SystemProperties ignored in spring-boot-maven-plugin if -Dspring.context.exit=onRefresh is specified about spring-boot

[scottfrederick]

@patbaumgartner It's not clear to me what you are trying to do. The CDS_TRAINING_JAVA_TOOL_OPTIONS environment variable can be configured on the spring-boot:build-image goal and passed to the Paketo Spring Boot buildpack. It should have no effect on the spring-boot:run goal, so I don't see how it could interfere with setting spring-boot.run.arguments on spring-boot:run.

Can you provide a more complete example of your pom.xml, the mvn commands you are running, and the behavior you expect?

[patbaumgartner]

Hi @scottfrederick

my main goal was to use App CDS with the spring-boot-maven plugin as it is described in the Buildpacks documentation. I realized that the parameter CDS_TRAINING_JAVA_TOOL_OPTIONS gets passed as JAVA_TOOL_OPTIONS successfully and was ignored. So I tried to find out where the issue might be.

            <plugin>
                <groupId>org.springframework.boot</groupId>
                <artifactId>spring-boot-maven-plugin</artifactId>
                <executions>
                    <execution>
                        <id>process-aot</id>
                        <goals>
                            <goal>process-aot</goal>
                        </goals>
                    </execution>
                </executions>
                <configuration>
                    <image>
                        <env>
                            <BP_SPRING_AOT_ENABLED>true</BP_SPRING_AOT_ENABLED>
                            <BP_JVM_CDS_ENABLED>true</BP_JVM_CDS_ENABLED>
                            <CDS_TRAINING_JAVA_TOOL_OPTIONS>-Dtelegrambots.enabled=false</CDS_TRAINING_JAVA_TOOL_OPTIONS>
                        </env>
                    </image>
                </configuration>
            </plugin>

Then I tried to run the app with java -Dtelegrambots.enabled=true -jar my.jar and it worked. Next step was the maven plugin which I got wrong above. It should have been mvn spring-boot:run -Dspring-boot.run.arguments="-Dreproducer.enabled=true" which passes the args to the app.

After that I realized that when adding -Dspring.context.exit=onRefresh my enabling property gets ignored. It doesn't matter if I do it via maven, buildpacks or java -jar.

I remember that I saw somewhere @sdeleuze used spring.flyway.enabled=false of FlywayAutoConfiguration for a training run but since the property gets ignored, it would not work as expected, which means to not creating a DB connection and migrating the data.

Since this is a pretty new feature, there is not much documentation and answers on SO. I hope you could follow my thoughts.

Best,
Patrick

[mhalbritter]

spring-boot.run.arguments are for arguments to the main class. I think jvmArguments is what you're looking for?

[sdeleuze]

FYI I did a quick test on aarch64 with the buildpacks branch of petclinic-efficient-container. If I remove spring.data.jdbc.dialect=postgresql from application.properties, the application training run fails as expected because it tries to connect to the database when building the container.

If I bring it back in the Spring Boot Maven configuration, it works again:

                <configuration>
                    <image>
                        <env>
                            <BP_SPRING_AOT_ENABLED>true</BP_SPRING_AOT_ENABLED>
                            <BP_JVM_CDS_ENABLED>true</BP_JVM_CDS_ENABLED>
                            <CDS_TRAINING_JAVA_TOOL_OPTIONS>-Dspring.data.jdbc.dialect=postgresql</CDS_TRAINING_JAVA_TOOL_OPTIONS>
                        </env>
                    </image>
                </configuration>

I did another test on x86 without the Spring Framework version customization, builder and image customization, it works as well. So the feature seems to work as expected with Buildacks. For Spring Boot run, the configuration is done differently as pointed out by @scottfrederick and @mhalbritter.