2010/5/26 Balazs Lecz <[email protected]>:
> + def _InstanceChrootQuarantineDir(cls, instance_name):
> + """Returns the name of the KVM chroot quarantine dir of the instance
> +
> + """
> + return utils.PathJoin(cls._CHROOT_QUARANTINE_DIR, "%s_%s"
> + % (instance_name, utils.TimestampForFilename()))
The operator goes to the previous line. Why don't you use
tempfile.mkdtemp? Is there any code avoiding collisions?
tempfile.mkdtemp(dir=cls._CHROOT_QUARANTINE_DIR,
prefix="%s-%s-" % (instance_name, utils.Time…))
Michael
