Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

build artifacts for rpm, deb, zip #5

Merged

Conversation

jngz-es
Copy link
Contributor

@jngz-es jngz-es commented Dec 4, 2019

issue #3

@jngz-es jngz-es self-assigned this Dec 4, 2019
@jngz-es jngz-es added the enhancement New feature or request label Dec 4, 2019
release = isSnapshot ? "0.1" : '1'
version = "${project.version}" - "-SNAPSHOT"

into '/usr/share/elasticsearch/plugins'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we are missing some folder of this path, would it be generated automatically?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All plugins will use the same path, like alerting, sql etc. So I don't think this is a problem.

@@ -31,6 +31,10 @@ buildscript {
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you tested the built artifacts on a redhat or debian machine?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, but I manually checked the artifacts which look having everything. The command "./gradlew buildPackages --refresh-dependencies --console=plain -Dbuild.snapshot=false" will generate all artifacts.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be safe, I suggest we spin up a debian or redhat host and install it there to see if AD can run properly. Otherwise, we are waiting for customers to test this for us.

@jngz-es jngz-es requested a review from kaituo December 5, 2019 18:10
dependsOn 'assemble'
}

task buildPackages(type: GradleBuild) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion/Question. Should the instruction on how to build be added to readme?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good suggestion, will add it later. By the way, is that you, Lai? I didn't realize it :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's me :)

@jngz-es jngz-es merged commit 3be6d8c into opendistro-for-elasticsearch:development Jan 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants