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

GHA migration: secrets check #365

Merged
merged 2 commits into from
Apr 24, 2024

Conversation

enrichman
Copy link
Contributor

This PR adds the steps that will use the migrated secrets, and a release.yml workflow that will be used for the releases.

The FOSSA secret with the API key is used during the CI workflow.
The Docker Hub username and password will be used during the release. At the moment we are just going to login to see that they are working fine.

This PR is needed because the branched of the CLI repo are protected, and we need PRs for them.
This is a first step "soft" PR used for the migration with no side-effects.

@enrichman enrichman force-pushed the 44987-v2.9-gha-migration-test branch from f77c30f to 7d7f4e8 Compare April 24, 2024 13:40
@enrichman enrichman mentioned this pull request Apr 24, 2024
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
@enrichman
Copy link
Contributor Author

Thanks for the review @andreas-kupries . I checked with @Anarkis and the error is expected because of the run on a different repo. Once merged we can check that the secrets are ok.
Do you think I should open the PR on a different branch? It's safe to merge on purpose.

@andreas-kupries
Copy link

Thanks for the review @andreas-kupries . I checked with @Anarkis and the error is expected because of the run on a different repo. Once merged we can check that the secrets are ok. Do you think I should open the PR on a different branch? It's safe to merge on purpose.

For my migration I actually did a test PR on a non-forked branch to check that the creds worked, and, of course the actions after also. For the FOSSA this might not be as important, as it is a code check, and not pushing anything.

Copy link

@bigkevmcd bigkevmcd 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!

@enrichman enrichman merged commit a85cc10 into rancher:v2.9 Apr 24, 2024
2 of 3 checks passed
@enrichman
Copy link
Contributor Author

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.

Migrate rancher/cli from Drone to GH Actions
3 participants