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

kibana 6.3.0 #28995

Closed
wants to merge 2 commits into from
Closed

kibana 6.3.0 #28995

wants to merge 2 commits into from

Conversation

ddillinger
Copy link
Contributor

Created with brew bump-formula-pr.

@jasontedor
Copy link
Contributor

@ilovezfs @MikeMcQuaid Sorry for the ping, but we need some help here. We have changed the licensing on our artifacts here so they do not carry a Apache 2.0 license anymore starting with 6.3.0. We want to work with Homebrew to allow distribution in an appropriate way. Is there someone that can reach out to us to start a conversation? Until we can sort that out though, we want to be conservative and ask that any PRs bumping the formula to 6.3.0 not be merged. I am happy to have a conversation here, or I can be reached at my first name at elastic.co.

@commitay
Copy link
Contributor

@jasontedor Does the license change also apply to the versioned formulae? [email protected] and [email protected] both have a new version 5.6.10?

@jasontedor
Copy link
Contributor

@commitay No, it only applies to 6.3.0 and later versions. All existing and future maintenance releases in the 5.6 series will continue to be under the same Apache 2.0 license as before. The 6.0, 6.1, and 6.2 series are also Apache 2.0 licensed. The change for the default distribution starts with the 6.3 series.

@ilovezfs
Copy link
Contributor

What is the new license?

@ilovezfs
Copy link
Contributor

ilovezfs commented Jun 14, 2018

OK, I see https://github.com/elastic/kibana/blob/master/licenses/ELASTIC-LICENSE.txt and https://github.com/elastic/kibana/blob/master/LICENSE.txt

Is there anything in the formula that is installing stuff from x-pack?

@ilovezfs
Copy link
Contributor

@jasontedor my recommendation is that upstream provide a source tarball that doesn't have the x-pack components at all so there are no misunderstandings. If that's not something upstream is willing to do, we can rm_rf "x-pack" at the beginning of the install method.

@mistydemeo
Copy link
Member

I agree with @ilovezfs; it would be great to get a copy of the tarballs with only the Apache 2 components and not the Elastic-licensed x-pack components.

@MikeMcQuaid
Copy link
Member

Also agree with @ilovezfs here.

@JCount
Copy link
Contributor

JCount commented Jun 14, 2018

I agree with @ilovezfs on this matter as well.

@ilovezfs
Copy link
Contributor

@jasontedor @ddillinger any update here?

@jasontedor
Copy link
Contributor

Heya, sorry, I’m on vacation since last Friday until next Monday so my replies will be terse and with high latency.

Briefly, the x-pack folder is integrated into our build scripts for some of the projects (if not all, I can’t check right now) so removing it at build time or providing a source tarball without the x-pack folder would not be possible at this time.

@ilovezfs
Copy link
Contributor

OK, we'll leave kibana, etc. at 6.2.x in Homebrew until upstream is able to provide an open source tarball and build script for 6.3.x. Thanks for the updated information @jasontedor

@ilovezfs ilovezfs closed this Jun 19, 2018
@ilovezfs
Copy link
Contributor

Thanks for the PR nonetheless @ddillinger

@ilovezfs ilovezfs mentioned this pull request Jun 19, 2018
@ilovezfs
Copy link
Contributor

if not all, I can’t check right now

