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

chore: update user creds for system test #948

Merged
merged 3 commits into from
Jan 19, 2022
Merged

Conversation

arithmetic1728
Copy link
Contributor

No description provided.

@arithmetic1728 arithmetic1728 requested review from silvolu and a team as code owners January 18, 2022 22:50
@parthea parthea added automerge Merge the pull request once unit tests and other checks pass. kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Jan 18, 2022
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Jan 18, 2022
@gcf-merge-on-green gcf-merge-on-green bot merged commit 1b42b63 into main Jan 19, 2022
@gcf-merge-on-green gcf-merge-on-green bot deleted the update_token branch January 19, 2022 00:10
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jan 19, 2022
@gnossen
Copy link

gnossen commented Jan 20, 2022

@arithmetic1728 I think this change may have broken the 2.4.0 release. google-auth does not declare a dependency on urllib3. grpcio is seeing this issue in CI:

  ...
  File "/var/local/git/grpc/src/python/grpcio_tests/tests/_loader.py", line 74, in walk_packages
    module = importer.find_module(module_name).load_module(module_name)
  File "<frozen importlib._bootstrap_external>", line 462, in _check_name_wrapper
  File "<frozen importlib._bootstrap_external>", line 962, in load_module
  File "<frozen importlib._bootstrap_external>", line 787, in load_module
  File "<frozen importlib._bootstrap>", line 265, in _load_module_shim
  File "<frozen importlib._bootstrap>", line 702, in _load
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 783, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/var/local/git/grpc/src/python/grpcio_tests/tests_aio/interop/client.py", line 23, in <module>
    from tests.interop import client as interop_client_lib
  File "/var/local/git/grpc/src/python/grpcio_tests/tests/interop/client.py", line 24, in <module>
    from tests.interop import methods
  File "/var/local/git/grpc/src/python/grpcio_tests/tests/interop/methods.py", line 33, in <module>
    from google.auth.transport import requests as google_auth_transport_requests
  File "/var/local/git/grpc/py38/lib/python3.8/site-packages/google/auth/transport/requests.py", line 40, in <module>
    from urllib3.util.ssl_ import (
ModuleNotFoundError: No module named 'urllib3'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants