Skip to content

Revert "chore(deps): update dependency grpc/grpc to v1.38.1"#6840

Merged
devjgm merged 1 commit intomainfrom
revert-6827-renovate/grpc-grpc-1.x
Jun 22, 2021
Merged

Revert "chore(deps): update dependency grpc/grpc to v1.38.1"#6840
devjgm merged 1 commit intomainfrom
revert-6827-renovate/grpc-grpc-1.x

Conversation

@devjgm
Copy link
Copy Markdown
Contributor

@devjgm devjgm commented Jun 22, 2021

Reverts #6827


This change is Reviewable

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 22, 2021
@devjgm
Copy link
Copy Markdown
Contributor Author

devjgm commented Jun 22, 2021

This revert is because after this PR our demo-centos-7-pr build fails. The issue seems to be related to pkg-config. Likely some diff in grpc/grpc@v1.37.1...v1.38.1

@google-cloud-cpp-bot
Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: ca78f08321067ee61c2dc4b2faaed1a95ee7bf37

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 22, 2021

Codecov Report

Merging #6840 (ca78f08) into main (e69438f) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6840      +/-   ##
==========================================
- Coverage   95.04%   95.03%   -0.01%     
==========================================
  Files        1264     1264              
  Lines      113312   113312              
==========================================
- Hits       107694   107685       -9     
- Misses       5618     5627       +9     
Impacted Files Coverage Δ
...e/cloud/spanner/testing/cleanup_stale_instances.cc 42.42% <0.00%> (-12.13%) ⬇️
...bigtable/examples/bigtable_hello_instance_admin.cc 81.31% <0.00%> (-2.20%) ⬇️
.../cloud/storage/benchmarks/throughput_experiment.cc 74.76% <0.00%> (-0.47%) ⬇️
...cloud/pubsub/internal/subscription_session_test.cc 97.78% <0.00%> (-0.25%) ⬇️
...sub/internal/batching_publisher_connection_test.cc 97.58% <0.00%> (-0.21%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e69438f...ca78f08. Read the comment docs.

@devjgm devjgm marked this pull request as ready for review June 22, 2021 18:13
@devjgm devjgm requested a review from a team June 22, 2021 18:13
@devjgm devjgm merged commit c0be918 into main Jun 22, 2021
@devjgm devjgm deleted the revert-6827-renovate/grpc-grpc-1.x branch June 22, 2021 18:25
devjgm added a commit to devjgm/google-cloud-cpp that referenced this pull request Jun 22, 2021
There's a [bug] w/ CentOS-7's version of `pkg-config` that can cause
invocations to take a reeeeeaaaaly long time to complete. We discovered
this when upgrading to grpc-1.38.1 and had to [rollback] the upgrade.

The fix to `pkg-config` was rolled out about 8 years ago in `pkg-config`
0.28. This PR upgrades the pkg-config we use on CentOS-7 to the latest
as of this writing. After this, I think we should be able to upgrade
grpc again.

[bug]: https://bugs.freedesktop.org/show_bug.cgi?id=54716
[rollback]: googleapis#6840
devjgm added a commit to devjgm/google-cloud-cpp that referenced this pull request Jun 22, 2021
There's a [bug] w/ CentOS-7's version of `pkg-config` that can cause
invocations to take a reeeeeaaaaly long time to complete. We discovered
this when upgrading to grpc-1.38.1 and had to [rollback] the upgrade.

The fix to `pkg-config` was rolled out about 8 years ago in `pkg-config`
0.28. This PR upgrades the pkg-config we use on CentOS-7 to the latest
as of this writing. After this, I think we should be able to upgrade
grpc again.

[bug]: https://bugs.freedesktop.org/show_bug.cgi?id=54716
[rollback]: googleapis#6840
devjgm added a commit to devjgm/google-cloud-cpp that referenced this pull request Jun 23, 2021
There's a [bug] w/ CentOS-7's version of `pkg-config` that can cause
invocations to take a reeeeeaaaaly long time to complete. We discovered
this when upgrading to grpc-1.38.1 and had to [rollback] the upgrade.

The fix to `pkg-config` was rolled out about 8 years ago in `pkg-config`
0.28. This PR upgrades the pkg-config we use on CentOS-7 to the latest
as of this writing. After this, I think we should be able to upgrade
grpc again.

[bug]: https://bugs.freedesktop.org/show_bug.cgi?id=54716
[rollback]: googleapis#6840
devjgm added a commit that referenced this pull request Jun 23, 2021
There's a [bug] w/ CentOS-7's version of `pkg-config` that can cause
invocations to take a reeeeeaaaaly long time to complete. We discovered
this when upgrading to grpc-1.38.1 and had to [rollback] the upgrade.

The fix to `pkg-config` was rolled out about 8 years ago in `pkg-config`
0.28. This PR upgrades the pkg-config we use on CentOS-7 to the latest
as of this writing. After this, I think we should be able to upgrade
grpc again.

[bug]: https://bugs.freedesktop.org/show_bug.cgi?id=54716
[rollback]: #6840
@devjgm
Copy link
Copy Markdown
Contributor Author

devjgm commented Jun 23, 2021

Following up: The underlying issue was that centos-7's pkg-config is too old and suffers from a bad performance bug. I fixed this in #6846, so our centos-7 image now has a newer version of pkg-config. We should be able to accept this grpc upgrade now, but I'll wait to see if renovate sends another upgrade PR for it.

@coryan
Copy link
Copy Markdown
Contributor

coryan commented Jun 24, 2021

Renovate tries to minimize the "noise", and only sends one PR (ever) for each version. I think renovate would not send a new PR unless:

  • There is a newer release, such as 1.38.2 or 1.39.0
  • You rename the old PRs to upgrade to 1.38.1

I just renamed the old PRs, let's see what happens.

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

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants