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

Full Text Search script breaks installation during first setup #649

Closed
nxtester opened this issue Sep 4, 2018 · 18 comments
Closed

Full Text Search script breaks installation during first setup #649

nxtester opened this issue Sep 4, 2018 · 18 comments
Labels

Comments

@nxtester
Copy link

nxtester commented Sep 4, 2018

As a routined tester of this wonderful script I encountered two serious glitches trying to install latest NC VM. Double verified, so it might be serious:

  1. "Read Error" during installation of "Nextcloud Talk" if system ask "Do you want change port?" (see screenshot)

putty_2018-09-04_20-46-00

  1. At the end of the main script (and after all components have been installed) system will ask for a new UNIX password. At this stage script will fall in a endless loop and script can't finish properly and has to be stopped by CTRL+C

putty_2018-09-04_20-52-06

If I restart the so stopped installation, script will start again and has to be stopped to prevent damaging existing installation. Anyway it seems that all is working properly.

Assumed faulty part of https://github.com/nextcloud/vm/blob/master/nextcloud-startup-script.sh:

Line 502 - 527

Q: Can I manually finish those steps which have been left out because of the endless loop? I need manually execute all lines from 502 until the end of script.

Best regards,

nxtester

@enoch85
Copy link
Member

enoch85 commented Sep 4, 2018

@nxtester Thanks for your report!

Just tested and everything works here. Not sure what could be wrong: https://cloud.techandme.se/s/Lp4W5426zmei8HQ

@enoch85
Copy link
Member

enoch85 commented Sep 4, 2018

Did you install any other apps during the setup? There have been issues with Full Text Search before.

@enoch85
Copy link
Member

enoch85 commented Sep 4, 2018

I could reproduce it if I installed ALL apps.

Which apps did you install to be exact?

@enoch85 enoch85 added the bug label Sep 4, 2018
@enoch85
Copy link
Member

enoch85 commented Sep 4, 2018

OK, I didn't install Full Text Search this time, and it worked. Will run it once more and see if that's the issue here.

@enoch85
Copy link
Member

enoch85 commented Sep 4, 2018

Yup, same error as last time. FTS script breaks the password change.

@Ark74 Please have a look at this. Indexing FTS does something nasty with the shell.

@enoch85
Copy link
Member

enoch85 commented Sep 4, 2018

@Ark74
Copy link
Collaborator

Ark74 commented Sep 4, 2018

Weird.
Sounds like a printf "\n" is being passed to password change.

I'll have a look later today.

@enoch85 enoch85 changed the title Latest NC script breaks during installation Full Text Search script breaks installation during first setup Sep 5, 2018
@enoch85
Copy link
Member

enoch85 commented Sep 5, 2018

@Ark74 I made some minor changes to the any_key function and corrected some vars. I don't know if it fixed it, but you could try.

@nxtester
Copy link
Author

nxtester commented Sep 5, 2018

Hi Guys,
thanks for your fast response.
I installed: Talk, Collabora, Full-Text-Search, Password-Manager
Allow one more question: Do I have to repeat whole installation from the scratch if this issue will be fixed or is it possible to finish VM installation properly by manual investigation?

Fresh install would be no problem but Let's encrypt will soon block my domains for a while because of too much tries :-)

Thanks for your appreciated work!
Best regards,
nxtester

@enoch85
Copy link
Member

enoch85 commented Sep 5, 2018

@nxtester Full Text Search is the failing one here.

Allow one more question: Do I have to repeat whole installation from the scratch if this issue will be fixed or is it possible to finish VM installation properly by manual investigation?

I'd recommend a clean install for a clean experience.

As I've changed some $vars and functions, could you please try again without enabling SSL? Just do a local installation with only Full Text Search to see if the issue is solved.

Thanks!

@nxtester
Copy link
Author

nxtester commented Sep 5, 2018

OK, tried it but script stopped by error before SSL part:

putty_2018-09-05_16-45-29

Thanks!

@enoch85
Copy link
Member

enoch85 commented Sep 5, 2018

@nxtester It fails because the curl command isn't run and the vars and functions isn't imported. You're testing it the wrong way.

Can you please tell me how you did the test?

@Ark74
Copy link
Collaborator

Ark74 commented Sep 5, 2018

@enoch85 did you enable debug by default?

@enoch85
Copy link
Member

enoch85 commented Sep 5, 2018

@Ark74 No?

@Ark74
Copy link
Collaborator

Ark74 commented Sep 5, 2018

Weird installing fts (default) the output seems like debug ON.

@Ark74
Copy link
Collaborator

Ark74 commented Sep 5, 2018

Ok, I can confirm that latest master keeps looping the "passwd: Authentication token manipulation error".
Looking into it.

@nxtester
Copy link
Author

nxtester commented Sep 5, 2018

Hello Guys,

I simply tested it by running your script according to https://github.com/nextcloud/vm.
Please tell me if I should test it by a different way.

Greetings

enoch85 pushed a commit that referenced this issue Sep 5, 2018
@enoch85 enoch85 closed this as completed in c179edb Sep 5, 2018
@enoch85
Copy link
Member

enoch85 commented Sep 5, 2018

Yup, I fixed it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants