py tests depend on /etc/apparmor/logprof.conf

Bug #1393979 reported by Christian Boltz
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
AppArmor
Fix Released
Undecided
Christian Boltz

Bug Description

The python tests (utils/test) depend on /etc/apparmor/logprof.conf:

# mv /etc/apparmor/logprof.conf{,_AWAY}
# cd utils
# make check
[...]
make[1]: Entering directory '/home/cb/apparmor/HEAD/utils/test'
Traceback (most recent call last):
  File "test-ptrace_parse.py", line 12, in <module>
    import apparmor.aa as aa
  File "/home/cb/apparmor/HEAD/utils/apparmor/aa.py", line 4658, in <module>
    if cfg['settings'].get('default_owner_prompt', False):
  File "/usr/lib64/python3.4/configparser.py", line 937, in __getitem__
    raise KeyError(key)
KeyError: 'settings'
Makefile:43: recipe for target 'check' failed
make[1]: *** [check] Error 1
make[1]: Leaving directory '/home/cb/apparmor/HEAD/utils/test'
Makefile:98: recipe for target 'check' failed
make: *** [check] Error 2

Tags: aa-tools

Related branches

Revision history for this message
Christian Boltz (cboltz) wrote :

Patch sent to the ML for review.

Changed in apparmor:
assignee: nobody → Christian Boltz (cboltz)
milestone: none → 2.10.1
status: New → In Progress
Revision history for this message
Christian Boltz (cboltz) wrote :

Fix commited to bzr r3263.

I wouldn't be surprised if we notice more dependencies on logprof.conf when adding more test coverage, but we can handle that whenever we notice it.

Changed in apparmor:
status: In Progress → Fix Committed
Changed in apparmor:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.