-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ability to provision persistent disks to agents #590
Conversation
type: "noop" | ||
resources: | ||
disk: | ||
enabled: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
enabled ? it seems redundant
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
because maybe you want to temporary disable it without removing the configuration.
it is hard to distinguish "not enabled" from nulls that we have in all the code paths that deal with disks, like merging configurations with defaults or handling backward/forward compatibility
723cc70
to
1078f35
Compare
I continued the work:
|
dev/start-local.sh
Outdated
@@ -63,7 +69,7 @@ kubectl apply -f helm/examples/herddb-dev.yaml | |||
# Start LangStream | |||
helm repo add langstream https://langstream.github.io/charts | |||
helm repo update | |||
helm install langstream langstream/langstream --values helm/examples/local.yaml || helm upgrade langstream langstream/langstream --values helm/examples/local.yaml | |||
helm install langstream ../langstream-charts/charts/langstream --values helm/examples/local.yaml || helm upgrade langstream ../langstream-charts/charts/langstream --values helm/examples/local.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should probably rollback this line
#590 (comment)