Bug #29065
closedHammer filter create fails if default location is set.
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1792339
Description of problem:
"hammer role create" now assigns "Default Location" by default. It did not do that previously.
Attempting to create a filter for a role fails with error:
Could not create the permission filter:
Location ids You can't assign locations to this resource
How reproducible:
every time
Steps to Reproduce:
[root@dhcp-3-165 ~]# hammer role create --name test
~]# hammer filter create --permissions="view_activation_keys,create_activation_keys,edit_activation_keys,destroy_activation_keys" --role-id=32
Could not create the permission filter:
Location ids You can't assign locations to this resource
[root@dhcp-3-165 ~]# hammer role info --name test
Id: 32
Name: test
Builtin: no
Description:
Locations:
Default Location
[root@dhcp-3-165 ~]# hammer role create --name test2
User role [test2] created.
[root@dhcp-3-165 ~]# hammer role info --name test2
Id: 33
Name: test2
Builtin: no
Description:
Locations:
Default Location
Actual results:
Could not create the permission filter
Expected results:
hammer filter can be created
Updated by Tomer Brisker almost 5 years ago
- Subject changed from Hammer filter create fails if default location is set. to Hammer filter create fails if default location is set.
- Category set to Organizations and Locations
Updated by Tomer Brisker almost 5 years ago
- Related to Bug #28272: can't create a filter for a role with view_organizations or view_locations permissions via API/hammer added
Updated by Tomer Brisker almost 5 years ago
- Related to Bug #29056: Hammer filter create fails if default organization is set. added
Updated by Shira Maximov almost 5 years ago
- Status changed from New to Closed
The bug is no longer reproducible.
Updated by Tomer Brisker over 4 years ago
- Status changed from Closed to Resolved
Updated by The Foreman Bot over 4 years ago
- Status changed from New to Ready For Testing
- Assignee set to Ondřej Pražák
- Pull request https://github.com/theforeman/foreman/pull/7493 added
Updated by The Foreman Bot over 4 years ago
- Pull request https://github.com/theforeman/foreman/pull/7494 added
Updated by Oleh Fedorenko over 4 years ago
- Pull request deleted (
https://github.com/theforeman/foreman/pull/7493)
Updated by Ondřej Pražák over 4 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset cce34197464941f38ea131d2715d012dfa9f58a7.
Updated by Tomer Brisker over 4 years ago
- Fixed in Releases 1.24.3, 2.0.0 added
- Fixed in Releases deleted (
2.1.0)