spring-guides / gs-spring-boot-for-azure Goto Github PK

Describe the issue:
Link address error, there are redundant symbols )：

Suggestion:
Remove ) in the link address.

@hemarina for notification.

In my understanding, there files / folders are not used anymore. How about deleting them?

Hi, I am new to Spring and Azure.
I follow the step 6 in 'Build and run the web app locally' and run the command: ./mvnw clean package
There is failure. The error message is:

[ERROR] Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 2.71 s <<< FAILURE! - in hello.GreetingControllerTest
[ERROR] greetingShouldReturnDefaultMessage Time elapsed: 0.583 s <<< FAILURE!
java.lang.AssertionError: Status expected:<200> but was:<404>
at hello.GreetingControllerTest.greetingShouldReturnDefaultMessage(GreetingControllerTest.java:43)

Hi there,

I'm a PM from Java on Azure team and would like to help improve this article.

The content of this article is outdated and the repo used is no longer under maintenance. The workflow has changed a lot and users can now achieve the same with following command:

git clone https://github.com/spring-projects/spring-petclinic.git
cd spring-petclinic
./mvnw package

mvn com.microsoft.azure:azure-webapp-maven-plugin:1.8.0:config
mvn azure-webapp:deploy

A similar docs can be found here.

BRs,
Yuchen

Describe the issue:
When executing ./mvnw azure-spring-apps:deploy according to README.md, the error is as follows:

Repro Steps:

1. Create Azure Spring Apps in portal.
2. Clone code to local and switch to complete folder.
3. Config the web app:

export MAVEN_OPTS="--add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.io=ALL-UNNAMED"
./mvnw com.microsoft.azure:azure-spring-apps-maven-plugin:1.14.0:config -DadvancedOptions

4. Deploy the app: ./mvnw azure-spring-apps:deploy

Environment:

• OS: Windows desktop.

Expected behavior:
Deploying the app can pass.

@hemarina for notification.

Describe the issue:
Link documents using Azure CLI instead of Azure Developer CLI(azd)：

Suggestion:
Update the link address to point to azd related documents.

@hemarina for notification.