Skip to content

Allow flags config via chart #38

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

Conversation

shivtej-opsverse
Copy link
Contributor

Update The Chart to Support the "Allow filtering Out Normal Events Flag".

Following is the output of the deployment template with values:

args:
  - "--ignore-normal=true"

The following is the deployment template :

---
# Source: k8s-event-logger/templates/deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: release-name-k8s-event-logger
  labels:
    app.kubernetes.io/name: k8s-event-logger
    helm.sh/chart: k8s-event-logger-1.2.1
    app.kubernetes.io/instance: release-name
    app.kubernetes.io/version: "2.2"
    app.kubernetes.io/managed-by: Helm
spec:
  replicas: 1
  selector:
    matchLabels:
      app.kubernetes.io/name: k8s-event-logger
      app.kubernetes.io/instance: release-name
  template:
    metadata:
      labels:
        app.kubernetes.io/name: k8s-event-logger
        app.kubernetes.io/instance: release-name
    spec:
      serviceAccountName: release-name-k8s-event-logger
      containers:
        - name: app
          image: "maxrocketinternet/k8s-event-logger:2.2"
          imagePullPolicy: IfNotPresent
          securityContext:
            allowPrivilegeEscalation: false
            capabilities:
              drop:
              - ALL
            readOnlyRootFilesystem: true
            runAsGroup: 10001
            runAsNonRoot: true
            runAsUser: 10001
            seccompProfile:
              type: RuntimeDefault
          env:
          args:
            - --ignore-normal=true
          resources:
            limits:
              cpu: 100m
              memory: 128Mi
            requests:
              cpu: 10m
              memory: 32Mi

Copy link
Owner

@max-rocket-internet max-rocket-internet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks @shivtej-opsverse 💚

@max-rocket-internet max-rocket-internet merged commit b0c3f1f into max-rocket-internet:master Sep 6, 2023
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.

2 participants