Skip to content

Conversation

@maximilianoPizarro
Copy link
Contributor

What is this PR About?

Deploy Botpress Server Community Helm Charts on Red Hat OpenShift

Redhat kubernetes Helm shell linkedin Artifact Hub

Open

Watch a demo video
Botpress on Red Hat Developer Sandbox

The purpose of this project is to generate the kubernetes objects based on the node image from the official repository botpress for deployment on container platforms using the Helm Charts strategy. Verified in Sandbox RedHat OpenShift Dedicated (Openshift 4.14.1).

How do we test this?

Installation

Charts Values Parameters

Parameter Value Default
env.EXTERNAL_URL https://my-botpress-NAMESPACE-dev.apps.sandbox-m2.ll9k.p1.openshiftapps.com https://botpress-server-maximilianopizarro5-dev.apps.sandbox-m2.ll9k.p1.openshiftapps.com
route.host my-botpress-NAMESPACE-dev.apps.sandbox-m2.ll9k.p1.openshiftapps.com botpress-server-maximilianopizarro5-dev.apps.sandbox-m2.ll9k.p1.openshiftapps.com
image.repository botpress/server quay.io/maximilianopizarro/botpress-server-v12

Add repository

helm repo add botpress https://redhat-cop.github.io/helm-charts

Install Chart with parameters

helm install botpress botpress/botpress --version VERSION --set route.host="Your-WilcardDNS-hostname",env.EXTERNAL_URL="Your-WilcardDNS-with-https"
Example:
helm install botpress botpress/botpress --version 0.1.1 --set route.host="my-botpress-maximilianopizarro5-dev.apps.sandbox-m2.ll9k.p1.openshiftapps.com",env.EXTERNAL_URL="https://my-botpress-maximilianopizarro5-dev.apps.sandbox-m2.ll9k.p1.openshiftapps.com"

Uninstall Chart

helm uninstall botpress

cc: @redhat-cop/day-in-the-life

Signed-off-by: Maximiliano Pizarro <[email protected]>
@ckavili
Copy link
Contributor

ckavili commented Feb 21, 2025

hey @maximilianoPizarro, thanks for this PR! our linter is a little bit picky 😇 if you can address those new line etc warnings, I'd be happy to merge it 🙏

Signed-off-by: Maximiliano Pizarro <[email protected]>
Signed-off-by: Maximiliano Pizarro <[email protected]>
@maximilianoPizarro
Copy link
Contributor Author

hey @maximilianoPizarro, thanks for this PR! our linter is a little bit picky 😇 if you can address those new line etc warnings, I'd be happy to merge it 🙏

Hi @ckavili! Don´t worry I send the changes on the last commit. Thanks in advance 💯

@garethahealy
Copy link
Contributor

@maximilianoPizarro ; can you rebase please

gniltaws and others added 20 commits April 15, 2025 08:14
…call to approve InstallPlan fails

Add more robost and verbose error handling

Signed-off-by: Maximiliano Pizarro <[email protected]>
…call to approve InstallPlan fails

Switch from modify_and_apply to patch do too sometimes getting errors
about apply failing due to annotations getting to big

Signed-off-by: Maximiliano Pizarro <[email protected]>
Signed-off-by: Maximiliano Pizarro <[email protected]>
Signed-off-by: Maximiliano Pizarro <[email protected]>
@maximilianoPizarro
Copy link
Contributor Author

@maximilianoPizarro ; can you rebase please

Hi @garethahealy, thanks for your help. The rebase is done.

@garethahealy
Copy link
Contributor

charts/botpress/Chart.yaml
  Error: 1:15 [new-lines] wrong new line character: expected \n
  Error: 30:48 [new-line-at-end-of-file] no new line character at the end of file

@maximilianoPizarro
Copy link
Contributor Author

charts/botpress/Chart.yaml
  Error: 1:15 [new-lines] wrong new line character: expected \n
  Error: 30:48 [new-line-at-end-of-file] no new line character at the end of file

@garethahealy now validation maitainers step fail

Validation success! 👍
Validating maintainers...
Error: failed linting charts: failed processing charts

------------------------------------------------------------------------------------------------------------------------
✖︎ botpress => (version: "0.1.1", path: "charts/botpress") > failed validating maintainer "Maximiliano Pizarro": 404 Not Found
------------------------------------------------------------------------------------------------------------------------

@maximilianoPizarro
Copy link
Contributor Author

Thanks for the feedback @garethahealy . I rebased main, and for some reason, the operator-installer wasn't working with my version of main 3.2.1, so I updated it to version 3.2.2, and it works.

@garethahealy
Copy link
Contributor

@ckavili you happy with everything?

@ckavili
Copy link
Contributor

ckavili commented Apr 15, 2025

yup yup all good :)

@garethahealy garethahealy merged commit e26d00c into redhat-cop:main Apr 15, 2025
5 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.

6 participants