-
Notifications
You must be signed in to change notification settings - Fork 2
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 helm chart #19
Add helm chart #19
Conversation
Coverage after merging add-helm-chart into main will be
Coverage Report
|
Coverage after merging add-helm-chart into main will be
Coverage Report
|
Coverage after merging add-helm-chart into main will be
Coverage Report
|
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.
Let's discuss values file design a bit. Also I think a values.example.yaml would be great
metadata: | ||
name: streaming-job-templates.streaming.sneaksanddata.com | ||
spec: | ||
group: streaming.sneaksanddata.com |
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.
I'm curious if we should use our domain here. I'll ask Jakob
.helm/templates/crd-streamclass.yaml
Outdated
shortNames: | ||
- streamclass | ||
- streamcls | ||
- sc |
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.
scls?
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.
Yes, would be better.
- INITIALIZING | ||
- READY | ||
- FAILED | ||
- STOPPED |
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.
Indent seems off?
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.
No, the indentation is correct.
selector: | ||
matchLabels: | ||
{{- include "app.selectorLabels" $ | nindent 6 }} | ||
# For now Arcane Operator supports only single replica |
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.
What's the todo here? Leader election?
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.
Yes, we can implement failover using Kubernetes leases: https://kubernetes.io/docs/concepts/architecture/leases/#custom-workload
# Fullname override | ||
fullnameOverride: "" | ||
|
||
# Service account configuration | ||
serviceAccount: |
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.
Indent off here?
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.
No, the indentation is correct.
# The name of the service account to use. | ||
# If not set and create is true, a name is generated using the fullname template | ||
name: "" | ||
|
||
# Additional labels for the deployment and pods | ||
additionalLabels: {} |
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.
Let's do operator.labels, maintainer.labels?
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.
The Operator and Maintainer work in the same process.
additionalLabels: {} | ||
|
||
# Additional labels for the deployment and pods | ||
additionalAnnotations: {} |
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.
Same here
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.
Same here :)
additionalAnnotations: {} | ||
|
||
# Extra environment variables to set in the deployment | ||
extraEnv: {} |
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.
Same here?
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.
Same here :)
Coverage after merging add-helm-chart into main will be
Coverage Report
|
Coverage after merging add-helm-chart into main will be
Coverage Report
|
Coverage after merging add-helm-chart into main will be
Coverage Report
|
Coverage after merging add-helm-chart into main will be
Coverage Report
|
Coverage after merging add-helm-chart into main will be
Coverage Report
|
Part of #8
Scope
Implemented:
Additional changes:
Checklist
latest
commit.