stakater / Forecastle

Forecastle is a control panel which dynamically discovers and provides a launchpad to access applications deployed on Kubernetes – [✩Star] if you're using it!
https://stakater.com
Apache License 2.0
601 stars 62 forks source link

Patch chart deployment #322

Closed c-dst closed 1 year ago

c-dst commented 1 year ago

Hi,

This PR update the helm chart rename securityContext in podSecurityContext add container securityContext for both containers values and update default values.yaml.

It's include also PR#313 patch.

The patch was tested again the lastest (v1.0.118) forcastle helm release.

github-actions[bot] commented 1 year ago

@hussnain612 Image is available for testing. docker pull stakater/forecastle:SNAPSHOT-PR-322-7d80214e

syphernl commented 1 year ago

This change appears to be causing problems on K8s v1.24.12:

error validating data: ValidationError(Deployment.spec.template.spec.securityContext): unknown field "fsgroup" in io.k8s.api.core.v1.PodSecurityContext

Workaround (ArgoCD Application values):

forecastle:
    deployment:
        podSecurityContext:
            fsgroup: ~
sstoffregen commented 1 year ago

same problem here

This change appears to be causing problems on K8s v1.24.12:

error validating data: ValidationError(Deployment.spec.template.spec.securityContext): unknown field "fsgroup" in io.k8s.api.core.v1.PodSecurityContext
sstoffregen commented 1 year ago

It still does not work within https://artifacthub.io/packages/helm/stakater/forecastle/1.0.128

Problem is still the same when trying to deploy via AgoCD: error validating data: ValidationError(Deployment.spec.template.spec.securityContext): unknown field "fsgroup" in io.k8s.api.core.v1.PodSecurityContext

I have also recognized that there is no 1.0.128-Release on Github yet