-
Notifications
You must be signed in to change notification settings - Fork 87
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
Add new integration tests for Azure OIDC for git repositories #793
Add new integration tests for Azure OIDC for git repositories #793
Conversation
8945f44
to
b8d25d0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I finally ran this in my account. Good to see it working 🙂
Left more comments based on my observations after running it.
472a7a5
to
f86cd49
Compare
0d79654
to
6648217
Compare
6229ec9
to
f11bd0c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
The git and auth package dependencies need to be updated when those packages are available before merging this.
Thanks for all the work in adding git tests.
f11bd0c
to
5e77f04
Compare
ddde871
to
d3b4de7
Compare
This can now be rebased on top of main with the following packages updates:
|
Signed-off-by: Dipti Pai <[email protected]> Signed-off-by: Sunny <[email protected]> Co-authored-by: Dipti Pai <[email protected]> Co-authored-by: Sunny <[email protected]>
d3b4de7
to
42a5c0e
Compare
@stefanprodan I rebased on top of upstream main and see the auth pkg is now updated to v0.0.1. However, git package is at v0.20.0 and git/gogit is at v0.19.0 which is lower than the versions you shared above. Is this ok? The new git integration tests are green. EDIT: In source controller, the updated packages are used correctly. In pkg/oci/tests/integration, the replace directives are for local paths so I understand this should be ok. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks @dipti-pai
Changes include enhancements to tests/integration to add git tests
testapp
is modified to accept category git and test cloning azure devops git repository using workload identity.Azure terraform files are separated into oci and git. Azure terraform files for git require an organization and PAT and create ADO project and repository
suite_test.go
is modified to add new git category and setup provider specific test configuration and permissions on git repositoryazure_test.go
has new functions to return test configuration based on the git terraform file output and API to give the workload identity access to the ADO repository.repo_list_test.go
is deleted with new files for oci_test.go with existing oci tests and git_test.go with tests to validate Azure OIDC for ADO repository.util_test.go
is a new utility with functions to get the gogit client to clone, commit and push changes to a git repository.In a subsequent PR, we can consider moving oci//tests/integration/ out of oci package to a root directory tests/integration/
Depends on PR in test-infra #44 and PR in pkg #789
Sample test run outputs -