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

Invenio versioning #127

Open
jwhitelbl opened this issue Sep 25, 2024 · 3 comments
Open

Invenio versioning #127

jwhitelbl opened this issue Sep 25, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@jwhitelbl
Copy link

Is this helm chart expecting a maximum or minimum version of invenio?

Versioning:
Helm master pulled on sept 25, 2024
container version derived from registry.cern.ch/inveniosoftware/almalinux:latest including default latest invenio from pip (invenio-3.4.1)

Describe the bug

Installer pod outright fails:

jwhite@proxmox0:~/invenio$ kubectl logs install-init-l5vtw -n invenio
Usage: invenio [OPTIONS] COMMAND [ARGS]...
Try 'invenio --help' for help.

Error: No such command 'db'.

Steps to Reproduce

  1. git clone https://github.com/inveniosoftware/helm-invenio.git; cd helm-invenio/charts/invenio
  2. customize overrides.yml for site including local docker repo with above-mentioned versioning for container
  3. helm install -f overrides.yaml -n invenio invenio .
  4. check kubectl logs for installer pod

Expected behavior

installer pod completes without error

@jwhitelbl jwhitelbl added the bug Something isn't working label Sep 25, 2024
@lindhe
Copy link
Contributor

lindhe commented Sep 26, 2024

I'm not sure the init Job works. It's been there since before I did my pretty significant rewrites of the chart and I never used the init Job so maybe it's broken. I might look into it some day, unless someone fixes it before I do.

What I typically do exec into the pod and run the wipe_recreate.sh script. It's similar to what the init Job does.

WORKER_POD_NAME="$(kubectl get pods -l app=worker -o jsonpath='{.items[0].metadata.name}')"
kubectl exec -it "${WORKER_POD_NAME:?}" -- bash -c 'yes | /opt/invenio/src/wipe_recreate.sh'

@jwhitelbl
Copy link
Author

Thanks for that. Do you folks have a recommended base dockerfile for the web/worker/etc? Or even "what packages should I be jamming on top of registry.cern.ch/inveniosoftware/almalinux:latest ? I keep running into paths missing, etc.

jwhite@proxmox0:~/invenio$ kubectl -n invenio exec -it "${WORKER_POD_NAME:?}" -- bash -c 'yes | /opt/invenio/src/wipe_recreate.sh'
bash: line 1: /opt/invenio/src/wipe_recreate.sh: No such file or directory
command terminated with exit code 127
jwhite@proxmox0:~/invenio$ 

@lindhe
Copy link
Contributor

lindhe commented Sep 27, 2024

I have requested that they set a "known good image" by default, so it's easier to get started. I think it was delayed because v12 was not released. Now it is, so perhaps it's possible to revisit that topic.

When I've been testing things, I've used ghcr.io/inveniosoftware/demo-inveniordm/demo-inveniordm@sha256:2193abc2caec9bc599061d6a5874fd2d7d201f55d1673a545af0a0406690e8a4 I think that should work! Except maybe for this bug which (if it's still an issue) is resolved by removing the health probes for web.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants