Skip to content

Conversation

@benzaidfoued
Copy link

Gate Agent CR templates behind enabled flags .

Fix for : #1169

Copy link
Contributor

@EItanya EItanya left a comment

Choose a reason for hiding this comment

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

A couple comments here:

  1. The sub-charts don't need the enabled piece, it's just a feature of subcharts.
  2. The parent chart already has sections for all of the agents

EItanya
EItanya previously approved these changes Jan 16, 2026
@EItanya
Copy link
Contributor

EItanya commented Jan 16, 2026

You will need to sign the commits before I can approve

@benzaidfoued benzaidfoued force-pushed the bug-fix branch 2 times, most recently from a8c1141 to 35b14bd Compare January 16, 2026 18:55
Signed-off-by: Foued Benzaid <benzaid.foued@gmail.com>
Signed-off-by: Foued Benzaid <benzaid.foued@gmail.com>
Signed-off-by: Foued Benzaid <benzaid.foued@gmail.com>
Signed-off-by: Foued Benzaid <benzaid.foued@gmail.com>
Signed-off-by: Foued Benzaid <benzaid.foued@gmail.com>
Signed-off-by: Foued Benzaid <benzaid.foued@gmail.com>
Signed-off-by: Foued Benzaid <benzaid.foued@gmail.com>
@benzaidfoued
Copy link
Author

You will need to sign the commits before I can approve

ouach, done

@benzaidfoued benzaidfoued requested a review from EItanya January 16, 2026 19:48
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