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.

Sign up for GitHub

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 #23310 - Display formatted message for reboot api #41

Closed
wants to merge 1 commit into from
Closed

Fixes #23310 - Display formatted message for reboot api #41

wants to merge 1 commit into from

Conversation

rabajaj0509
Copy link
Member

No description provided.

@theforeman-bot
Copy link
Member

Issues: #23310

@lzap
Copy link
Member

lzap commented Apr 19, 2018

[test]

@rabajaj0509
Copy link
Member Author

closing the pr and re-opening to trigger the test process.

@rabajaj0509 rabajaj0509 reopened this May 25, 2018
@lzap
Copy link
Member

lzap commented Jul 31, 2019

@rahulbajaj0509 can you rebase please?

@lzap
Copy link
Member

lzap commented Jul 31, 2019 •
edited

I am getting error for this one when reboot reports errors:

[lzap@box hammer-cli]$ hammer discovery reboot --all
Could not reboot the host:
  Error: undefined method `map' for nil:NilClass

Edit: However the error appears with or without the patch. Anyway, can you fix that? Please add a test for both cases (no errors + errors).

@stejskalleos
Copy link
Contributor

Closing the PR, it's been four years since the last update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Needs testing Not yet reviewed
Projects
None yet
Milestone
No milestone
4 participants
@rabajaj0509 @theforeman-bot @lzap @stejskalleos