MonkeyCanCode commented on code in PR #1610:
URL: https://github.com/apache/polaris/pull/1610#discussion_r2103738051


##########
site/content/in-dev/unreleased/getting-started/quickstart.md:
##########
@@ -36,9 +36,38 @@ cd ~/polaris
   :polaris-quarkus-admin:assemble --rerun \
   -Dquarkus.container-image.tag=postgres-latest \
   -Dquarkus.container-image.build=true
-docker compose -f getting-started/eclipselink/docker-compose-postgres.yml -f 
getting-started/eclipselink/docker-compose-bootstrap-db.yml -f 
getting-started/eclipselink/docker-compose.yml up
+```
+- **For standalone**: Omit the `-Dquarkus.container-image.tag` and 
`-Dquarkus.container-image.build` options if you do not need to build a Docker 
image.
+
+2. **Set the Assets Path**
+```shell
+export ASSETS_PATH=$(pwd)/getting-started/assets/
 ```
 
+3. **Set Authentication Credentials**
+
+Polaris supports multiple authentication methods, including the use of 
`CLIENT_ID` and `CLIENT_SECRET` environment variables. If you choose to use 
these credentials, you can set them as follows:
+
+```shell
+export CLIENT_ID=root
+export CLIENT_SECRET=s3cr3t
+```
+- **For Docker**: These variables are configured in the `getting-started.env` 
file. To use custom values, export them as shown above and remove the 
`--env-file` option from the `docker compose` command.

Review Comment:
   Fine with me. The only problem is I don't have a way to test the cloud 
deployment (so I am assuming it should work...PR is updated). There are more 
issues with those inconsistencies.  Lets get this merge first then we can fix 
other issues as the existed one is not even use-able. WDYT? 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@polaris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to