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

Fixes #25446, #25647, #22832, #22831 - Blank entry, filtering #2

Merged
merged 1 commit into from
Mar 26, 2019

Conversation

ofedoren
Copy link
Member

@ofedoren ofedoren commented Feb 5, 2019

Requires theforeman/foreman_ansible#225 for assign_roles commands.

Copy link
Member

@ares ares left a comment

Choose a reason for hiding this comment

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

just one thing, otherwise all works fine

command_name 'play'
action :play_roles

success_message _("Ansible roles are being played. Task's UUID:\n%{task_id}")
Copy link
Member

Choose a reason for hiding this comment

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

would it be possible to display job invocation id rather? here and in every other output we have here, it's the id that can be found in the response

@ofedoren ofedoren force-pushed the feat-25446-25647-22832-22831 branch from 2c4c24b to 09d224d Compare March 25, 2019 13:38
@ofedoren
Copy link
Member Author

@ares, changed as you suggested.

@ofedoren ofedoren force-pushed the feat-25446-25647-22832-22831 branch 2 times, most recently from cabad0b to b98cb83 Compare March 25, 2019 13:47
Copy link
Member

@ares ares left a comment

Choose a reason for hiding this comment

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

One little change in two strings and good to go

action :multiple_play_roles
command_name 'play-hosts'

success_message _("Ansible roles are being played. Task's id: %{id}")
Copy link
Member

Choose a reason for hiding this comment

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

s/Task's id/Job ID/

action :multiple_play_roles
command_name 'play-hostgroups'

success_message _("Ansible roles are being played. Task's id: %{id}")
Copy link
Member

Choose a reason for hiding this comment

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

s/Task's id/Job ID/

@ofedoren ofedoren force-pushed the feat-25446-25647-22832-22831 branch from b98cb83 to eaf09d1 Compare March 26, 2019 13:50
@ofedoren
Copy link
Member Author

@ares, done. Thanks!

@ares ares merged commit 6f1d112 into theforeman:master Mar 26, 2019
@ares
Copy link
Member

ares commented Mar 26, 2019

Thanks @ofedoren, merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants