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
python3Packages.google_auth: 1.20.1 -> 1.22.1 #101029
Conversation
13b53eb
to
ccc83b0
Compare
As I figured it might, this package requiring an updated
|
d24f006
to
e3d40e5
Compare
8bd28ed
to
8098d2a
Compare
All right, worked through all the issues except a Tensorflow package that doesn't build on master anyways. |
@jonringer @costrouc mind taking a look? Also now that 20.09 is released would I still backport this? |
you may have to remove some "broken = true;", but yea. I'm fine with backporting non-major bumps. And if it doesn't cause regressions. |
only one I'm concerned about backporting:
|
pyarrow | ||
]; | ||
|
||
# prevent google directory from shadowing google imports | ||
# call_api_applying_custom_retry_on_timeout requires credentials | ||
# test_magics requires modifying sys.path | ||
checkPhase = '' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe disable this test?
builder for '/nix/store/2nm6nj24sswksj2p5gw7myyddpbywg4l-python3.8-google-cloud-bigquery-2.1.0.drv' failed with exit code 1; last 10 log lines:
E datetime.datetime(4567, 1, 1, 0, 0)
E
E + [datetime.datetime(4567, 1, 1, 0, 0), datetime.datetime(9999, 12, 31, 0, 0)]
E - [datetime.datetime(4567, 1, 1, 0, 0, tzinfo=<UTC>),
E - datetime.datetime(9999, 12, 31, 0, 0, tzinfo=<UTC>)]
tests/unit/test_table.py:2294: AssertionError
=========================== short test summary info ============================
FAILED tests/unit/test_table.py::TestRowIterator::test_to_dataframe_timestamp_out_of_pyarrow_bounds
========== 1 failed, 1372 passed, 54 skipped, 1 deselected in 14.60s ===========
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pytestCheckHook
would make this easy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Strange, all tests pass on my machine. Just checked again with nix-build -A python38Packages.google_cloud_bigquery
. I can disable the test, though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Disabled.
Not sure what you mean. I didn't explicitly add any
Are you still evaluating or do you want me to look into if it's possible to keep this on 1.x? |
I did on release-20.09, as part of the release #101736 |
For backporting, I would prefer not to backport this, unless it's already broken |
8098d2a
to
b924871
Compare
Just checked and this one was not broken on 20.09. So I will keep that in mind when backporting. |
87dce67
to
1706241
Compare
@jonringer would you mind taking another look? I think this is good to go. |
@jonringer pretty please? 🙇 |
oh yea, sorry, last time it was just taking longer than expected to review these packages |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
failures are broken on target branch
https://github.com/NixOS/nixpkgs/pull/101029
2 packages marked as broken and skipped:
kmymoney python37Packages.rl-coach
1 package failed to build:
python37Packages.tensorflow-bin_2
118 packages built:
beancount deja-dup duplicity duply dvc-with-remotes fava gcalcli ledger2beancount lieer patroni python37Packages.beancount python37Packages.beancount_docverif python37Packages.cirq python37Packages.goobook python37Packages.google-auth-httplib2 python37Packages.google-auth-oauthlib python37Packages.google-cloud-access-context-manager python37Packages.google-cloud-iam python37Packages.google-cloud-org-policy python37Packages.google_api_core python37Packages.google_api_python_client python37Packages.google_auth python37Packages.google_cloud_asset python37Packages.google_cloud_automl python37Packages.google_cloud_bigquery python37Packages.google_cloud_bigquery_datatransfer python37Packages.google_cloud_bigtable python37Packages.google_cloud_container python37Packages.google_cloud_core python37Packages.google_cloud_dataproc python37Packages.google_cloud_datastore python37Packages.google_cloud_dlp python37Packages.google_cloud_dns python37Packages.google_cloud_error_reporting python37Packages.google_cloud_firestore python37Packages.google_cloud_iot python37Packages.google_cloud_kms python37Packages.google_cloud_language python37Packages.google_cloud_logging python37Packages.google_cloud_monitoring python37Packages.google_cloud_pubsub python37Packages.google_cloud_redis python37Packages.google_cloud_resource_manager python37Packages.google_cloud_runtimeconfig python37Packages.google_cloud_secret_manager python37Packages.google_cloud_securitycenter python37Packages.google_cloud_spanner python37Packages.google_cloud_speech python37Packages.google_cloud_storage python37Packages.google_cloud_tasks python37Packages.google_cloud_testutils python37Packages.google_cloud_texttospeech python37Packages.google_cloud_trace python37Packages.google_cloud_translate python37Packages.google_cloud_videointelligence python37Packages.google_cloud_vision python37Packages.google_cloud_websecurityscanner python37Packages.gspread python37Packages.kubernetes python37Packages.proto-plus python37Packages.pydrive python37Packages.tensorflow-tensorboard_2 python37Packages.weboob python38Packages.beancount_docverif python38Packages.cirq python38Packages.goobook python38Packages.google-auth-httplib2 python38Packages.google-auth-oauthlib python38Packages.google-cloud-access-context-manager python38Packages.google-cloud-iam python38Packages.google-cloud-org-policy python38Packages.google_api_core python38Packages.google_api_python_client python38Packages.google_auth python38Packages.google_cloud_asset python38Packages.google_cloud_automl python38Packages.google_cloud_bigquery python38Packages.google_cloud_bigquery_datatransfer python38Packages.google_cloud_bigtable python38Packages.google_cloud_container python38Packages.google_cloud_core python38Packages.google_cloud_dataproc python38Packages.google_cloud_datastore python38Packages.google_cloud_dlp python38Packages.google_cloud_dns python38Packages.google_cloud_error_reporting python38Packages.google_cloud_firestore python38Packages.google_cloud_iot python38Packages.google_cloud_kms python38Packages.google_cloud_language python38Packages.google_cloud_logging python38Packages.google_cloud_monitoring python38Packages.google_cloud_pubsub python38Packages.google_cloud_redis python38Packages.google_cloud_resource_manager python38Packages.google_cloud_runtimeconfig python38Packages.google_cloud_secret_manager python38Packages.google_cloud_securitycenter python38Packages.google_cloud_spanner python38Packages.google_cloud_speech python38Packages.google_cloud_storage python38Packages.google_cloud_tasks python38Packages.google_cloud_testutils python38Packages.google_cloud_texttospeech python38Packages.google_cloud_trace python38Packages.google_cloud_translate python38Packages.google_cloud_videointelligence python38Packages.google_cloud_vision python38Packages.google_cloud_websecurityscanner python38Packages.gspread python38Packages.kubernetes python38Packages.proto-plus python38Packages.pydrive python38Packages.tensorflow-tensorboard_2 python38Packages.weboob s3ql tuijam wal_e
All good, was just making me anxious since it was a large amount of packages and already had to merge in new changes once or twice. Thank you! |
Motivation for this change
Upgrades
google_auth
, fixes many Google Cloud packages. Will backport to 20.09 for ZHF.https://hydra.nixos.org/build/127647323
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)