SKIP UPGRADE Resolve "Update Nexus 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
Upgrade Notices
N/A
Closes #2356 (closed)
Merge request reports
Activity
added kindmaintenance priority6 teamDevelopment & Ops labels
for reviewer:
Check for BB test-values dupes in main branch:
comm -1 -2 \ <(yq -P 'sort_keys(..)' -o=props <(curl -s https://repo1.dso.mil/big-bang/product/packages/nexus/-/raw/main/chart/values.yaml) | sort) \ <(yq -P '.addons.nexusRepositoryManager.values' -o=props <(curl -s https://repo1.dso.mil/big-bang/bigbang/-/raw/master/tests/test-values.yaml) | sort) \ | sed '/^$/d'
Check for BB test-values dupes on THIS (2356-update-nexus-test-values-yaml) branch:
comm -1 -2 \ <(yq -P 'sort_keys(..)' -o=props <(curl -s https://repo1.dso.mil/big-bang/product/packages/nexus/-/raw/main/chart/values.yaml) | sort) \ <(yq -P '.addons.nexusRepositoryManager.values' -o=props <(curl -s https://repo1.dso.mil/big-bang/bigbang/-/raw/2356-update-nexus-test-values-yaml/tests/test-values.yaml) | sort) \ | sed '/^$/d'
Command should only output the following, which is a false positive as the list here has unique items/values between default values.yaml and BB test-values.yaml.
nexus.env.0.name = INSTALL4J_ADD_VM_PARAMS nexus.env.1.name = NEXUS_SECURITY_RANDOMPASSWORD nexus.env.1.value = true nexus.repository.repo.0.repo_data.cleanup.policyNames.0 = string nexus.repository.repo.0.repo_data.component.proprietaryComponents = true nexus.repository.repo.0.repo_data.online = true nexus.repository.repo.0.repo_data.storage.blobStoreName = default nexus.repository.repo.0.repo_data.storage.strictContentTypeValidation = true nexus.repository.repo.0.repo_data.storage.writePolicy = allow_once nexus.repository.repo.0.type = hosted
Edited by Matt Vasquezassigned to @matt.vasquez
added statusreview label
added nexus label
added nexusRepositoryManager label
removed nexus label
requested review from @chris.oconnell
mentioned in commit 59871e11
Please register or sign in to reply