@jasontedor when you are able to check, of course please feel free to comment on any of the other PRs if you determine Elastic license is not an issue for them (assuming using 6.3.x for some of the packages doesn't interfere with using 6.2.x for others).

@jrosengren
Copy link

Apache 2.0 licensed downloads (at least for metricbeat) appear to be available here: https://www.elastic.co/downloads/beats/metricbeat-oss

Are you able to use the "-oss" versions of the beats or is there still a question about whether those are fully Apache 2.0 compatible?

@ilovezfs
Copy link
Contributor

ilovezfs commented Jul 3, 2018

@jrosengren those should be fine! @jasontedor please correct me if that's not the case.

@ilovezfs
Copy link
Contributor

ilovezfs commented Jul 3, 2018

@jrosengren I think the problem there is that there's no way for us to build those from source unless I'm not seeing it. So actually the situation is unchanged.

@felixbuenemann
Copy link
Sponsor Contributor

@jasontedor Would removing the bin/x-pack and modules/x-pack directories from the elasticsearch tar-ball be enough to keep only the Apache 2.0 licensed parts?

@jrosengren
Copy link

It seems somewhat ridiculous that this hasn't been resolved yet. What are we waiting for to be able to include Metricbeat 6.3.x in Homebrew?

@MikeMcQuaid
Copy link
Member

@jrosengren The ability to build a package from source with an open source license.

@jrosengren
Copy link

Elastic publishes -oss docker images that presumably only contain the open source bits, so it seems like it should be possible for Homebrew to build similarly open source bits.

https://raw.githubusercontent.com/elastic/beats/99824df163863478d27688206e18ec27bc1b3ac5/LICENSE.txt

Based on the text in the LICENSE.txt, should it be possible to just download the source, run the build, and only use the -oss package? Or does Homebrew have a requirement to not download any non-open source bits?

@mschneid
Copy link

@MikeMcQuaid As I have seen, Elastic offers OSS downloads for the new 6.4.0 series:

@MikeMcQuaid
Copy link
Member

MikeMcQuaid commented Aug 24, 2018

Based on the text in the LICENSE.txt, should it be possible to just download the source, run the build, and only use the -oss package? Or does Homebrew have a requirement to not download any non-open source bits?

If there's a guaranteed way of building only the OSS version; yeh we can do that, I don't think we're saying we can't download it.

@felixbuenemann
Copy link
Sponsor Contributor

@mschneid At least some of those packages contain platform specific tar-balls, so since they contain pre-built binaries I don't think they can be used for homebrew packages.

@chrmoritz
Copy link
Contributor

At least kibana (not sure about the other ones) seems to have an option -oss in their build script to build only the OSS version now.

I was able to build the OSS version after rm_rf "x-pack" with the upgraded formula in chrmoritz@1f31903, which also follows upstream by switching from npm to yarn and upgrades the bundled node to the 8.x release line (which should fix the 10.14 bottle #31741).

But I don't know if (and how) we could do similar OSS-only builds for the other elastic formulae.

@MikeMcQuaid
Copy link
Member

@chrmoritz Sounds like a good call for a PR 👍

@chrmoritz
Copy link
Contributor

chrmoritz commented Sep 9, 2018

@MikeMcQuaid The main problem is, that this PR would be blocked on upgrading elasticsearch to 6.4.0 (as [email protected] isn't compatible with [email protected]), which itself is blocked on upgrading most (all?) other elastic formulae to 6.4.0.

I have no idea if (and how) building only the OSS part is possible for these other formulae as they are all using quite different build systems. But if you want I could open this as another do-not-merge PR to start the effort of upgrading the elastic formulae to 6.4.0 and hope that someone else, who is more familar with these build systems, will chime in and continue the work.

@MikeMcQuaid
Copy link
Member

I have no idea if (and how) building only the OSS part is possible for these other formulae as they are all using quite different build systems. But if you want I could open this as another do-not-merge PR to start the effort of upgrading the elastic formulae to 6.4.0 and hope that someone else, who is more familar with these build systems, will chime in and continue the work.

This sounds good 👍

@chrmoritz
Copy link
Contributor

chrmoritz commented Sep 10, 2018

I've currently kibana, elasticsearch and logstash upgraded to 6.4.0 OSS.

Still need help for the go based:

Edit: I give up here for now, as I can't find a way to build only the OSS part after removing x-pack for the go based formulae.

But according their their compatibility table these other updates are not strictly needed for shipping kibana and elasticsearch 6.4.0 and considering their high popularity vs the low popularity of the *beats formulas it might be worth shipping them anyway.

@chrmoritz chrmoritz mentioned this pull request Sep 10, 2018
4 tasks
@lock lock bot added the outdated PR was locked due to age label Oct 10, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Oct 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.