[CHANGE ME] Re-generated to pick up changes in the API or client library generator. #66

yoshi-automation · 2020-04-03T12:14:22Z

This PR was generated using Autosynth. 🌈

Log from Synthtool

2020-04-03 05:13:49,388 synthtool > Executing /tmpfs/src/git/autosynth/working_repo/synth.py.
2020-04-03 05:13:49,441 synthtool > Ensuring dependencies.
2020-04-03 05:13:49,445 synthtool > Pulling artman image.
latest: Pulling from googleapis/artman
Digest: sha256:ef1a5b367dbe1e37cea1c7c814c801a638473e8dd66f87f4a2b8c2a146013673
Status: Image is up to date for googleapis/artman:latest
2020-04-03 05:13:50,843 synthtool > Cloning googleapis.
2020-04-03 05:13:51,355 synthtool > Running generator for google/pubsub/artman_pubsub.yaml.
2020-04-03 05:14:09,299 synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/python/pubsub-v1.
2020-04-03 05:14:09,300 synthtool > Copy: /home/kbuilder/.cache/synthtool/googleapis/google/pubsub/v1/pubsub.proto to /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/python/pubsub-v1/google/cloud/pubsub_v1/proto/pubsub.proto
2020-04-03 05:14:09,300 synthtool > Placed proto files into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/python/pubsub-v1/google/cloud/pubsub_v1/proto.
2020-04-03 05:14:09,333 synthtool > Replaced 'from google.cloud import pubsub_v1' in tests/unit/gapic/v1/test_publisher_client_v1.py.
2020-04-03 05:14:09,334 synthtool > Replaced ' pubsub_v1' in tests/unit/gapic/v1/test_publisher_client_v1.py.
2020-04-03 05:14:09,334 synthtool > Replaced 'from google.cloud import pubsub_v1' in tests/unit/gapic/v1/test_subscriber_client_v1.py.
2020-04-03 05:14:09,335 synthtool > Replaced ' pubsub_v1' in tests/unit/gapic/v1/test_subscriber_client_v1.py.
2020-04-03 05:14:09,336 synthtool > Replaced '# The name of the interface for this client. This is the key used to' in google/cloud/pubsub_v1/gapic/subscriber_client.py.
2020-04-03 05:14:09,336 synthtool > Replaced '# The name of the interface for this client. This is the key used to' in google/cloud/pubsub_v1/gapic/publisher_client.py.
2020-04-03 05:14:09,337 synthtool > Replaced 'import google.api_core.gapic_v1.method\n' in google/cloud/pubsub_v1/gapic/publisher_client.py.
2020-04-03 05:14:09,338 synthtool > Replaced 'DESCRIPTOR = _MESSAGESTORAGEPOLICY,\n\\s+__module__.*\n\\s+,\n\\s+__doc__ = """' in google/cloud/pubsub_v1/proto/pubsub_pb2.py.
2020-04-03 05:14:09,340 synthtool > No replacements made in google/cloud/pubsub_v1/gapic/subscriber_client.py for pattern subscription \(str\): The subscription whose backlog .*
(.*
)+?\s+Format is .*, maybe replacement is no longer needed?
2020-04-03 05:14:09,341 synthtool > Replaced 'import functools\n' in google/cloud/pubsub_v1/gapic/publisher_client.py.
2020-04-03 05:14:09,341 synthtool > Replaced 'import pkg_resources\n' in google/cloud/pubsub_v1/gapic/publisher_client.py.
2020-04-03 05:14:09,342 synthtool > Replaced 'class PublisherClient' in google/cloud/pubsub_v1/gapic/publisher_client.py.
2020-04-03 05:14:09,343 synthtool > Replaced 'client_config \\(dict\\): DEPRECATED.' in google/cloud/pubsub_v1/gapic/publisher_client.py.
2020-04-03 05:14:09,344 synthtool > Replaced '# Raise deprecation warnings .*\n.*\n.*\n.*\n.*\n.*\n' in google/cloud/pubsub_v1/gapic/publisher_client.py.
2020-04-03 05:14:09,344 synthtool > Replaced '~google.api_core.page_iterator.PageIterator' in google/cloud/pubsub_v1/gapic/publisher_client.py.
2020-04-03 05:14:09,345 synthtool > Replaced '~google.api_core.page_iterator.PageIterator' in google/cloud/pubsub_v1/gapic/subscriber_client.py.
2020-04-03 05:14:09,345 synthtool > Replaced 'from google.iam.v1 import iam_policy_pb2' in google/cloud/pubsub_v1/gapic/transports/publisher_grpc_transport.py.
2020-04-03 05:14:09,346 synthtool > Replaced 'from google.iam.v1 import iam_policy_pb2' in google/cloud/pubsub_v1/gapic/transports/subscriber_grpc_transport.py.
.coveragerc
.flake8
.github/CONTRIBUTING.md
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/release-please.yml
.gitignore
.kokoro/build.sh
.kokoro/continuous/common.cfg
.kokoro/continuous/continuous.cfg
.kokoro/docs/common.cfg
.kokoro/docs/docs.cfg
.kokoro/presubmit/common.cfg
.kokoro/presubmit/presubmit.cfg
.kokoro/publish-docs.sh
.kokoro/release.sh
.kokoro/release/common.cfg
.kokoro/release/release.cfg
.kokoro/trampoline.sh
CODE_OF_CONDUCT.md
CONTRIBUTING.rst
LICENSE
MANIFEST.in
docs/_static/custom.css
docs/_templates/layout.html
docs/conf.py.j2
noxfile.py.j2
renovate.json
setup.cfg
2020-04-03 05:14:09,731 synthtool > No replacements made in noxfile.py for pattern session\.install\("-e", "\.\./test_utils/"\), maybe replacement is no longer needed?
Running session blacken
Creating virtual environment (virtualenv) using python3.6 in .nox/blacken
pip install black==19.3b0
black docs google tests noxfile.py setup.py
reformatted /tmpfs/src/git/autosynth/working_repo/google/__init__.py
reformatted /tmpfs/src/git/autosynth/working_repo/google/cloud/__init__.py
reformatted /tmpfs/src/git/autosynth/working_repo/google/cloud/pubsub.py
reformatted /tmpfs/src/git/autosynth/working_repo/docs/conf.py
reformatted /tmpfs/src/git/autosynth/working_repo/google/cloud/pubsub_v1/gapic/publisher_client_config.py
reformatted /tmpfs/src/git/autosynth/working_repo/google/cloud/pubsub_v1/gapic/transports/publisher_grpc_transport.py
reformatted /tmpfs/src/git/autosynth/working_repo/google/cloud/pubsub_v1/gapic/subscriber_client_config.py
reformatted /tmpfs/src/git/autosynth/working_repo/google/cloud/pubsub_v1/gapic/transports/subscriber_grpc_transport.py
reformatted /tmpfs/src/git/autosynth/working_repo/google/cloud/pubsub_v1/gapic/publisher_client.py
reformatted /tmpfs/src/git/autosynth/working_repo/google/cloud/pubsub_v1/proto/pubsub_pb2_grpc.py
reformatted /tmpfs/src/git/autosynth/working_repo/google/cloud/pubsub_v1/gapic/subscriber_client.py
reformatted /tmpfs/src/git/autosynth/working_repo/tests/system/gapic/v1/test_system_publisher_v1.py
reformatted /tmpfs/src/git/autosynth/working_repo/noxfile.py
reformatted /tmpfs/src/git/autosynth/working_repo/tests/unit/gapic/v1/test_publisher_client_v1.py
reformatted /tmpfs/src/git/autosynth/working_repo/tests/unit/gapic/v1/test_subscriber_client_v1.py
reformatted /tmpfs/src/git/autosynth/working_repo/google/cloud/pubsub_v1/proto/pubsub_pb2.py
All done! ✨ 🍰 ✨
16 files reformatted, 56 files left unchanged.
Session blacken was successful.
2020-04-03 05:14:17,612 synthtool > Wrote metadata to synth.metadata.

…ary generator.

plamut

More of a quick review, but the test dependency installation is messed up in noxfile. We either need to fix the template, or make a manual adjustment in synth.py.

plamut · 2020-04-06T18:17:31Z

CONTRIBUTING.rst

@@ -22,7 +22,7 @@ In order to add a feature:
  documentation.

 - The feature must work fully on the following CPython versions:  2.7,
-  3.5, 3.6, and 3.7 on both UNIX and Windows.
+  3.5, 3.6, 3.7 and 3.8 on both UNIX and Windows.


Missing Oxford comma here, but this is probably the mother of all nits. :D

plamut · 2020-04-06T18:19:30Z

noxfile.py


    session.install("-e", "test_utils")
+    session.install("-e", "psutil")


Psutil is an external test dependency, why has it been changed to an editable install? This looks like something that could fail.

Edit: Indeed, that's the cause of the failing system tests.

@hongalex I think you were looking at the psutil issue?

Hm, yeah I added psutil in this PR but I didn't realize it was an editable install. Every time we try to merge in a gapic generated PR, it tries to remove psutil from the dependencies, so we have to manually add it in.

@busunkim96, I tried looking for other places where psutil should be (re)added, but can't seem to find it. Any ideas?

I asked Alex to add the extra dependency via synth.py so it doesn't get overridden in subsequent PRs. But the noxfile template incorrectly assumes that all extra dependencies are local. :) I'll open a PR in synthtool.

https://github.com/googleapis/synthtool/blob/master/synthtool/gcp/templates/python_library/noxfile.py.j2

googleapis/synthtool#469

Thanks for the quick fix of the synthtool!

I pushed a commit with the regeneration. PTAL and merge when you're ready. :)

plamut

Looks good IMO.

plamut · 2020-04-21T08:23:59Z

noxfile.py

-    session.install("-e", "test_utils")
-    session.install("-e", "psutil")
+    session.install("mock", "pytest", "psutil")
+    session.install("git+https://github.com/googleapis/python-test-utils")


Nice, thanks! ❤️

BTW, is now time to remove the local test_utils directory? (I can do it, in a separate PR)

plamut · 2020-04-21T08:28:59Z

noxfile.py

@@ -141,7 +139,7 @@ def docs(session):
    """Build the docs for this library."""

    session.install("-e", ".")
-    session.install("sphinx", "alabaster", "recommonmark")
+    session.install("sphinx<3.0.0", "alabaster", "recommonmark")


Pinning Sphinx to < 3.x is not needed anymore, we fixed all the warnings in #70.

Not a show-stopper, though, just something we can clean in the next autogen pass.

Edit:
It turns out that some of the new changes introduced new Sphinx warnings, thus the pin is (again) necessary until it gets fixed.

docstring of google.cloud.pubsub_v1.types.AcknowledgeRequest:3:Unexpected indentation.

@busunkim96 Is this generator issue known, i.e. accidentally introducing docs warnings in Sphinx 3.x? Do we have a feedback loop mechanism to make this visible to generator maintainers?

[CHANGE ME] Re-generated to pick up changes in the API or client libr…

96f4f82

…ary generator.

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 3, 2020

hongalex requested review from crwilcox and pradn and removed request for crwilcox April 6, 2020 17:41

pradn approved these changes Apr 6, 2020

View reviewed changes

pradn requested a review from plamut April 6, 2020 17:59

plamut suggested changes Apr 6, 2020

View reviewed changes

busunkim96 and others added 2 commits April 21, 2020 00:18

chore: manual regen

3355f9c

Merge branch 'master' into autosynth

1f049c0

plamut approved these changes Apr 21, 2020

View reviewed changes

plamut merged commit 4ce898e into master Apr 21, 2020

plamut deleted the autosynth branch April 21, 2020 08:49

release-please bot mentioned this pull request Aug 11, 2022

chore(main): release 2.13.6 #761

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[CHANGE ME] Re-generated to pick up changes in the API or client library generator. #66

[CHANGE ME] Re-generated to pick up changes in the API or client library generator. #66

yoshi-automation commented Apr 3, 2020

plamut left a comment •

edited

plamut Apr 6, 2020

plamut Apr 6, 2020

pradn Apr 6, 2020

hongalex Apr 6, 2020

busunkim96 Apr 6, 2020

busunkim96 Apr 7, 2020

plamut Apr 7, 2020

busunkim96 Apr 21, 2020

plamut left a comment

plamut Apr 21, 2020

plamut Apr 21, 2020 •

edited


		session.install("-e", "test_utils")
		session.install("-e", "psutil")

[CHANGE ME] Re-generated to pick up changes in the API or client library generator. #66

[CHANGE ME] Re-generated to pick up changes in the API or client library generator. #66

Conversation

yoshi-automation commented Apr 3, 2020

plamut left a comment • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

plamut left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

plamut Apr 21, 2020 • edited

Choose a reason for hiding this comment

plamut left a comment •

edited

plamut Apr 21, 2020 •

edited