This should fix the apparmor dep8 test in debian CI, which runs on LXC configured in a way that loading a profile will fail. Last lines of a run in a debian LXC container on a debian VM as host:
...
Enabling site default-ssl.
To activate the new configuration, you need to run:
systemctl reload apache2
test_CVE_2011_3205 (__main__.BasicTest)
Test parsing lines > 4096 in length (CVE-2011-3205) ... ok
test_daemons (__main__.BasicTest)
Test daemon ... ok
test_ftp_proxy (__main__.BasicTest)
Test ftp ... ok
test_http_proxy (__main__.BasicTest)
Test http ... ok
test_https_proxy (__main__.BasicTest)
Test https ... ok
test_squidclient (__main__.BasicTest)
Test squidclient ... ok
test_zz_apparmor (__main__.BasicTest)
Test apparmor ... ok
----------------------------------------------------------------------
Ran 7 tests in 270.069s
OK
autopkgtest [09:44:35]: test squid: -----------------------]
autopkgtest [09:44:35]: test squid: - - - - - - - - - - results - - - - - - - - - -
squid PASS
autopkgtest [09:44:35]: @@@@@@@@@@@@@@@@@@@@ summary
upstream-test-suite PASS
squid PASSIn that same configuration, the original squid-4.8-1 fails to pass, so I'm confident I replicated the failure that happened in debian CI.
I didn't squash the two commits because I wanted to show the first commit is just a revert of what was added before, and then I added my fix. I think it's more visible that way, but, if this PR is accepted, feel free to squash the commits, or ask me to do the squashing.