Skip to content

(PA-7608) Update artifactory URL instead nightlies for yum source#787

Merged
skyamgarp merged 1 commit into
mainfrom
PA-7608
Jul 14, 2025
Merged

(PA-7608) Update artifactory URL instead nightlies for yum source#787
skyamgarp merged 1 commit into
mainfrom
PA-7608

Conversation

@skyamgarp

Copy link
Copy Markdown
Contributor

No description provided.

@skyamgarp skyamgarp marked this pull request as ready for review July 14, 2025 08:29
@skyamgarp skyamgarp requested review from a team and bastelfreak as code owners July 14, 2025 08:29

@amitkarsale amitkarsale left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@skyamgarp skyamgarp merged commit f9b4eb6 into main Jul 14, 2025
4 checks passed
@skyamgarp skyamgarp deleted the PA-7608 branch July 14, 2025 08:36
Comment thread tasks/install_shell.sh
yum_source='https://yum-puppetcore.puppet.com/public'
elif [ "$nightly" = true ]; then
yum_source='http://nightlies.puppet.com/yum'
yum_source='https://artifactory.delivery.puppetlabs.net:443/artifactory/internal_nightly__local/yum'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think we want to do this globally or at least not until a new major version.

Can't the test/caller pass the parameter for now? Alternatively, we could create a puppetcore8-nightly collection and update the {yum,apt,mac,windows}_source. The way the new behavior is strictly opt-in.

@skyamgarp skyamgarp Jul 16, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Okay we wanted to test if this is accessible with twingate. Do we need to revert this change?

@mhashizume mhashizume added the maintenance Maintenance chores are typically excluded from changelogs label Oct 28, 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.

4 participants