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

WIP: grpcio: 1.23.0 -> 1.24.2 #71371

Merged
merged 3 commits into from Oct 23, 2019
Merged

WIP: grpcio: 1.23.0 -> 1.24.2 #71371

merged 3 commits into from Oct 23, 2019

Conversation

tomberek
Copy link
Contributor

Motivation for this change

There is an added feature, wait_for_termination, that was needed

Things done

Bump grpcio src and sha.

  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.
Notify maintainers

cc @vanschelven

@marsam
Copy link
Contributor

marsam commented Oct 19, 2019

Would you mind also updating grpc, and pythonPackages.grpcio-tools? those packages need to be in-sync.
Also, can you change the commit message to pythonPackages.grpcio: 1.23.0 -> 1.24.2

@tomberek
Copy link
Contributor Author

there isn't a release of 1.24.2 for pythonPackages.grpcio-tools. Not sure if we caught upstream mid-release. I've updated this PR with 1.24.2 for grpc and 1.24.1 for pythonPackages.grpcio-tools

@ofborg ofborg bot requested review from LnL7 and marsam October 20, 2019 22:07
@tomberek tomberek changed the title grpcio: 1.23.0 -> 1.24.2 WIP: grpcio: 1.23.0 -> 1.24.2 Oct 20, 2019
@tomberek
Copy link
Contributor Author

This causes problems with pythonPackages.tensorflow. Investigating...

Copy link
Contributor

@jonringer jonringer left a comment

Choose a reason for hiding this comment

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

a lot of the python38 failures are not associated with this PR:

[122 built (11 failed), 572 copied (7783.4 MiB), 2948.4 MiB DL]
error: build of '/nix/store/h6qlb3rhyz0phl1pc7d1jprfcl48d2vi-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/71371
4 package are marked as broken and were skipped:
linuxPackages_4_4.sysdig linuxPackages_4_9.sysdig linuxPackages_hardkernel_4_14.sysdig linuxPackages_hardkernel_latest.sysdig

67 package failed to build:
python27Packages.baselines python27Packages.dm-sonnet python27Packages.edward python27Packages.graph_nets python27Packages.tensorflow python27Packages.tensorflow-probability python27Packages.tensorflowWithCuda python27Packages.tflearn python37Packages.baselines python37Packages.dm-sonnet python37Packages.edward python37Packages.graph_nets python37Packages.optuna python37Packages.rl-coach python37Packages.tensorflow python37Packages.tensorflow-probability python37Packages.tensorflowWithCuda python37Packages.tflearn python38Packages.baselines python38Packages.dm-sonnet python38Packages.edward python38Packages.google_api_core 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_securitycenter python38Packages.google_cloud_spanner python38Packages.google_cloud_speech python38Packages.google_cloud_storage python38Packages.google_cloud_tasks 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.graph_nets python38Packages.grpc_google_iam_v1 python38Packages.grpcio python38Packages.grpcio-gcp python38Packages.grpcio-tools python38Packages.optuna python38Packages.rl-coach python38Packages.tensorflow python38Packages.tensorflow-probability python38Packages.tensorflow-tensorboard python38Packages.tensorflowWithCuda python38Packages.tflearn

95 package were build:
google-cloud-cpp grpc linuxPackages-libre.sysdig linuxPackages.sysdig linuxPackages_4_14.sysdig linuxPackages_5_2.sysdig linuxPackages_5_3.sysdig linuxPackages_hardened.sysdig linuxPackages_latest-libre.sysdig linuxPackages_latest_hardened.sysdig linuxPackages_latest_xen_dom0.sysdig linuxPackages_testing_bcachefs.sysdig linuxPackages_testing_hardened.sysdig linuxPackages_xen_dom0.sysdig python27Packages.google_api_core python27Packages.google_cloud_asset python27Packages.google_cloud_automl python27Packages.google_cloud_bigquery python27Packages.google_cloud_bigquery_datatransfer python27Packages.google_cloud_bigtable python27Packages.google_cloud_container python27Packages.google_cloud_core python27Packages.google_cloud_dataproc python27Packages.google_cloud_datastore python27Packages.google_cloud_dlp python27Packages.google_cloud_dns python27Packages.google_cloud_error_reporting python27Packages.google_cloud_firestore python27Packages.google_cloud_iot python27Packages.google_cloud_kms python27Packages.google_cloud_language python27Packages.google_cloud_logging python27Packages.google_cloud_monitoring python27Packages.google_cloud_pubsub python27Packages.google_cloud_redis python27Packages.google_cloud_resource_manager python27Packages.google_cloud_runtimeconfig python27Packages.google_cloud_securitycenter python27Packages.google_cloud_spanner python27Packages.google_cloud_speech python27Packages.google_cloud_storage python27Packages.google_cloud_tasks python27Packages.google_cloud_texttospeech python27Packages.google_cloud_trace python27Packages.google_cloud_translate python27Packages.google_cloud_videointelligence python27Packages.google_cloud_vision python27Packages.google_cloud_websecurityscanner python27Packages.grpc_google_iam_v1 python27Packages.grpcio python27Packages.grpcio-gcp python27Packages.grpcio-tools python27Packages.tensorflow-bin python27Packages.tensorflow-tensorboard python37Packages.google_api_core 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_securitycenter python37Packages.google_cloud_spanner python37Packages.google_cloud_speech python37Packages.google_cloud_storage python37Packages.google_cloud_tasks 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.grpc_google_iam_v1 python37Packages.grpcio python37Packages.grpcio-gcp python37Packages.grpcio-tools python37Packages.tensorflow-bin python37Packages.tensorflow-tensorboard sysdig

marsam and others added 2 commits October 23, 2019 04:20
Co-authored-by: Tom Bereknyei <tomberek@users.noreply.github.com>
@marsam
Copy link
Contributor

marsam commented Oct 23, 2019

@tomberek thanks for your contribution; I've made minor cleanups, I hope you don't mind

@GrahamcOfBorg build grpc pythonPackages.grpcio pythonPackages.grpcio-tools

@marsam marsam merged commit 54c4d89 into NixOS:master Oct 23, 2019
@tomberek
Copy link
Contributor Author

@marsam no problem at all. Thank you!

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

Successfully merging this pull request may close these issues.

None yet

3 participants