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

Metacat is deployable on Kubernetes, using a Helm Chart #1623

Closed
artntek opened this issue May 11, 2023 · 7 comments
Closed

Metacat is deployable on Kubernetes, using a Helm Chart #1623

artntek opened this issue May 11, 2023 · 7 comments
Assignees
Labels
Milestone

Comments

@artntek
Copy link
Contributor

artntek commented May 11, 2023

Built on top of the work for #1218 (build a docker image for metacat) - the goal is to be able to use helm to install or upgrade metacat on a k8s cluster.

Issues related to this epic:

@artntek
Copy link
Contributor Author

artntek commented May 11, 2023

ToDo Items

For MVP Release, in Priority Order: see Issue #1665

Post-MVP -- see Issue #1666

@mbjones
Copy link
Member

mbjones commented Jun 28, 2023

Hey @artntek nice to see your progress on the Postgres sub chart over in that branch. A small thought -- I've been thinking it would be better to not commit files, and particularly binary files, in the charts directory, with a similar rationale to not committing other build artifacts like jar files. Otherwise, our repo is going to start accumulating versions of the pg chart as we upgrade versions. Better to stick with just the Chart.yaml defining the dependency.

For context: https://stackoverflow.com/q/73599035

When the chart gets built with helm dependency builds the dependency will be resolved, and when we package and release it it will be included in the artifact from helm package`. Does that resonate with you?

@artntek
Copy link
Contributor Author

artntek commented Jun 28, 2023

Totally! I was in two minds about this, and ended up following the example I found in dataone_indexer (which has a solr binary checked in). I’d feel better leaving it out. What about the lock file - should that be checked in?

@mbjones
Copy link
Member

mbjones commented Jun 28, 2023

Yeah, sorry about the indexer -- I've been meaning to change that one too. I'm not sure about the lockfile. It's a good question. I think it is probably not needed and will be generated anyways. Putting it in place would lock upgrades to a specific version for anyone who installed the chart, even if the chart label requested is fairly generic. If we have a really specific chart version listed in the dependencies, then the lock is probably not doing anything different.

Let's add charts to the .gitignore so it doesn't arise for others.

@artntek
Copy link
Contributor Author

artntek commented Jun 28, 2023

Removed binary and lock file in #1651

Let's add charts to the .gitignore so it doesn't arise for others.

Done - #1652

@artntek
Copy link
Contributor Author

artntek commented Aug 8, 2023

Exclusions for k8s Metacat v3.0.0+

  • Don't need cgi-bin setup
  • Not supporting Morpho client
    • can omit Context/useHttpOnly='false' in $TOMCAT/conf/context.xml

@mbjones mbjones added this to the 3.0.0 milestone Nov 3, 2023
@artntek
Copy link
Contributor Author

artntek commented Dec 1, 2023

closing this ticket - now tracking separately for:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants