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

Jump to bottom

Fixes #29111 - Ansible Inventory includes newlines in response #8289

Merged
merged 1 commit into from Feb 1, 2021
Merged

Fixes #29111 - Ansible Inventory includes newlines in response #8289

merged 1 commit into from Feb 1, 2021

Conversation

anandkumaragrawal
Copy link
Contributor

No description provided.

@theforeman-bot
Copy link
Member

Can one of the admins verify this patch?

2 similar comments
@theforeman-bot
Copy link
Member

Can one of the admins verify this patch?

@theforeman-bot
Copy link
Member

Can one of the admins verify this patch?

@theforeman-bot
Copy link
Member

Issues: #29111

@ofedoren
Copy link
Member

ok to test

ofedoren
ofedoren approved these changes Jan 31, 2021
View reviewed changes
Copy link
Member

@ofedoren ofedoren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @anandkumaragrawal, fixes the issue; checked with hammer.

@xprazak2 xprazak2 merged commit 540c577 into theforeman:develop Feb 1, 2021
@xprazak2
Copy link
Contributor

xprazak2 commented Feb 1, 2021

Thanks everyone!

@anandkumaragrawal anandkumaragrawal deleted the report_template branch February 2, 2021 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xprazak2 xprazak2 xprazak2 approved these changes

@ofedoren ofedoren ofedoren approved these changes

Assignees
No one assigned
Labels
Needs testing Templates
Projects
None yet
Milestone
No milestone
4 participants
@anandkumaragrawal @theforeman-bot @ofedoren @xprazak2