Skip to content

Add Makefile and update protobuf#106

Merged
estesp merged 3 commits intocontainerd:mainfrom
dmcgowan:update-protobuf-process
Mar 7, 2022
Merged

Add Makefile and update protobuf#106
estesp merged 3 commits intocontainerd:mainfrom
dmcgowan:update-protobuf-process

Conversation

@dmcgowan
Copy link
Copy Markdown
Member

@dmcgowan dmcgowan commented Mar 4, 2022

Updates protobuf version to v3.11.4 to match with containerd.
Adds a Makefile for the workflow and updates ci to use Makefile.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 4, 2022

Codecov Report

Merging #106 (f8e8cc4) into main (aa5c947) will decrease coverage by 0.19%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
- Coverage   63.87%   63.67%   -0.20%     
==========================================
  Files          11       11              
  Lines        1049     1049              
==========================================
- Hits          670      668       -2     
- Misses        334      336       +2     
  Partials       45       45              
Impacted Files Coverage Δ
src/github.com/containerd/ttrpc/server.go 76.01% <0.00%> (-0.63%) ⬇️

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 aa5c947...f8e8cc4. Read the comment docs.

@dmcgowan dmcgowan force-pushed the update-protobuf-process branch 3 times, most recently from a4f5d8d to 7cf1e14 Compare March 4, 2022 03:09
@dmcgowan dmcgowan marked this pull request as ready for review March 4, 2022 04:05
@dmcgowan
Copy link
Copy Markdown
Member Author

dmcgowan commented Mar 4, 2022

Marked this as ready for review. There was originally a failure on Windows showing a closed that was not caught. It stopped showing up but I added more logging so we can see the type of error to check in the future.

dmcgowan added 3 commits March 4, 2022 18:17
Use makefile and installation script for protoc.
Update protoc version to match containerd.

Signed-off-by: Derek McGowan <derek@mcg.dev>
Signed-off-by: Derek McGowan <derek@mcg.dev>
Signed-off-by: Derek McGowan <derek@mcg.dev>
@dmcgowan dmcgowan force-pushed the update-protobuf-process branch from 7cf1e14 to f8e8cc4 Compare March 5, 2022 02:17
Copy link
Copy Markdown
Member

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp merged commit b194a10 into containerd:main Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants