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

Seed job SCM may now be configured with github-branch-source/GitHubAppCredentials #719

Merged

Conversation

andlaz
Copy link

@andlaz andlaz commented Feb 20, 2022

Changes

resolves #614

Submitter Checklist

Reviewer Notes

Release Notes

@andlaz
Copy link
Author

andlaz commented Feb 25, 2022

i could use some hints as to, where i need to make a change in e2e tests to make this pass please? It's indirectly an API change because of the extension of CR validation to allow the use of GitHubAppCredentials in the seedJobs array

thanks @Sig00rd for re-running, looks like the issue could have been in the env

@Sig00rd
Copy link

Sig00rd commented Mar 1, 2022

Hi @andlaz and thanks for contributing this, could you also add some test cases for validating this credential type in https://github.com/jenkinsci/kubernetes-operator/blob/master/pkg/configuration/user/seedjobs/validate_test.go ?

@stale
Copy link

stale bot commented Apr 16, 2022

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If this pull request is still relevant, just comment with any updates and we'll keep it open. Thank you for your contributions.

@stale stale bot added the stale label Apr 16, 2022
@andlaz
Copy link
Author

andlaz commented Apr 16, 2022

Could this please be merged?

@stale stale bot removed the stale label Apr 16, 2022
@mario21ic
Copy link

this is the PR that I just need.
could somebody merge this PR? 🙏

@Sig00rd
Copy link

Sig00rd commented May 31, 2022

Sorry for the delay!

@Sig00rd Sig00rd merged commit 46f64fe into jenkinsci:master May 31, 2022
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.

Allow seed job scm to be configured with github-branch-source/GitHubAppCredentials
6 participants