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

(PE-37376) Initialize one JRuby instance first when filling a pool #118

Merged
merged 3 commits into from
Jun 3, 2024

Conversation

justinstoller
Copy link
Member

Puppet may apply a "settings catalog" when it initializes. This will check the filesystem and create any directories specified by settings that do not already exist. This will cause errors if Puppet does need to create directories and is initialized in parallel. To resolve this when priming the pool we now instantiate one instance first before initializing the remaining instances in parallel.

@justinstoller justinstoller requested review from a team as code owners May 30, 2024 16:46
Puppet may apply a "settings catalog" when it initializes. This will
check the filesystem and create any directories specified by settings
that do not already exist. This will cause errors if Puppet does need to
create directories and is initialized in parallel. To resolve this when
priming the pool we now instantiate one instance first before
initializing the remaining instances in parallel.
@justinstoller justinstoller force-pushed the parallelize-creation branch from a7b1f7b to ef6a947 Compare June 3, 2024 16:37
@justinstoller justinstoller changed the title (PE-37376) Initialize one JRuby instance first when initializing pool (PE-37376) Initialize one JRuby instance first when filling a pool Jun 3, 2024
@mcdonaldseanp mcdonaldseanp merged commit 44a8af7 into puppetlabs:main Jun 3, 2024
5 checks passed
@justinstoller justinstoller deleted the parallelize-creation branch June 3, 2024 17: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