Skip to content

chore: use java-graalvm-ci-prod gcp project for running tests#3349

Merged
mpeddada1 merged 5 commits intomainfrom
use-graalvm-ci-project
Nov 13, 2024
Merged

chore: use java-graalvm-ci-prod gcp project for running tests#3349
mpeddada1 merged 5 commits intomainfrom
use-graalvm-ci-project

Conversation

@mpeddada1
Copy link
Contributor

@mpeddada1 mpeddada1 commented Nov 7, 2024 •
edited
Loading

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

mpeddada1 commented Nov 7, 2024

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 •
edited
Loading

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

mpeddada1 commented Nov 13, 2024

/gcbrun

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 13, 2024

Quality Gate Passed Quality Gate passed for 'gapic-generator-java-root'

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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 13, 2024

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

mpeddada1 commented Nov 13, 2024

/gcbrun

@mpeddada1
Copy link
Contributor Author

mpeddada1 commented Nov 13, 2024

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

jinseopkim0
jinseopkim0 reviewed Nov 13, 2024
View reviewed changes
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

suztomo
suztomo approved these changes Nov 13, 2024
View reviewed changes
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

burkedavison commented Nov 13, 2024

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

@burkedavison
Copy link
Member

burkedavison commented Nov 13, 2024

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

@mpeddada1 mpeddada1 merged commit 4c5a43c into main Nov 13, 2024
@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

Reviewers

@jinseopkim0 jinseopkim0 jinseopkim0 left review comments

@suztomo suztomo suztomo approved these changes

@burkedavison burkedavison burkedavison approved these changes

Assignees

No one assigned

Labels

size: xs Pull request size is extra small.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@mpeddada1 @burkedavison @suztomo @jinseopkim0 @diegomarquezp