Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: new install options: sshDir and extraEnvs #1422

Merged
merged 5 commits into from
Sep 10, 2024

Conversation

link89
Copy link
Contributor

@link89 link89 commented Sep 9, 2024

close #1421 and #1423

Copy link

changeset-bot bot commented Sep 9, 2024

🦋 Changeset detected

Latest commit: 3680938

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages
Name Type
@scow/cli Patch
@scow/auth Patch
@scow/gateway Patch
@scow/mis-server Patch
@scow/mis-web Patch
@scow/portal-server Patch
@scow/portal-web Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@link89 link89 changed the title sshDir as an option feat: sshDir as an option Sep 9, 2024
@link89 link89 changed the title feat: sshDir as an option feat: new install options: sshDir and extraEnvs Sep 9, 2024
Copy link
Member

@ddadaal ddadaal left a comment

Choose a reason for hiding this comment

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

非常感谢您的贡献!留下了一些评价意见,另外:请在apps/cli/assets/init-full/install.yaml中增加您新增的配置项的简介以及示例值,以使得新用户可以直接看到新的配置项。

@@ -160,7 +170,7 @@ export const createComposeSpec = (config: InstallConfigSchema) => {
const authVolumes = {
"/etc/hosts": "/etc/hosts",
"./config": "/etc/scow",
"~/.ssh": "/root/.ssh",
SSH_DIR: "/root/.ssh",
Copy link
Member

Choose a reason for hiding this comment

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

这里应该使用[SSH_DIR],使用SSH_DIR变量的值作为key,而不是使用字符串"SSH_DIR"作为key

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fix, 在本地通过 npm run build 验证

apps/cli/src/compose/index.ts Outdated Show resolved Hide resolved
@ddadaal
Copy link
Member

ddadaal commented Sep 10, 2024

非常感谢您的贡献!

@ddadaal ddadaal merged commit 35f3b60 into PKUHPC:master Sep 10, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature/Enhancement] 允许在 install.yaml 中指定 ssh_dir
2 participants