Bug #23331

Permission edit_discovered_hosts interfere with edit_hosts

Added by Lukas Zapletal 3 months ago. Updated 10 days ago.

Status:Closed
Priority:Normal
Assignee:Lukas Zapletal
Category:Authorization
Target version:1.19.0
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link: Found in Releases:
Pull request:https://github.com/theforeman/foreman/pull/5468

Description

Method Authorizable#permission_name does not work with edit_discovered_hosts and will break core tests, because it expects to find "edit_hosts" while it pulls "edit_discovered_hosts". This fails randomly because it depends on SQL result order which is random.

Associated revisions

Revision 401a54db
Added by Lukas Zapletal 3 months ago

Fixes #23331 - grouped permissions for host disabled (#5468)

History

#1 Updated by The Foreman Bot 3 months ago

  • Assignee set to Lukas Zapletal
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_discovery/pull/428 added

#2 Updated by Lukas Zapletal 3 months ago

  • Category changed from Discovery plugin to Authorization
  • Project changed from Discovery to Foreman

#3 Updated by The Foreman Bot 3 months ago

  • Pull request https://github.com/theforeman/foreman/pull/5468 added

#4 Updated by Marek Hulán 3 months ago

  • Legacy Backlogs Release (now unused) set to 353
  • Pull request deleted (https://github.com/theforeman/foreman_discovery/pull/428)

#5 Updated by Lukas Zapletal 3 months ago

  • % Done changed from 0 to 100
  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF