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(ci): Fix network parameter in continous-delivery.yml, and add network labels to GCP jobs #5710

Merged
merged 9 commits into from
Nov 25, 2022

Conversation

teor2345
Copy link
Contributor

@teor2345 teor2345 commented Nov 23, 2022

Motivation

We want to add the Zcash network as a GCP label. We need to lowercase the network to do that.

This also fixes a workflow input handling bug in continous-delivery.yml, which made it impossible to correctly implement a network label in PR #5693.

Solution

  • fix network parameter in continous-delivery.yml
  • standardise network usage in zcashd-manual-deploy
  • add Network as a label
  • use lowercase network labels

This PR also fixes some shellcheck errors using search and replace, including scripts in the unchanged parts of these workflows.

Review

This is a routine change.

Reviewer Checklist

  • Will the PR name make sense to users?
    • Does it need extra CHANGELOG info? (new features, breaking changes, large changes)
  • Are the PR labels correct?
  • Does the code do what the ticket and PR says?
  • How do you know it works? Does it have tests?

@teor2345 teor2345 added C-bug Category: This is a bug A-devops Area: Pipelines, CI/CD and Dockerfiles labels Nov 23, 2022
@teor2345 teor2345 self-assigned this Nov 23, 2022
@teor2345 teor2345 requested a review from a team as a code owner November 23, 2022 21:30
@teor2345 teor2345 requested review from arya2 and removed request for a team November 23, 2022 21:30
@github-actions github-actions bot added the C-trivial Category: A trivial change that is not worth mentioning in the CHANGELOG label Nov 23, 2022
@teor2345 teor2345 changed the title Suggestion: Fix network parameter in continous-delivery.yml fix(ci): Fix network parameter in continous-delivery.yml, and add network labels to GCP jobs Nov 23, 2022
@teor2345 teor2345 added C-enhancement Category: This is an improvement P-Medium ⚡ A-diagnostics Area: Diagnosing issues or monitoring performance labels Nov 23, 2022
@teor2345 teor2345 changed the base branch from add-instance-labels to main November 23, 2022 21:40
@teor2345 teor2345 force-pushed the continous-delivery-net-fix branch from 0b6bd6b to bc48288 Compare November 23, 2022 21:49
@teor2345 teor2345 changed the base branch from main to add-instance-labels November 23, 2022 21:49
@github-actions github-actions bot added the C-feature Category: New features label Nov 23, 2022
@teor2345 teor2345 force-pushed the continous-delivery-net-fix branch from bc48288 to 5f8eb12 Compare November 23, 2022 22:07
@teor2345
Copy link
Contributor Author

@gustavovalverde I'm not sure how to test continous-delivery.yml, are there instructions?

zcashd-manual-deploy.yml isn't so important to test, because it is not used right now.

@teor2345 teor2345 force-pushed the continous-delivery-net-fix branch from dfc3e02 to c2f3c59 Compare November 23, 2022 22:16
@teor2345 teor2345 removed the C-feature Category: New features label Nov 23, 2022
Copy link
Contributor

@arya2 arya2 left a comment

Choose a reason for hiding this comment

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

I'm not sure how to test continous-delivery.yml

The workflow dispatch is available here if that helps.

.github/workflows/continous-delivery.yml Show resolved Hide resolved
@teor2345
Copy link
Contributor Author

I'm not sure how to test continous-delivery.yml

The workflow dispatch is available here if that helps.

Thanks!

I started a manual run here:
https://github.com/ZcashFoundation/zebra/actions/runs/3536561823

But we also need to test the release trigger, and check the labels, and maybe the logs for the instances?

I'm not sure how to do any of that.

@github-actions github-actions bot added the C-feature Category: New features label Nov 24, 2022
Base automatically changed from add-instance-labels to main November 24, 2022 03:34
@teor2345 teor2345 force-pushed the continous-delivery-net-fix branch from fe7af64 to 3b46d1e Compare November 24, 2022 20:47
@teor2345 teor2345 force-pushed the continous-delivery-net-fix branch from 3b46d1e to cf5c764 Compare November 24, 2022 20:48
@teor2345
Copy link
Contributor Author

Failed due to #5384 again.

@teor2345 teor2345 requested a review from arya2 November 25, 2022 02:17
@teor2345
Copy link
Contributor Author

@gustavovalverde I get this warning when running the workflow, but I don't think I made any changes to that part of the workflow:

Deploy single instance
The "create_credentials_file" option is true, but the current GitHub workspace is empty. Did you forget to use "actions/checkout" before this step? If you do not intend to share authentication with future steps in this job, set "create_credentials_file" to false.

https://github.com/ZcashFoundation/zebra/actions/runs/3536561823

@arya2
Copy link
Contributor

arya2 commented Nov 25, 2022

@gustavovalverde I get this warning when running the workflow, but I don't think I made any changes to that part of the workflow:

Thise seems to be normal, it says that for all runs going back at least ~6 months:

https://github.com/ZcashFoundation/zebra/actions/runs/2411825703

Copy link
Contributor

@arya2 arya2 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

The "current GitHub workspace is empty" warning isn't new so I'm not worried about it.

Versioning and deploy-nodes were skipped in the manual run, but:

  • versioning wasn't changed (except the comment)
  • changes to deploy-nodes and zcashd-manual-deploy were tested in CI jobs

mergify bot added a commit that referenced this pull request Nov 25, 2022
@mergify mergify bot merged commit a763eec into main Nov 25, 2022
@mergify mergify bot deleted the continous-delivery-net-fix branch November 25, 2022 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles A-diagnostics Area: Diagnosing issues or monitoring performance C-bug Category: This is a bug C-enhancement Category: This is an improvement C-feature Category: New features C-trivial Category: A trivial change that is not worth mentioning in the CHANGELOG
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants