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

Update Hyrax chart #5768

Merged
merged 11 commits into from Aug 24, 2023
Merged

Update Hyrax chart #5768

merged 11 commits into from Aug 24, 2023

Conversation

mcritchlow
Copy link
Contributor

@mcritchlow mcritchlow commented Jul 5, 2022

refs #5357

Depends on:

  • Bump chart to 1.0 samvera-labs/fcrepo-charts#13
  • An updated dassie image for 3.4.1 to use as the chart default image/tag
  • A dassie-worker image for 3.4.1 to use as the chart worker default image/tag (currently no image exists at all in ghcr.io/samvera as far as I can tell)

Upgrade Hyrax Helm chart to 2.0.0

Changes proposed in this pull request:

  • Create a new release of the Hyrax helm chart at version 2.0 (major version bump)
  • Bump all chart dependencies to their latest versions
  • Bump and set chart image.tag for each dependency, with the idea that Add a dependency management bot to Hyrax #5357 will be done soon and automate keeping these up to date
  • Update chart configuration as needed for each dependency upgrade

@mcritchlow mcritchlow changed the title Chart 2dot0 upgrades Update Hyrax Chart to 2.0 Jul 5, 2022
@mcritchlow
Copy link
Contributor Author

mcritchlow commented Jul 6, 2022

Just dropping a note that I tried to publish the new fcrepo chart, but am getting access denied. This feels familiar to when we were initially trying to test publishing the hyrax chart using oci/ghcr.io

$ cd fcrepo-charts
$ helm package ./
$ helm push fcrepo-1.0.0.tgz oci://ghcr.io/samvera
Error: unexpected status: 403 Forbidden

I have successfully pushed Hyrax chart updates in the past, so I'm wondering if it's the submission path/directory or something?

If anyone can help while I'm out today that would be great, otherwise I'll circle back.

Same goes for dassie and dassie-worker images for 3.4.1. We don't have a current dassie image (3.0.1 is most recent) or a dassie-worker (no image at all in ghcr.io) for the chart default images.

cc: @dunn @no-reply @orangewolf

@mcritchlow mcritchlow force-pushed the chart-2dot0-upgrades branch 3 times, most recently from fad72dc to 28547a2 Compare July 12, 2022 14:28
@mcritchlow
Copy link
Contributor Author

Current status:

  • New dassie 4.0 beta images pushed to ghcr and referenced as chart defaults
  • New fcrepo 1.0 chart used as dependency
  • Failing to start on a Valkyrie migration added to dassie 8fea026 with error:
│ db-setup PG::InsufficientPrivilege: ERROR:  permission denied to create extension "uuid-ossp"                                              │
│ db-setup HINT:  Must be superuser to create this extension.  

@mcritchlow mcritchlow marked this pull request as ready for review July 18, 2022 20:20
@dunn dunn force-pushed the chart-2dot0-upgrades branch 2 times, most recently from 83f0913 to 633091c Compare July 18, 2022 20:53
dunn
dunn previously approved these changes Jul 18, 2022
mcritchlow and others added 11 commits August 24, 2023 14:26
- set the image tag to latest rolling debian 11 release
- update `values.yaml`, `ConfigMap`, `Secret`, and `_helpers`
- set the image tag to latest rolling debian 11 release
- update `values.yaml`, `ConfigMap`, `Secret`, and `_helpers` as needed
- add support for custom `externalSolrPort` for folks using an external
  Solr setup
- Pin fcrepo and postgresql images on 12.x, as the version of Fedora that fcrepo supports is not compatible with 13+.
- Update image repository for default image and worker image to `ghcr.io/samvera/dassie` and `ghcr.io/samvera/dassie-worker`
- Update `appVersion` to `3.4.1`
- Use new `fcrepo` chart URL
@dunn dunn changed the title Update Hyrax Chart to 2.0 Update Hyrax chart Aug 24, 2023
@dunn dunn merged commit 0b6b4ea into main Aug 24, 2023
7 checks passed
@dunn dunn deleted the chart-2dot0-upgrades branch August 24, 2023 22:20
@dlpierce dlpierce added the notes-container Release Notes: Docker, Helm, etc label Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notes-container Release Notes: Docker, Helm, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants