CONTRIBUTING.md

@@ -1,16 +0,0 @@
-# Contributing to SMARTER
-
-SMARTER is an open-source, software platform for the Edge. Contributions are an important part of the platform, and our goal is to make it as simple as possible to become a contributor.
-
-# Project license, and developer certificate of origin
-
-The SMARTER codebase is licensed under the Apache 2.0 open source license, individual components may have their own licenses. 
-Please see the LICENSE file in the SMARTER documentation repository for the full text of this license, or the LICENSE file
-in respective repositories.  New contributions are expected to be lisensed under the Apache 2.0 license or something compatible
-with that license.
-
-Please note that we expect contributors to the project to self-certify that they are authorized to contribute code 
-using the Linux Foundation's Developer Certificate of Origin. See http://developercertificate.org for more details.
-
-Contributors sign-off that they adhere to these requirements by adding a Signed-off-by line to commit messages.
-Pull requests for contributions without a signed-off-by line will not be accepted.

CONTRIBUTIONS

@@ -0,0 +1,4 @@
+We sincerely appreciate your interest of contributing to this project but for now
+we are not able to accept contributions. We expect to change this policy soon and
+provide a contribution procedure. We suggest that the project be maintained on a
+different branch until the contribution policy is changed.

Dockerfile

@@ -5,18 +5,19 @@ RUN apk update && apk upgrade && apk add tar ca-certificates build-base
 ENV GOPATH /go
 RUN go version
 
-WORKDIR /arm.com/smarter-device-management
+WORKDIR /go/src/smarter-device-management
 COPY . .
 
 RUN echo $PATH;export CGO_LDFLAGS_ALLOW='-Wl,--unresolved-symbols=ignore-in-object-files' && \
-    go mod init arm.com/smarter-device-management && go mod tidy && go mod vendor && \
+    go install -ldflags="-s -w" -v smarter-device-management
-    CGO_ENABLED=0 go build -ldflags='-s -w -extldflags="-static"' .
 
-FROM scratch
+FROM alpine
+
+RUN apk update && apk upgrade
 
 WORKDIR /root
 
 COPY conf.yaml /root/config/conf.yaml
-COPY --from=build /arm.com/smarter-device-management/smarter-device-management /usr/bin/smarter-device-management
+COPY --from=build /go/bin/smarter-device-management /usr/bin/smarter-device-management
 
-CMD ["/usr/bin/smarter-device-management","-logtostderr=true","-v=0"]
+CMD ["smarter-device-management","-logtostderr=true","-v=0"]

README.md

@@ -17,10 +17,6 @@ The smarter-device-manager starts by reading a YAML configuration file. This con
   nummaxdevices: 10
 ```
 
-Devices in subdirectories have the slash replaced with underscore in the
-resource name, due to kubernetes naming restrictions: e.g. `/dev/net/tun`
-becomes `smarter-devices/net_tun`.
-
 The default config file provided will enable most of the devices available on a Raspberry Pi (vers 1-4) or equivalent boards. I2C, SPI, video devices, sound and others would be enabled. The config file can be replaced using a configmap to enable or disable access to different devices, like accelerators, GPUs, etc.
 
 The node will show the devices it recognizes as resources in the node object in Kubernetes. The example below shows a raspberry PI.

argus.go

@@ -4,7 +4,7 @@ package main
 
 import (
 	"github.com/golang/glog"
-        pluginapi "k8s.io/kubelet/pkg/apis/deviceplugin/v1beta1"
+	pluginapi "k8s.io/kubernetes/pkg/kubelet/apis/deviceplugin/v1beta1"
 )
 
 func check(err error) {

compile.sh

@@ -5,12 +5,6 @@
 function printHelp() {
                 echo $(basename $0)" options:";
                 echo "    -A <Architectures to use> # Compiling to ${ARCHS} now, examples: linux/amd64,linux/arm/v7,linux/arm/v6,linux/arm64"
-                if [ ${FLAG_NOCACHE} -gt 0 ]
-                then
-                        echo "    -C # Do not use cache"
-                else 
-                        echo "    -C # Use cache" 
-                fi
                 if [ ${FLAG_UPLOADIMAGES} -gt 0 ]
                 then
                         echo "    -U # Do not upload images - the default is upload the images to the registry"
@@ -49,9 +43,8 @@ FLAG_UPLOADMANIFEST=1
 ADDITIONAL_TAG=""
 ADDITIONAL_IMAGE_NAME=""
 PUSH_OPTION=""
-FLAG_NOCACHE=0
 
-while getopts hA:B:MST:UC name
+while getopts hA:B:MST:U name
 do
         case $name in
         h)
@@ -59,10 +52,6 @@ do
                 exit 0;;
         A)
                 ARCHS="$OPTARG";;
-        C)
-                [ ${FLAG_NOCACHE} -gt 0 ] && FLAG_NOCACHE=0;
-                [ ${FLAG_NOCACHE} -eq 0 ] && FLAG_NOCACHE=1;
-                ;;
         U)
                 [ ${FLAG_UPLOADIMAGES} -gt 0 ] && FLAG_UPLOADIMAGES=0;
                 [ ${FLAG_UPLOADIMAGES} -eq 0 ] && FLAG_UPLOADIMAGES=1;
@@ -104,13 +93,6 @@ EOF
         fi
 fi
 
-if [ $FLAG_NOCACHE -gt 0 ]
-then
-        CACHE_OPTION="--no-cache"
-else
-        CACHE_OPTION="" 
-fi
-        
 if [ $FLAG_UPLOADIMAGES -gt 0 ]
 then
         PUSH_OPTION="--push"
@@ -118,6 +100,6 @@ else
         PUSH_OPTION="--load"
 fi
         
-docker buildx build ${CACHE_OPTION}  -t "${REPOSITORY_NAME}${IMAGE_NAME}${ADDITIONAL_IMAGE_NAME}:${BUILD_TAG}" --platform=${ARCHS} ${PUSH_OPTION} .
+docker buildx build  -t "${REPOSITORY_NAME}${IMAGE_NAME}${ADDITIONAL_IMAGE_NAME}:${BUILD_TAG}" --platform=${ARCHS} ${PUSH_OPTION} .
 
 exit 0

main.go

@@ -14,7 +14,7 @@ import (
 	"github.com/fsnotify/fsnotify"
 	"github.com/golang/glog"
 	"gopkg.in/yaml.v2"
-        pluginapi "k8s.io/kubelet/pkg/apis/deviceplugin/v1beta1"
+	pluginapi "k8s.io/kubernetes/pkg/kubelet/apis/deviceplugin/v1beta1"
 )
 
 var confFileName string
@@ -55,60 +55,18 @@ func init() {
 	flag.Parse()
 }
 
-func readDevDirectory(dirToList string, allowedRecursions uint8) (files []string, err error) {
+func readDevDirectory(dirToList string) (files []string, err error) {
-        var foundFiles []string
+	f, err := os.Open(dirToList)
-
+	if err != nil {
-        fType, err := os.Stat(dirToList)
+		return nil, err
-        if err != nil {
-                return nil, err
-        }
-
-        if !fType.IsDir()  {
-                return nil, nil
-        }
-
-        f, err := os.Open(dirToList)
-        if err != nil {
-                return nil, err
-        }
-        files, err = f.Readdirnames(-1)
-        if err != nil {
-                f.Close()
-                return nil, err
-        }
-        f.Close()
-        for _, subDir := range files {
-                foundFiles = append(foundFiles, subDir)
-                if allowedRecursions > 0 {
-                        filesDir, err := readDevDirectory(dirToList+"/"+subDir,allowedRecursions-1)
-                        if err == nil {
-                                for _, fileName := range filesDir {
-                                        foundFiles = append(foundFiles, subDir+"/"+fileName)
-                                }
-                        }
-                }
-        }
-
-        return foundFiles, nil
-}
-
-func sanitizeName(path string) string {
-        sanitizeChar := func(r rune) rune {
-		switch {
-		case r >= 'A' && r <= 'Z':
-			return r
-		case r >= 'a' && r <= 'z':
-			return r
-		case r >= '0' && r <= '9':
-			return r
-		case r == '_':
-			return r
-		case r == '-':
-			return r
-		}
-		return '_'
 	}
-        return strings.Map(sanitizeChar, path)
+	files, err = f.Readdirnames(-1)
+	f.Close()
+	if err != nil {
+		return nil, err
+	}
+
+	return files, nil
 }
 
 func findDevicesPattern(listDevices []string, pattern string) ([]string,error) {
@@ -144,13 +102,13 @@ func main() {
         }
 
 	glog.V(0).Info("Reading existing devices on /dev")
-	ExistingDevices, err := readDevDirectory("/dev",10)
+	ExistingDevices, err := readDevDirectory("/dev")
 	if err != nil {
 		glog.Errorf(err.Error())
 		os.Exit(1)
 	}
 
-	ExistingDevicesSys, err := readDevDirectory("/sys/devices",0)
+	ExistingDevicesSys, err := readDevDirectory("/sys/devices")
 	if err != nil {
 		glog.Errorf(err.Error())
 		os.Exit(1)
@@ -193,10 +151,9 @@ func main() {
                         if len(foundDevices) > 0 {
                                 for _, deviceToCreate := range foundDevices {
                                         var newDevice DeviceInstance
-                                        deviceSafeName := sanitizeName(deviceToCreate)
                                         newDevice.deviceType = deviceFileType
-                                        newDevice.deviceName = "smarter-devices/" + deviceSafeName
+                                        newDevice.deviceName = "smarter-devices/" + deviceToCreate
-                                        newDevice.socketName = pluginapi.DevicePluginPath + "smarter-" + deviceSafeName + ".sock"
+                                        newDevice.socketName = pluginapi.DevicePluginPath + "smarter-" + deviceToCreate + ".sock"
                                         newDevice.deviceFile = "/dev/" + deviceToCreate
                                         newDevice.numDevices = deviceToTest.NumMaxDevices
                                         listDevicesAvailable = append(listDevicesAvailable, newDevice)
@@ -236,17 +193,17 @@ L:
 			}
 
 			var err error
-			for id, _ := range listDevicesAvailable {
+			for _, devicesInUse := range listDevicesAvailable {
-                                switch listDevicesAvailable[id].deviceType {
+                                switch devicesInUse.deviceType {
                                 case deviceFileType :
-                                        listDevicesAvailable[id].devicePluginSmarter = NewSmarterDevicePlugin(listDevicesAvailable[id].numDevices, listDevicesAvailable[id].deviceFile, listDevicesAvailable[id].deviceName, listDevicesAvailable[id].socketName)
+                                        devicesInUse.devicePluginSmarter = NewSmarterDevicePlugin(devicesInUse.numDevices, devicesInUse.deviceFile, devicesInUse.deviceName, devicesInUse.socketName)
-                                        if err = listDevicesAvailable[id].devicePluginSmarter.Serve(); err != nil {
+                                        if err = devicesInUse.devicePluginSmarter.Serve(); err != nil {
                                                 glog.V(0).Info("Could not contact Kubelet, retrying. Did you enable the device plugin feature gate?")
                                                 break
                                         }
                                 case nvidiaSysType :
-                                        listDevicesAvailable[id].devicePluginNvidia = NewNvidiaDevicePlugin(listDevicesAvailable[id].numDevices, listDevicesAvailable[id].deviceName,"NVIDIA_VISIBLE_DEVICES", listDevicesAvailable[id].socketName, listDevicesAvailable[id].deviceId)
+                                        devicesInUse.devicePluginNvidia = NewNvidiaDevicePlugin(devicesInUse.numDevices, devicesInUse.deviceName,"NVIDIA_VISIBLE_DEVICES", devicesInUse.socketName, devicesInUse.deviceId)
-                                        if err = listDevicesAvailable[id].devicePluginNvidia.Serve(); err != nil {
+                                        if err = devicesInUse.devicePluginNvidia.Serve(); err != nil {
                                                 glog.V(0).Info("Could not contact Kubelet, retrying. Did you enable the device plugin feature gate?")
                                                 break
                                         }
@@ -277,18 +234,13 @@ L:
 			default:
 				glog.V(0).Infof("Received signal \"%v\", shutting down.", s)
 				for _, devicesInUse := range listDevicesAvailable {
-                                        glog.V(0).Info("Stopping device ", devicesInUse.deviceName)
                                         switch devicesInUse.deviceType {
                                         case deviceFileType :
-				                glog.V(0).Info("Smarter device type")
                                                 if devicesInUse.devicePluginSmarter != nil {
-				                        glog.V(0).Info("Stopping device")
                                                         devicesInUse.devicePluginSmarter.Stop()
                                                 }
                                         case nvidiaSysType :
-				                glog.V(0).Info("Nvidia device type")
                                                 if devicesInUse.devicePluginNvidia != nil {
-				                        glog.V(0).Info("Stopping device")
                                                         devicesInUse.devicePluginNvidia.Stop()
                                                 }
                                         }

nvidia-server.go

@@ -13,7 +13,7 @@ import (
 	"github.com/golang/glog"
 	"golang.org/x/net/context"
 	"google.golang.org/grpc"
-        pluginapi "k8s.io/kubelet/pkg/apis/deviceplugin/v1beta1"
+	pluginapi "k8s.io/kubernetes/pkg/kubelet/apis/deviceplugin/v1beta1"
 )
 
 var passDeviceSpecs = flag.Bool("pass-device-specs", false, "pass the list of DeviceSpecs to the kubelet on Allocate()")
@@ -182,10 +182,6 @@ func (m *NvidiaDevicePlugin) PreStartContainer(context.Context, *pluginapi.PreSt
 	return &pluginapi.PreStartContainerResponse{}, nil
 }
 
-func (m *NvidiaDevicePlugin) GetPreferredAllocation(context.Context, *pluginapi.PreferredAllocationRequest) (*pluginapi.PreferredAllocationResponse, error) {
-	return &pluginapi.PreferredAllocationResponse{}, nil
-}
-
 func (m *NvidiaDevicePlugin) cleanup() error {
 	if err := os.Remove(m.socket); err != nil && !os.IsNotExist(err) {
 		return err

server.go

@@ -13,7 +13,7 @@ import (
 	"github.com/golang/glog"
 	"golang.org/x/net/context"
 	"google.golang.org/grpc"
-        pluginapi "k8s.io/kubelet/pkg/apis/deviceplugin/v1beta1"
+	pluginapi "k8s.io/kubernetes/pkg/kubelet/apis/deviceplugin/v1beta1"
 )
 
 const (
@@ -94,7 +94,6 @@ func (m *SmarterDevicePlugin) Start() error {
 
 // Stop the gRPC server
 func (m *SmarterDevicePlugin) Stop() error {
-	glog.V(0).Infof("Stopping server with socket ",m.socket)
 	if m.server == nil {
 		return nil
 	}
@@ -102,7 +101,6 @@ func (m *SmarterDevicePlugin) Stop() error {
 	m.server.Stop()
 	m.server = nil
 	close(m.stop)
-	glog.V(0).Info("Server stopped with socket ",m.socket)
 
 	return m.cleanup()
 }
@@ -180,12 +178,7 @@ func (m *SmarterDevicePlugin) PreStartContainer(context.Context, *pluginapi.PreS
 	return &pluginapi.PreStartContainerResponse{}, nil
 }
 
-func (m *SmarterDevicePlugin) GetPreferredAllocation(context.Context, *pluginapi.PreferredAllocationRequest) (*pluginapi.PreferredAllocationResponse, error) {
-	return &pluginapi.PreferredAllocationResponse{}, nil
-}
-
 func (m *SmarterDevicePlugin) cleanup() error {
-	glog.V(0).Info("Removing file ",m.socket)
 	if err := os.Remove(m.socket); err != nil && !os.IsNotExist(err) {
 		return err
 	}

smarter-device-management-pod-k3s-test-xavier.yaml

@@ -15,7 +15,7 @@ spec:
   nodeName: smarter-jetson-xavier-4bcc2584
   containers:
   - name: smarter-device-manager
-    image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.20.11
+    image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.1.1
     imagePullPolicy: IfNotPresent
     securityContext:
       allowPrivilegeEscalation: false

smarter-device-management-pod-k3s.yaml

@@ -15,7 +15,7 @@ spec:
   nodeName: <replace with node to run>
   containers:
   - name: smarter-device-manager
-    image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.20.11
+    image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.1.1
     imagePullPolicy: IfNotPresent
     securityContext:
       allowPrivilegeEscalation: false

smarter-device-management-pod.yaml

@@ -15,7 +15,7 @@ spec:
   nodeName: <replace with node to run>
   containers:
   - name: smarter-device-manager
-    image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.20.11
+    image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.1.1
     imagePullPolicy: IfNotPresent
     securityContext:
       allowPrivilegeEscalation: false
@@ -33,8 +33,6 @@ spec:
         mountPath: /var/lib/kubelet/device-plugins
       - name: dev-dir
         mountPath: /dev
-      - name: sys-dir
-        mountPath: /sys
   volumes:
     - name: device-plugin
       hostPath:
@@ -42,7 +40,4 @@ spec:
     - name: dev-dir
       hostPath:
             path: /dev
-    - name: sys-dir
+      terminationGracePeriodSeconds: 30
-      hostPath:
-            path: /sys
-  terminationGracePeriodSeconds: 30

smarter-device-manager-ds-k3s.yaml

@@ -34,7 +34,7 @@ spec:
       dnsPolicy: ClusterFirstWithHostNet
       containers:
       - name: smarter-device-manager
-        image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.20.11
+        image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.1.1
         imagePullPolicy: IfNotPresent
         securityContext:
           allowPrivilegeEscalation: false

smarter-device-manager-ds-with-configmap-rpi-k3s.yaml

@@ -34,7 +34,7 @@ spec:
       dnsPolicy: ClusterFirstWithHostNet
       containers:
       - name: smarter-device-manager
-        image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.20.11
+        image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.1.1
         imagePullPolicy: IfNotPresent
         securityContext:
           allowPrivilegeEscalation: false

smarter-device-manager-ds-with-configmap-rpi.yaml

@@ -34,7 +34,7 @@ spec:
       dnsPolicy: ClusterFirstWithHostNet
       containers:
       - name: smarter-device-manager
-        image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.20.11
+        image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.1.1
         imagePullPolicy: IfNotPresent
         securityContext:
           allowPrivilegeEscalation: false

smarter-device-manager-ds-with-configmap-xavier-k3s.yaml

@@ -34,7 +34,7 @@ spec:
       dnsPolicy: ClusterFirstWithHostNet
       containers:
       - name: smarter-device-manager
-        image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.20.11
+        image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.1.1
         imagePullPolicy: IfNotPresent
         securityContext:
           allowPrivilegeEscalation: false

smarter-device-manager-ds-with-configmap-xavier.yaml

@@ -34,7 +34,7 @@ spec:
       dnsPolicy: ClusterFirstWithHostNet
       containers:
       - name: smarter-device-manager
-        image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.20.11
+        image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.1.1
         imagePullPolicy: IfNotPresent
         securityContext:
           allowPrivilegeEscalation: false

smarter-device-manager-ds.yaml

@@ -34,7 +34,7 @@ spec:
       dnsPolicy: ClusterFirstWithHostNet
       containers:
       - name: smarter-device-manager
-        image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.20.11
+        image: registry.gitlab.com/arm-research/smarter/smarter-device-manager:v1.1.1
         imagePullPolicy: IfNotPresent
         securityContext:
           allowPrivilegeEscalation: false