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

Upgrade to Solr 9.3.0 #108

Merged
merged 9 commits into from
Oct 20, 2023
Merged

Upgrade to Solr 9.3.0 #108

merged 9 commits into from
Oct 20, 2023

Conversation

extracts
Copy link
Contributor

@extracts extracts commented Oct 6, 2023

PR für #35

@extracts extracts requested a review from j3nsch October 6, 2023 08:46
@j3nsch
Copy link
Member

j3nsch commented Oct 6, 2023

Danke, das sieht gut aus. Wie sind die Ergebnisse bei den Application Tests?

@extracts
Copy link
Contributor Author

extracts commented Oct 6, 2023

Wie sind die Ergebnisse bei den Application Tests?

DIe Application Tests laufen erfolgreich durch und ich sehe (ausser PHPUnit Deprecation Warnings) keine Solr-spezifischen Warnings:

https://github.com/OPUS4/application/actions/runs/6430171390/job/17460649008

@extracts extracts linked an issue Oct 6, 2023 that may be closed by this pull request
@j3nsch
Copy link
Member

j3nsch commented Oct 6, 2023

Gut, dann mach mal ein entsprechendes Ticket für Application auf. Die 4.8.x Versionen von OPUS 4 müssen PHP 7 kompatibel sein. Erst ab OPUS 4.9 wird nur noch PHP 8 unterstützt. Das heißt abhängig von der PHP Version müsste mit unterschiedlichen Solr-Versionen getestet werden. In Vagrant können wir uns auf PHP 8 festlegen. Für GitHub geben wir die PHP Version ja in einer "Matrix" an. Da kann man auch noch weitere Parameter spezifizieren (schau mal in die Doku) und so auch die SOLR Version festlegen. Das muss dann an die entsprechenden Skripte usw. weitergereicht werden. Alternativ könnte man schauen, ob die PHP 7 Version von Search auch mit Solr 9 läuft.

Wenn es keine Lösung mit akzeptable Aufwand gibt, kommt die SOLR 9 Kompatiblität erst mit OPUS 4.9.

@j3nsch
Copy link
Member

j3nsch commented Oct 6, 2023

Probiere mal so etwas in der Matrix-Konfiguration:

strategy:
  matrix:
    php-versions: [{php: '7.1', solr: '7.7.3'}, {php: '8.1', solr: '9.3.0'}]

Mit den entsprechenden Anpassungen am Rest. "php-versions" sollte dann umbenannt werden.

@extracts
Copy link
Contributor Author

extracts commented Oct 6, 2023

Ticket & PR für application

conf/solrconfig.xml Outdated Show resolved Hide resolved
@extracts
Copy link
Contributor Author

extracts commented Oct 11, 2023

Folgende Punkte sind mir noch aufgefallen:

  1. Mit Vagrant lässt sich die Solr-Admin-Oberfläche vom Host OS über http://localhost:9983 momentan nicht aufrufen (ERR_CONNECTION_RESET). Hierzu habe ich bislang noch keine Lösung gefunden...

  2. Die Download-Geschwindigkeit von https://archive.apache.org/dist/solr/solr/ ist deutlich langsamer als von https://dlcdn.apache.org/solr/solr/. In den GitHub Workflow Tests scheint dies kein Problem zu sein, bei einer lokalen Installation mit Vagrant kann der Download allerdings recht lange dauern. Daher habe ich den Solr-Download für das Vagrantfile nun so ergänzt, dass zumindest ein Fortschrittsbalken angezeigt wird.

@j3nsch
Copy link
Member

j3nsch commented Oct 12, 2023

  1. Mit Vagrant lässt sich die Solr-Admin-Oberfläche vom Host OS über http://localhost:9983 momentan nicht aufrufen (ERR_CONNECTION_RESET). Hierzu habe ich bislang noch keine Lösung gefunden...

Das ist ein Ticket, oder?

@extracts
Copy link
Contributor Author

  1. Mit Vagrant lässt sich die Solr-Admin-Oberfläche vom Host OS über http://localhost:9983 momentan nicht aufrufen (ERR_CONNECTION_RESET).

Siehe Ticket #109.

@j3nsch j3nsch merged commit 0cb9324 into 4.8.1 Oct 20, 2023
2 checks passed
@j3nsch j3nsch deleted the upgradeSolr35 branch October 20, 2023 14:52
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

Successfully merging this pull request may close these issues.

Umstieg auf aktuelle Solr Version (9.x)
2 participants