Bug 681648

Summary: RFE: Update low-latency packaging to provide configuration file
Product: Red Hat Enterprise MRG Reporter: Robert Rati <rrati>
Component: condor-low-latencyAssignee: Robert Rati <rrati>
Status: CLOSED ERRATA QA Contact: Lubos Trilety <ltrilety>
Severity: medium Docs Contact:
Priority: low    
Version: DevelopmentCC: jneedle, ltoscano, ltrilety, matt, mkudlej
Target Milestone: 2.0.1   
Target Release: ---   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: condor-low-latency-1.2-2 Doc Type: Bug Fix
Doc Text:
The condor-low-latency package now provides an example configuration file that can be modified and placed into the directory pointed to by the LOCAL_CONFIG_DIR variable. Using LOCAL_CONFIG_DIR allows for greater flexibility in configuring external features, and configuration of Low-Latency is now easier.
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-09-07 16:44:52 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 723971    
Bug Blocks: 723887    

Description Robert Rati 2011-03-02 20:23:27 UTC
Description of problem:
Update the low-latency packaging to provide a configuration file that is dropped in /etc/condor/config.d.  It should also provide a configuration file for job-hooks specific to low-latency.

It would also be worthwhile to change the subsystem name in the configuration and documentation from LL_DAEMON to CAROD as well.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Matthew Farrellee 2011-03-04 12:59:31 UTC
This will need backward compatibility support for LL_DAEMON in configuration

Comment 2 Robert Rati 2011-07-01 19:23:31 UTC
There is now an included example configuration file, but it needs human interaction before it can be used.

Also, configuration params have changed:
LL_BROKER -> CAROD_BROKER
LL_DAEMON -> CAROD

Older params are still accepted, with the exception of the tests.  Only the CAROD params are accepted in the tests.

Fixed on master

Comment 3 Robert Rati 2011-07-06 14:23:31 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
C: The switch to using LOCAL_CONFIG_DIR allows for ease in configuration of external features
C: There was no example configuration file that could be easily modified and dropped into LOCAL_CONFIG_DIR
C: An example configuration file is included in the condor-low-latency rpm
R: Configuration of Low-Latency is much easier

Comment 4 Luigi Toscano 2011-07-21 12:45:57 UTC
It seems that the old parameters (LL_*) are not working anymore, and this is not expected in a minor update.

== RHEL6.1:
Traceback (most recent call last):
  File "/usr/sbin/carod", line 1101, in <module>
    sys.exit(main())
  File "/usr/sbin/carod", line 959, in main
    base_logger = create_file_logger(log_name, file['log'], logging.INFO, size=size)
  File "/usr/lib/python2.6/site-packages/condorutils/log.py", line 23, in create_file_logger
    backupCount=bcknum)
  File "/usr/lib/python2.6/logging/handlers.py", line 112, in __init__
    BaseRotatingHandler.__init__(self, filename, mode, encoding, delay)
  File "/usr/lib/python2.6/logging/handlers.py", line 64, in __init__
    logging.FileHandler.__init__(self, filename, mode, encoding, delay)
  File "/usr/lib/python2.6/logging/__init__.py", line 827, in __init__
    StreamHandler.__init__(self, self._open())
  File "/usr/lib/python2.6/logging/__init__.py", line 846, in _open
    stream = open(self.baseFilename, self.mode)
IOError: [Errno 21] Is a directory: '/var/log/condor'


== RHEL5.6:
Traceback (most recent call last):
  File "/usr/sbin/carod", line 1101, in ?
    sys.exit(main())
  File "/usr/sbin/carod", line 959, in main
    base_logger = create_file_logger(log_name, file['log'], logging.INFO, size=size)
  File "/usr/lib/python2.4/site-packages/condorutils/log.py", line 23, in create_file_logger
    backupCount=bcknum)
  File "/usr/lib64/python2.4/logging/handlers.py", line 107, in __init__
    BaseRotatingHandler.__init__(self, filename, mode, encoding)
  File "/usr/lib64/python2.4/logging/handlers.py", line 59, in __init__
    logging.FileHandler.__init__(self, filename, mode, encoding)
  File "/usr/lib64/python2.4/logging/__init__.py", line 757, in __init__
    stream = open(filename, mode)
IOError: [Errno 21] Is a directory: '/var/log/condor'

Comment 5 Robert Rati 2011-07-22 21:41:13 UTC
Issue fixed in latest package in FiV

Comment 7 Lubos Trilety 2011-08-02 11:57:36 UTC
There is a new example of configuration file with CAROD_ parameters as part of low latency package. Older params are still accepted.

The change of subsystem name from LL_DAEMON to CAROD in the configuration is covered by bug 718348.

Tested with:
condor-low-latency-1.2-2

Tested on:
RHEL5 x86_64,i386
RHEL6 x86_64,i386

>>> VERIFIED

Comment 8 Douglas Silas 2011-08-09 14:19:16 UTC
    Technical note updated. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    Diffed Contents:
@@ -1,4 +1 @@
-C: The switch to using LOCAL_CONFIG_DIR allows for ease in configuration of external features
+The condor-low-latency package now provides an example configuration file that can be modified and  placed into the directory pointed to by the $LOCAL_CONFIG_DIR environnment variable. Using $LOCAL_CONFIG_DIR allows for greater flexibility in configuring external features, and configuration of Low-Latency is now easier.-C: There was no example configuration file that could be easily modified and dropped into LOCAL_CONFIG_DIR
-C: An example configuration file is included in the condor-low-latency rpm
-R: Configuration of Low-Latency is much easier

Comment 9 Douglas Silas 2011-08-09 14:41:33 UTC
    Technical note updated. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    Diffed Contents:
@@ -1 +1 @@
-The condor-low-latency package now provides an example configuration file that can be modified and  placed into the directory pointed to by the $LOCAL_CONFIG_DIR environnment variable. Using $LOCAL_CONFIG_DIR allows for greater flexibility in configuring external features, and configuration of Low-Latency is now easier.+The condor-low-latency package now provides an example configuration file that can be modified and  placed into the directory pointed to by the LOCAL_CONFIG_DIR variable. Using LOCAL_CONFIG_DIR allows for greater flexibility in configuring external features, and configuration of Low-Latency is now easier.

Comment 10 errata-xmlrpc 2011-09-07 16:44:52 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHSA-2011-1249.html