Skip to content

Commit

Permalink
Update examples using ResourceRequirements
Browse files Browse the repository at this point in the history
Signed-off-by: Elliot Gunton <[email protected]>
  • Loading branch information
elliotgunton committed Dec 12, 2024
1 parent d0dbcda commit 88e7b68
Show file tree
Hide file tree
Showing 8 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion docs/examples/workflows/misc/user_container.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ This example showcases the user of a user container with a volume mount.
metadata=m.ObjectMeta(name="something"),
spec=m.PersistentVolumeClaimSpec(
access_modes=["ReadWriteOnce"],
resources=m.ResourceRequirements(requests={"storage": "64Mi"}),
resources=m.VolumeResourceRequirements(requests={"storage": "64Mi"}),
),
)
],
Expand Down
2 changes: 1 addition & 1 deletion docs/examples/workflows/upstream/ci_output_artifact.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ The upstream example can be [found here](https://github.com/argoproj/argo-workfl
metadata=m.ObjectMeta(name="workdir"),
spec=m.PersistentVolumeClaimSpec(
access_modes=["ReadWriteOnce"],
resources=m.ResourceRequirements(
resources=m.VolumeResourceRequirements(
requests={
"storage": m.Quantity(__root__="1Gi"),
}
Expand Down
2 changes: 1 addition & 1 deletion docs/examples/workflows/upstream/ci_workflowtemplate.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ The upstream example can be [found here](https://github.com/argoproj/argo-workfl
metadata=m.ObjectMeta(name="work"),
spec=m.PersistentVolumeClaimSpec(
access_modes=["ReadWriteOnce"],
resources=m.ResourceRequirements(
resources=m.VolumeResourceRequirements(
requests={
"storage": m.Quantity(__root__="64Mi"),
}
Expand Down
2 changes: 1 addition & 1 deletion docs/examples/workflows/use-cases/fine_tune_llama.md
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,7 @@ There are several implicit dependencies in this script:
m.PersistentVolumeClaim(
metadata=m.ObjectMeta(name=f"rank-{i}"),
spec=m.PersistentVolumeClaimSpec(
resources=m.ResourceRequirements(requests={"storage": "20Gi"}, limits={"storage": "20Gi"}),
resources=m.VolumeResourceRequirements(requests={"storage": "20Gi"}, limits={"storage": "20Gi"}),
# TODO: it's possible to spin up pods in one zone of a region and a disk in another zone of a region!
# I recommend setting a `storage_class_name` that specifically tells K8s that it should create
# the volumes only when pods actually want to _mount_ a volume! That way the disks are
Expand Down
2 changes: 1 addition & 1 deletion examples/workflows/misc/user_container.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ def foo():
metadata=m.ObjectMeta(name="something"),
spec=m.PersistentVolumeClaimSpec(
access_modes=["ReadWriteOnce"],
resources=m.ResourceRequirements(requests={"storage": "64Mi"}),
resources=m.VolumeResourceRequirements(requests={"storage": "64Mi"}),
),
)
],
Expand Down
2 changes: 1 addition & 1 deletion examples/workflows/upstream/ci_output_artifact.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
metadata=m.ObjectMeta(name="workdir"),
spec=m.PersistentVolumeClaimSpec(
access_modes=["ReadWriteOnce"],
resources=m.ResourceRequirements(
resources=m.VolumeResourceRequirements(
requests={
"storage": m.Quantity(__root__="1Gi"),
}
Expand Down
2 changes: 1 addition & 1 deletion examples/workflows/upstream/ci_workflowtemplate.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
metadata=m.ObjectMeta(name="work"),
spec=m.PersistentVolumeClaimSpec(
access_modes=["ReadWriteOnce"],
resources=m.ResourceRequirements(
resources=m.VolumeResourceRequirements(
requests={
"storage": m.Quantity(__root__="64Mi"),
}
Expand Down
2 changes: 1 addition & 1 deletion examples/workflows/use_cases/fine_tune_llama.py
Original file line number Diff line number Diff line change
Expand Up @@ -253,7 +253,7 @@
m.PersistentVolumeClaim(
metadata=m.ObjectMeta(name=f"rank-{i}"),
spec=m.PersistentVolumeClaimSpec(
resources=m.ResourceRequirements(requests={"storage": "20Gi"}, limits={"storage": "20Gi"}),
resources=m.VolumeResourceRequirements(requests={"storage": "20Gi"}, limits={"storage": "20Gi"}),
# TODO: it's possible to spin up pods in one zone of a region and a disk in another zone of a region!
# I recommend setting a `storage_class_name` that specifically tells K8s that it should create
# the volumes only when pods actually want to _mount_ a volume! That way the disks are
Expand Down

0 comments on commit 88e7b68

Please sign in to comment.