github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-08-04 09:49:50 +00:00
Code Issues Projects Releases Wiki Activity

Set fake docker client to minimum required version.

Browse Source
This commit is contained in:
Random-Liu 2016-04-04 16:01:11 -07:00
parent ce988c8d75
commit 8fb7ee38b0
3 changed files with 17 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
pkg/kubelet/dockertools/docker.go
View File

@ -292,9 +292,7 @@ func getDockerClient(dockerEndpoint string) (*dockerapi.Client, error) {
// will be returned. The program exits if error occurs. // will be returned. The program exits if error occurs.
func ConnectToDockerOrDie(dockerEndpoint string) DockerInterface { func ConnectToDockerOrDie(dockerEndpoint string) DockerInterface {
if dockerEndpoint == "fake://" { if dockerEndpoint == "fake://" {
return &FakeDockerClient{ return NewFakeDockerClient()
VersionInfo: docker.Env{"ApiVersion=1.18", "Version=1.6.0"},
}
} }
client, err := getDockerClient(dockerEndpoint) client, err := getDockerClient(dockerEndpoint)
if err != nil { if err != nil {

23
pkg/kubelet/dockertools/docker_test.go
View File

@ -81,7 +81,7 @@ func findPodContainer(dockerContainers []*docker.APIContainers, podFullName stri
} }
func TestGetContainerID(t *testing.T) { func TestGetContainerID(t *testing.T) {
fakeDocker := &FakeDockerClient{} fakeDocker := NewFakeDockerClient()
fakeDocker.SetFakeRunningContainers([]*docker.Container{ fakeDocker.SetFakeRunningContainers([]*docker.Container{
{ {
ID: "foobar", ID: "foobar",
@ -156,7 +156,7 @@ func TestContainerNaming(t *testing.T) {
} }
func TestVersion(t *testing.T) { func TestVersion(t *testing.T) {
fakeDocker := &FakeDockerClient{VersionInfo: docker.Env{"Version=1.1.3", "ApiVersion=1.15"}} fakeDocker := NewFakeDockerClientWithVersion("1.1.3", "1.15")
manager := &DockerManager{client: fakeDocker} manager := &DockerManager{client: fakeDocker}
version, err := manager.Version() version, err := manager.Version()
if err != nil { if err != nil {
@ -214,7 +214,7 @@ func TestPullWithNoSecrets(t *testing.T) {
} }
for _, test := range tests { for _, test := range tests {
fakeKeyring := &credentialprovider.FakeKeyring{} fakeKeyring := &credentialprovider.FakeKeyring{}
fakeClient := &FakeDockerClient{} fakeClient := NewFakeDockerClient()
dp := dockerPuller{ dp := dockerPuller{
client: fakeClient, client: fakeClient,
@ -257,9 +257,9 @@ func TestPullWithJSONError(t *testing.T) {
} }
for i, test := range tests { for i, test := range tests {
fakeKeyring := &credentialprovider.FakeKeyring{} fakeKeyring := &credentialprovider.FakeKeyring{}
fakeClient := &FakeDockerClient{ fakeClient := NewFakeDockerClient()
Errors: map[string]error{"pull": test.err}, fakeClient.InjectError("pull", test.err)
}
puller := &dockerPuller{ puller := &dockerPuller{
client: fakeClient, client: fakeClient,
keyring: fakeKeyring, keyring: fakeKeyring,
@ -327,7 +327,7 @@ func TestPullWithSecrets(t *testing.T) {
builtInKeyRing := &credentialprovider.BasicDockerKeyring{} builtInKeyRing := &credentialprovider.BasicDockerKeyring{}
builtInKeyRing.Add(test.builtInDockerConfig) builtInKeyRing.Add(test.builtInDockerConfig)
fakeClient := &FakeDockerClient{} fakeClient := NewFakeDockerClient()
dp := dockerPuller{ dp := dockerPuller{
client: fakeClient, client: fakeClient,
@ -353,9 +353,8 @@ func TestPullWithSecrets(t *testing.T) {
func TestDockerKeyringLookupFails(t *testing.T) { func TestDockerKeyringLookupFails(t *testing.T) {
fakeKeyring := &credentialprovider.FakeKeyring{} fakeKeyring := &credentialprovider.FakeKeyring{}
fakeClient := &FakeDockerClient{ fakeClient := NewFakeDockerClient()
Errors: map[string]error{"pull": fmt.Errorf("test error")}, fakeClient.InjectError("pull", fmt.Errorf("test error"))
}
dp := dockerPuller{ dp := dockerPuller{
client: fakeClient, client: fakeClient,
@ -493,7 +492,7 @@ func (f *imageTrackingDockerClient) InspectImage(name string) (image *docker.Ima
} }
func TestIsImagePresent(t *testing.T) { func TestIsImagePresent(t *testing.T) {
cl := &imageTrackingDockerClient{&FakeDockerClient{}, ""} cl := &imageTrackingDockerClient{NewFakeDockerClient(), ""}
puller := &dockerPuller{ puller := &dockerPuller{
client: cl, client: cl,
} }
@ -671,7 +670,7 @@ func TestFindContainersByPod(t *testing.T) {
nil, nil,
}, },
} }
fakeClient := &FakeDockerClient{} fakeClient := NewFakeDockerClient()
np, _ := network.InitNetworkPlugin([]network.NetworkPlugin{}, "", nettest.NewFakeHost(nil)) np, _ := network.InitNetworkPlugin([]network.NetworkPlugin{}, "", nettest.NewFakeHost(nil))
// image back-off is set to nil, this test should not pull images // image back-off is set to nil, this test should not pull images
containerManager := NewFakeDockerManager(fakeClient, &record.FakeRecorder{}, nil, nil, &cadvisorapi.MachineInfo{}, options.GetDefaultPodInfraContainerImage(), 0, 0, "", containertest.FakeOS{}, np, nil, nil, nil) containerManager := NewFakeDockerManager(fakeClient, &record.FakeRecorder{}, nil, nil, &cadvisorapi.MachineInfo{}, options.GetDefaultPodInfraContainerImage(), 0, 0, "", containertest.FakeOS{}, np, nil, nil, nil)

6
pkg/kubelet/dockertools/fake_docker_client.go
View File

@ -55,8 +55,12 @@ type FakeDockerClient struct {
EnableSleep bool EnableSleep bool
} }
// We don't check docker version now, just set the docker version of fake docker client to 1.8.1.
// Notice that if someday we also have minimum docker version requirement, this should also be updated.
const fakeDockerVersion = "1.8.1"
func NewFakeDockerClient() *FakeDockerClient { func NewFakeDockerClient() *FakeDockerClient {
return NewFakeDockerClientWithVersion("1.8.1", "1.20") return NewFakeDockerClientWithVersion(fakeDockerVersion, minimumDockerAPIVersion)
} }
func NewFakeDockerClientWithVersion(version, apiVersion string) *FakeDockerClient { func NewFakeDockerClientWithVersion(version, apiVersion string) *FakeDockerClient {