Skip to content

Add deploy-sno-reconfig-ipv6 test job for jetlag#76602

Closed
mcornea wants to merge 2 commits intoopenshift:mainfrom
mcornea:jetlag-deploy-sno-reconfig-ipv6
Closed

Add deploy-sno-reconfig-ipv6 test job for jetlag#76602
mcornea wants to merge 2 commits intoopenshift:mainfrom
mcornea:jetlag-deploy-sno-reconfig-ipv6

Conversation

@mcornea
Copy link
Contributor

@mcornea mcornea commented Mar 20, 2026

Summary

Add a test job that verifies the assisted-service pod properly picks up configuration changes when redeploying on the same bastion. The job:

  1. Deploys SNO with standard IPv4 configuration
  2. Reconfigures all.yml for single-stack IPv6 with use_bastion_registry: true
  3. Re-runs setup-bastion.yml + sno-deploy.yml without cleaning up containers
  4. Verifies the IPv6 deployment succeeds (assisted-service pod picks up new config)

This tests the fix in redhat-performance/jetlag#787.

Changes

  • ref YAML: Add REDEPLOY_IPV6 env var (default false)
  • commands script: After initial deploy, if REDEPLOY_IPV6=true, append IPv6 network config + setup_bastion_registry: true + use_bastion_registry: true to all.yml and redeploy without clean-resources.sh
  • test config: Add deploy-sno-reconfig-ipv6 test entry with JETLAG_PR: "787"

After jetlag PR merges

Remove JETLAG_PR: "787" from the test entry and keep the job as a permanent regression test.

Test plan

  • /test deploy-sno-reconfig-ipv6 — verifies the full IPv4→IPv6 reconfig path

mtulio and others added 2 commits August 14, 2025 22:23
)

- Update Platform External job notification channel
  prepending icon for the platform and provider executed
  for this method.

Signed-off-by: Marius Cornea <mcornea@redhat.com>
Add a test job that deploys SNO with IPv4, then reconfigures for
single-stack IPv6 with bastion registry and redeploys on the same
bastion without cleaning up. This tests that the assisted-service
pod properly picks up configuration changes on redeploy.

New env var REDEPLOY_IPV6 controls the reconfig behavior.
Uses JETLAG_PR=787 to test the companion jetlag fix.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 20, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 20, 2026

PR needs rebase.

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.

@openshift-ci openshift-ci bot requested review from afcollins and akrzos March 20, 2026 13:14
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 20, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mcornea
Once this PR has been reviewed and has the lgtm label, please assign mtulio, neisw 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

@mcornea mcornea closed this Mar 20, 2026
@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Mar 20, 2026
@openshift-ci-robot
Copy link
Contributor

[REHEARSALNOTIFIER]
@mcornea: no rehearsable tests are affected by this change

Note: If this PR includes changes to step registry files (ci-operator/step-registry/) and you expected jobs to be found, try rebasing your PR onto the base branch. This helps pj-rehearse accurately detect changes when the base branch has moved forward.

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

Labels

needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. rehearsals-ack Signifies that rehearsal jobs have been acknowledged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants