Skip to content

[chore][CI] Remove EOL distros from test matrices #6073

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 2 commits into from
Apr 1, 2025

Conversation

pjanotti
Copy link
Contributor

Both OpenSUSE 12 (leap 42) and CentOS 7 reached EOL. Removing them from the test matrices.

@pjanotti pjanotti requested review from a team as code owners March 31, 2025 20:40
@pjanotti pjanotti marked this pull request as draft March 31, 2025 20:43
Copy link
Contributor

@crobert-1 crobert-1 left a comment

Choose a reason for hiding this comment

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

We may want to consider adding a breaking changelog to notify users. (We added one the last time we dropped testing for an EOL distro)

@pjanotti
Copy link
Contributor Author

Interesting, it seems that an initial round of failures was due to the new release being published concurrently https://github.com/signalfx/splunk-otel-collector/actions/runs/14181034561/attempts/1?pr=6073

@pjanotti pjanotti marked this pull request as ready for review March 31, 2025 21:30
@pjanotti pjanotti requested a review from a team as a code owner March 31, 2025 21:30
@pjanotti
Copy link
Contributor Author

puppet-rake-spec failure is a known issue.

Copy link
Contributor

@jinja2 jinja2 left a comment

Choose a reason for hiding this comment

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

It looks like we have a copy of the readme content on the puppetforge page for our module. Should we be releasing a new version to push latest readme for the same? I am not sure how the docs are synced.

@pjanotti
Copy link
Contributor Author

pjanotti commented Apr 1, 2025

I think the readme for all mass deployment tools is only updated when we make a new release for those (which are not tied to the collector release). That said it is time to make a new release for those.

@pjanotti
Copy link
Contributor Author

pjanotti commented Apr 1, 2025

@jinja2 yes, after this one is merged we should do a release for Ansible and Chef. There are no changes to be published for Puppet so no new release is needed and the version that was published won't be affected by the deprecations here. I will create a separate PR for that but this one should be merged first.

@pjanotti pjanotti merged commit e56e25d into signalfx:main Apr 1, 2025
320 of 321 checks passed
@pjanotti pjanotti deleted the remote-eol-distros-from-ci branch April 1, 2025 19:39
@github-actions github-actions bot locked and limited conversation to collaborators Apr 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants