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

Indexer Can Be Installed as a Helm Sub-Chart for Metacat in Kubernetes #49

Closed
artntek opened this issue Oct 19, 2023 · 3 comments
Closed
Milestone

Comments

@artntek
Copy link
Collaborator

artntek commented Oct 19, 2023

In order to support being deployed as a helm sub chart for metacat k8s deployment (see Metacat issue 1665), indexer needs:

  • cleanup of hardcoded parameters in yaml templates and values files
  • code changes to support healthchecks -- liveness & readiness probes
  • and a placeholder for anything else that is discovered during this process...
@artntek artntek added this to the 3.0.0 milestone Oct 19, 2023
@artntek artntek changed the title Indexer Can Be Installed as a Helm Sub-Chart in Kubernetes Indexer Can Be Installed as a Helm Sub-Chart for Metacat in Kubernetes Oct 19, 2023
@mbjones
Copy link
Member

mbjones commented Oct 19, 2023

That sounds great. Do you have a list of those hardcoded params? I didn't think there were many (any), as we've been able to deploy both locally on RD and on several namespaces of the dev cluster. Let's generate a list of what needs to be de-hardcoded, and it should be straightforward to fix those.

@artntek
Copy link
Collaborator Author

artntek commented Oct 19, 2023

As I work through getting everything up and running, I'm collecting the values I've had to override in a separate yaml file. Most of the overrides are because of the namespace being hardcoded (d1index), and while some of those will always need to be overridden manually, there is scope to auto-configure some of them.

There are also the credentials for rmq and solr that need to be moved to secrets

@artntek
Copy link
Collaborator Author

artntek commented Nov 30, 2023

Done - see branch feature-49-subchart-changes-image-auto-publish, resulting in PRs:
#50
#53
#55
#56
#57
...plus corresponding work in metacat feature-1665-misc-fixes branch

@artntek artntek closed this as completed Nov 30, 2023
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

No branches or pull requests

2 participants