Bug 681648 - RFE: Update low-latency packaging to provide configuration file
Summary: RFE: Update low-latency packaging to provide configuration file
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise MRG
Classification: Red Hat
Component: condor-low-latency
Version: Development
Hardware: All
OS: All
low
medium
Target Milestone: 2.0.1
: ---
Assignee: Robert Rati
QA Contact: Lubos Trilety
URL:
Whiteboard:
Depends On: 723971
Blocks: 723887
TreeView+ depends on / blocked
 
Reported: 2011-03-02 20:23 UTC by Robert Rati
Modified: 2012-02-07 10:07 UTC (History)
5 users (show)

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.
Clone Of:
Environment:
Last Closed: 2011-09-07 16:44:52 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 724907 0 medium CLOSED change low-latency conf. entry names 2021-02-22 00:41:40 UTC
Red Hat Product Errata RHSA-2011:1249 0 normal SHIPPED_LIVE Moderate: Red Hat Enterprise MRG Grid 2.0 security, bug fix and enhancement update 2011-09-07 16:40:45 UTC

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


Note You need to log in before you can comment on or make changes to this bug.