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

openvswitch_db: Make value optional when state is absent #61092

Open
wants to merge 1 commit into
base: devel
from

Conversation

@krsacme
Copy link
Contributor

commented Aug 22, 2019

SUMMARY

When the state is absent, value is mandatory. But removal of a key
does not require the value, as the key and value both can be removed
with 'ovs-vsctl remove' command. Making value as optional, it will
be used if provided. And value will be mandatory, when the state
is present.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

openvswitch_db

openvswitch_db: Make value options when state is absent
When the state is absent, value is mandatory. But removal of a key
does not require the value, as the key and value both can be removed
with 'ovs-vsctl remove' command. Making value as optional, it will
be used if provided. And value will be mandatory, when the state
is present.
@ansibot

This comment has been minimized.

Copy link
Contributor

commented Aug 22, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Aug 22, 2019

@krsacme, just so you are aware we have a dedicated Working Group for network.
You can find other people interested in this in #ansible-network on Freenode IRC
For more information about communities, meetings and agendas see https://github.com/ansible/community

click here for bot help

@krsacme krsacme changed the title openvswitch_db: Make value options when state is absent openvswitch_db: Make value optional when state is absent Aug 22, 2019

@ansible-zuul

This comment has been minimized.

Copy link

commented Aug 22, 2019

@ansibot ansibot added the stale_ci label Aug 30, 2019

@danielmellado danielmellado self-requested a review Sep 4, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.