FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Avoid protobuf 3.0.0.b2.post1. by tseaver · Pull Request #1571 · googleapis/google-cloud-python · GitHub

Avoid protobuf 3.0.0.b2.post1. - #1571

Merged
tseaver merged 2 commits into
googleapis:masterfrom
tseaver:1570-avoid-protobuf-3.0.0b2.post1
Mar 4, 2016
Merged

Avoid protobuf 3.0.0.b2.post1.#1571
tseaver merged 2 commits into
googleapis:masterfrom
tseaver:1570-avoid-protobuf-3.0.0b2.post1

Conversation

tseaver commented Mar 4, 2016

Copy link
Copy Markdown
Contributor

See: #1570

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 4, 2016
tseaver closed this Mar 4, 2016

dhermes commented Mar 4, 2016

Copy link
Copy Markdown
Contributor

Did you mean to close? I would say I couldn't verify the syntax for the != but I assume you ran it.

tseaver commented Mar 4, 2016

Copy link
Copy Markdown
Contributor Author

The syntax was right, but I ran into other stupidity, and lost the will to live. :)

tseaver reopened this Mar 4, 2016

dhermes commented Mar 4, 2016

Copy link
Copy Markdown
Contributor

Branch has conflicts?

dhermes commented Mar 4, 2016

Copy link
Copy Markdown
Contributor

Also the CLA check is the only check that ran

tseaver commented Mar 4, 2016

Copy link
Copy Markdown
Contributor Author

grpc/grpc#5280 defeated me. I need to back out the forced use of grpcio in the testenv:py27 section to move forward.

tseaver commented Mar 4, 2016

Copy link
Copy Markdown
Contributor Author

@dhermes PTAL

Comment thread tox.ini
PYTHONPATH =
# Until grpcio 0.13.1 ships, this environment is broken on UCS2 builds.
# See: https://github.com/grpc/grpc/issues/5280 and
# https://github.com/grpc/grpc/pull/5319

This comment was marked as spam.

This comment was marked as spam.

dhermes commented Mar 4, 2016

Copy link
Copy Markdown
Contributor

LGTM. One day gRPC will be able to play with the big kids :)

tseaver added a commit that referenced this pull request Mar 4, 2016
tseaver merged commit 69391eb into googleapis:master Mar 4, 2016
tseaver deleted the 1570-avoid-protobuf-3.0.0b2.post1 branch March 4, 2016 20:26

bgenchel commented Aug 2, 2016

Copy link
Copy Markdown

what is the actual fix here? brought here by #1570, saying the issue was fixed in this issue, but this conversation seems pretty unintelligible to me.

dhermes commented Aug 2, 2016

Copy link
Copy Markdown
Contributor

Our fix is just to avoid a b0rken version that the protobuf maintainers uploaded to PyPI.

If you have a stacktrace that seems to contradict, please post it on that issue or open a new one.

parthea added a commit that referenced this pull request Nov 24, 2025
* chore(deps): update all dependencies

* revert

Co-authored-by: Anthonios Partheniou <partheniou@google.com>
parthea pushed a commit that referenced this pull request Nov 26, 2025
* chore: initial setup for async auth sessions api (#1571)

* chore: initial setup for async auth sessions api

* fix whitespace

* add init file

* update file names to aiohttp

* update import statement

* feat: Implement asynchronous timeout context manager (#1569)

* feat: implement async timeout guard

* add docstring

* clean whitespace

* update import file name

* add missing return statement

* update test cases

* update test cases

* include underlying timeout exception in trace

* avoid the cost of actual time

* feat: Implement asynchronous `AuthorizedSession` api response class (#1575)

* feat: implement asynchronous response class for AuthorizedSessions API

* check if aiohttp is installed and avoid tests dependency

* update content to be async

* update docstring to be specific to aiohttp

* add type checking and avoid leaking underlying API responses

* add test case for iterating chunks

* add read method to response interface

* address PR comments

* fix lint issues

* feat: Implement asynchronous `AuthorizedSession` api request class (#1579)

* feat: implement request class for asynchoronous AuthorizedSession API

* add type checking and address TODOs

* remove default values from interface methods

* aiohttp reponse close method must not be awaited

* cleanup

* update Request class docstring

* feat: Implement asynchronous `AuthorizedSession` class (#1580)

* feat: Implement Asynchronous AuthorizedSession class

* add comment for implementing locks within refresh

* move timeout guard to sessions

* add unit tests and code cleanup

* implement async exponential backoff iterator

* cleanup

* add testing for http methods and cleanup

* update number of retries to 3

* refactor test cases

* fix linter and mypy issues

* fix pytest code coverage

* fix: avoid leaking api error for closed session

* add error handling for response

* cleanup default values and add test coverage

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* cleanup: minor code cleanup (#1589)

* chore: Add aiohttp requirements test constraint. (#1566)

See https://github.com/googleapis/google-auth-library-python/issues/1565
for more information.

* chore(main): release 2.33.0 (#1560)

* chore(main): release 2.33.0

* fix: retry token request on retryable status code (#1563)

* fix: retry token request on retryable status code

* feat(auth): Update get_client_ssl_credentials to support X.509 workload certs (#1558)

* feat(auth): Update get_client_ssl_credentials to support X.509 workload certs

* feat(auth): Update has_default_client_cert_source

* feat(auth): Fix formatting

* feat(auth): Fix test__mtls_helper.py

* feat(auth): Fix function name in tests

* chore: Refresh system test creds.

* feat(auth): Fix style

* feat(auth): Fix casing

* feat(auth): Fix linter issue

* feat(auth): Fix coverage issue

---------

Co-authored-by: Carl Lundin <clundin@google.com>
Co-authored-by: Carl Lundin <108372512+clundin25@users.noreply.github.com>

* chore: Update ECP deps. (#1583)

* chore(main): release 2.34.0 (#1574)

* cleanup: minor code cleanup

* fix lint issues

---------

Co-authored-by: Carl Lundin <108372512+clundin25@users.noreply.github.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Andy Zhao <andyzhao@google.com>
Co-authored-by: Carl Lundin <clundin@google.com>

* update secrets from forked repo

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: arithmetic1728 <58957152+arithmetic1728@users.noreply.github.com>
Co-authored-by: Carl Lundin <108372512+clundin25@users.noreply.github.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Andy Zhao <andyzhao@google.com>
Co-authored-by: Carl Lundin <clundin@google.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. packaging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL