Skip to content

Use netavark create to fill out network config#610

Open
ashley-cui wants to merge 2 commits intocontainers:mainfrom
ashley-cui:netavarkcreate
Open

Use netavark create to fill out network config#610
ashley-cui wants to merge 2 commits intocontainers:mainfrom
ashley-cui:netavarkcreate

Conversation

@ashley-cui
Copy link
Copy Markdown
Member

@ashley-cui ashley-cui commented Jan 27, 2026

Netavark now supports a new command, netavark create, which will take incomplete network configs, validate options, and return a completed config.
Migrate c/common/libnetwork to use the netavark create command

Requires a new netavark, but mostly non-breaking, other than now we enforce true/false for no default route - 0/1 is no longer supported

@github-actions github-actions Bot added the common Related to "common" package label Jan 27, 2026
@ashley-cui ashley-cui force-pushed the netavarkcreate branch 2 times, most recently from fa4eabd to 0a9721c Compare April 7, 2026 19:40
@packit-as-a-service
Copy link
Copy Markdown

Packit jobs failed. @containers/packit-build please check.

@ashley-cui ashley-cui force-pushed the netavarkcreate branch 2 times, most recently from aaf14f5 to c34e22b Compare April 14, 2026 18:48
@ashley-cui ashley-cui force-pushed the netavarkcreate branch 3 times, most recently from 23535d7 to dac307f Compare April 16, 2026 14:35
@ashley-cui
Copy link
Copy Markdown
Member Author

Tests are passing here, but would love to do a vendor into podman before this merges, but that depends on new images which are currently failing. Hold tight..

Comment thread common/libnetwork/netavark/config.go Outdated
Comment thread common/libnetwork/types/network.go Outdated
Comment thread common/libnetwork/netavark/config.go
@ashley-cui ashley-cui force-pushed the netavarkcreate branch 2 times, most recently from c896fb5 to e635760 Compare April 22, 2026 15:02
@Luap99 Luap99 added the podman 6 breaking changes that should go only into podman 6 only label Apr 22, 2026
ashley-cui added a commit to ashley-cui/podman that referenced this pull request Apr 23, 2026
Signed-off-by: Ashley Cui <acui@redhat.com>
ashley-cui added a commit to ashley-cui/podman that referenced this pull request Apr 23, 2026
Signed-off-by: Ashley Cui <acui@redhat.com>
@ashley-cui ashley-cui marked this pull request as ready for review April 24, 2026 00:41
@ashley-cui
Copy link
Copy Markdown
Member Author

ashley-cui commented Apr 24, 2026

Ready to review. Tests are passing in containers/podman#28570 except for one, which will be fixed in containers/netavark#1440. Please merge the netavark PR first to unblock podman vendoring.

@containers/container-libs-maintainers PTAL

ashley-cui added a commit to ashley-cui/podman that referenced this pull request Apr 24, 2026
For containers/container-libs#610

Signed-off-by: Ashley Cui <acui@redhat.com>
From containers/automation_images#443

Signed-off-by: Ashley Cui <acui@redhat.com>
Netavark now supports a new command, netavark create, which will take incomplete network configs, validate options, and return a completed config.
Migrate c/common/libnetwork to use the netavark create command

Requires a new netavark, but mostly non-breaking, other than now we enforce true/false for no default route - 0/1 is no longer supported

Signed-off-by: Ashley Cui <acui@redhat.com>
ashley-cui added a commit to ashley-cui/podman that referenced this pull request Apr 25, 2026
For containers/container-libs#610

Signed-off-by: Ashley Cui <acui@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

common Related to "common" package podman 6 breaking changes that should go only into podman 6 only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants