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

Bugfix for aggregate function in junos_user module #59016

Open
wants to merge 1 commit into
base: stable-2.8
from

Conversation

Projects
None yet
3 participants
@kr3ator
Copy link

commented Jul 12, 2019

Removed unnecessary conditionals that break the aggregate function of the module.

SUMMARY

This change fixes bug described in bug "junos_user: wrong SSH-key assignment in aggregate mode #58978".

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

junos_user

ADDITIONAL INFORMATION

Please see bug report "junos_user: wrong SSH-key assignment in aggregate mode #58978".

Bugfix for aggregate function
Removed unnecessary conditionals that break the aggregate functionality of the module.
@ansibot

This comment has been minimized.

Copy link
Contributor

commented Jul 12, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Jul 12, 2019

@kr3ator, 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

@kr3ator kr3ator changed the title Bugfix for aggregate function Bugfix for aggregate function in junos_user module Jul 12, 2019

@ansible-zuul

This comment has been minimized.

Copy link

commented Jul 12, 2019

Build succeeded (third-party-check pipeline).

@abadger

This comment has been minimized.

Copy link
Member

commented Jul 17, 2019

@kr3ator This bugfix needs to be merged into devel before it can be backported to a stable branch. When it does get backported, it will also need a changelog fragment. Please see this link for details of the backporting process: http://docs.testing.ansible.com/ansible/devel/community/development_process.html#backporting-merged-prs

@ansibot ansibot removed the needs_triage label Jul 17, 2019

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