-
Notifications
You must be signed in to change notification settings - Fork 523
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
fix (jkube-kit) : Remove ExposeEnricher (#1309) #1447
Conversation
Eclipse JKube CI ReportStarted new GH workflow run for #1447 (2022-08-09T14:49:28Z) ⚙️ JKube E2E Tests (2825980007)
|
Codecov Report
@@ Coverage Diff @@
## master #1447 +/- ##
============================================
+ Coverage 52.61% 52.69% +0.08%
+ Complexity 3936 3924 -12
============================================
Files 461 461
Lines 20814 20752 -62
Branches 2834 2819 -15
============================================
- Hits 10951 10935 -16
+ Misses 8724 8693 -31
+ Partials 1139 1124 -15
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
84b0122
to
e11fac7
Compare
Kudos, SonarCloud Quality Gate passed! |
I think E2E tests should pass once eclipse-jkube/jkube-integration-tests#139 gets merged |
We've removed a lot of functionality in this PR, we should discuss and re-evaluate alternatives. |
e11fac7
to
331a7c7
Compare
Kudos, SonarCloud Quality Gate passed! |
...r/generic/src/main/java/org/eclipse/jkube/enricher/generic/NetworkingV1IngressGenerator.java
Outdated
Show resolved
Hide resolved
...richer/generic/src/main/java/org/eclipse/jkube/enricher/generic/openshift/RouteEnricher.java
Outdated
Show resolved
Hide resolved
Remove all occurences of ExposeEnricher from project. Signed-off-by: Rohan Kumar <[email protected]>
4335901
to
c1bd2f6
Compare
70603f0
to
67722c6
Compare
Kudos, SonarCloud Quality Gate passed! |
Looks good, Thanks! 👍 |
Description
Fix #1309
ExposeEnricher
from project.kubernetes
orkubernetes-ro
.expose: false
label.jkube.openshift.generateRoute
orjkube.enricher.jkube-openshift-route.generateRoute
properties.expose: true
.jkube.createExternalUrls
property.expose: false
.jkube.createExternalUrls
property.expose: false
.Type of change
test, version modification, documentation, etc.)
Checklist