@jayunit100 please can you suggest a reviewer(s)?

Hi satnam. Possbly @brendanburns , and yourself @satnam6502 might be able to review this . It should be very easy to test (just run k8petstore.sh and modify the variables at the top, which are self explanatory, as you see fit)

(fyi, the official issue tracker is #5260 )

Thanks for the suggestions. I'm on call today so I've asked Brendan to help out.

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

CLAs look good, thanks!

Cool, some minor comments on the docs, but generally LGTM.

Thanks for doing this!
--brendan

okay updated @brendanburns !

Sorry, one more thing, can you run gofmt on:

examples/k8petstore/web-server/src/main/PetStoreBook.go

Thanks!
--brendan

okay, done. thats a neat tool. didn't know about it

Ugh, travis is trying to build your example code:

https://travis-ci.org/GoogleCloudPlatform/kubernetes/jobs/54292082

We should either include the right libraries in our Gopath, or block travis from building this somehow.

hiya, okay, i;ll look into adding the libraries. Im not sure where to add them so if its non obvious just ping in this thread :)... will update when im done.
UPDATE
okay, I think I see how it works, updating.... GoDeps.json,

okay, pushed an to GoDeps ........... we will see if it works. thanks for noticing that !~

Well, looks like the build still fails.
Locally I get a funny error building kube ... so I can't really reproduce what travice is doing.
I guess one option is to rename src to _src (like you did in a the other guestbook-go examples/ app) .... but that is only a temporary fix.

Well, I got the build working locally:

[jay@rhbd kubernetes]$  go build -v ./... 2>&1 | grep examples
_/home/jay/gopath/src/github.com/GoogleCloudPlatform/kubernetes/examples
_/home/jay/gopath/src/github.com/GoogleCloudPlatform/kubernetes/examples/liveness/image
_/home/jay/gopath/src/github.com/GoogleCloudPlatform/kubernetes/examples/k8petstore/web-server/src/main
[jay@rhbd kubernetes]$ pwd
/home/jay/gopath/src/github.com/GoogleCloudPlatform/kubernetes

I will see if i can follow the directions in the developer docs to build it cleanly. maybe its just working b.c of the local deps which are on my machine, (i.e. possibly i didn't add them correctly into the Godeps json file)

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.

CLAs look good, thanks!

@brendandburns okay, i did the temporary disablement of the src dir by moving to _src.

As a follow on , lets work on #5465, as part of which we can get rid of needing to do stuff like that, and bring examples/ into the build

(bump) tl;dr this is now passing and ready to merge - i shaded the src package to _src, and updated the dependencies. maybe in a future build, it will pass once travis integrates examples/ src properly.

Squash before merge (once final sign offs are given) please.

Squashed into 2 commits (original patch in the first ; cleanup in the second).

LGTM, will merge on green.

lgtm, but needs rebase and squash.

okay , rebased and squashed into 3 commits (one containing the fixes which jeff pushed, one containing the cleanups which i made after the original patch, and one with the original code dump). thanks for your help Jeff and Brendan !

merged!