Skip to content
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

@corrupt 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 [email protected]

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

Signed-off-by: corrupt <[email protected]>
@netlify

netlify Bot commented Jan 10, 2023

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

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).

@Ilyesbdlala

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 ?

@Ilyesbdlala

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 :)

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