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

Bug #1393979 reported by Christian Boltz on 2014-11-18
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
AppArmor
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

Related branches

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
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  Edit
Everyone can see this information.

Other bug subscribers