Skip to content

Conversation

@wk8
Copy link
Contributor

@wk8 wk8 commented Oct 4, 2018

Trying to repro the build failure from #2757

Signed-off-by: rongzhang <[email protected]>
Signed-off-by: Jean Rouge <[email protected]>
@olljanat
Copy link
Contributor

olljanat commented Oct 4, 2018

@wk8 all builds fails currently.

We should get #2756 merged to fix it.

@wk8
Copy link
Contributor Author

wk8 commented Oct 4, 2018

@olljanat thanks, was actually doing this dummy PR to debug Circle, but hadn't had time to actually do it :)

Reviewed your own PR, approved with one minor comment: #2756 (review)

@olljanat
Copy link
Contributor

olljanat commented Oct 4, 2018

@wk8 yes I noticed.

#2756 is not actually mine but I did cherry-pick that commit to my #2758 to get build working.

@kolyshkin
Copy link
Contributor

CI builds were failing because of containerd/protobuild#20 which broke protobuild for protoc 3.5.0. The fix is either containerd/protobuild#21 (not yet merged) or update to protoc 3.6.1 (#2756, merged, so CI is back to normal).

Sorry for the mess 🐕

@wk8 wk8 closed this Oct 11, 2018
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.

4 participants