Skip to content

fix(reproducer): enable IPv6 forwarding for network routing - #4226

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
pderenda:OSPNW-1776-fix-IPV6-forwarding
Oct 9, 2026
Merged

fix(reproducer): enable IPv6 forwarding for network routing#4226
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
pderenda:OSPNW-1776-fix-IPV6-forwarding

Conversation

@pderenda

@pderenda pderenda commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

IPv6 forwarding was not enabled during premetal provisioning, causing failures in IPv6-enabled scenarios when OCP VMs on IPv6 networks become unreachable with 'no route to host' errors.

@qodo-code-review

qodo-code-review Bot commented Oct 2, 2026 •
edited
Loading

Copy link
Copy Markdown

Code Review by Qodo

Grey Divider

Sorry, something went wrong

We weren't able to complete the code review on our side. Please try again manually by commenting /agentic_review on this PR.

Grey Divider

Qodo Logo

@openshift-ci

openshift-ci Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

Hi @pderenda. Thanks for your PR.

I'm waiting for a openstack-k8s-operators member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

@qodo-code-review

qodo-code-review Bot commented Oct 2, 2026

Copy link
Copy Markdown

PR Summary by Qodo

Enable IPv6 forwarding during reproducer network setup

🐞 Bug fix 🕐 Less than 10 minutes

Grey Divider

AI Description 

• Enable IPv6 forwarding on the reproducer host during network preparation.
• Restore routing to OCP VMs on IPv6 networks that were unreachable with “no route to host.”
Diagram 

graph TD
  A["Reproducer setup"] --> B["Host sysctl"] --> C["IPv6 VM routing"]
Loading
High-Level Assessment

Keep the new sysctl task beside the existing IPv4 forwarding task. It uses the role’s established mechanism; a separate sysctl configuration path would add complexity without a clear benefit.

Files changed (1) +6 / -0

Bug fix (1) +6 / -0
prepare_networking.ymlEnable host IPv6 forwarding during network preparation +6/-0

Enable host IPv6 forwarding during network preparation

• Adds an ansible.posix.sysctl task that sets net.ipv6.conf.all.forwarding to 1 in the existing privileged host-configuration block. This allows the reproducer host to forward traffic for IPv6-enabled OCP VM networks.

roles/reproducer/tasks/prepare_networking.yml

@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Oct 2, 2026

Copy link
Copy Markdown

Build succeeded (check pipeline).
https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/4e0bc8536989454193f48f3b09d15ee6

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 53m 23s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 32m 24s
✔️ podified-multinode-edpm-deployment-crc-centos-10 SUCCESS in 1h 20m 53s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 32m 49s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 21s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 26s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 20s
✔️ cifmw-molecule-reproducer SUCCESS in 16m 29s

evallesp
evallesp reviewed Oct 5, 2026
View reviewed changes
Comment thread roles/reproducer/tasks/prepare_networking.yml
@pderenda
pderenda force-pushed the OSPNW-1776-fix-IPV6-forwarding branch from b4ea10c to 37070e9 Compare October 5, 2026 14:33
@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Oct 6, 2026

Copy link
Copy Markdown

Build succeeded (check pipeline).
https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/1d03292ba88e42dd851a0392c05dae86

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 55m 05s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 19m 06s
✔️ podified-multinode-edpm-deployment-crc-centos-10 SUCCESS in 1h 22m 05s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 34m 56s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 36s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 40s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 26s
✔️ cifmw-molecule-reproducer SUCCESS in 16m 10s

evallesp
evallesp approved these changes Oct 8, 2026
View reviewed changes

@evallesp evallesp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Oct 8, 2026
@evallesp

evallesp commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

/ok-to-test

@pderenda
[reproducer] Enable IPv6 Router Advertisements for network routing
1479332 
IPv6 forwarding and Router Advertisement acceptance were not properly
configured during premetal provisioning, causing failures in IPv6-enabled
scenarios when OCP VMs on IPv6 networks become unreachable with 'no route
to host' errors.

This change enables:
- IPv6 forwarding on all interfaces (net.ipv6.conf.all.forwarding)
- IPv6 Router Advertisement acceptance on all interfaces (accept_ra=2)
- IPv6 Router Advertisement acceptance on new interfaces (default.accept_ra=2)

Signed-off-by: PDerenda <pderenda@redhat.com>
@pderenda
pderenda force-pushed the OSPNW-1776-fix-IPV6-forwarding branch from 37070e9 to 1479332 Compare October 8, 2026 08:21
@openshift-ci openshift-ci Bot removed the lgtm label Oct 8, 2026
@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Oct 9, 2026

Copy link
Copy Markdown

Build succeeded (check pipeline).
https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/19b2fa1cd3c14c28b777938ce7fb4b21

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 19m 54s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 21m 42s
✔️ podified-multinode-edpm-deployment-crc-centos-10 SUCCESS in 1h 18m 56s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 34m 08s
✔️ cifmw-pod-zuul-files SUCCESS in 6m 02s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 40s
✔️ cifmw-pod-pre-commit SUCCESS in 10m 18s
✔️ cifmw-molecule-reproducer SUCCESS in 14m 03s

evallesp
evallesp approved these changes Oct 9, 2026
View reviewed changes

@evallesp evallesp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm
/approve

@openshift-ci openshift-ci Bot added the lgtm label Oct 9, 2026
@openshift-ci

openshift-ci Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: evallesp

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

The pull request process is described 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

@openshift-ci openshift-ci Bot added the approved label Oct 9, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit f09c64d into openstack-k8s-operators:main Oct 9, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Valkyrie00 Valkyrie00 Valkyrie00 approved these changes
@evallesp evallesp evallesp approved these changes

Assignees

@evallesp evallesp

Labels

approved lgtm ok-to-test Ready For Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pderenda @evallesp @Valkyrie00