Project

General

Profile

Bug #27888

hammer job-invocation info unclear on whether the Randomized ordering is enabled

Added by Peter Ondrejka almost 3 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Difficulty:
Triaged:
No
Bugzilla link:

Description

Introduced in https://github.com/theforeman/hammer_cli_foreman_remote_execution/pull/24/files

If randomized is enabled, the job-invocation info shows 'Randomized ordering: true', which is fine, but if it is disabled no value is displayed 'Randomized ordering: ' which is confusing.

Expected behavior: 'false' should be displayed if the option is off. Or alternatively, we could change the option to match UI: 'Execution order: randomized' / 'Execution order: alphabetical', which is clearer about what is the alternative option and also more extensible in case we decide to implement another order type

Associated revisions

Revision e8daad1b (diff)
Added by Peter Koprda over 1 year ago

Fixes #27888 - Add false value for disabled Randomized ordering (#39)

History

#1 Updated by The Foreman Bot over 1 year ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/hammer_cli_foreman_remote_execution/pull/39 added

#2 Updated by The Foreman Bot over 1 year ago

  • Fixed in Releases hammer_cli_foreman_remote_execution 0.2.2 added

#3 Updated by Adam Ruzicka over 1 year ago

  • Target version set to hammer_cli_foreman_remote_execution 0.2.2
  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF