Skip to content

chore: update graalvm container images to 22.3.3#2212

Merged
mpeddada1 merged 2 commits intomainfrom
update-graal-jobs
Oct 30, 2023
Merged

chore: update graalvm container images to 22.3.3#2212
mpeddada1 merged 2 commits intomainfrom
update-graal-jobs

Conversation

@mpeddada1
Copy link
Contributor

@mpeddada1 mpeddada1 commented Oct 30, 2023

Thank you for opening a Pull Request! For general contributing guidelines, please refer to contributing guide

Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕️

@mpeddada1 mpeddada1 requested a review from a team October 30, 2023 18:22
@product-auto-label product-auto-label bot added the size: xs Pull request size is extra small. label Oct 30, 2023
@product-auto-label product-auto-label bot added size: s Pull request size is small. and removed size: xs Pull request size is extra small. labels Oct 30, 2023
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 30, 2023

[gapic-generator-java-root] 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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 30, 2023

[java_showcase_integration_tests] 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

@mpeddada1
Copy link
Contributor Author

mpeddada1 commented Oct 30, 2023 •
edited
Loading

Kokoro GraalVM Java 17 test failed with:

JUnit Vintage:ITLongRunningOperation:testHttpJson_LROSuccessfulResponse_doesNotExceedTotalTimeout
    MethodSource [className = 'com.google.showcase.v1beta1.it.ITLongRunningOperation', methodName = 'testHttpJson_LROSuccessfulResponse_doesNotExceedTotalTimeout', methodParameterTypes = '']
    => java.util.concurrent.ExecutionException: com.google.api.gax.rpc.NotFoundException: Not Found
       com.google.common.util.concurrent.AbstractFuture.getDoneValue(AbstractFuture.java:592)
       com.google.common.util.concurrent.AbstractFuture.get(AbstractFuture.java:571)
       com.google.common.util.concurrent.FluentFuture$TrustedFuture.get(FluentFuture.java:91)
       com.google.common.util.concurrent.ForwardingFuture.get(ForwardingFuture.java:67)
       com.google.api.gax.longrunning.OperationFutureImpl.get(OperationFutureImpl.java:125)
       [...]
     Caused by: com.google.api.gax.rpc.NotFoundException: Not Found
       com.google.api.gax.rpc.ApiExceptionFactory.createException(ApiExceptionFactory.java:50)
       com.google.api.gax.httpjson.HttpJsonApiExceptionFactory.createApiException(HttpJsonApiExceptionFactory.java:76)
       com.google.api.gax.httpjson.HttpJsonApiExceptionFactory.create(HttpJsonApiExceptionFactory.java:54)
       com.google.api.gax.httpjson.HttpJsonExceptionCallable$ExceptionTransformingFuture.onFailure(HttpJsonExceptionCallable.java:97)
       com.google.api.core.ApiFutures$1.onFailure(ApiFutures.java:84)
       [...]

Retry results in success.

@mpeddada1 mpeddada1 merged commit 6bccaec into main Oct 30, 2023
@mpeddada1 mpeddada1 deleted the update-graal-jobs branch October 30, 2023 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@burkedavison burkedavison burkedavison approved these changes

Assignees

No one assigned

Labels

size: s Pull request size is small.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mpeddada1 @burkedavison