Skip to content
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

drop sys-v leftovers; assume systemctl is available for unknown platforms #1868

Merged
merged 2 commits into from
May 11, 2024

Conversation

bastelfreak
Copy link
Member

@bastelfreak bastelfreak commented May 10, 2024

This also drops support for linux systems without systemd, for example EL6. But we don't support those anymore since a long time.

@bastelfreak bastelfreak force-pushed the systemd branch 4 times, most recently from 7ba1253 to 7713a1e Compare May 10, 2024 13:53
@bastelfreak bastelfreak changed the title Assume systemctl as default for restarting sshd.service drop sys-v leftovers; assume systemctl is available for unknown platforms May 10, 2024
@bastelfreak bastelfreak force-pushed the systemd branch 4 times, most recently from 0db7b9a to cb575da Compare May 10, 2024 14:20
@bastelfreak bastelfreak force-pushed the systemd branch 4 times, most recently from d325cce to 97a4a43 Compare May 10, 2024 14:28
Copy link

codecov bot commented May 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.29%. Comparing base (2a89cfc) to head (be79965).
Report is 27 commits behind head on master.

❗ Current head be79965 differs from pull request most recent head a9e0bff. Consider uploading reports for the commit a9e0bff to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1868      +/-   ##
==========================================
- Coverage   74.62%   74.29%   -0.33%     
==========================================
  Files          81       79       -2     
  Lines        4492     4396      -96     
==========================================
- Hits         3352     3266      -86     
+ Misses       1140     1130      -10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

I'd prefer to first drop network devices and then revisit this.

spec/helpers.rb Outdated Show resolved Hide resolved
@bastelfreak
Copy link
Member Author

will wait with merging until EOS/cisco are dropped

@joshcooper
Copy link
Contributor

joshcooper commented May 10, 2024

We still support EL6 in puppet 7, though have plans to drop it soonish.

RHEL 7 is offering extended support until June 30, 2028, so we don't have plans on dropping it.

We also support SLES 11.

@bastelfreak bastelfreak force-pushed the systemd branch 5 times, most recently from 9a8a474 to c35497b Compare May 10, 2024 16:06
lib/beaker/host/unix/exec.rb Outdated Show resolved Hide resolved
@bastelfreak bastelfreak force-pushed the systemd branch 2 times, most recently from f008145 to be79965 Compare May 10, 2024 16:28
This also drops support for linux systems without systemd, for example
EL6. But we don't support those anymore since a long time.
@bastelfreak bastelfreak merged commit 88954b4 into voxpupuli:master May 11, 2024
7 checks passed
@bastelfreak bastelfreak deleted the systemd branch May 11, 2024 14:32
Comment on lines -287 to -288
when /opensuse|sles/
exec(Beaker::Command.new("/usr/sbin/rcsshd restart"))
Copy link
Member

Choose a reason for hiding this comment

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

I think @joshcooper wanted to say that SLES 11 still hit this code. IMHO we should treat it like EL 6: special case it and less SLES 12+ fall through to systemd.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, I'm good with defaulting to systemd and special casing older platforms. Can you restore the ability to start services on SLES 11 using /usr/sbin/rcsshd restart?

Copy link
Member Author

@bastelfreak bastelfreak May 15, 2024

Choose a reason for hiding this comment

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

#1883

but the question is the same as for EL6: How long will Puppet support those platforms? does it make sense for us to carry on with non-systemd support?

Copy link
Contributor

@joshcooper joshcooper May 15, 2024

Choose a reason for hiding this comment

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

RHEL 6 will have Extended LIfecycle Support until June 30, 2024 based on https://access.redhat.com/articles/4665701#6-how-can-i-continue-to-receive-software-maintenance-bug-and-security-fixes-and-full-technical-support-for-red-hat-enterprise-linux-6-6

But they could extend it, similar to how Oracle extended Solaris 10 until Jan 2027 https://blogs.oracle.com/solaris/post/new-end-date-of-extended-support-for-oracle-solaris-10-and-113

Also SLES 11 is something we will be supporting for some time.

end
end

PlatformHelpers::SYSTEMVPLATFORMS.each do |platform|
Copy link
Member

Choose a reason for hiding this comment

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

Would have been nice to keep this for EL6.

Copy link
Member Author

Choose a reason for hiding this comment

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

@joshcooper do you still rely on it? Vox Pupuli doesn't support EL6 anymore and I suspect you won't update to beaker 6 soonish and drop EL6 from your pipelines anyways?

Copy link
Member

Choose a reason for hiding this comment

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

We special cased it in the code. Either we should have dropped it or implemented tests. We kept it in at @joshcooper's request so I'd like to test it

@bastelfreak bastelfreak mentioned this pull request May 24, 2024
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.

5 participants