Skip to content

Commit bcfb91c

Browse files
authored
ci: minor updates in pipeline (#93)
1 parent 23386cc commit bcfb91c

File tree

3 files changed

+28
-24
lines changed

3 files changed

+28
-24
lines changed

.github/workflows/push_pr.yml

Lines changed: 13 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: Push/PR pipelien
1+
name: Push/PR pipeline
22

33
on:
44
push:
@@ -24,6 +24,14 @@ jobs:
2424
- name: Validate code
2525
run: make ci/validate
2626

27+
snyk:
28+
name: Run security checks via snyk
29+
runs-on: ubuntu-20.04
30+
steps:
31+
- uses: actions/checkout@v2
32+
- name: Scan code for vulnerabilities
33+
run: make ci/snyk-test
34+
2735
test-nix:
2836
name: Run unit tests on *Nix
2937
runs-on: ubuntu-20.04
@@ -48,7 +56,7 @@ jobs:
4856
- name: Install Go
4957
uses: actions/setup-go@v2
5058
with:
51-
go-version: ${{env.GO_VERSION}}
59+
go-version: ${{env.GO_VERSION}}
5260
- name: Running unit tests
5361
shell: pwsh
5462
run: |
@@ -73,22 +81,13 @@ jobs:
7381
go-version: ${{env.GO_VERSION}}
7482
- name: Integration test
7583
env:
76-
GOPATH: "${{ github.workspace }}"
84+
GOPATH: ${{ github.workspace }}
7785
run: make integration-test
7886

79-
build:
80-
name: Build binary for all platforms:arch
87+
test-build:
88+
name: Test binary compilation for all platforms:arch
8189
runs-on: ubuntu-20.04
8290
steps:
8391
- uses: actions/checkout@v2
8492
- name: Build all platforms:arch
8593
run: make ci/build
86-
87-
snyk:
88-
name: Run security checks via snyk
89-
runs-on: ubuntu-20.04
90-
steps:
91-
- uses: actions/checkout@v2
92-
- name: Scan code for vulnerabilities
93-
run: make ci/snyk-test
94-

build/.goreleaser.yml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
builds:
2-
- id: nri-redis-nix
2+
- id: nri-nix
33
main: ./src
44
binary: nri-redis
55
gobinary: "go1.9.7"
@@ -17,7 +17,7 @@ builds:
1717
- goos: darwin
1818
goarch: 386
1919

20-
- id: nri-redis-win
20+
- id: nri-win
2121
main: ./src
2222
binary: nri-redis
2323
gobinary: "go1.9.7"
@@ -32,9 +32,9 @@ builds:
3232
pre: build/windows/set_exe_properties.sh {{ .Env.TAG }} "redis"
3333

3434
nfpms:
35-
- id: nri-redis-deb
35+
- id: nri-deb
3636
builds:
37-
- nri-redis-nix
37+
- nri-nix
3838
file_name_template: "{{ .ProjectName }}_{{ .Version }}-1_{{ .Arch }}"
3939
vendor: "New Relic, Inc."
4040
homepage: "https://www.newrelic.com/infrastructure"
@@ -55,9 +55,9 @@ nfpms:
5555
dependencies:
5656
- "newrelic-infra"
5757

58-
- id: nri-redis-rpm
58+
- id: nri-rpm
5959
builds:
60-
- nri-redis-nix
60+
- nri-nix
6161
file_name_template: "{{ .ProjectName }}-{{ .Version }}-{{ .Arch }}"
6262
vendor: "New Relic, Inc."
6363
homepage: "https://www.newrelic.com/infrastructure"
@@ -83,18 +83,18 @@ nfpms:
8383
amd64: 1.x86_64
8484

8585
archives:
86-
- id: nri-redis-nix
86+
- id: nri-nix
8787
builds:
88-
- nri-redis-nix
88+
- nri-nix
8989
name_template: "{{ .ProjectName }}_{{ .Os }}_{{ .Version }}_{{ .Arch }}_dirty"
9090
files:
9191
- redis-config.yml.sample
9292
- redis-definition.yml
9393
format: tar.gz
9494

95-
- id: nri-redis-win
95+
- id: nri-win
9696
builds:
97-
- nri-redis-win
97+
- nri-win
9898
name_template: "{{ .ProjectName }}-{{ .Arch }}.{{ .Version }}_dirty"
9999
files:
100100
- redis-win-config.yml.sample

build/ci.mk

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ ci/deps:
77
.PHONY : ci/debug-container
88
ci/debug-container: ci/deps
99
@docker run --rm -it \
10+
--name "nri-$(INTEGRATION)-debug" \
1011
-v $(CURDIR):/go/src/github.com/newrelic/nri-$(INTEGRATION) \
1112
-w /go/src/github.com/newrelic/nri-$(INTEGRATION) \
1213
-e PRERELEASE=true \
@@ -21,13 +22,15 @@ ci/debug-container: ci/deps
2122
.PHONY : ci/validate
2223
ci/validate: ci/deps
2324
@docker run --rm -t \
25+
--name "nri-$(INTEGRATION)-validate" \
2426
-v $(CURDIR):/go/src/github.com/newrelic/nri-$(INTEGRATION) \
2527
-w /go/src/github.com/newrelic/nri-$(INTEGRATION) \
2628
$(BUILDER_TAG) make validate
2729

2830
.PHONY : ci/test
2931
ci/test: ci/deps
3032
@docker run --rm -t \
33+
--name "nri-$(INTEGRATION)-test" \
3134
-v $(CURDIR):/go/src/github.com/newrelic/nri-$(INTEGRATION) \
3235
-w /go/src/github.com/newrelic/nri-$(INTEGRATION) \
3336
$(BUILDER_TAG) make test
@@ -45,6 +48,7 @@ ci/snyk-test:
4548
ci/build: ci/deps
4649
ifdef TAG
4750
@docker run --rm -t \
51+
--name "nri-$(INTEGRATION)-build" \
4852
-v $(CURDIR):/go/src/github.com/newrelic/nri-$(INTEGRATION) \
4953
-w /go/src/github.com/newrelic/nri-$(INTEGRATION) \
5054
-e INTEGRATION \
@@ -59,6 +63,7 @@ endif
5963
ci/prerelease: ci/deps
6064
ifdef TAG
6165
@docker run --rm -t \
66+
--name "nri-$(INTEGRATION)-prerelease" \
6267
-v $(CURDIR):/go/src/github.com/newrelic/nri-$(INTEGRATION) \
6368
-w /go/src/github.com/newrelic/nri-$(INTEGRATION) \
6469
-e INTEGRATION \

0 commit comments

Comments
 (0)