Merge ~sergiodj/prometheus-snap:rock into prometheus-snap:20.04

Proposed by Sergio Durigan Junior
Status: Merged
Merge reported by: Sergio Durigan Junior
Merged at revision: f426b01f00f507b50183b9e85bf14f4ef3fe8ce8
Proposed branch: ~sergiodj/prometheus-snap:rock
Merge into: prometheus-snap:20.04
Diff against target: 37 lines (+2/-12)
1 file modified
snapcraft.yaml (+2/-12)
Reviewer Review Type Date Requested Status
Athos Ribeiro Approve
Canonical Server Pending
Review via email: mp+416567@code.launchpad.net

Description of the change

This MP serves two purposes:

- It updates the prometheus-alertmanager snap to the latest upstream version (2.33.5)

- It it the starting point of the new "rock" branch that will be used to release this snap from now on. For more information, see:

https://forum.snapcraft.io/t/improvements-on-rocks-snap-tracks/29046

I took the opportunity to also:

- Update nodejs to the latest LTS version (16.14.0)

- Remove a (now unnecessary) hack used to make the snap build with a previous upstream version. This bug has since been fixed.

Everything builds fine and my local test also succeeded.

To post a comment you must log in.
Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

FWIW, this MP is targeting the 20.04 branch just to make things easier to review. As mentioned above, this will actually be a new branch called "rock".

Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

Thanks, Sergio! This LGTM. The commit message does refer to prometheus-alertmanager here. Shouldn't it be prometheus instead?

review: Needs Information
Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Thanks for the review, Athos. You're correct: it should refer to prometheus. I've fixed the message and force-pushed the commit. Care to take another quick look?

Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/snapcraft.yaml b/snapcraft.yaml
2index 05401d6..fb2e8b7 100644
3--- a/snapcraft.yaml
4+++ b/snapcraft.yaml
5@@ -1,5 +1,5 @@
6 name: prometheus
7-version: 2.32.1
8+version: 2.33.5
9 base: core20
10 grade: stable
11 summary: The Prometheus monitoring system and time series database
12@@ -36,7 +36,7 @@ parts:
13 build-packages: [build-essential, curl, make, wget]
14 override-build: |
15 set -x
16- NODE_VERSION=16.13.2
17+ NODE_VERSION=16.14.0
18 NODE_ARCH=$(dpkg --print-architecture)
19 case "${NODE_ARCH}" in
20 "amd64")
21@@ -52,16 +52,6 @@ parts:
22 export PATH=/usr/local/lib/nodejs/node-v${NODE_VERSION}-linux-${NODE_ARCH}/bin:$PATH
23 NODE_ENV="production"
24 npm install yarn -g
25- # See https://github.com/prometheus/prometheus/issues/10185
26- # for an explanation regarding the snippet below.
27- if [ "${SNAPCRAFT_PROJECT_VERSION}" = "2.32.1" ]; then
28- rm -fv template/template_test.go
29- else
30- echo
31- echo "FIXME: Please verify if the template/template_test.go still fails."
32- echo
33- exit 1
34- fi
35 make build
36 make common-test-short
37 mkdir -p $SNAPCRAFT_PART_INSTALL/bin

Subscribers

People subscribed via source and target branches