Skip to content

configure resource requests and limits for native k8s executors#678

Merged
loujar merged 2 commits into
mainfrom
lsj/executor-resources
May 14, 2025
Merged

configure resource requests and limits for native k8s executors#678
loujar merged 2 commits into
mainfrom
lsj/executor-resources

Conversation

@loujar
Copy link
Copy Markdown
Contributor

@loujar loujar commented May 14, 2025

configure resource request and limits for native k8s executors for consistency with other sourcegraph helm charts.

Checklist

Test plan

verified with helm template . --show-only templates/executor.Deployment.yaml

@loujar loujar merged commit f8dd02a into main May 14, 2025
8 checks passed
@loujar loujar deleted the lsj/executor-resources branch May 14, 2025 20:46
jdpleiness pushed a commit that referenced this pull request May 14, 2025
…s executors (#680)

configure resource request and limits for native k8s executors for consistency with other sourcegraph helm charts.

### Checklist

- [ ] Follow the [manual testing process](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/TEST.md)
- [ ] Update [changelog](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/charts/sourcegraph/CHANGELOG.md)
- [ ] Update [Kubernetes update doc](https://docs.sourcegraph.com/admin/updates/kubernetes)

### Test plan

verified with `helm template . --show-only templates/executor.Deployment.yaml`
 <br> Backport f8dd02a from #678
enriquegh pushed a commit that referenced this pull request Jul 10, 2025
configure resource request and limits for native k8s executors for
consistency with other sourcegraph helm charts.

### Checklist

- [ ] Follow the [manual testing
process](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/TEST.md)
- [ ] Update
[changelog](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/charts/sourcegraph/CHANGELOG.md)
- [ ] Update [Kubernetes update
doc](https://docs.sourcegraph.com/admin/updates/kubernetes)

### Test plan

verified with `helm template . --show-only
templates/executor.Deployment.yaml`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants