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

Un-mute and fix BuildExamplePluginsIT #38899

Merged
merged 6 commits into from
Mar 4, 2019

Conversation

alpar-t
Copy link
Contributor

@alpar-t alpar-t commented Feb 14, 2019

There doesn't seem to be anything wrong with the test iteself.
I think the failure were CI performance related, but while it was muted,
some failures managed to sneak in.

@jaymode , @tvernum note that this does fix the test, but users will still not be able to run it unless we publish the specific jars to maven central. This is done by the release manager.
The apply plugin: "nebula.maven-scm" causes the POMs to be generated so RM can publish them, but the RM config still needs to be updated.
This is necessary to make it possible for users to get to all the dependencies of the example plugin, including transitive dependencies.
The reason this works in the regular build is that all the dependencies are taken from the current build, but the example plugins are also meant to be built outside the build.

Closes #38784

There doesn't seem to be anything wrong with the test iteself.
I think the failure were CI performance related, but while it was muted,
some failures managed to sneak in.

Closes elastic#38784
@alpar-t alpar-t added :Delivery/Build Build or test infrastructure >test-failure Triaged test failures from CI v7.0.0 v8.0.0 v7.2.0 labels Feb 14, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@alpar-t
Copy link
Contributor Author

alpar-t commented Feb 14, 2019

@elasticmachine run elasticsearch-ci/oss-distro-docs

@alpar-t alpar-t requested a review from mark-vieira February 27, 2019 15:49
@@ -39,6 +39,7 @@ import org.gradle.api.logging.Logger
import org.gradle.api.tasks.Copy
import org.gradle.api.tasks.Delete
import org.gradle.api.tasks.Exec
import org.gradle.internal.os.OperatingSystem
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like this import is unused.

@@ -16,6 +16,7 @@
* specific language governing permissions and limitations
* under the License.
*/
apply plugin: "nebula.maven-scm"
Copy link
Contributor

Choose a reason for hiding this comment

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

Applying this plugin twice now.

@alpar-t alpar-t merged commit 41a29a7 into elastic:master Mar 4, 2019
@alpar-t alpar-t deleted the fix-example-plugin-build branch March 4, 2019 06:37
alpar-t added a commit that referenced this pull request Mar 4, 2019
* Un-mute and fix BuildExamplePluginsIT

There doesn't seem to be anything wrong with the test iteself.
I think the failure were CI performance related, but while it was muted,
some failures managed to sneak in.

Closes #38784

* PR review
alpar-t added a commit that referenced this pull request Mar 4, 2019
* Un-mute and fix BuildExamplePluginsIT

There doesn't seem to be anything wrong with the test iteself.
I think the failure were CI performance related, but while it was muted,
some failures managed to sneak in.

Closes #38784

* PR review
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
@jakelandis jakelandis removed the v8.0.0 label Jul 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team >test-failure Triaged test failures from CI v7.0.0-rc2 v7.2.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI BuildExamplePluginsIT fails
5 participants