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

Add new integration tests for Azure OIDC for git repositories #793

Merged
merged 1 commit into from
Sep 13, 2024

Conversation

dipti-pai
Copy link
Member

@dipti-pai dipti-pai commented Jul 22, 2024

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 repository
  • azure_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 -

$ make test-azure GO_TEST_ARGS="-retain -existing"
make test PROVIDER_ARG="-provider azure"
make[1]: Entering directory '/home/dipti/code/pkg/oci/tests/integration'
docker image inspect fluxcd/testapp:test >/dev/null
TEST_IMG=fluxcd/testapp:test go test -timeout 30m -v ./ -retain -existing -provider azure --tags=integration
2024/08/22 12:19:26 Terraform binary:  /snap/bin/terraform
2024/08/22 12:19:26 Init Terraform
2024/08/22 12:19:28 Applying Terraform
2024/08/22 12:22:07 pushing flux test image fluxtesthipomule.azurecr.io/app:test
2024/08/22 12:22:08 Git is enabled, get test config
URL without username: https://dev.azure.com/diptipai-flux/fluxProjhipomule/_git/fluxRepohipomule
2024/08/22 12:22:08 Giving permissions to workload identity to access repository
2024/08/22 12:22:10 Added managed identity to organization:
2024/08/22 12:22:10 Workload identity is enabled, initializing service account with annotations
=== RUN   TestGitCloneUsingProvider
=== RUN   TestGitCloneUsingProvider/Git_oidc_credential_test
--- PASS: TestGitCloneUsingProvider (6.70s)
    --- PASS: TestGitCloneUsingProvider/Git_oidc_credential_test (5.94s)
=== RUN   TestOciImageRepositoryListTags
    oci_test.go:30: Skipping test as oci is not enabled in env
--- SKIP: TestOciImageRepositoryListTags (0.00s)
=== RUN   TestOciRepositoryRootLoginListTags
    oci_test.go:46: Skipping test as oci is not enabled in env
--- SKIP: TestOciRepositoryRootLoginListTags (0.00s)
=== RUN   TestOciOIDCLoginListTags
    oci_test.go:64: Skipping test as oci is not enabled in env
--- SKIP: TestOciOIDCLoginListTags (0.00s)
PASS
ok      github.com/fluxcd/pkg/oci/tests/integration     171.350s
make[1]: Leaving directory '/home/dipti/code/pkg/oci/tests/integration'
source .env (After enabling oci in env)
$ make test-azure GO_TEST_ARGS="-retain -existing"
make test PROVIDER_ARG="-provider azure"
make[1]: Entering directory '/home/dipti/code/pkg/oci/tests/integration'
docker image inspect fluxcd/testapp:test >/dev/null
TEST_IMG=fluxcd/testapp:test go test -timeout 30m -v ./ -retain -existing -provider azure --tags=integration
2024/08/22 12:23:28 Terraform binary:  /snap/bin/terraform
2024/08/22 12:23:28 Init Terraform
2024/08/22 12:23:31 Applying Terraform
2024/08/22 12:26:11 pushing flux test image fluxtesthipomule.azurecr.io/app:test
2024/08/22 12:26:12 OCI is enabled, push oci test images
2024/08/22 12:26:15 pushing test image fluxtesthipomule.azurecr.io/dytqs:v0.1.0
2024/08/22 12:26:17 pushing test image fluxtesthipomule.azurecr.io/dytqs:v0.1.2
2024/08/22 12:26:18 pushing test image fluxtesthipomule.azurecr.io/dytqs:v0.1.3
2024/08/22 12:26:19 pushing test image fluxtesthipomule.azurecr.io/dytqs:v0.1.4
2024/08/22 12:26:20 Git is enabled, get test config
URL without username: https://dev.azure.com/diptipai-flux/fluxProjhipomule/_git/fluxRepohipomule
2024/08/22 12:26:20 Giving permissions to workload identity to access repository
2024/08/22 12:26:21 Added managed identity to organization:
2024/08/22 12:26:21 Workload identity is enabled, initializing service account with annotations
=== RUN   TestGitCloneUsingProvider
=== RUN   TestGitCloneUsingProvider/Git_oidc_credential_test
--- PASS: TestGitCloneUsingProvider (6.38s)
    --- PASS: TestGitCloneUsingProvider/Git_oidc_credential_test (5.74s)
=== RUN   TestOciImageRepositoryListTags
--- PASS: TestOciImageRepositoryListTags (0.00s)
=== RUN   TestOciRepositoryRootLoginListTags
--- PASS: TestOciRepositoryRootLoginListTags (0.00s)
=== RUN   TestOciOIDCLoginListTags
--- PASS: TestOciOIDCLoginListTags (0.00s)
PASS
ok      github.com/fluxcd/pkg/oci/tests/integration     179.322s
make[1]: Leaving directory '/home/dipti/code/pkg/oci/tests/integration'
$ make test-azure-git GO_TEST_ARGS="-retain -existing"
make test PROVIDER_ARG="-provider azure" GO_TEST_PREFIX="-run TestGit"
make[1]: Entering directory '/home/dipti/code/pkg/oci/tests/integration'
docker image inspect fluxcd/testapp:test >/dev/null
TEST_IMG=fluxcd/testapp:test go test -timeout 30m -v ./ -run TestGit -retain -existing -provider azure --tags=integration
2024/08/22 12:43:05 Terraform binary:  /snap/bin/terraform
2024/08/22 12:43:05 Init Terraform
2024/08/22 12:43:07 Applying Terraform
2024/08/22 12:45:50 pushing flux test image fluxtesthipomule.azurecr.io/app:test
2024/08/22 12:45:51 OCI is enabled, push oci test images
2024/08/22 12:45:55 pushing test image fluxtesthipomule.azurecr.io/6vxpb:v0.1.0
2024/08/22 12:45:56 pushing test image fluxtesthipomule.azurecr.io/6vxpb:v0.1.2
2024/08/22 12:45:57 pushing test image fluxtesthipomule.azurecr.io/6vxpb:v0.1.3
2024/08/22 12:45:58 pushing test image fluxtesthipomule.azurecr.io/6vxpb:v0.1.4
2024/08/22 12:45:59 Git is enabled, get test config
URL without username: https://dev.azure.com/diptipai-flux/fluxProjhipomule/_git/fluxRepohipomule
2024/08/22 12:45:59 Giving permissions to workload identity to access repository
2024/08/22 12:46:01 Added managed identity to organization:
2024/08/22 12:46:01 Workload identity is enabled, initializing service account with annotations
=== RUN   TestGitCloneUsingProvider
=== RUN   TestGitCloneUsingProvider/Git_oidc_credential_test
--- PASS: TestGitCloneUsingProvider (6.97s)
    --- PASS: TestGitCloneUsingProvider/Git_oidc_credential_test (5.95s)
PASS
ok      github.com/fluxcd/pkg/oci/tests/integration     183.375s
make[1]: Leaving directory '/home/dipti/code/pkg/oci/tests/integration'
$ make test-azure-oci GO_TEST_ARGS="-retain -existing"
make test PROVIDER_ARG="-provider azure" GO_TEST_PREFIX="-run TestOci"
make[1]: Entering directory '/home/dipti/code/pkg/oci/tests/integration'
docker image inspect fluxcd/testapp:test >/dev/null
TEST_IMG=fluxcd/testapp:test go test -timeout 30m -v ./ -run TestOci -retain -existing -provider azure --tags=integration
2024/08/22 12:46:32 Terraform binary:  /snap/bin/terraform
2024/08/22 12:46:32 Init Terraform
2024/08/22 12:46:34 Applying Terraform
2024/08/22 12:49:23 pushing flux test image fluxtesthipomule.azurecr.io/app:test
2024/08/22 12:49:24 OCI is enabled, push oci test images
2024/08/22 12:49:28 pushing test image fluxtesthipomule.azurecr.io/1uavx:v0.1.0
2024/08/22 12:49:29 pushing test image fluxtesthipomule.azurecr.io/1uavx:v0.1.2
2024/08/22 12:49:30 pushing test image fluxtesthipomule.azurecr.io/1uavx:v0.1.3
2024/08/22 12:49:31 pushing test image fluxtesthipomule.azurecr.io/1uavx:v0.1.4
2024/08/22 12:49:32 Git is enabled, get test config
URL without username: https://dev.azure.com/diptipai-flux/fluxProjhipomule/_git/fluxRepohipomule
2024/08/22 12:49:32 Giving permissions to workload identity to access repository
2024/08/22 12:49:33 Added managed identity to organization:
2024/08/22 12:49:33 Workload identity is enabled, initializing service account with annotations
=== RUN   TestOciImageRepositoryListTags
--- PASS: TestOciImageRepositoryListTags (0.00s)
=== RUN   TestOciRepositoryRootLoginListTags
--- PASS: TestOciRepositoryRootLoginListTags (0.00s)
=== RUN   TestOciOIDCLoginListTags
--- PASS: TestOciOIDCLoginListTags (0.00s)
PASS
ok      github.com/fluxcd/pkg/oci/tests/integration     180.909s
make[1]: Leaving directory '/home/dipti/code/pkg/oci/tests/integration'

