Bug 1252823 - Please include plugin script
Please include plugin script
Product: Fedora
Classification: Fedora
Component: elasticsearch (Show other bugs)
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: jiri vanek
Fedora Extras Quality Assurance
: 1226445 (view as bug list)
Depends On:
  Show dependency treegraph
Reported: 2015-08-12 05:57 EDT by Pavel Alexeev
Modified: 2016-01-19 18:27 EST (History)
5 users (show)

See Also:
Fixed In Version: elasticsearch-1.7.1-3.fc23
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2016-01-19 18:27:55 EST
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Fix for 1252823 and 1296308 bugs (7.69 KB, patch)
2016-01-06 16:21 EST, Pavel Alexeev
no flags Details | Diff

  None (edit)
Description Pavel Alexeev 2015-08-12 05:57:56 EDT

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?
Comment 1 Zbigniew Jędrzejewski-Szmek 2015-08-12 13:20:33 EDT
I see how that could be useful. Jiri, what do you think?
Comment 2 jiri vanek 2015-08-13 02:36:16 EDT
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...
Comment 3 Pavel Alexeev 2015-12-08 17:01:39 EST
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?
Comment 4 jiri vanek 2015-12-09 02:18:55 EST
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:)

Comment 5 jiri vanek 2015-12-16 05:23:13 EST
*** Bug 1226445 has been marked as a duplicate of this bug. ***
Comment 6 Pavel Alexeev 2016-01-06 16:16:21 EST
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).
Comment 7 Pavel Alexeev 2016-01-06 16:21 EST
Created attachment 1112248 [details]
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.
Comment 8 jiri vanek 2016-01-07 10:41:28 EST
Yup sure, I added you to maintainers. Go on and fix what you need. Considering it will be kept working :)
Comment 9 Pavel Alexeev 2016-01-10 14:00:43 EST
Thank you!
Sure I'll try be careful.
Comment 10 Fedora Update System 2016-01-10 14:54:40 EST
elasticsearch-1.7.1-3.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-1885bffc8c
Comment 11 Fedora Update System 2016-01-11 08:52:53 EST
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
Comment 12 Fedora Update System 2016-01-19 18:27:48 EST
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.

Note You need to log in before you can comment on or make changes to this bug.