# voxpupuli
- n
natemccurdy
04/25/2023, 8:37 PM@Craig Gumbley What in there specifically? - c
Craig Gumbley
04/25/2023, 8:37 PMWhoops - c
- c
Craig Gumbley
04/25/2023, 8:39 PMAlthough Tim might have already done that.. in the PR - s
Slackbot
04/25/2023, 8:39 PMThis message was deleted.vnb- 4
- 3
- n
natemccurdy
04/25/2023, 8:40 PM
But it's pulling in 7.21.0.PUPPET_GEM_VERSION: "~> 7.0" - n
natemccurdy
04/25/2023, 8:40 PMInstead of 7.23.0 - b
bastelfreak
04/25/2023, 8:40 PMthose are different jobs - b
bastelfreak
04/25/2023, 8:41 PMI updated the
job to pull in puppetsetup_matrix
. Previously we only defined the puppet version in the Gemfile as:7.xCopy codepuppetversion = ENV['PUPPET_GEM_VERSION'] || '>= 6.0' gem 'puppet', puppetversion, :require => false, :groups => [:test] - b
bastelfreak
04/25/2023, 8:41 PMnow what's failing are the unit tests for the
job: https://github.com/voxpupuli/gha-puppet/pull/30/files#diff-a5f1ef1c4e2526c1092b3d50254581c35462653f2b0f467628f6c36aeea60becL80unit - b
bastelfreak
04/25/2023, 8:42 PMbecause it doesn't pull in the latest puppet 7 but 7.21, which allows a newer ruby-concurrent, which doesn't work well with the used ruby/puppet version. but that worked in the past (like 3 hours ago) - c
Craig Gumbley
04/25/2023, 8:44 PMI’ll pull your branch in a sec - b
bastelfreak
04/25/2023, 8:46 PM - c
Craig Gumbley
04/25/2023, 8:57 PM@bastelfreak do you mind if i clear the workflow cache and re-kick puppet-example/32? - b
bastelfreak
04/25/2023, 8:57 PMna, just go on - c
Craig Gumbley
04/25/2023, 8:59 PMI’m hoping it’s just some bundler magic.. because it appears to pull 7.24 locally in a couple of test envs - b
bastelfreak
04/25/2023, 9:00 PMmhm it doesn't make sense. why does it fail now - c
Craig Gumbley
04/25/2023, 9:02 PM - c
Craig Gumbley
04/25/2023, 9:02 PMthinking party - b
bastelfreak
04/25/2023, 9:02 PMI can also reproduce that locally - b
bastelfreak
04/25/2023, 9:03 PMin theory, direct deps (as in declared in a gemfile) have high prio over transitive deps - b
bastelfreak
04/25/2023, 9:03 PMand we declare puppet, not ruby-concurrent. so puppet should have a higher prio and we should get 7.24 - b
bastelfreak
04/25/2023, 9:03 PMand that worked until a few hours ago - b
bastelfreak
04/25/2023, 9:04 PMwe could add ruby-concurreny to our gemfile and pin < 1.2 but that requires a modulesync - b
bastelfreak
04/25/2023, 9:05 PMI could hack puppet_metadata so we depend on puppet ~> 7.24 - c
Craig Gumbley
04/25/2023, 9:07 PMhmm - n
natemccurdy
04/25/2023, 9:07 PM
was the only way I found to get something higher than~> 7.22
and below7.21
.8 - b
bastelfreak
04/25/2023, 9:07 PMbut I would like to understand why it fails - b
bastelfreak
04/25/2023, 9:08 PMit feels like a bundler bug - c
Craig Gumbley
04/25/2023, 9:08 PMwhat version of bundler is everyone running locally (and what OS)