Skip to content

Comments

feat: Added documentation for skipping test deployments#1796

Merged
ReneHezser merged 14 commits intoAzure:mainfrom
ReneHezser:skip-bicep-test
Mar 6, 2025
Merged

feat: Added documentation for skipping test deployments#1796
ReneHezser merged 14 commits intoAzure:mainfrom
ReneHezser:skip-bicep-test

Conversation

@ReneHezser
Copy link
Contributor

@ReneHezser ReneHezser commented Jan 17, 2025

Overview/Summary

Deployment of tests can be skipped by the CI

This PR fixes/adds/changes/removes

Azure/bicep-registry-modules#4203

Breaking Changes

none

As part of this Pull Request I have

  • Read the Contribution Guide and ensured this PR is compliant with the guide
  • Checked for duplicate Pull Requests
  • Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Ensured PR tests are passing
  • Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: Triage 🔍 Maintainers need to triage still label Jan 17, 2025
@ReneHezser ReneHezser marked this pull request as ready for review February 24, 2025 13:29
@ReneHezser ReneHezser requested a review from a team as a code owner February 24, 2025 13:30
@ReneHezser
Copy link
Contributor Author

Thanks @AlexanderSehr. I've adjusted the text.

ReneHezser and others added 3 commits March 4, 2025 17:34
…SNFR2.md

Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
…SNFR2.md

Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
@ReneHezser ReneHezser enabled auto-merge (squash) March 4, 2025 17:02
@ReneHezser ReneHezser merged commit 06144de into Azure:main Mar 6, 2025
4 checks passed
@ReneHezser ReneHezser deleted the skip-bicep-test branch March 7, 2025 07:18
eriqua added a commit to Azure/bicep-registry-modules that referenced this pull request Jun 5, 2025
## Description

Allow testcases to be excluded from deplyoments by the CI deployment.

Azure/Azure-Verified-Modules#1796 has been
merged and the documentation is here:
https://azure.github.io/Azure-Verified-Modules/spec/SNFR2/#skipping-deployments-should-not

## Pipeline Reference

| Pipeline |
| -------- |
|
[![avm.ptn.deployment-script.import-image-to-acr](https://github.com/ReneHezser/bicep-registry-modules/actions/workflows/avm.ptn.deployment-script.import-image-to-acr.yml/badge.svg?branch=enable-skip-ci-deployment-tests)](https://github.com/ReneHezser/bicep-registry-modules/actions/workflows/avm.ptn.deployment-script.import-image-to-acr.yml)
|
|
https://github.com/ReneHezser/bicep-registry-modules/actions/runs/13542323177
--> expected failing static validation proving both added pester tests
work as expected |

## Type of Change

- [x] Update to CI Environment or utilities (Non-module affecting
changes)
- [ ] Azure Verified Module updates:
- [ ] Bugfix containing backwards-compatible bug fixes, and I have NOT
bumped the MAJOR or MINOR version in `version.json`:
- [ ] Someone has opened a bug report issue, and I have included "Closes
#{bug_report_issue_number}" in the PR description.
- [ ] The bug was found by the module author, and no one has opened an
issue to report it yet.
- [ ] 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

## Checklist

- [x] I'm sure there are no other open Pull Requests for the same
update/change
- [ ] I have run `Set-AVMModule` locally to generate the supporting
module files.
- [x] My corresponding pipelines / checks run clean and green without
any errors or warnings

---------

Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
yffok pushed a commit to yffok/bicep-registry-modules that referenced this pull request Jan 15, 2026
## Description

Allow testcases to be excluded from deplyoments by the CI deployment.

Azure/Azure-Verified-Modules#1796 has been
merged and the documentation is here:
https://azure.github.io/Azure-Verified-Modules/spec/SNFR2/#skipping-deployments-should-not

## Pipeline Reference

| Pipeline |
| -------- |
|
[![avm.ptn.deployment-script.import-image-to-acr](https://github.com/ReneHezser/bicep-registry-modules/actions/workflows/avm.ptn.deployment-script.import-image-to-acr.yml/badge.svg?branch=enable-skip-ci-deployment-tests)](https://github.com/ReneHezser/bicep-registry-modules/actions/workflows/avm.ptn.deployment-script.import-image-to-acr.yml)
|
|
https://github.com/ReneHezser/bicep-registry-modules/actions/runs/13542323177
--> expected failing static validation proving both added pester tests
work as expected |

## Type of Change

- [x] Update to CI Environment or utilities (Non-module affecting
changes)
- [ ] Azure Verified Module updates:
- [ ] Bugfix containing backwards-compatible bug fixes, and I have NOT
bumped the MAJOR or MINOR version in `version.json`:
- [ ] Someone has opened a bug report issue, and I have included "Closes
#{bug_report_issue_number}" in the PR description.
- [ ] The bug was found by the module author, and no one has opened an
issue to report it yet.
- [ ] 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

## Checklist

- [x] I'm sure there are no other open Pull Requests for the same
update/change
- [ ] I have run `Set-AVMModule` locally to generate the supporting
module files.
- [x] My corresponding pipelines / checks run clean and green without
any errors or warnings

---------

Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Triage 🔍 Maintainers need to triage still

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants