-
Notifications
You must be signed in to change notification settings - Fork 87
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
Springboot code #162
Springboot code #162
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.
Great. I made a few comments but it's ok to merge if they are addressed later/soon.
Please confirm that you have run all of the tests (ie source test.sh) successfully and go ahead and merge. Thanks!
@@ -0,0 +1,17 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> |
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.
please remove this file if/as it is not necessary
@@ -0,0 +1,4 @@ | |||
activeProfiles= |
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.
please remove this file if/as it is not necessary
@@ -2,7 +2,7 @@ FROM maven:3.6.3-openjdk-11 AS maven_build | |||
COPY pom.xml /tmp/ | |||
COPY src /tmp/src/ | |||
WORKDIR /tmp/ | |||
RUN mvn package | |||
RUN mvn clean package spring-boot:repackage |
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.
please move the build out into build.sh rather than in the dockerfile (see inventory-helidon for example)
</dependency> | ||
<dependency> | ||
<groupId>com.oracle.database.observability</groupId> | ||
<artifactId>dms</artifactId> |
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 think there are some redundancies re the oracle dependencies here but we can address this when we apply 21.3 versions soon
Inventory- springboot