FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update installation.md by corrupt · Pull Request #290 · secureCodeBox/documentation · GitHub

This repository was archived by the owner on Sep 6, 2023. It is now read-only.

Update installation.md - #290

Closed
corrupt wants to merge 1 commit into
secureCodeBox:mainfrom
corrupt:patch-1
Closed

Update installation.md#290
corrupt wants to merge 1 commit into
secureCodeBox:mainfrom
corrupt:patch-1

Conversation

corrupt commented Jan 10, 2023

Copy link
Copy Markdown

Adapted the instructions to the situation found when installing from helm in 2023.

Signed-off-by: corrupt corrupt@users.noreply.github.com

Adapted the instructions to the situation found when installing from helm in 2023.

Signed-off-by: corrupt <corrupt@users.noreply.github.com>

netlify Bot commented Jan 10, 2023
edited
Loading

Copy link
Copy Markdown

Deploy Preview for docs-securecodebox failed.

Name Link
🔨 Latest commit 54bb2d9
🔍 Latest deploy log https://app.netlify.com/sites/docs-securecodebox/deploys/63bd916b389bdb0009646e58

corrupt commented Jan 27, 2023

Copy link
Copy Markdown
Author

I honestly don't understand why the checks fail. It's not in anything I changed but in docs/architecture/09_architecture_decisions where index.md links local files that don't exist in main. And more specifically, in one file that isn't even linked (adr_0017.md).

Copy link
Copy Markdown
Member

Hello, Thank you for contribution ! :) I am however confused with what you mean by "installing from helm in 2023" ?
We currently use the Bitnami minio helm chart. It requires the use of root-user/root-password. The change was deliberate, as can be seen in the following PRs:

Can you please clarify ?

Copy link
Copy Markdown
Member

Hello, Thank you for contribution ! :) I am however confused with what you mean by "installing from helm in 2023" ? We currently use the Bitnami minio helm chart. It requires the use of root-user/root-password. The change was deliberate, as can be seen in the following PRs:

* [Upgraded embedded minio chart to latest release v11.9.4 (closes #755) #1340 ](https://github.com/secureCodeBox/secureCodeBox/pull/1340)

* [ fix(minio): Fixed port forward script to fit change to bitnami minio charts #1401 ](https://github.com/secureCodeBox/secureCodeBox/pull/1401)

Can you please clarify ?

In review, we realized that even though the SCB v4.0.0 is still not released, we have updated our docs prematurely to fit the changes (oops). We use bitnami minio charts in v4, and we're planning to release it soon. So I'll close this PR without merging. Thank you for letting us find out a mistake in our release process :)

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL