Skip to content

Remove Solaris From Vagrant Checker#4238

Merged
karianna merged 2 commits intoadoptium:masterfrom
karianna:remove_solaris
Feb 17, 2026
Merged

Remove Solaris From Vagrant Checker#4238
karianna merged 2 commits intoadoptium:masterfrom
karianna:remove_solaris

Conversation

@karianna
Copy link
Contributor

@karianna karianna commented Feb 4, 2026

We no longer build Solaris

Copy link
Contributor

@steelhead31 steelhead31 left a comment

Choose a reason for hiding this comment

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

Looks good.

Copy link
Member

@sxa sxa left a comment

Choose a reason for hiding this comment

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

I'm neutral on this - the code is still usable if someone outside adoptium wishes to use it, even if we won't actively maintain it. Noting that unless this is removed from the jenkins job too this will likely break our VPC job. Perhaps there should be a corresponding linked issue to cover all actions required here? ;-)

I'm not sure why the markdown has been shoehorned into this PR though - I feel that should be separate from the Solaris changes especially since they are not mentioned in the PR/commit title.

@karianna
Copy link
Contributor Author

karianna commented Feb 16, 2026

I'm neutral on this - the code is still usable if someone outside adoptium wishes to use it, even if we won't actively maintain it. Noting that unless this is removed from the jenkins job too this will likely break our VPC job. Perhaps there should be a corresponding linked issue to cover all actions required here? ;-)

I'm not sure why the markdown has been shoehorned into this PR though - I feel that should be separate from the Solaris changes especially since they are not mentioned in the PR/commit title.

I've only made the Vagrant Checker change as that was a GH Action triggered check on PR submissions (trying to keep that green). I agree there's a larger/further issue WRT to removing Solaris everywhere.

Have removed Markdown, will submit that separately.

@karianna
Copy link
Contributor Author

#4259 Now has the bot

@sxa
Copy link
Member

sxa commented Feb 16, 2026

I've only made the Vagrant Checker change as that was a GH Action triggered check on PR submissions (trying to keep that green). I agree there's a larger/further issue WRT to removing Solaris everywhere.

vagrantPlayBookcheck.sh is not exclusive to GH actions. It was originally written (and is still run and is green) via https://ci.adoptium.net/job/VagrantPlaybookCheck/ - having said that I believe @steelhead31 was prevented from doing some updates on the host machine due to still supporting this particular VM :-)

@karianna
Copy link
Contributor Author

@steelhead31 - Thoughts on this, do you prefer seeing the removal to unblock you or?

@steelhead31
Copy link
Contributor

I'd be happy to have Solaris removed from the VPC, on my ever growing to do list, is a plan to update/migrate the Jenkins based VPC jobs, Im just waiting on the destiny of its current host. If we lose the host, then I'll likely rework some parts of this, and losing solaris even in the short term means I can look at bumping the pinned versions of Virtual Box, Vagrant, Python & Ansible, as they all had to be held back to support Solaris.

@karianna karianna requested a review from sxa February 17, 2026 09:19
Copy link
Member

@sxa sxa left a comment

Choose a reason for hiding this comment

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

Based on Scott's comment and the intention to remove the ability to run it that will occur on the upgrade I'm ok with this.

@karianna karianna merged commit d0a4470 into adoptium:master Feb 17, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants