Update CI config to include a release stage to publish github binaries

Signed-off-by: Jesus Vazquez <jesus.vazquez@grafana.com>

.circleci/config.yml

@@ -1,4 +1,38 @@
-version: 2
+version: 2.1
+workflows:
+  pr_acceptance:
+    jobs:
+      - test
+      - build
+  build_accept_deploy:
+    jobs:
+      - test
+      - build
+      - package:
+          requires:
+            - build
+      - deploy:
+          requires:
+            - test
+            - build
+            - package
+  publish_release:
+    jobs:
+      - test:
+          filters:
+            branches:
+              ignore: /.*/
+            tags:
+              only: /v[0-9]+(\.[0-9]+)*(-.*)*/
+      - github_binaries:
+          requires:
+            - test
+          filters:
+            branches:
+              ignore: /.*/
+            tags:
+              only: /v[0-9]+(\.[0-9]+)*(-.*)*/
+
 jobs:
   test:
     working_directory: /home/circleci/go/src/github.com/grafana/carbon-relay-ng
@@ -96,21 +130,12 @@ jobs:
 
                docker push grafana/carbon-relay-ng:$tag
             fi
-workflows:
-  version: 2
-  pr_acceptance:
-    jobs:
-      - test
-      - build
-  build_accept_deploy:
-    jobs:
-      - test
-      - build
-      - package:
-          requires:
-            - build
-      - deploy:
-          requires:
-            - test
-            - build
-            - package
+
+  github_binaries:
+      docker:
+        - image: cimg/go:1.18
+      steps:
+        - checkout
+        - run: curl -sfL https://goreleaser.com/static/run | bash
+        # For dry runs use the following and comment the above
+        # - run: curl -sfL https://goreleaser.com/static/run | bash -s -- --rm-dist --skip-publish --snapshot

go-whisper/whisper.go

@@ -59,13 +59,13 @@ func parseRetentionPart(retentionPart string) (int, error) {
 }
 
 /*
-Parse a retention definition as you would find in the storage-schemas.conf of a Carbon install.
-Note that this only parses a single retention definition, if you have multiple definitions (separated by a comma)
-you will have to split them yourself.
+  Parse a retention definition as you would find in the storage-schemas.conf of a Carbon install.
+  Note that this only parses a single retention definition, if you have multiple definitions (separated by a comma)
+  you will have to split them yourself.
 
-ParseRetentionDef("10s:14d") Retention{10, 120960}
+  ParseRetentionDef("10s:14d") Retention{10, 120960}
 
-See: http://graphite.readthedocs.org/en/1.0/config-carbon.html#storage-schemas-conf
+  See: http://graphite.readthedocs.org/en/1.0/config-carbon.html#storage-schemas-conf
 */
 func ParseRetentionDef(retentionDef string) (*Retention, error) {
 	parts := strings.Split(retentionDef, ":")
@@ -128,10 +128,10 @@ func validateRetentions(retentions Retentions) error {
 }
 
 /*
-A retention level.
+  A retention level.
 
-Retention levels describe a given archive in the database. How detailed it is and how far back
-it records.
+  Retention levels describe a given archive in the database. How detailed it is and how far back
+  it records.
 */
 type Retention struct {
 	secondsPerPoint int
@@ -178,8 +178,8 @@ func (r retentionsByPrecision) Less(i, j int) bool {
 }
 
 /*
-Implementation of modulo that works like Python
-Thanks @timmow for this
+   Implementation of modulo that works like Python
+   Thanks @timmow for this
 */
 func mod(a, b int) int {
 	return a - (b * int(math.Floor(float64(a)/float64(b))))

ui/web/web.go

@@ -266,21 +266,20 @@ func parseRewriterRequest(r *http.Request) (rewriter.RW, *handlerError) {
 	return rw, nil
 }
 
-/*
-needs updating, but using what api?
-
-	func updateRoute(w http.ResponseWriter, r *http.Request) (interface{}, *handlerError) {
-		payload, err := parseRouteRequest(r)
-		if err != nil {
-			return nil, err
-		}
-
-		e := routes.Update(payload.Key, &payload.Addr, &payload.Patt)
-		if e != nil {
-			return nil, &handlerError{e, "Could not update route (" + e.Error() + ")", http.StatusBadRequest}
-		}
-		return routes.Map[payload.Key], nil
+/* needs updating, but using what api?
+func updateRoute(w http.ResponseWriter, r *http.Request) (interface{}, *handlerError) {
+	payload, err := parseRouteRequest(r)
+	if err != nil {
+		return nil, err
 	}
+
+	e := routes.Update(payload.Key, &payload.Addr, &payload.Patt)
+	if e != nil {
+		return nil, &handlerError{e, "Could not update route (" + e.Error() + ")", http.StatusBadRequest}
+	}
+	return routes.Map[payload.Key], nil
+}
+
 */
 func addAggregate(w http.ResponseWriter, r *http.Request) (interface{}, *handlerError) {
 	aggregate, err := parseAggregateRequest(r)