Project

General

Profile

Actions

Bug #13164

closed

view_params is missing

Added by Ori Rabin about 8 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Users, Roles and Permissions
Target version:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1296662
Description of problem:
The view_params permission in the Parameter resource is missing. This prevents us from allowing users to be able to view parameters.

Version-Release number of selected component (if applicable):
6.1.5

How reproducible:
Repeatable:

Actual results:

  1. hammer filter available-permissions --resource-type Parameter
    ---|----------------|----------
    ID | NAME | RESOURCE
    ---|----------------|----------
    67 | create_params | Parameter
    68 | edit_params | Parameter
    69 | destroy_params | Parameter
    ---|----------------|----------

Expected results:

I'd expect to be assign the view_params permission to users (as edit_params is useless without this!).


Related issues 5 (1 open4 closed)

Related to Foreman - Refactor #15496: Simplify all parameter kinds permissionsNew06/22/2016Actions
Related to Foreman - Bug #15599: Unattended controller can't access host parameters in kickstart templatesClosedOri Rabin07/07/2016Actions
Related to Foreman Remote Execution - Bug #15605: Tests failing with latest ForemanClosedIvan Necas07/07/2016Actions
Related to Foreman - Bug #15629: Validation errors for Domain and OS Parameters tab not showingClosedOri Rabin07/08/2016Actions
Related to Foreman - Bug #17395: @host.params are not interpreted when was used in ptable kickstart template.ClosedDominic Cleal11/18/2016Actions
Actions #1

Updated by The Foreman Bot about 8 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3060 added
Actions #2

Updated by Dominic Cleal about 8 years ago

  • Category set to Users, Roles and Permissions
  • Status changed from Ready For Testing to Assigned
  • Priority changed from High to Normal
Actions #3

Updated by Dominic Cleal about 8 years ago

Using Foreman 1.10.2-el7, I noticed that non-admins are unable to access parameters within the "Edit Locations" section when adding all parameter permissions from within the UI.
Reviewing the filter permissions for the Parameters category, it appears the "view_params" permission does not show up which is needed for this feature to be shown to non-admins.

I've added the view_params permission into the permissions table then added the view_params permission to the role and everything worked as expected :)
insert into permissions (name,resource_type,created_at,updated_at) values ('view_params','Parameter',NOW(),NOW());

I'm unsure if this permission has been deprecated and needs to be altered within app/views/taxonomies/_form.html.erb or if we need to alter the seeds.d/03-permissions.rb to include the perm.

from foreman-dev. The use of view_params should probably be removed from this one view or implemented everywhere.

Actions #4

Updated by The Foreman Bot about 8 years ago

  • Status changed from Assigned to Ready For Testing
Actions #5

Updated by The Foreman Bot almost 8 years ago

  • Pull request https://github.com/theforeman/foreman/pull/3592 added
Actions #6

Updated by Marek Hulán over 7 years ago

  • Related to Refactor #15496: Simplify all parameter kinds permissions added
Actions #7

Updated by Dominic Cleal over 7 years ago

  • translation missing: en.field_release set to 160
Actions #8

Updated by Ori Rabin over 7 years ago

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

Updated by Tomer Brisker over 7 years ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/3060)
Actions #10

Updated by The Foreman Bot over 7 years ago

  • Pull request https://github.com/theforeman/foreman/pull/3624 added
Actions #11

Updated by Dominic Cleal over 7 years ago

  • Related to Bug #15599: Unattended controller can't access host parameters in kickstart templates added
Actions #12

Updated by Ivan Necas over 7 years ago

  • Related to Bug #15605: Tests failing with latest Foreman added
Actions #13

Updated by Ivan Necas over 7 years ago

  • Related to Bug #15629: Validation errors for Domain and OS Parameters tab not showing added
Actions #14

Updated by Dominic Cleal over 7 years ago

  • Related to Bug #17395: @host.params are not interpreted when was used in ptable kickstart template. added
Actions

Also available in: Atom PDF