-
-
Notifications
You must be signed in to change notification settings - Fork 663
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
FTS reinstallation is broken #2326
Comments
@Ark74 Please have a look at this, I have no clue (or time). :) |
Hello!
Sorry I was out the whole day, I'll be taking a look shortly.
Cheers!
El 16/05/22 a las 14:43, Daniel Hansson escribió:
…
@Ark74 <https://github.com/Ark74> Please have a look at this, I have
no clue (or time). :)
—
Reply to this email directly, view it on GitHub
<#2326 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABYZQJSHDYPSCEJ2OJNSW7LVKKQPVANCNFSM5WCRXGYQ>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I guess the first problem is here, this volume is not been stopped and deleted correctly, as it is still running.
|
@enoch85 fixed! |
This is just when starting the default container we rewrite it, that's the reason to use docker-compose instead of docker images as the resulting container is a custom one, all security gets rewritten and changes applied here: Lines 240 to 251 in e3721d4
and here: Lines 198 to 205 in e3721d4
So no demo at all. |
Thanks @Ark74! As always, good work. |
Dear enoch85 and Ark74, this is what I am getting after reinstalling. Even purging docker doesn't help. ERROR: The Compose file './docker-compose.yml' is invalid because: When I remove the name:fts_os-data from /opt/opensearch/docker-compose.yml I get the node installed but then I get this after executing sudo -u www-data php /var/www/nextcloud/occ fulltextsearch:index An unhandled exception has been thrown: |
Baah, I should have tested first. |
@vigorio what ubuntu version are you running? |
@Ark74 20.04.4 LTS |
I see, please run fulltextsearch.sh with that change (#2329), and select: "Uninstall fulltextsearch", then install again. |
The final fix is now merged. @vigorio Please confirm that it works to both reinstall, and uninstall. Thanks! |
His current state seems to be failing, it should be better to first Uninstall then Install, and test the Reinstall afterwards. |
@Ark74 I have tested reinstall, uninstall and install and even purging the whole docker instance with images config files etc and I still get the same error… |
@vigorio this issue was opened for a docker issue. I would invite fill another bug with a step by step details on how to reproduce the error. |
@vigorio Please open another issue. Thanks! |
Issue has been resolved. Seems like my installation has been pulling the old script?! Everything works as expected. Thank you! |
That make sense, thanks for reporting back. |
Steps To Reproduce
Expected Result
Everything should ve removed and it should reinstall 100% fresh
Actual Result
Screenshots, Videos, or Pastebins
@Ark74
I also wonder why we are using a demo installer?
Additional Context
No response
Build Version
latest (24.0.0)
Environment
By using the scripts
Environment Details
No response
The text was updated successfully, but these errors were encountered: