-
Notifications
You must be signed in to change notification settings - Fork 96
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add tests for recycling services #787
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing!
{Key: "k8s.pod.uid", Type: "string", Value: k8s.UUIDRegex}, | ||
{Key: "k8s.pod.start_time", Type: "string", Value: k8s.TimeRegex}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't know if needed, but maybe you can also verify that the pod UID and start time have changed from the previous pod, to double-check the restart.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great idea! I'll add this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👏🏻 thanks for addressing the suggestions!
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #787 +/- ##
==========================================
- Coverage 76.70% 68.94% -7.76%
==========================================
Files 107 104 -3
Lines 9276 9242 -34
==========================================
- Hits 7115 6372 -743
- Misses 1776 2410 +634
- Partials 385 460 +75
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
This PR adds k8s tests where we prove Beyla works on restart of services.