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

Add CentOS 8 support #917

Merged
merged 1 commit into from
Mar 10, 2020
Merged

Add CentOS 8 support #917

merged 1 commit into from
Mar 10, 2020

Conversation

traylenator
Copy link
Contributor

@traylenator traylenator commented Mar 6, 2020

Pull Request (PR) description

Add CentOS 8 support

The main differences for CentOS 8 are:

  • There is now a new collectd-utils package containing collectdctl
  • collectd-python is now a seperate sub package
  • python is python3.
  • pip command is now pip3
  • The rabbitmq plugin no longer works due to lack of python3 support

The python_dir fact will fall back to /usr/libexec/platform-python
if python3 does not exist on the path. This is the python
that collectd uses and significantly all that collectd-python requires.

Note that setting collectd::ci_package_repo true will currently
result in yum being configured with a non existant repo since
https://pkg.ci.collectd.org/rpm/ does not exist
for 8. request

@traylenator
Copy link
Contributor Author

I think after this it makes sense to add a Boolean collectd::utils = false param that ensures
collectdctl is present installing the collectd-utils package depending on the OS.

@vox-pupuli-tasks
Copy link

Dear @traylenator, thanks for the PR!

This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks

1 similar comment
@vox-pupuli-tasks
Copy link

Dear @traylenator, thanks for the PR!

This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks

The main differences for CentOS 8 are:
* There is now a new collectd-utils package containing collectdctl
* collectd-python is now a seperate sub package
* python is python3.
* pip command is now pip3
* The rabbitmq plugin no longer works due to lack of python3 support

The python_dir fact will fall back to `/usr/libexec/platform-python`
if `python3` does not exist on the path. This is the python
that collectd is built against and significantly requires.

Note that setting `collectd::ci_package_repo` true will currently
result in yum being configured with a non existant repo since
https://pkg.ci.collectd.org/rpm/ does not exist
for 8.

collectd/collectd-ci#35
@bastelfreak
Copy link
Member

@traylenator thanks for the awesome work! Is this still WIP?

@traylenator traylenator changed the title WIP: Add CentOS 8 support Add CentOS 8 support Mar 9, 2020
@traylenator traylenator removed the needs-work not ready to merge just yet label Mar 9, 2020
@traylenator
Copy link
Contributor Author

One more feature to add I'll make a release hopefully.

@traylenator traylenator merged commit eb8cfee into voxpupuli:master Mar 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants