Project

General

Profile

Actions

Bug #36588

closed

hammer settings set returns incomplete message for False values

Added by Oleh Fedorenko 11 months ago. Updated 10 months ago.

Status:
Closed
Priority:
Normal
Category:
Other commands
Target version:
-
Difficulty:
Triaged:
Yes
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=2221673

Description of problem:
hammer settings set returns incomplete message when setting some boolean with False value.

Version-Release number of selected component (if applicable):
6.14.0 snap 6

How reproducible:
always

Steps to Reproduce:
1. Try to set some (no matter which) boolean setting to False:
  1. hammer settings set --name db_pending_seed --value 0

Actual results:
Setting [db_pending_seed] updated to [].

Expected results:
Setting [db_pending_seed] updated to [false].

Additional info:
Works fine for True values
  1. hammer settings set --name db_pending_seed --value False
    Setting [db_pending_seed] updated to [].
  2. hammer settings set --name db_pending_seed --value false
    Setting [db_pending_seed] updated to [].
  3. hammer settings set --name db_pending_seed --value 0
    Setting [db_pending_seed] updated to [].
  1. hammer settings set --name db_pending_seed --value True
    Setting [db_pending_seed] updated to [true].
  2. hammer settings set --name db_pending_seed --value true
    Setting [db_pending_seed] updated to [true].
  3. hammer settings set --name db_pending_seed --value 1
    Setting [db_pending_seed] updated to [true].
Actions #1

Updated by Oleh Fedorenko 11 months ago

  • Status changed from New to Assigned
  • Assignee set to Oleh Fedorenko
  • Triaged changed from No to Yes
Actions #2

Updated by The Foreman Bot 11 months ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli/pull/371 added
Actions #3

Updated by The Foreman Bot 10 months ago

  • Fixed in Releases hammer-cli-3.8.0 added
Actions #4

Updated by Oleh Fedorenko 10 months ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF