Anthonios Partheniou comments

Results 300 comments of


                                            Anthonios Partheniou

Resolve lint issues when running flake8

Also see these lines where we're excluding linting on generated code https://github.com/googleapis/gapic-generator-python/blob/30cd1a49353048fc190114cfb7a73e14ec2eff31/gapic/templates/.flake8.j2#L23-L25

Improve gRPC error message when request is invalid

Hi @ithomaslin , This appears to be an issue with the API. Thanks for reporting the issue in the API specific issue tracker (https://issuetracker.google.com/issues/343845155). We'll keep this issue open and...

Improve gRPC error message when request is invalid

I was able to re-create the error using the following code ``` from google.cloud.run_v2 import ListJobsRequest import google.cloud.run_v2 as run_v2 run_client = run_v2.JobsClient() request = ListJobsRequest( parent="projects/" ) run_client.list_jobs(request=request) ```...

Improve gRPC error message when request is invalid

Thanks for the update @ajoy39. Please let me know if you're still unable to resolve the issue after trying `REST`. I've reached out to the API team to request a...

Improve gRPC error message when request is invalid

@ajoy39 Thanks for the update. I'm glad that you found the issue! I'll keep this issue open as the error message for gRPC could be improved.

Improve gRPC error message when request is invalid

I'm going to transfer this issue to the repository for the code generator where we can work to improve the gRPC error message.

`ValueError: metadata was invalid` occurs when key in request metadata contains uppercase charactors

The exception `ValueError: metadata was invalid` is raised by gRPC. From https://github.com/grpc/grpc/blob/ca0d85cea5242d6740ba6cf1dc5c0b9647856fca/src/python/grpcio/grpc/_cython/_cygrpc/channel.pyx.pxi#L22-L23, ``` cdef str _call_error_metadata(metadata): return 'metadata was invalid: %s' % metadata ``` From https://github.com/grpc/grpc/blob/ca0d85cea5242d6740ba6cf1dc5c0b9647856fca/src/core/lib/surface/validate_metadata.cc#L33-L42, Uppercase characters are not...

Test coverage is incorrectly configured for gapic-generator-python, and is trying to ananlyze non-python files as if they are python code

This is related to a recent release of `coverage` : https://pypi.org/project/coverage/7.11.1/. Let's exclude this version temporarily and see if it comes back. https://github.com/nedbat/coveragepy/issues/2077

Test coverage is incorrectly configured for gapic-generator-python, and is trying to ananlyze non-python files as if they are python code

Closing as obsolete. https://github.com/coveragepy/coveragepy/issues/2077 was fixed

Test coverage is incorrectly configured for gapic-generator-python, and is trying to ananlyze non-python files as if they are python code

Re-opening. This is still a problem. see build log https://github.com/googleapis/gapic-generator-python/actions/runs/19243623779/job/55012060872?pr=2482