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

Install: Ensure package installed before file resources #422

Merged
merged 1 commit into from
Sep 4, 2024

Conversation

sagepe
Copy link
Contributor

@sagepe sagepe commented Sep 3, 2024

This sets a require on the package for all file resources in the install class, as sometimes Puppet tries to create these before it's installed. It also chains the DB schema copy exec to the necessary directory.

Fixes #421

This sets a require on the package for all file resources in the install
class, as sometimes Puppet tries to create these before it's installed.
It also chains the DB schema copy exec to the necessary directory.
@lbetz
Copy link
Contributor

lbetz commented Sep 4, 2024

Strange. It hasn't caused any problems for years. Have you changed the order of “processing” on the Puppet server?
But your changes don't hurt either.

@lbetz lbetz added the bug label Sep 4, 2024
@lbetz lbetz added this to the v5.0.1 milestone Sep 4, 2024
@lbetz lbetz merged commit 0c5e2ed into voxpupuli:main Sep 4, 2024
5 checks passed
@sagepe
Copy link
Contributor Author

sagepe commented Sep 4, 2024

Have you changed the order of “processing” on the Puppet server?

I don't think so, although it's quite a large and old codebase, so it's not impossible there's something non-obvious tucked away somewhere. Having said that we do set Apt::Source <| |> -> Package <| |>, which might have some unintended side effect that has resulted in this.

@lbetz
Copy link
Contributor

lbetz commented Sep 4, 2024

No, you didn't say that, but I also don't see any connection with the problem that is occurring.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

First run on new server fails due to ordering of package and file resources
2 participants