Remove hardcoded uid and gid, they'll default to the ones from the docker images now

Author: lfrancke

CHANGELOG.md

@@ -21,6 +21,8 @@ All notable changes to this project will be documented in this file.
   - Replace stackable-operator `print_startup_string` with `tracing::info!` with fields.
 - BREAKING: Inject the vector aggregator address into the vector config using the env var `VECTOR_AGGREGATOR_ADDRESS` instead
     of having the operator write it to the vector config ([#704]).
+- Previously this operator would hardcode the UID and GID of the Pods being created to 1000/0, this has changed now ([#718])
+  - The defaults from the docker images itself will now apply, which will be different from 1000/0 going forward
 
 ### Fixed
 
@@ -37,6 +39,7 @@ All notable changes to this project will be documented in this file.
 [#710]: https://github.com/stackabletech/druid-operator/pull/710
 [#714]: https://github.com/stackabletech/druid-operator/pull/714
 [#716]: https://github.com/stackabletech/druid-operator/pull/716
+[#718]: https://github.com/stackabletech/druid-operator/pull/718
 
 ## [25.3.0] - 2025-03-21
 

rust/operator-binary/src/druid_controller.rs

@@ -88,7 +88,6 @@ use crate::{
 pub const DRUID_CONTROLLER_NAME: &str = "druidcluster";
 pub const FULL_CONTROLLER_NAME: &str = concatcp!(DRUID_CONTROLLER_NAME, '.', OPERATOR_NAME);
 
-const DRUID_UID: i64 = 1000;
 const DOCKER_IMAGE_BASE_NAME: &str = "druid";
 
 // volume names
@@ -1140,8 +1139,6 @@ fn build_rolegroup_statefulset(
         .service_account_name(service_account.name_any())
         .security_context(
             PodSecurityContextBuilder::new()
-                .run_as_user(DRUID_UID)
-                .run_as_group(0)
                 .fs_group(1000)
                 .build(),
         );