existingConfigMapName is not working for ProxyFleet

[stevefan1999-personal]

What happened?

Setting an existingConfigMapName for ProxyFleet is not working. It will always set to be the default template. Modifying the default template will always revert back to the default anyway.

I need to set force-key-authentication to false in velocity-config.toml because I'm not using Local external traffic load balancer mode, because I'm self hosting a Kubernetes cluster running with Cilium. The default load balancer coming with Cilium does not have good support for Local mode because it is based on SNAT (despite there is a DSR mode that theoretically supports direct source IP forward with GENEVE, I failed to make it work correctly and I'm stuck with SNAT).

What components are involved in your issue?

No response

Version

0.7.0

Kubernetes Version

1.29

Relevant log output

No response

Code of Conduct

• I agree to follow this project's Code of Conduct

[jeremylvln]

Hi @stevefan1999-personal!

Thanks a lot for reporting the issue. The configuration field was present but was not used at all for the proxies. I've opened a PR to fix it. Let me know once merged & built if it fixes your issue (use the next image tag for the operator) :)

[TehNeon]

I may have attempted to utilize this feature incorrectly. But it sounds like it's meant to allow for the overriding of the config map that provides the proxy config (velocity-config.toml as defined in the Shulker provided configmap). But in actual usage, it appears that it wants theinit-fs.sh file to be defined in the config map, also likely other files like the probe-readiness.sh. I'm not sure if this is the intended behaviour to allow for the overriding of the init-fs.sh.

[jeremylvln]

It's currently a total overwrite you are write. I can't cherry-pick parts of the ConfigMap, I really want to keep this on the Kubernetes-level by changing the name of the ConfigMap. However I understand it could be an issue. Maybe it would be useful to split the ConfigMap to isolate the bootstrapping scripts from the config files themselves.

[github-actions]

🎉 This issue has been resolved in version 0.8.0 🎉

The release is available on:

• v0.8.0
• v0.8.0
• GitHub release

Your semantic-release bot 📦🚀

[stevefan1999-personal]

Not working for MinecraftServerFleet either

Shulker/packages/shulker-operator/src/reconcilers/minecraft_server_fleet/fleet.rs

Lines 59 to 61 in 5937788

	let mut config_clone = minecraft_server_fleet.spec.template.spec.config.clone();
	config_clone.existing_config_map_name =
	Some(ConfigMapBuilder::name(minecraft_server_fleet));

@jeremylvln

[jeremylvln]

Jeez. Nice catch. I totally forgot about the fleet. Will fix shortly.

Thanks!