log application configuration file used in generators and health check enrichers #2880
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Eclipse JKube CI ReportStarted new GH workflow run for #2880 (2024-09-03T14:00:12Z) ⚙️ JKube E2E Tests (10684202791)
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2880 +/- ##
=============================================
+ Coverage 59.36% 70.62% +11.26%
- Complexity 4586 5012 +426
=============================================
Files 500 486 -14
Lines 21211 19453 -1758
Branches 2830 2505 -325
=============================================
+ Hits 12591 13739 +1148
+ Misses 7370 4491 -2879
+ Partials 1250 1223 -27 ☔ View full report in Codecov by Sentry. |
arsenalzp
force-pushed
the
k8s-probes-port
branch
from
0c6dd01
to
4f19318
Compare
|
Let's discuss this on the issue first. |
arsenalzp
force-pushed
the
k8s-probes-port
branch
from
2bd807b
to
57f9e2a
Compare
|
...generator/api/src/test/java/org/eclipse/jkube/generator/api/DefaultGeneratorManagerTest.java
Outdated
Show resolved
Hide resolved
...kit/generator/api/src/main/java/org/eclipse/jkube/generator/api/DefaultGeneratorManager.java
Outdated
Show resolved
Hide resolved
jkube-kit/common/src/main/java/org/eclipse/jkube/kit/common/PropertiesExtender.java
Outdated
Show resolved
Hide resolved
| import java.net.URL; | ||
| import java.util.Properties; | ||
|
|
||
| public class PropertiesExtender extends Properties { |
There was a problem hiding this comment.
I'm not sure about the motivation behind using this POJO. Why not use a URL field directly?
There was a problem hiding this comment.
Hello,
Sorry for delayed response, I was preparing for certification exam.
java.util.Properties class is returned by SpringBootUtil.getSpringBootApplicationProperties(), however that class doesn't have necessary fields which might be used for storing URL/File object so I decided to extend it to bring properties file location together with its properties list:
public static SpringBootConfiguration from(JavaProject project) {
final PropertiesExtender properties = SpringBootUtil.getSpringBootApplicationProperties(
SpringBootUtil.getSpringBootActiveProfile(project),
JKubeProjectUtil.getClassLoader(project));
project.setPropertiesFile(properties.getPropertiesFile());
|
@arsenalzp : We discussed this internally and decided that we do not want to add this much complication to tackling this issue. I think the simplest approach would be to log a message (see #2832 (comment)) Would it be possible for you to update it to make this a bit simpler? |
Hello, |
|
Fixed. |
|
|
@arsenalzp : Is it possible to just have log statements without relying on |
Technically it is possible to show the message somewhere in |
|
Hello colleagues, |
|
@arsenalzp : Hello, I took a look at your changes again. However, it's a bit of a complex change for just logging the used properties file. I think you can extract the property file getting used from this method here (maybe create a separate method that receives JavaProject, KitLogger) and log the relative path of the Could you please check if this approach would work out? |
rohanKanojia
force-pushed
the
k8s-probes-port
branch
2 times, most recently
from
0b28296
to
bfa28f5
Compare
Merged
17 tasks
rohanKanojia
force-pushed
the
k8s-probes-port
branch
3 times, most recently
from
b4f831c
to
319bd03
Compare
rohanKanojia
force-pushed
the
k8s-probes-port
branch
from
319bd03
to
c5cf279
Compare
rohanKanojia
force-pushed
the
k8s-probes-port
branch
4 times, most recently
from
7f2fa66
to
e9eb254
Compare
Signed-off-by: Oleksandr Krutko <alexander.krutko@gmail.com> fix typo of resourcesDirectory javadoc in JavaProject class Signed-off-by: Oleksandr Krutko <alexander.krutko@gmail.com>
Signed-off-by: Oleksandr Krutko <alexander.krutko@gmail.com>
Signed-off-by: Oleksandr Krutko <alexander.krutko@gmail.com>
Signed-off-by: Oleksandr Krutko <alexander.krutko@gmail.com>
…erties file Signed-off-by: Oleksandr Krutko <alexander.krutko@gmail.com>
rohanKanojia
force-pushed
the
k8s-probes-port
branch
4 times, most recently
from
a68736e
to
36f49ae
Compare
Add a debug log statement in generators of these frameworks: - Spring Boot - Thorntail - Helidon - Quarkus - Micronaut Signed-off-by: Rohan Kumar <rohaan@redhat.com>
rohanKanojia
force-pushed
the
k8s-probes-port
branch
from
36f49ae
to
c7e0c96
Compare
|
rohanKanojia
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fixes #2832
jkube.internal.application-config-file.pathjkube.internal.application-config-file.pathproperty to log application config file path:Type of change
test, version modification, documentation, etc.)
Checklist