-
Notifications
You must be signed in to change notification settings - Fork 107
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(ci): remove warnings caused by missing actions/checkout
step
#5874
Conversation
Running the workflow here: https://github.com/ZcashFoundation/zebra/actions/runs/3734154152 |
https://github.com/ZcashFoundation/zebra/actions/runs/3734154152/jobs/6335869442#step:6:74 |
@gustavovalverde I don't see why this PR would cause this issue, but it's not happening on the main branch or anywhere else. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WARNING: Default device-name for disk name [auto-delete=yes] will be [auto-delete=yes] because it is being mounted to a container with [
--container-mount-disk
]
ERROR: (gcloud.compute.instances.create-with-container) Could not fetch resource:
- Invalid value for field 'resource.disks[1].deviceName': 'auto-delete=yes'. Must be a match of regex '(^$|\w[\w.-]{0,254})'
https://github.com/ZcashFoundation/zebra/actions/runs/3734154152/jobs/6335869442#step:6:74
@gustavovalverde I don't see why this PR would cause this issue, but it's not happening on the main branch or anywhere else.
This seems to be an existing bug in the workflow, which is only triggered by a workflow_dispatch
.
The name=
field is missing for manual dispatch:
--create-disk=name=auto-delete=yes,size=300GB,type=pd-ssd \ |
This is what it should be like:
--create-disk=name=zebrad-cache-${{ env.GITHUB_SHA_SHORT }},auto-delete=yes,size=300GB,type=pd-ssd \ |
This seems like a blocker for this PR, because we can't test the PR without fixing it.
Is this warning a problem?
https://github.com/ZcashFoundation/zebra/actions/runs/3877453287/jobs/6612495110#step:6:75 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good otherwise!
That's part of the issues with the GCP documentation that I reported to them |
Motivation
We get this warning when running the
continous-delivery.yml
workflow:Fixes #5729
Solution
Add missing
actions/checkout
steps to allow the credentials file to be written correctlyReview
Anyone can review this
Reviewer Checklist