Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: 📦 Provide RudderStack Chart with HPA #44

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

feat: 📦 Provide RudderStack Chart with HPA #44

wants to merge 18 commits into from

Conversation

afranzi
Copy link

@afranzi afranzi commented Feb 12, 2022

Description of the change

This PR aims to provide the RudderStack helm as a Chart while providing the following features:

  • Feature: Provide Chart structure.
  • Feature: Release Chart on master merge.
  • Feature: Move (on demand) Postgres resources to a sidecar approach.
  • Feature: Enable HPA for the Transformations Deployment resource.
  • Feature: Enable HPA for the Backend Statefulset resource.
  • Feature: PreStop Hook to prevent data loss on new deployments or autoscaling down processes.
  • Fix: backend.uploadFreqInS typo.
  • Feature: Add env tag to Telegraf global tags.
  • Feature: Allow to define extraEnvVars in the Telegraf pod.

⚠️ Note: It requires the RudderStack team to enable gh-pages to host the Chart.

Related issues

PR #43

Checklists

Development

  • Lint rules pass locally
  • The code changed/added as part of this pull request has been covered with tests
  • All tests related to the changed code pass in development

Code review

  • This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
  • "Ready for review" label attached to the PR and reviewers mentioned in a comment
  • Changes have been reviewed by at least one other engineer
  • Issue from task tracker has a link to this pull request

afranzi and others added 16 commits January 31, 2022 18:47
@afranzi
feat: [DAT-3850] - 🐳 - Productionalize chart (#1)
72acd72 
* feat: 🐳 - Add GHA to publish helm

* Set theme jekyll-theme-time-machine

* Set theme jekyll-theme-cayman

* feat: 🐳 - Add GHA to publish helm

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* refactor: 📦 - Create subcharts & proper chart folder

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* fix: 📦 Move templates folder under the Chart one

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* fix: 📦 Add dependencies

* fix: 📦 Add dependencies

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* fix: 📦 Fix if identations

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* feat: 📝 - Allow to provide external config

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* feat: 🔒 - Allow to define the ServiceAccount

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* feat: 🔒 - Fix ServiceAccount position

* feat: 🔒 - Bump version to 0.4.5

* Prepare PR

* feat: [DAT-3878] - Allow to define extra env vars in telegraf

* feat: [DAT-3878] - Bump version to 0.4.6

* feat: [DAT-3936] 🔧 Move postgresql from statefulset to sidecar

* feat: [DAT-3936] 🔧 Add missing env var

* feat: [DAT-3936] 🔧 Fix rediness command

* feat: [DAT-3936] 🔧 Bump version

* feat: [DAT-3936] 🔧 Fix postgres host

* feat: [DAT-3936] 🔧 Remove postgres subchart

* feat: [DAT-3936] 🔧 Bump version to 0.4.11

* doc: [DAT-3898] - 📝 Update Documentation and add missing postgres values

* feat: [DAT-3850] 🔧 Allow to define the terminationGracePeriodSeconds value

* feat: [DAT-3850] - 💄 Cleaning values

* feat: [DAT-3850] 📦 Put postgres back

* fix [DAT-3850] - Define COMPUTE_DB_HOST_IN_K8S

* fix [DAT-3850] - Define ephemeral volume

Co-authored-by: afranzi <[email protected]>
@afranzi
fix [DAT-3850] - Fix ephemeral volumen
725066b
@afranzi
fix [DAT-3850] - Fix value
115c5bd
@afranzi
feat [DAT-3850] - Provide rudderWorkspaceToken if defined
f43ed9a
@afranzi
feat [DAT-4136] - 🔧 Provide a pre-stop script (#2)
73797b9 
* feat [DAT-4136] - 🔧 Provide a pre-stop script

* feat [DAT-4136] - 🔧 Solve script path
@afranzi
feat [DAT-4136] - 🔧 Move pre-stop hook to scripts configmap
8468b52
@afranzi
feat [DAT-4136] - 🔧 Bump chart version to v0.4.2
5edafb8
@afranzi
feat [DAT-4136] - 🔧 Enable Exec mode
5813767
@afranzi
feat [DAT-4136] - 🔧 Emit statsd pre-stop hook points (#3)
b6c0f86 
* feat [DAT-4136] - 🔧 Emit statsd pre-stop hook points

* feat [DAT-4136] - 🚀 Bump Chart version to v0.4.3
@afranzi
feat [DAT-4136] - :clock: Emit pre-stop duration time (#4)
4c06dcd
@afranzi
fix: [DAT-4136] - Fix bash issue
4356920
@afranzi
fix: [DAT-4136] - Expose pre-stop log
187a00b
@afranzi
bump: [DAT-4136] - Bump chart version to v0.4.6
683fed3
@afranzi
feat: [DAT-4136] - Add sleep time to postgres and telegraf sidecars o…
50f6fb8 
…n pre-stopping them
@afranzi
feat: [DAT-4136] - Add env tag to the telegraf global tags section
b931f2c
@afranzi
feat: [DAT-3850] - 📦 Add HPA resources
21b2142
@afranzi afranzi mentioned this pull request Feb 12, 2022
Closed
13 tasks
@tf-seti tf-seti deleted the branch rudderlabs:master December 19, 2023 11:24
@tf-seti tf-seti deleted the master branch December 19, 2023 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@afranzi @JavierMonton @cesarlugoe