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.

Sign up for GitHub

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

Jump to bottom

eos_config module exit session gracefully #37920

Merged
merged 1 commit into from
Mar 29, 2018
Merged

eos_config module exit session gracefully #37920

merged 1 commit into from
Mar 29, 2018

Conversation

ganeshrn
Copy link
Member

SUMMARY

Fixes #36979
If abort is not issued in the top level session prompt
the existing session goes to pending state.
The fix is to come out of config mode by issuing end command
and again to same config session and execute abort which
abort is issued at the top level session prompt.

(cherry picked from commit 017ea01)

Merged to devel #37244

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

eos/eos.py

ANSIBLE VERSION
2.5
ADDITIONAL INFORMATION

@ganeshrn
eos_config module exit session gracefully
9a1ccba 
Fixes ansible#36979
If `abort` is not issued in the top level session prompt
the existing session goes to pending state.
The fix is to come out of config mode by issuing `end` command
and again to same config session and execute `abort` which
`abort` is issued at the top level session prompt.

(cherry picked from commit 017ea01)
@ansibot
Copy link
Contributor

ansibot commented Mar 26, 2018

cc @Qalthos @gdpak @gundalow @privateip @rcarrillocruz @trishnaguha
click here for bot help

@ansibot ansibot added affects_2.5 This issue/PR affects Ansible v2.5 backport This PR does not target the devel branch. 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. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Mar 26, 2018
@sivel sivel removed the needs_triage Needs a first human triage before being processed. label Mar 26, 2018
@ganeshrn ganeshrn merged commit 4dffcd3 into ansible:stable-2.5 Mar 29, 2018
@ganeshrn ganeshrn deleted the backport/2.5/37244 branch March 29, 2018 13:28
@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.
Reviewers
No reviews
Assignees
No one assigned
Labels
affects_2.5 This issue/PR affects Ansible v2.5 backport This PR does not target the devel branch. bug This issue/PR relates to a bug. 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.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ganeshrn @ansibot @sivel