github-actions[bot] commented on code in PR #62748:
URL: https://github.com/apache/doris/pull/62748#discussion_r3201031400
##########
docker/runtime/doris-compose/resource/init_fdb.sh:
##########
@@ -27,6 +27,21 @@ init_db() {
return
fi
+ if [ -n "$DORIS_CLOUD_AK" ] && [ -n "$DORIS_CLOUD_SK" ] && [ -n
"$DORIS_CLOUD_ENDPOINT" ]; then
Review Comment:
`fdb.conf` now always starts `backup_agent` with `--blob-credentials
/opt/apache-doris/fdb/conf/blob_creds.json`, but this file is only created
after the initialized-cluster guard. Any existing compose cloud cluster with
`HAS_INIT_FDB_FILE` already present will return at the top of `init_db`, so
after upgrading/restarting to this config the backup agent starts with a
credentials path that does not exist. That breaks backup/restore workflows for
precisely the existing-cluster recovery case. Please create/refresh
`blob_creds.json` before the early return, or make the backup-agent credential
argument conditional on the file actually being present.
--
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]
