https://bugzilla.redhat.com/show_bug.cgi?id=1252823
Bug ID: 1252823 Summary: Please include plugin script Product: Fedora Version: 22 Component: elasticsearch Assignee: jvanek@redhat.com Reporter: pahan@hubbitus.info QA Contact: extras-qa@fedoraproject.org CC: bobjensen@gmail.com, java-sig-commits@lists.fedoraproject.org, jvanek@redhat.com, pbrobinson@gmail.com, zbyszek@in.waw.pl
Hello.
In repack.sh you delete "elasticsearch elasticsearch.bat elasticsearch-service-mgr.exe elasticsearch-service-x64.exe elasticsearch-service-x86.exe plugin plugin.bat service.bat".
I understand what binaries must be stripped. But what with plugin, which is just shell script? I assume it is not sane way install such plugions which is not in rpm form, and it is main reason of excludin, is not? But may we let it on administrator choose when desired plugin is not packaged?
Could you please include that script in package as it done in upstream rpm?
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
--- Comment #1 from Zbigniew Jędrzejewski-Szmek zbyszek@in.waw.pl --- I see how that could be useful. Jiri, what do you think?
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
--- Comment #2 from jiri vanek jvanek@redhat.com --- I have nothing against adding shell for adding plugins. but... Maybe I miss something, but upstream plugin.sh as it is, is unusable. We have to rewrite it from scratch or heavily patch to fit fedora installation. Same as was necessary with upstream launching script:(
Pavel, are you volunteer? I even dont knw how to test whether it works. I'm using elastic for really simple tasks so I'm happy with current state of things. If people needs more, they have to help. (as eg. zbyszek do)
Upstream rpm have everything in single directory. And all dependences built in. Fedora can't do that and upstream is exactly not cooperative...
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
--- Comment #3 from Pavel Alexeev pahan@hubbitus.info --- Jiri thank you for the answer.
I'm use several plugins for elasticsearch like kopf, gui, Russian morphology, etc. Do we ever have some guidelines how to package such plugins for elastic in rpm? And that is very useful even for Fedora installation install it as plugins via upstream mechanism. As I understand there only paths adjustment needed. Are you willing I try provide patch for that?
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
--- Comment #4 from jiri vanek jvanek@redhat.com --- No guidelines at all :)
I can see are 2x2 approaches I would guess. I font know which will be more valid, so yours usecase - first usecase. (global space plugin x user's space plugin) x (rpm packed plugin x upstream-like handled plugin)
I guess that global space + upstream-like handled would be most simple, but as I told. Your usecase is the way to go, as whole this thing is goig to evole...now. (and you may be the only user for some time...)
If you will try to provide patch, I will try to include it in RPMS:)
TY!
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
jiri vanek jvanek@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jimmyjones2@gmx.co.uk
--- Comment #5 from jiri vanek jvanek@redhat.com --- *** Bug 1226445 has been marked as a duplicate of this bug. ***
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
Pavel Alexeev pahan@hubbitus.info changed:
What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://github.com/elastic/ | |elasticsearch/issues/11797
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
Pavel Alexeev pahan@hubbitus.info changed:
What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugzilla.redhat.com | |/show_bug.cgi?id=1296308
--- Comment #6 from Pavel Alexeev pahan@hubbitus.info --- Proper solution for that should come from https://github.com/elastic/elasticsearch/issues/11797
Now I could provide simple solution requested initially. Please revisit attached patch. It also fixes bz#1296308 (which is very annoying even for tests).
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
--- Comment #7 from Pavel Alexeev pahan@hubbitus.info --- Created attachment 1112248 --> https://bugzilla.redhat.com/attachment.cgi?id=1112248&action=edit Fix for 1252823 and 1296308 bugs
If you want I could push it myself.
I also request access to package to co-maintain it if you are willing.
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
--- Comment #8 from jiri vanek jvanek@redhat.com --- Yup sure, I added you to maintainers. Go on and fix what you need. Considering it will be kept working :)
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
Pavel Alexeev pahan@hubbitus.info changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|22 |23
--- Comment #9 from Pavel Alexeev pahan@hubbitus.info --- Thank you! Sure I'll try be careful.
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
Pavel Alexeev pahan@hubbitus.info changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
--- Comment #10 from Fedora Update System updates@fedoraproject.org --- elasticsearch-1.7.1-3.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-1885bffc8c
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #11 from Fedora Update System updates@fedoraproject.org --- elasticsearch-1.7.1-3.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-1885bffc8c
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
--- Comment #12 from Fedora Update System updates@fedoraproject.org --- elasticsearch-1.7.1-3.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=1252823
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |elasticsearch-1.7.1-3.fc23 Resolution|--- |ERRATA Last Closed| |2016-01-19 18:27:55
java-sig-commits@lists.fedoraproject.org