search

Azure-Samples / azure-search-openai-demo-java

This repo is the Java version of Microsoft's sample app for ChatGPT + Enterprise data.
MIT License
67 stars 66 forks source link

Failed to "azd up" in VS Code Remote Containers #8

Closed Menghua1 closed 9 months ago

Menghua1 commented 9 months ago

Describe the issue: When running azd up command, we are encountering a file permission issue. The error details are as follows. Besides, azd up can run successfully after we give permission to the file. image Repro Steps:

  1. According to README.md, click the buttons of the VS Code Remote Containers environment in the document.
  2. Run azd auth login.
  3. Run azd up.

Environment：

Expected behavior: In VS Code Remote Containers, azd up can run successfully.

@dantelmomsft and @hemarina for notification.

dantelmomsft commented 9 months ago

@Menghua1 can't reproduce. It's not happening on my side. Can you share the command you used to set the permission, so we can add an entry in the troubleshooting guide.

However in your screenshot I've seen you have a notebooks folder. It has been deleted a couple of commits ago. Can you please update the code and try again. thanks

Menghua1 commented 9 months ago

@dantelmomsft Remote Containers button link should use https://vscode.dev/redirect?url=vscode://ms-vscode-remote.remote-containers/cloneInVolume?url=https://github.com/azure-samples/azure-search-openai-demo-java/ instead of https://vscode.dev/redirect?url=vscode://ms-vscode-remote.remote-containers/cloneInVolume?url=https://github.com/dantelmomsft/azure-search-openai-demo-java/ in Readme.md.

Through correct link open VS Code Remote Containers, running azd up still exsits this issue, however, it works after running chmod +x app/backend/mvnw to give permissions.

In addition, clone the repo code to local and reopen in Containers, it works.

dantelmomsft commented 9 months ago

closing as this commit 446c935 should solve the issue

Menghua1 commented 8 months ago

@dantelmomsft The link to the "OPEN" button in the picture below is still wrong. Your commit 446c935 does not solve the link jump issue at the location shown below.

Because your commit 446c935 only solves the linking error for the "OPEN" button at the location shown below:

image
dantelmomsft commented 8 months ago

@Menghua1 thanks. I've fixed this with a new commit: https://github.com/Azure-Samples/azure-search-openai-demo-java/commit/8a615d425d9c66d98ef59470c83bb9cad20a7b15