/srv/irclogs.ubuntu.com/2015/10/26/#cloud-init.txt

=== alexpilotti_ is now known as alexpilotti
=== frickler_ is now known as frickler
=== smoser` is now known as smoser
nilujeIs there a reason why cloud-init doesn't have a test_requires in its setup.py?14:27
smoserniluje, not a good one. i'm a python-tools neophyte14:32
nilujesmoser: you are the one who packaged cloud-init?14:33
nilujesmoser: https://github.com/brmzkw/dyntftpd/blob/master/setup.py14:34
nilujeI have a basic setup.py here14:34
smoserniluje, we use tox, and tox reads requirements and test-requirements14:34
nilujetests_require is a nice to have, so you can run "pip install -e ." to install the packaages in editable mode14:34
nilujeoh14:35
smoserso theres probably just some overlap in functionality14:35
nilujeok great, I didn't see the tox.ini, only the Makefile and I tried to run nosetests manually14:35
nilujeI guess make test should install and run nose then14:35
smoserniluje, its kind of in both places...14:36
smoseri have 2 different thing si need to support14:36
smosera.) running and installing with ubuntu packaging (system python dependencies)14:36
smoserb.) testing without installing those packages (tox)14:37
nilujeok14:37
nilujethen adding tests_require in the setup.py seems to be a good idea :p14:37
=== jroll|dupe is now known as jroll
natorioussmoser: been looking at cloudbase-init for reference.  Their using wmi and win32api both which are not part of stdlib.  It'd be nice to get everything working using ctypes.windll or similar18:10
smoseryeah, agree.18:10
smoserwant as much as possible to be stdlib18:10
natoriousdid a bit more refactoring and I think moving away from network interfaces content can be better accomplished by using a sources helper to convert that into a similar network json format.  As in data source network_config should always return the same type and structure.  If static networking is moving towards network json, for backwards compat, it should convert an interfaces content file to network json to return18:13
natoriousI'm gonna dive into the windows partition label detection and hopefully get something decent soon18:14
alexpilotti_natorious: what is the reason why you want to avoid pywin32?18:15
natoriousalexpilotti: if were trying to make things as close to stdlib as possible, might as well not start w/ dependencies18:17
natoriousalexpilotti_: ^^18:17
alexpilotti_natorious: and for networks, we are already working on the new json format, that’s independent from pywin32 as we use ctypes for all networking stuff18:18
alexpilotti_WMI wont depend on pywin32 for long18:18
alexpilotti_we’re rewriting that module to avoid any COM usage18:19
alexpilotti_most modern APIs dont use Win32, but WMI18:19
alexpilotti_for example storage management is no entirely based on WMI, while previously was based on COM18:20
natoriousalexpilotti_: right.  The networking bit is for linux as much as windows.  How the datasources should handle returning either or but return the same type consistently18:20
alexpilotti_there should be an abstraction, with a clear interface18:20
alexpilotti_that’s what we are doing for most stuff in cloud-init V218:20
alexpilotti_btw smoser: how’s work going on V2?18:21
natoriousim at a point of needing to detect configdrive partitions for v2 via datasource18:21
natoriousand would like to attempt using stdlib before the alternatives18:22
natoriousfwiw18:22
natoriousbbiab18:22
akostadinov1Hello, I was wondering how to make cloud-init restart machine after package upgrade?19:32
akostadinov1it is usually required to restart so updates take effect. Like e.g. kernel upgrades/19:32
akostadinov1otherwise package upgrade is of limited help19:33
akostadinov1is there any switch to request that?19:33
smoserakostadinov1, in ubuntu it shoudl generally work.19:56
smoserprobably not supported in rpm based19:57
smoserhttp://bazaar.launchpad.net/~cloud-init-dev/cloud-init/trunk/view/head:/doc/examples/cloud-config.txt#L1619:57
akostadinov1smoser: too bad :/ running on RPM20:11

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!