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

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

Merged
merged 1 commit into from
Jun 22, 2021

Conversation

devjgm
Copy link
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
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
Collaborator

Google Cloud Build Logs
For commit: ca78f08321067ee61c2dc4b2faaed1a95ee7bf37

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

@codecov
Copy link

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 as a code owner 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
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
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