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.

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

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

Merged
merged 3 commits into from Apr 21, 2020

Conversation

@yoshi-automation
Copy link
Contributor

@yoshi-automation yoshi-automation commented Apr 3, 2020

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.

@hongalex hongalex requested review from crwilcox and pradn and removed request for crwilcox Apr 6, 2020
pradn
pradn approved these changes Apr 6, 2020
@pradn pradn requested a review from plamut Apr 6, 2020
Copy link
Contributor

@plamut plamut left a comment

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.

Loading

@@ -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.
Copy link
Contributor

@plamut plamut Apr 6, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Loading

noxfile.py Outdated

session.install("-e", "test_utils")
session.install("-e", "psutil")
Copy link
Contributor

@plamut plamut Apr 6, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Loading

Copy link
Contributor

@pradn pradn Apr 6, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Loading

Copy link
Member

@hongalex hongalex Apr 6, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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?

Loading

Copy link
Collaborator

@busunkim96 busunkim96 Apr 6, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Loading

Copy link
Collaborator

@busunkim96 busunkim96 Apr 7, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Loading

Copy link
Contributor

@plamut plamut Apr 7, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick fix of the synthtool!

Loading

Copy link
Collaborator

@busunkim96 busunkim96 Apr 21, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Loading

plamut
plamut approved these changes Apr 21, 2020
Copy link
Contributor

@plamut plamut left a comment

Looks good IMO.

Loading

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

@plamut plamut Apr 21, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks! ❤️

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

Loading

@@ -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")
Copy link
Contributor

@plamut plamut Apr 21, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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?

Loading

@plamut plamut merged commit 4ce898e into master Apr 21, 2020
2 of 3 checks passed
Loading
@plamut plamut deleted the autosynth branch Apr 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants