Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix resource generation ipcidr dependency #1204

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

2fa
Copy link
Contributor

@2fa 2fa commented Apr 2, 2024

Summary

This change should fix partial issue with resource generation:

Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Could not autoload puppet/type/firewallchain: Could not autoload puppet/provider/firewallchain/firewallchain: no such file to load -- puppet_x/puppetlabs/firewall/ipcidr (file: /etc/puppetlabs/code/puppet-control/dev/profile_modules/profile/manifests/example/sample.pp, line: 40, column: 3) on node host.example.com
Warning: Not using cache on failed catalog
Error: Could not retrieve catalog; skipping run

You still need to do manual steps after upgrading from the previous version:

  1. Find and delete firewall-related cache
    find $(puppet config print vardir) -ipath '*firewall*' -type f -delete
  2. Regenerate resource types
    puppet generate types --environment <environment>

Additional Context

As far as i understand puppet tries to load providers and types from the previous version that he keeps in his cache. This problem especially tricky if you have multiple environments and are trying to test new version in a non-default one. I was forced to create separate puppetserver instance to properly test all the changes.

Related Issues (if any)

Related to #1162

@2fa 2fa requested a review from a team as a code owner April 2, 2024 18:55
@CLAassistant
Copy link

CLAassistant commented Apr 2, 2024

CLA assistant check
All committers have signed the CLA.

@Ramesh7 Ramesh7 merged commit c5a7e58 into puppetlabs:main Jul 19, 2024
41 of 42 checks passed
@Ramesh7 Ramesh7 added the bugfix label Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants