Add TODOs

Author: matyasselmeci

osg_configure/configure_modules/gratia.py

@@ -227,6 +227,9 @@ def configure(self, attributes):
             else:
                 self.options['resource'].value = attributes['OSG_SITE_NAME']
 
+        # TODO Why is gratia looking at attributes instead of just using the options
+        # directly?  If it's not necessary, drop the OSG_HOSTNAME attribute from
+        # siteinformation.py.
         if 'OSG_HOSTNAME' not in attributes:
             self.log('Hostname of this machine not specified. Please give this '
                      'in the host_name option in the Site Information section',

osg_configure/configure_modules/siteinformation.py

@@ -33,6 +33,7 @@ def __init__(self, *args, **kwargs):
                             configfile.Option(name='host_name',
                                               required=MANDATORY_ON_CE,
                                               mapping='OSG_HOSTNAME'),
+                        # ^^ the OSG_HOSTNAME mapping is used by the gratia module, though not sure why.
                         'resource':
                             configfile.Option(name='resource',
                                               required=MANDATORY,