Re: [openstack-dev] Freezer - tmp_file issue

2016-07-21 Thread Saad Zaher
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

2016-07-21 Thread Mathieu, Pierre-Arthur
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

2016-07-20 Thread Straub, Albert
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