Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 564604 - =app-misc/elasticsearch-2.1.0 version bump
Summary: =app-misc/elasticsearch-2.1.0 version bump
Status: RESOLVED TEST-REQUEST
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Ferenc Erki
URL: https://www.elastic.co/guide/en/elast...
Whiteboard:
Keywords:
Depends on:
Blocks: 564606
  Show dependency tree
 
Reported: 2015-11-01 07:31 UTC by Tomáš Mózes
Modified: 2015-12-17 14:00 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
app-misc/elasticsearch-2.0.0.ebuild (elasticsearch-2.0.0.ebuild,1.46 KB, text/plain)
2015-11-01 17:38 UTC, Julian Simioni
Details
files/elasticsearch.init4 (elasticsearch.init4,2.27 KB, text/plain)
2015-11-01 17:39 UTC, Julian Simioni
Details
app-misc/elasticsearch-2.0.0.ebuild (elasticsearch-2.0.0.ebuild,1.46 KB, text/plain)
2015-11-01 17:55 UTC, Julian Simioni
Details
files/elasticsearch.init5 (elasticsearch.init5,2.27 KB, text/plain)
2015-11-01 17:55 UTC, Julian Simioni
Details
files/elasticsearch.service5 (elasticsearch.service5,289 bytes, text/plain)
2015-11-01 17:56 UTC, Julian Simioni
Details
elasticsearch-2.1.0.ebuild (elasticsearch-2.1.0.ebuild,1.53 KB, text/plain)
2015-12-08 10:41 UTC, Ferenc Erki
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tomáš Mózes 2015-11-01 07:31:23 UTC
https://www.elastic.co/blog/elasticsearch-2-0-0-released

This is needed by other stuff like elasticsearch-py 2.0, logstash 2.0...
Comment 1 Julian Simioni 2015-11-01 17:38:43 UTC
Created attachment 415890 [details]
app-misc/elasticsearch-2.0.0.ebuild

elasticsearch-2.0.0 ebuild that Works For Me(tm), would love others to test.

Changes are simple:
* URI updated
* unneeded pruning of library no longer shipped is removed
* change to quoting in init file (the quotes were being picked up as part of the path by Elasticsearch, perhaps there's a better way to fix this)
Comment 2 Julian Simioni 2015-11-01 17:39:20 UTC
Created attachment 415892 [details]
files/elasticsearch.init4
Comment 3 Julian Simioni 2015-11-01 17:55:28 UTC
Created attachment 415894 [details]
app-misc/elasticsearch-2.0.0.ebuild

New ebuild/files that don't conflict with files for elasticsearch-1.7.1.ebuild
Comment 4 Julian Simioni 2015-11-01 17:55:56 UTC
Created attachment 415896 [details]
files/elasticsearch.init5
Comment 5 Julian Simioni 2015-11-01 17:56:22 UTC
Created attachment 415898 [details]
files/elasticsearch.service5

Simple copy of service file
Comment 6 Ferenc Erki 2015-11-02 00:32:11 UTC
@Julian: thanks for your efforts! I was also playing with 2.0.0 when RC1 came out, but it didn't worked fully at first try.

- service file: it doesn't seem to be changed, so I don't think the name shuold be bumped to service5 from service4

- initscript: I think you just found my initial issues with RC1 (quotes get picked up by elasticsearch as part of path), but if we omit the quotes, then pathes with spaces in it will cause problem. I'll look into how we can resolve this.

- ebuild: at first glance it looks OK, thanks for the cleanup! I'm going to give it a test round.
Comment 7 Tomáš Mózes 2015-11-02 02:44:31 UTC
A side note about java:
https://www.elastic.co/guide/en/elasticsearch/reference/current/setup.html

How about we prioritize Java 8?
Comment 8 Ferenc Erki 2015-11-02 15:24:07 UTC
Yeah, upstream states it requires Java 8, but two lines later they recommend Java 8 or Java 7. I might send an update to them about clarifying this :)

@Tomas: what do you mean by prioritizing Java 8?
Comment 9 Tomáš Mózes 2015-11-02 19:43:24 UTC
(In reply to Ferenc Erki from comment #8)
> @Tomas: what do you mean by prioritizing Java 8?

Currently we have:

RDEPEND="|| ( virtual/jre:1.7 virtual/jre:1.8 )"

If I'm not mistaken this means that if no virtual/jre is available, it will install 1.7.
Comment 10 Ferenc Erki 2015-11-09 15:33:34 UTC
@Tomas:, ah, yes, preferring Java 8 is a good idea, thanks! Though I didn't find documentation about this behaviour of RDEPEND yet. Will test it.
Comment 11 Tomáš Mózes 2015-12-03 19:47:19 UTC
@Ferenc, can we push to the repo (we can kindly consult the proxy team)?
Comment 12 Ferenc Erki 2015-12-08 10:41:17 UTC
Created attachment 418756 [details]
elasticsearch-2.1.0.ebuild
Comment 13 Ferenc Erki 2015-12-08 10:55:37 UTC
Sorry for the delay, I attached an ebuild for elasticsearch-2.1.0 now, based on the 2.0.0 ebuild by Julian Simioni (thanks!).

The initscript (elasticsearch.init5) attached by Julian can be used too. The service file does not need any change.

List of changes:
- updated SRC_URI
- preferring Java 8 if not installed yet
- cleaned up src_prepare (much less stuff needs to be dropped from upstream packages)
- creating new required directories (/etc/elasticsearch/scripts and /usr/share/elasticsearch/plugins)
- updated initscript
Comment 14 Ian Delaney (RETIRED) gentoo-dev 2015-12-16 10:58:59 UTC
With personal consent from the maintainer Chainsaw, dropping his maintainer tag and setting proxy maintained under proxy-maintainers.(In reply to Ferenc Erki from comment #12)
> Created attachment 418756 [details]
> elasticsearch-2.1.0.ebuild

Would you also like to nominate which old versions to drop.
Currently have
Available versions:  (~)1.6.2^s (~)1.7.1^s (~)1.7.3^s

add the new makes four. Also Can you consider if you'd like to have one put forward for stabilisation, and select the candidate.

commit f279fce9617b2e3ddbf3ef99df8f65629617e959
Author: Ian Delaney <idella4@gentoo.org>
Date:   Wed Dec 16 17:48:07 2015 +0800

    app-misc/elasticsearch: bump to version 2.1.0
    
    ebuild written by the proxy maintainer, submitted via the gentoo bug
    Also informed has been runtested by 'Patrick'
    
    Gentoo bug: #564604
    
commit 20c1bcbaa6346c6e4643f50b904deaa8e9c06af2
Author: Ian Delaney <idella4@gentoo.org>
Date:   Wed Dec 16 17:07:00 2015 +0800

    app-misc/elasticsearch: Drop set maintainer, add proxy-maintainers herd
    
    Arrangement made directly by consult with Chainsaw via irc channel today,
    Dec 16. proxy maintainer remains unchanged

will leave in state of IN_PROGRESS for now. Ideally, removing old should be done though a fresh bug
Comment 15 Ian Delaney (RETIRED) gentoo-dev 2015-12-16 11:23:27 UTC
commit 5f03e659c405fcd0baa49b780a71a711376bee7f
Author: Ian Delaney <idella4@gentoo.org>
Date:   Wed Dec 16 19:21:43 2015 +0800

    app-misc/elasticsearch: add the elasticsearch.init5 script to FILESDIR
    
    Notes from the bug;
    ebuild based on the 2.0.0 ebuild by Julian Simioni
    updated SRC_URI, preferring Java 8 if not installed yet
    cleaned up src_prepare.
    updated initscript (added here)
    
    Gentoo bug: 564604
Comment 16 Ferenc Erki 2015-12-16 11:29:42 UTC
(In reply to Ian Delaney from comment #15)
> commit 5f03e659c405fcd0baa49b780a71a711376bee7f
> Author: Ian Delaney <idella4@gentoo.org>
> Date:   Wed Dec 16 19:21:43 2015 +0800
> 
>     app-misc/elasticsearch: add the elasticsearch.init5 script to FILESDIR

I believe the systemd `.service5` file contents has been pushed to the git repo on that commit as an `.init5` file by mistake.
Comment 17 Ferenc Erki 2015-12-16 11:32:38 UTC
(In reply to Ian Delaney from comment #14)
>Ideally, removing old should be done though a fresh bug

Opened bug 568418 for that, thanks!
Comment 18 Josh Holmer 2015-12-16 17:40:38 UTC
(In reply to Ferenc Erki from comment #16)
> (In reply to Ian Delaney from comment #15)
> > commit 5f03e659c405fcd0baa49b780a71a711376bee7f
> > Author: Ian Delaney <idella4@gentoo.org>
> > Date:   Wed Dec 16 19:21:43 2015 +0800
> > 
> >     app-misc/elasticsearch: add the elasticsearch.init5 script to FILESDIR
> 
> I believe the systemd `.service5` file contents has been pushed to the git
> repo on that commit as an `.init5` file by mistake.

I believe so as well. Cannot start elasticsearch-2.1.0 under openrc because of this. (service: Exec format error)
Comment 19 Ian Delaney (RETIRED) gentoo-dev 2015-12-17 14:00:40 UTC
commit 1a0ea0914d676d18ddc19c8f7b48cd6c22f68cf9
Author: Ian Delaney <idella4@gentoo.org>
Date:   Thu Dec 17 21:56:10 2015 +0800

    app-misc/elasticsearch: switch to correct file elasticsearch.init5
    
    file mistakenly mixed with ".service5", now obsoleted in the gentoo bug
    
    Gentoo bug: 564604

Sorry for the mixup