Skip to content

add container resource limits#89

Merged
andymeierdev merged 1 commit intomainfrom
andymeierdev/add-container-resource-limits
Apr 9, 2026
Merged

add container resource limits#89
andymeierdev merged 1 commit intomainfrom
andymeierdev/add-container-resource-limits

Conversation

@andymeierdev
Copy link
Copy Markdown
Collaborator

Summary

  • add CPU and memory requests/limits for the app container
  • add CPU and memory requests/limits for the cloudflared sidecar

Testing

  • not run

Copy link
Copy Markdown

@minniemeierdev minniemeierdev left a comment

Choose a reason for hiding this comment

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

LGTM! Resource limits look reasonable.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

🍹 preview on andymeier/prod

Pulumi report

View in Pulumi Cloud

  Previewing update (prod)

View Live: https://app.pulumi.com/meiermade/andymeier/prod/previews/5d524231-adda-46e8-afce-1a3e5dc85baf

pulumi:pulumi:Stack: (same)
  [urn=urn:pulumi:prod::andymeier::pulumi:pulumi:Stack::andymeier-prod]
  ~ kubernetes:apps/v1:Deployment: (update)
      [id=andymeier/app]
      [urn=urn:pulumi:prod::andymeier::kubernetes:apps/v1:Deployment::app]
    ~ spec: {
        ~ template: {
            ~ spec: {
                ~ containers: [
                    ~ [0]: {
                            + resources: {
                                + limits  : {
                                    + cpu   : "250m"
                                    + memory: "256Mi"
                                  }
                                + requests: {
                                    + cpu   : "25m"
                                    + memory: "64Mi"
                                  }
                              }
                          }
                    ~ [1]: {
                            + resources: {
                                + limits  : {
                                    + cpu   : "100m"
                                    + memory: "128Mi"
                                  }
                                + requests: {
                                    + cpu   : "10m"
                                    + memory: "32Mi"
                                  }
                              }
                          }
                  ]
              }
          }
      }
Resources:
  ~ 1 to update
  15 unchanged
  

@andymeierdev andymeierdev merged commit 64132cf into main Apr 9, 2026
2 checks passed
@andymeierdev andymeierdev deleted the andymeierdev/add-container-resource-limits branch April 9, 2026 22:57
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