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.

Sign up for GitHub

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

Jump to bottom

Attempt to fix the permred "PostCommit Java IO Performance Tests - another attempt #32741

Merged
merged 1 commit into from
Oct 10, 2024
Merged

Attempt to fix the permred "PostCommit Java IO Performance Tests - another attempt #32741

merged 1 commit into from
Oct 10, 2024
+0 −5

Conversation

shunping
Copy link
Contributor

A follow-up of PR (#32736)

  • remove setup-gcloud as they are not in java_tests and python_tests.

@github-actions github-actions bot added the build label Oct 10, 2024
@damondouglas damondouglas self-requested a review October 10, 2024 16:39
@damccorm damccorm merged commit 538db45 into apache:master Oct 10, 2024
3 checks passed
@shunping
Copy link
Contributor Author

I see the following exception with the latest run(https://github.com/apache/beam/actions/runs/11278385415/job/31366711759):

org.apache.beam.it.gcp.bigtable.BigTableIOLT > testBigtableWriteAndRead FAILED
    org.apache.beam.it.gcp.bigtable.BigtableResourceManagerException at BigTableIOLT.java:120
        Caused by: com.google.api.gax.rpc.UnauthenticatedException at ApiExceptionFactory.java:116
            Caused by: io.grpc.StatusRuntimeException at Status.java:533

So sadly my proposed fix does not work...

reeba212 pushed a commit to reeba212/beam that referenced this pull request Dec 4, 2024
@shunping @reeba212
Remove setup-gcloud as they are not in java_tests and python_tests (a…
082bf2f 
…pache#32741)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@damondouglas damondouglas damondouglas approved these changes

Assignees
No one assigned
Labels
build
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@shunping @damondouglas @damccorm