Skip to content

fix(014-galust-umbrella-chart): Galust Umbrella Chart#209

Merged
aghamyan44 merged 7 commits into
mainfrom
014-galust-umbrella-chart
May 12, 2026
Merged

fix(014-galust-umbrella-chart): Galust Umbrella Chart#209
aghamyan44 merged 7 commits into
mainfrom
014-galust-umbrella-chart

Conversation

@aghamyan44
Copy link
Copy Markdown
Contributor

No description provided.

@what-the-diff
Copy link
Copy Markdown

what-the-diff Bot commented May 11, 2026

🎉 Fabulous news, team! 🎉 We've made significant progress on the Helm chart for our Galust AI layer. Here's a kick-off overview of the improvements:

💾 Package Improvements

  • We've now included a .helmignore file, which helps keep our package neat and tidy by excluding non-relevant patterns. 🗑
  • Our new Chart.yaml file neatly organizes the Galust AI layer chart and clearly spells out its dependencies. 📜

📚 Documentation and Guides

  • Our newly crafted README.md is a comprehensive guide that clearly outlines our chart's purpose, deployment instructions, component details and some checks post-deployment - perfect for new team members! 📖
  • We've created a NOTES.txt template, which will offer insightful output information upon chart installation. 📝

🔧 Handy Tools and Templates

  • We have helper templates in _helpers.tpl that will assist in chart naming and label generation - a great time-saving feature! ⏳
  • We've got a new template for the image pull secret in image-pull-secret.yaml. It's now easier than ever to create dynamic Docker registry secrets. 👏

👀 Configurations

  • We've set up default specifications in values.yaml, which details settings for various components- it's a one-stop file for all the configuration needs. 🛠️
  • For testing purposes, we've created values.test.yaml to provide minimal configurations. 🧪

🥳 But that's not all! Here's more that we've done:

🚀 New Deployments

  • We've added a new Helm chart, charts/galust-ai-layer, to smoothly deploy various Galust AI layer components on Kubernetes! 🌐

📋 Task Outline

  • We've neatly jotted down the outline for setting up, implementing and validating the new chart in tasks.md- now, tracking tasks is a breeze! 📌

🔄 Backward Compatibility

  • We've preserved backward compatibility by maintaining existing deployment defaults from the ai-layer! No worries about breaking changes! 👍

🛡️ Security and Traffic Management

  • We now have ingress configurations for all components to manage incoming traffic with settings for TLS and CORS support. 🚦

Shout out to everyone who was involved! Your efforts are definitely moving us forward. Let's continue this fantastic work! 🚀

Comment thread charts/galust-ai-layer/values.test.yaml Outdated
dockerConfigJson: ""
dockerConfigJsonBase64: ""

imagePullSecrets: &galustImagePullSecrets
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we seems do not need this additional config, we can just mark imagePullSecret and use as anchor in places we need it

@@ -0,0 +1,25 @@
# Minimal test overlay for externally reachable Galust AI layer URLs.
imagePullSecret:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there I remember a known issue that aws ecr secrets getting expired after some time, and the pods fail to pull image until we manually renew the token again.
can we maybe integrate solution to renew thesecret here directly? as I remembre Hovo has this kind of fix/implementation for cloudbrowser.

Comment thread charts/galust-ai-layer/values.test.yaml Outdated
@@ -0,0 +1,25 @@
# Minimal test overlay for externally reachable Galust AI layer URLs.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets move example to root examples folder

Comment thread charts/galust-ai-layer/Chart.yaml Outdated
dependencies:
- name: base
alias: backend
version: 0.3.2
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the base chart version seems quite old, lets use the latest one

Comment thread charts/galust-ai-layer/values.yaml Outdated
key: host
DATABASE_PASSWORD:
secretKeyRef:
name: cnpg-main-user
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where do we create cnpg-main-urls and cnpg-main-user secrets, if its being created out of this helm chart then lets have it in readme
will not it be mre convinent to have host and password in same secret?

@aghamyan44 aghamyan44 changed the title fix(014-galust-umbrella-chart): Galust Imbrella Chart fix(014-galust-umbrella-chart): Galust Umbrella Chart May 12, 2026
@aghamyan44 aghamyan44 merged commit 93660bf into main May 12, 2026
3 checks passed
@aghamyan44 aghamyan44 deleted the 014-galust-umbrella-chart branch May 12, 2026 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants