-
Notifications
You must be signed in to change notification settings - Fork 51
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
E2E test improvements #108
Conversation
Generated and curated from nri-kubernetes 2.4.0 in minikube v1.20.0 on Kubernetes v1.19.11
1f5d439
to
d33dab4
Compare
e2e/schema/service.json
Outdated
@@ -94,7 +93,6 @@ | |||
} | |||
], | |||
"required": [ | |||
"clusterIP", |
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.
I agree with this change, but I don't understand why some services (or versions) are not returning this. Do you know the root cause?
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.
I am investigating that
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.
clusterIP: None |
For some reason the service created forces the clusterIP to be none @roobre
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.
I am currently testing locally to re-add clusterIP
9bce801
to
2a8cb2b
Compare
I removed the e2e support for helm3 in favour of #110 |
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.
Looking HELLA RAD! 👍🏻
The main objective of this PR is: