Skip to content

Commit

Permalink
Add missing -L flag when using curl command to download release
Browse files Browse the repository at this point in the history
  • Loading branch information
Etienne Tremel committed Mar 13, 2018
1 parent 61e3f5a commit 8ce3a46
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 8 deletions.
10 changes: 10 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
Changelog
=========

## v0.1.1

- install.sh: add missing `-L` flag to curl command and do not rely on bash

## v0.1.0

- Initial release
6 changes: 3 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,11 @@ build:
dist:
mkdir -p $(DIST)
GOOS=linux GOARCH=amd64 go build -o $(BINARY) -ldflags $(LDFLAGS) ./cmd/...
tar -zcvf $(DIST)/helm-monitor_linux_$(VERSION).tgz $(BINARY) README.md LICENSE plugin.yaml
tar -zcvf $(DIST)/helm-monitor_linux_$(VERSION).tar.gz $(BINARY) README.md LICENSE plugin.yaml
GOOS=darwin GOARCH=amd64 go build -o $(BINARY) -ldflags $(LDFLAGS) ./cmd/...
tar -zcvf $(DIST)/helm-monitor_darwin_$(VERSION).tgz $(BINARY) README.md LICENSE plugin.yaml
tar -zcvf $(DIST)/helm-monitor_darwin_$(VERSION).tar.gz $(BINARY) README.md LICENSE plugin.yaml
GOOS=windows GOARCH=amd64 go build -o $(BINARY).exe -ldflags $(LDFLAGS) ./cmd/...
tar -zcvf $(DIST)/helm-monitor_windows_$(VERSION).tgz $(BINARY).exe README.md LICENSE plugin.yaml
tar -zcvf $(DIST)/helm-monitor_windows_$(VERSION).tar.gz $(BINARY).exe README.md LICENSE plugin.yaml

test-all: vet lint test

Expand Down
8 changes: 4 additions & 4 deletions install.sh
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/bin/bash
#!/bin/sh

set -e

Expand All @@ -7,7 +7,7 @@ HELM_MONITOR_VERSION=${HELM_MONITOR_VERSION:-$current_version}

dir=${HELM_PLUGIN_DIR:-"$(helm home)/plugins/helm-monitor"}
os=$(uname -s | tr '[:upper:]' '[:lower:]')
release_file="helm-monitor_${os}_${HELM_MONITOR_VERSION}.tgz"
release_file="helm-monitor_${os}_${HELM_MONITOR_VERSION}.tar.gz"
url="https://github.com/ContainerSolutions/helm-monitor/releases/download/v${HELM_MONITOR_VERSION}/${release_file}"

mkdir -p $dir
Expand All @@ -16,10 +16,10 @@ if command -v wget
then
wget -O ${dir}/${release_file} $url
elif command -v curl; then
curl -o ${dir}/${release_file} $url
curl -L -o ${dir}/${release_file} $url
fi

tar xf ${dir}/${release_file} -C $dir
tar xvf ${dir}/${release_file} -C $dir

chmod +x ${dir}/helm-monitor

Expand Down
2 changes: 1 addition & 1 deletion plugin.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
---
name: "monitor"
version: "0.1.0"
version: "0.1.1"
usage: "monitor and rollback in case of failure based on metrics or logs"
description: |-
Query at a given interval a Prometheus or ElasticSearch instance, a rollback
Expand Down

0 comments on commit 8ce3a46

Please sign in to comment.