Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

fix(deps): update dependency org.graalvm.sdk:graal-sdk to v22.2.0 #1740

Merged
merged 1 commit into from
Aug 22, 2022
Merged

fix(deps): update dependency org.graalvm.sdk:graal-sdk to v22.2.0 #1740

merged 1 commit into from
Aug 22, 2022

Conversation

renovate-bot
Copy link
Contributor

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
org.graalvm.sdk:graal-sdk 22.1.0.1 -> 22.2.0 age adoption passing confidence

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

  • If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Mend Renovate. View repository job log here.

@renovate-bot renovate-bot requested review from a team as code owners July 19, 2022 13:23
@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 19, 2022
@lqiu96 lqiu96 added kokoro:force-run Add this label to force Kokoro to re-run the tests. and removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Jul 20, 2022
@mpeddada1 mpeddada1 added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jul 22, 2022
@mpeddada1
Copy link
Contributor

Want to test this out with a couple of repos before merging this in so adding a do-not-merge label.

@renovate-bot renovate-bot force-pushed the renovate/org.graalvm.sdk-graal-sdk-22.x branch from 44b8810 to 94d7d82 Compare July 25, 2022 22:28
@meltsufin meltsufin added the automerge Merge the pull request once unit tests and other checks pass. label Jul 27, 2022
@gcf-merge-on-green
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jul 28, 2022
@renovate-bot renovate-bot force-pushed the renovate/org.graalvm.sdk-graal-sdk-22.x branch from 94d7d82 to a891c1b Compare July 29, 2022 16:59
@meltsufin
Copy link
Member

Want to test this out with a couple of repos before merging this in so adding a do-not-merge label.

@mpeddada1 Any updates on this testing? Thanks!

@mpeddada1
Copy link
Contributor

mpeddada1 commented Jul 29, 2022 •
edited
Loading

@meltsufin noticed a couple of issues when testing with Pub/Sub:

java -version

openjdk 11.0.16 2022-07-19
OpenJDK Runtime Environment GraalVM CE 22.2.0 (build 11.0.16+8-jvmci-22.2-b06)
OpenJDK 64-Bit Server VM GraalVM CE 22.2.0 (build 11.0.16+8-jvmci-22.2-b06, mixed mode, shar

Stacktrace:

GraalVM Native Image: Generating 'native-tests' (executable)...
========================================================================================================================
[1/7] Initializing...                                                                                   (13.8s @ 0.27GB)
 Version info: 'GraalVM 22.2.0 Java 11 CE'
 Java version info: '11.0.16+8-jvmci-22.2-b06'
 C compiler: gcc (linux, x86_64, 11.3.0)
 Garbage collector: Serial GC
 6 user-specific feature(s)
 - com.google.api.gax.grpc.nativeimage.GrpcNettyFeature
 - com.google.api.gax.grpc.nativeimage.ProtobufMessageFeature
 - com.google.api.gax.nativeimage.GoogleJsonClientFeature
 - com.google.api.gax.nativeimage.OpenCensusFeature
 - com.oracle.svm.thirdparty.gson.GsonFeature
 - org.graalvm.junit.platform.JUnitPlatformFeature
[junit-platform-native] Running in 'test listener' mode using files matching pattern [junit-platform-unique-ids*] found in folder [/usr/local/google/home/mpeddada/IdeaProjects/java-pubsub/google-cloud-pubsub/target/test-ids] and its subfolders.

Fatal error: java.lang.NoClassDefFoundError: org/graalvm/util/GuardedAnnotationAccess
	at org.graalvm.junit.platform.config.util.AnnotationUtils.getAnnotations(AnnotationUtils.java:99)
	at org.graalvm.junit.platform.config.util.AnnotationUtils.forEachAnnotationOnClassMembers(AnnotationUtils.java:63)
	at org.graalvm.junit.platform.config.util.AnnotationUtils.registerClassesFromAnnotationForReflection(AnnotationUtils.java:124)
	at org.graalvm.junit.platform.config.jupiter.JupiterConfigProvider.onTestClassRegistered(JupiterConfigProvider.java:91)
	at org.graalvm.junit.platform.JUnitPlatformFeature.lambda$registerTestClassForReflection$3(JUnitPlatformFeature.java:153)
	at org.graalvm.junit.platform.JUnitPlatformFeature.forEachProvider(JUnitPlatformFeature.java:162)
	at org.graalvm.junit.platform.JUnitPlatformFeature.registerTestClassForReflection(JUnitPlatformFeature.java:153)
	at java.base/java.util.stream.ForEachOps$ForEachOp$OfRef.accept(ForEachOps.java:183)
	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:195)
	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:195)
	at java.base/java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:177)
	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:195)
	at java.base/java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:177)
	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:195)
	at java.base/java.util.Iterator.forEachRemaining(Iterator.java:133)
	at java.base/java.util.Spliterators$IteratorSpliterator.forEachRemaining(Spliterators.java:1801)
	at java.base/java.util.stream.ReferencePipeline$Head.forEach(ReferencePipeline.java:658)
	at java.base/java.util.stream.ReferencePipeline$7$1.accept(ReferencePipeline.java:274)
	at java.base/java.util.Iterator.forEachRemaining(Iterator.java:133)
	at java.base/java.util.Spliterators$IteratorSpliterator.forEachRemaining(Spliterators.java:1801)
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:484)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:474)
	at java.base/java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:150)
	at java.base/java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential(ForEachOps.java:173)
	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.base/java.util.stream.ReferencePipeline.forEach(ReferencePipeline.java:497)
	at org.graalvm.junit.platform.JUnitPlatformFeature.discoverTestsAndRegisterTestClassesForReflection(JUnitPlatformFeature.java:145)
	at org.graalvm.junit.platform.JUnitPlatformFeature.beforeAnalysis(JUnitPlatformFeature.java:94)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGenerator.lambda$runPointsToAnalysis$9(NativeImageGenerator.java:722)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.FeatureHandler.forEachFeature(FeatureHandler.java:78)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGenerator.runPointsToAnalysis(NativeImageGenerator.java:722)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGenerator.doRun(NativeImageGenerator.java:564)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGenerator.run(NativeImageGenerator.java:521)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGeneratorRunner.buildImage(NativeImageGeneratorRunner.java:407)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGeneratorRunner.build(NativeImageGeneratorRunner.java:585)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGeneratorRunner.main(NativeImageGeneratorRunner.java:128)
