autotest-firewalld-add-service: read firewalld system settings before trying default/fallback settings

Problem: We need to be able to modify firewalld zone on systems that
pre-date the firewall-cmd --permanent option.

In order to do this we must manually read/write the xml in
/etc/firewalld.  The file system layout for firewalld is such that
default/fallback configuration is stored in /usr/lib/firewalld and
system configuration is stored in /etc/firewalld/

To accommodate this default/fallback and system setting behaviour when
add the service to the node file we must first attempt to read existing
settings from /etc/firewalld/, and if not such file exists in
/etc/firewalld, only then will we read the default/fallback settings in
/usr/lib/firewalld.

This patch implements this behavior using a try/open/except function.
We use open() instead of os.access because open is a more exact test of
file readability as os.access does not attempt access with EUID/EGID or
other effective permissions, whereas open() does.

Lightly tested on Fedora 18.

Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
autotest-firewall-add-service: autopep8 and remove unused import

general code cleanups

Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
autotest-firewalld-add-service: remove unused variable

Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
autotest-firewalld-add-service: spelling fix thanks to codespell

Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
autotest_firewalld_add_service_test: add unittest

Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>