Skip to content

Migrate webhook CI to GHA #394

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

Closed
wants to merge 1 commit into from
Closed

Conversation

ericpromislow
Copy link
Contributor

@ericpromislow ericpromislow commented Jun 11, 2024

Related to rancher/rancher#45231

This is a continuation of PR 383 -- we have to make a PR off the repo to access secrets

See #383 for previous comments.

Some changes from the drone-based scripts:

  • Use different names for shell vars TAG and IMAGE_TAG

  • HELM_VERSION and HELM_CHART_VERSION are two different things (version of helm, and the version of the chart to generate)

  • TAG and IMAGE_TAG also denote different things

  • Can't use make from the GHA scripts: dapper doesn't pass on env vars

Some changes from the drone-based scripts:

- Use different names for shell vars TAG and IMAGE_TAG
- `HELM_VERSION` and `HELM_CHART_VERSION` are two different things (version of helm,
  and the version of the chart to generate)
- `TAG` and `IMAGE_TAG` also denote different things

- Can't use make from the GHA scripts: `dapper` doesn't pass on env vars
@ericpromislow ericpromislow requested a review from a team as a code owner June 11, 2024 17:35
@ericpromislow
Copy link
Contributor Author

We're going to stick with the PR from the fork. Ignore this PR

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.

1 participant