FastGPT icon indicating copy to clipboard operation
FastGPT copied to clipboard

更新Oceanbase镜像版本;修复容器部署时在healthcheck中出现的问题

Open Amaranterre opened this issue 8 months ago • 8 comments

将Oceanbase镜像版本更新到了最新长期支持版本4.3.5-lts

同时解决了 Issue 4642 提到的部署问题。 部署失败的原因在于,docker-compose.yml中的,

      test:
        [
          'CMD-SHELL',
          'obclient -h$OB_SERVER_IP -P2881 -uroot@$OB_TENANT_NAME -p$OB_TENANT_PASSWORD -e "SELECT 1;"'
        ]

在被docker-compose解析时,可能会存在环境变量不能被解析的情况,这时command就变成了

                "Test": [
                    "CMD-SHELL",
                    "obclient -h -P2881 -uroot@ -p -e \"SELECT 1;\""
                ],

中的

"obclient -h -P2881 -uroot@ -p -e \"SELECT 1;\""

(见这个comment中打印出的 log)

因此将写法改为了

      test:
        [
          'CMD-SHELL',
          'obclient -h$${OB_SERVER_IP} -P2881 -uroot@$${OB_TENANT_NAME} -p$${OB_TENANT_PASSWORD} -e "SELECT 1;"'
        ]

这样环境变量会在容器内部的bash被解析,而不是被docker-compose解析。

Amaranterre avatar Apr 27 '25 06:04 Amaranterre

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: Amaranterre
:x: sa-buc


sa-buc seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

cla-assistant[bot] avatar Apr 27 '25 06:04 cla-assistant[bot]

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: Amaranterre
:x: sa-buc


sa-buc seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

cla-assistant[bot] avatar Apr 27 '25 06:04 cla-assistant[bot]

TestGru Assignment

Summary

Link CommitId Status Reason
Detail ab64777fbc8f5f6a5792c5fb911b7c45dbb4980c 🚫 Skipped No files need to be tested {"deploy/docker/docker-compose-oceanbase/docker-compose.yml":"target file(deploy/docker/docker-compose-oceanbase/docker-compose.yml) not in work scope \n include: projects/app/src//*.ts \n exclude: node_modules,/.test.ts,**/.test.tsx,/*.spec.ts,/.spec.tsx,**/.d.ts,/*.test.js,/*.spec.js"}

[!TIP] You can @gru-agent and leave your feedback. TestGru will make adjustments based on your input

gru-agent[bot] avatar Apr 27 '25 06:04 gru-agent[bot]

Preview mcp_server Image: ghcr.io/labring/fastgpt-pr:fatsgpt_mcp_server_ab64777fbc8f5f6a5792c5fb911b7c45dbb4980c

github-actions[bot] avatar Apr 27 '25 06:04 github-actions[bot]

Preview sandbox Image: ghcr.io/labring/fastgpt-pr:fatsgpt_sandbox_ab64777fbc8f5f6a5792c5fb911b7c45dbb4980c

github-actions[bot] avatar Apr 27 '25 06:04 github-actions[bot]

Preview fastgpt Image: ghcr.io/labring/fastgpt-pr:fatsgpt_ab64777fbc8f5f6a5792c5fb911b7c45dbb4980c

github-actions[bot] avatar Apr 27 '25 06:04 github-actions[bot]

另外那个issue也可以close了 @c121914yu

Amaranterre avatar Apr 27 '25 07:04 Amaranterre

另外那个issue也可以close了 @c121914yu

Amaranterre avatar Apr 27 '25 07:04 Amaranterre