microsoft / azure-maven-plugins

Maven plugins for Azure
MIT License
274 stars 148 forks source link

[FEATURE_REQ] Support pre-definition for Spring Apps maven plugin #2296

Open moarychan opened 1 year ago

moarychan commented 1 year ago

Plugin name and version

azure-spring-apps-maven-plugin, 1.17.0

Plugin configuration in your pom.xml

<plugin>
    <groupId>com.microsoft.azure</groupId>
    <artifactId>azure-spring-apps-maven-plugin</artifactId>
    <version>1.17.0</version>
    <configuration>
        <appName>customers-service</appName>
        <isPublic>false</isPublic>
        <deployment>
            <cpu>1</cpu>
            <memoryInGB>1</memoryInGB>
            <instanceCount>1</instanceCount>
            <runtimeVersion>Java 17</runtimeVersion>
        </deployment>
    </configuration>
</plugin>

Expected behavior

The node configuration of the maven plugin azure-spring-apps-maven-plugin .

The configuration of the Maven plugin can define default configuration items or deployment items. When executing xxx:config, if the user does not want to use the default value, give a new value, such as <appName>customers-service</appName>, <runtimeVersion>Java 17</runtimeVersion>, etc. When saving into the POM file, all configured configurations will be merged into the maven plugin configuration node.

Actual behavior

Will skip config the module if there's a pre-definition for the Spring apps maven plugin.

Steps to reproduce the problem

Flanker32 commented 1 year ago

@moarychan Thanks for your report and sorry for the late response, currently spring apps maven plugin does not support modify existing configuration, but if you want to set cpu/memory during config, you may add -DadvancedOptions, here is the screen shot

image