Skip to content

feat: usage stats for HELM chart + replace DB scripts with migration container (ETL-627: ETL-438) #37

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
Merged
sudopower merged 35 commits into from
Jan 19, 2026
Merged

feat: usage stats for HELM chart + replace DB scripts with migration container (ETL-627: ETL-438) #37

sudopower merged 35 commits into from
Jan 19, 2026

Conversation

@sudopower
Copy link
Member

@sudopower sudopower commented Dec 8, 2025 •
edited
Loading

  1. Added hooks for reporting usages states
  2. If enabled (default) creates a secret with installation ID used by other components
  3. Replaced migration init container script with migration container image (build in CH-ETL)
  4. Removed unused roles and bindings
  5. Service account is now a hook resource
  • This is necessary for pre-install hook, since service account must exist before it's creation to allow the hook to create installation ID secret
  • Added a cleanup hook to ensure it gets cleaned before next installation
  1. Merged PR ETL-628: Feat/add support for changing sslmode #47 from @jessealkema (SSL mode is now expected to be part of connection URL)
  • for chart created POSTGRES default is "disabled", added option to override in values

ashish-bagri
ashish-bagri reviewed Dec 10, 2025
View reviewed changes
sudopower and others added 21 commits December 10, 2025 16:55
@sudopower
refactor: tracking.enabled from values.yaml
eb20373 
Removed tracking.enabled setting from values.yaml.
@sudopower
refactor: service account creation order for pre-install hook require…
42fcb7f 
…ments
@sudopower
Merge remote-tracking branch 'origin/sudopower/etl-438-tracking-helm-…
c041ef5 
…chart-usage' into sudopower/etl-438-tracking-helm-chart-usage
@sudopower
Merge branch 'refs/heads/main' into sudopower/etl-438-tracking-helm-c…
7e9da09 
…hart-usage
@sudopower
fix: naming of usageStats service
c852876
@sudopower
Merge branch 'refs/heads/main' into sudopower/etl-438-tracking-helm-c…
77825e8 
…hart-usage

# Conflicts:
#	charts/glassflow-etl/templates/deployment.yaml
@sudopower
refactor: make usagestats secrets optional, ignore if usagestats hook…
ef58ad8 
… fails
@sudopower
refactor: usage stats secret defaults
44da821
@jessealkema
feat: add support for SSL mode in PostgreSQL configuration and simpli…
6d32b91 
…fied passing connection string
@jessealkema
feat: add sslmode configuration for PostgreSQL in deployment and schema
81be47d
@sudopower
Merge branch 'refs/heads/main' into sudopower/etl-438-tracking-helm-c…
8d6cddd 
…hart-usage

# Conflicts:
#	charts/glassflow-etl/templates/deployment.yaml
@sudopower
refactor: remove unused role and role binding for reading k8s Job
4641fa2 
This is leftover from a migration job requirement
@sudopower
refactor: remove unused role binding for a role that does not exist
f6691fe
@sudopower
revert: api-role binding changes
9bf89f7 
unused, will be removed in a seperate PR
@sudopower
revert: api-role binding changes
14d1aa7 
unused, will be removed in a seperate PR
@sudopower
revert: api-role binding changes
3e26559 
unused, will be removed in a seperate PR
@sudopower
revert: api-role binding changes
555ff36 
unused, will be removed in a seperate PR
@sudopower
fix: stats svc var name
83aef67
@sudopower
refactor: remove unnecessary ns scope
d718347
@sudopower
Merge pull request #49 from glassflow/sudopower/refactor-chart
bd60d9d 
refactor: remove unused roles and role bindings
@sudopower
Merge pull request #47 from jessealkema/feat/add-support-for-changing…
47e2272 
…-sslmode

Feat/add support for changing sslmode
@sudopower sudopower changed the title feat: analytics tracking for HELM chart usage feat: usage stats for HELM chart usage Jan 15, 2026
@sudopower sudopower changed the title feat: usage stats for HELM chart usage feat: usage stats for HELM chart Jan 15, 2026
@sudopower sudopower changed the title feat: usage stats for HELM chart feat: usage stats for HELM chart (ETL-627: ETL-438) Jan 16, 2026
@sudopower sudopower changed the title feat: usage stats for HELM chart (ETL-627: ETL-438) feat: usage stats for HELM chart + replace DB scripts with migration container (ETL-627: ETL-438) Jan 16, 2026
@ashish-bagri ashish-bagri self-requested a review January 19, 2026 13:37
@sudopower sudopower merged commit 5a92936 into main Jan 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ashish-bagri ashish-bagri ashish-bagri approved these changes

@PabloPardoGarcia PabloPardoGarcia Awaiting requested review from PabloPardoGarcia

Assignees

No one assigned

Labels

do-not-merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sudopower @ashish-bagri @jessealkema