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

Avoid Compilation errors on Semeru JDK flavour. #2835

Conversation

krmahadevan
Copy link
Member

No description provided.

@krmahadevan
Copy link
Member Author

@juherr - Now I saw a different error again on the same osgi dependencies (with not being able to resolve artifacts). I have pushed an additional commit here that bumps up the test versions (Hope that would fix it)

https://github.com/krmahadevan/testng/actions/runs/3467189157/jobs/5791820079

JDK 11 Zulu

FAILURE  11.5sec, org.testng.test.osgi.PlainOsgiTest > versionShouldStartWithDigit
    shaded.org.eclipse.aether.resolution.ArtifactResolutionException: Error resolving artifact org.apache.geronimo.specs:geronimo-atinject_1.0_spec:jar:1.0

https://github.com/cbeust/testng/actions/runs/3467191687/jobs/5791824963

JDK17 SapMachine

Error:   6.1sec org.testng.test.osgi.PlainOsgiTest > versionShouldStartWithDigit
  shaded.org.eclipse.aether.resolution.ArtifactResolutionException: Error resolving artifact org.ops4j.pax.swissbox:pax-swissbox-core:jar:1.8.2

org.testng.test.osgi.PlainOsgiTest is flaky and 
Randomly fails with dependency resolution on some 
JDK flavours and not all. So configuring the build
To ignore this failure and move forward.
@juherr
Copy link
Member

juherr commented Nov 15, 2022

Hi @vlsi , we are stuck. Do you have any idea about the random issue?

@vlsi
Copy link
Contributor

vlsi commented Nov 15, 2022

Is it reproducible?
Frankly speaking, I've been facing connectivity issues recently between GitHub Actions and repo1.maven.org in pgjdbc.

See https://github.com/pgjdbc/pgjdbc/actions/runs/3459838388/jobs/5775689519#step:8:539

                        Caused by: shaded.org.apache.http.conn.ConnectTimeoutException: Connect to repo1.maven.org:443 [repo1.maven.org/146.75.28.209] failed: connect timed out
                            at app//shaded.org.apache.http.impl.conn.DefaultHttpClientConnectionOperator.connect(DefaultHttpClientConnectionOperator.java:151)
                            at app//shaded.org.apache.http.impl.conn.PoolingHttpClientConnectionManager.connect(PoolingHttpClientConnectionManager.java:373)
                            at app//shaded.org.apache.http.impl.execchain.MainClientExec.establishRoute(MainClientExec.java:394)
                            at app//shaded.org.apache.http.impl.execchain.MainClientExec.execute(MainClientExec.java:237)
                            at app//shaded.org.apache.http.impl.execchain.ProtocolExec.execute(ProtocolExec.java:185)
                            at app//shaded.org.apache.http.impl.execchain.RetryExec.execute(RetryExec.java:89)
                            at app//shaded.org.apache.http.impl.execchain.RedirectExec.execute(RedirectExec.java:110)
                            at app//shaded.org.apache.http.impl.client.InternalHttpClient.doExecute(InternalHttpClient.java:185)
                            at app//shaded.org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:83)
                            at app//org.ops4j.pax.url.mvn.internal.wagon.ConfigurableHttpWagon.execute(ConfigurableHttpWagon.java:162)
                            at app//shaded.org.apache.maven.wagon.shared.http.AbstractHttpClientWagon.fillInputData(AbstractHttpClientWagon.java:1095)
                            ... 42 more
                            Caused by: java.net.SocketTimeoutException: connect timed out
                                at java.base/java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:412)
                                at java.base/java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:255)
                                at java.base/java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:237)
                                at java.base/java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
                                at java.base/java.net.Socket.connect(Socket.java:609)
                                at shaded.org.apache.http.conn.ssl.SSLConnectionSocketFactory.connectSocket(SSLConnectionSocketFactory.java:339)
                                at shaded.org.apache.http.impl.conn.DefaultHttpClientConnectionOperator.connect(DefaultHttpClientConnectionOperator.java:142)
                                ... 52 more
                Caused by: shaded.org.eclipse.aether.resolution.ArtifactResolutionException: Error resolving artifact org.osgi:org.osgi.service.jdbc:jar:1.0.0
                    at app//shaded.org.eclipse.aether.internal.impl.DefaultArtifactResolver.resolve(DefaultArtifactResolver.java:413)
                    at app//shaded.org.eclipse.aether.internal.impl.DefaultArtifactResolver.resolveArtifacts(DefaultArtifactResolver.java:215)
                    at app//shaded.org.eclipse.aether.internal.impl.DefaultArtifactResolver.resolveArtifact(DefaultArtifactResolver.java:192)
                    at app//shaded.org.eclipse.aether.internal.impl.DefaultRepositorySystem.resolveArtifact(DefaultRepositorySystem.java:247)
                    at app//org.ops4j.pax.url.mvn.internal.AetherBasedResolver.resolve(AetherBasedResolver.java:767)
                    ... 24 more

As it did not fail much, I have not explored if there's a way to configure retries or something like that.

@vlsi
Copy link
Contributor

vlsi commented Nov 15, 2022

The artifact you mention is listed on Central, so the version should not be an issue: https://repo1.maven.org/maven2/org/apache/geronimo/specs/geronimo-atinject_1.0_spec/1.0/

@krmahadevan
Copy link
Member Author

@vlsi - It happens almost for every build and every time its a different artifact and confined to only one of these https://github.com/cbeust/testng/blob/a678d0586e2cf2ccf39d8997a24517c25f7be366/testng-test-osgi/testng-test-osgi-build.gradle.kts#L28-L31 and its not across all JDK flavours. Its random. I am not able to see/determine a pattern on when this happens

@vlsi
Copy link
Contributor

vlsi commented Nov 15, 2022

I think ArtifactResolutionException is not related to Gradle build script.
The error comes from shaded.org.eclipse.aether.internal.impl.DefaultArtifactResolver which is osgi container trying to resolve the dependency.

@krmahadevan
Copy link
Member Author

@vlsi - Any suggestions on how to get this sorted out ?

@krmahadevan krmahadevan merged commit e4adf0f into testng-team:master Nov 15, 2022
@krmahadevan krmahadevan deleted the fix_kotlin_compilation_errors_on_semeru branch November 15, 2022 14:54
@vlsi
Copy link
Contributor

vlsi commented Nov 15, 2022

I would suggest waiting for a couple of days and checking if that resolves on its own.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants