Skip to content
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

fix(locking-service): remove enableServiceLinks from helm chart #2202

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

tyriis
Copy link
Owner

@tyriis tyriis commented Dec 7, 2023

No description provided.

@tyriis-automation
Copy link
Contributor

--- kubernetes HelmRelease: home-automation/locking-service Deployment: home-automation/locking-service

+++ kubernetes HelmRelease: home-automation/locking-service Deployment: home-automation/locking-service

@@ -24,13 +24,13 @@

     metadata:
       labels:
         app.kubernetes.io/component: main
         app.kubernetes.io/instance: locking-service
         app.kubernetes.io/name: locking-service
     spec:
-      enableServiceLinks: false
+      enableServiceLinks: true
       serviceAccountName: default
       automountServiceAccountToken: true
       securityContext:
         fsGroup: 1000
         fsGroupChangePolicy: OnRootMismatch
         runAsGroup: 1000

@tyriis-automation
Copy link
Contributor

--- kubernetes/talos-flux/apps/home-automation/locking-service/app Kustomization: flux-system/apps-locking-service HelmRelease: home-automation/locking-service

+++ kubernetes/talos-flux/apps/home-automation/locking-service/app Kustomization: flux-system/apps-locking-service HelmRelease: home-automation/locking-service

@@ -50,13 +50,12 @@

               protocol: TCP
             resources:
               requests:
                 cpu: 10m
                 memory: 128Mi
         pod:
-          enableServiceLinks: false
           securityContext:
             fsGroup: 1000
             fsGroupChangePolicy: OnRootMismatch
             runAsGroup: 1000
             runAsUser: 1000
           topologySpreadConstraints:

@tyriis tyriis merged commit 0edd625 into main Dec 7, 2023
8 checks passed
@tyriis tyriis deleted the feature/app-template-v2-migration branch December 7, 2023 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant