Skip to content
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

Fix broken GCP console link #598

Conversation

campionfellin
Copy link
Contributor

This fixes #596

With the link to the Tasks API.

Though note that there are 2 other broken links on that page, and it's hard to tell where they're supposed to go:

Line 3:
[Google Cloud Tasks](https://cloud.google.com/cloud-tasks/) maybe it should link to:
https://cloud.google.com/sdk/gcloud/reference/alpha/tasks/ instead?

Line 86:
https://cloud.google.com/cloud-tasks should be changed similarly.

@campionfellin
Copy link
Contributor Author

Hey, I'm curious about the circleci build and why it fails sometimes. It looks like the 2nd step in dependencies is:

gcloud auth activate-service-account --key-file /home/ubuntu/nodejs-docs-samples/key.json || true

and in failing builds, you get this:

ERROR: (gcloud.auth.activate-service-account) Could not read json file /home/ubuntu/nodejs-docs-samples/key.json: No JSON object could be decoded

but since the || true, it passes this portion of the build. I think this causes problems later with the first part of the functions test when it tries:

functions-emulator config set projectId $GCLOUD_PROJECT but fails to find the env variable $GCLOUD_PROJECT and then times out.

Maybe we should remove the || true from when we activate the service account so it fails there?

Just a thought, though this could be a separate issue or PR

Copy link
Contributor

@fhinkel fhinkel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@fhinkel
Copy link
Contributor

fhinkel commented Jun 5, 2018

@campionfellin Thanks so much. Sorry that this was sitting here forever.
Any chance you did a quick grep if the wrong URL is used anywhere else?

About the CI: It's not in good shape 😢 All the recent runs didn't even get to that error.

@campionfellin
Copy link
Contributor Author

Hey @fhinkel are you on googlecloud-community.slack.com channel? We're having similar problems with our CI over at clasp and I'm thinking of a solution, I'd love to run it by you.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 6, 2018
@campionfellin
Copy link
Contributor Author

Ah looks like the build passed... @JustinBeckwith did you change the pipeline recently? Looks like it's still getting this error:

gcloud auth activate-service-account --key-file /home/ubuntu/nodejs-docs-samples/key.json || true
ERROR: (gcloud.auth.activate-service-account) Could not read json file /home/ubuntu/nodejs-docs-samples/key.json: No JSON object could be decoded

Yet somehow passes all the tests later.

@JustinBeckwith
Copy link
Contributor

I haven't changed a thing 🤷‍♂️

@JustinBeckwith JustinBeckwith merged commit 613b91a into GoogleCloudPlatform:master Jun 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broken link in App Engine/Cloud Tasks ReadMe
4 participants