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

增加kubernetes环境中部署halo文档 #89

Closed
wants to merge 3 commits into from
Closed

增加kubernetes环境中部署halo文档 #89

wants to merge 3 commits into from

Conversation

marionxue
Copy link

@marionxue marionxue commented Aug 9, 2022

@f2c-ci-robot f2c-ci-robot bot added the do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. label Aug 9, 2022
@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Aug 9, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign ruibaby after the PR has been reviewed.
You can assign the PR to them by writing /assign @ruibaby in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@netlify
Copy link

netlify bot commented Aug 9, 2022

Deploy Preview for halo-docs ready!

Name Link
🔨 Latest commit 32490ff
🔍 Latest deploy log https://app.netlify.com/sites/halo-docs/deploys/62f45ffaa8f3e8000850d7fa
😎 Deploy Preview https://deploy-preview-89--halo-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@f2c-ci-robot f2c-ci-robot bot added release-note-none Denotes a PR that doesn't merit a release note. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Aug 9, 2022
Copy link
Member

@ruibaby ruibaby left a comment

Choose a reason for hiding this comment

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

@marionxue 感谢你的贡献,这里有一些文档格式方面的建议,如果你认同我的修改,可以点击 Commit suggestion 按钮。

docs/getting-started/install/other/kubernetes.md Outdated Show resolved Hide resolved
docs/getting-started/install/other/kubernetes.md Outdated Show resolved Hide resolved
docs/getting-started/install/other/kubernetes.md Outdated Show resolved Hide resolved
docs/getting-started/install/other/kubernetes.md Outdated Show resolved Hide resolved
docs/getting-started/install/other/kubernetes.md Outdated Show resolved Hide resolved
docs/getting-started/install/other/kubernetes.md Outdated Show resolved Hide resolved
docs/getting-started/install/other/kubernetes.md Outdated Show resolved Hide resolved
docs/getting-started/install/other/kubernetes.md Outdated Show resolved Hide resolved
docs/getting-started/install/other/kubernetes.md Outdated Show resolved Hide resolved
pnpm-lock.yaml Outdated Show resolved Hide resolved
@ruibaby
Copy link
Member

ruibaby commented Aug 9, 2022

/cc @halo-dev/sig-docs

docs/getting-started/install/other/kubernetes.md Outdated Show resolved Hide resolved
docs/getting-started/install/other/kubernetes.md Outdated Show resolved Hide resolved
Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

Update docs/getting-started/install/other/kubernetes.md

Co-authored-by: Ryan Wang <i@ryanc.cc>

commit suggestions
type: ClusterIP
```

## 配置证书
Copy link
Member

Choose a reason for hiding this comment

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

如果能够提供两种配置证书的方式就更好了 : )

  1. 已有证书的配置
  2. Let's Encrypt 免费证书配置(可参考 cert-manager

Copy link
Author

Choose a reason for hiding this comment

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

文章已提供已有证书配置,Let's Encrypt 免费证书配置 暂没计划

Copy link
Member

Choose a reason for hiding this comment

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

当前文档已经非常棒了!关于免费证书配置,可以后期让社区用户继续完善。

近期我会抽空手动测试一下 Kubernetes 环境中部署 Halo 的文档。

@vercel
Copy link

vercel bot commented Aug 17, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
halo-docs ✅ Ready (Inspect) Visit Preview Aug 17, 2022 at 3:53AM (UTC)

@JohnNiang
Copy link
Member

非常抱歉最近没有时间测试 Kubernetes 部署。目前我们正在紧张地开发 Halo 2.0。

建议当前 PR 保持打开的状态,继续等待社区的反馈。

/reopen

@f2c-ci-robot f2c-ci-robot bot reopened this Sep 13, 2022
@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Sep 13, 2022

@JohnNiang: Reopened this PR.

In response to this:

非常抱歉最近没有时间测试 Kubernetes 部署。目前我们正在紧张地开发 Halo 2.0。

建议当前 PR 保持打开的状态,继续等待社区的反馈。

/reopen

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@f2c-ci-robot f2c-ci-robot bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 29, 2022
@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Sep 29, 2022

@marionxue: PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@JohnNiang
Copy link
Member

Hi @marionxue , 我们已经提供了 Helm Charts,并提供了对应的部署文档,请参考 #233。相对于手动编写 Manifest 文件,不容易出错,更易维护,更简单。

再次感谢 @marionxue 的参与!我们将关闭当前 PR。如果有任何问题可随时重新打开。

@JohnNiang JohnNiang closed this Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. release-note-none Denotes a PR that doesn't merit a release note.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants