From 5b1d875f27876a9d410ff218d9d6cff2a1a7e073 Mon Sep 17 00:00:00 2001 From: Daniel Smith Date: Thu, 1 Dec 2016 11:47:24 -0800 Subject: [PATCH] Revert "Modify GCI mounter to enable NFSv3" --- cluster/gce/gci/configure.sh | 4 +-- cluster/gce/gci/mounter/Changelog | 4 --- cluster/gce/gci/mounter/Dockerfile | 4 +-- cluster/gce/gci/mounter/Makefile | 2 +- cluster/gce/gci/mounter/mounter | 20 +++++---------- cluster/gce/gci/mounter/startmounter.sh | 26 ------------------- test/e2e/common/volumes.go | 34 ------------------------- test/e2e_node/jenkins/gci-init.yaml | 2 +- 8 files changed, 12 insertions(+), 84 deletions(-) delete mode 100644 cluster/gce/gci/mounter/startmounter.sh diff --git a/cluster/gce/gci/configure.sh b/cluster/gce/gci/configure.sh index 1435972d6eef..bc903ab571d2 100644 --- a/cluster/gce/gci/configure.sh +++ b/cluster/gce/gci/configure.sh @@ -100,10 +100,10 @@ function split-commas { function install-gci-mounter-tools { local -r rkt_version="v1.18.0" - local -r gci_mounter_version="v3" + local -r gci_mounter_version="v2" local -r rkt_binary_sha1="75fc8f29c79bc9e505f3e7f6e8fadf2425c21967" local -r rkt_stage1_fly_sha1="474df5a1f934960ba669b360ab713d0a54283091" - local -r gci_mounter_sha1="6ae71998bf9deba06468b036c2ba126626e0050d" + local -r gci_mounter_sha1="851e841d8640d6a05e64e22c493f5ac3c4cba561" download-or-bust "${rkt_binary_sha1}" "https://storage.googleapis.com/kubernetes-release/rkt/${rkt_version}/rkt" download-or-bust "${rkt_stage1_fly_sha1}" "https://storage.googleapis.com/kubernetes-release/rkt/${rkt_version}/stage1-fly.aci" download-or-bust "${gci_mounter_sha1}" "https://storage.googleapis.com/kubernetes-release/gci-mounter/gci-mounter-${gci_mounter_version}.aci" diff --git a/cluster/gce/gci/mounter/Changelog b/cluster/gce/gci/mounter/Changelog index 07d14b96483c..ced9cdd097da 100644 --- a/cluster/gce/gci/mounter/Changelog +++ b/cluster/gce/gci/mounter/Changelog @@ -5,7 +5,3 @@ ## v2 (Fri Oct 28 2016 Vishnu Kannan ) - Adding netbase package. - Digest: sha256:c7dfe059fbbf976fc4284a87eb18adf0f8e0c4cf30a30f5a852842c772a64c2d - -## v3 (Fri Nov 11 2016 Jing Xu ) - - Add startmounter.sh script to start rpcbind before mounting for NFSv3 - - Digest: sha256:35bc48a1ca97c3971611dc4662d08d131869daa692acb281c7e9e052924e38b1 diff --git a/cluster/gce/gci/mounter/Dockerfile b/cluster/gce/gci/mounter/Dockerfile index 57d60350f34a..bfdc3f6b7da4 100644 --- a/cluster/gce/gci/mounter/Dockerfile +++ b/cluster/gce/gci/mounter/Dockerfile @@ -16,5 +16,5 @@ FROM ubuntu:xenial MAINTAINER vishh@google.com RUN apt-get update && apt-get install -y netbase nfs-common=1:1.2.8-9ubuntu12 glusterfs-client=3.7.6-1ubuntu1 -ADD startmounter.sh /startmounter.sh -ENTRYPOINT ["/startmounter.sh"] + +ENTRYPOINT ["/bin/mount"] diff --git a/cluster/gce/gci/mounter/Makefile b/cluster/gce/gci/mounter/Makefile index af717c757573..72efa3b77c4f 100644 --- a/cluster/gce/gci/mounter/Makefile +++ b/cluster/gce/gci/mounter/Makefile @@ -12,7 +12,7 @@ # See the License for the specific language governing permissions and # limitations under the License. -TAG=v3 +TAG=v2 REGISTRY=gcr.io/google_containers IMAGE=gci-mounter diff --git a/cluster/gce/gci/mounter/mounter b/cluster/gce/gci/mounter/mounter index 7761325030c5..6f45a56abf43 100755 --- a/cluster/gce/gci/mounter/mounter +++ b/cluster/gce/gci/mounter/mounter @@ -18,7 +18,7 @@ set -o errexit set -o nounset set -o pipefail -MOUNTER_VERSION=v3 +MOUNTER_VERSION=v2 MOUNTER_USER=root ROOT_DIR=/home/kubernetes/bin RKT_BINARY=${ROOT_DIR}/rkt @@ -46,18 +46,10 @@ fi echo "Running mount using a rkt fly container" -if [ "$#" -eq 0 ]; then - ${RKT_BINARY} run --stage1-path=${STAGE1_ACI} \ - --insecure-options=image \ - --volume=kubelet,kind=host,source=/var/lib/kubelet,readOnly=false,recursive=true \ - --mount volume=kubelet,target=/var/lib/kubelet \ - ${MOUNTER_IMAGE} --user=${MOUNTER_USER} --exec /bin/mount -else - ${RKT_BINARY} run --stage1-path=${STAGE1_ACI} \ - --insecure-options=image \ - --volume=kubelet,kind=host,source=/var/lib/kubelet,readOnly=false,recursive=true \ - --mount volume=kubelet,target=/var/lib/kubelet \ - ${MOUNTER_IMAGE} --user=${MOUNTER_USER} --exec /startmounter.sh -- "$@" -fi +${RKT_BINARY} run --stage1-path=${STAGE1_ACI} \ + --insecure-options=image \ + --volume=kubelet,kind=host,source=/var/lib/kubelet,readOnly=false,recursive=true \ + --mount volume=kubelet,target=/var/lib/kubelet \ + ${MOUNTER_IMAGE} --user=${MOUNTER_USER} --exec /bin/mount -- "$@" echo "Successfully ran mount using a rkt fly container" diff --git a/cluster/gce/gci/mounter/startmounter.sh b/cluster/gce/gci/mounter/startmounter.sh deleted file mode 100644 index 511ac7d17385..000000000000 --- a/cluster/gce/gci/mounter/startmounter.sh +++ /dev/null @@ -1,26 +0,0 @@ -#!/bin/bash - -# Copyright 2016 The Kubernetes Authors. -# -# Licensed under the Apache License, Version 2.0 (the "License"); -# you may not use this file except in compliance with the License. -# You may obtain a copy of the License at -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, software -# distributed under the License is distributed on an "AS IS" BASIS, -# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -# See the License for the specific language governing permissions and -# limitations under the License. - -# start rpcbind if it is not started yet -s=`/etc/init.d/rpcbind status` -if [[ $s == *"not running"* ]]; then - echo "Starting rpcbind: /sbin/rpcbind -w" - /sbin/rpcbind -w -fi -echo `/etc/init.d/rpcbind status` - -# mount with passing paramaters -/bin/mount "${@}" diff --git a/test/e2e/common/volumes.go b/test/e2e/common/volumes.go index 1abd31563873..532d363a587f 100644 --- a/test/e2e/common/volumes.go +++ b/test/e2e/common/volumes.go @@ -393,40 +393,6 @@ var _ = framework.KubeDescribe("GCP Volumes", func() { }) }) - //////////////////////////////////////////////////////////////////////// - // NFSv3 - //////////////////////////////////////////////////////////////////////// - - framework.KubeDescribe("NFSv3", func() { - It("should be mountable for NFSv3", func() { - config := VolumeTestConfig{ - namespace: namespace.Name, - prefix: "nfs", - serverImage: "gcr.io/google_containers/volume-nfs:0.8", - serverPorts: []int{2049, 20048}, - } - - defer func() { - if clean { - volumeTestCleanup(f, config) - } - }() - pod := startVolumeServer(f, config) - serverIP := pod.Status.PodIP - framework.Logf("NFS server IP address: %v", serverIP) - - volume := v1.VolumeSource{ - NFS: &v1.NFSVolumeSource{ - Server: serverIP, - Path: "/exports", - ReadOnly: true, - }, - } - // Must match content of test/images/volumes-tester/nfs/index.html - testVolumeClient(f, config, volume, nil, "Hello from NFS!") - }) - }) - //////////////////////////////////////////////////////////////////////// // Gluster //////////////////////////////////////////////////////////////////////// diff --git a/test/e2e_node/jenkins/gci-init.yaml b/test/e2e_node/jenkins/gci-init.yaml index 94b82d503a7b..8b3cb6c7bc95 100644 --- a/test/e2e_node/jenkins/gci-init.yaml +++ b/test/e2e_node/jenkins/gci-init.yaml @@ -8,5 +8,5 @@ runcmd: - mount -B -o remount,exec /home/kubernetes/bin - wget https://storage.googleapis.com/kubernetes-release/rkt/v1.18.0/rkt -O /home/kubernetes/bin/rkt - wget https://storage.googleapis.com/kubernetes-release/rkt/v1.18.0/stage1-fly.aci -O /home/kubernetes/bin/stage1-fly.aci - - wget https://storage.googleapis.com/kubernetes-release/gci-mounter/gci-mounter-v3.aci -O /home/kubernetes/bin/gci-mounter-v3.aci + - wget https://storage.googleapis.com/kubernetes-release/gci-mounter/gci-mounter-v2.aci -O /home/kubernetes/bin/gci-mounter-v2.aci - chmod a+x /home/kubernetes/bin/rkt