Skip to content

enable dnsplugin for network create#4187

Merged
openshift-merge-robot merged 1 commit intocontainers:masterfrom
baude:dnspluginenable
Oct 29, 2019
Merged

enable dnsplugin for network create#4187
openshift-merge-robot merged 1 commit intocontainers:masterfrom
baude:dnspluginenable

Conversation

@baude
Copy link
Member

@baude baude commented Oct 3, 2019

when users create a new network and the dnsname plugin can be found by
podman, we will enable container name resolution on the new network.
there is an option to opt out as well.

tests cannot be added until we solve the packaging portion of the
dnsname plugin.

Signed-off-by: baude bbaude@redhat.com

@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: baude

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-robot openshift-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/M labels Oct 3, 2019
@baude baude force-pushed the dnspluginenable branch 2 times, most recently from ee387e1 to dcf6a4c Compare October 22, 2019 12:29
@baude
Copy link
Member Author

baude commented Oct 27, 2019

ping, merge?

@rhatdan
Copy link
Member

rhatdan commented Oct 28, 2019

when users create a new network and the dnsname plugin can be found by
podman, we will enable container name resolution on the new network.
there is an option to opt *out* as well.

tests cannot be added until we solve the packaging portion of the
dnsname plugin.

Signed-off-by: baude <bbaude@redhat.com>
@baude
Copy link
Member Author

baude commented Oct 28, 2019

fixed up

@giuseppe
Copy link
Member

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 29, 2019
@openshift-merge-robot openshift-merge-robot merged commit 815bd56 into containers:master Oct 29, 2019
@baude baude deleted the dnspluginenable branch December 22, 2019 18:49
@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 Sep 26, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants