Missing dep8 tests
Bug #1906333 reported by
Sergio Durigan Junior
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
rsyslog (Ubuntu) |
Fix Released
|
Wishlist
|
Andreas Hasenack |
Bug Description
rsyslog is missing dep8 tests, which would be really good to have given the importance of this package and the amount of delta we're currently carrying.
Related branches
~xypron/ubuntu/+source/rsyslog:merge-request
Superseded
for merging
into
ubuntu/+source/rsyslog:ubuntu/devel
- git-ubuntu import: Pending requested
-
Diff: 215 lines (+174/-5) (has conflicts)5 files modifieddebian/changelog (+96/-0)
debian/rsyslog.logcheck.ignore.server (+4/-5)
debian/salsa-ci.yml (+3/-0)
debian/tests/control (+7/-0)
debian/tests/logcheck (+64/-0)
~enr0n/ubuntu/+source/rsyslog:merge-mantic-lp2025678
Ready for review
for merging
into
ubuntu/+source/rsyslog:debian/sid
- Simon Chopin (community): Approve
-
Diff: 2856 lines (+2335/-31)33 files modifieddebian/00rsyslog.conf (+12/-0)
debian/50-default.conf (+48/-0)
debian/NEWS (+30/-0)
debian/README.apparmor (+132/-0)
debian/README.apparmor.rsyslog.d (+16/-0)
debian/apparmor/rsyslog-gnutls.apparmor (+3/-0)
debian/apparmor/rsyslog-mysql.apparmor (+20/-0)
debian/apparmor/rsyslog-openssl.apparmor (+3/-0)
debian/apparmor/rsyslog-pgsql.apparmor (+9/-0)
debian/changelog (+1627/-0)
debian/clean (+6/-0)
debian/control (+7/-2)
debian/dmesg.service (+13/-0)
debian/reload-apparmor-profile (+14/-0)
debian/rsyslog-gnutls.install (+1/-0)
debian/rsyslog-mysql.install (+1/-0)
debian/rsyslog-openssl.install (+1/-0)
debian/rsyslog-pgsql.install (+1/-0)
debian/rsyslog.conf (+10/-26)
debian/rsyslog.dirs (+1/-0)
debian/rsyslog.docs (+1/-0)
debian/rsyslog.install (+6/-1)
debian/rsyslog.postinst (+29/-0)
debian/rsyslog.postrm (+13/-0)
debian/rsyslog.service (+2/-0)
debian/rules (+8/-2)
debian/tests/apparmor-include-mechanism (+92/-0)
debian/tests/control (+19/-0)
debian/tests/simple-logger (+25/-0)
debian/tests/simple-mysql (+29/-0)
debian/tests/simple-pgsql (+25/-0)
debian/tests/utils (+76/-0)
debian/usr.sbin.rsyslogd (+55/-0)
~ahasenack/ubuntu/+source/rsyslog:lunar-rsyslog-merge
- Athos Ribeiro (community): Approve
- git-ubuntu bot: Approve
- Canonical Server Core Reviewers: Pending requested
- Canonical Server Reporter: Pending requested
-
Diff: 2762 lines (+2248/-31)33 files modifieddebian/00rsyslog.conf (+12/-0)
debian/50-default.conf (+48/-0)
debian/NEWS (+30/-0)
debian/README.apparmor (+132/-0)
debian/README.apparmor.rsyslog.d (+16/-0)
debian/apparmor/rsyslog-gnutls.apparmor (+3/-0)
debian/apparmor/rsyslog-mysql.apparmor (+20/-0)
debian/apparmor/rsyslog-openssl.apparmor (+3/-0)
debian/apparmor/rsyslog-pgsql.apparmor (+9/-0)
debian/changelog (+1540/-0)
debian/clean (+6/-0)
debian/control (+7/-2)
debian/dmesg.service (+13/-0)
debian/reload-apparmor-profile (+14/-0)
debian/rsyslog-gnutls.install (+1/-0)
debian/rsyslog-mysql.install (+1/-0)
debian/rsyslog-openssl.install (+1/-0)
debian/rsyslog-pgsql.install (+1/-0)
debian/rsyslog.conf (+10/-26)
debian/rsyslog.dirs (+1/-0)
debian/rsyslog.docs (+1/-0)
debian/rsyslog.install (+6/-1)
debian/rsyslog.postinst (+29/-0)
debian/rsyslog.postrm (+13/-0)
debian/rsyslog.service (+2/-0)
debian/rules (+8/-2)
debian/tests/apparmor-include-mechanism (+92/-0)
debian/tests/control (+19/-0)
debian/tests/simple-logger (+36/-0)
debian/tests/simple-mysql (+42/-0)
debian/tests/simple-pgsql (+38/-0)
debian/tests/utils (+41/-0)
debian/usr.sbin.rsyslogd (+53/-0)
~ahasenack/ubuntu/+source/rsyslog:lunar-rsyslog-enable-apparmor-dep8-take4-dot-d
Merged
into
ubuntu/+source/rsyslog:ubuntu/devel
at
revision 76271daab8cbf6a7bf649bd3e5a3c2e15139d8c1
- git-ubuntu bot: Approve
- Athos Ribeiro (community): Approve
- Simon Déziel (community): Approve
- Canonical Server Core Reviewers: Pending requested
- Canonical Server Reporter: Pending requested
-
Diff: 774 lines (+556/-24)26 files modifieddebian/NEWS (+30/-0)
debian/README.apparmor (+132/-0)
debian/README.apparmor.rsyslog.d (+16/-0)
debian/apparmor/rsyslog-gnutls.apparmor (+3/-0)
debian/apparmor/rsyslog-mysql.apparmor (+20/-0)
debian/apparmor/rsyslog-openssl.apparmor (+3/-0)
debian/apparmor/rsyslog-pgsql.apparmor (+9/-0)
debian/changelog (+42/-0)
debian/reload-apparmor-profile (+14/-0)
debian/rsyslog-gnutls.install (+1/-0)
debian/rsyslog-mysql.install (+1/-0)
debian/rsyslog-openssl.install (+1/-0)
debian/rsyslog-pgsql.install (+1/-0)
debian/rsyslog.dirs (+1/-1)
debian/rsyslog.docs (+1/-0)
debian/rsyslog.install (+2/-0)
debian/rsyslog.postinst (+7/-0)
debian/rsyslog.preinst (+0/-15)
debian/rsyslog.service (+2/-0)
debian/tests/apparmor-include-mechanism (+92/-0)
debian/tests/control (+19/-0)
debian/tests/simple-logger (+36/-0)
debian/tests/simple-mysql (+42/-0)
debian/tests/simple-pgsql (+38/-0)
debian/tests/utils (+41/-0)
debian/usr.sbin.rsyslogd (+2/-8)
Changed in rsyslog (Ubuntu): | |
assignee: | nobody → Andreas Hasenack (ahasenack) |
status: | New → In Progress |
To post a comment you must log in.
This bug was fixed in the package rsyslog - 8.2210.0-3ubuntu2
---------------
rsyslog (8.2210.0-3ubuntu2) lunar; urgency=medium
* Support apparmor profile snippets: rsyslogd: add "include if exists" for the rsyslog.d d/rsyslog. d/ rsyslog- mysql,rsyslog- mysql.install} : add apparmor rsyslog- pgsql,rsyslog- pgsql.install} : add apparmor rsyslog- gnutls. apparmor, rsyslog- gnutls. install} : add rsyslog- openssl. apparmor, rsyslog- gnutls. install} : add apparmor- profile apparmor- profile: script to reload the apparmor. rsyslog. d, d/rsyslog.install: install a specific include- mechanism: DEP8 test for the rsyslog.d
- d/usr.sbin.
directory, and remove the now unnecessary mysql and postgresql
sections
- d/rsyslog.preinst: don't disable the apparmor profile on install
- d/rsyslog.postinst: remove disabling of apparmor on upgrades if we
are upgrading from a version older than $now.
- d/rsyslog.dirs: install /etc/apparmor.
- d/{apparmor/
profile for mysql plugin
- d/{apparmor/
profile for postgresql plugin
- d/{apparmor/
apparmor profile for the gnutls plugin
- d/{apparmor/
apparmor profile for the openssl plugin
- New script to reload apparmor profile:
+ d/rsyslog.service: reload apparmor profile in ExecStartPre and
set StandardError to journal so we can see errors from the
script
+ d/rsyslog.install: install reload-
+ d/reload-
rsyslogd apparmor profile
- d/NEWS: add info about apparmor changes in the Ubuntu packaging
- d/rsyslog.docs, d/README.apparmor: explains how the dynamic
component of the rsyslog apparmor profile is applied
- d/README.
README file in the apparmor include directory for rsyslog
* Add DEP8 tests (LP: #1906333):
- d/t/control, d/t/simple-logger: simple logger test
- d/t/utils: common function(s)
- d/t/control, d/t/simple-mysql: DEP8 test using rsyslog with a
MySQL server
- d/t/control, d/t/simple-pgsql: DEP8 test using rsyslog with a
PostgreSQL server
- d/t/apparmor-
include mechanism used by the rsyslog apparmor profile
-- Andreas Hasenack <email address hidden> Fri, 17 Feb 2023 14:22:27 -0300