Update server-logger to function with multiple securityContext settings #555

burmanm · 2023-07-18T13:58:19Z

What this PR does:
Modifies the server-logger to use an EmptyVolume to ensure that it doesn't have securityContext restrictions.

Update the CI tests to using a :version instead of :latest so that we don't accidentially test with incorrect version
Deprecate runDockerAsCassandraUser since our current security changes have made it non-functional.

Which issue(s) this PR fixes:
Fixes #553
Fixes #554

Checklist

… image. Also, remove dockerRunsAsCassandra property handling

pkg/reconciliation/construct_podtemplatespec.go

Add missing shell directive Fix checkout-strategy for the docker build job

burmanm · 2023-07-20T14:32:37Z

That CI error is not relevant, it's really just CI issue with download issue (originally, later the date won't match so it won't work).

burmanm added 2 commits July 18, 2023 14:07

Update kind and envtests to 1.27

d6c5ccc

Use EmptyVolume for /var/lib/vector and mount it to the system-logger…

ea7386e

… image. Also, remove dockerRunsAsCassandra property handling

burmanm requested a review from a team as a code owner July 18, 2023 13:58

emerkle826 reviewed Jul 18, 2023

View reviewed changes

pkg/reconciliation/construct_podtemplatespec.go Show resolved Hide resolved

Use versioned logger image in the testing instead of :latest

28b4273

Add missing shell directive Fix checkout-strategy for the docker build job

burmanm force-pushed the logger_changes branch from 650eec9 to 28b4273 Compare July 18, 2023 16:38

emerkle826 approved these changes Jul 26, 2023

View reviewed changes

burmanm merged commit 672ecf6 into k8ssandra:master Jul 26, 2023
39 of 40 checks passed

Provide feedback