Copilot commented on code in PR #4252:
URL: https://github.com/apache/hertzbeat/pull/4252#discussion_r3659661293


##########
script/docker-compose/hertzbeat-postgresql-victoria-metrics/docker-compose.yaml:
##########
@@ -34,8 +34,8 @@ services:
     ports:
       - '15432:5432'
     environment:
-      POSTGRES_USER: root
-      POSTGRES_PASSWORD: 123456
+      POSTGRES_USER: ${POSTGRES_USER:-root}
+      POSTGRES_PASSWORD: ${POSTGRES_PASSWORD:?Please set POSTGRES_PASSWORD 
(e.g., in .env or your environment)
       TZ: Asia/Shanghai

Review Comment:
   Line 38 is missing the closing `}` (and currently appears unterminated), 
which will make the Compose file invalid and prevent `docker compose up` from 
parsing. Close the variable expansion and keep the error message on a single 
line (optionally quote the whole value to avoid YAML parsing pitfalls with 
punctuation).



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to