Caused by: java.lang.ClassNotFoundException: org.graalvm.util.GuardedAnnotationAccess
	at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:476)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:589)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:522)
	... 36 more

There was a change that was introduced with 22.2.0 where the package name for GuardedAnnotationAccess was modified from org.graalvm.util to com.oracle.svm.util. This was addressed in native-maven-plugin 0.9.13. However, this issue is occurring in Pub/Sub since it still using an older version of the native-maven-plugin which is transitively brought in via java-shared-config. A few changes are needed in order to make this work:

  1. Update the java-shared-config version in pubsub to at least 1.5.2. UPDATE: This was completed with build(deps): update dependency com.google.cloud:google-cloud-shared-config to v1.5.3 java-pubsub#1211
  2. Add the following config to gax-grpc: --initialize-at-run-time=io.grpc.netty.shaded.io.netty.internal.tcnative.CertificateCompressionAlgo to resolve the following native image compilation issue once shared-config is upgraded. UPDATE: this was updated with fix(java): initialize CertificateCompressionAlgo netty class at run-time #1758 :
Error: Class initialization of io.grpc.netty.shaded.io.netty.internal.tcnative.CertificateCompressionAlgo failed. Use the option --initialize-at-run-time=io.grpc.netty.shaded.io.netty.internal.tcnative.CertificateCompressionAlgo to explicitly request delayed initialization of this class.
Detailed message:

@mpeddada1 mpeddada1 mentioned this pull request Aug 1, 2022
Merged
@mpeddada1
Copy link
Contributor

mpeddada1 commented Aug 1, 2022 •
edited
Loading

Created #1758 for the class initialization issue mentioned above. However, unable to test storage and bigquery tests with SNAPSHOT versions of gax and shared-dependencies as they are resulting in the following error:

[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running com.google.cloud.bigquery.it.ITBigQueryTest
Jul 29, 2022 9:10:43 PM com.google.auth.oauth2.DefaultCredentialsProvider warnAboutProblematicCredentials
WARNING: Your application has authenticated using end user credentials from Google Cloud SDK. We recommend that most server applications use service accounts instead. If your application continues to use end user credentials from Cloud SDK, you might receive a "quota exceeded" or "API not enabled" error. For more information about service accounts, see https://cloud.google.com/docs/authentication/.
[ERROR] Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.49 s <<< FAILURE! - in com.google.cloud.bigquery.it.ITBigQueryTest
[ERROR] com.google.cloud.bigquery.it.ITBigQueryTest  Time elapsed: 0.489 s  <<< ERROR!
java.lang.ExceptionInInitializerError
	at com.google.cloud.bigquery.it.ITBigQueryTest.beforeClass(ITBigQueryTest.java:683)
Caused by: java.lang.IllegalStateException: You are currently running with version 2.0.0 of google-api-client. You need at least version 1.31.1 of google-api-client to run version 1.32.1 of the BigQuery API library.
	at com.google.cloud.bigquery.it.ITBigQueryTest.beforeClass(ITBigQueryTest.java:683)

Have we seen this before?

@renovate-bot renovate-bot force-pushed the renovate/org.graalvm.sdk-graal-sdk-22.x branch from a891c1b to 82cd91c Compare August 1, 2022 23:09
@blakeli0 blakeli0 added the automerge Merge the pull request once unit tests and other checks pass. label Aug 3, 2022
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Aug 3, 2022
@alicejli alicejli added the automerge Merge the pull request once unit tests and other checks pass. label Aug 16, 2022
@renovate-bot renovate-bot force-pushed the renovate/org.graalvm.sdk-graal-sdk-22.x branch from 82cd91c to b8c4436 Compare August 16, 2022 16:25
@renovate-bot renovate-bot force-pushed the renovate/org.graalvm.sdk-graal-sdk-22.x branch from b8c4436 to 64c9552 Compare August 16, 2022 17:58
@alicejli alicejli removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 16, 2022
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@alicejli alicejli added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 16, 2022
@alicejli alicejli removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 16, 2022
@alicejli alicejli added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 16, 2022
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Aug 17, 2022
@alicejli alicejli added kokoro:force-run Add this label to force Kokoro to re-run the tests. automerge Merge the pull request once unit tests and other checks pass. and removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Aug 17, 2022
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Aug 18, 2022
@alicejli
Copy link

@mpeddada1 Have you been able to confirm the testing works now? I think the previous error was due to not having the latest libraries bom release, but that should have gotten fixed August 12.

@alicejli alicejli added the automerge Merge the pull request once unit tests and other checks pass. label Aug 19, 2022
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Aug 19, 2022
@mpeddada1
Copy link
Contributor

Thanks for following up and the update @alicejli! I've just confirmed that after the fix, running bigquery tests with graal-sdk 22.2.0 results in a successful build! This PR can be merged.

Screen Shot 2022-08-22 at 3 04 57 PM

@mpeddada1 mpeddada1 removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 22, 2022
@alicejli alicejli merged commit ded44a6 into googleapis:main Aug 22, 2022
@renovate-bot renovate-bot deleted the renovate/org.graalvm.sdk-graal-sdk-22.x branch August 22, 2022 19:14
@release-please release-please bot mentioned this pull request Aug 22, 2022
Merged
gcf-merge-on-green bot pushed a commit that referenced this pull request Aug 23, 2022
@release-please
chore(main): release 2.19.0 (#1776)
de1235b 
🤖 I have created a release *beep* *boop*
---


## [2.19.0](v2.18.7...v2.19.0) (2022-08-22)


### Features

* Add numeric enum support. ([#1743](#1743)) ([3f7628e](3f7628e))


### Bug Fixes

* **deps:** update dependency com.google.auth:google-auth-library-credentials to v1.10.0 ([#1768](#1768)) ([3f2188d](3f2188d))
* **deps:** update dependency com.google.auth:google-auth-library-credentials to v1.9.0 ([#1765](#1765)) ([103db3c](103db3c))
* **deps:** update dependency com.google.auth:google-auth-library-oauth2-http to v1.10.0 ([#1769](#1769)) ([0b1eb92](0b1eb92))
* **deps:** update dependency com.google.auth:google-auth-library-oauth2-http to v1.9.0 ([#1766](#1766)) ([2677f07](2677f07))
* **deps:** update dependency com.google.code.gson:gson to v2.9.1 ([#1757](#1757)) ([ea2a075](ea2a075))
* **deps:** update dependency com.google.protobuf:protobuf-bom to v3.21.5 ([#1772](#1772)) ([d7a48d1](d7a48d1))
* **deps:** update dependency io.grpc:grpc-bom to v1.48.1 ([#1763](#1763)) ([e5e4232](e5e4232))
* **deps:** update dependency org.graalvm.sdk:graal-sdk to v22.2.0 ([#1740](#1740)) ([ded44a6](ded44a6))
* **deps:** update dependency org.mockito:mockito-core to v4.7.0 ([#1774](#1774)) ([29678c8](29678c8))
* **deps:** update dependency org.threeten:threetenbp to v1.6.1 ([#1773](#1773)) ([d2c84e6](d2c84e6))
* **test:** testThrottlingBlocking flakyness fix ([#1775](#1775)) ([e69393c](e69393c))


### Documentation

* explaining UNIX environment is required ([#1760](#1760)) ([1d31e90](1d31e90))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
suztomo pushed a commit to googleapis/sdk-platform-java that referenced this pull request Dec 16, 2022
@release-please
chore(main): release 2.19.0 (#1776)
aa85c68 
🤖 I have created a release *beep* *boop*
---


## [2.19.0](googleapis/gax-java@v2.18.7...v2.19.0) (2022-08-22)


### Features

* Add numeric enum support. ([#1743](googleapis/gax-java#1743)) ([fa87970](googleapis/gax-java@fa87970))


### Bug Fixes

* **deps:** update dependency com.google.auth:google-auth-library-credentials to v1.10.0 ([#1768](googleapis/gax-java#1768)) ([6008b0d](googleapis/gax-java@6008b0d))
* **deps:** update dependency com.google.auth:google-auth-library-credentials to v1.9.0 ([#1765](googleapis/gax-java#1765)) ([ba597b4](googleapis/gax-java@ba597b4))
* **deps:** update dependency com.google.auth:google-auth-library-oauth2-http to v1.10.0 ([#1769](googleapis/gax-java#1769)) ([db37c42](googleapis/gax-java@db37c42))
* **deps:** update dependency com.google.auth:google-auth-library-oauth2-http to v1.9.0 ([#1766](googleapis/gax-java#1766)) ([fe181b0](googleapis/gax-java@fe181b0))
* **deps:** update dependency com.google.code.gson:gson to v2.9.1 ([#1757](googleapis/gax-java#1757)) ([36a0136](googleapis/gax-java@36a0136))
* **deps:** update dependency com.google.protobuf:protobuf-bom to v3.21.5 ([#1772](googleapis/gax-java#1772)) ([c9a2a56](googleapis/gax-java@c9a2a56))
* **deps:** update dependency io.grpc:grpc-bom to v1.48.1 ([#1763](googleapis/gax-java#1763)) ([5535101](googleapis/gax-java@5535101))
* **deps:** update dependency org.graalvm.sdk:graal-sdk to v22.2.0 ([#1740](googleapis/gax-java#1740)) ([77f66d3](googleapis/gax-java@77f66d3))
* **deps:** update dependency org.mockito:mockito-core to v4.7.0 ([#1774](googleapis/gax-java#1774)) ([e192e0b](googleapis/gax-java@e192e0b))
* **deps:** update dependency org.threeten:threetenbp to v1.6.1 ([#1773](googleapis/gax-java#1773)) ([b9e4767](googleapis/gax-java@b9e4767))
* **test:** testThrottlingBlocking flakyness fix ([#1775](googleapis/gax-java#1775)) ([9aef46a](googleapis/gax-java@9aef46a))


### Documentation

* explaining UNIX environment is required ([#1760](googleapis/gax-java#1760)) ([1555924](googleapis/gax-java@1555924))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@alicejli alicejli alicejli approved these changes

@blakeli0 blakeli0 blakeli0 approved these changes

Assignees
No one assigned
Labels
kokoro:force-run Add this label to force Kokoro to re-run the tests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@renovate-bot @mpeddada1 @meltsufin @alicejli @blakeli0 @lqiu96