~ubuntu-core-dev/ubuntu/xenial/ubuntu-release-upgrader/xenial

« back to all changes in this revision

Viewing changes to DistUpgrade/DistUpgradeAufs.py

  • Committer: Dimitri John Ledkov
  • Date: 2015-01-20 22:30:06 UTC
  • mto: This revision was merged to the branch mainline in revision 2859.
  • Revision ID: dimitri.j.ledkov@intel.com-20150120223006-ba30fbqtp4x2n0og
Fix resource warnings & logging.warn deprecation warning.

Show diffs side-by-side

added added

removed removed

Lines of Context:
117
117
 
118
118
def is_aufs_mount(dir):
119
119
    " test if the given dir is already mounted with aufs overlay "
120
 
    for line in open("/proc/mounts"):
121
 
        (device, mountpoint, fstype, options, a, b) = line.split()
122
 
        if device == "none" and fstype == "aufs" and mountpoint == dir:
123
 
            return True
 
120
    with open("/proc/mounts") as f:
 
121
        for line in f:
 
122
            (device, mountpoint, fstype, options, a, b) = line.split()
 
123
            if device == "none" and fstype == "aufs" and mountpoint == dir:
 
124
                return True
124
125
    return False
125
126
 
126
127
 
182
183
    # create something vaguely rollbackable
183
184
 
184
185
    # get the mount points before the aufs buisiness starts
185
 
    mounts = open("/proc/mounts").read()
 
186
    with open("/proc/mounts") as f:
 
187
        mounts = f.read()
186
188
    from .DistUpgradeMain import SYSTEM_DIRS
187
189
    systemdirs = SYSTEM_DIRS
188
190
    
234
236
    # include sub mounts)
235
237
    needs_bind_mount = set()
236
238
    needs_bind_mount.add("/var/cache/apt/archives")
237
 
    for line in open("/proc/mounts"):
238
 
        (device, mountpoint, fstype, options, a, b) = line.split()
239
 
        if is_real_fs(fstype) and is_submount(mountpoint, systemdirs):
240
 
            logging.warning("mountpoint %s submount of systemdir" % mountpoint)
241
 
            return False
242
 
        if (fstype != "aufs" and
243
 
                not is_real_fs(fstype) and
244
 
                is_submount(mountpoint, systemdirs)):
245
 
            logging.debug("found %s that needs bind mount", mountpoint)
246
 
            needs_bind_mount.add(mountpoint)
 
239
    with open("/proc/mounts") as f:
 
240
        for line in f:
 
241
            (device, mountpoint, fstype, options, a, b) = line.split()
 
242
            if is_real_fs(fstype) and is_submount(mountpoint, systemdirs):
 
243
                logging.warning("mountpoint %s submount of systemdir" % mountpoint)
 
244
                return False
 
245
            if (fstype != "aufs" and
 
246
                    not is_real_fs(fstype) and
 
247
                    is_submount(mountpoint, systemdirs)):
 
248
                logging.debug("found %s that needs bind mount", mountpoint)
 
249
                needs_bind_mount.add(mountpoint)
247
250
 
248
251
    # aufs mounts do not support stacked filesystems, so
249
252
    # if we mount /var we will loose the tmpfs stuff