10666: Replaced 'version' package with 'version' var
authorLucas Di Pentima <ldipentima@veritasgenetics.com>
Thu, 30 Nov 2017 20:26:42 +0000 (17:26 -0300)
committerLucas Di Pentima <ldipentima@veritasgenetics.com>
Thu, 30 Nov 2017 20:26:42 +0000 (17:26 -0300)
on every go program.

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <ldipentima@veritasgenetics.com>

21 files changed:
build/run-tests.sh
sdk/go/version/version.go [deleted file]
sdk/go/version/version_test.go [deleted file]
services/arv-git-httpd/main.go
services/crunch-dispatch-local/crunch-dispatch-local.go
services/crunch-dispatch-slurm/crunch-dispatch-slurm.go
services/crunch-run/crunchrun.go
services/crunchstat/crunchstat.go
services/health/main.go
services/keep-balance/main.go
services/keep-web/handler.go
services/keep-web/main.go
services/keepproxy/keepproxy.go
services/keepstore/handlers.go
services/keepstore/keepstore.go
services/ws/main.go
services/ws/router.go
tools/arv-sync-groups/arv-sync-groups.go
tools/keep-block-check/keep-block-check.go
tools/keep-exercise/keep-exercise.go
tools/keep-rsync/keep-rsync.go

index 8369632921b7e66391120cd35370873f55e43765..ffe0e5098bae1268a2fe62481c58565a88957b3b 100755 (executable)
@@ -102,7 +102,6 @@ sdk/go/manifest
 sdk/go/blockdigest
 sdk/go/streamer
 sdk/go/stats
-sdk/go/version
 sdk/go/crunchrunner
 sdk/cwl
 tools/arv-sync-groups
@@ -831,7 +830,6 @@ gostuff=(
     sdk/go/streamer
     sdk/go/crunchrunner
     sdk/go/stats
-    sdk/go/version
     lib/crunchstat
     services/arv-git-httpd
     services/crunchstat
diff --git a/sdk/go/version/version.go b/sdk/go/version/version.go
deleted file mode 100644 (file)
index ef9abe7..0000000
+++ /dev/null
@@ -1,15 +0,0 @@
-package version
-
-var (
-       // Version will get assigned the release number at compile time
-       Version string
-)
-
-// GetVersion returns the release number if it was assigned by the compiler
-// or "dev" otherwise.
-func GetVersion() string {
-       if Version != "" {
-               return Version
-       }
-       return "dev"
-}
diff --git a/sdk/go/version/version_test.go b/sdk/go/version/version_test.go
deleted file mode 100644 (file)
index 426c124..0000000
+++ /dev/null
@@ -1,30 +0,0 @@
-// Copyright (C) The Arvados Authors. All rights reserved.
-//
-// SPDX-License-Identifier: Apache-2.0
-
-package version
-
-import (
-       "testing"
-
-       . "gopkg.in/check.v1"
-)
-
-// Gocheck boilerplate
-func Test(t *testing.T) {
-       TestingT(t)
-}
-
-type TestSuite struct {
-}
-
-var _ = Suite(&TestSuite{})
-
-func (s *TestSuite) TestVersion(c *C) {
-       // Default version string when Version is not set
-       c.Assert(Version, Equals, "")
-       c.Assert(GetVersion(), Equals, "dev")
-       // Simulate linker flag setting Version var
-       Version = "1.0.0"
-       c.Assert(GetVersion(), Equals, "1.0.0")
-}
index 2750ab9eaf0ab51b8517deba914f8726592ce674..76aacc4e9d5d4c9b7515705720c8d5dc7469f7c3 100644 (file)
@@ -14,10 +14,11 @@ import (
 
        "git.curoverse.com/arvados.git/sdk/go/arvados"
        "git.curoverse.com/arvados.git/sdk/go/config"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
        "github.com/coreos/go-systemd/daemon"
 )
 
+var version = "dev"
+
 // Server configuration
 type Config struct {
        Client          arvados.Client
@@ -62,7 +63,7 @@ func main() {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
@@ -93,7 +94,7 @@ func main() {
        if _, err := daemon.SdNotify(false, "READY=1"); err != nil {
                log.Printf("Error notifying init daemon: %v", err)
        }
-       log.Printf("arv-git-httpd %q started", arvadosVersion.GetVersion())
+       log.Printf("arv-git-httpd %q started", version)
        log.Println("Listening at", srv.Addr)
        log.Println("Repository root", theConfig.RepoRoot)
        if err := srv.Wait(); err != nil {
index c1d6d562a2c963209be94a590fcd14bc8eea272b..1529c4fbab20b98b08ac738b61f889a33ddf26fd 100644 (file)
@@ -21,9 +21,10 @@ import (
        "git.curoverse.com/arvados.git/sdk/go/arvados"
        "git.curoverse.com/arvados.git/sdk/go/arvadosclient"
        "git.curoverse.com/arvados.git/sdk/go/dispatch"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
 )
 
+var version = "dev"
+
 func main() {
        err := doMain()
        if err != nil {
@@ -61,11 +62,11 @@ func doMain() error {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
-       log.Printf("crunch-dispatch-local %q started", arvadosVersion.GetVersion())
+       log.Printf("crunch-dispatch-local %q started", version)
 
        runningCmds = make(map[string]*exec.Cmd)
 
index e743fa585451b732734e2a3e10ec6d1c60298221..5eeb32e62fd071326661da8b0c365d85799dccbd 100644 (file)
@@ -23,10 +23,11 @@ import (
        "git.curoverse.com/arvados.git/sdk/go/arvadosclient"
        "git.curoverse.com/arvados.git/sdk/go/config"
        "git.curoverse.com/arvados.git/sdk/go/dispatch"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
        "github.com/coreos/go-systemd/daemon"
 )
 
+var version = "dev"
+
 // Config used by crunch-dispatch-slurm
 type Config struct {
        Client arvados.Client
@@ -79,11 +80,11 @@ func doMain() error {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
-       log.Printf("crunch-dispatch-slurm %q started", arvadosVersion.GetVersion())
+       log.Printf("crunch-dispatch-slurm %q started", version)
 
        err := readConfig(&theConfig, *configPath)
        if err != nil {
index c45a3f0599f3bd4967c6e9edfe7a8eed053fedf0..a957874f91504215b875a3e352e5c57d5d5c56b1 100644 (file)
@@ -32,7 +32,6 @@ import (
        "git.curoverse.com/arvados.git/sdk/go/arvadosclient"
        "git.curoverse.com/arvados.git/sdk/go/keepclient"
        "git.curoverse.com/arvados.git/sdk/go/manifest"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
 
        dockertypes "github.com/docker/docker/api/types"
        dockercontainer "github.com/docker/docker/api/types/container"
@@ -40,6 +39,8 @@ import (
        dockerclient "github.com/docker/docker/client"
 )
 
+var version = "dev"
+
 // IArvadosClient is the minimal Arvados API methods used by crunch-run.
 type IArvadosClient interface {
        Create(resourceType string, parameters arvadosclient.Dict, output interface{}) error
@@ -1419,7 +1420,7 @@ func (runner *ContainerRunner) NewArvLogWriter(name string) io.WriteCloser {
 
 // Run the full container lifecycle.
 func (runner *ContainerRunner) Run() (err error) {
-       runner.CrunchLog.Printf("crunch-run %q started", arvadosVersion.GetVersion())
+       runner.CrunchLog.Printf("crunch-run %q started", version)
        runner.CrunchLog.Printf("Executing container '%s'", runner.Container.UUID)
 
        hostname, hosterr := os.Hostname()
@@ -1600,11 +1601,11 @@ func main() {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
-       log.Printf("crunch-run %q started", arvadosVersion.GetVersion())
+       log.Printf("crunch-run %q started", version)
 
        containerId := flag.Arg(0)
 
index 454a5fc3a4363d24b6f858183da6d404750ea6a1..248ec6259db11a94c68b4c602169f107ab531c27 100644 (file)
@@ -17,7 +17,6 @@ import (
        "time"
 
        "git.curoverse.com/arvados.git/lib/crunchstat"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
 )
 
 const MaxLogLine = 1 << 14 // Child stderr lines >16KiB will be split
@@ -25,6 +24,7 @@ const MaxLogLine = 1 << 14 // Child stderr lines >16KiB will be split
 var (
        signalOnDeadPPID  int = 15
        ppidCheckInterval     = time.Second
+       version               = "dev"
 )
 
 func main() {
@@ -44,11 +44,11 @@ func main() {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
-       reporter.Logger.Printf("crunchstat %q started", arvadosVersion.GetVersion())
+       reporter.Logger.Printf("crunchstat %q started", version)
 
        if reporter.CgroupRoot == "" {
                reporter.Logger.Fatal("error: must provide -cgroup-root")
index d04b55bc6c1a111b5ad5c1b8418dcddf131fe4e0..4b553659ed4c467b9f87c7713237789e6fb3046c 100644 (file)
@@ -9,10 +9,11 @@ import (
        "git.curoverse.com/arvados.git/sdk/go/arvados"
        "git.curoverse.com/arvados.git/sdk/go/health"
        "git.curoverse.com/arvados.git/sdk/go/httpserver"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
        log "github.com/Sirupsen/logrus"
 )
 
+var version = "dev"
+
 func main() {
        configFile := flag.String("config", arvados.DefaultConfigFile, "`path` to arvados configuration file")
        getVersion := flag.Bool("version", false, "Print version information and exit.")
@@ -20,14 +21,14 @@ func main() {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
        log.SetFormatter(&log.JSONFormatter{
                TimestampFormat: "2006-01-02T15:04:05.000000000Z07:00",
        })
-       log.Printf("arvados health %q started", arvadosVersion.GetVersion())
+       log.Printf("arvados health %q started", version)
 
        cfg, err := arvados.GetConfig(*configFile)
        if err != nil {
index fe4a8c41c572932853076a8b0465d5e0bee01049..ecc3f2696f6ab2ef29b4ece9d3071dd75c91389b 100644 (file)
@@ -16,9 +16,10 @@ import (
 
        "git.curoverse.com/arvados.git/sdk/go/arvados"
        "git.curoverse.com/arvados.git/sdk/go/config"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
 )
 
+var version = "dev"
+
 const defaultConfigPath = "/etc/arvados/keep-balance/keep-balance.yml"
 
 // Config specifies site configuration, like API credentials and the
@@ -93,7 +94,7 @@ func main() {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
@@ -106,7 +107,7 @@ func main() {
                log.Fatal(config.DumpAndExit(cfg))
        }
 
-       log.Printf("keep-balance %q started", arvadosVersion.GetVersion())
+       log.Printf("keep-balance %q started", version)
 
        if *debugFlag {
                debugf = log.Printf
index 161864a8b948581299744ee4ea16020241fe6dc6..7bdd521226a5ab8789862a2e5d3f3f274a059493 100644 (file)
@@ -24,7 +24,6 @@ import (
        "git.curoverse.com/arvados.git/sdk/go/health"
        "git.curoverse.com/arvados.git/sdk/go/httpserver"
        "git.curoverse.com/arvados.git/sdk/go/keepclient"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
        "golang.org/x/net/webdav"
 )
 
@@ -94,7 +93,7 @@ func (h *handler) serveStatus(w http.ResponseWriter, r *http.Request) {
                Version string
        }{
                cacheStats: h.Config.Cache.Stats(),
-               Version:    arvadosVersion.GetVersion(),
+               Version:    version,
        }
        json.NewEncoder(w).Encode(status)
 }
index 8bd890322e2cae303c508304c33d852038a2557d..938377b5256e4b5dfad6ba6f333e527b5dd10873 100644 (file)
@@ -13,12 +13,12 @@ import (
 
        "git.curoverse.com/arvados.git/sdk/go/arvados"
        "git.curoverse.com/arvados.git/sdk/go/config"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
        "github.com/coreos/go-systemd/daemon"
 )
 
 var (
        defaultConfigPath = "/etc/arvados/keep-web/keep-web.yml"
+       version           = "dev"
 )
 
 // Config specifies server configuration.
@@ -94,7 +94,7 @@ func main() {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
@@ -115,7 +115,7 @@ func main() {
                log.Fatal(config.DumpAndExit(cfg))
        }
 
-       log.Printf("keep-web %q started", arvadosVersion.GetVersion())
+       log.Printf("keep-web %q started", version)
 
        os.Setenv("ARVADOS_API_HOST", cfg.Client.APIHost)
        srv := &server{Config: cfg}
index ec074cff3f4a772a39e7a345d3e4307be5a04c7f..59eec0bb04a3e67842334cbd429a006ba178f80d 100644 (file)
@@ -26,12 +26,13 @@ import (
        "git.curoverse.com/arvados.git/sdk/go/config"
        "git.curoverse.com/arvados.git/sdk/go/health"
        "git.curoverse.com/arvados.git/sdk/go/keepclient"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
        "github.com/coreos/go-systemd/daemon"
        "github.com/ghodss/yaml"
        "github.com/gorilla/mux"
 )
 
+var version = "dev"
+
 type Config struct {
        Client          arvados.Client
        Listen          string
@@ -80,7 +81,7 @@ func main() {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
@@ -107,7 +108,7 @@ func main() {
                log.Fatal(config.DumpAndExit(cfg))
        }
 
-       log.Printf("keepproxy %q started", arvadosVersion.GetVersion())
+       log.Printf("keepproxy %q started", version)
 
        arv, err := arvadosclient.New(&cfg.Client)
        if err != nil {
index 5cd3aa8b7146d18a6eb4506ddee8cb9a3bcdb7a0..daf4fc69ddff2702f0da43fd0af4d4927374bb2c 100644 (file)
@@ -31,7 +31,6 @@ import (
 
        "git.curoverse.com/arvados.git/sdk/go/health"
        "git.curoverse.com/arvados.git/sdk/go/httpserver"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
        log "github.com/Sirupsen/logrus"
 )
 
@@ -348,7 +347,7 @@ func (rtr *router) StatusHandler(resp http.ResponseWriter, req *http.Request) {
 
 // populate the given NodeStatus struct with current values.
 func (rtr *router) readNodeStatus(st *NodeStatus) {
-       st.Version = arvadosVersion.GetVersion()
+       st.Version = version
        vols := KeepVM.AllReadable()
        if cap(st.Volumes) < len(vols) {
                st.Volumes = make([]*volumeStatusEnt, len(vols))
index 23af6906bbcf89ec0832e8416a3198096edbef1c..06a26984d002ad3cbe9372cdaaad2a651b998990 100644 (file)
@@ -18,11 +18,12 @@ import (
        "git.curoverse.com/arvados.git/sdk/go/config"
        "git.curoverse.com/arvados.git/sdk/go/httpserver"
        "git.curoverse.com/arvados.git/sdk/go/keepclient"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
        log "github.com/Sirupsen/logrus"
        "github.com/coreos/go-systemd/daemon"
 )
 
+var version = "dev"
+
 // A Keep "block" is 64MB.
 const BlockSize = 64 * 1024 * 1024
 
@@ -104,7 +105,7 @@ func main() {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
@@ -119,7 +120,7 @@ func main() {
                log.Fatal(config.DumpAndExit(theConfig))
        }
 
-       log.Printf("keepstore %q started", arvadosVersion.GetVersion())
+       log.Printf("keepstore %q started", version)
 
        err = theConfig.Start()
        if err != nil {
index 30d569c98890264aa4a1028085d94764e01a7da4..df462eafc2859e6d8aa071dbe851737cf3be2037 100644 (file)
@@ -11,10 +11,10 @@ import (
 
        "git.curoverse.com/arvados.git/sdk/go/config"
        "git.curoverse.com/arvados.git/sdk/go/ctxlog"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
 )
 
 var logger = ctxlog.FromContext
+var version = "dev"
 
 func main() {
        log := logger(nil)
@@ -27,7 +27,7 @@ func main() {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
@@ -48,7 +48,7 @@ func main() {
                return
        }
 
-       log.Printf("arvados-ws %q started", arvadosVersion.GetVersion())
+       log.Printf("arvados-ws %q started", version)
 
        log.Info("started")
        srv := &server{wsConfig: &cfg}
index 5fa16743520dd2eb93a3eb4cdcead5824a327186..987c225eac8ba5a456aa0c2af2e7f56eede749be 100644 (file)
@@ -15,7 +15,6 @@ import (
 
        "git.curoverse.com/arvados.git/sdk/go/ctxlog"
        "git.curoverse.com/arvados.git/sdk/go/health"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
        "github.com/Sirupsen/logrus"
        "golang.org/x/net/websocket"
 )
@@ -124,7 +123,7 @@ func (rtr *router) DebugStatus() interface{} {
 func (rtr *router) Status() interface{} {
        return map[string]interface{}{
                "Clients": atomic.LoadInt64(&rtr.status.ReqsActive),
-               "Version": arvadosVersion.GetVersion(),
+               "Version": version,
        }
 }
 
index 3530ce56deb54528186afbc5a360b164baea95a7..e88fac66e001b3b1fb93e76ee93020cbb3ea7fac 100644 (file)
@@ -17,9 +17,10 @@ import (
        "strings"
 
        "git.curoverse.com/arvados.git/sdk/go/arvados"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
 )
 
+var version = "dev"
+
 type resourceList interface {
        Len() int
        GetItems() []interface{}
@@ -165,7 +166,7 @@ func ParseFlags(config *ConfigParams) error {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
@@ -287,7 +288,7 @@ func doMain(cfg *ConfigParams) error {
        }
        defer f.Close()
 
-       log.Printf("arv-sync-groups %q started. Using %q as users id and parent group UUID %q", arvadosVersion.GetVersion(), cfg.UserID, cfg.ParentGroupUUID)
+       log.Printf("arv-sync-groups %q started. Using %q as users id and parent group UUID %q", version, cfg.UserID, cfg.ParentGroupUUID)
 
        // Get the complete user list to minimize API Server requests
        allUsers := make(map[string]arvados.User)
index 9d3b45a6bd988a20f01838dfa8b176098152c7f9..9f6ff984aec2b9d26dd153a53df701aefe6ead96 100644 (file)
@@ -18,9 +18,10 @@ import (
 
        "git.curoverse.com/arvados.git/sdk/go/arvadosclient"
        "git.curoverse.com/arvados.git/sdk/go/keepclient"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
 )
 
+var version = "dev"
+
 func main() {
        err := doMain(os.Args[1:])
        if err != nil {
@@ -73,7 +74,7 @@ func doMain(args []string) error {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
index d5386b0ef5d4cce751b0ee7731e8682cc8ae7d41..4c8a3449be4b5d1fd819c0cb423a91a8fa6eba98 100644 (file)
@@ -32,9 +32,10 @@ import (
 
        "git.curoverse.com/arvados.git/sdk/go/arvadosclient"
        "git.curoverse.com/arvados.git/sdk/go/keepclient"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
 )
 
+var version = "dev"
+
 // Command line config knobs
 var (
        BlockSize     = flag.Int("block-size", keepclient.BLOCKSIZE, "bytes per read/write op")
@@ -54,11 +55,11 @@ func main() {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }
 
-       log.Printf("keep-exercise %q started", arvadosVersion.GetVersion())
+       log.Printf("keep-exercise %q started", version)
 
        arv, err := arvadosclient.MakeArvadosClient()
        if err != nil {
index df7f1caf3e0f990aaf781792475871d0f4dbbe64..55c1a3b09054fb82a2de3fc3fbbd7d9e1e8904b8 100644 (file)
@@ -19,9 +19,10 @@ import (
 
        "git.curoverse.com/arvados.git/sdk/go/arvadosclient"
        "git.curoverse.com/arvados.git/sdk/go/keepclient"
-       arvadosVersion "git.curoverse.com/arvados.git/sdk/go/version"
 )
 
+var version = "dev"
+
 func main() {
        err := doMain()
        if err != nil {
@@ -80,7 +81,7 @@ func doMain() error {
 
        // Print version information if requested
        if *getVersion {
-               fmt.Printf("Version: %s\n", arvadosVersion.GetVersion())
+               fmt.Printf("Version: %s\n", version)
                os.Exit(0)
        }