Skip to content

Develop -> Main: Merge resource heirarchy changes. #225

Merged
kapilgarg1996 merged 3 commits into
mainfrom
develop
Jun 23, 2026
Merged

Develop -> Main: Merge resource heirarchy changes. #225
kapilgarg1996 merged 3 commits into
mainfrom
develop

Conversation

@gyoza

@gyoza gyoza commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

No description provided.

akshat-goyal-harness and others added 3 commits June 22, 2026 19:22
…vice

- Add global.services.resourceHierarchy config to values.yaml
- Create harnesscommon.services.rhsEnv helper function in _services.tpl
- Enables automatic RESOURCE_HIERARCHY_SERVICE_SECRET injection
- Pattern mirrors existing database credential injection (dbv3)
- Eliminates need for 200+ per-service secret overrides
@gyoza gyoza changed the title Develop -> Main: Merge resource heirchy changes. Develop -> Main: Merge resource heirarchy changes. Jun 23, 2026
@gyoza gyoza requested a review from kapilgarg1996 June 23, 2026 02:09
env:
{{- include "harnesscommon.services.rhsEnv" (dict "ctx" $) | indent 12 }}
*/}}
{{- define "harnesscommon.services.rhsEnv" }}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ideal scenario, make the function accept service name and derive all values using that. Maybe in next version

@kapilgarg1996 kapilgarg1996 merged commit c657581 into main Jun 23, 2026
5 checks passed
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.

3 participants