@dipti-pai dipti-pai requested a review from stefanprodan as a code owner July 22, 2024 22:34
oci/tests/integration/.env.sample Outdated Show resolved Hide resolved
oci/tests/integration/suite_test.go Outdated Show resolved Hide resolved
oci/tests/integration/suite_test.go Outdated Show resolved Hide resolved
oci/tests/integration/README.md Show resolved Hide resolved
oci/tests/integration/suite_test.go Outdated Show resolved Hide resolved
oci/tests/integration/suite_test.go Outdated Show resolved Hide resolved
oci/tests/integration/repo_list_test.go Outdated Show resolved Hide resolved
oci/tests/integration/repo_list_test.go Outdated Show resolved Hide resolved
oci/tests/integration/.env.sample Outdated Show resolved Hide resolved
oci/tests/integration/.env.sample Outdated Show resolved Hide resolved
oci/tests/integration/suite_test.go Outdated Show resolved Hide resolved
oci/tests/integration/suite_test.go Outdated Show resolved Hide resolved
oci/tests/integration/suite_test.go Outdated Show resolved Hide resolved
oci/tests/integration/suite_test.go Outdated Show resolved Hide resolved
oci/tests/integration/terraform/azure/main.tf Outdated Show resolved Hide resolved
oci/tests/integration/README.md Outdated Show resolved Hide resolved
oci/tests/integration/README.md Outdated Show resolved Hide resolved
oci/tests/integration/Makefile Outdated Show resolved Hide resolved
@dipti-pai dipti-pai force-pushed the git-oidc-integration-tests branch from 8945f44 to b8d25d0 Compare August 27, 2024 07:08
oci/tests/integration/azure_test.go Outdated Show resolved Hide resolved
oci/tests/integration/azure_test.go Outdated Show resolved Hide resolved
oci/tests/integration/azure_test.go Outdated Show resolved Hide resolved
oci/tests/integration/testapp/main.go Outdated Show resolved Hide resolved
oci/tests/integration/util_test.go Outdated Show resolved Hide resolved
oci/tests/integration/job_test.go Show resolved Hide resolved
oci/tests/integration/suite_test.go Outdated Show resolved Hide resolved
Copy link
Contributor

@darkowlzz darkowlzz left a 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.

oci/tests/integration/azure_test.go Outdated Show resolved Hide resolved
oci/tests/integration/azure_test.go Outdated Show resolved Hide resolved
oci/tests/integration/suite_test.go Outdated Show resolved Hide resolved
oci/tests/integration/README.md Outdated Show resolved Hide resolved
oci/tests/integration/azure_test.go Outdated Show resolved Hide resolved
oci/tests/integration/azure_test.go Outdated Show resolved Hide resolved
oci/tests/integration/terraform/azure/outputs.tf Outdated Show resolved Hide resolved
oci/tests/integration/util_test.go Outdated Show resolved Hide resolved
@dipti-pai dipti-pai force-pushed the git-oidc-integration-tests branch from 0d79654 to 6648217 Compare September 9, 2024 16:31
@dipti-pai dipti-pai requested a review from a team as a code owner September 9, 2024 16:31
@darkowlzz darkowlzz force-pushed the git-oidc-integration-tests branch 2 times, most recently from 6229ec9 to f11bd0c Compare September 13, 2024 14:10
Copy link
Contributor

@darkowlzz darkowlzz left a 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.

@dipti-pai dipti-pai force-pushed the git-oidc-integration-tests branch from f11bd0c to 5e77f04 Compare September 13, 2024 19:33
@dipti-pai dipti-pai changed the base branch from azure-git-oidc to main September 13, 2024 19:36
@dipti-pai dipti-pai force-pushed the git-oidc-integration-tests branch 4 times, most recently from ddde871 to d3b4de7 Compare September 13, 2024 19:51
@stefanprodan
Copy link
Member

This can now be rebased on top of main with the following packages updates:

  • auth/v0.0.1
  • cache/v0.0.4
  • git/v0.21.0
  • git/gogit/v0.21.0
  • oci/v0.41.1

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]>
@dipti-pai dipti-pai force-pushed the git-oidc-integration-tests branch from d3b4de7 to 42a5c0e Compare September 13, 2024 20:08
@dipti-pai
Copy link
Member Author

dipti-pai commented Sep 13, 2024

This can now be rebased on top of main with the following packages updates:

  • auth/v0.0.1
  • cache/v0.0.4
  • git/v0.21.0
  • git/gogit/v0.21.0
  • oci/v0.41.1

@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.

@stefanprodan stefanprodan added area/testing Testing related issues and pull requests area/git Git and SSH related issues and pull requests labels Sep 13, 2024
Copy link
Member

@stefanprodan stefanprodan left a 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

@stefanprodan stefanprodan merged commit 30c101f into fluxcd:main Sep 13, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/git Git and SSH related issues and pull requests area/testing Testing related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants