Skip to content

Conversation

saschagrunert
Copy link
Member

What type of PR is this?

/kind failing-test

What this PR does / why we need it:

This fixes the build of the release-note tool, which previously failed
with:

vendor/google.golang.org/api/internal/conn_pool.go:29:2: undefined: grpc.ClientConnInterface

Which issue(s) this PR fixes:

Refers to 15aa81b
Unblocks the release after #4188

Special notes for your reviewer:

@haircommander I think we have to re-align the v1.19.0 tag afterwards.

Does this PR introduce a user-facing change?

None

This fixes the build of the release-note tool, which previously failed
with:

```
vendor/google.golang.org/api/internal/conn_pool.go:29:2: undefined: grpc.ClientConnInterface
```

Signed-off-by: Sascha Grunert <sgrunert@suse.com>
@openshift-ci-robot openshift-ci-robot added release-note-none Denotes a PR that doesn't merit a release note. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. labels Sep 14, 2020
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: saschagrunert

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 14, 2020
@saschagrunert
Copy link
Member Author

/test e2e_crun

@codecov
Copy link

codecov bot commented Sep 14, 2020

Codecov Report

Merging #4190 into release-1.19 will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           release-1.19    #4190   +/-   ##
=============================================
  Coverage         41.75%   41.75%           
=============================================
  Files               110      110           
  Lines              9163     9163           
=============================================
  Hits               3826     3826           
  Misses             4994     4994           
  Partials            343      343           

@TomSweeneyRedHat
Copy link
Contributor

LGTM
and happy green test buttons!

@haircommander
Copy link
Member

/lgtm

thanks @saschagrunert

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 14, 2020
@openshift-merge-robot openshift-merge-robot merged commit 99c925b into cri-o:release-1.19 Sep 14, 2020
@saschagrunert saschagrunert deleted the release-1.19-grpc branch September 14, 2020 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants