Just to let you know, I have pushed the suggested changes already.

On 8 February 2014 20:39, Vaidas Jablonskis <jablons...@gmail.com> wrote:

> Yep. I will push my changes tonight. Thanks!
>
>
> On 8 February 2014 20:35, Joshua Harlow <harlo...@gmail.com> wrote:
>
> > Seems ok, one change.
> >
> > resp = url_helper.readurl(url=url_map[mkey], headers=headers)
> >
> > Can u wrap this with a try: except IOError: block.
> >
> > Thanks
> > --
> >
> >
> https://code.launchpad.net/~jablonskis/cloud-init/add-gce-datasource/+merge/204464
> > You are the owner of lp:~jablonskis/cloud-init/add-gce-datasource.
> >
>
>
>
> --
> Vaidas Jablonskis
>
>
> https://code.launchpad.net/~jablonskis/cloud-init/add-gce-datasource/+merge/204464
> You are the owner of lp:~jablonskis/cloud-init/add-gce-datasource.
>



-- 
Vaidas Jablonskis

https://code.launchpad.net/~jablonskis/cloud-init/add-gce-datasource/+merge/204464
Your team cloud init development team is requested to review the proposed merge 
of lp:~jablonskis/cloud-init/add-gce-datasource into lp:cloud-init.

_______________________________________________
Mailing list: https://launchpad.net/~cloud-init-dev
Post to     : cloud-init-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~cloud-init-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to