Skip to content
This repository has been archived by the owner on Mar 9, 2022. It is now read-only.

Update CRI to d779e9c9561b732adf06263c5424889e7564fdbd. #85

Merged
merged 1 commit into from
Jun 21, 2017
Merged

Update CRI to d779e9c9561b732adf06263c5424889e7564fdbd. #85

merged 1 commit into from
Jun 21, 2017

Conversation

Random-Liu
Copy link
Member

Use newest CRI in Kubernetes 1.7.

Signed-off-by: Lantao Liu lantaol@google.com

@Random-Liu
Copy link
Member Author

CRI Validation test result:

Summarizing 5 Failures:

[Fail] [k8s.io] Security Context runtime should support container with security context [It] runtime should support RunAsUser [security context] 
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/container.go:326

[Fail] [k8s.io] Streaming runtime should support streaming interfaces [It] runtime should support attach [Conformance] 
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/streaming.go:183

[Fail] [k8s.io] Streaming runtime should support streaming interfaces [It] runtime should support exec [Conformance] 
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/streaming.go:127

[Fail] [k8s.io] Streaming runtime should support streaming interfaces [It] runtime should support portforward [Conformance] 
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/streaming.go:231

[Fail] [k8s.io] Security Context runtime should support container with security context [It] runtime should support RunAsUserName [security context] 
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/container.go:326

Ran 35 of 36 Specs in 31.574 seconds
FAIL! -- 30 Passed | 5 Failed | 0 Pending | 1 Skipped --- FAIL: TestE2ECRI (31.58s)
FAIL

Ginkgo ran 1 suite in 31.666567593s
Test Suite Failed
exit status 1

@Random-Liu Random-Liu added this to the v0.1.0-alpha.1 milestone Jun 20, 2017
@Random-Liu
Copy link
Member Author

Let's hold this PR, we need to update again after kubernetes/kubernetes#47818 is merged.

@Random-Liu Random-Liu changed the title Update CRI to 9cc2ecc347908faf0a79104f645241da24d97558. Update CRI to d779e9c9561b732adf06263c5424889e7564fdbd. Jun 21, 2017
@Random-Liu
Update CRI to d779e9c9561b732adf06263c5424889e7564fdbd.
862d00a 
Signed-off-by: Lantao Liu <lantaol@google.com>
@Random-Liu
Copy link
Member Author

CRI Validation test result:

Summarizing 5 Failures:

[Fail] [k8s.io] Security Context runtime should support container with security context [It] runtime should support RunAsUserName [security context] 
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/container.go:326

[Fail] [k8s.io] Security Context runtime should support container with security context [It] runtime should support RunAsUser [security context] 
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/container.go:326

[Fail] [k8s.io] Streaming runtime should support streaming interfaces [It] runtime should support exec [Conformance] 
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/streaming.go:127

[Fail] [k8s.io] Streaming runtime should support streaming interfaces [It] runtime should support attach [Conformance] 
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/streaming.go:183

[Fail] [k8s.io] Streaming runtime should support streaming interfaces [It] runtime should support portforward [Conformance] 
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/streaming.go:231

Ran 35 of 36 Specs in 28.586 seconds
FAIL! -- 30 Passed | 5 Failed | 0 Pending | 1 Skipped --- FAIL: TestE2ECRI (28.59s)
FAIL

Ginkgo ran 1 suite in 28.674270743s
Test Suite Failed
exit status 1

mikebrow
mikebrow approved these changes Jun 21, 2017
View reviewed changes
Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

LGTM

@mikebrow mikebrow added the lgtm label Jun 21, 2017
@Random-Liu Random-Liu merged commit 78b74a6 into containerd:master Jun 21, 2017
@Random-Liu Random-Liu deleted the update-cri branch June 21, 2017 20:18
@Random-Liu Random-Liu mentioned this pull request Jun 21, 2017
Closed
lanchongyizu pushed a commit to lanchongyizu/cri-containerd that referenced this pull request Sep 3, 2017
@Random-Liu
Merge pull request containerd#85 from Random-Liu/update-cri
163bb4f 
Update CRI to d779e9c9561b732adf06263c5424889e7564fdbd.
kevpar added a commit to kevpar/cri that referenced this pull request Dec 7, 2020
@kevpar
Merge pull request containerd#85 from dcantah/revendor-containerd-1.4.3
bfec7b9 
Update containerd vendoring to upstream 1.4.3
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mikebrow mikebrow mikebrow approved these changes

Assignees

@mikebrow mikebrow

Labels
cncf-cla: yes lgtm
Projects
None yet
Milestone
v0.1.0-alpha.1
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Random-Liu @mikebrow @k8s-ci-robot