Conversation
b27ca26 to
877614d
Compare
|
Copied some more stuff from the rpb java update, now the tests result in the following as before: @fsteeg maybe this could be a fix adjustment, but this is not urgent. |
Doesn't work for me ( |
1c5c4d0 to
5ec32c7
Compare
|
@fsteeg
This works for me |
It's |
Okay, that is new to me, in context of lobid-organisations and -resources, the build of the web app is not done with |
|
|
|
I think we should stick with our process and do a separate functional review (e.g. by @acka47) before code review. |
|
It seems functional review is not complete, see #665 (comment). |
It is, see #665 (comment) |
|
Ah, I see, I had to delete the old local embedded Elasticsearch index to make it work ( |
|
@Phu2 can you deploy and merge? |
|
I tried to deploy on test (q13):
errors shown up in |
|
I'll do a rollback for now to bring up the test instance again. |
|
@fsteeg can you help, I am not sure how to solve this. |
|
Needed to rebuild the embedded Elasticsearch index (triggered by deleting the old), started with:
Hint was this in the logs:
Now running on test: https://test.nwbib.de/ Will deploy to production and merge. |
Quick test with regard to updating the java version to 11
Related to #665
Maybe it needs some adjustments inspired by https://github.com/hbz/rpb/pull/130/changes#diff-75ff37996f3bf481e0e9b6566d5cd9eec8c4a09a75e76a6470edcec1c9e0142dR28