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

Set interface name same as the network_interface name. #21318

Closed
wants to merge 3 commits into from
Closed

Set interface name same as the network_interface name. #21318

wants to merge 3 commits into from

Conversation

vikas-goel
Copy link
Contributor

@vikas-goel vikas-goel commented Jan 20, 2024

When interface_name attribute in containers.conf is set to "device", then set interface names inside containers same as the network_interface names.

If the interface name is set in the user request, then that takes precedence.

Fixes: #21313

Signed-off-by: Vikas Goel vikas.goel@gmail.com

Does this PR introduce a user-facing change? Yes


Copy link
Contributor

openshift-ci bot commented Jan 20, 2024

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

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/test-infra repository.

@openshift-ci openshift-ci bot added the do-not-merge/release-note-label-needed Enforce release-note requirement, even if just None label Jan 20, 2024
Copy link

Ephemeral COPR build failed. @containers/packit-build please check.

@vikas-goel vikas-goel changed the title Set interface name to the parent interface name of the bridge. Set interface name to the host interface name of the bridge. Jan 23, 2024
@vikas-goel vikas-goel changed the title Set interface name to the host interface name of the bridge. Set interface name same as the network_interface name. Jan 24, 2024
@TomSweeneyRedHat
Copy link
Member

TomSweeneyRedHat commented Jan 25, 2024

/approved

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 25, 2024
@TomSweeneyRedHat
Copy link
Member

@vikas-goel It looks like you have some syntax errors:

# github.com/containers/podman/v4/libpod
libpod/networking_common.go:470:11: undefined: nicName
libpod/networking_common.go:486:36: undefined: nicName
libpod/networking_common.go:486:81: c.runtime.config.Containers.InterfaceName undefined (type "github.com/containers/common/pkg/config".ContainersConfig has no field or method InterfaceName)
libpod/networking_common.go:487:27: undefined: nicName
libpod/runtime_ctr.go:274:82: ctr.runtime.config.Containers.InterfaceName undefined (type "github.com/containers/common/pkg/config".ContainersConfig has no field or method InterfaceName)
make[1]: *** [Makefile:350: bin/podman] Error 1
make[1]: Leaving directory '/var/tmp/go/src/github.com/containers/podman'
make: *** [Makefile:740: podman-release-amd64.tar.gz] Error 2

@vikas-goel
Copy link
Contributor Author

vikas-goel commented Jan 26, 2024

@vikas-goel It looks like you have some syntax errors:

# github.com/containers/podman/v4/libpod
libpod/networking_common.go:470:11: undefined: nicName
libpod/networking_common.go:486:36: undefined: nicName
libpod/networking_common.go:486:81: c.runtime.config.Containers.InterfaceName undefined (type "github.com/containers/common/pkg/config".ContainersConfig has no field or method InterfaceName)
libpod/networking_common.go:487:27: undefined: nicName
libpod/runtime_ctr.go:274:82: ctr.runtime.config.Containers.InterfaceName undefined (type "github.com/containers/common/pkg/config".ContainersConfig has no field or method InterfaceName)
make[1]: *** [Makefile:350: bin/podman] Error 1
make[1]: Leaving directory '/var/tmp/go/src/github.com/containers/podman'
make: *** [Makefile:740: podman-release-amd64.tar.gz] Error 2

This is waiting for containers/common#1814 to get merged and promoted.

Copy link
Contributor

openshift-ci bot commented Jan 26, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: vikas-goel
Once this PR has been reviewed and has the lgtm label, please assign giuseppe for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

@openshift-ci openshift-ci bot removed the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 26, 2024
When interface_name attribute in containers.conf file is set to "device", then set interface names inside containers same as the network_interface names of the respective bridges.

If the interface name is set in the user request, then that takes precedence.

Fixes: #21313

Signed-off-by: Vikas Goel <vikas.goel@gmail.com>
@vikas-goel
Copy link
Contributor Author

/retest

Copy link
Contributor

openshift-ci bot commented Jan 30, 2024

@vikas-goel: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test message.

In response to this:

/retest

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/test-infra repository.

@vikas-goel
Copy link
Contributor Author

@rhatdan , @TomSweeneyRedHat , @mheon ,
As this repo is updated to use the latest common repo, can you add /ok-to-test tag to retest this PR?

When interface_name attribute in containers.conf file is set to "device", then set interface names inside containers same as the network_interface names of the respective network.

If the interface name is set in the user request, then that takes precedence.

Fixes: #21313

Signed-off-by: Vikas Goel <vikas.goel@gmail.com>
@vikas-goel vikas-goel closed this by deleting the head repository Jan 30, 2024
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Apr 30, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
do-not-merge/release-note-label-needed Enforce release-note requirement, even if just None locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
3 participants