Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@jacobweinstock
Copy link
Member

Description

Clean up unneeded fields in the values.yaml. Implement fields like affinity, tolerations, and additionalArgs.

Why is this needed

Fixes: #

How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

Clean up unneeded fields in the values.yaml.
Implement fields like affinity, tolerations,
and additionalArgs.

Signed-off-by: Jacob Weinstock <[email protected]>
@codecov
Copy link

codecov bot commented Jun 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 48.19%. Comparing base (de82b7e) to head (5cc0890).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #228   +/-   ##
=======================================
  Coverage   48.19%   48.19%           
=======================================
  Files         101      101           
  Lines        8596     8596           
=======================================
  Hits         4143     4143           
  Misses       4197     4197           
  Partials      256      256           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jacobweinstock jacobweinstock added the ready-to-merge Signal Mergify to merge the PR label Jun 27, 2025
@mergify mergify bot merged commit 0dd45d9 into tinkerbell:main Jun 27, 2025
13 checks passed
@jacobweinstock jacobweinstock deleted the clean-up-helm-values branch August 20, 2025 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

helm-chart ready-to-merge Signal Mergify to merge the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant