Skip to content

CI: Dont output debug infos twice#177

Merged
bastelfreak merged 2 commits intoOpenVoxProject:mainfrom
bastelfreak:parallellllllllllllll
Aug 24, 2025
Merged

CI: Dont output debug infos twice#177
bastelfreak merged 2 commits intoOpenVoxProject:mainfrom
bastelfreak:parallellllllllllllll

Conversation

@bastelfreak
Copy link
Contributor

At a previous step, we already run bundle env, we don't need to duplicate the information.

@bastelfreak bastelfreak added the skip-changelog Excluded from CHANGELOG label Aug 24, 2025
@bastelfreak bastelfreak force-pushed the parallellllllllllllll branch from fbd8dce to 77c8312 Compare August 24, 2025 13:11
sebastianrakel and others added 2 commits August 24, 2025 16:38
A bit of backstory. I did a lot of JRuby debugging in OpenVoxProject#122. Yesterday @sebastianrakel and I did a bit of deep diving. Some numbers:

* puppetserver 8.7.0, the last open source release from Perforce, ships JRuby 9.4.8.0
* openvox-server 8.9.0 ships JRuby 9.4.8.0
* puppet enterprise 2025.4.0 (latest version at 2025-07-27) ships pe-puppetserver JRuby 9.4.8.0
* latest JRuby 9.4 is 9.4.13.0
* JRuby 9.4 series is compatible with MRI Ruby 3.1
* puppetserver 7 uses JRuby 9.3 which is compatible with Ruby 2.6
* Perforce never tested JRuby 9.4.8.0 in CI, JRuby was pinned to 9.4.3.0

Since openvox runs tests, the Rspec JRuby pipeline fails. We cannot access the old perforce pipeline logs. Their last successful CI run I could fine was https://github.com/puppetlabs/puppet/actions/runs/11693993097. This was triggered by commit puppetlabs/puppet@4ea6ee2 on 2024-11-06.

In https://github.com/puppetlabs/puppet/actions/runs/12021455668 we see the first failed pipelline for puppetserver 8, triggered by  puppetlabs/puppet@e967115 on 2024-11-24. The used JRuby verison in both jobs was 9.4.3.0

Something changed between 2024-11-06 and 2024-11-24 and I doubt it was related to puppetlabs/puppet@e967115 (puppetlabs/puppet#9523).

Whatever changed in that timeframe modified the error reporting. Basic example:

```
class a {}
class a {}
```

raises:

```
Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Class 'a' is already defined (file: /etc/puppetlabs/code/environments/server/manifests/site.pp, line: 1); cannot redefine (file: /etc/puppetlabs/code/environments/server/manifests/site.pp, line: 2) (file: /etc/puppetlabs/code/environments/server/manifests/site.pp, line: 2) on node puppettestinfra.voxpupu.li
```

After "cannot redefine", the file is listed twice. This issue is reproducible puppetserver 8.7.0, Puppet Enterprise 2025.4.0 & openvox-server 8.9.0.

After some testing in OpenVoxProject#122 I noticed that JRuby 9.4.3.0 indeed raises errors now, but others. I assume the JRuby issues are, at least in part, triggered by a webmock update. I don't know yet what causes the duplicate file lines in the output. The old rspec tests demand a single file line in the output.

In OpenVoxProject#164 I'm testing different Jruby Versions with a webmock pinned to 3.25.0. 3.25.0 was released on 2025-02-06. The version before that, 3.24.0, on 2024-09-30. That's what Perforce was using back then.

And after diffing CI output from JRuby 9.4.3, 9.4.4, 9.4.5 & 9.4.6 we can see that the duplicate line messages start with JRuby 9.4.6: https://gist.github.com/bastelfreak/025c23417f9dd7215b5c5af205fc8b76
At a previous step, we already run `bundle env`, we don't need to
duplicate the information.
@bastelfreak bastelfreak force-pushed the parallellllllllllllll branch from 77c8312 to 201b1cd Compare August 24, 2025 14:54
@bastelfreak bastelfreak enabled auto-merge August 24, 2025 15:06
@bastelfreak bastelfreak merged commit 2b9578a into OpenVoxProject:main Aug 24, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Excluded from CHANGELOG

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants