Skip to content

Comments

apache-kafka: Fix, update and adjust JRE/JVM#98446

Merged
srhb merged 5 commits intoNixOS:masterfrom
srhb:fix-kafka
Mar 10, 2021
Merged

apache-kafka: Fix, update and adjust JRE/JVM#98446
srhb merged 5 commits intoNixOS:masterfrom
srhb:fix-kafka

Conversation

@srhb
Copy link
Contributor

@srhb srhb commented Sep 22, 2020

Most Kafka versions were broken by the recent jdk/jre changes in #89731 but it needed some cleanup work anyway. 😄

I've enriched the package versions with relevant JRE compatibility, and passed it through so that the module can take advantage of it to set a jre by default (new option) that corresponds in the service.

Also remove the default jvmOptions to avoid having to pass these through in version-compatible permutations. Users should adjust them according to the version they use, so add a release note that warns about this.

Also adds the latest kafka 2.6, sets it to the default apacheKafka and adjust the Scala version used for 2.5.X according to upstream recommendations.

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@ofborg ofborg bot added 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: changelog This PR adds or changes release notes 8.has: documentation This PR adds or changes documentation 8.has: module (update) This PR changes an existing module in `nixos/` labels Sep 22, 2020
@ofborg ofborg bot added the 8.has: package (new) This PR adds a new package label Sep 22, 2020
@ofborg ofborg bot requested a review from ragnard September 22, 2020 07:38
@ofborg ofborg bot added 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Sep 22, 2020
@srhb
Copy link
Contributor Author

srhb commented Sep 23, 2020

Force pushed to use the headless variants, the default ones pull in a ton of X deps.

@ofborg ofborg bot added 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. and removed 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Sep 23, 2020
@ryantm ryantm added 2.status: merge conflict This PR has merge conflicts with the target branch and removed 2.status: merge conflict This PR has merge conflicts with the target branch labels Oct 3, 2020
@GTrunSec
Copy link
Contributor

GTrunSec commented Oct 12, 2020

@srhb, please resolve the conflict. furthermore, about the JVM option. #100115 thanks @NeQuissimus are you maintainter here?

@srhb srhb force-pushed the fix-kafka branch 2 times, most recently from 717bc06 to 4af0b62 Compare October 13, 2020 05:32
@srhb
Copy link
Contributor Author

srhb commented Oct 13, 2020

Conflicts resolved. 🙂

@ofborg ofborg bot removed the 2.status: merge conflict This PR has merge conflicts with the target branch label Oct 13, 2020
@NeQuissimus
Copy link
Member

@srhb LGTM but can you add 2.6 to the test suite, please? :)

@srhb
Copy link
Contributor Author

srhb commented Oct 13, 2020

Thought I did already in a912e93276835b377dd0ad02fcfebf9e5e5f8cfe - did I miss another place?

@NeQuissimus
Copy link
Member

Oh wow, I must be blind :D

@SuperSandro2000 SuperSandro2000 added the 2.status: merge conflict This PR has merge conflicts with the target branch label Jan 18, 2021
@bbigras
Copy link
Contributor

bbigras commented Mar 9, 2021

Any progress on this?

@NeQuissimus
Copy link
Member

This should be good to go after a rebase

@srhb
Copy link
Contributor Author

srhb commented Mar 9, 2021

I'm sorry, I forgot all about this. I'll fix it up now.

@bbigras
Copy link
Contributor

bbigras commented Mar 9, 2021

I'm sorry, I forgot all about this. I'll fix it up now.

Hey no worries :) Thank you very much for the PR and the quick rebase.

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-already-reviewed/2617/356

@srhb
Copy link
Contributor Author

srhb commented Mar 10, 2021

I'm going to go ahead and merge once ofborg passes since there's some approval at least, and I don't want to play whack-a-mole with the release note merge conflicts. :-)

@ofborg ofborg bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. and removed 2.status: merge conflict This PR has merge conflicts with the target branch 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. labels Mar 10, 2021
@srhb srhb merged commit 33d6e71 into NixOS:master Mar 10, 2021
@srhb srhb deleted the fix-kafka branch March 10, 2021 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: changelog This PR adds or changes release notes 8.has: documentation This PR adds or changes documentation 8.has: module (update) This PR changes an existing module in `nixos/` 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants