cloud-provider-azure icon indicating copy to clipboard operation
cloud-provider-azure copied to clipboard

Use managed identity for aks pipeline

Open lzhecheng opened this issue 1 year ago • 3 comments

What type of PR is this?

/kind testing

What this PR does / why we need it:

Use managed identity for aks pipeline

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:


lzhecheng avatar Apr 29 '24 07:04 lzhecheng

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lzhecheng

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Apr 29 '24 07:04 k8s-ci-robot

/hold

lzhecheng avatar Apr 30 '24 07:04 lzhecheng

@nilo19 @feiskyer the pipeline builds are good. Please help take a look.

lzhecheng avatar May 01 '24 00:05 lzhecheng

/lgtm

nilo19 avatar May 06 '24 00:05 nilo19

/unhold

lzhecheng avatar May 06 '24 01:05 lzhecheng

/cherrypick release-1.30

lzhecheng avatar May 08 '24 05:05 lzhecheng

/cherrypick release-1.29

lzhecheng avatar May 08 '24 05:05 lzhecheng

/cherrypick release-1.28

lzhecheng avatar May 08 '24 05:05 lzhecheng

/cherrypick release-1.27

lzhecheng avatar May 08 '24 05:05 lzhecheng

@lzhecheng: new pull request created: #6147

In response to this:

/cherrypick release-1.30

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@lzhecheng: new pull request created: #6148

In response to this:

/cherrypick release-1.29

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@lzhecheng: #6048 failed to apply on top of branch "release-1.28":

Applying: Use managed identity for aks pipeline
Using index info to reconstruct a base tree...
M	.pipelines/ccm-image.yml
M	.pipelines/cnm-image.yml
M	.pipelines/run-autoscaling-e2e.yml
M	.pipelines/run-autoscaling-multipool-e2e.yml
M	.pipelines/scripts/run-e2e.sh
M	tests/e2e/e2e_test.go
M	tests/e2e/utils/azure_auth.go
Falling back to patching base and 3-way merge...
Auto-merging tests/e2e/utils/azure_auth.go
CONFLICT (content): Merge conflict in tests/e2e/utils/azure_auth.go
Auto-merging tests/e2e/e2e_test.go
Auto-merging .pipelines/scripts/run-e2e.sh
Auto-merging .pipelines/run-autoscaling-multipool-e2e.yml
Auto-merging .pipelines/run-autoscaling-e2e.yml
Auto-merging .pipelines/cnm-image.yml
Auto-merging .pipelines/ccm-image.yml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Use managed identity for aks pipeline
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherrypick release-1.28

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@lzhecheng: #6048 failed to apply on top of branch "release-1.27":

Applying: Use managed identity for aks pipeline
Using index info to reconstruct a base tree...
M	.pipelines/ccm-image.yml
M	.pipelines/cnm-image.yml
M	.pipelines/run-autoscaling-e2e.yml
M	.pipelines/run-autoscaling-multipool-e2e.yml
A	.pipelines/run-vmas-e2e.yml
M	.pipelines/scripts/run-e2e.sh
M	tests/e2e/e2e_test.go
M	tests/e2e/utils/azure_auth.go
Falling back to patching base and 3-way merge...
Auto-merging tests/e2e/utils/azure_auth.go
CONFLICT (content): Merge conflict in tests/e2e/utils/azure_auth.go
Auto-merging tests/e2e/e2e_test.go
Auto-merging .pipelines/scripts/run-e2e.sh
CONFLICT (content): Merge conflict in .pipelines/scripts/run-e2e.sh
CONFLICT (modify/delete): .pipelines/run-vmas-e2e.yml deleted in HEAD and modified in Use managed identity for aks pipeline. Version Use managed identity for aks pipeline of .pipelines/run-vmas-e2e.yml left in tree.
Auto-merging .pipelines/run-autoscaling-multipool-e2e.yml
Auto-merging .pipelines/run-autoscaling-e2e.yml
Auto-merging .pipelines/cnm-image.yml
Auto-merging .pipelines/ccm-image.yml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Use managed identity for aks pipeline
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherrypick release-1.27

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.