better handle appending \n on stderr #39019
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ansibot
added
bug
|
The test |
ansibot
added
the
needs_revision
mkrizek
removed
the
needs_triage
|
@bcoca - saw that this is going into |
|
@jladdjr 2.5.2 at earliest, 2.5.1 was already released |
|
@bcoca has this been cherry-picked into stable-2.5? |
|
@bcoca Why is this not in 2.5.2? |
|
@bcoca - I was also wondering the same thing. In case this hasn't been communicated, this is considered a breaking change to Tower and should be shipped ASAP. |
|
because it was scheduled for next week initially and i had scheduled merging a bunch of backports for this Friday, the sudden change in release date meant I would not have time to do so. Expect it for 2.5.3 |
nitzmahone
added
the
tower_blocker
jaywcarman
added a commit
to jaywcarman/power-up
that referenced
this pull request
Jun 29, 2018
Ansible version '2.5.2' has a bug that will display "[ERROR]:" when running a dynamic inventory script (ansible/ansible#39007) due to an appended new line character. This issue is resolved by ansible/ansible#39019 which is included in Ansible version '2.5.5'.
jaywcarman
added a commit
to IBM/power-up
that referenced
this pull request
Jul 3, 2018
Ansible version '2.5.2' has a bug that will display "[ERROR]:" when running a dynamic inventory script (ansible/ansible#39007) due to an appended new line character. This issue is resolved by ansible/ansible#39019 which is included in Ansible version '2.5.5'.
jaywcarman
added a commit
to jaywcarman/power-up
that referenced
this pull request
Jul 3, 2018
Ansible version '2.5.2' has a bug that will display "[ERROR]:" when running a dynamic inventory script (ansible/ansible#39007) due to an appended new line character. This issue is resolved by ansible/ansible#39019 which is included in Ansible version '2.5.5'.
jaywcarman
added a commit
to IBM/power-up
that referenced
this pull request
Jul 3, 2018
Ansible version '2.5.2' has a bug that will display "[ERROR]:" when running a dynamic inventory script (ansible/ansible#39007) due to an appended new line character. This issue is resolved by ansible/ansible#39019 which is included in Ansible version '2.5.5'.
rbrud
pushed a commit
to rbrud/power-up
that referenced
this pull request
Jul 22, 2018
Ansible version '2.5.2' has a bug that will display "[ERROR]:" when running a dynamic inventory script (ansible/ansible#39007) due to an appended new line character. This issue is resolved by ansible/ansible#39019 which is included in Ansible version '2.5.5'.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
SUMMARY
fixes #39007
ISSUE TYPE
COMPONENT NAME
inventory/script
ANSIBLE VERSION