UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Cluster auditor: contributing guide addition, helm test change

Merged Micah Nagel requested to merge ca-changes into master
All threads resolved!

Package Merge Request

Package Changes

Add contributing doc for cluster auditor

Remove overrides for helm test in BB CI values - effectively this allows the test to pass even if there is no data present in the dashboard. After a number of back and forth changes and attempts this appears to be the best path forward:

  • Too many retries will time out the helm test (it already runs 5 times)
  • Consistently inconsistent behavior with data showing up or not, definitely seems to be timing with data populating in the dashboard
  • Test will continue to check both the prometheus target and the dashboards existence
  • Cluster auditor will be going away/changing in BB 2.0 so I think this modification is acceptable for the short term in between period

Package MR

https://repo1.dso.mil/platform-one/big-bang/apps/core/cluster-auditor/-/merge_requests/89

Edited by Micah Nagel

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
  • Micah Nagel resolved all threads

    resolved all threads

  • Micah Nagel added 1 commit

    added 1 commit

    • 5b4b0888 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Micah Nagel marked this merge request as ready

    marked this merge request as ready

  • Micah Nagel changed milestone to %1.46.0

    changed milestone to %1.46.0

  • Ryan Garcia approved this merge request

    approved this merge request

  • merged

  • Ryan Garcia mentioned in commit fe011681

    mentioned in commit fe011681

  • Please register or sign in to reply
    Loading