Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IPv6 network attachements - no dual stack #689

Conversation

hjensas
Copy link
Contributor

@hjensas hjensas commented Jan 13, 2024

Add support for IPv6 in network attachement definitions. This does not enable dual-stack support.

NOTE:
Dual stack should be possible by utilizing ipRanges in the whearabouts ipam config, but attempts to use ipRanges fails with an error indicating the wherabouts version in OCP does not have this feature.
See: #687 (comment)

@hjensas hjensas requested a review from stuggi January 13, 2024 16:58
@openshift-ci openshift-ci bot requested review from fao89 and olliewalsh January 13, 2024 16:58
Makefile Outdated
@@ -2010,6 +2012,16 @@ ifeq ($(NETWORK_BGP), true)
netattach: export INTERFACE_BGP_1=${NNCP_BGP_1_INTERFACE}
netattach: export INTERFACE_BGP_2=${NNCP_BGP_2_INTERFACE}
endif
ifeq ($(NETWORK_ISOLATION_IPV4), true)
netattach: export IPV4_ENABLED=true
netattach: export CTLPLANE_IPV6_ADDRESS_PREFIX=${NNCP_CTLPLANE_IP_ADDRESS_PREFIX}
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be *_IP_* instead of *_IPV6_*?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh my, yes it should. Fixed.

@hjensas hjensas force-pushed the netatt-ipv6-no-dual-stack branch from ccfa50c to 1b6332d Compare January 15, 2024 15:37
Copy link
Contributor

@abays abays left a comment

Choose a reason for hiding this comment

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

/lgtm

Copy link
Contributor

openshift-ci bot commented Jan 15, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, hjensas

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

The pull request process is described here

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

Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://review.rdoproject.org/zuul/buildset/52efca4cf5304d29a889b74780947818

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 24m 05s
install-yamls-crc-podified-edpm-baremetal RETRY_LIMIT in 4s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 09m 51s

@abays
Copy link
Contributor

abays commented Jan 15, 2024

recheck

RETRY_LIMIT

Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://review.rdoproject.org/zuul/buildset/ab9cfb0323cd47ed97f2324a83a8c8d2

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 23m 28s
install-yamls-crc-podified-edpm-baremetal RETRY_LIMIT in 4m 47s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 09m 33s

scripts/gen-netatt.sh Outdated Show resolved Hide resolved
Add support for IPv6 in network attachement definitions.
This does not enable dual-stack support.

NOTE: Dual stack should be possible by utilizing `ipRanges`
      in the whearabouts ipam config, but attempts to use
      `ipRanges` fails with an error indicating the wherabouts
      version in OCP does not have this feature.

Signed-off-by: Harald Jensås <[email protected]>
@hjensas hjensas force-pushed the netatt-ipv6-no-dual-stack branch from 1b6332d to b747422 Compare January 16, 2024 07:51
@openshift-ci openshift-ci bot removed the lgtm label Jan 16, 2024
@rabi
Copy link
Contributor

rabi commented Jan 16, 2024

/lgtm

Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://review.rdoproject.org/zuul/buildset/edc39c53b4804140b784bb9a8b93e0fa

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 30m 10s
install-yamls-crc-podified-edpm-baremetal FAILURE in 1h 13m 00s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 06m 26s

@abays
Copy link
Contributor

abays commented Jan 16, 2024

recheck

Compute kernel panic:

[   11.739532] ---[ end Kernel panic - not syncing: Jitter RNG permanent health test failure ]---

@openshift-merge-bot openshift-merge-bot bot merged commit f64f03e into openstack-k8s-operators:main Jan 16, 2024
5 checks passed
@hjensas hjensas deleted the netatt-ipv6-no-dual-stack branch January 16, 2024 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants