Re: IGNITE-7107 Apache Ignite RPM packages

Just as a follow up for the community. We had a private conversation with Peter and decided that RPMs will be initially hosted in Ignite root. Going forward (Ignite 2.5) we might to store RPMs and DEBs on JFrog Bintray [1] (this is what Cassandra does for its DEB packages [2]). So, Peter pleas

Re: IGNITE-7107 Apache Ignite RPM packages

Hi, Denis. Proposed layout will require changes to release procedure. Also, I’d like to add to this layout symlink to latest repository, which we will update every release. But my concern will be — are we allowed to add and store about half a gigabyte of artifacts every 3 months? > On 18 Ja

[GitHub] ignite pull request #3396: IGNITE-7107 Apache Ignite RPM packages

Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/3396 ---

Re: IGNITE-7107 Apache Ignite RPM packages

Hi Petr, I would go for the approach implemented for Cassandra. Specifically: Cassandra root repository includes all the most recent versions and packages for RPM and DEB: http://www.apache.org/dist/cassandra/ This is a subdirectory for RPMs there: http://www.apache.org/dist/cassandra/redhat/

[GitHub] ignite pull request #3396: IGNITE-7107 Apache Ignite RPM packages

GitHub user vveider opened a pull request: https://github.com/apache/ignite/pull/3396 IGNITE-7107 Apache Ignite RPM packages * added changelog to package specification You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain

Re: IGNITE-7107 Apache Ignite RPM packages

Hi, Igniters! As part of "Apache Ignite in Packages" initiative, I’ve introduced update to Release procedure, which consists of: - RPM-build instruction [1]. - new "vote_3_step_1[rpm]create_rpm_repository.sh” script which does everything else to prepare RPM-repository layout along with source

Re: IGNITE-7107 Apache Ignite RPM packages

Hi, Denis. I was glad to contribute. Concerning DEB package — for 2.4 release I’m planning to introduce instructions and spec files for conversion RPM package to DEB, substantially reducing efforts for now because of file structure and install scripts inheritance. Later, of course, I’m going t

Re: IGNITE-7107 Apache Ignite RPM packages

Peter, thanks for this Christmas/New Year gift for the community! :) I’ll be back soon putting together a documentation for this capability. BTW, what’s the plan in regards DEB packages? https://issues.apache.org/jira/browse/IGNITE-7108 Do you

[GitHub] ignite pull request #3171: IGNITE-7107 Apache Ignite RPM packages

Github user vveider closed the pull request at: https://github.com/apache/ignite/pull/3171 ---

Re: IGNITE-7107 Apache Ignite RPM packages

Guys Thanks for your efforts to make it available in 2.4! On Thu, Dec 28, 2017 at 4:15 PM, Andrey Gura wrote: > Guys, > > I've merged change to master branch. Thanks a lot! > > On Thu, Dec 28, 2017 at 3:54 PM, Petr Ivanov wrote: > > Fixed remarks and nitpicks. > > > > Also, I purpose to delay

Re: IGNITE-7107 Apache Ignite RPM packages

Guys, I've merged change to master branch. Thanks a lot! On Thu, Dec 28, 2017 at 3:54 PM, Petr Ivanov wrote: > Fixed remarks and nitpicks. > > Also, I purpose to delay service autostart implementation until we’ll get > some first-hand usability feedback. > Added notes to DEVNOTES.txt about how

Re: IGNITE-7107 Apache Ignite RPM packages

Fixed remarks and nitpicks. Also, I purpose to delay service autostart implementation until we’ll get some first-hand usability feedback. Added notes to DEVNOTES.txt about how to start service. > On 28 Dec 2017, at 14:32, Ilya Kasnacheev wrote: > > Hello once more! > > Two more nitpicks: >

Re: IGNITE-7107 Apache Ignite RPM packages

Hello once more! Two more nitpicks: right now we install /usr/bin/ignitevisorcmd alternative, but it does't work since a) visor wants to write to /usr/share, and b) we moved it to examples for that. Please remove that alternative for now, create a ticket. rpm -ql apache-ignite | grep \\.java som

Re: IGNITE-7107 Apache Ignite RPM packages

Hello again! I have reviewed the modified patch. All the things in my critical list were fixed. I have just two minor remarks: - Please move sqlline.sh back from examples to /usr/share/a-i/bin. It works just as well from there as it was from our distribution. visor doesn't, hence it stays in exam

Re: IGNITE-7107 Apache Ignite RPM packages

Removed replacement for default-config.xml. Also reimplemented service to be able to run multiple instances of Ignite. See updated PR [1]. [1] https://github.com/apache/ignite/pull/3171 > On 25 Dec 2017, at 18:57, Pavel Tupitsyn wrote: > > PDS a

Re: IGNITE-7107 Apache Ignite RPM packages

PDS and discovery settings are unrelated to the packaging task. Andrey is right, just use existing default config. On Mon, Dec 25, 2017 at 6:51 PM, Petr Ivanov wrote: > Can we change default configuration file then? > So that both binary and package deliveries contained PDS turned on by > defaul

Re: IGNITE-7107 Apache Ignite RPM packages

Can we change default configuration file then? So that both binary and package deliveries contained PDS turned on by default? And what about Multicast Discovery? > On 25 Dec 2017, at 18:41, Dmitriy Setrakyan wrote: > > I agree with Andrey. The product should be identical regardless of how you

Re: IGNITE-7107 Apache Ignite RPM packages

I agree with Andrey. The product should be identical regardless of how you download and install it. On Mon, Dec 25, 2017 at 7:18 AM, Andrey Gura wrote: > Hi, > > I think we should provide the same default configuration for all > binary builds. From my point of view RPM/DEB/etc packages should us

Re: IGNITE-7107 Apache Ignite RPM packages

Hi, I think we should provide the same default configuration for all binary builds. From my point of view RPM/DEB/etc packages should use default configuration file like to standard binary release. On Mon, Dec 25, 2017 at 4:39 PM, Ilya Kasnacheev wrote: > Hello Igniters! > > What's your take on

Re: IGNITE-7107 Apache Ignite RPM packages

Hello Igniters! What's your take on enabling PDS in 2.4 in default config? This way we could enable it in RPM build with easy heart. The rest of your answers seem spot on. I'll happily review your amended patch when it is ready (later this week?) -- Ilya Kasnacheev 2017-12-22 18:27 GMT+03:00 v

Re: IGNITE-7107 Apache Ignite RPM packages

Ilya Kasnacheev wrote > I have noticed that both spec and DEVNOTES are made to use tabs alongside > with spaces. I thought we are avoiding tabs? Please confirm that usage of > tabs is desired in this case. My fault - got used to editing such files in default vim. Fixed and updated vim settings to

Re: IGNITE-7107 Apache Ignite RPM packages

Hello! I have reviewed your code and commented IGNITE-7107 for a list of potential problem. I think everything from the first "blocker" section ought to be fixed before this patch is merged, while "nice to have" things may be spinned out as separate tickets. I suggest moving scripts that don't wo

Re: IGNITE-7107 Apache Ignite RPM packages

Done. See updated PR#3171 [1] [1] https://github.com/apache/ignite/pull/3171 -- Sent from: http://apache-ignite-developers.2346864.n4.nabble.com/

Re: IGNITE-7107 Apache Ignite RPM packages

Hi Petr, Proposed to rename Ignite binary packages so that there is no “fabric” mentioning: http://apache-ignite-developers.2346864.n4.nabble.com/Removing-quot-fabric-quot-from-Ignite-binary-package-name-td25419.html

Re: IGNITE-7107 Apache Ignite RPM packages

Hi, Denis! Can you be a little bit more specific about "fabric" mentions? What exactly should be renamed? Package has final name "apache-ignite-fabric" in order to follow the convention of upstream source naming and DEVNOTES.txt instructions just name files as they are. -- Sent from: http://apa

Re: IGNITE-7107 Apache Ignite RPM packages

Hi Peter, Not being an expert in Linux packing system, the overall description looks clear to me. A minor comment, is that all ‘apache-ignite-fabric’ occurrences have to be renamed to ‘apache-ignite’. Ignite is presently called as an in-memory computing platform (that is also a questionable n

Re: IGNITE-7107 Apache Ignite RPM packages

I've added IEP document [1] with current packages design for overview. [1] https://cwiki.apache.org/confluence/display/IGNITE/IEP-11%3A+Introduce+Apache+Ignite+delivery+in+RPM+and+DEB+packages -- Sent from: http://apache-ignite-developers.2346864.n4.nabble.com/

Re: IGNITE-7107 Apache Ignite RPM packages

Thanks Petr, this is awesome. To be honest, I can hardly wait until it will be possible to install Ignite with yum or apt-get. D. On Thu, Dec 7, 2017 at 7:02 AM, Petr Ivanov wrote: > HI, all! > > > Within the task IGNITE-7107 I’ve prepare pull request #3171 [1] with: > - specification to build

IGNITE-7107 Apache Ignite RPM packages

HI, all! Within the task IGNITE-7107 I’ve prepare pull request #3171 [1] with: - specification to build RPM package out of built binary archive; - corresponding instruction in DEVNOTES.txt Also took responsibility to introduced style correction and unification to DEVNOTES.txt [1] - https://

[GitHub] ignite pull request #3171: IGNITE-7107 Apache Ignite RPM packages

GitHub user vveider opened a pull request: https://github.com/apache/ignite/pull/3171 IGNITE-7107 Apache Ignite RPM packages * added spec for RPM build * added instructions to DEVNOTES.txt You can merge this pull request into a Git repository by running: $ git pull https

[jira] [Created] (IGNITE-7107) Apache Ignite RPM packages

Peter Ivanov created IGNITE-7107: Summary: Apache Ignite RPM packages Key: IGNITE-7107 URL: https://issues.apache.org/jira/browse/IGNITE-7107 Project: Ignite Issue Type: Improvement