Merge ~peppepetra/grafana-snap:lp1995279 into grafana-snap:master

Proposed by Giuseppe Petralia
Status: Merged
Approved by: Eric Chen
Approved revision: fa984287cdba5dd7a6a30d106c47d4e744d31752
Merged at revision: fa984287cdba5dd7a6a30d106c47d4e744d31752
Proposed branch: ~peppepetra/grafana-snap:lp1995279
Merge into: grafana-snap:master
Diff against target: 49 lines (+15/-9)
1 file modified
snapcraft.yaml (+15/-9)
Reviewer Review Type Date Requested Status
Eric Chen Needs Information
Erhan Sunar (community) Approve
Chi Wai CHAN Approve
Mert Kirpici (community) Approve
BootStack Reviewers Pending
Review via email: mp+432555@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Mert Kirpici (mertkirpici) wrote :

+1

review: Approve
Revision history for this message
Chi Wai CHAN (raychan96) wrote :

lgtm

review: Approve
Revision history for this message
Eric Chen (eric-chen) :
review: Needs Information
Revision history for this message
Eric Chen (eric-chen) wrote :
review: Needs Information
Revision history for this message
Giuseppe Petralia (peppepetra) wrote :

Reply in line

Revision history for this message
Giuseppe Petralia (peppepetra) wrote :

> Will this patch also solve this?
> https://bugs.launchpad.net/charm-grafana/+bug/1995279

Yes version 7.5.17 address the issue with LP#1995279

Revision history for this message
Erhan Sunar (esunar) wrote :

LGTM

review: Approve
Revision history for this message
Eric Chen (eric-chen) wrote :

Reply inline.

review: Needs Information
Revision history for this message
Mert Kirpici (mertkirpici) :

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 966c991..97d5228 100644
3--- a/snapcraft.yaml
4+++ b/snapcraft.yaml
5@@ -1,5 +1,5 @@
6 name: grafana
7-version: 7.4.1
8+version: 7.5.17
9 summary: feature rich metrics dashboard and graph editor
10 base: core20
11 description: |
12@@ -50,23 +50,29 @@ parts:
13 cp -p -r ${SNAPCRAFT_PART_BUILD}/conf/* ${SNAPCRAFT_PART_INSTALL}/conf/
14 grafana-ui:
15 after: [grafana]
16- plugin: npm
17+ plugin: nil
18 source-type: git
19 source: https://github.com/grafana/grafana.git
20 source-tag: v${SNAPCRAFT_PROJECT_VERSION}
21- npm-node-version: '14.15.5'
22- build-packages: [python2-minimal,python2-dev]
23+ build-packages: [curl,python2-minimal,python2-dev]
24 override-build: |
25 set -x
26- # https://forum.snapcraft.io/t/issue-with-nodejs-plugin-when-using-base-keyword/11109/8
27- echo "unsafe-perm = true" > ~/.npmrc
28- snapcraftctl build
29+ curl -sL https://deb.nodesource.com/setup_14.x | sudo -E bash -
30+ apt install -y nodejs
31+ npm install --global yarn patch-package
32+ # on arm, s390x, and ppc systems remove phantomjs as required from the
33+ # package.json and yarn lock as it's not available from npm on those
34+ # architectures
35+ ARCH=$(uname -m)
36+ case $ARCH in
37+ arm*|s390*|ppc*)
38+ yarn remove phantomjs-prebuilt --ignore-workspace-root-check;;
39+ esac
40 export PYTHON=/usr/bin/python2
41- # XXX check if needed
42 PATH="$PATH:$SNAPCRAFT_PART_BUILD/node_modules/.bin:$SNAPCRAFT_PART_INSTALL/bin"
43 NODE_ENV="production"
44- npm install -g --prefix $SNAPCRAFT_PART_INSTALL yarn
45 yarn install --pure-lockfile --no-progress
46+ npx browserslist@latest --update-db
47 yarn build
48 cp -rf public ${SNAPCRAFT_PART_INSTALL}/public
49 cp -rf tools ${SNAPCRAFT_PART_INSTALL}/tools

Subscribers

People subscribed via source and target branches

to all changes: