Skip to content

Conversation

@heyvister1
Copy link
Collaborator

@heyvister1 heyvister1 commented Jul 6, 2025

Adding global attribute in values.yaml, to be able to propagate root chart values into sub-charts. For example network-operator global imagePullSecrets into node-feature-discovery sub-chart

@heyvister1 heyvister1 force-pushed the fix-image-pull-secrets branch 2 times, most recently from 4ecf04d to 4f50e4c Compare July 6, 2025 19:56
@heyvister1 heyvister1 requested review from almaslennikov and e0ne July 6, 2025 19:57
@coveralls
Copy link
Collaborator

coveralls commented Jul 6, 2025

Pull Request Test Coverage Report for Build 16109248994

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.03%) to 60.302%

Totals Coverage Status
Change from base Build 16072601436: 0.03%
Covered Lines: 3594
Relevant Lines: 5960

💛 - Coveralls

@heyvister1 heyvister1 force-pushed the fix-image-pull-secrets branch from 4f50e4c to aa95443 Compare July 7, 2025 06:19
… network-operator sub-charts

Signed-off-by: Ido Heyvi <[email protected]>
@heyvister1 heyvister1 added the on hold This enhancement is currently on hold pending additional clarification and evaluation label Jul 7, 2025
@heyvister1
Copy link
Collaborator Author

Should be on-hold until changes will get merged in nfd upstream

imagePullSecrets: []

# -- Global values to be propagated into Network-Operator sub-charts
global:
Copy link
Member

Choose a reason for hiding this comment

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

The format seems different between charts. May be problematic.
Some uses:

imagePullSecrets:
- name: my-secret

Other uses:

imagePullSecrets:
-  my-secret

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

on hold This enhancement is currently on hold pending additional clarification and evaluation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants