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

Debian: Add support for Splunk 9.4 and newer #393

Merged
merged 2 commits into from
Jan 9, 2025

Conversation

muncjack
Copy link

@muncjack muncjack commented Jan 8, 2025

this change is to change the file name of the debian packer after version 9.4.0
it was: -linux-2.6-
it needs to be: -linux-

Pull Request (PR) description

the is a problem when trying to install splunk 9.4.0 in ubuntu hosts due to package file name change

This Pull Request (PR) fixes the following issues

Fixes #392

version 9.4.0
it was: -linux-2.6-
if needs to be: -linux-
Copy link
Contributor

@maxadamo maxadamo 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 to me.
I checked the RPM and it is still using the same old name. This change is required only for DEB packages.

@vchepkov
Copy link

vchepkov commented Jan 8, 2025

I don't think this is specific for Debian, for example, rpm is also changed:

"https://download.splunk.com/products/splunk/releases/9.4.0/linux/splunk-9.4.0-6b4ebe426ca6.x86_64.rpm"

@bastelfreak bastelfreak changed the title this is a fix for debian package download of version 9.4.0 #392 Debian: Add support for Splunk 9.4 and newer Jan 8, 2025
@bastelfreak
Copy link
Member

Hi, thanks for the PR!

We've some acceptance tests in https://github.com/voxpupuli/puppet-splunk/tree/master/spec/acceptance . Could you extend them to also run on Splunk 9.4?

@maxadamo
Copy link
Contributor

maxadamo commented Jan 8, 2025

I don't think this is specific for Debian, for example, rpm is also changed:

"https://download.splunk.com/products/splunk/releases/9.4.0/linux/splunk-9.4.0-6b4ebe426ca6.x86_64.rpm"

@vchepkov I don't think so.
I quickly checked with puppet-debugger:

1:>> $version = '9.4.0'
 => "9.4.0"
2:>> $linux_prefix = (versioncmp($version, '9.0.5') >= 0 or (versioncmp($version, '8.2.11') >= 0 and versioncmp($version, '9.0.0') == -1)) ? {
  3:>> true  => '.',
  4:>> false => '-linux-2.6-',
  5:>> }
 => "."

and it prints a dot "." (with 9.4.0 and higher), which is correct.
But please feel free to double-check at the following URL, because it's really cluttered: https://www.splunk.com/en_us/download/previous-releases.html

@vchepkov
Copy link

vchepkov commented Jan 8, 2025

I don't think this is specific for Debian, for example, rpm is also changed:
"https://download.splunk.com/products/splunk/releases/9.4.0/linux/splunk-9.4.0-6b4ebe426ca6.x86_64.rpm"

@vchepkov I don't think so. I quickly checked with puppet-debugger:

What I meant was, similar update should be applied to rpm based linux. I don't use module myself, but someone might benefit from it. Thanks.

@maxadamo
Copy link
Contributor

maxadamo commented Jan 8, 2025

@vchepkov I don't think so. I quickly checked with puppet-debugger:

What I meant was, similar update should be applied to rpm based linux. I don't use module myself, but someone might benefit from it. Thanks.

Now I understand. Thanks to you. That was my first thought but the RPM is unchanged.

@muncjack
Copy link
Author

muncjack commented Jan 9, 2025

Hello,

I have added a entry to spec/spec_helper_acceptance.rb for 9.4.0, so I presum this will be added to the init test....

@maxadamo
Copy link
Contributor

maxadamo commented Jan 9, 2025

@bastelfreak bastelfreak added enhancement New feature or request and removed needs-tests labels Jan 9, 2025
@bastelfreak bastelfreak merged commit acbd76e into voxpupuli:master Jan 9, 2025
19 checks passed
@siegy22
Copy link
Contributor

siegy22 commented Jan 17, 2025

It looks like they didn't change it for their RPMs: https://download.splunk.com/products/splunk/releases/9.4.0/linux/splunk-9.4.0-6b4ebe426ca6.x86_64.rpm

They changed it for RPMs earlier, and they completely removed the -linux-2.6- part from it. 🤦

So RPMs will stil work with 9.4.0, I just checked 😄

The change for the RPMs is here: https://github.com/voxpupuli/puppet-splunk/blob/master/manifests/params.pp#L323-L328

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.

Debian Splunk package name is incorrect for version 9.4.0
6 participants