search

kubernetes-sigs / windows-testing

Containers, scripts and documentation for running Kubernetes tests with Windows nodes
Apache License 2.0
40 stars 55 forks source link

debug az login issues #462

Closed jackfrancis closed 2 months ago

jackfrancis commented 2 months ago

This PR shows us the az login context during execution flow progression

k8s-ci-robot commented 2 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jackfrancis Once this PR has been reviewed and has the lgtm label, please assign jsturtevant for approval. For more information see the Kubernetes Code Review Process.

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

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/kubernetes-sigs/windows-testing/blob/master/OWNERS)** 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 commented 2 months ago

@jackfrancis: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-e2e-capz-windows-2022-extension c1e470b1db4d6f25f38ac48bff408d9d7e65b9e8 link false /test pull-e2e-capz-windows-2022-extension

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
jsturtevant commented 2 months ago

The issues we were seeing were due to terraform scripts in https://github.com/kubernetes/k8s.io/pull/7101 and https://github.com/kubernetes/k8s.io/pull/7102 being applied separately instead of as a group which caused some authentication/authorization errors.

The other issues around Keyvault accessing the wrong tenant is resolved with https://github.com/kubernetes-sigs/windows-testing/pull/459/commits/a7dbf481f457e4daf159d570d3ca5baacde08bf6 which points to the correct tennant

jsturtevant commented 2 months ago

/close

k8s-ci-robot commented 2 months ago

@jsturtevant: Closed this PR.

In response to [this](https://github.com/kubernetes-sigs/windows-testing/pull/462#issuecomment-2259401829): >/close Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.