Code review comment for ~ahasenack/ubuntu/+source/squid:eoan-squid-4.8-merge

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

shouldn't the conditional be true for us?

if os.path.exists(self.aa_abs_profile):

with:
        self.aa_profile = "usr.sbin.squid"
        self.aa_abs_profile = "/etc/apparmor.d/%s" % self.aa_profile

in Container:
root@e:~# apt install squid3
root@e:~# ll /etc/apparmor.d/usr.sbin.squid
-rw-r--r-- 1 root root 2462 Jul 19 12:01 /etc/apparmor.d/usr.sbin.squid

So can't we just keep the conditional and get the tests running without added Delta?

review: Needs Information

« Back to merge proposal