Skip to content

Comments

Fix build-boot-artifacts-x86-host-sa and build-boot-artifacts-bfb-sa stages#317

Open
shevchenko-evgeny wants to merge 1 commit intoNVIDIA:mainfrom
shevchenko-evgeny:fix_sa_build
Open

Fix build-boot-artifacts-x86-host-sa and build-boot-artifacts-bfb-sa stages#317
shevchenko-evgeny wants to merge 1 commit intoNVIDIA:mainfrom
shevchenko-evgeny:fix_sa_build

Conversation

@shevchenko-evgeny
Copy link

Description

Fix build process described in the installation guide.
This PR fixes issues in the build process, specifically related to boot artifacts preparation.

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Related Issues (Optional)

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

Additional Notes

@shevchenko-evgeny shevchenko-evgeny requested a review from a team as a code owner February 17, 2026 19:08
@copy-pr-bot
Copy link

copy-pr-bot bot commented Feb 17, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ajf
Copy link
Collaborator

ajf commented Feb 18, 2026

/ok to test dd771ef

@ajf
Copy link
Collaborator

ajf commented Feb 18, 2026

@shevchenko-evgeny: to pass the DCO (Developer Certificate of Origin), add the Signed-Off-By: git header in the commit with git commit -s

@shevchenko-evgeny
Copy link
Author

@ajf The commit has been signed per your request.

The build-release-container-x86_64 task is currently failing on carbide-ssh-console --test main. I suspect the build container does not have ipmi_sim and ipmitool installed. Addressing that is outside the scope of this fix.

@ajf
Copy link
Collaborator

ajf commented Feb 19, 2026

/ok to test ba84966

@ajf
Copy link
Collaborator

ajf commented Feb 19, 2026

@shevchenko-evgeny this LGTM, but I need to check internally if we're actually enforcing commit signatures for non-NVIDIA org people.

@ajf
Copy link
Collaborator

ajf commented Feb 19, 2026

Signed-off-by: Evgeny Shevchenko <eshevchenko@mirantis.com>
@shevchenko-evgeny
Copy link
Author

@ajf signed

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