diff options
author | Michael DeHaan <mdehaan@redhat.com> | 2008-02-13 13:59:43 -0500 |
---|---|---|
committer | Michael DeHaan <mdehaan@redhat.com> | 2008-02-13 13:59:43 -0500 |
commit | 72fc0db631b5d64e8de59ff5478b61316e7e4c43 (patch) | |
tree | 068857943501f0604b373005b73f316c612efa05 | |
parent | fd7f8f2bfa888b749c9746631e55e76ce726e243 (diff) | |
download | third_party-cobbler-72fc0db631b5d64e8de59ff5478b61316e7e4c43.tar.gz third_party-cobbler-72fc0db631b5d64e8de59ff5478b61316e7e4c43.tar.xz third_party-cobbler-72fc0db631b5d64e8de59ff5478b61316e7e4c43.zip |
Nicer error reporting when user doesn't have permissions on the log files
-rw-r--r-- | CHANGELOG | 10 | ||||
-rw-r--r-- | cobbler/api.py | 8 |
2 files changed, 15 insertions, 3 deletions
@@ -1,7 +1,13 @@ Cobbler CHANGELOG (all entries mdehaan@redhat.com unless noted otherwise) -* Thu Jan 31 2007 - 0.7.2 +* XXX Feb XX 2008 - 0.8.0 (TBD) +- stable release of 0.7.* branch plus ... +- fixed potential user problem with source_repos in upgrade scenario +- additional higher level API functions for find, fixes for other higher level API functions +- better messaging when insufficient permissions on needed files + +* Thu Jan 31 2008 - 0.7.2 (0.8 rc) - default_virt_file_size and default_virt_ram added to settings - enforce permissions/selinux context after reposync - better API for copying/renames, API consistancy cleanup @@ -11,7 +17,7 @@ Cobbler CHANGELOG - deletes can now be done recursively (optional --recursive on distro/profile remove) - 'cobbler list' is now (re)sorted -* Wed Jan 09 2007 - 0.7.1 +* Wed Jan 09 2008 - 0.7.1 - allow imports to force usage of a specific kickstart template with --kickstart - added --yumopts parameter to repos (works just like --kopts/--ksmeta) - minor doc fixes diff --git a/cobbler/api.py b/cobbler/api.py index 1a6beb3..13fc5f3 100644 --- a/cobbler/api.py +++ b/cobbler/api.py @@ -22,11 +22,14 @@ import action_import import action_reposync import action_status import action_validate +from cexceptions import * import sub_process import module_loader + import logging import os import fcntl +from rhpl.translate import _, N_, textdomain, utf8 ERROR = 100 INFO = 10 @@ -81,7 +84,10 @@ class BootAPI: def __setup_logger(self,name): logger = logging.getLogger(name) logger.setLevel(logging.INFO) - ch = logging.FileHandler("/var/log/cobbler/cobbler.log") + try: + ch = logging.FileHandler("/var/log/cobbler/cobbler.log") + except: + raise CX(_("No write permissions on log file. Are you root?")) ch.setLevel(logging.INFO) formatter = logging.Formatter("%(asctime)s - %(name)s - %(message)s") ch.setFormatter(formatter) |