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

v3.0.0 #34

Merged
merged 39 commits into from
Mar 22, 2024
Merged

v3.0.0 #34

merged 39 commits into from
Mar 22, 2024

Conversation

BWibo
Copy link
Member

@BWibo BWibo commented Feb 26, 2024

@BWibo BWibo self-assigned this Feb 26, 2024
@BWibo
Copy link
Member Author

BWibo commented Feb 27, 2024

@gislab-augsburg, @klml: I just released 3.0.0-beta1.
Please go through the CHANGELOG and test the new version. When everything is fine, we can release 3.0.0.

Note: I incuded some dependency updates. Please make sure to go through the changelog before testing this in an existing setup. The Solr upgrade should not require any additional action or cause problem, but make sure to make backups and read the docs first.

@BWibo BWibo marked this pull request as ready for review February 27, 2024 14:54
@BWibo BWibo added semantic-release type: fix Iterations on existing features or infrastructure. type: feature Brand new functionality, features, pages, workflows, endpoints, etc. state: feedback required Additional user feedback is required to proceed. labels Feb 27, 2024
@gislab-augsburg
Copy link

@BWibo :
I tested release sddi-ckan-3.0.0-beta1. Whith disabling the solr init container in our values with

solr:	
  initContainers: []

it works perfectly. No problems with the new Solr version or the other the upgrades.

So from our side, you can can release 3.0.0 🚀

@BWibo
Copy link
Member Author

BWibo commented Mar 1, 2024

@gislab-augsburg: OK, really cool, thx for testing.

Did you apply the new version to an instance with content? I'm asking to make sure the new Solr does migrate without issues.

One more thing before I make the release. Can you provide a minimal example of the values that work on OpenShift.
Just strip any sensitive data and replace all URLs with something like: "https://www.test.de".
I would like to add an example of how to get this running on OpenShift to the docs here: https://github.com/tum-gis/sddi-ckan-k8s/tree/main/examples

@BWibo
Copy link
Member Author

BWibo commented Mar 1, 2024

Just post the anonymized values.yml here. I'll document it and add the example.

@gislab-augsburg
Copy link

gislab-augsburg commented Mar 4, 2024

@BWibo :
Yes, I applied the new version to an instance with content. I ran ckan search-index rebuild and the new Solr version worked without issues. Actually, I first tried the default Solr version:

image:
# -- [Image repository](https://kubernetes.io/docs/concepts/containers/images/)
repository: ckan/ckan-solr
# -- [Image pull policy](https://kubernetes.io/docs/concepts/containers/images/#image-pull-policy)
pullPolicy: IfNotPresent
# -- Overrides the image tag whose default is the chart `appVersion`.
tag: ""

Then changed to tag: 2.9-solr9-spatial because we had tag: 2.9-solr8-spatial before. Both, the default version and the 2.9-solr9-spatial version, are working without any issues.

I will provide the example values for OpenShift the next days.

@BWibo
Copy link
Member Author

BWibo commented Mar 11, 2024

@gislab-augsburg Hey, I wanted to ask about the example file. I would like to include this in the release, so I can point users to the example for OpenShift usage, for which we add support in this release. Thx in advance.

@gislab-augsburg
Copy link

Hey @BWibo,
Sorry for delay. Please see attached draft for minimal example values for openShift: openshift.zip

Some comments on it:

  • I did not include the extra volumes, volume mounts and CMs for local licenses, local .ini and custom ckan-schema because this is not OpenShift but LHM specific. If you are interested, I can provide an example of these, too.
  • We use route instead of ingress at the moment, so I included our local route template which we use with an extra chart and the value ckan.host.
  • We do not use datapusher, but replace it with ckanext-xloader which is installed in our docker image sddi-urban.
    Datapusher raised an OpenShift related error and the pod failed when we included it at the beginning. We did not resolve the error but decided to just disable datapusher and use xloader.

Error in datapusher pod:

*** Starting uWSGI 2.0.19.1 (64bit) on [Fri Sep 29 13:42:09 2023] ***
compiled with version: 10.2.1 20201203 on 01 March 2023 09:19:15
os: Linux-4.18.0-372.64.1.el8_6.x86_64 #1 SMP Thu Jun 29 14:42:09 EDT 2023
nodename: datapusher-5b9c686959-k9s87
machine: x86_64
clock source: unix
detected number of CPU cores: 8
current working directory: /srv/app
detected binary path: /usr/bin/uwsgi
!!! no internal routing support, rebuild with pcre support !!!
cannot setgid() as non-root user
  • We do not use certIssuer. Didn't try that at all.

@BWibo BWibo merged commit fa1d1df into main Mar 22, 2024
1 check passed
@BWibo BWibo deleted the release/3.0.0 branch March 22, 2024 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semantic-release state: feedback required Additional user feedback is required to proceed. type: feature Brand new functionality, features, pages, workflows, endpoints, etc. type: fix Iterations on existing features or infrastructure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants