Skip to content

pipelines: comment out BaremetalDeploymentTesting dependsOn in publish stage - #756

Merged
bfjelds (bfjelds) merged 1 commit into
mainfrom
user/bfjelds/remove-baremetal-publish-dependson
Aug 28, 2026
Merged

pipelines: comment out BaremetalDeploymentTesting dependsOn in publish stage#756
bfjelds (bfjelds) merged 1 commit into
mainfrom
user/bfjelds/remove-baremetal-publish-dependson

Conversation

@bfjelds

@bfjelds bfjelds (bfjelds) commented Aug 28, 2026

Copy link
Copy Markdown
Member

Comments out BaremetalDeploymentTesting_host and BaremetalDeploymentTesting_container in the dependsOn list of the Publishing stage in .pipelines/templates/stages/publishing/publish.yml.

@bfjelds
bfjelds (bfjelds) requested a review from a team as a code owner August 28, 2026 17:56
Copilot AI lite review requested due to automatic review settings August 28, 2026 17:56
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

…h stage

Comment out BaremetalDeploymentTesting_host and BaremetalDeploymentTesting_container
in the Publishing stage dependsOn list.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bfjelds
bfjelds (bfjelds) force-pushed the user/bfjelds/remove-baremetal-publish-dependson branch from 873e8af to 9fc9ccf Compare August 28, 2026 17:57
@bfjelds bfjelds (bfjelds) changed the title pipelines: remove BaremetalDeploymentTesting dependsOn from publish stage pipelines: comment out BaremetalDeploymentTesting dependsOn in publish stage Aug 28, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Azure Pipelines publishing stage template to stop gating the Publishing stage on the baremetal deployment test stages, so publishing on main is no longer blocked by BaremetalDeploymentTesting_* completion.

Changes:

  • Removed BaremetalDeploymentTesting_host and BaremetalDeploymentTesting_container from the dependsOn list for the Publishing stage when stageType == 'pre'.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI review requested due to automatic review settings August 28, 2026 17:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

- FunctionalTesting
- BaremetalDeploymentTesting_host
- BaremetalDeploymentTesting_container
# - BaremetalDeploymentTesting_host

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

i'd rather do this via a checkbox param, but not blocking

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

as discussed, leaving as is for now.

@bfjelds
bfjelds (bfjelds) merged commit 45414db into main Aug 28, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants