Skip to content

refactor: docker deploy#392

Open
luojiyin1987 wants to merge 10 commits intomainfrom
refactor/docker-deploy
Open

refactor: docker deploy#392
luojiyin1987 wants to merge 10 commits intomainfrom
refactor/docker-deploy

Conversation

@luojiyin1987
Copy link
Contributor

@luojiyin1987 luojiyin1987 commented Jun 21, 2025

PR-392 PR-392 PR-392 Powered by Pull Request Badge

Checklist(清单):

  • Labels
  • Assignees
  • Reviewers

Closes #XXXXX

@luojiyin1987 luojiyin1987 self-assigned this Jun 21, 2025
@luojiyin1987 luojiyin1987 added the enhancement Some improvements label Jun 21, 2025
@luojiyin1987 luojiyin1987 requested a review from TechQuery June 21, 2025 08:49
@luojiyin1987
Copy link
Contributor Author

luojiyin1987 commented Jun 21, 2025

这个改动比较大,用的 SonOfBytes/unregistry-action 很新。 需要进行比较多的详细测试。

背景知识
https://www.perplexity.ai/search/build-and-push-docker-imageszh-al6hUzbdTKqL0dnW1d5B2A

Comment on lines +68 to +65
password: ${{ secrets.SSH_KEY }}
strip_components: 1
image: ${{ env.BOX_URL }}:latest
destination: ${{ secrets.USER }}@${{ secrets.HOST }}
ssh_key: ${{ secrets.SSH_KEY }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个项目用的是密码认证。

Copy link
Contributor Author

@luojiyin1987 luojiyin1987 Jun 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SonOfBytes/unregistry-action#1 已经提issue, 等维护者的确认。

psviderski/unregistry#16 也往更上游提 issue 寻求帮助。

@luojiyin1987
Copy link
Contributor Author

服务器上已经是 sudo 用户组 用户执行sudo时, 免密码的。
echo '${{ secrets.SSH_KEY }}' | sudo -S ,是没必要的。 可以直接执行 sudo 命令的.。 @TechQuery

@TechQuery
Copy link
Contributor

服务器上已经是 sudo 用户组 用户执行sudo时, 免密码的。
echo '${{ secrets.SSH_KEY }}' | sudo -S ,是没必要的。 可以直接执行 sudo 命令的.。 @TechQuery

主要这套脚本是要复用的,希望能应对更多不同的情况。

@luojiyin1987
Copy link
Contributor Author

服务器上已经是 sudo 用户组 用户执行sudo时, 免密码的。
echo '${{ secrets.SSH_KEY }}' | sudo -S ,是没必要的。 可以直接执行 sudo 命令的.。 @TechQuery

主要这套脚本是要复用的,希望能应对更多不同的情况。

已经改回去了。

TechQuery
TechQuery previously approved these changes Jun 22, 2025
@luojiyin1987
Copy link
Contributor Author

要等上游确认后, 再进行测试后, 再合并到主分支。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Some improvements

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

2 participants