Skip to content

Commit

Permalink
Fix: test release on PRs
Browse files Browse the repository at this point in the history
Everything _but_ make release-snapshot is running for PRs. That doesn't
make sense.

Remove arm because k6 does not support it. We might restore it in the
future if we can modify the Goreleaser file to conditionally ignore it
for that architecture.

Signed-off-by: Marcelo E. Magallon <[email protected]>
  • Loading branch information
mem committed Dec 13, 2023
1 parent 82cbec1 commit daee222
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 18 deletions.
14 changes: 1 addition & 13 deletions .drone.yml
Original file line number Diff line number Diff line change
Expand Up @@ -213,10 +213,6 @@ steps:
NFPM_SIGNING_KEY_FILE: /drone/src/release-private-key.key
image: ghcr.io/grafana/grafana-build-tools:v0.4.0
name: test release
when:
ref:
- refs/heads/main
- refs/tags/v*.*.*
- commands:
- ./scripts/package/verify-deb-install.sh
image: docker
Expand All @@ -225,10 +221,6 @@ steps:
volumes:
- name: docker
path: /var/run/docker.sock
when:
ref:
- refs/heads/main
- refs/tags/v*.*.*
- commands:
- ./scripts/package/verify-rpm-install.sh
image: docker
Expand All @@ -237,10 +229,6 @@ steps:
volumes:
- name: docker
path: /var/run/docker.sock
when:
ref:
- refs/heads/main
- refs/tags/v*.*.*
- commands:
- make release
environment:
Expand Down Expand Up @@ -312,6 +300,6 @@ kind: secret
name: gpg_private_key
---
kind: signature
hmac: f3e837fc29e4096762e3d698581dce4b67160fa79766910e8898efadf99a6010
hmac: 5fb7efe6e79672a68aadbece06cb2ab9a879000265c58781d68d0998967ca9bb

...
3 changes: 1 addition & 2 deletions .goreleaser.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ builds:
- darwin
goarch:
- amd64
- arm
- arm64
ignore:
- goos: darwin
Expand Down Expand Up @@ -59,7 +58,7 @@ nfpms:
dst: /etc/systemd/system/synthetic-monitoring-agent.service

# Copy k6 as sm-k6 to prevent clashing with k6 if it's installed.
- src: ./dist/synthetic-monitoring-agent_{{.Os}}_{{.Arch}}/k6
- src: ./dist/synthetic-monitoring-agent_{{.Os}}_{{.Arch}}{{ with .Amd64 }}_{{ . }}{{ end}}/k6
dst: /usr/bin/sm-k6
rpm:
signature:
Expand Down
6 changes: 3 additions & 3 deletions scripts/configs/drone/main.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -271,13 +271,13 @@ local docker_publish(repo, auth, tag, os, arch, version='') =
NFPM_SIGNING_KEY_FILE: '/drone/src/release-private-key.key',
},
},
step('test release', [ 'make release-snapshot' ]) + devAndRelease + {
step('test release', [ 'make release-snapshot' ]) + {
environment: {
NFPM_DEFAULT_PASSPHRASE: { from_secret: 'gpg_passphrase' },
NFPM_SIGNING_KEY_FILE: '/drone/src/release-private-key.key',
},
},
step('test deb package', [ './scripts/package/verify-deb-install.sh' ], image='docker') + devAndRelease + {
step('test deb package', [ './scripts/package/verify-deb-install.sh' ], image='docker') + {
volumes: [
{
name: 'docker',
Expand All @@ -286,7 +286,7 @@ local docker_publish(repo, auth, tag, os, arch, version='') =
],
privileged: true,
},
step('test rpm package', [ './scripts/package/verify-rpm-install.sh' ], image='docker') + devAndRelease + {
step('test rpm package', [ './scripts/package/verify-rpm-install.sh' ], image='docker') + {
volumes: [
{
name: 'docker',
Expand Down

0 comments on commit daee222

Please sign in to comment.