microsoft / terraform-provider-azuredevops

Terraform Azure DevOps provider
https://www.terraform.io/docs/providers/azuredevops/
MIT License
372 stars 260 forks source link

`azuredevops_pipeline_authorization` - Support cross project authorization #989

Closed xuzhang3 closed 3 months ago

xuzhang3 commented 3 months ago

All Submissions:

What about the current behavior has changed?

Inspired by #973 @josh-barker I updated the azuredevops_pipeline_authorization to allow cross project authorization, this change is a breaking change to users who do the authorization for repository. Any suggestions?

Does this introduce a change to go.mod, go.sum or vendor/?

Does this introduce a breaking change?

Any relevant logs, error output, etc?

(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)

Other information

xuzhang3 commented 3 months ago
==> Checking that code complies with gofmt requirements...
==> Sourcing .env file if available
if [ -f .env ]; then set -o allexport; . ./.env; set +o allexport; fi; \
        TF_ACC=1 go test -tags "all" $(go list ./azuredevops/internal/acceptancetests |grep -v 'vendor') -v -run=TestAccPipelineAuthorization_ -timeout 120m
=== RUN   TestAccPipelineAuthorization_allPipeline_queue
=== PAUSE TestAccPipelineAuthorization_allPipeline_queue
=== RUN   TestAccPipelineAuthorization_pipeline_queue
=== PAUSE TestAccPipelineAuthorization_pipeline_queue
=== RUN   TestAccPipelineAuthorization_multiPipeline_queue
=== PAUSE TestAccPipelineAuthorization_multiPipeline_queue
=== RUN   TestAccPipelineAuthorization_allPipelineWithPipeline_queue
=== PAUSE TestAccPipelineAuthorization_allPipelineWithPipeline_queue
=== RUN   TestAccPipelineAuthorization_allPipeline_environment
=== PAUSE TestAccPipelineAuthorization_allPipeline_environment
=== RUN   TestAccPipelineAuthorization_pipeline_environment
=== PAUSE TestAccPipelineAuthorization_pipeline_environment
=== RUN   TestAccPipelineAuthorization_allPipeline_variableGroup
=== PAUSE TestAccPipelineAuthorization_allPipeline_variableGroup
=== RUN   TestAccPipelineAuthorization_pipeline_variableGroup
=== PAUSE TestAccPipelineAuthorization_pipeline_variableGroup
=== RUN   TestAccPipelineAuthorization_allPipeline_endpoint
=== PAUSE TestAccPipelineAuthorization_allPipeline_endpoint
=== RUN   TestAccPipelineAuthorization_pipeline_endpoint
=== PAUSE TestAccPipelineAuthorization_pipeline_endpoint
=== RUN   TestAccPipelineAuthorization_allPipeline_repository
=== PAUSE TestAccPipelineAuthorization_allPipeline_repository
=== RUN   TestAccPipelineAuthorization_pipeline_repository
=== PAUSE TestAccPipelineAuthorization_pipeline_repository
=== RUN   TestAccPipelineAuthorization_allPipeline_repository_crossProject
=== PAUSE TestAccPipelineAuthorization_allPipeline_repository_crossProject
=== RUN   TestAccPipelineAuthorization_pipeline_repository_crossProject
=== PAUSE TestAccPipelineAuthorization_pipeline_repository_crossProject
=== CONT  TestAccPipelineAuthorization_allPipeline_queue
=== CONT  TestAccPipelineAuthorization_pipeline_variableGroup
=== CONT  TestAccPipelineAuthorization_pipeline_repository
=== CONT  TestAccPipelineAuthorization_allPipeline_environment
=== CONT  TestAccPipelineAuthorization_pipeline_endpoint
=== CONT  TestAccPipelineAuthorization_allPipelineWithPipeline_queue
=== CONT  TestAccPipelineAuthorization_allPipeline_endpoint
=== CONT  TestAccPipelineAuthorization_pipeline_repository_crossProject
--- PASS: TestAccPipelineAuthorization_pipeline_repository (64.22s)
=== CONT  TestAccPipelineAuthorization_allPipeline_variableGroup
--- PASS: TestAccPipelineAuthorization_allPipeline_environment (73.20s)
=== CONT  TestAccPipelineAuthorization_multiPipeline_queue
--- PASS: TestAccPipelineAuthorization_allPipelineWithPipeline_queue (73.69s)
=== CONT  TestAccPipelineAuthorization_pipeline_environment
--- PASS: TestAccPipelineAuthorization_allPipeline_queue (74.01s)
=== CONT  TestAccPipelineAuthorization_pipeline_queue
--- PASS: TestAccPipelineAuthorization_pipeline_repository_crossProject (74.50s)
=== CONT  TestAccPipelineAuthorization_allPipeline_repository_crossProject
--- PASS: TestAccPipelineAuthorization_pipeline_endpoint (84.55s)
=== CONT  TestAccPipelineAuthorization_allPipeline_repository
--- PASS: TestAccPipelineAuthorization_pipeline_variableGroup (87.99s)
--- PASS: TestAccPipelineAuthorization_allPipeline_endpoint (95.61s)
--- PASS: TestAccPipelineAuthorization_allPipeline_variableGroup (60.70s)
--- PASS: TestAccPipelineAuthorization_allPipeline_repository_crossProject (50.97s)
--- PASS: TestAccPipelineAuthorization_pipeline_queue (51.98s)
--- PASS: TestAccPipelineAuthorization_multiPipeline_queue (52.82s)
--- PASS: TestAccPipelineAuthorization_pipeline_environment (52.39s)
--- PASS: TestAccPipelineAuthorization_allPipeline_repository (53.36s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        160.469s