Bug 2228287 - Puppet config group edit page breaks with "undefined method "environment' for #‹ForemanPuppet:ConfigGroup:0x0000555d6a93b90" [NEEDINFO]
Summary: Puppet config group edit page breaks with "undefined method "environment' for...
Keywords:
Status: ON_QA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Puppet
Version: 6.14.0
Hardware: x86_64
OS: Linux
unspecified
high
Target Milestone: 6.14.0
Assignee: Leos Stejskal
QA Contact: Gaurav Talreja
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-08-01 22:19 UTC by Gaurav Talreja
Modified: 2023-08-16 17:45 UTC (History)
3 users (show)

Fixed In Version: foreman_puppet-6.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Target Upstream Version:
Embargoed:
gtalreja: needinfo? (ekohlvan)

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github theforeman foreman_puppet pull 366 0 None open Fix #357 - Undefined method 'environment' for config groups 2023-08-10 07:19:23 UTC
Red Hat Issue Tracker SAT-19306 0 None None None 2023-08-01 22:19:49 UTC

Description Gaurav Talreja 2023-08-01 22:19:17 UTC 
Description of problem:


Version-Release number of selected component (if applicable):
Satellite 6.14.0 Snap 9.0

How reproducible:
Always

Steps to Reproduce:
1. Enable puppet on Satellite 6.14
2. Create config group with some puppet_classes assigned
3. Click on the config group created from all page.

Actual results:
Oops, we're sorry but something went wrong undefined method "environment' for #‹ForemanPuppet:ConfigGroup:0x0000555d6a93b90

Expected results:
ConfigGroup Edit page should be visible

Additional info:
Same works for 6.13, so adding a `Regression` keyword
Comment 5 Leos Stejskal 2023-08-10 07:19:23 UTC 
Fix available here: https://github.com/theforeman/foreman_puppet/pull/366

Can you verify the fix + check that the styling is the same as before?
I did small refactoring, it should behave the same but want to be 100% sure.
Comment 6 Gaurav Talreja 2023-08-10 09:51:10 UTC 
@lstejska Thanks for the quick fix, I've tested the patch above, and it fixes the issue. Also Ack'd the PR.
Note You need to log in before you can comment on or make changes to this bug.