Skip to content

feat(java-base): Add openjdk-devel package#537

Merged
sbernauer merged 5 commits intomainfrom
feat/openjdk-devel
Jan 25, 2024
Merged

feat(java-base): Add openjdk-devel package#537
sbernauer merged 5 commits intomainfrom
feat/openjdk-devel

Conversation

@sbernauer
Copy link
Copy Markdown
Member

@sbernauer sbernauer commented Jan 25, 2024

Description

As decided in Arch meeting on 2024-01-25
For stackabletech/spark-k8s-operator#340

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes
- [x] Changes are OpenShift compatible
- [x] All added packages (via microdnf or otherwise) have a comment on why they are added
- [x] Things not downloaded from Red Hat repositories should be mirrored in the Stackable repository and downloaded from there
- [x] All packages should have (if available) signatures/hashes verified
- [ ] Does your change affect an SBOM? Make sure to update all SBOMs

Comment thread java-base/Dockerfile Outdated
Co-authored-by: Lars Francke <lars.francke@stackable.tech>
@sbernauer sbernauer requested a review from lfrancke January 25, 2024 07:39
@lfrancke
Copy link
Copy Markdown
Member

This should probably also change the other Dockerfiles which might manually install devel right now including the parameters we added for that

Examples:
https://github.com/stackabletech/docker-images/blob/main/hadoop/Dockerfile#L17
https://github.com/stackabletech/docker-images/blob/main/conf.py#L63-L64

@sbernauer
Copy link
Copy Markdown
Member Author

sbernauer commented Jan 25, 2024

Good catch, wasn't aware of them!

@sbernauer sbernauer self-assigned this Jan 25, 2024
@sbernauer sbernauer added this pull request to the merge queue Jan 25, 2024
Merged via the queue into main with commit a51da74 Jan 25, 2024
@sbernauer sbernauer deleted the feat/openjdk-devel branch January 25, 2024 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants