-
Notifications
You must be signed in to change notification settings - Fork 38.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update guestbook frontend php script as suggested in issue #18640. #21327
Conversation
Labelling this PR as size/XS |
LGTM |
cc @carlosrdrz |
It would be useful for someone with a non-privileged account to confirm that they can pull the new image. |
GCE e2e test build/test passed for commit 2d8f857. |
Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist") If this message is too spammy, please complain to ixdy. |
@bgrant0607 Brian, I got Takashi, who is not on the google-samples project, to do a test pull of gcr.io/google-samples/gb-frontend:v4, and he was successful. |
Thanks, @amygdala. LGTM |
Automatic merge from submit-queue |
Auto commit by PR queue bot
Update guestbook frontend php script as suggested in issue #18640.
(also created new GCR image version with the changes).