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

Upgrade default image #6

Merged
merged 2 commits into from
Jan 30, 2024
Merged

Conversation

AnchorArray
Copy link
Contributor

@AnchorArray AnchorArray commented Jan 30, 2024

Azure deployment diff

langkit, langkit, Deployment (apps) has changed:
  # Source: langkit/templates/deployment.yaml
  apiVersion: apps/v1
  kind: Deployment
  metadata:
    name: langkit
    labels:
-     helm.sh/chart: langkit-0.4.0
+     helm.sh/chart: langkit-0.5.0
      app.kubernetes.io/name: langkit
      app.kubernetes.io/instance: langkit
-     app.kubernetes.io/version: "1.0.1"
+     app.kubernetes.io/version: "1.0.2.dev1"
      app.kubernetes.io/managed-by: Helm
  spec:
    replicas: 1
    selector:
      matchLabels:
        app.kubernetes.io/name: langkit
        app.kubernetes.io/instance: langkit
    template:
      metadata:
        labels:
-         helm.sh/chart: langkit-0.4.0
+         helm.sh/chart: langkit-0.5.0
          app.kubernetes.io/name: langkit
          app.kubernetes.io/instance: langkit
-         app.kubernetes.io/version: "1.0.1"
+         app.kubernetes.io/version: "1.0.2.dev1"
          app.kubernetes.io/managed-by: Helm
      spec:
        securityContext:
          {}
        containers:
          - name: langkit
            securityContext:
              readOnlyRootFilesystem: true
            image: "whylabs/whylogs:py-llm-1.0.2.dev0"
            imagePullPolicy: IfNotPresent
            ports:
              - containerPort: 8000
            livenessProbe:
              initialDelaySeconds: 15
              periodSeconds: 10
              tcpSocket:
                port: 8000
            readinessProbe:
              initialDelaySeconds: 15
              periodSeconds: 10
              tcpSocket:
                port: 8000
            resources:
              limits:
                cpu: "8"
                memory: 16Gi
              requests:
                cpu: "4"
                memory: 8Gi
            envFrom:
              - secretRef:
                  name: whylabs-api-key
              - secretRef:
                  name: langkit-api-secret
            volumeMounts:
              - mountPath: /tmp
                name: temp-dir
              - mountPath: /root/.config
                name: root-config
        volumes:
          - emptyDir: {}
            name: temp-dir
          - emptyDir: {}
            name: root-config
        affinity:
          podAntiAffinity:
            preferredDuringSchedulingIgnoredDuringExecution:
            - podAffinityTerm:
                labelSelector:
                  matchExpressions:
                  - key: app.kubernetes.io/name
                    operator: In
                    values:
                    - langkit
                topologyKey: kubernetes.io/hostname
              weight: 100
        tolerations:
          - effect: NoSchedule
            key: kubernetes.azure.com/scalesetpriority
            operator: Equal
            value: spot
langkit, langkit, Service (v1) has changed:
  # Source: langkit/templates/service.yaml
  apiVersion: v1
  kind: Service
  metadata:
    name: langkit
    labels:
-     helm.sh/chart: langkit-0.4.0
+     helm.sh/chart: langkit-0.5.0
      app.kubernetes.io/name: langkit
      app.kubernetes.io/instance: langkit
-     app.kubernetes.io/version: "1.0.1"
+     app.kubernetes.io/version: "1.0.2.dev1"
      app.kubernetes.io/managed-by: Helm
  spec:
    type: LoadBalancer
    ports:
      - protocol: TCP
        port: 80
        targetPort: 8000
    selector:
      app.kubernetes.io/name: langkit
      app.kubernetes.io/instance: langkit

@AnchorArray AnchorArray self-assigned this Jan 30, 2024
@AnchorArray AnchorArray requested a review from andyndang January 30, 2024 18:22
@AnchorArray AnchorArray marked this pull request as ready for review January 30, 2024 18:22
Copy link
Contributor

@andyndang andyndang left a comment

Choose a reason for hiding this comment

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

LGTM

@AnchorArray AnchorArray merged commit 8df9f86 into mainline Jan 30, 2024
1 check passed
@AnchorArray AnchorArray deleted the dev/aberg/bump-image-version branch January 30, 2024 18:33
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.

3 participants