add team_name to dagBundleConfigList#64906
add team_name to dagBundleConfigList#64906stephen-bracken wants to merge 3 commits intoapache:mainfrom
Conversation
c2f831c to
9408d28
Compare
9408d28 to
c10924f
Compare
jscheffl
left a comment
There was a problem hiding this comment.
Multi-Team is a non-goal in the helm chart for the moment. There are plenty more things needed and a lot of work. We decided not adding this for the moment. So I'd rather propose not to add it here - otherwise there is an expectation that multi-team is in some way supported.
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds a team_name field to the Helm chart values JSON schema for dagBundleConfigList, enabling multi-team deployments to annotate DAG bundles with an owning team.
Changes:
- Extend
dagBundleConfigListitem schema with a newteam_namestring property.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: stephen-bracken <18257727+stephen-bracken@users.noreply.github.com>
|
Putting as |
|
Converting to draft as there is still a request for a change. It can be reopened whenever it is ready, thanks! |
Add team_name to dagBundleConfigList values schema
team_nameis used by multi-team deployments to identify which team owns which dag bundles, it needs to be added to the values schema.Was generative AI tooling used to co-author this PR?