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

Use --force with puppet generate types #2013

Merged
merged 1 commit into from
Jan 10, 2024
Merged

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Jan 10, 2024

When installing a new module it maintains the file timestamps from the original tarball. puppet generate types uses the timestamp of when it ran.

If you update a module to a release that was created before it ran puppet types generate, it will never update the types. This can happen with a module downgrade, or just when you regenerated types manually. By using --force the timestamps are ignored.

When installing a new module it maintains the file timestamps from the
original tarball. puppet generate types uses the timestamp of when it
ran.

If you update a module to a release that was created before it ran
puppet types generate, it will never update the types. This can happen
with a module downgrade, or just when you regenerated types manually. By
using --force the timestamps are ignored.
@evgeni
Copy link
Member

evgeni commented Jan 10, 2024

I could swear I had already submitted such a PR with exactly that explanation 🙈

@ekohl
Copy link
Member Author

ekohl commented Jan 10, 2024

puppetlabs/puppet#9206 would be a proper fix in Puppet, but until we can rely on that I think this is better.

@ekohl
Copy link
Member Author

ekohl commented Jan 10, 2024

I do recall you complaining about it, but I didn't see any PR.

@ekohl ekohl merged commit 47d0ab5 into theforeman:master Jan 10, 2024
2 checks passed
@ekohl ekohl deleted the use-force branch January 10, 2024 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants