SKIP UPGRADE Resolve "Re-evaluate test-values.yaml"

General MR

Summary

Remove duplicate values present in test-values.yaml that are already present in default values.yaml

Relevant logs/screenshots

N/A

Linked Issue

issue

Upgrade Notices

N/A

Closes #274 (closed)

added kindmaintenance priority6 teamDevelopment & Ops labels

assigned to @alfredodiaz53

added statusdoing label

added 1 commit

• c79b1be8 - remove duplicate defaulted overrides from test-values.yaml

Compare with previous version

changed title from Draft: Resolve "Re-evaluate test-values.yaml" to Draft: SKIP UPDATE CHECK Resolve "Re-evaluate test-values.yaml"

changed the description

assigned to @matt.vasquez and unassigned @alfredodiaz53

added 1 commit

• fd0e3359 - correct removing duplicates from test-values.yaml

Compare with previous version

added 1 commit

• 554b3bea - typo on syntax

Compare with previous version

changed title from Draft: SKIP UPDATE CHECK Resolve "Re-evaluate test-values.yaml" to Draft: SKIP UPGRADE Resolve "Re-evaluate test-values.yaml"

added 6 commits

• b4ba0314 - Big Bang Bot: Update CODEOWNERS
• 61ab830d - Update Ironbank
• 1cde8231 - Update Ironbank to v17.2.7
• 3ce64eec - Gitlab Dep Update
• 01df1a34 - Big Bang Bot: Update CODEOWNERS
• 809b00ec - SKIP UPDATE CHECK Update Ironbank to v17.2.8

Compare with previous version

added 1 commit

• 0f81bb96 - add CHANGELOG and update README

Compare with previous version

added 13 commits

• 0f81bb96...fd67855a - 12 commits from branch main
• c3f5fc51 - Merge branch 'main' into '274-re-evaluate-test-values-yaml'

Compare with previous version

For Reviewer:

Here is a check you can do to check for duplicate between test-values and ./chart/values.yaml:

comm -1 -2 \
<(yq -P 'sort_keys(..)' -o=props <(curl -s https://repo1.dso.mil/big-bang/product/packages/gitlab/-/raw/main/chart/values.yaml) | sort) \
<(yq -P 'sort_keys(..)' -o=props <(curl -s https://repo1.dso.mil/big-bang/product/packages/gitlab/-/raw/274-re-evaluate-test-values-yaml/tests/test-values.yml) | sort) \
| sed '/^$/d'

output should be empty if no duplicates are detected.

marked this merge request as ready

approved this merge request

mentioned in commit 8e60b5ea

merged

mentioned in merge request big-bang/bigbang!5183 (closed)

changed milestone to %2.38.0

added statusreview label and removed statusdoing label

hey team. For future reference, no need to bump package chart version when just modifying the test-values. That's not included in the package