Closed jsturtevant closed 1 month ago
Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jsturtevant
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/test ls
@jsturtevant: The specified target(s) for /test
were not found.
The following commands are available to trigger optional jobs:
/test pull-e2e-capz-validate-pr-templates
/test pull-e2e-capz-windows-2022-extension
/test pull-e2e-capz-windows-2022-extension-community
/test pull-e2e-capz-windows-2022-extension-gmsa
/test pull-e2e-run-capz-sh-windows-2022-hyperv
Use /test all
to run the following jobs that were automatically triggered:
pull-e2e-capz-windows-2022-extension
pull-e2e-capz-windows-2022-extension-gmsa
/test pull-e2e-capz-windows-2022-extension-gmsa
/test pull-e2e-capz-windows-2022-extension-gmsa
/test pull-e2e-capz-windows-2022-extension-gmsa
Authentication is working but goofed up the print statement function
/test pull-e2e-capz-windows-2022-extension-gmsa
/test pull-e2e-capz-windows-2022-extension-gmsa /hold for test
/assign @marosset @ritikaguptams
@jsturtevant: GitHub didn't allow me to assign the following users: ritikaguptams.
Note that only kubernetes-sigs members with read permissions, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. For more information please see the contributor guide
capacity error, we are hitting it often going to drop westeurope in separate PR /test pull-e2e-capz-windows-2022-extension-gmsa
/test pull-e2e-capz-windows-2022-extension-gmsa
westeurope three times in a row?!
/test pull-e2e-capz-windows-2022-extension-gmsa
/hold cancel
/lgtm
After switching to the Workload identity the old SDK didn't support the credentials.
This updates the SDK to use the latest version. The
DefaultAzureCredential
will work with both ENV variables and workload identity: https://github.com/Azure/azure-sdk-for-go/tree/sdk/azidentity/v1.7.0/sdk/azidentity#defaultazurecredential