Merge ~rafaeldtinoco/ubuntu/+source/cacti:lp1863739-focal into ubuntu/+source/cacti:debian/sid

Proposed by Rafael David Tinoco
Status: Merged
Merge reported by: Rafael David Tinoco
Merged at revision: 17e9b5454a6f074f486e837f46160f5e7ba55258
Proposed branch: ~rafaeldtinoco/ubuntu/+source/cacti:lp1863739-focal
Merge into: ubuntu/+source/cacti:debian/sid
Diff against target: 160 lines (+94/-4)
6 files modified
debian/README.Debian (+8/-3)
debian/cacti.config (+1/-0)
debian/changelog (+38/-0)
debian/control (+2/-1)
debian/patches/general-instructions-no-auto-create-user.patch (+44/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
Graham Inggs Pending
Canonical Server MOTU reviewers Pending
Canonical Server Core Reviewers Pending
Canonical Server Pending
Review via email: mp+379397@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Rafael David Tinoco (rafaeldtinoco) wrote :

I got this reviewed by @ginggs and comments were given in the LP bug:

"""
I think the only change needed is to mention in the changelog that the 'Replace php-php-gettext dependency' change was dropped.
"""

Revision history for this message
Rafael David Tinoco (rafaeldtinoco) wrote :

$ git push -f rafaeldtinoco lp1863739-focal
Enumerating objects: 12, done.
Counting objects: 100% (12/12), done.
Delta compression using up to 8 threads
Compressing objects: 100% (8/8), done.
Writing objects: 100% (8/8), 1.03 KiB | 211.00 KiB/s, done.
Total 8 (delta 6), reused 0 (delta 0)
To ssh://git.launchpad.net/~rafaeldtinoco/ubuntu/+source/cacti
 + 14a2ccf...17e9b54 lp1863739-focal -> lp1863739-focal (forced update)

$ dput ubuntu cacti_1.2.9+ds1-1ubuntu1_source.changes
Checking signature on .changes
gpg: /home/rafaeldtinoco/work/sources/ubuntu/cacti_1.2.9+ds1-1ubuntu1_source.changes: Valid signature from A93E0E0AD83C0D0F
Checking signature on .dsc
gpg: /home/rafaeldtinoco/work/sources/ubuntu/cacti_1.2.9+ds1-1ubuntu1.dsc: Valid signature from A93E0E0AD83C0D0F
Package includes an .orig.tar.gz file although the debian revision suggests
that it might not be required. Multiple uploads of the .orig.tar.gz may be
rejected by the upload queue management software.
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading cacti_1.2.9+ds1-1ubuntu1.dsc: done.
  Uploading cacti_1.2.9+ds1.orig-docs-source.tar.gz: done.
  Uploading cacti_1.2.9+ds1.orig.tar.gz: done.
  Uploading cacti_1.2.9+ds1-1ubuntu1.debian.tar.xz: done.
  Uploading cacti_1.2.9+ds1-1ubuntu1_source.buildinfo: done.
  Uploading cacti_1.2.9+ds1-1ubuntu1_source.changes: done.
Successfully uploaded packages.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/README.Debian b/debian/README.Debian
index 13220d4..ce92a33 100644
--- a/debian/README.Debian
+++ b/debian/README.Debian
@@ -11,9 +11,14 @@ cacti for Debian
11 and load up the cacti skeleton. Assuming you chose a database and11 and load up the cacti skeleton. Assuming you chose a database and
12 user both named 'cacti' and your database is on localhost:12 user both named 'cacti' and your database is on localhost:
1313
14 mysql -u root -p -e "create database cacti"14 mysql -u root (-p) -e "create database cacti"
15 mysql -u root -p -e "grant all privileges on cacti.* to cacti@localhost identified by 'yourpasswordhere'; flush privileges"15 mysql -u root (-p) -e "create user 'cacti'@'localhost';"
16 cat /usr/share/doc/cacti/cacti.sql | mysql -u cacti -p cacti 16 mysql -u root (-p) -e "alter user 'cacti'@'localhost' identified with 'sha256_password';
17 mysql -u root (-p) -e "alter user 'cacti'@'localhost' identified by 'yourpasswordhere';"
18 mysql -u root (-p) -e "grant all privileges on cacti.* to 'cacti'@'localhost';"
19 mysql -u root (-p) -e "flush privileges;"
20
21 cat /usr/share/doc/cacti/cacti.sql | mysql -u cacti -p cacti
1722
18 Next, go to http://$yourhost/cacti/, and follow the on-screen directions.23 Next, go to http://$yourhost/cacti/, and follow the on-screen directions.
19 The default login/password is admin/admin.24 The default login/password is admin/admin.
diff --git a/debian/cacti.config b/debian/cacti.config
index 3d5b0a2..a1ffaa2 100644
--- a/debian/cacti.config
+++ b/debian/cacti.config
@@ -13,6 +13,7 @@ if [ -f /usr/share/dbconfig-common/dpkg/config.mysql ]; then
13 dbc_first_version="0.8.6g-3"13 dbc_first_version="0.8.6g-3"
14 dbc_load_include="php:/etc/cacti/debian.php"14 dbc_load_include="php:/etc/cacti/debian.php"
15 dbc_load_include_args="--dbname=database_default --dbpass=database_password --dbuser=database_username --dbserver=database_hostname --dbport=database_port"15 dbc_load_include_args="--dbname=database_default --dbpass=database_password --dbuser=database_username --dbserver=database_hostname --dbport=database_port"
16 dbc_authplugin="sha256_password"
16 dbc_go cacti $@17 dbc_go cacti $@
17fi18fi
1819
diff --git a/debian/changelog b/debian/changelog
index b8e4887..81e2743 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,16 @@
1cacti (1.2.9+ds1-1ubuntu1) focal; urgency=medium
2
3 * Merge with Debian unstable (LP: #1863739). Remaining changes:
4 - General installing instructions update for NO_AUTO_CREATE_USER.
5 - Use new dbconfig "dbc_authplugin" variable to mitigate MySQL 8 issues.
6 * Dropped changes [upstream]:
7 - MySQL 8 change needs: NO_AUTO_CREATE_USER and grouping keyword.
8 * Dropped changes [debian]:
9 - Replace php-php-gettext dependency in order to fix translations
10 (LP #1844070)
11
12 -- Rafael David Tinoco <rafaeldtinoco@ubuntu.com> Tue, 18 Feb 2020 13:28:26 +0000
13
1cacti (1.2.9+ds1-1) unstable; urgency=medium14cacti (1.2.9+ds1-1) unstable; urgency=medium
215
3 * New upstream version 1.2.9+ds116 * New upstream version 1.2.9+ds1
@@ -60,6 +73,31 @@ cacti (1.2.6+ds1-1) unstable; urgency=medium
6073
61 -- Paul Gevers <elbrus@debian.org> Thu, 05 Sep 2019 17:47:08 +020074 -- Paul Gevers <elbrus@debian.org> Thu, 05 Sep 2019 17:47:08 +0200
6275
76cacti (1.2.4+ds1-2ubuntu3) eoan; urgency=medium
77
78 * Replace php-php-gettext dependency in order to fix translations
79 (LP: #1844070)
80
81 -- Graham Inggs <ginggs@ubuntu.com> Thu, 19 Sep 2019 10:30:52 +0000
82
83cacti (1.2.4+ds1-2ubuntu2) eoan; urgency=medium
84
85 * Fix CREATE FUNCTION errors if binary logging is enabled:
86 - d/p/MySQL-8-change-needs-NO_AUTO_CREATE_USER_and_groupin.patch
87 - d/cacti.admin.sql
88 - d/cacti.admin.rm.sql
89
90 -- Rafael David Tinoco <rafaeldtinoco@ubuntu.com> Fri, 09 Aug 2019 06:10:31 +0000
91
92cacti (1.2.4+ds1-2ubuntu1) eoan; urgency=medium
93
94 * Fixes for MySQL upstream updates. (Closes: #933683)
95 - General installing instructions update for NO_AUTO_CREATE_USER.
96 - Use new dbconfig "dbc_authplugin" variable to mitigate MySQL 8 issues.
97 - Debian Instructions have to contain workaround for NO_AUTO_CREATE_USER.
98
99 -- Rafael David Tinoco <rafaeldtinoco@ubuntu.com> Thu, 08 Aug 2019 09:42:54 -0300
100
63cacti (1.2.4+ds1-2) unstable; urgency=medium101cacti (1.2.4+ds1-2) unstable; urgency=medium
64102
65 * tests: add new IMPORT messages to ignore filter103 * tests: add new IMPORT messages to ignore filter
diff --git a/debian/control b/debian/control
index a60b9ea..3433974 100644
--- a/debian/control
+++ b/debian/control
@@ -1,7 +1,8 @@
1Source: cacti1Source: cacti
2Section: web2Section: web
3Priority: optional3Priority: optional
4Maintainer: Cacti Maintainer <pkg-cacti-maint@lists.alioth.debian.org>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5XSBC-Original-Maintainer: Cacti Maintainer <pkg-cacti-maint@lists.alioth.debian.org>
5Uploaders: Paul Gevers <elbrus@debian.org>6Uploaders: Paul Gevers <elbrus@debian.org>
6Build-Depends: debhelper (>= 11~),7Build-Depends: debhelper (>= 11~),
7 dh-linktree (>=0.5~),8 dh-linktree (>=0.5~),
diff --git a/debian/patches/general-instructions-no-auto-create-user.patch b/debian/patches/general-instructions-no-auto-create-user.patch
8new file mode 1006449new file mode 100644
index 0000000..6f75dc4
--- /dev/null
+++ b/debian/patches/general-instructions-no-auto-create-user.patch
@@ -0,0 +1,44 @@
1From cbb63ddf62e01dfb8f9df81de8f087764bceecc3 Mon Sep 17 00:00:00 2001
2From: Rafael David Tinoco <rafaeldtinoco@ubuntu.com>
3Date: Fri, 9 Aug 2019 16:25:49 +0000
4Subject: [PATCH] General installing instructions update for
5 NO_AUTO_CREATE_USER
6
7After MySQL 8 started using NO_AUTO_CREATE_USER by default, the general
8installing instructions need update to instruct one on how to proper
9create the user separately from the ALTER SQL command given by example.
10
11Signed-off-by: Rafael David Tinoco <rafaeldtinoco@ubuntu.com>
12
13Author: Rafael David Tinoco <rafaeldtinoco@ubuntu.com>
14Forwarded: https://salsa.debian.org/cacti-team/cacti/merge_requests/1
15Bug: https://github.com/Cacti/cacti/issues/2872
16Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=933683
17---
18 docs-source/General-Installing-Instructions.md | 10 +++++++---
19 1 file changed, 7 insertions(+), 3 deletions(-)
20
21diff --git a/docs-source/General-Installing-Instructions.md b/docs-source/General-Installing-Instructions.md
22index bc2c765..4d5ffcd 100644
23--- a/docs-source/General-Installing-Instructions.md
24+++ b/docs-source/General-Installing-Instructions.md
25@@ -250,9 +250,13 @@ back into rotation.
26
27 ```sql
28 shell> mysql --user=root mysql
29- MySQL> GRANT ALL ON cacti.* TO cactiuser@localhost IDENTIFIED BY 'somepassword';
30- MySQL> GRANT SELECT ON mysql.time_zone_name TO cactiuser@localhost IDENTIFIED BY 'somepassword';
31- MySQL> flush privileges;
32+ mysql> CREATE DATABASE cacti
33+ mysql> CREATE USER 'cacti'@'localhost';
34+ mysql> ALTER USER 'cacti'@'localhost' IDENTIFIED WITH 'sha256_password';
35+ mysql> ALTER USER 'cacti'@'localhost' IDENTIFIED BY 'somepassword';
36+ mysql> GRANT ALL PRIVILEGES ON cacti.* to 'cacti'@'localhost';
37+ mysql> GRANT SELECT ON mysql.time_zone_name TO 'cacti'@'localhost';
38+ mysql> FLUSH PRIVILEGES;
39 ```
40
41 Note that if your `root` (or equivalent) user does not have `SUPER` permissions,
42--
432.20.1
44
diff --git a/debian/patches/series b/debian/patches/series
index 9635522..75b7da0 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -2,3 +2,4 @@
2enable-system-jqueryui-by-putting-cacti-changes-in-main.css.patch2enable-system-jqueryui-by-putting-cacti-changes-in-main.css.patch
3perl-path.patch3perl-path.patch
4font-awesome-path.patch4font-awesome-path.patch
5general-instructions-no-auto-create-user.patch

Subscribers

People subscribed via source and target branches