Skip to content

fix: CI - Add timout to cert creation#6658

Open
AlexanderSehr wants to merge 1 commit intomainfrom
users/alsehr/certTimeout
Open

fix: CI - Add timout to cert creation#6658
AlexanderSehr wants to merge 1 commit intomainfrom
users/alsehr/certTimeout

Conversation

@AlexanderSehr
Copy link
Copy Markdown
Collaborator

@AlexanderSehr AlexanderSehr commented Mar 7, 2026 •
edited
Loading

Description

To avoid pipelines from running into a timeout after 6h because of a certificate not being correclty created, this PR adds a maxRetries mechanism to the script.

Result in deployment script:
image

Example timout: ref

Pipeline Reference

Pipeline
avm.res.web.hosting-environment

Type of Change

  • Azure Verified Module updates:
    • Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:
    • Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
    • Breaking changes and I have bumped the MAJOR version in version.json.
    • Update to documentation
  • Update to CI Environment or utilities (Non-module affecting changes)

AlexanderSehr
AlexanderSehr commented Mar 7, 2026
View reviewed changes
utilities/e2e-template-assets/scripts/Set-CertificateInKeyVault.ps1
Copy link
Copy Markdown
Collaborator Author

@AlexanderSehr AlexanderSehr Mar 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Disable actions before merging

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs: Triage 🔍 Maintainers need to triage still Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue labels Mar 7, 2026
@AlexanderSehr AlexanderSehr marked this pull request as ready for review March 8, 2026 09:59
@AlexanderSehr AlexanderSehr requested a review from a team as a code owner March 8, 2026 09:59
@AlexanderSehr AlexanderSehr self-assigned this Mar 8, 2026
@AlexanderSehr AlexanderSehr added Needs: Core Team 🧞 This item needs the AVM Core Team to review it Type: CI 🚀 This issue is related to the AVM CI and removed Needs: Triage 🔍 Maintainers need to triage still labels Mar 8, 2026
@AlexanderSehr
Copy link
Copy Markdown
Collaborator Author

AlexanderSehr commented Apr 3, 2026

ping @Azure/avm-core-team-technical

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@AlexanderSehr AlexanderSehr

Labels

Needs: Core Team 🧞 This item needs the AVM Core Team to review it Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue Type: CI 🚀 This issue is related to the AVM CI

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@AlexanderSehr