Skip to content

Conversation

@Manas23601
Copy link
Collaborator

@Manas23601 Manas23601 commented Aug 25, 2025

Description

Checks if any pod restarted.

Prom Query

avg(kube_pod_container_status_running{} != 0) by("uid", pod, namespace)[24h:5m]

Testing

Test is expected to fail


Note

Adds a no-pod-restarts integration test using Prometheus pod UID aggregation, extends the client to parse uid, and updates the README checklist.

  • Tests:
    • Add test/integration/prometheus/no_pod_restarts_test.go checking duplicate pods (by uid, pod, namespace) over 24h/168h; wired into test/integration/prometheus/test.bats.
  • Prometheus client:
    • Extend Metric with uid and handle it in UnmarshalJSON in pkg/prometheus/client.go.
  • Docs:
    • Mark "Ensure no pod restarts on OpenCost" as complete in README.md.

Written by Cursor Bugbot for commit 59806f3. This will update automatically on new commits. Configure here.

@Manas23601 Manas23601 requested review from ameijer and mbolt35 August 25, 2025 21:40
@Manas23601
Copy link
Collaborator Author

This test will only pass when chaos monkey is shut down

@Manas23601
Copy link
Collaborator Author

bugbot run

@ameijer
Copy link
Member

ameijer commented Nov 18, 2025

bugbot review

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no bugs!


@Manas23601
Copy link
Collaborator Author

bugbot run

@cursor
Copy link

cursor bot commented Nov 18, 2025

Skipping Bugbot: Bugbot is disabled for this repository

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.

2 participants