Skip to content

Change UID & GID to 1000/0 #1193

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

Merged
merged 4 commits into from
Jul 11, 2025
Merged

Change UID & GID to 1000/0 #1193

merged 4 commits into from
Jul 11, 2025

Conversation

lfrancke
Copy link
Member

@lfrancke lfrancke commented Jul 8, 2025

Description

Caution

Merge after #1200 is resolved (so we don't consume runners until we need to build everything).

Reverting the previous change from PR #1164

Reason: I thought I had tested this and everything would work with FSGroup set.
But it turns out that some tools seem to write some data with a permission set that is only accessible by the user (e.g. NiFi journals) so this will require more work and that'll happen after 25.7

Definition of Done Checklist

Note

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

  • Changes are OpenShift compatible
  • All added packages (via microdnf or otherwise) have a comment on why they are added
  • Things not downloaded from Red Hat repositories should be mirrored in the Stackable repository and downloaded from there
  • All packages should have (if available) signatures/hashes verified
  • Add an entry to the CHANGELOG.md file
  • Integration tests ran successfully
TIP: Running integration tests with a new product image

The image can be built and uploaded to the kind cluster with the following commands:

bake --product <product> --image-version <stackable-image-version>
kind load docker-image <image-tagged-with-the-major-version> --name=<name-of-your-test-cluster>

See the output of bake to retrieve the image tag for <image-tagged-with-the-major-version>.

@lfrancke lfrancke self-assigned this Jul 8, 2025
@lfrancke lfrancke moved this to Development: In Progress in Stackable Engineering Jul 8, 2025
@lfrancke
Copy link
Member Author

lfrancke commented Jul 8, 2025

Before merging this I'd like to see if the OPA & NiFi (& possibly others) tests work again.

@lfrancke lfrancke changed the title Change UID & GID to 1000/1000 Change UID & GID to 1000/0 Jul 11, 2025
@NickLarsenNZ NickLarsenNZ enabled auto-merge July 11, 2025 09:40
@NickLarsenNZ NickLarsenNZ moved this from Development: In Progress to Development: Waiting for Review in Stackable Engineering Jul 11, 2025
@NickLarsenNZ NickLarsenNZ disabled auto-merge July 11, 2025 09:44
@NickLarsenNZ NickLarsenNZ added this pull request to the merge queue Jul 11, 2025
Merged via the queue into main with commit 8de1825 Jul 11, 2025
3 checks passed
@NickLarsenNZ NickLarsenNZ deleted the chore/revert-uid-gid branch July 11, 2025 11:39
@sbernauer sbernauer moved this from Development: Waiting for Review to Development: Done in Stackable Engineering Jul 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Development: Done
Development

Successfully merging this pull request may close these issues.

3 participants