-
Notifications
You must be signed in to change notification settings - Fork 84
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
feat: extend support without orbit #150
feat: extend support without orbit #150
Conversation
bbbf3c4
to
74b2fa1
Compare
Impressive ! all access keys are in secrets ? |
I hope access keys are fake in the doc ;) |
62dc3d5
to
b84fd08
Compare
@@ -11,6 +11,8 @@ spec: | |||
replicas: {{ .Values.replication.consumer.replicaCount }} | |||
template: | |||
metadata: | |||
annotations: |
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.
It doesn't seem that this should be here if orbit isn't enabled. Should prob add a conditional for this.
@@ -11,6 +11,8 @@ spec: | |||
replicas: {{ .Values.replication.producer.replicaCount }} | |||
template: | |||
metadata: | |||
annotations: |
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 for orbit conditional
@@ -11,6 +11,8 @@ spec: | |||
replicas: {{ .Values.replication.status.replicaCount }} | |||
template: | |||
metadata: | |||
annotations: |
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 for orbit conditional
@@ -15,17 +15,30 @@ spec: | |||
release: {{ .Release.Name }} | |||
template: | |||
metadata: | |||
annotations: |
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 for orbit conditional
@@ -1,8 +1,6 @@ | |||
# Default values for backbeat. | |||
# This is a YAML-formatted file. | |||
# Declare variables to be passed into your templates. | |||
orbit: |
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.
Instead of completely removing .Values.orbit
consider moving it to .Values.global.orbit
in each of the sub charts (the Zenko parent values will still overwrite these)
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.
Done! 👍
cd1eeae
to
7fd9b77
Compare
f29760c
to
097016f
Compare
@NicolasT Any thoughts before we merge this? |
charts/zenko/values.yaml
Outdated
# When 'orbit.enabled' is 'true', these aren't used, please use | ||
# https://zenko.io to manage your deployment | ||
locationConstraints: {} | ||
# awsbackend: |
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.
@ssalaues Do you think we should leave these comments under locationConstraints
?
a100f9a
to
9d1aa30
Compare
7c107f3
to
56785bb
Compare
56785bb
to
1d6b042
Compare
1d6b042
to
6dbda62
Compare
This adds several features to Zenko when Orbit is disabled:
Documentation
Instrucctions on the setup can be found in
charts/setup-no-orbit.md
.Upgrades
If related configuration values are changed in
values.yml
, ahelm dependency build zenko && helm upgrade zenko zenko
command (assuming zenko is the release name) will restart the pods with the updated configuration.