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-36494,PE-37833) Standardize pe-installer-runtime on shared format with more deps #806

Merged

Conversation

nmburgan
Copy link
Contributor

This standardizes the pe-installer-runtime definition to the format used by the other runtimes, using a shared definition for common things between 2021.7.x and main. It also expands the list of components so we are providing all dependencies to Bolt and Puppet. This is so that we don't get surprised when pe-installer-vanagon gem installs Bolt and pulls in dependencies that may break on the version of Ruby we're using.

…e deps

This standardizes the pe-installer-runtime definition to the format used by the other runtimes, using a shared definition for common things between 2021.7.x and main. It also expands the list of components so we are providing all dependencies to Bolt and Puppet. This is so that we don't get surprised when pe-installer-vanagon gem installs Bolt and pulls in dependencies that may break on the version of Ruby we're using.
@nmburgan nmburgan requested review from a team as code owners March 22, 2024 19:33
@nmburgan nmburgan added the blocked PRs blocked on work external to the PR itself label Mar 22, 2024
Copy link
Contributor

@donoghuc donoghuc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want to just manage bolt as a gem here too? Unless i'm missing something I dont think you would ever need to do a gem install bolt for installer.

@nmburgan
Copy link
Contributor Author

We have machinery for updating the bolt version in pe-installer-vanagon on-demand, I think so we can take up new versions more lazily. But I'm not sure we necessarily have to keep doing it that way. Might do that as a follow up. This at least gets us past https://perforce.atlassian.net/browse/PE-37833.

@nmburgan nmburgan changed the title (PE-36494) Standardize pe-installer-runtime on shared format with more deps (PE-36494,PE-37833) Standardize pe-installer-runtime on shared format with more deps Mar 22, 2024
@nmburgan nmburgan removed the blocked PRs blocked on work external to the PR itself label Mar 22, 2024
@nmburgan
Copy link
Contributor Author

Tested on 2021.7.x, looks all good.

1 similar comment
@nmburgan
Copy link
Contributor Author

Tested on 2021.7.x, looks all good.

@nmburgan nmburgan merged commit 3eedb9e into puppetlabs:master Mar 25, 2024
3 checks passed
@joshcooper joshcooper added the skip changelog Will be excluded from changelog label Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip changelog Will be excluded from changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants