← Back to branch summary

~smoser/cloud-init/lp-1077700

« back to all changes in this revision

Viewing changes to cloudinit/config/cc_mounts.py

  • Committer: harlowja
  • Date: 2012-10-28 02:25:48 UTC
  • mto: This revision was merged to the branch mainline in revision 700.
  • Revision ID: harlowja@virtualbox.rhel-20121028022548-2m4y18xiepzn8l2e
Helpful cleanups.

1. Remove the usage of the path.join function
   now that all code should be going through
   the util file methods (and they can be
   mocked out as needed).
2. Adjust all occurences of the above join
   function to either not use it or replace
   it with the standard os.path.join (which
   can also be mocked out as needed)
3. Fix pylint from complaining about the
   tests folder 'helpers.py' not being found
4. Add a pylintrc file that is used instead
   of the options hidden in the 'run_pylint' 
   tool.

Show diffs side-by-side

added added

removed removed

Lines of Context:
cloudinit/config/cc_mounts.py
28
28
SHORTNAME_FILTER = r"^[x]{0,1}[shv]d[a-z][0-9]*$"
29
29
SHORTNAME = re.compile(SHORTNAME_FILTER)
30
30
WS = re.compile("[%s]+" % (whitespace))
 
31
FSTAB_PATH = "/etc/fstab"
31
32
 
32
33
 
33
34
def is_mdname(name):
167
168
        cc_lines.append('\t'.join(line))
168
169
 
169
170
    fstab_lines = []
170
 
    fstab = util.load_file(cloud.paths.join(True, "/etc/fstab"))
171
 
    for line in fstab.splitlines():
 
171
    for line in util.load_file(FSTAB_PATH).splitlines():
172
172
        try:
173
173
            toks = WS.split(line)
174
174
            if toks[3].find(comment) != -1:
179
179
 
180
180
    fstab_lines.extend(cc_lines)
181
181
    contents = "%s\n" % ('\n'.join(fstab_lines))
182
 
    util.write_file(cloud.paths.join(False, "/etc/fstab"), contents)
 
182
    util.write_file(FSTAB_PATH, contents)
183
183
 
184
184
    if needswap:
185
185
        try:
188
188
            util.logexc(log, "Activating swap via 'swapon -a' failed")
189
189
 
190
190
    for d in dirs:
191
 
        real_dir = cloud.paths.join(False, d)
192
191
        try:
193
 
            util.ensure_dir(real_dir)
 
192
            util.ensure_dir(d)
194
193
        except:
195
194
            util.logexc(log, "Failed to make '%s' config-mount", d)
196
195