-
Notifications
You must be signed in to change notification settings - Fork 119
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
Framework for Kubernetes integration tests #214
Conversation
Codecov Report
@@ Coverage Diff @@
## main #214 +/- ##
===========================================
- Coverage 81.29% 40.35% -40.94%
===========================================
Files 35 34 -1
Lines 2705 2649 -56
===========================================
- Hits 2199 1069 -1130
- Misses 373 1517 +1144
+ Partials 133 63 -70
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
LGTM!
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.
Looks like a great test setup. I skipped the ~2360 files inside the vendor folder. Thanks.
* k8s base test cluster (untested) * Fixed k8s deployments * barely working k8s test deployment * working smoke test * working version * Release candidate
Some future tasks might require testing Beyla in kubernetes. For example, to test Kubernetes metadata decoration.
This only adds a smoke test that can be skipped in later PRs, if we verify that K8s tests work properly in GitHub.
This also introduces a breaking change for the
PROMETHEUS_PORT
variable, in advance to the work that has to be done in the issue #213