Skip to content

feat(chart): initial chart files #2

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

Merged
merged 15 commits into from
Mar 16, 2022
Merged

Conversation

ebaron
Copy link
Member

@ebaron ebaron commented Nov 24, 2021

Here is an initial Helm chart for Cryostat. It's mainly designed to quickly trial Cryostat in any Kubernetes cluster. As such, it's missing a lot of features of the operator, such as end-to-end encryption, authentication, persistent storage. Some of these should be not too difficult to add later.

It should work with a variety of network setups:

  • ClusterIP services, and kubectl port-forward
  • NodePort services
  • LoadBalancer services
  • Ingress
  • Routes (on OpenShift)

I've tested it out manually using Minikube and CodeReady Containers. If new to Helm, this is a good resource to learn the basics: https://helm.sh/docs/intro/.

To install:
helm install <name> ./cryostat/

To uninstall:
helm uninstall <name>

To render the YAML files that would be created during installation:
helm template <name> ./cryostat/

The install and template commands allow you to override values to test different scenarios without modifiying the values.yaml directly, e.g. --set core.service.type=NodePort,grafana.service.type=NodePort or --set core.route.enabled=true,grafana.route.enabled=true.

Fixes: #1

# runAsNonRoot: true
# runAsUser: 1000

grafana:
Copy link

Choose a reason for hiding this comment

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

shall it be possible, to use an existing grafana instance as well?

Copy link
Member Author

Choose a reason for hiding this comment

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

Not at first, but I could see that being a useful feature. I've filed #3 to make a note of this (linking to our operator since this would also be useful there).

name: cryostat
description: Securely manage JFR recordings for your containerized Java workloads

# A chart can be either an 'application' or a 'library' chart.
Copy link

Choose a reason for hiding this comment

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

guess you'd like to remove the default comments.

Copy link
Member Author

Choose a reason for hiding this comment

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

Definitely, thanks for the reminder!

@ebaron ebaron added the feat New feature or request label Mar 15, 2022
@ebaron ebaron marked this pull request as ready for review March 15, 2022 20:45
Copy link

@heubeck heubeck left a comment

Choose a reason for hiding this comment

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

Thank you.
Will you serve this chart from a http repo, e.g. by using GH pages?

@ebaron
Copy link
Member Author

ebaron commented Mar 15, 2022

Thank you. Will you serve this chart from a http repo, e.g. by using GH pages?

We still have to explore release options for this chart, but serving it on our GH page could be a good option. This PR references images from our latest development branch. Once we release our 2.1 version of Cryostat, we would like to release a Helm chart to go with it. It would also be nice if we could release a separate chart version pointing the latest development branch as well. I've created #8 to explore this further.

@heubeck
Copy link

heubeck commented Mar 16, 2022

Thank you. Will you serve this chart from a http repo, e.g. by using GH pages?

We still have to explore release options for this chart, but serving it on our GH page could be a good option. This PR references images from our latest development branch. Once we release our 2.1 version of Cryostat, we would like to release a Helm chart to go with it. It would also be nice if we could release a separate chart version pointing the latest development branch as well. I've created #8 to explore this further.

Sounds great.
A nice setup is e.g. here: https://github.com/bitnami-labs/sealed-secrets
They are creating GH releases with the packaged chart using GH actions, and host the helm repo index with GH actions.

@ebaron
Copy link
Member Author

ebaron commented Mar 16, 2022

Sounds great. A nice setup is e.g. here: https://github.com/bitnami-labs/sealed-secrets They are creating GH releases with the packaged chart using GH actions, and host the helm repo index with GH actions.

Thanks for the tip, I'll take a look!

@ebaron ebaron merged commit 47db2fd into cryostatio:main Mar 16, 2022
@ebaron ebaron linked an issue Mar 21, 2022 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a basic Helm chart for installing Cryostat
3 participants