Used new xds agent v1.0.0-rc1 and lib xaapiv1.
authorSebastien Douheret <sebastien.douheret@iot.bzh>
Thu, 30 Nov 2017 00:40:47 +0000 (01:40 +0100)
committerSebastien Douheret <sebastien.douheret@iot.bzh>
Thu, 30 Nov 2017 15:33:16 +0000 (16:33 +0100)
Signed-off-by: Sebastien Douheret <sebastien.douheret@iot.bzh>
.vscode/settings.json
Makefile
gdb-xds.go
glide.yaml
main.go

index c9bea18..496fc84 100644 (file)
@@ -1,22 +1,19 @@
 // Place your settings in this file to overwrite default and user settings.
 {
-  // Configure glob patterns for excluding files and folders.
-  "files.exclude": {
-    "**/.tmp": true,
-    ".git": true,
-    "glide.lock": true,
-    "vendor/": true,
-    "debug": true,
-    "bin": true,
-    "tools": true
-  },
-  // Words to add to dictionary for a workspace.
-  "cSpell.words": [
-    "apiv",
-    "iosk",
-    "zhouhui",
-    "ldflags",
-    "socketio",
-    "xdsconfig"
-  ]
+    // Configure glob patterns for excluding files and folders.
+    "files.exclude": {
+        "**/.tmp": true,
+        ".git": true,
+        "glide.lock": true,
+        "vendor/": true,
+        "debug": true,
+        "bin": true,
+        "tools": true
+    },
+    // Words to add to dictionary for a workspace.
+    "cSpell.words": [
+        "apiv", "iosk", "zhouhui", "ldflags", "socketio", "xdsconfig", "golib",
+        "sebd", "ccmd", "aargs", "eenv", "gdbserver", "NOFIX", "XDSAGENT",
+        "xaapiv"
+    ]
 }
index f256c1b..a951275 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -1,22 +1,22 @@
 # Makefile used to build xds-gdb commands
 
-# Application Version
-VERSION := 0.1.0
+# Application Name
 TARGET=xds-gdb
 
-# Retrieve git tag/commit to set sub-version string
-ifeq ($(origin SUB_VERSION), undefined)
-       SUB_VERSION := $(shell git describe --exact-match --tags 2>/dev/null | sed 's/^v//')
-       ifneq ($(SUB_VERSION), )
-               VERSION := $(firstword $(subst -, ,$(SUB_VERSION)))
-               SUB_VERSION := $(word 2,$(subst -, ,$(SUB_VERSION)))
-       endif
-       ifeq ($(SUB_VERSION), )
-               SUB_VERSION := $(shell git rev-parse --short HEAD)
-               ifeq ($(SUB_VERSION), )
-                       SUB_VERSION := unknown-dev
-               endif
-       endif
+# Retrieve git tag/commit to set version & sub-version strings
+GIT_DESC := $(shell git describe --always --tags)
+VERSION := $(firstword $(subst -, ,$(GIT_DESC)))
+SUB_VERSION := $(subst $(VERSION)-,,$(GIT_DESC))
+ifeq ($(VERSION), )
+       VERSION := unknown-dev
+endif
+ifeq ($(SUB_VERSION), )
+       SUB_VERSION := $(shell date +'%Y-%m-%d_%H%M%S')
+endif
+
+# Configurable variables for installation (default /opt/AGL/...)
+ifeq ($(origin DESTDIR), undefined)
+       DESTDIR := /opt/AGL/xds/gdb
 endif
 
 HOST_GOOS=$(shell go env GOOS)
@@ -29,15 +29,15 @@ ifeq ($(HOST_GOOS), windows)
        EXT=.exe
 endif
 
-
 mkfile_path := $(abspath $(lastword $(MAKEFILE_LIST)))
 ROOT_SRCDIR := $(patsubst %/,%,$(dir $(mkfile_path)))
