Skip to content

(PA-8445) Pull released Facter from Artifactory#43

Merged
joshcooper merged 1 commit into
mainfrom
PA-8445/main/use-puppetcore-facter
May 27, 2026
Merged

(PA-8445) Pull released Facter from Artifactory#43
joshcooper merged 1 commit into
mainfrom
PA-8445/main/use-puppetcore-facter

Conversation

@mhashizume
Copy link
Copy Markdown
Contributor

Summary

Adds --source <artifactory-rubygems-mirror> to gem install puppet.gem in the shared unit_tests_with_nightly_puppet_gem.yaml workflow so Puppet's Facter runtime dependency is resolved from the Artifactory proxy of Puppet Core (latest released Facter) instead of public rubygems.org (stuck on Facter 4.10.0).

Why

Without the source flag, RubyGems falls back to public rubygems.org for transitive deps. The latest Facter on public rubygems is 4.10.0, several versions behind the current released Facter on Puppet Core. The Artifactory mirror at artifactory.delivery.puppetlabs.net/artifactory/api/gems/rubygems/ is the same URL the next bundler step already uses for GEM_SOURCE, so the source choice is consistent across the job.

Jira: PA-8445

Test plan

  • yamllint passes on the workflow file
  • Downstream module CI consuming this reusable workflow installs Facter from the Artifactory mirror at a version > 4.10.0

🤖 Generated with Claude Code

The shared GitHub Action `unit_tests_with_nightly_puppet_gem.yaml` runs
`gem install puppet.gem -N` to install a nightly Puppet build. Because
no gem source was passed to that command, RubyGems resolved Puppet's
Facter dependency against public rubygems.org, which still serves the
last publicly-released Facter (4.10.0). The current released Facter
gems live on Puppet Core, several versions ahead.

Add `--source <artifactory-rubygems-mirror>` to that `gem install`. The
mirror is the same URL the very next step already uses for `GEM_SOURCE`
and is an Artifactory proxy for Puppet Core, so Facter resolves from
there at the latest released version. Twingate is already connected
earlier in the workflow, so no extra credentials are required.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mhashizume mhashizume requested a review from a team as a code owner May 27, 2026 18:20
@joshcooper joshcooper merged commit b03a4eb into main May 27, 2026
2 checks passed
@mhashizume mhashizume deleted the PA-8445/main/use-puppetcore-facter branch May 27, 2026 20:52
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