Skip to content

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

@tseavertseaver commented Mar 4, 2016

Copy link
Copy Markdown
Contributor

See: #1570

@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 4, 2016
@tseavertseaver closed this Mar 4, 2016
@dhermes

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

tseaver commented Mar 4, 2016

Copy link
Copy Markdown
ContributorAuthor

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

@tseavertseaver reopened this Mar 4, 2016
@dhermes

dhermes commented Mar 4, 2016

Copy link
Copy Markdown
Contributor

Branch has conflicts?

@dhermes

dhermes commented Mar 4, 2016

Copy link
Copy Markdown
Contributor

Also the CLA check is the only check that ran

@tseaver

tseaver commented Mar 4, 2016

Copy link
Copy Markdown
ContributorAuthor

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

@tseaver

tseaver commented Mar 4, 2016

Copy link
Copy Markdown
ContributorAuthor

@dhermes PTAL

dhermes
dhermes reviewed Mar 4, 2016
View reviewed changes
Comment threadtox.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.

Sign in to view

This comment was marked as spam.

Sign in to view

@dhermes

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
Merge pull request#1571from tseaver/1570-avoid-protobuf-3.0.0b2.post1
69391eb
Avoid protobuf 3.0.0.b2.post1.
@tseaver
tseaver merged commit 69391eb into googleapis:masterMar 4, 2016
@tseaver
tseaver deleted the 1570-avoid-protobuf-3.0.0b2.post1 branch March 4, 2016 20:26
@theacodestheacodes mentioned this pull request Mar 4, 2016
Merged
@dhermesdhermes mentioned this pull request Mar 8, 2016
Closed
@bgenchel

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

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
@renovate-bot@parthea
chore(deps): update all dependencies (#1571)
0dabd4e
* 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
@ohmayr@gcf-owl-bot
@arithmetic1728@clundin25@release-please@andyrzhao
feat: Add support for asynchronousAuthorizedSessionapi (#1577)
0d38ef8
* 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>
parthea pushed a commit that referenced this pull request Apr 1, 2026
@chelsea-lin
test: move test_struct_from_dataframe with allow_large_result=False (#…
7c3292f
…1571)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

cla: yesThis human has signed the Contributor License Agreement.packaging

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@tseaver@dhermes@bgenchel@googlebot