Skip to content

add cpu/memory resources to deployment containers#26

Merged
andymeierdev merged 1 commit intomainfrom
andymeierdev/yeet-deployment-fix
Apr 9, 2026
Merged

add cpu/memory resources to deployment containers#26
andymeierdev merged 1 commit intomainfrom
andymeierdev/yeet-deployment-fix

Conversation

@andymeierdev
Copy link
Copy Markdown
Collaborator

Summary

  • add CPU/memory requests and limits to the main app container in pulumi/src/k8s/deployment.ts
  • add CPU/memory requests and limits to the sidecar container in the same deployment
  • keep existing probes, commands, and env configuration unchanged

Why

  • ensure both containers have explicit resource boundaries for more predictable scheduling and runtime behavior

Testing

  • not run (infrastructure definition change only)

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

🍹 preview on fsharp-view-engine/prod

Pulumi report

View in Pulumi Cloud

  Previewing update (prod)

View Live: https://app.pulumi.com/meiermade/fsharp-view-engine/prod/previews/4d019e06-9321-4a2f-995d-97a58d05bb7d

pulumi:pulumi:Stack: (same)
  [urn=urn:pulumi:prod::fsharp-view-engine::pulumi:pulumi:Stack::fsharp-view-engine-prod]
  ~ docker-build:index:Image: (update)
      [id=sha256:6560af848738fff6c7f31e2b1e44eb2f751aac27dc268bb8f7ad4af0746c5155]
      [urn=urn:pulumi:prod::fsharp-view-engine::docker-build:index:Image::fsharpviewengine]
    ~ cacheFrom : [
        + [0]: {
                + disabled: false
                + gha     : {
                    + scope: "buildkit"
                  }
                + raw     : ""
              }
      ]
    ~ cacheTo   : [
        + [0]: {
                + disabled: false
                + gha     : {
                    + ignoreError: true
                    + mode       : "max"
                    + scope      : "buildkit"
                  }
                + raw     : ""
              }
      ]
    ~ context   : {
        ~ location: "/Users/andy/repos/meiermade/FSharp.ViewEngine/sln" => "/home/runner/work/FSharp.ViewEngine/FSharp.ViewEngine/sln"
      }
    ~ dockerfile: {
        ~ location: "/Users/andy/repos/meiermade/FSharp.ViewEngine/sln/Dockerfile" => "/home/runner/work/FSharp.ViewEngine/FSharp.ViewEngine/sln/Dockerfile"
      }
  ~ kubernetes:apps/v1:Deployment: (update)
      [id=fsharpviewengine/fsharpviewengine]
      [urn=urn:pulumi:prod::fsharp-view-engine::kubernetes:apps/v1:Deployment::fsharpviewengine]
    ~ spec: {
        ~ template: {
            ~ spec: {
                ~ containers: [
                    ~ [0]: {
                            ~ image    : "us-east1-docker.pkg.dev/meiermade-platform/platform/fsharpviewengine:latest@sha256:de6c5673a5f70b171b5a7aafbd43726b021deb3535528411ed8a3e9d4bd67309" => [unknown]
                            + resources: {
                                + limits  : {
                                    + cpu   : "250m"
                                    + memory: "256Mi"
                                  }
                                + requests: {
                                    + cpu   : "25m"
                                    + memory: "64Mi"
                                  }
                              }
                          }
                    ~ [1]: {
                            + resources: {
                                + limits  : {
                                    + cpu   : "100m"
                                    + memory: "128Mi"
                                  }
                                + requests: {
                                    + cpu   : "10m"
                                    + memory: "32Mi"
                                  }
                              }
                          }
                  ]
              }
          }
      }
Resources:
  ~ 2 to update
  8 unchanged
  

@andymeierdev andymeierdev merged commit c03e159 into main Apr 9, 2026
2 checks passed
@andymeierdev andymeierdev deleted the andymeierdev/yeet-deployment-fix branch April 9, 2026 22:58
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