Turn on snippetgen by default #1043

Activity

1. busunkim96

   added

   type: processA process-related concern. May include testing, release, or the like.A process-related concern. May include testing, release, or the like.

   on Oct 27, 2021
2. busunkim96

   self-assigned this

   on Oct 27, 2021
3. busunkim96 commented on Oct 29, 2021

   

   busunkim96

   on Oct 29, 2021

   ContributorAuthor

   More actions

   There appears to be a bug for client streaming so I wasn't able to enable this globally:

   If it takes a while to fix I will back out #1044 so it doesn't block other changes.

       sample["request"] = v.validate_and_transform_request(
     File "/usr/local/google/home/busunkim/.cache/bazel/_bazel_busunkim/5f7ad274936c83fa5125630e373edd54/sandbox/linux-sandbox/1362/execroot/com_google_googleapis/bazel-out/host/bin/external/gapic_generator_python/gapic_plugin.runfiles/gapic_generator_python/gapic/samplegen/samplegen.py", line 560, in validate_and_transform_request
       raise types.InvalidRequestSetup(
   gapic.samplegen_utils.types.InvalidRequestSetup: Too many base parameters for client side streaming form
   --python_gapic_out: protoc-gen-python_gapic: Plugin failed with status code 1.
   Target //google/firestore/v1:firestore-v1-py failed to build
4. busunkim96

   added a commit that references this issue on Nov 8, 2021

   fix(snippetgen): fix client streaming samples (#1061)

   ...

   64b9ad6
5. busunkim96 commented on Nov 10, 2021

   

   busunkim96

   on Nov 10, 2021

   Last edited by busunkim96

   ContributorAuthor

   More actions

   Aand I've found at least one other bug - it was not caught by the goldens but was discovered when I tried to use #1077 as the generator version in the WORKSPACE file. I think it is related to #1012.

   I'm planning on manually generating locally with 0.56.1 with every API to shake out remaining bugs blocking turning on snippetgen by default.
6. busunkim96 commented on Nov 15, 2021

   

   busunkim96

   on Nov 15, 2021

   ContributorAuthor

   More actions

   #1012 started to use the resource path logic which wasn't written to handle nested sub-fields. The generation was failing as matching resource paths were not found for resource paths that are not on the base request.
7. busunkim96 commented on Jan 20, 2022

   

   busunkim96

   on Jan 20, 2022

   ContributorAuthor

   More actions

   I've opened googleapis/google-cloud-python#16441 to track re-adding f-string support for resource paths (it requires addressing the bug mentioned in the last comment).

   The resource path logic appears to have been the last blocking issue. I ran the internal googleapis presubmit with #1134 and all the APIs appear to be generate successfully. 😃
8. parthea commented on Apr 1, 2022

   

   parthea

   on Apr 1, 2022

   Contributor

   More actions

   @busunkim96 Please can you check if there are any outstanding items? If not could we close this issue?
9. busunkim96

   closed this as completedon Apr 8, 2022