Skip to content

Conversation

@mpeddada1
Copy link
Contributor

@mpeddada1 mpeddada1 commented Nov 7, 2024

This PR migrates the presubmits in sdk-platform-java to java-graalvm-ci-prod.

@product-auto-label product-auto-label bot added the size: xs Pull request size is extra small. label Nov 7, 2024
@mpeddada1
Copy link
Contributor Author

Stacktrace:

�[0m �[0m�[1m�[31mError: �[0m�[0m�[1mError when reading or editing Project Service : Request `List Project Services java-graalvm-ci-prod` returned error: Batch request and retried single request "List Project Services java-graalvm-ci-prod" both failed. Final error: Failed to list enabled services for project java-graalvm-ci-prod: googleapi: Error 403: Permission denied to list services for consumer container

@mpeddada1
Copy link
Contributor Author

mpeddada1 commented Nov 12, 2024

Stacktrace after enabling cloudkms.googleapis.com in java-graalvm-ci-prod:

 ITKmsTest.ensureKmsKeyRingExists:60->getKeyRing:76 » StatusRuntime PERMISSION_DENIED: Permission 'cloudkms.keyRings.get' denied on resource ...` (or it may not exist).

@mpeddada1
Copy link
Contributor Author

/gcbrun

@sonarqubecloud
Copy link

@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed for 'java_showcase_integration_tests'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@mpeddada1
Copy link
Contributor Author

/gcbrun

@mpeddada1
Copy link
Contributor Author

The presubmits running on java-graalvm-ci-prod are now passing.

Copy link
Contributor

@jinseopkim0 jinseopkim0 left a comment

Choose a reason for hiding this comment

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

Retrying the failed library_generation test

error: RPC failed; curl 56 GnuTLS recv error (-9): Error decoding the received TLS packet.
error: 2366 bytes of body are still expected

https://github.com/googleapis/sdk-platform-java/actions/runs/11823555090/job/32943111452?pr=3349

Copy link
Member

@suztomo suztomo left a comment

Choose a reason for hiding this comment

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

The required GraalVM checks passed

@burkedavison
Copy link
Member

Please don't merge until all the GraalVM checks are passing.

@burkedavison
Copy link
Member

Discussed with Mridula. Approved. The failing checks are duplicates and legacy.

@mpeddada1 mpeddada1 merged commit 4c5a43c into main Nov 13, 2024
50 of 55 checks passed
@mpeddada1 mpeddada1 deleted the use-graalvm-ci-project branch November 13, 2024 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants