Re: [openstack-dev] Freezer - tmp_file issue
Hi Albert, Could you please share with us your config file, log file and the exact error message that you see ? BR, Saad! On Thu, Jul 21, 2016 at 11:45 AM, Mathieu, Pierre-Arthur < pierre-arthur.math...@hpe.com> wrote: > Hi Albert, > > > The mailing list is one place where you can report this kind of things. > > You can also just pop up in the IRC room related to the corresponding > project (#openstack-freezer in our case), this is usualy the fastest way. > > You can also create a bug in lauchpad (I did that for you: > https://bugs.launchpad.net/freezer/+bug/1605178). > > Someone is going to patch this shortly. > > > Cheers, > > - Pierre > > > PS: When interacting with OpenStack mailing lists, you need to add start > the subject of the email by [mailing-list-name][project-name] > ([openstack-dev][freezer] in this case) so that readers can sort through > the big number of emails easily. > > > > > From: Straub, Albert <albert.str...@charter.com> > Sent: Wednesday, July 20, 2016 10:12:08 PM > To: openstack-dev@lists.openstack.org > Subject: [openstack-dev] Freezer - tmp_file issue > > Hi! > > I’m rather new at this so let me know if this is the wrong place to ask > for a change. I was trying to run the Freezer code without the trickle > executable but with a configuration file. It wouldn’t run. So, I took a > look and it appears that if you do not have a trickle executable but you do > have a config file that it will add a tmp_file key to the backup_args > dictionary. However, if you don’t have a trickle executable, it will try > to pop out tmp_file which doesn’t exist and thus an exception is thrown and > the program exits. Would it be possible to have someone move the if > backup_args.config: \ backup_args.__dict__[‘tmp_file’] = conf_file.name > above and on the same indent as the if trickle_executable and then move the > part in the else statement if backup_args.config to the same indent level > as well? That should prevent it from having a happy heart attack and > exiting. > > Thanks, > > Al > > > if trickle_executable: > LOG.info("Info: Starting trickle ...") > trickle_command = '{0} -d {1} -u {2} '.\ > format(trickle_executable, >getattr(backup_args, 'download_limit') or -1, >getattr(backup_args, 'upload_limit') or -1) > backup_args.__dict__['trickle_command'] = trickle_command > if backup_args.config: > backup_args.__dict__['tmp_file'] = conf_file.name > > # maintain env variable not to get into infinite loop > if "tricklecount" in os.environ: > tricklecount = int(os.environ.get("tricklecount", 1)) > tricklecount += 1 > os.environ["tricklecount"] = str(tricklecount) > > else: > os.environ["tricklecount"] = str(1) > else: > LOG.warn("Trickle not found. Switching to normal mode without " > "limiting bandwidth") > if backup_args.config: > # remove index tmp_file from backup arguments dict > backup_args.__dict__.pop('tmp_file') > utils.delete_file(conf_file.name) > > > __ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > > __ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > -- -- Best Regards, Saad! __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] Freezer - tmp_file issue
Hi Albert, The mailing list is one place where you can report this kind of things. You can also just pop up in the IRC room related to the corresponding project (#openstack-freezer in our case), this is usualy the fastest way. You can also create a bug in lauchpad (I did that for you: https://bugs.launchpad.net/freezer/+bug/1605178). Someone is going to patch this shortly. Cheers, - Pierre PS: When interacting with OpenStack mailing lists, you need to add start the subject of the email by [mailing-list-name][project-name] ([openstack-dev][freezer] in this case) so that readers can sort through the big number of emails easily. From: Straub, Albert <albert.str...@charter.com> Sent: Wednesday, July 20, 2016 10:12:08 PM To: openstack-dev@lists.openstack.org Subject: [openstack-dev] Freezer - tmp_file issue Hi! I’m rather new at this so let me know if this is the wrong place to ask for a change. I was trying to run the Freezer code without the trickle executable but with a configuration file. It wouldn’t run. So, I took a look and it appears that if you do not have a trickle executable but you do have a config file that it will add a tmp_file key to the backup_args dictionary. However, if you don’t have a trickle executable, it will try to pop out tmp_file which doesn’t exist and thus an exception is thrown and the program exits. Would it be possible to have someone move the if backup_args.config: \ backup_args.__dict__[‘tmp_file’] = conf_file.name above and on the same indent as the if trickle_executable and then move the part in the else statement if backup_args.config to the same indent level as well? That should prevent it from having a happy heart attack and exiting. Thanks, Al if trickle_executable: LOG.info("Info: Starting trickle ...") trickle_command = '{0} -d {1} -u {2} '.\ format(trickle_executable, getattr(backup_args, 'download_limit') or -1, getattr(backup_args, 'upload_limit') or -1) backup_args.__dict__['trickle_command'] = trickle_command if backup_args.config: backup_args.__dict__['tmp_file'] = conf_file.name # maintain env variable not to get into infinite loop if "tricklecount" in os.environ: tricklecount = int(os.environ.get("tricklecount", 1)) tricklecount += 1 os.environ["tricklecount"] = str(tricklecount) else: os.environ["tricklecount"] = str(1) else: LOG.warn("Trickle not found. Switching to normal mode without " "limiting bandwidth") if backup_args.config: # remove index tmp_file from backup arguments dict backup_args.__dict__.pop('tmp_file') utils.delete_file(conf_file.name) __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
[openstack-dev] Freezer - tmp_file issue
Hi! I’m rather new at this so let me know if this is the wrong place to ask for a change. I was trying to run the Freezer code without the trickle executable but with a configuration file. It wouldn’t run. So, I took a look and it appears that if you do not have a trickle executable but you do have a config file that it will add a tmp_file key to the backup_args dictionary. However, if you don’t have a trickle executable, it will try to pop out tmp_file which doesn’t exist and thus an exception is thrown and the program exits. Would it be possible to have someone move the if backup_args.config: \ backup_args.__dict__[‘tmp_file’] = conf_file.name above and on the same indent as the if trickle_executable and then move the part in the else statement if backup_args.config to the same indent level as well? That should prevent it from having a happy heart attack and exiting. Thanks, Al if trickle_executable: LOG.info("Info: Starting trickle ...") trickle_command = '{0} -d {1} -u {2} '.\ format(trickle_executable, getattr(backup_args, 'download_limit') or -1, getattr(backup_args, 'upload_limit') or -1) backup_args.__dict__['trickle_command'] = trickle_command if backup_args.config: backup_args.__dict__['tmp_file'] = conf_file.name # maintain env variable not to get into infinite loop if "tricklecount" in os.environ: tricklecount = int(os.environ.get("tricklecount", 1)) tricklecount += 1 os.environ["tricklecount"] = str(tricklecount) else: os.environ["tricklecount"] = str(1) else: LOG.warn("Trickle not found. Switching to normal mode without " "limiting bandwidth") if backup_args.config: # remove index tmp_file from backup arguments dict backup_args.__dict__.pop('tmp_file') utils.delete_file(conf_file.name) __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev