UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

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

Merged Matt Vasquez requested to merge 205-re-evaluate-test-values-yaml into main

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 #205 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Matt Vasquez changed title from Draft: Resolve "Re-evaluate test-values.yaml" to Draft: SKIP UPGRADE Resolve "Re-evaluate test-values.yaml"

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

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

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

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

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

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

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

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

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

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

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

  • Matt Vasquez added 1 commit

    added 1 commit

    • 6c7ed777 - remove duplicate test-values

    Compare with previous version

  • Matt Vasquez marked this merge request as ready

    marked this merge request as ready

  • for reviewer:

    Check for Package 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/loki/-/raw/main/chart/values.yaml) | sort) \
    <(yq -P 'sort_keys(..)' -o=props <(curl -s https://repo1.dso.mil/big-bang/product/packages/loki/-/raw/main/tests/test-values.yaml) | sort) \
    | sed '/^$/d'

    Check for Package test-values dupes on THIS (205-re-evaluate-test-values-yaml) branch:

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

    image

    Command should have no output if there are no dupes found.

  • Blair Bowden approved this merge request

    approved this merge request

  • merged

  • Blair Bowden mentioned in commit 6bee8a95

    mentioned in commit 6bee8a95

Please register or sign in to reply
Loading