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

[CI] TestingConventionsPrecommitPluginFuncTest skips convention check if no tests available failing #87913

Closed
iverase opened this issue Jun 22, 2022 · 8 comments
Assignees
Labels
:Delivery/Tooling Developer tooliing and automation Team:Delivery Meta label for Delivery team >test-failure Triaged test failures from CI

Comments

@iverase
Copy link
Contributor

iverase commented Jun 22, 2022

It seems all suite is failing in windows.

Build scan:
https://gradle-enterprise.elastic.co/s/odbr7rn44bbja/tests/:integTest/org.elasticsearch.gradle.internal.precommit.TestingConventionsPrecommitPluginFuncTest/skips%20convention%20check%20if%20no%20tests%20available

Reproduction line:
null

Applicable branches:
master

Reproduces locally?:
Didn't try

Failure history:
https://gradle-enterprise.elastic.co/scans/tests?tests.container=org.elasticsearch.gradle.internal.precommit.TestingConventionsPrecommitPluginFuncTest&tests.test=skips%20convention%20check%20if%20no%20tests%20available

Failure excerpt:

org.gradle.testkit.runner.UnexpectedBuildFailure: Unexpected build execution failure in C:\Users\jenkins\AppData\Local\Temp\junit5057803447775395944 with arguments [precommit, -Dtest.external=true]

Output:

FAILURE: Build failed with an exception.

* Where:
Build file 'C:\Users\jenkins\AppData\Local\Temp\junit5057803447775395944\build.gradle' line: 16

* What went wrong:
Could not compile build file 'C:\Users\jenkins\AppData\Local\Temp\junit5057803447775395944\build.gradle'.
> startup failed:
  build file 'C:\Users\jenkins\AppData\Local\Temp\junit5057803447775395944\build.gradle': 16: Unexpected character: '"' @ line 16, column 19.
                 url = "C:\Users\jenkins\AppData\Local\Temp\junit7233216055668629539\local-repo"
                       ^
  
  1 error


* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 81ms

  at org.gradle.testkit.runner.internal.DefaultGradleRunner.lambda$build$2(DefaultGradleRunner.java:264)
  at org.gradle.testkit.runner.internal.DefaultGradleRunner.run(DefaultGradleRunner.java:338)
  at org.gradle.testkit.runner.internal.DefaultGradleRunner.build(DefaultGradleRunner.java:262)
  at org.elasticsearch.gradle.internal.test.InternalAwareGradleRunner.build(InternalAwareGradleRunner.java:145)
  at org.elasticsearch.gradle.internal.test.NormalizeOutputGradleRunner.build(NormalizeOutputGradleRunner.java:145)
  at org.elasticsearch.gradle.internal.precommit.TestingConventionsPrecommitPluginFuncTest.skips convention check if no tests available(TestingConventionsPrecommitPluginFuncTest.groovy:54)

@iverase iverase added :Delivery/Tooling Developer tooliing and automation >test-failure Triaged test failures from CI labels Jun 22, 2022
@elasticmachine elasticmachine added the Team:Delivery Meta label for Delivery team label Jun 22, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@breskeby
Copy link
Contributor

breskeby commented Jul 4, 2022

Fixed in #88240

@breskeby breskeby closed this as completed Jul 4, 2022
@albertzaharovits
Copy link
Contributor

Reopening because it seems related to new failures in the same clas, still on windows, but 7.17 now.
The error seems different though?

TestingConventionsPrecommitPluginFuncTest > applies conventions on #sourceSetName tests > org.elasticsearch.gradle.internal.precommit.TestingConventionsPrecommitPluginFuncTest.applies conventions on internalClusterTest tests FAILED
    org.spockframework.runtime.ConditionFailedWithExceptionError at TestingConventionsPrecommitPluginFuncTest.groovy:270
        Caused by: java.lang.NullPointerException at TestingConventionsPrecommitPluginFuncTest.groovy:270

@mark-vieira
Copy link
Contributor

Probably fallout from the backport. Do you mind taking a look @breskeby?

@breskeby
Copy link
Contributor

checking

breskeby added a commit to breskeby/elasticsearch that referenced this issue Jul 13, 2022
…c#88240)

Addressing elastic#87913 by fixing LocalRepositoryFixture on windows
# Conflicts:
#	build-tools-internal/src/integTest/groovy/org/elasticsearch/gradle/internal/precommit/TestingConventionsPrecommitPluginFuncTest.groovy
@breskeby
Copy link
Contributor

Should be fixed by this backport: #88524 I'll keep an eye on this till confirmed

elasticsearchmachine pushed a commit that referenced this issue Jul 13, 2022
#88524)

Addressing #87913 by fixing LocalRepositoryFixture on windows
# Conflicts:
#	build-tools-internal/src/integTest/groovy/org/elasticsearch/gradle/internal/precommit/TestingConventionsPrecommitPluginFuncTest.groovy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Delivery/Tooling Developer tooliing and automation Team:Delivery Meta label for Delivery team >test-failure Triaged test failures from CI
Projects
None yet
Development

No branches or pull requests

5 participants