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 #12926 - add rerun to host row in job detail #93
Conversation
def template_invocation_actions(task, host, job_invocation) | ||
[ | ||
display_link_if_authorized(_('View host'), hash_for_host_path(host).merge(:auth_object => host, :permission => :view_hosts)), | ||
display_link_if_authorized(_('Rerun on %s') % host.name, hash_for_rerun_job_invocation_path(:id => job_invocation, :host_ids => [ host.id ])), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason to show the hostname? Don't think we do that anywhere else in Foreman, do we? I think it looks OK, but it'd be weird if there were multiple action items because you already know the host based on the row the button's in.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to be clear about on which host it's going to be run, without following the whole line to the left. But I can easily change it to 'Rerun' only if you prefer. Or maybe 'Rerun on this host'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No strong preference either way, the way it is now is OK
rebased, still ongoing discussion in outdated diff |
APJ |
Fixes #12926 - add rerun to host row in job detail
Add package installation to README
based on #92 to avoid conflicts, please review just the second commit or ideally review the second PR first...