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

Add troubleshooting URL for persistent socket path related issue #38542

Merged
merged 2 commits into from Apr 11, 2018

Conversation

ganeshrn
Copy link
Member

SUMMARY

socket path timeout related error messages are not displayed on the
console as the ansible-connection process does not have access to it.
Until this is fixed need to point to troubleshooting URL so that
users can take corrective actions.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

module_utils/connection.py

ANSIBLE VERSION
2.5
ADDITIONAL INFORMATION

socket path timeout related error messages are not displayed on the
console as the ansible-connection process does not have access to it.
Until this is fixed need to point to troubleshooting URL so that
users can take corrective actions.
@ganeshrn ganeshrn requested a review from gundalow April 10, 2018 16:48
@ansibot ansibot added bug This issue/PR relates to a bug. needs_triage Needs a first human triage before being processed. support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Apr 10, 2018
@ganeshrn ganeshrn removed the needs_triage Needs a first human triage before being processed. label Apr 10, 2018
@ansibot
Copy link
Contributor

ansibot commented Apr 10, 2018

The test ansible-test sanity --test pep8 [explain] failed with 1 error:

lib/ansible/module_utils/connection.py:125:161: E501 line too long (179 > 160 characters)

click here for bot help

@ansibot ansibot added ci_verified Changes made in this PR are causing tests to fail. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Apr 10, 2018
@gundalow gundalow added this to Need CP into 2.5.1 in zzz NOT USED: Networking Bugs Apr 10, 2018
@ansibot ansibot removed ci_verified Changes made in this PR are causing tests to fail. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Apr 11, 2018
@ganeshrn ganeshrn merged commit 53d3e7e into ansible:devel Apr 11, 2018
@ganeshrn ganeshrn deleted the socket_path_url_issue branch April 11, 2018 05:53
ganeshrn added a commit to ganeshrn/ansible that referenced this pull request Apr 11, 2018
…ible#38542)

* Add troubleshooting URL for persistent socket path related issue

socket path timeout related error messages are not displayed on the
console as the ansible-connection process does not have access to it.
Until this is fixed need to point to troubleshooting URL so that
users can take corrective actions.

* Fix CI issue

* Update changelog fragments
(cherry picked from commit 53d3e7e)
ganeshrn added a commit that referenced this pull request Apr 12, 2018
) (#38577)

* Add troubleshooting URL for persistent socket path related issue

socket path timeout related error messages are not displayed on the
console as the ansible-connection process does not have access to it.
Until this is fixed need to point to troubleshooting URL so that
users can take corrective actions.

* Fix CI issue

* Update changelog fragments
(cherry picked from commit 53d3e7e)
@ganeshrn ganeshrn moved this from Need CP into 2.5.1 to Done in zzz NOT USED: Networking Bugs Apr 16, 2018
ilicmilan pushed a commit to ilicmilan/ansible that referenced this pull request Nov 7, 2018
…ible#38542)

* Add troubleshooting URL for persistent socket path related issue

socket path timeout related error messages are not displayed on the
console as the ansible-connection process does not have access to it.
Until this is fixed need to point to troubleshooting URL so that
users can take corrective actions.

* Fix CI issue
@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. support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants