Skip to content

Add nvidia_aspeed_bmc marker#563

Draft
william8545 wants to merge 1 commit intosonic-net:masterfrom
william8545:nvidia_bmc_marker
Draft

Add nvidia_aspeed_bmc marker#563
william8545 wants to merge 1 commit intosonic-net:masterfrom
william8545:nvidia_bmc_marker

Conversation

@william8545
Copy link
Copy Markdown

Add the nvidia_aspeed_bmc marker for NVIDIA/Mellanox hw-mgmt integration scripts.

Signed-off-by: William Tsai <willtsai@nvidia.com>
@william8545 william8545 requested a review from a team as a code owner April 20, 2026 19:50
@mssonicbld
Copy link
Copy Markdown

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@william8545 william8545 reopened this May 1, 2026
@william8545 william8545 marked this pull request as draft May 1, 2026 00:11
@william8545 william8545 marked this pull request as ready for review May 1, 2026 00:12
@william8545 william8545 marked this pull request as draft May 1, 2026 00:12
@mssonicbld
Copy link
Copy Markdown

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@william8545 william8545 closed this May 1, 2026
@william8545 william8545 deleted the nvidia_bmc_marker branch May 1, 2026 02:50
@william8545 william8545 restored the nvidia_bmc_marker branch May 1, 2026 02:50
@william8545 william8545 reopened this May 1, 2026
@mssonicbld
Copy link
Copy Markdown

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Contributor

@paulmenzel paulmenzel left a comment

Choose a reason for hiding this comment

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

In my opinion, the marker should be introduced, when the other changes depending on it get added. No separate merge/pull request is needed.

Additionally, it’d be really great, if you wrote a commit message body.

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.

4 participants