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

Backport of release pipeline: fix ref arguments in invoking workflow into release/1.3.x #17685

Conversation

hc-github-team-nomad-core
Copy link
Contributor

Backport

This PR is auto-generated from #17684 to be assessed for backporting due to the inclusion of the label backport/1.3.x.

The below text is copied from the body of the original PR.


Although #17669 fixed the permissions of the release pipeline to push new commits, there was still an error when invoking the build workflow.

The format of the reference was changed in #17103 such that we're sending the git ref (a SHA) and not the "--ref" argument required by the GH actions workflow API, which in this case is apparently specially defined as "The branch or tag name which contains the version of the workflow file you'd like to run" and not what git calls a "ref".

This changeset:

  • Removes the third-party action entirely so that we're using GitHub's own tooling. This removes one more thing from the supply chain to pin and ensures a 1:1 mapping of args to what's documented by GitHub.
  • Removes the --ref argument entirely, which causes it to default to the current branch that the release workflow is running on (which is always what we want).

Tested out in https://github.com/hashicorp/nomad-enterprise/actions/runs/5349266786 (after many many iterations 😀 )

@hc-github-team-nomad-core hc-github-team-nomad-core force-pushed the backport/release-workflow-permissions/sharply-guided-man branch from b365592 to c2e762b Compare June 22, 2023 19:33
@hc-github-team-nomad-core hc-github-team-nomad-core merged commit 1481c79 into release/1.3.x Jun 22, 2023
@hc-github-team-nomad-core hc-github-team-nomad-core deleted the backport/release-workflow-permissions/sharply-guided-man branch June 22, 2023 19:33
@vercel vercel bot temporarily deployed to Preview – nomad June 22, 2023 19:37 Inactive
@vercel vercel bot temporarily deployed to Preview – nomad-storybook-and-ui June 22, 2023 19:41 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants