-
Notifications
You must be signed in to change notification settings - Fork 47
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
Deploying springBoot applications #4746
Comments
moving to 23.0.0.8 to align with the planned sb3 feature release |
epic moved to 23.0.0.9 Issue to update the guide for sb3 |
Plan to document (single topic and possibly config examples on feature page):
|
topics to migrate: Deploying a Spring Boot application to Liberty |
Not sure how this gets populated, but for https://docs-draft-openlibertyio.mqj6zf7jocq.us-south.codeengine.appdomain.cloud/docs/latest/reference/feature/springBoot-1.5.html it lists the Supported Java versions including JavaSE-1.8 - JavaSE-20. While Spring Boot 1.5 only supports JavaSE-1.8 |
We need to clarify that the If I was doing this with proper mathematical range notation that would be:
But maybe simply putting a
|
For the section Configuring thin Spring Boot applications it has an example of calling I would rather use a command like this that places the thinned result into the
And then updates the I recommend there also be a statement that the original |
@dmuelle @cbridgha Given Tom's comments above which I am okay with if he likes that better, I just want to make sure that we have something documented about using the |
@hlhoots I added a sub-section to the first part of the page specifically for using /dropins, which doesn't have you configure anything but the necessary features in server.xml: Deploying a Spring Boot JAR or WAR application from the dropins directory Does this cover that scenario? |
thanks for reviewing @tjwatson
|
Peer Review:
Feels like
Should we be mentioning what application?
Period (.) missing.
Period (.) missing.
Rephrased: Not sure if it implies the same. (Please verify my suggestion). When an app is in the |
Peer Review:To deploy a Spring Boot application to Open Liberty by using Maven, see the Containerizing, packaging, and running a Spring Boot application Open Liberty guide. I think "for deploying" is a weaker verb phrase and "Open Liberty guide" is how we generally refer to the collectionIn the following steps, you create a Liberty server instance, deploy your Spring Boot JAR or WAR application, and specify the default HTTP port for the server instance. There is only one server instance here, which "the" makes more clear than "that" By default, Liberty deploys the Spring Boot application with the default host configuration. not much change here and I think it reads better as is Run the Specify the application location with a |
@dmuelle Hi David, A few minor comments...
|
Peer ReviewConfigure and Deploy Spring Boot applications to Open Liberty Although the examples in the following sections use an example JAR application file, the Open Liberty configuration applies equally to JAR and WAR files. Alternatively, you can deploy a Spring Boot application without configuring the Test the application in a browser by going to the Without stopping the server, add another application argument in a new The Spring Boot application stops and restarts with the same context path. Test the application in a browser by going to the The result is a fat artifact that is easy to run on any server that has a JVM. However, this result is a large artifact, even for the smallest Also, spell check: artifact or artefact ? For example, if you use Docker to deploy your application to the cloud, you need to build a new Docker layer that includes your updated application content and all of the Spring Boot framework dependencies." Acrolinx recommends "go to" for URLs The following example uses the Run the Replace the value of the Link not working Links to guides dont work on docs-draft. Spring Boot 3 link is working now that autogen is updated |
Peer ReviewConfiguring non-default settings for the Spring Boot Actuator The IDs are ordered by precedence by using the Test the actuator health endpoint at the http://localhost:8080actuator/health URL. URL -> Please check / If the If the Start the server in the foreground with the Start the server in the foreground by using the |
Peer ReviewThis feature enables support for Spring Boot applications using Spring Boot version 2.x.x. This is from the metatype, not in manually written doc All other server configuration is handled by Spring Boot application properties. Only one server here and "configuration" is a collective noun to encompass all configuration on that server The following table lists the Open Liberty features that are required to support certain Spring Boot 2.0 starters. Per IBM style, need "that are" here To enable the Spring Boot Support 2.0 feature, add the following element declaration into your This is from the metatype, not in manually written doc |
Thanks for reviewing @chirp1 @ramkumar-k-9286 . I implemented both of karen's suggestions. Ram, I implemented all your suggestions except for a few noted in edited comments above. |
@dmuelle Hi David, the updates that you made based on my review comments look good to me. Thanks! |
Content is on vNext and will publish with 23.0.0.9. Closing as completed. |
Found this comment from @tjwatson in a closed issue
The topic was never moved over. We also have this comment from Laura/ Alasdair in the KC tracking sheet:
Analyze the entire Deploying a Spring Boot application to Liberty section and see what might go into a single OL topic in the deployment section, possibly with config examples added to the Spring Boot Support feature- but noten that the IBM docs topic uses v 1.5 and 2.0 is current so need to confirm if any changes from the IBM docs examples
The text was updated successfully, but these errors were encountered: