Bug #29720
closed
NoMethodError: undefined method `>’ for nil:NilClass when promoting to new environment after upgrade to Katello 3.15
Description
When puppet is set to false under content_types in katello.yaml, the content_types field on a content view version does not get a puppet_modules key. This causes the above error in the promote_puppet_environment? method in app/models/katello/content_view_version.rb.
Need to update to account for this.
Updated by Partha Aji over 4 years ago
The code here should handle case where PuppetModule::CONTENT_TYPE is not set
https://github.com/Katello/katello/blob/master/app/models/katello/content_view_version.rb#L164
def promote_puppet_environment?
(!content_counts.blank? && content_counts.dig(PuppetModule::CONTENT_TYPE) > 0) || self.content_view.force_puppet_environment?
end
Updated by James Jeffers over 4 years ago
- Target version set to Katello 3.16.0
- Triaged changed from No to Yes
Updated by Samir Jha over 4 years ago
This has been fixed by this change here by adding the puppet module key explicitly.
: https://github.com/Katello/katello/pull/8702/files#diff-806e40dd93ec377d04d685b9fc99ba73R317
Updated by Samir Jha over 4 years ago
- Is duplicate of Bug #29448: Skip puppet env import on cv publish/promote if smart proxy feature is not present added
Updated by Samir Jha over 4 years ago
- Status changed from New to Duplicate
- Target version changed from Katello 3.16.0 to Katello Recycle Bin
- Triaged changed from Yes to No
Untriaging to consider for 3.15 release..
Updated by Samir Jha over 4 years ago
- Status changed from Duplicate to New
- Target version deleted (
Katello Recycle Bin)
Updated by Jonathon Turel over 4 years ago
- Status changed from New to Rejected
- Target version set to Katello Recycle Bin
- Triaged changed from No to Yes
This is resolved in 3.16 and there is a workaround provided in the above comments - so closing this out for 3.15