Skip to content

OCPSTRAT-3298: Add IEEE 829 test plan for NodePool rollout control - #1

Open
redhat-chai-bot wants to merge 5 commits into
hypershift-community:mainfrom
redhat-chai-bot:ocpstrat-9238-test-plan
Open

redhat-chai-bot wants to merge 5 commits into
hypershift-community:mainfrom
redhat-chai-bot:ocpstrat-9238-test-plan

Conversation

@redhat-chai-bot

@redhat-chai-bot redhat-chai-bot commented Sep 5, 2026 •

Copy link
Copy Markdown

Summary

Add plans/ocpstrat-3298.md, an IEEE 829-2008-compliant manual test plan for the NodePool rollout-control behavior covered by openshift/hypershift#8698.

Coverage

The plan documents three scenarios from the HyperShift e2e coverage:

  • Management-side HAProxy annotation changes do not trigger a rollout.
  • A spec-driven MachineConfig change triggers a rollout and updates the relevant hashes.
  • Removing the rollout annotation and forcing reconciliation re-seeds the annotation without triggering a rollout.

The plan maps verified OCPSTRAT-3298 acceptance criteria to these scenarios and makes automated e2e tests primary, automated unit tests supplementary, and manual procedures fallback-only when automation is infeasible.

Validation

  • Exactly one plan file is added: plans/ocpstrat-3298.md.
  • All 14 documented IEEE 829 sections are present.
  • The automation/evidence matrix and fallback policy are included.
  • The three test cases were cross-checked against the functions and constants in HyperShift PR #8698.
  • No stale OCPSTRAT-9238 references remain.

redhat-chai-bot and others added 5 commits September 5, 2026 07:54
…t control

Add manual test plan covering the three e2e scenarios from HyperShift
PR #8698: management-side image changes must not trigger rollouts,
spec-driven MachineConfig changes must trigger rollouts and update
hash annotations, and operator upgrade annotation seeding must occur
without triggering rollouts.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@redhat-chai-bot redhat-chai-bot changed the title OCPSTRAT-9238: Add IEEE 829 test plan for NodePool rollout control OCPSTRAT-3298: Add IEEE 829 test plan for NodePool rollout control Sep 7, 2026
Comment thread plans/ocpstrat-3298.md
```
oc get nodepool upgrade-seed-test -n <hc-namespace> -o jsonpath='{.metadata.annotations.hypershift\.openshift\.io/nodePoolCurrentRolloutConfig}'
```
Expected: a non-empty value appears within 1 minute.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should it verify that re-seeded value equals BASELINE_ROLLOUT_HASH?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I checked OperatorUpgradeNoRolloutTest in HyperShift PR #8698. It does not record a baseline rollout-hash value or assert that the re-seeded value equals one. It asserts that the annotation is restored within one minute and non-empty, while separately verifying UpdatingConfig remains False, nodePoolCurrentConfig is unchanged, and node identities are preserved.

The plan intentionally mirrors those automated assertions, so I did not add a stronger equality requirement only to the manual fallback path. If the upstream e2e test is strengthened to assert baseline-hash equality, the plan should be updated to match.


AI-generated. Review for accuracy.

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