Skip to content

Fix breaking changes in non-core repos #762

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

Merged
merged 1 commit into from
Mar 3, 2025

Conversation

joshcooper
Copy link
Contributor

@joshcooper joshcooper commented Mar 3, 2025

Commit e6f817c introduced a breaking change when downloading the GPG key from non-core repos on SLES.

It also incorrectly attempted to manage credentials for non-core repos for all rpm-based platforms.

This commit restores the GPG location on yum.puppet.com and guards the sed commands to only run when the repo collection contains "core"

This should fix https://jenkins-platform.delivery.puppetlabs.net/view/puppet-agent/view/Acceptance%20Suites/view/CI%20Goalie/job/forge-module_puppetlabs-puppet-agent-module_intn-run_task_acceptance-main/591/

Commit e6f817c introduced a breaking
change when downloading the GPG key from non-core repos on SLES.

It also incorrectly attempted to manage credentials for non-core repos
for all rpm-based platforms.

This commit restores the GPG location on yum.puppet.com and guards the
sed commands to only run when the repo collection contains "core"
@joshcooper joshcooper requested review from bastelfreak and a team as code owners March 3, 2025 17:53
Copy link
Collaborator

@bastelfreak bastelfreak left a comment

Choose a reason for hiding this comment

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

@mhashizume mhashizume merged commit 0f71bcf into puppetlabs:main Mar 3, 2025
16 checks passed
@mhashizume mhashizume added the maintenance Maintenance chores are typically excluded from changelogs label Mar 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Maintenance chores are typically excluded from changelogs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants