Actions
Bug #26756
closedDrop invalid scsi_controller_type attribute
Difficulty:
trivial
Triaged:
No
Bugzilla link:
Description
In the #26402 we started to pass the attribute instead of ignoring it.
But now if someone had an old hammer command working, with invalid type of the `scsi_controller_type`, the command will fail on new version.
For not breaking old hammer commands, which dropped the attribute altogether, I would like to drop invalid values for this attribute.
Better solution would be validate and give the user meaningful feedback, but it is out of scope and would not reach the same version as the fix.
Updated by Ondřej Ezr over 5 years ago
- Related to Bug #26402: vmware compute-attributes scsi_controller_type not honored added
Updated by The Foreman Bot over 5 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/6744 added
Updated by Ondřej Ezr over 5 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset a317b6ebbd1f917fd136bf8a334bc7494805918c.
Updated by The Foreman Bot over 5 years ago
- Pull request https://github.com/theforeman/foreman/pull/6800 added
Updated by Tomer Brisker over 5 years ago
- Fixed in Releases 1.22.0 added
- Fixed in Releases deleted (
1.23.0)
Actions