Skip to content

NO-JIRA: feat(tnf): add resource-agents steps under baremetalds/two-node/fencing#75815

Open
jaypoulz wants to merge 1 commit intoopenshift:mainfrom
jaypoulz:tnf-update-rhcos-resource-agents
Open

NO-JIRA: feat(tnf): add resource-agents steps under baremetalds/two-node/fencing#75815
jaypoulz wants to merge 1 commit intoopenshift:mainfrom
jaypoulz:tnf-update-rhcos-resource-agents

Conversation

@jaypoulz
Copy link
Contributor

@jaypoulz jaypoulz commented Mar 6, 2026

  • Move update-rhcos-resource-agents and delete-rhcos-resource-agents-image from agent/ to baremetalds/two-node/fencing; rename to baremetalds-two-node-fencing-*
  • Wire agent and baremetalds-two-node-fencing-recovery workflows to new step refs; set OWNERS to edge-enablement approvers/reviewers
  • Add local hack/tnf-resource-agents-build test (Stream 9/10 Dockerfiles + script) to verify resource-agents build; images tagged in local registry only
  • Align CI step with local build: enable CRB/EPEL, add libxslt/systemd/which; on missing libqb-devel (e.g. Stream 10) build libqb from source and install stub RPM for make rpm
  • Add TNF REPO job variant (repo-ra-techpreview) and nested-podman in release nightlies (4.22, 4.23, 5.0) and installer presubmits

Made-with: Cursor

@openshift-ci-robot
Copy link
Contributor

@jaypoulz: This pull request explicitly references no jira issue.

Details

In response to this:

  • Move update-rhcos-resource-agents and delete-rhcos-resource-agents-image from agent/ to baremetalds/two-node/fencing; rename to baremetalds-two-node-fencing-*
  • Wire agent and baremetalds-two-node-fencing-recovery workflows to new step refs; set OWNERS to edge-enablement approvers/reviewers
  • Add local hack/tnf-resource-agents-build test (Stream 9/10 Dockerfiles + script) to verify resource-agents build; images tagged in local registry only
  • Align CI step with local build: enable CRB/EPEL, add libxslt/systemd/which; on missing libqb-devel (e.g. Stream 10) build libqb from source and install stub RPM for make rpm
  • Add TNF REPO job variant (repo-ra-techpreview) and nested-podman in release nightlies (4.22, 4.23, 5.0) and installer presubmits

Made-with: Cursor

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Mar 6, 2026
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 6, 2026
@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Mar 6, 2026
@openshift-ci openshift-ci bot requested review from bear-redhat and deepsm007 March 6, 2026 14:46
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 6, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jaypoulz
Once this PR has been reviewed and has the lgtm label, please assign danilo-gemoli for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jaypoulz jaypoulz force-pushed the tnf-update-rhcos-resource-agents branch from 918c12a to 57e8ba8 Compare March 6, 2026 14:50
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 6, 2026
@openshift-ci-robot openshift-ci-robot removed the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Mar 6, 2026
@jaypoulz jaypoulz force-pushed the tnf-update-rhcos-resource-agents branch 2 times, most recently from 017321d to bde5021 Compare March 6, 2026 15:20
- nested-podman
cron: 19 10,20 * * *
steps:
cluster_profile: equinix-edge-enablement

Choose a reason for hiding this comment

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

Silly question - is actually pointing to OFICR?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, the legacy name was likely kept due to too many places to find/replace

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@eggfoobar is the expert here, but I don't think we changed the cluster profile name to offcir. We just added offcir secrets to the old profile.

Choose a reason for hiding this comment

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

ack. Let's make sure we have a Jira somewhere in the backlog to fix this since it falls into the "tribal knowledge" category.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yup, it's just an alias for our creds, future housekeeping we can change it, just needs some coordination. It's low priority until we get some cycles since there might be CI interruptions during the move.

More info on it can be found in the ci docs, https://docs.ci.openshift.org/how-tos/adding-a-cluster-profile/#what-is-a-cluster-profile

@jaypoulz
Copy link
Contributor Author

jaypoulz commented Mar 6, 2026

/pj-rehease periodic-ci-openshift-release-main-nightly-4.22-e2e-metal-ovn-two-node-fencing-recovery-repo-ra-techpreview periodic-ci-openshift-release-main-nightly-5.0-e2e-metal-ovn-two-node-fencing-recovery-repo-ra-techpreview

@jaypoulz
Copy link
Contributor Author

jaypoulz commented Mar 6, 2026

/pj-rehearse periodic-ci-openshift-release-main-nightly-4.22-e2e-metal-ovn-two-node-fencing-recovery-repo-ra-techpreview periodic-ci-openshift-release-main-nightly-5.0-e2e-metal-ovn-two-node-fencing-recovery-repo-ra-techpreview

@openshift-ci-robot
Copy link
Contributor

@jaypoulz: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

…ode/fencing

- Move update-rhcos-resource-agents and delete-rhcos-resource-agents-image to baremetalds/two-node/fencing; rename to baremetalds-two-node-fencing-*
- Wire agent and baremetalds-two-node-fencing-recovery workflows to new step refs; set OWNERS to edge-enablement approvers/reviewers
- REPO path: get release image and version from deployed cluster (clusterversion) only; step must run after cluster deployment
- Add hack/tnf-resource-agents-build test (Stream 9/10) and align CI step with local build (CRB/EPEL, libqb from source when needed)
- Add TNF REPO job variant and nested-podman in release nightlies (4.22, 5.0) and installer presubmits

Made-with: Cursor
@jaypoulz jaypoulz force-pushed the tnf-update-rhcos-resource-agents branch from bde5021 to dd11412 Compare March 10, 2026 19:39
@jaypoulz
Copy link
Contributor Author

/pj-rehearse periodic-ci-openshift-release-main-nightly-4.22-e2e-metal-ovn-two-node-fencing-recovery-repo-ra-techpreview periodic-ci-openshift-release-main-nightly-5.0-e2e-metal-ovn-two-node-fencing-recovery-repo-ra-techpreview

@openshift-ci-robot
Copy link
Contributor

@jaypoulz: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 10, 2026

@jaypoulz: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/rehearse/periodic-ci-openshift-release-main-nightly-5.0-e2e-metal-ovn-two-node-fencing-recovery-repo-ra-techpreview dd11412 link unknown /pj-rehearse periodic-ci-openshift-release-main-nightly-5.0-e2e-metal-ovn-two-node-fencing-recovery-repo-ra-techpreview
ci/rehearse/periodic-ci-openshift-release-main-nightly-4.22-e2e-metal-ovn-two-node-fencing-recovery-repo-ra-techpreview dd11412 link unknown /pj-rehearse periodic-ci-openshift-release-main-nightly-4.22-e2e-metal-ovn-two-node-fencing-recovery-repo-ra-techpreview

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants