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

Use workflow configuration variables for quay organization names #23145

Merged
merged 1 commit into from Jan 18, 2023

Conversation

michi-covalent
Copy link
Contributor

We've defined QUAY_ORGANIZATION and QUAY_ORGANIZATION_DEV workflow configuration variables, so we don't need to use the composite action to set these as environment variables anymore.

Ref: https://github.blog/changelog/2023-01-10-github-actions-support-for-configuration-variables-in-workflows/

Suggested-by: Nicolas Busseneau nicolas@isovalent.com
Signed-off-by: Michi Mutsuzaki michi@isovalent.com

Use workflow configuration variables for quay organization names

We've defined QUAY_ORGANIZATION and QUAY_ORGANIZATION_DEV workflow
configuration variables, so we don't need to use the composite action
to set these as environment variables anymore.

Ref: https://github.blog/changelog/2023-01-10-github-actions-support-for-configuration-variables-in-workflows/

Suggested-by: Nicolas Busseneau <nicolas@isovalent.com>
Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
@michi-covalent michi-covalent added the release-note/ci This PR makes changes to the CI. label Jan 18, 2023
@michi-covalent michi-covalent requested review from a team as code owners January 18, 2023 01:19
@github-actions github-actions bot added the kind/community-contribution This was a contribution made by a community member. label Jan 18, 2023
@michi-covalent michi-covalent added the needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch label Jan 18, 2023
Copy link
Member

@nbusseneau nbusseneau left a comment

Choose a reason for hiding this comment

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

@aanm aanm merged commit f28b8e2 into master Jan 18, 2023
@aanm aanm deleted the pr/michi/variables branch January 18, 2023 13:32
@nbusseneau
Copy link
Member

@bimmlerd Do we need to backport this for 1.12?

@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.12.6 Jan 18, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.11.13 Jan 18, 2023
@bimmlerd
Copy link
Member

yeah I think we want this for all stable branches, actually.

@aanm aanm removed needs-backport/1.11 needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels Jan 23, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed this from Needs backport from master in 1.11.13 Jan 23, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed this from Needs backport from master in 1.12.6 Jan 23, 2023
@aanm
Copy link
Member

aanm commented Jan 23, 2023

removed needs-backport labels since this PR was reverted in #23169

@aanm aanm mentioned this pull request Jan 23, 2023
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/community-contribution This was a contribution made by a community member. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants