Skip to content

feat: support pods that are owned by deployments#22

Open
cgetzen wants to merge 2 commits into
SlinkyProject:mainfrom
taichi-dev:feat/pods-created-from-deployments
Open

feat: support pods that are owned by deployments#22
cgetzen wants to merge 2 commits into
SlinkyProject:mainfrom
taichi-dev:feat/pods-created-from-deployments

Conversation

@cgetzen
Copy link
Copy Markdown
Contributor

@cgetzen cgetzen commented May 20, 2026

Summary

Resolves #21

The deployment example in https://github.com/SlinkyProject/slurm-bridge/blob/main/docs/workload.md#annotations does not work.

Given that Pod is a supported workload, pods owned by deployments should also be supported.

Checklist

  • I have read
    CONTRIBUTING.md
    and the
    Code of Conduct.
  • New or existing tests cover these changes (where applicable).
  • Documentation is updated if user-visible behavior changes.

Breaking Changes

No breaking changes

Testing Notes

Integration tests have been added for the scheduler, similar to controller and admissions integration tests.

Applying the example from the readme now schedules correctly:

apiVersion: apps/v1
kind: Deployment
metadata:
  name: pause
  # `slurm-bridge` annotations on parent object
  annotations:
    slinky.slurm.net/timelimit: "5"
    slinky.slurm.net/account: foo
spec:
  replicas: 2
  selector:
    matchLabels:
      app: pause
  template:
    metadata:
      labels:
        app: pause
    spec:
      schedulerName: slurm-bridge-scheduler
      containers:
        - name: pause
          image: registry.k8s.io/pause:3.6
          resources:
            limits:
              cpu: "1"
              memory: 100Mi

Additional Context

cgetzen added 2 commits May 19, 2026 12:03
Signed-off-by: Charlie Getzen <charliegetzen@meshy.ai>
Signed-off-by: Charlie Getzen <charliegetzen@meshy.ai>
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.

[BUG]: SlurmBridge missing rbac around deployments

2 participants