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

[bookstack] Setup mysql tls connection for external database and other fixes #6

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

dod38fr
Copy link

@dod38fr dod38fr commented Nov 8, 2024

Special notes for your reviewer:

  • updated chart for recent kubernetes
  • add APP_KEY as bookstack does not start without
  • app.url is now mandatory to avoid bookstack crash

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • [ x] DCO signed
  • [ x] Chart Version bumped
  • [ x] Variables are documented in the README.md
  • [ x] Title of the PR starts with chart name (e.g. [velero])

Signed-off-by: Dominique Dumont <[email protected]>
Otherwise bookstack crashes on missing APP_KEY value

Signed-off-by: Dominique Dumont <[email protected]>
- port was changed from 80 to 8080 in solidnerd/bookstack 22.04 image.
  see https://hub.docker.com/r/solidnerd/bookstack
- ingress declaration was too old for k8s 1.29

Signed-off-by: Dominique Dumont <[email protected]>
Signed-off-by: Dominique Dumont <[email protected]>
Otherwise bookstack crashes on missing APP_URL value

Signed-off-by: Dominique Dumont <[email protected]>
Signed-off-by: Dominique Dumont <[email protected]>
Signed-off-by: Dominique Dumont <[email protected]>
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.

1 participant