Skip to content

Add monitoring config api-tests and update server to pass tests#1977

Open
chilagrow wants to merge 10 commits intov2from
monitoring-config-api-tests
Open

Add monitoring config api-tests and update server to pass tests#1977
chilagrow wants to merge 10 commits intov2from
monitoring-config-api-tests

Conversation

@chilagrow
Copy link
Copy Markdown
Contributor

@chilagrow chilagrow commented Mar 30, 2026

This PR adds monitoring config API tests

  • Add monitoring config API tests to run in CI
  • v2 branch uses v2 branch of helm-charts in CI
  • Fix monitoring config impl in server so tests pass
  • PMM is optional field in MonitoringConfig CR

Closes #1799.

Signed-off-by: Chi Fujii <chi.fujii@solanica.io>
Signed-off-by: Chi Fujii <chi.fujii@solanica.io>
Signed-off-by: Chi Fujii <chi.fujii@solanica.io>
@chilagrow chilagrow force-pushed the monitoring-config-api-tests branch from 33299ee to 856e20d Compare March 31, 2026 03:02
Signed-off-by: Chi Fujii <chi.fujii@solanica.io>
@chilagrow chilagrow force-pushed the monitoring-config-api-tests branch from f924e43 to d2b9b96 Compare March 31, 2026 05:13
Signed-off-by: Chi Fujii <chi.fujii@solanica.io>
Signed-off-by: Chi Fujii <chi.fujii@solanica.io>
Signed-off-by: Chi Fujii <chi.fujii@solanica.io>
@chilagrow chilagrow force-pushed the monitoring-config-api-tests branch from 3fb788b to 83a9433 Compare April 3, 2026 05:37
Signed-off-by: Chi Fujii <chi.fujii@solanica.io>
@chilagrow chilagrow force-pushed the monitoring-config-api-tests branch from ad36f1d to 0319aa5 Compare April 3, 2026 07:31
@chilagrow chilagrow marked this pull request as ready for review April 3, 2026 07:57
@chilagrow chilagrow requested a review from a team as a code owner April 3, 2026 07:57
Signed-off-by: Chi Fujii <chi.fujii@solanica.io>
@chilagrow chilagrow changed the title Add monitoring config api-tests Add monitoring config api-tests and update server to pass tests Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant