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

Fix junos modules check_mode issue #37311

Merged
merged 1 commit into from Mar 26, 2018

Conversation

ganeshrn
Copy link
Member

SUMMARY

Fixes #37208

If check_mode is enabled instead of committing the config need to
discard all the changes to candidate db
In case of cli to discard changes issue rollback 0 command
and for netconf execute discard-changes rpc call

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

module_utils/network/junos/junos.py

ANSIBLE VERSION
2.5
ADDITIONAL INFORMATION

Fixes ansible#37208

If check_mode is enabled instead of committing th config need to
discard all the chnages to cnadidate db
In case of cli to discard changes issue `rollback 0` command
and for netconf execute `discard-changes` rpc call
@ansibot
Copy link
Contributor

ansibot commented Mar 12, 2018

@ansibot ansibot added bug This issue/PR relates to a bug. needs_triage Needs a first human triage before being processed. networking Network category support:core This issue/PR relates to code supported by the Ansible Engineering Team. support:network This issue/PR relates to code supported by the Ansible Network Team. test This PR relates to tests. labels Mar 12, 2018
@ganeshrn ganeshrn removed the needs_triage Needs a first human triage before being processed. label Mar 12, 2018
@ansibot ansibot added the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label Mar 12, 2018
@ansibot ansibot removed the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label Mar 13, 2018
@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Mar 21, 2018
@ganeshrn ganeshrn merged commit 8eaa9cc into ansible:devel Mar 26, 2018
@ganeshrn ganeshrn deleted the junos_banner_check_mode branch March 26, 2018 11:27
ganeshrn added a commit to ganeshrn/ansible that referenced this pull request Mar 26, 2018
Fixes ansible#37208

If check_mode is enabled instead of committing th config need to
discard all the chnages to cnadidate db
In case of cli to discard changes issue `rollback 0` command
and for netconf execute `discard-changes` rpc call
(cherry picked from commit 8eaa9cc)
ganeshrn added a commit that referenced this pull request Mar 29, 2018
Fixes #37208

If check_mode is enabled instead of committing th config need to
discard all the chnages to cnadidate db
In case of cli to discard changes issue `rollback 0` command
and for netconf execute `discard-changes` rpc call
(cherry picked from commit 8eaa9cc)
@ganeshrn
Copy link
Member Author

Fix will be available in 2.5.1

ryancurrah pushed a commit to ryancurrah/ansible that referenced this pull request Apr 4, 2018
Fixes ansible#37208

If check_mode is enabled instead of committing th config need to
discard all the chnages to cnadidate db
In case of cli to discard changes issue `rollback 0` command
and for netconf execute `discard-changes` rpc call
@ansible ansible locked and limited conversation to collaborators Apr 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug This issue/PR relates to a bug. networking Network category stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. support:core This issue/PR relates to code supported by the Ansible Engineering Team. support:network This issue/PR relates to code supported by the Ansible Network Team. test This PR relates to tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

check_mode breaks on junos_banner
2 participants