*thought
--
https://code.launchpad.net/~harlowja/cloud-init/+git/cloud-init/+merge/305882
Your team cloud init development team is requested to review the proposed merge
of ~harlowja/cloud-init:sys-io-errors into cloud-init:master.
___
Mailing list: ht
Will double check, though I fixed that recently.
--
https://code.launchpad.net/~harlowja/cloud-init/+git/cloud-init/+merge/305882
Your team cloud init development team is requested to review the proposed merge
of ~harlowja/cloud-init:sys-io-errors into cloud-init:master.
Josh,
can you take another look here?
--
https://code.launchpad.net/~harlowja/cloud-init/+git/cloud-init/+merge/305882
Your team cloud init development team is requested to review the proposed merge
of ~harlowja/cloud-init:sys-io-errors into cloud-init:master.
__
it looks like your test case is busted, flake8 fails which makes me wonder if
its working correctly.
tests/unittests/test_net.py:478:36: F821 undefined name 'name'
tests/unittests/test_net.py:478:42: F821 undefined name 'field'
that and a few other changes:
http://paste.ubuntu.com/23354704
Data point: We tried this patch out locally under RHEL7 and it's working like
a champ.
--
https://code.launchpad.net/~harlowja/cloud-init/+git/cloud-init/+merge/305882
Your team cloud init development team is requested to review the proposed merge
of ~harlowja/cloud-init:sys-io-errors into clou
Oh, lol.
--
https://code.launchpad.net/~harlowja/cloud-init/+git/cloud-init/+merge/305882
Your team cloud init development team is requested to review the proposed merge
of ~harlowja/cloud-init:sys-io-errors into cloud-init:master.
___
Mailing list: ht
hm.. i had just inteded to say lets drop sys_netdev_info and instead make the
callers use read_sys_net
--
https://code.launchpad.net/~harlowja/cloud-init/+git/cloud-init/+merge/305882
Your team cloud init development team is requested to review the proposed merge
of ~harlowja/cloud-init:sys-io-
Ya, that makes sense to me, I can adjust this one.
--
https://code.launchpad.net/~harlowja/cloud-init/+git/cloud-init/+merge/305882
Your team cloud init development team is requested to review the proposed merge
of ~harlowja/cloud-init:sys-io-errors into cloud-init:master.
__
Josh,
I had sort of done this as:
http://paste.ubuntu.com/23207571/
The person who had reproduced this on arch said that the patch above it fixes
it.
Another option...
it looks like callers of sys_netdev_info could use read_sys_net.
And read_sys_net actually handles the IOError already.
I thin
9 matches
Mail list logo