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

grpc: 1.25.0 -> 1.26.0 #75917

Merged
merged 2 commits into from Dec 30, 2019
Merged

grpc: 1.25.0 -> 1.26.0 #75917

merged 2 commits into from Dec 30, 2019

Conversation

marsam
Copy link
Contributor

@marsam marsam commented Dec 19, 2019

Motivation for this change

Changelog: https://github.com/grpc/grpc/releases/tag/v1.26.0

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • 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 @

@jonringer
Copy link
Contributor

this causes ~150 rebuilds, staging might be more appropriate

@marsam
Copy link
Contributor Author

marsam commented Dec 19, 2019

AFAIK ~150 rebuilds should be fine for master, I'm not sure if is written somewhere but according #nixos-dev, ~2500 is the "limit" #67976 (comment)

@jonringer
Copy link
Contributor

I think it's more a courtesy to others hacking on master, so they don't have to build a potentially large number of packages to build the package they are modifying.

Personally, I have a 3900X that generally eats through most builds :).

@jonringer
Copy link
Contributor

sigh... tensorflow and pytorch are killing me when reviewing this :(

@FRidh
Copy link
Member

FRidh commented Dec 19, 2019

<500 is typically fine for master. With @r-ryantm we typically put it at 100~500 because it's often a batch of changes.

@jonringer
Copy link
Contributor

there is a regression in one of these packages, but I'm not sure which:

44 package failed to build:
dvc-with-remotes python27Packages.tensorflow-bin python37Packages.dm-sonnet python37Packages.google_cloud_automl python37Packages.google_cloud_bigquery python37Packages.google_cloud_bigtable python37Packages.google_cloud_container python37Packages.google_cloud_datastore python37Packages.google_cloud_dns python37Packages.google_cloud_error_reporting python37Packages.google_cloud_logging python37Packages.google_cloud_resource_manager python37Packages.google_cloud_runtimeconfig python37Packages.google_cloud_spanner python37Packages.google_cloud_storage python37Packages.graph_nets python37Packages.optuna python37Packages.pytorch python37Packages.rl-coach python37Packages.tensorflow-bin python37Packages.tensorflowWithCuda python37Packages.torchvision python38Packages.baselines python38Packages.dm-sonnet python38Packages.edward python38Packages.google_cloud_automl python38Packages.google_cloud_bigtable python38Packages.google_cloud_container python38Packages.google_cloud_datastore python38Packages.google_cloud_dns python38Packages.google_cloud_error_reporting python38Packages.google_cloud_logging python38Packages.google_cloud_resource_manager python38Packages.google_cloud_runtimeconfig python38Packages.google_cloud_spanner python38Packages.google_cloud_storage python38Packages.graph_nets python38Packages.pytorch python38Packages.pytorchWithCuda python38Packages.tensorflow python38Packages.tensorflow-probability python38Packages.tensorflowWithCuda python38Packages.tflearn python38Packages.torchvision
nix build -f . --keep-going dvc-with-remotes python27Packages.tensorflow-bin python37Packages.dm-sonnet python37Packages.google_cloud_automl python37Packages.google_cloud_bigquery python37Packages.google_cloud_bigtable python37Packages.google_cloud_container python37Packages.google_cloud_datastore python37Packages.google_cloud_dns python37Packages.google_cloud_error_reporting python37Packages.google_cloud_logging python37Packages.google_cloud_resource_manager python37Packages.google_cloud_runtimeconfig python37Packages.google_cloud_spanner python37Packages.google_cloud_storage python37Packages.graph_nets python37Packages.pytorch python37Packages.rl-coach python37Packages.tensorflow-bin python37Packages.tensorflowWithCuda python37Packages.torchvision python38Packages.baselines python38Packages.dm-sonnet python38Packages.edward python38Packages.google_cloud_automl python38Packages.google_cloud_bigtable python38Packages.google_cloud_container python38Packages.google_cloud_datastore python38Packages.google_cloud_dns python38Packages.google_cloud_error_reporting python38Packages.google_cloud_logging python38Packages.google_cloud_resource_manager python38Packages.google_cloud_runtimeconfig python38Packages.google_cloud_spanner python38Packages.google_cloud_storage python38Packages.graph_nets python38Packages.pytorch python38Packages.pytorchWithCuda python38Packages.tensorflow python38Packages.tensorflow-probability python38Packages.tensorflowWithCuda python38Packages.tflearn python38Packages.torchvision
....
[1 built (30 failed), 12 copied (321.9 MiB), 47.6 MiB DL]
error: build of '/nix/store/072c75lbh6bqqxqz6jj4ay7gip10dsil-python3.7-tensorflow-gpu-1.15.0.drv', '/nix/store/168xml4sbf8b7112zkrzv2lmnhc5r1p5-python3.7-google-cloud-container-0.3.0.drv', '/nix/store/1bay0jr7px3203mabgzf9nb00hnb7h6z-python2.7-tensorflow-1.14.0.drv', '/nix/store/28hhi8c0ymq2kl0mfzspvzvb0plgdh6k-python3.8-google-cloud-spanner-1.11.0.drv', '/nix/store/2nl20xfn08dra6029fy97v8gngq2fja1-python3.7-google-cloud-resource-manager-0.30.0.drv', '/nix/store/3604ccdwinh30nhgjr418wdh5m92jisx-dvc-0.24.3.drv', '/nix/store/3vsr3vj8hwvri46zbggn5vqk44z5vx0w-python3.8-google-cloud-logging-1.14.0.drv', '/nix/store/4dxx60fsmmylz5wy8nl7i0vba4v3g4xy-python3.8-google-cloud-runtimeconfig-0.30.0.drv', '/nix/store/4rl1mx24ib7vvkfbsk5g1q996pzm52nr-python3.8-google-cloud-bigtable-1.1.0.drv', '/nix/store/4y5lv4f57nbl5nfi670xb49yjvybpr2l-python3.8-baselines-0.1.6.drv', '/nix/store/65f4barc1h6v02hwvxb9jig08wj02j3v-python3.8-google-cloud-dns-0.31.0.drv', '/nix/store/6cxnj286hpv3zp9bgqx972bycxkvmgky-python3.8-google-cloud-automl-0.7.1.drv', '/nix/store/6k82hdhr6n207qlzxzk2mi2bq868mmis-python3.8-edward-1.3.5.drv', '/nix/store/7f9kzc3fk07m83xwj96ygrvklf3v6x0w-python3.7-google-cloud-bigtable-1.1.0.drv', '/nix/store/8rfjjwpjp12cg9dv4zdqzb0j0y40r7ry-python3.8-dm-sonnet-1.33.drv', '/nix/store/aaks3g8rrm4pk0vfc8xsnvnirv1fz6yg-python3.8-google-cloud-error-reporting-0.33.0.drv', '/nix/store/dq4172sh2dbwifgz1fgldzgw2izpp924-python3.7-google-cloud-logging-1.14.0.drv', '/nix/store/fs0r56klkxyay2ij4fx34cr5ax7g32wd-python3.7-google-cloud-bigquery-1.21.0.drv', '/nix/store/hgzy3l4wqsi7w17xgw2yhrrngg2qn6dl-python3.8-google-cloud-datastore-1.10.0.drv', '/nix/store/hnc15vb59fnp2f4yrrw904hs18f829gj-python3.7-google-cloud-storage-1.20.0.drv', '/nix/store/j17g9qp3d5h5lwj3jhxv6vyw0b7zcp2m-python3.7-google-cloud-datastore-1.10.0.drv', '/nix/store/jgmmmsavra6697k35z418nzm5sxwr8dw-python3.7-google-cloud-automl-0.7.1.drv', '/nix/store/kmvjmyglg7r6zvfnx1rcqz3vbyd5hsyr-python3.8-tensorflow-1.15.0.drv', '/nix/store/m6iw01xxi8vcyk3b955zzrh8vcy3zdm8-python3.8-google-cloud-storage-1.20.0.drv', '/nix/store/nz4156l77wmv0yrqcqas49c7bbhbznaq-python3.7-rl-coach-1.0.1.drv', '/nix/store/p88wfmb5zps6xwbr9is6i9i4w297bxb4-python3.7-google-cloud-dns-0.31.0.drv', '/nix/store/r1ga4ln21yp74m8r9m36apw0v36h9xga-python3.8-google-cloud-container-0.3.0.drv', '/nix/store/r9v9d0ffk93aa8x3ql54p6db8l1fkhdi-python3.8-torchvision-0.2.1.drv', '/nix/store/riq2804wjzwsxs0j5gr72m5iq4g40s75-python3.7-dm-sonnet-1.33.drv', '/nix/store/ry8yq8w299wk2p0l2nzgad0a06mk1r9y-python3.7-graph_nets-1.0.5.drv', '/nix/store/sfh0d417f2iz08pshxp0bhzmlpn6vvnz-python3.8-graph_nets-1.0.5.drv', '/nix/store/vx82ca4v812jr08xn6vq2ggkp2j9i7li-python3.8-tflearn-0.3.2.drv', '/nix/store/wf85gha22v26n7c8jy88rlkaz3z6yhs4-python3.8-tensorflow-gpu-1.15.0.drv', '/nix/store/wfa8gildy2hkmqcqnh7q9dgknjj0n4rl-python3.8-pytorch-1.2.0.drv', '/nix/store/wih38y9kr5mfzg8byb271zk0b5xkllw7-python3.8-google-cloud-resource-manager-0.30.0.drv', '/nix/store/xazz8f110ybibc13hlkzchqx142bga42-python3.7-google-cloud-error-reporting-0.33.0.drv', '/nix/store/y6x67xifxkvg9fl079vlrskqxqi18gsm-python3.8-tensorflow_probability-0.7.drv', '/nix/store/yv3jcskxfy9m55bbw2wi1g2rk6g2gfz6-python3.7-tensorflow-1.14.0.drv', '/nix/store/zai5wczvjb7n7siq3wrqqrqkvw0jv5sk-python3.8-pytorch-1.2.0.drv', '/nix/store/zhji2a0mpm6jn1kbh3hj1pz13lj8lviv-python3.7-google-cloud-spanner-1.11.0.drv', '/nix/store/ziiyd22p6wb5hkywdvwprq4khjnh6711-python3.7-google-cloud-runtimeconfig-0.30.0.drv' failed

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.

LGTM

failures seem to be mostly related to google python packages are broken, or they are broken on master

[48 built (31 failed), 58 copied (1890.5 MiB), 807.9 MiB DL]
error: build of '/nix/store/nw7b7wbhrjf193mp0dpmhdn5zn2dc8mi-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/75917
16 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 python27Packages.baselines python27Packages.dm-sonnet python27Packages.edward python27Packages.graph_nets python27Packages.tensorflow python27Packages.tensorflow-build python27Packages.tensorflow-probability python27Packages.tensorflowWithCuda python27Packages.tensorflowWithoutCuda python27Packages.tflearn python37Packages.pyro-ppl python38Packages.pyro-ppl

51 package failed to build:
dvc-with-remotes python27Packages.tensorflow-bin python37Packages.arviz python37Packages.baselines python37Packages.dm-sonnet python37Packages.edward python37Packages.google_cloud_automl python37Packages.google_cloud_bigquery python37Packages.google_cloud_bigtable python37Packages.google_cloud_container python37Packages.google_cloud_datastore python37Packages.google_cloud_dns python37Packages.google_cloud_error_reporting python37Packages.google_cloud_logging python37Packages.google_cloud_resource_manager python37Packages.google_cloud_runtimeconfig python37Packages.google_cloud_spanner python37Packages.google_cloud_storage python37Packages.graph_nets python37Packages.optuna python37Packages.pytorch python37Packages.pytorchWithCuda python37Packages.rl-coach python37Packages.tensorflow python37Packages.tensorflow-bin python37Packages.tensorflow-probability python37Packages.tensorflowWithCuda python37Packages.tflearn python37Packages.torchvision python38Packages.baselines python38Packages.dm-sonnet python38Packages.edward python38Packages.google_cloud_automl python38Packages.google_cloud_bigtable python38Packages.google_cloud_container python38Packages.google_cloud_datastore python38Packages.google_cloud_dns python38Packages.google_cloud_error_reporting python38Packages.google_cloud_logging python38Packages.google_cloud_resource_manager python38Packages.google_cloud_runtimeconfig python38Packages.google_cloud_spanner python38Packages.google_cloud_storage python38Packages.graph_nets python38Packages.pytorch python38Packages.pytorchWithCuda python38Packages.tensorflow python38Packages.tensorflow-probability python38Packages.tensorflowWithCuda python38Packages.tflearn python38Packages.torchvision

73 package were built:
google-cloud-cpp grpc linuxPackages-libre.sysdig linuxPackages.sysdig linuxPackages_4_14.sysdig linuxPackages_5_3.sysdig linuxPackages_5_4.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.grpc_google_iam_v1 python27Packages.grpcio python27Packages.grpcio-gcp python27Packages.grpcio-tools python27Packages.tensorflow-tensorboard python37Packages.google_api_core python37Packages.google_cloud_asset python37Packages.google_cloud_bigquery_datatransfer python37Packages.google_cloud_core python37Packages.google_cloud_dataproc python37Packages.google_cloud_dlp python37Packages.google_cloud_firestore python37Packages.google_cloud_iot python37Packages.google_cloud_kms python37Packages.google_cloud_language python37Packages.google_cloud_monitoring python37Packages.google_cloud_pubsub python37Packages.google_cloud_redis python37Packages.google_cloud_securitycenter python37Packages.google_cloud_speech 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-tensorboard python38Packages.google_api_core python38Packages.google_cloud_asset python38Packages.google_cloud_bigquery_datatransfer python38Packages.google_cloud_core python38Packages.google_cloud_dataproc python38Packages.google_cloud_dlp python38Packages.google_cloud_firestore python38Packages.google_cloud_iot python38Packages.google_cloud_kms python38Packages.google_cloud_language python38Packages.google_cloud_pubsub python38Packages.google_cloud_redis python38Packages.google_cloud_securitycenter python38Packages.google_cloud_speech 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.grpc_google_iam_v1 python38Packages.grpcio python38Packages.grpcio-gcp python38Packages.grpcio-tools python38Packages.tensorflow-tensorboard sysdig

@jonringer jonringer merged commit e1ee43f into NixOS:master Dec 30, 2019
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