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
diff --git a/snapcraft.yaml b/snapcraft.yaml
index 966c991..97d5228 100644
--- a/snapcraft.yaml
+++ b/snapcraft.yaml
@@ -1,5 +1,5 @@
1name: grafana1name: grafana
2version: 7.4.12version: 7.5.17
3summary: feature rich metrics dashboard and graph editor3summary: feature rich metrics dashboard and graph editor
4base: core204base: core20
5description: |5description: |
@@ -50,23 +50,29 @@ parts:
50 cp -p -r ${SNAPCRAFT_PART_BUILD}/conf/* ${SNAPCRAFT_PART_INSTALL}/conf/50 cp -p -r ${SNAPCRAFT_PART_BUILD}/conf/* ${SNAPCRAFT_PART_INSTALL}/conf/
51 grafana-ui:51 grafana-ui:
52 after: [grafana]52 after: [grafana]
53 plugin: npm53 plugin: nil
54 source-type: git54 source-type: git
55 source: https://github.com/grafana/grafana.git55 source: https://github.com/grafana/grafana.git
56 source-tag: v${SNAPCRAFT_PROJECT_VERSION}56 source-tag: v${SNAPCRAFT_PROJECT_VERSION}
57 npm-node-version: '14.15.5'57 build-packages: [curl,python2-minimal,python2-dev]
58 build-packages: [python2-minimal,python2-dev]
59 override-build: |58 override-build: |
60 set -x59 set -x
61 # https://forum.snapcraft.io/t/issue-with-nodejs-plugin-when-using-base-keyword/11109/860 curl -sL https://deb.nodesource.com/setup_14.x | sudo -E bash -
62 echo "unsafe-perm = true" > ~/.npmrc61 apt install -y nodejs
63 snapcraftctl build62 npm install --global yarn patch-package
63 # on arm, s390x, and ppc systems remove phantomjs as required from the
64 # package.json and yarn lock as it's not available from npm on those
65 # architectures
66 ARCH=$(uname -m)
67 case $ARCH in
68 arm*|s390*|ppc*)
69 yarn remove phantomjs-prebuilt --ignore-workspace-root-check;;
70 esac
64 export PYTHON=/usr/bin/python271 export PYTHON=/usr/bin/python2
65 # XXX check if needed
66 PATH="$PATH:$SNAPCRAFT_PART_BUILD/node_modules/.bin:$SNAPCRAFT_PART_INSTALL/bin"72 PATH="$PATH:$SNAPCRAFT_PART_BUILD/node_modules/.bin:$SNAPCRAFT_PART_INSTALL/bin"
67 NODE_ENV="production"73 NODE_ENV="production"
68 npm install -g --prefix $SNAPCRAFT_PART_INSTALL yarn
69 yarn install --pure-lockfile --no-progress74 yarn install --pure-lockfile --no-progress
75 npx browserslist@latest --update-db
70 yarn build76 yarn build
71 cp -rf public ${SNAPCRAFT_PART_INSTALL}/public77 cp -rf public ${SNAPCRAFT_PART_INSTALL}/public
72 cp -rf tools ${SNAPCRAFT_PART_INSTALL}/tools78 cp -rf tools ${SNAPCRAFT_PART_INSTALL}/tools

Subscribers

People subscribed via source and target branches

to all changes: