Code review comment for ~ahasenack/ubuntu/+source/volatildap:focal-volatildap-apparmor-dep8

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Because we are writing to /etc and changing the environment outside of packaging. From the spec at https://people.debian.org/~mpitt/autopkgtest/README.package-tests.html:
"""
    The test, when run, is liable to break the testbed system. This includes causing data loss, causing services that the machine is running to malfunction, or permanently disabling services; it does not include causing services on the machine to temporarily fail.

    When this restriction is present the test will usually be skipped unless the testbed's virtualisation arrangements are sufficiently powerful, or alternatively if the user explicitly requests.
"""

An alternative would be a shell trap in the test to remove that file when it exits or fails. Would you rather I go that route?

« Back to merge proposal