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 Solr #243

Closed
nigelgbanks opened this issue Sep 7, 2022 · 6 comments
Closed

Upgrade Solr #243

nigelgbanks opened this issue Sep 7, 2022 · 6 comments

Comments

@nigelgbanks
Copy link
Contributor

nigelgbanks commented Sep 7, 2022

Currently, Solr is using arguments for ps that are not supported by busybox (therefore also Alpine).

idc-solr-1        | [cont-init.d] 02-setup-environment.sh: exited 0.
idc-solr-1        | [cont-init.d] 02-solr-setup.sh: executing... 
idc-solr-1        | [cont-init.d] 02-solr-setup.sh: exited 0.
idc-solr-1        | [cont-init.d] done.
idc-solr-1        | [services.d] starting services
idc-solr-1        | [services.d] done.
idc-solr-1        | ps: unrecognized option: p
idc-solr-1        | BusyBox v1.33.1 () multi-call binary.
idc-solr-1        | 
idc-solr-1        | Usage: ps [-o COL1,COL2=HEADER] [-T]
idc-solr-1        | 
idc-solr-1        | Show list of processes
idc-solr-1        | 
idc-solr-1        | 	-o COL1,COL2=HEADER	Select columns for display
idc-solr-1        | 	-T			Show threads
idc-solr-1        | This script relies on a version of ps that supports the -p flag.
idc-solr-1        | 
idc-solr-1        | Please install a POSIX compliant version and try again.
idc-solr-1        | [cont-finish.d] executing container finish scripts...
idc-solr-1        | [cont-finish.d] done.
idc-solr-1        | [s6-finish] waiting for services.
idc-solr-1        | [s6-finish] sending all processes the TERM signal.
idc-solr-1        | [s6-finish] sending all processes the KILL signal and exiting.
idc-solr-1 exited with code 0

This has been resolved here.

Upgrading to versions 8.11.2 and 9.1 or greater should resolve the issue.

It is a low-priority issue, as only a warning is printed. Since the process always starts from a clean slate, no existing process for solr should be present.

Doesn't seem to be an issue on our older version of solr presently, that being said, solr should be updated regardless.

@nigelgbanks nigelgbanks self-assigned this Sep 7, 2022
@nigelgbanks
Copy link
Contributor Author

This should wait until after #215, as upgrading Solr will constitute a major release, as folks will need to update their Drupal site as well.

@DonRichards
Copy link
Member

@nigelgbanks Is this not the issue causing my exit code?

@nigelgbanks
Copy link
Contributor Author

@DonRichards I can't actually replicate this on the 1.0.7 release, do you have a custom solr image?

@nigelgbanks
Copy link
Contributor Author

My bad for not reading the whole log, I just assumed it was the same as the activemq issue.

@DonRichards
Copy link
Member

Yes. We forked this a while back and I'm now playing catchup.

@nigelgbanks
Copy link
Contributor Author

nigelgbanks commented Sep 7, 2022

Oh I'd check your image then, quick fix might be to install the GNU version of ps i.e. apk add procps or changing your version of solr.

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

No branches or pull requests

3 participants