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

ipsec: Treat DHCPv4 off with no static address as IPv4 disabled #2556

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

cathay4t
Copy link
Member

When user desired dhcp: off with no static address for IPSec interface,
nmstate will fail as verification error.

For IPSec connection, the DHCPv4 off with empty IP address should be
treated by IP disabled as we have no intention on supporting routes on
IPSec connection yet. The route should be set by IPSec daemon or to xfrm
interface.

Unit test case and integration test cases included.

Copy link

Congratulations! One of the builds has completed. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/nmstate-nmstate-2556
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

When user desired `dhcp: off` with no static address for IPSec interface,
nmstate will fail as verification error.

For IPSec connection, the DHCPv4 off with empty IP address should be
treated by IP disabled as we have no intention on supporting routes on
IPSec connection yet. The route should be set by IPSec daemon or to xfrm
interface.

Unit test case and integration test cases included.

Signed-off-by: Gris Ge <fge@redhat.com>
@cathay4t cathay4t enabled auto-merge (rebase) February 21, 2024 23:51
@cathay4t cathay4t merged commit 4d0f082 into nmstate:base Feb 22, 2024
25 of 26 checks passed
@cathay4t cathay4t deleted the ipsec_dhcp branch February 22, 2024 00:10
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.

None yet

2 participants