-BINDIR := $(ROOT_SRCDIR)/bin
 ROOT_GOPRJ := $(abspath $(ROOT_SRCDIR)/../../../..)
+LOCAL_BINDIR := $(ROOT_SRCDIR)/bin
+LOCAL_TOOLSDIR := $(ROOT_SRCDIR)/tools/${HOST_GOOS}
 PACKAGE_DIR := $(ROOT_SRCDIR)/package
 
 export GOPATH := $(shell go env GOPATH):$(ROOT_GOPRJ)
-export PATH := $(PATH):$(ROOT_SRCDIR)/tools
+export PATH := $(PATH):$(LOCAL_TOOLSDIR)
 
 VERBOSE_1 := -v
 VERBOSE_2 := -v -x
@@ -57,9 +57,9 @@ endif
 
 
 ifeq ($(SUB_VERSION), )
-       PACKAGE_ZIPFILE := $(TARGET)_$(ARCH)-v$(VERSION).zip
+       PACKAGE_ZIPFILE := $(TARGET)_$(ARCH)-$(VERSION).zip
 else
-       PACKAGE_ZIPFILE := $(TARGET)_$(ARCH)-v$(VERSION)_$(SUB_VERSION).zip
+       PACKAGE_ZIPFILE := $(TARGET)_$(ARCH)-$(VERSION)_$(SUB_VERSION).zip
 endif
 
 .PHONY: all
@@ -67,24 +67,25 @@ all: vendor build
 
 .PHONY: build
 build:
-       @echo "### Build $(TARGET) (version $(VERSION), subversion $(SUB_VERSION)) - $(BUILD_MODE)";
-       @cd $(ROOT_SRCDIR); $(BUILD_ENV_FLAGS) go build $(VERBOSE_$(V)) -i -o $(BINDIR)/$(TARGET)$(EXT) -ldflags "$(GO_LDFLAGS) -X main.AppVersion=$(VERSION) -X main.AppSubVersion=$(SUB_VERSION)" -gcflags "$(GO_GCFLAGS)" .
+       @echo "### Build $(TARGET) (version $(VERSION), subversion $(SUB_VERSION) - $(BUILD_MODE))";
+       @cd $(ROOT_SRCDIR); $(BUILD_ENV_FLAGS) go build $(VERBOSE_$(V)) -i -o $(LOCAL_BINDIR)/$(TARGET)$(EXT) -ldflags "$(GO_LDFLAGS) -X main.AppVersion=$(VERSION) -X main.AppSubVersion=$(SUB_VERSION)" -gcflags "$(GO_GCFLAGS)" .
 
 test: tools/glide
-       go test --race $(shell ./tools/glide novendor)
+       go test --race $(shell $(LOCAL_TOOLSDIR)/glide novendor)
 
 vet: tools/glide
-       go vet $(shell ./tools/glide novendor)
+       go vet $(shell $(LOCAL_TOOLSDIR)/glide novendor)
 
 fmt: tools/glide
-       go fmt $(shell ./tools/glide novendor)
+       go fmt $(shell $(LOCAL_TOOLSDIR)/glide novendor)
 
 .PHONY: clean
 clean:
-       rm -rf $(BINDIR)/* debug $(ROOT_GOPRJ)/pkg/*/$(REPOPATH) $(PACKAGE_DIR)
+       rm -rf $(LOCAL_BINDIR)/* debug $(ROOT_GOPRJ)/pkg/*/$(REPOPATH) $(PACKAGE_DIR)
 
+.PHONY: distclean
 distclean: clean
-       rm -rf $(BINDIR) tools glide.lock vendor $(ROOT_SRCDIR)/*.zip
+       rm -rf $(LOCAL_BINDIR) $(ROOT_SRCDIR)/tools glide.lock vendor $(ROOT_SRCDIR)/*.zip
 
 .PHONY: release
 release:
@@ -92,7 +93,7 @@ release:
 
 package: clean vendor build
        @mkdir -p $(PACKAGE_DIR)/$(TARGET)
-       @cp -a $(BINDIR)/*gdb$(EXT) $(PACKAGE_DIR)/$(TARGET)
+       @cp -a $(LOCAL_BINDIR)/*gdb$(EXT) $(PACKAGE_DIR)/$(TARGET)
        @cp -r $(ROOT_SRCDIR)/conf.d $(ROOT_SRCDIR)/scripts $(PACKAGE_DIR)/$(TARGET)
        cd $(PACKAGE_DIR) && zip -r $(ROOT_SRCDIR)/$(PACKAGE_ZIPFILE) ./$(TARGET)
 
@@ -116,17 +117,20 @@ uninstall:
        export DESTDIR=$(DESTDIR) && $(ROOT_SRCDIR)/scripts/install.sh uninstall
 
 vendor: tools/glide glide.yaml
-       ./tools/glide install --strip-vendor
+       $(LOCAL_TOOLSDIR)/glide install --strip-vendor
 
 vendor/debug: vendor
        (cd vendor/github.com/iotbzh && \
                rm -rf xds-common && ln -s ../../../../xds-common && \
-               rm -rf xds-server && ln -s ../../../../xds-server )
+               rm -rf xds-agent && ln -s ../../../../xds-agent )
 
+.PHONY: tools/glide
 tools/glide:
-       @echo "Downloading glide"
-       mkdir -p tools
-       curl --silent -L https://glide.sh/get | GOBIN=./tools  sh
+       @test -f $(LOCAL_TOOLSDIR)/glide || { \
+               echo "Downloading glide"; \
+               mkdir -p $(LOCAL_TOOLSDIR); \
+               curl --silent -L https://glide.sh/get | GOBIN=$(LOCAL_TOOLSDIR)  sh; \
+       }
 
 help:
        @echo "Main supported rules:"
index 22e13ed..981b977 100644 (file)
@@ -3,17 +3,16 @@ package main
 import (
        "encoding/json"
        "fmt"
-       "net/http"
        "os"
+       "regexp"
+       "strconv"
        "strings"
        "syscall"
 
        "github.com/Sirupsen/logrus"
+       "github.com/iotbzh/xds-agent/lib/xaapiv1"
        common "github.com/iotbzh/xds-common/golib"
-       "github.com/iotbzh/xds-server/lib/apiv1"
-       "github.com/iotbzh/xds-server/lib/crosssdk"
-       "github.com/iotbzh/xds-server/lib/folder"
-       sio_client "github.com/zhouhui8915/go-socket.io-client"
+       sio_client "github.com/sebd71/go-socket.io-client"
 )
 
 // GdbXds -
@@ -28,11 +27,12 @@ type GdbXds struct {
        rPath   string
        listPrj bool
        cmdID   string
+       xGdbPid string
 
        httpCli *common.HTTPClient
        ioSock  *sio_client.Client
 
-       folders []folder.FolderConfig
+       projects []xaapiv1.ProjectConfig
 
        // callbacks
        cbOnError      func(error)
@@ -51,6 +51,7 @@ func NewGdbXds(log *logrus.Logger, args []string, env []string) *GdbXds {
                eenv:    env,
                httpCli: nil,
                ioSock:  nil,
+               xGdbPid: strconv.Itoa(os.Getpid()),
        }
 }
 
@@ -89,25 +90,45 @@ func (g *GdbXds) Init() (int, error) {
        g.log.Infoln("Connect HTTP client on ", baseURL)
        conf := common.HTTPClientConfig{
                URLPrefix:           "/api/v1",
-               HeaderClientKeyName: "XDS-SID",
+               HeaderClientKeyName: "Xds-Agent-Sid",
                CsrfDisable:         true,
+               LogOut:              g.log.Out,
+               LogLevel:            common.HTTPLogLevelWarning,
        }
        c, err := common.HTTPNewClient(baseURL, conf)
        if err != nil {
-               return int(syscallEBADE), err
+               errmsg := err.Error()
+               if m, err := regexp.MatchString("Get http.?://", errmsg); m && err == nil {
+                       i := strings.LastIndex(errmsg, ":")
+                       errmsg = "Cannot connection to " + baseURL + errmsg[i:]
+               }
+               return int(syscallEBADE), fmt.Errorf(errmsg)
        }
        g.httpCli = c
+       g.httpCli.SetLogLevel(g.log.Level.String())
+       g.log.Infoln("HTTP session ID:", g.httpCli.GetClientID())
+
+       // First call to check that xds-agent and server are alive
+       ver := xaapiv1.XDSVersion{}
+       if err := g.httpCli.Get("/version", &ver); err != nil {
+               return int(syscallEBADE), err
+       }
+       g.log.Infoln("XDS agent & server version:", ver)
+
+       // SEB Check that server is connected
+       // FIXME: add multi-servers support
 
-       // First call to check that xds-server is alive
+       // Get XDS projects list
        var data []byte
-       if err := c.HTTPGet("/folders", &data); err != nil {
+       if err := g.httpCli.HTTPGet("/projects", &data); err != nil {
                return int(syscallEBADE), err
        }
-       g.log.Infof("Result of /folders: %v", string(data[:]))
-       g.folders = []folder.FolderConfig{}
-       errMar := json.Unmarshal(data, &g.folders)
+
+       g.log.Infof("Result of /projects: %v", string(data[:]))
+       g.projects = []xaapiv1.ProjectConfig{}
+       errMar := json.Unmarshal(data, &g.projects)
        if errMar != nil {
-               g.log.Errorf("Cannot decode folders configuration: %s", errMar.Error())
+               g.log.Errorf("Cannot decode projects configuration: %s", errMar.Error())
        }
 
        // Check mandatory args
@@ -122,7 +143,7 @@ func (g *GdbXds) Init() (int, error) {
                Transport: "websocket",
                Header:    make(map[string][]string),
        }
-       opts.Header["XDS-SID"] = []string{c.GetClientID()}
+       opts.Header["XDS-AGENT-SID"] = []string{c.GetClientID()}
 
        iosk, err := sio_client.NewClient(baseURL, opts)
        if err != nil {
@@ -143,19 +164,19 @@ func (g *GdbXds) Init() (int, error) {
                }
        })
 
-       iosk.On(apiv1.ExecOutEvent, func(ev apiv1.ExecOutMsg) {
+       iosk.On(xaapiv1.ExecOutEvent, func(ev xaapiv1.ExecOutMsg) {
                if g.cbRead != nil {
                        g.cbRead(ev.Timestamp, ev.Stdout, ev.Stderr)
                }
        })
 
-       iosk.On(apiv1.ExecInferiorOutEvent, func(ev apiv1.ExecOutMsg) {
+       iosk.On(xaapiv1.ExecInferiorOutEvent, func(ev xaapiv1.ExecOutMsg) {
                if g.cbInferiorRead != nil {
                        g.cbInferiorRead(ev.Timestamp, ev.Stdout, ev.Stderr)
                }
        })
 
-       iosk.On(apiv1.ExecExitEvent, func(ev apiv1.ExecExitMsg) {
+       iosk.On(xaapiv1.ExecExitEvent, func(ev xaapiv1.ExecExitMsg) {
                if g.cbOnExit != nil {
                        g.cbOnExit(ev.Code, ev.Error)
                }
@@ -164,6 +185,7 @@ func (g *GdbXds) Init() (int, error) {
        return 0, nil
 }
 
+// Close frees allocated objects and close opened connections
 func (g *GdbXds) Close() error {
        g.cbOnDisconnect = nil
        g.cbOnError = nil
@@ -177,23 +199,23 @@ func (g *GdbXds) Close() error {
 
 // Start sends a request to start remotely gdb within xds-server
 func (g *GdbXds) Start(inferiorTTY bool) (int, error) {
-       var body []byte
        var err error
-       var folder *folder.FolderConfig
+       var project *xaapiv1.ProjectConfig
 
-       // Retrieve the folder definition
-       for _, f := range g.folders {
-               if f.ID == g.prjID {
-                       folder = &f
+       // Retrieve the project definition
+       for _, f := range g.projects {
+               // check as prefix to support short/partial id name
+               if strings.HasPrefix(f.ID, g.prjID) {
+                       project = &f
                        break
                }
        }
 
        // Auto setup rPath if needed
-       if g.rPath == "" && folder != nil {
+       if g.rPath == "" && project != nil {
                cwd, err := os.Getwd()
                if err == nil {
-                       fldRp := folder.ClientPath
+                       fldRp := project.ClientPath
                        if !strings.HasPrefix(fldRp, "/") {
                                fldRp = "/" + fldRp
                        }
@@ -209,7 +231,7 @@ func (g *GdbXds) Start(inferiorTTY bool) (int, error) {
        // except if XDS_GDBSERVER_OUTPUT_NOFIX is defined
        _, gdbserverNoFix := os.LookupEnv("XDS_GDBSERVER_OUTPUT_NOFIX")
 
-       args := apiv1.ExecArgs{
+       args := xaapiv1.ExecArgs{
                ID:              g.prjID,
                SdkID:           g.sdkID,
                Cmd:             g.ccmd,
@@ -220,24 +242,17 @@ func (g *GdbXds) Start(inferiorTTY bool) (int, error) {
                TTYGdbserverFix: !gdbserverNoFix,
                CmdTimeout:      -1, // no timeout, end when stdin close or command exited normally
        }
-       body, err = json.Marshal(args)
-       if err != nil {
-               return int(syscallEBADE), err
-       }
 
-       g.log.Infof("POST %s/exec %v", g.uri, string(body))
-       var res *http.Response
-       var found bool
-       res, err = g.httpCli.HTTPPostWithRes("/exec", string(body))
+       g.log.Infof("POST %s/exec %v", g.uri, args)
+       res := xaapiv1.ExecResult{}
+       err = g.httpCli.Post("/exec", args, &res)
        if err != nil {
                return int(syscall.EAGAIN), err
        }
-       dRes := make(map[string]interface{})
-       json.Unmarshal(g.httpCli.ResponseToBArray(res), &dRes)
-       if _, found = dRes["cmdID"]; !found {
-               return int(syscallEBADE), err
+       if res.CmdID == "" {
+               return int(syscallEBADE), fmt.Errorf("null CmdID")
        }
-       g.cmdID = dRes["cmdID"].(string)
+       g.cmdID = res.CmdID
 
        return 0, nil
 }
@@ -284,7 +299,7 @@ func (g *GdbXds) InferiorRead(f func(timestamp, stdout, stderr string)) {
 
 // Write writes message/string into gdb stdin
 func (g *GdbXds) Write(args ...interface{}) error {
-       return g.ioSock.Emit(apiv1.ExecInEvent, args...)
+       return g.ioSock.Emit(xaapiv1.ExecInEvent, args...)
 }
 
 // SendSignal is used to send a signal to remote process/gdb
@@ -293,26 +308,22 @@ func (g *GdbXds) SendSignal(sig os.Signal) error {
                return fmt.Errorf("cmdID not set")
        }
 
-       var body []byte
-       body, err := json.Marshal(apiv1.ExecSignalArgs{
+       sigArg := xaapiv1.ExecSignalArgs{
                CmdID:  g.cmdID,
                Signal: sig.String(),
-       })
-       if err != nil {
-               g.log.Errorf(err.Error())
        }
-       g.log.Debugf("POST /signal %s", string(body))
-       return g.httpCli.HTTPPost("/signal", string(body))
+       g.log.Debugf("POST /signal %v", sigArg)
+       return g.httpCli.Post("/signal", sigArg, nil)
 }
 
 //***** Private functions *****
 
 func (g *GdbXds) printProjectsList() (int, error) {
        msg := ""
-       if len(g.folders) > 0 {
+       if len(g.projects) > 0 {
                msg += "List of existing projects (use: export XDS_PROJECT_ID=<< ID >>): \n"
                msg += "  ID\t\t\t\t | Label"
-               for _, f := range g.folders {
+               for _, f := range g.projects {
                        msg += fmt.Sprintf("\n  %s\t | %s", f.ID, f.Label)
                        if f.DefaultSdk != "" {
                                msg += fmt.Sprintf("\t(default SDK: %s)", f.DefaultSdk)
@@ -321,27 +332,22 @@ func (g *GdbXds) printProjectsList() (int, error) {
                msg += "\n"
        }
 
-       var data []byte
-       if err := g.httpCli.HTTPGet("/sdks", &data); err != nil {
+       // FIXME : support multiple servers
+       sdks := []xaapiv1.SDK{}
+       if err := g.httpCli.Get("/servers/0/sdks", &sdks); err != nil {
                return int(syscallEBADE), err
        }
-       g.log.Infof("Result of /sdks: %v", string(data[:]))
-
-       sdks := []crosssdk.SDK{}
-       errMar := json.Unmarshal(data, &sdks)
-       if errMar == nil {
-               msg += "\nList of installed cross SDKs (use: export XDS_SDK_ID=<< ID >>): \n"
-               msg += "  ID\t\t\t\t\t | NAME\n"
-               for _, s := range sdks {
-                       msg += fmt.Sprintf("  %s\t | %s\n", s.ID, s.Name)
-               }
+       msg += "\nList of installed cross SDKs (use: export XDS_SDK_ID=<< ID >>): \n"
+       msg += "  ID\t\t\t\t\t | NAME\n"
+       for _, s := range sdks {
+               msg += fmt.Sprintf("  %s\t | %s\n", s.ID, s.Name)
        }
 
-       if len(g.folders) > 0 && len(sdks) > 0 {
+       if len(g.projects) > 0 && len(sdks) > 0 {
                msg += fmt.Sprintf("\n")
                msg += fmt.Sprintf("For example: \n")
                msg += fmt.Sprintf("  XDS_PROJECT_ID=%q XDS_SDK_ID=%q  %s -x myGdbConf.ini\n",
-                       g.folders[0].ID, sdks[0].ID, AppName)
+                       g.projects[0].ID, sdks[0].ID, AppName)
        }
 
        return 0, fmt.Errorf(msg)
index 1f80627..1fde7b8 100644 (file)
@@ -8,16 +8,14 @@ import:
   version: ^1.19.1
 - package: github.com/Sirupsen/logrus
   version: ^0.11.5
-- package: github.com/zhouhui8915/go-socket.io-client
-  version: master
-- package: github.com/iotbzh/xds-server
-  version: 6691c9f7c53d
+- package: github.com/sebd71/go-socket.io-client
+  version: 46defcb47f
+- package: github.com/iotbzh/xds-agent
+  version: 1.0.0-rc1
   subpackages:
-  - apiv1
-  - crosssdk
-  - xdsconfig
+  - lib/xaapiv1
 - package: github.com/iotbzh/xds-common
-  version: 363bac39b844
+  version: ^0.1.0
   subpackages:
   - golib/common
 - package: github.com/joho/godotenv
diff --git a/main.go b/main.go
index e7f14d6..dfe2a14 100644 (file)
--- a/main.go
+++ b/main.go
@@ -360,7 +360,7 @@ endloop:
                                sc := bufio.NewScanner(reader)
                                for sc.Scan() {
                                        data := sc.Text()
-                                       iosk.Emit(apiv1.ExecInferiorInEvent, data+"\n")
+                                       iosk.Emit(xaapiv1.ExecInferiorInEvent, data+"\n")
                                        log.Debugf("Inferior IN: <%v>", data)
                                }
                                if sc.Err() != nil {