Skip to content

feat(argo-workflows): Allow enrollment to prepare nodes for monitoring after inspection - #2562

Open
haseebsyed12 wants to merge 3 commits into
mainfrom
enroll-snmpv3-user
Open

haseebsyed12 wants to merge 3 commits into
mainfrom
enroll-snmpv3-user

Conversation

@haseebsyed12

@haseebsyed12 haseebsyed12 commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

What does this change do?

Adds --post-inspection-runbooks -- the ability to run an ordered list of Ironic runbooks on a node right after inspection, before RAID and firmware, as part of enrollment.

Upgrade impact

  • This change requires operator action to upgrade. If checked, add the
    upgrade-impact label and a release note: run scriv create from the
    repository root and describe the required action in the generated
    changelog.d/ file. See RELEASING.md.

Operator action means anything a deployment has to do beyond a normal resync:
deploy repo or values changes, new or removed secrets, enabling or disabling a
component, or a manual one-time step.

@github-actions

github-actions Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Python test results

466 tests   461 ✅  42s ⏱️
  1 suites    5 💤
  1 files      0 ❌

Results for commit 3a61a58.

♻️ This comment has been updated with latest results.

@haseebsyed12
haseebsyed12 force-pushed the enroll-snmpv3-user branch 3 times, most recently from 415fa6e to cfd80b8 Compare October 23, 2026 19:09
@haseebsyed12
haseebsyed12 requested a review from a team October 23, 2026 19:10
@haseebsyed12
haseebsyed12 marked this pull request as ready for review October 23, 2026 19:10
@haseebsyed12
haseebsyed12 force-pushed the enroll-snmpv3-user branch 2 times, most recently from 91fe273 to 812516f Compare October 23, 2026 19:44
…o downstream steps can consume it directly instead of parsing logs or resolving BMC addresses.

This branch has not been deployed

No deployments
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.

2 participants