| *** baoli has quit IRC | 00:00 | |
| *** david-lyle has joined #openstack-ironic | 00:09 | |
| *** gcb has quit IRC | 00:10 | |
| *** baoli has joined #openstack-ironic | 00:17 | |
| *** maratoid_ has quit IRC | 00:19 | |
| *** david-lyle has quit IRC | 00:20 | |
| *** pmannidi has joined #openstack-ironic | 00:24 | |
| *** pmannidi has quit IRC | 00:24 | |
| *** pmannidi has joined #openstack-ironic | 00:25 | |
| *** baoli has quit IRC | 00:27 | |
| *** rbudden has quit IRC | 00:37 | |
| *** pmannidi has quit IRC | 00:38 | |
| *** pmannidi has joined #openstack-ironic | 00:39 | |
| *** aNuposic has quit IRC | 00:39 | |
| *** pmannidi has quit IRC | 00:41 | |
| *** pmannidi has joined #openstack-ironic | 00:41 | |
| *** hoangcx has joined #openstack-ironic | 00:41 | |
| *** rajinir_ has quit IRC | 00:50 | |
| *** baoli has joined #openstack-ironic | 00:53 | |
| *** SerenaFeng has joined #openstack-ironic | 00:55 | |
| *** hw_wutianwei has joined #openstack-ironic | 00:56 | |
| *** SerenaFeng has quit IRC | 00:56 | |
| *** SerenaFeng has joined #openstack-ironic | 00:57 | |
| *** litao has joined #openstack-ironic | 01:01 | |
| *** phuongnh has joined #openstack-ironic | 01:04 | |
| *** aNuposic has joined #openstack-ironic | 01:04 | |
| *** crushil has joined #openstack-ironic | 01:09 | |
| *** oomichi has quit IRC | 01:12 | |
| *** yuanying has quit IRC | 01:12 | |
| *** oomichi has joined #openstack-ironic | 01:12 | |
| *** yuanying has joined #openstack-ironic | 01:13 | |
| *** zhurong has joined #openstack-ironic | 01:13 | |
| *** amotoki has joined #openstack-ironic | 01:14 | |
| *** amotoki has quit IRC | 01:17 | |
| *** amotoki_ has joined #openstack-ironic | 01:18 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Update OSC baremetal-node-set supporting dynamic drivers https://review.openstack.org/431542 | 01:18 |
|---|---|---|
| *** SerenaFeng has quit IRC | 01:23 | |
| *** aNuposic has quit IRC | 01:28 | |
| *** SerenaFeng has joined #openstack-ironic | 01:32 | |
| *** zhangjl has joined #openstack-ironic | 01:33 | |
| *** hamzy has joined #openstack-ironic | 01:45 | |
| *** zhangjl1 has joined #openstack-ironic | 01:51 | |
| *** zhangjl has quit IRC | 01:52 | |
| *** zhurong has quit IRC | 01:52 | |
| *** yufei_ has quit IRC | 01:53 | |
| *** Sukhdev has quit IRC | 01:56 | |
| *** gcb has joined #openstack-ironic | 01:56 | |
| *** zhangjl1 has quit IRC | 01:59 | |
| *** zhangjl has joined #openstack-ironic | 01:59 | |
| openstackgerrit | Merged openstack/ironic master: Source lib/ironic in grenade settings https://review.openstack.org/431369 | 02:02 |
| *** yufei_ has joined #openstack-ironic | 02:02 | |
| *** Syed__ has quit IRC | 02:05 | |
| openstackgerrit | wangxiyuan proposed openstack/ironic master: Change misc to test_utils for tempest test https://review.openstack.org/431809 | 02:07 |
| *** stack_ has quit IRC | 02:11 | |
| *** zhurong has joined #openstack-ironic | 02:12 | |
| *** zhurong has quit IRC | 02:25 | |
| *** crushil has quit IRC | 02:27 | |
| *** zhurong has joined #openstack-ironic | 02:30 | |
| *** zhurong has quit IRC | 02:38 | |
| *** baoli has quit IRC | 02:39 | |
| *** zhurong has joined #openstack-ironic | 02:51 | |
| *** ijw has quit IRC | 02:52 | |
| *** harlowja has quit IRC | 02:53 | |
| *** zhurong has quit IRC | 02:59 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic master: Add nodes tagging support - RPC objects https://review.openstack.org/233357 | 03:01 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic master: Add DB layer support for node tags filter https://review.openstack.org/272624 | 03:07 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic master: Add nodes tagging support - API https://review.openstack.org/250478 | 03:07 |
| *** amotoki_ has quit IRC | 03:08 | |
| *** amotoki has joined #openstack-ironic | 03:13 | |
| *** cdearborn has quit IRC | 03:14 | |
| *** rloo has quit IRC | 03:14 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-ui master: Correct horizon dir in installation doc https://review.openstack.org/431851 | 03:15 |
| *** sayantani01 has joined #openstack-ironic | 03:21 | |
| *** sdake has quit IRC | 03:28 | |
| *** crushil has joined #openstack-ironic | 03:29 | |
| *** sasha2 has quit IRC | 03:29 | |
| *** sayantani01 has quit IRC | 03:31 | |
| *** sayantani01 has joined #openstack-ironic | 03:32 | |
| *** Nisha_agarwal has joined #openstack-ironic | 03:42 | |
| *** tovin07 has quit IRC | 03:46 | |
| *** links has joined #openstack-ironic | 03:50 | |
| *** tovin07 has joined #openstack-ironic | 03:51 | |
| *** dsneddon_ has joined #openstack-ironic | 03:54 | |
| *** tuanluong has joined #openstack-ironic | 03:56 | |
| *** SerenaFeng has quit IRC | 03:57 | |
| *** sdake has joined #openstack-ironic | 04:07 | |
| *** stendulker has joined #openstack-ironic | 04:08 | |
| *** nmathew has joined #openstack-ironic | 04:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-oneviewclient master: Updated from global requirements https://review.openstack.org/389942 | 04:20 |
| *** udesale has joined #openstack-ironic | 04:24 | |
| *** tovin07 has quit IRC | 04:27 | |
| *** faizy has joined #openstack-ironic | 04:28 | |
| *** shu-mutou-AWAY has quit IRC | 04:29 | |
| *** dsneddon_ has quit IRC | 04:29 | |
| *** sayantan_ has joined #openstack-ironic | 04:33 | |
| *** sayantani01 has quit IRC | 04:33 | |
| *** tuanluong has quit IRC | 04:35 | |
| *** dsneddon_ has joined #openstack-ironic | 04:41 | |
| *** amotoki has quit IRC | 04:42 | |
| *** gcb has quit IRC | 04:51 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/431899 | 04:52 |
| *** gcb has joined #openstack-ironic | 04:53 | |
| *** sayantani01 has joined #openstack-ironic | 05:01 | |
| *** sayantan_ has quit IRC | 05:02 | |
| *** aparnav has joined #openstack-ironic | 05:09 | |
| *** ionutbalutoiu has quit IRC | 05:10 | |
| *** ionutbalutoiu has joined #openstack-ironic | 05:12 | |
| Nisha_agarwal | vsaienk0, Thanks :) gate passes after your patch is merged | 05:13 |
| Nisha_agarwal | :) | 05:13 |
| *** amotoki has joined #openstack-ironic | 05:22 | |
| *** amotoki has quit IRC | 05:24 | |
| *** amotoki has joined #openstack-ironic | 05:25 | |
| *** gcb has quit IRC | 05:29 | |
| *** gcb has joined #openstack-ironic | 05:30 | |
| *** owalsh_ has joined #openstack-ironic | 05:38 | |
| *** dsneddon_ has quit IRC | 05:39 | |
| *** owalsh has quit IRC | 05:39 | |
| *** SerenaFeng has joined #openstack-ironic | 05:40 | |
| openstackgerrit | M V P Nitesh proposed openstack/python-ironicclient master: Now for OSC 'node list' command with --provision-state argument for 'available' will correctly https://review.openstack.org/431556 | 05:41 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/bifrost master: Updated from global requirements https://review.openstack.org/431926 | 05:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/431952 | 05:49 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/431953 | 05:49 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/431954 | 05:49 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements https://review.openstack.org/431956 | 05:49 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Updated from global requirements https://review.openstack.org/431957 | 05:49 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/432088 | 05:58 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/431899 | 05:58 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-oneviewclient master: Updated from global requirements https://review.openstack.org/389942 | 05:59 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/virtualbmc master: Updated from global requirements https://review.openstack.org/432121 | 06:01 |
| *** dsneddon_ has joined #openstack-ironic | 06:04 | |
| *** tiendc has joined #openstack-ironic | 06:11 | |
| *** SerenaFeng has quit IRC | 06:13 | |
| *** MattMan_1 has quit IRC | 06:15 | |
| *** MattMan_1 has joined #openstack-ironic | 06:15 | |
| *** litao has quit IRC | 06:16 | |
| *** litao has joined #openstack-ironic | 06:17 | |
| *** gcb has quit IRC | 06:18 | |
| *** gcb has joined #openstack-ironic | 06:18 | |
| *** jaganathan has joined #openstack-ironic | 06:18 | |
| *** sacharya has quit IRC | 06:24 | |
| *** phuongnh has quit IRC | 06:25 | |
| *** crushil has quit IRC | 06:28 | |
| *** vsaienko has joined #openstack-ironic | 06:28 | |
| *** Nisha_agarwal has quit IRC | 06:38 | |
| *** Nisha_agarwal has joined #openstack-ironic | 06:39 | |
| *** pmannidi_ has joined #openstack-ironic | 06:44 | |
| *** makowals has quit IRC | 06:46 | |
| *** makowals has joined #openstack-ironic | 06:47 | |
| *** SerenaFeng has joined #openstack-ironic | 06:53 | |
| *** pmannidi_ has quit IRC | 06:55 | |
| *** mtanino has quit IRC | 07:00 | |
| *** rcernin has joined #openstack-ironic | 07:03 | |
| openstackgerrit | wanghongtaozz proposed openstack/ironic master: Delete the underline https://review.openstack.org/432144 | 07:04 |
| *** sayantani01 has quit IRC | 07:10 | |
| *** tiendc has quit IRC | 07:11 | |
| *** SerenaFeng has quit IRC | 07:13 | |
| *** SerenaFeng has joined #openstack-ironic | 07:14 | |
| *** vsaienko has quit IRC | 07:22 | |
| *** faizy has quit IRC | 07:28 | |
| *** pcaruana has joined #openstack-ironic | 07:30 | |
| *** faizy has joined #openstack-ironic | 07:31 | |
| *** Nisha_agarwal has quit IRC | 07:32 | |
| *** makowals has quit IRC | 07:33 | |
| *** fragatina has quit IRC | 07:36 | |
| *** mnaser has quit IRC | 07:41 | |
| *** afazekas has quit IRC | 07:43 | |
| *** makowals has joined #openstack-ironic | 07:45 | |
| *** dsneddon_ has quit IRC | 07:45 | |
| *** tiendc has joined #openstack-ironic | 07:45 | |
| *** tuanluong has joined #openstack-ironic | 07:45 | |
| *** foexle has joined #openstack-ironic | 07:45 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Adds wsgi to ironic-api in devstack https://review.openstack.org/430851 | 07:55 |
| *** faizy_ has joined #openstack-ironic | 08:02 | |
| *** sdake_ has joined #openstack-ironic | 08:04 | |
| *** faizy has quit IRC | 08:05 | |
| *** faizy__ has joined #openstack-ironic | 08:05 | |
| *** sdake has quit IRC | 08:07 | |
| *** faizy_ has quit IRC | 08:09 | |
| *** faizy has joined #openstack-ironic | 08:09 | |
| *** faizy__ has quit IRC | 08:12 | |
| openstackgerrit | Merged openstack/ironic master: No node interface settings for classic drivers https://review.openstack.org/430435 | 08:13 |
| *** foexle has quit IRC | 08:33 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Adds wsgi to ironic-api in devstack https://review.openstack.org/430851 | 08:36 |
| *** sdake_ has quit IRC | 08:37 | |
| *** sdake has joined #openstack-ironic | 08:38 | |
| *** sdake has quit IRC | 08:40 | |
| *** SerenaFeng has quit IRC | 08:40 | |
| *** SerenaFeng has joined #openstack-ironic | 08:42 | |
| vsaienk0 | Happy Friday Ironic'ers! gates are unblocked it is safe to do rechecks now :) | 08:42 |
| *** fragatina has joined #openstack-ironic | 08:46 | |
| *** SerenaFeng has quit IRC | 08:46 | |
| *** faizy has quit IRC | 08:46 | |
| *** faizy has joined #openstack-ironic | 08:47 | |
| *** fragatina has quit IRC | 08:51 | |
| *** ralonsoh has joined #openstack-ironic | 08:51 | |
| *** litao has quit IRC | 08:52 | |
| aparnav | Hey jroll, yuriyz, JayF, Is it possible for this documentation patch to go in today https://review.openstack.org/#/c/408441 ? | 08:55 |
| patchbot | patch 408441 - ironic - Doc for sanitize erase support in iLO drivers | 08:55 |
| *** mgoddard has joined #openstack-ironic | 08:55 | |
| *** mjura has joined #openstack-ironic | 08:58 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** amotoki has quit IRC | 09:01 | |
| *** zzzeek has joined #openstack-ironic | 09:03 | |
| *** derekh has joined #openstack-ironic | 09:07 | |
| *** jaganathan has quit IRC | 09:11 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Fix missed '_' import https://review.openstack.org/432191 | 09:14 |
| vsaienk0 | dtantsur, lucasagomes, vdrok it would be good to have this before releasing https://review.openstack.org/#/c/432191 1 liner | 09:15 |
| patchbot | patch 432191 - ironic - Fix missed '_' import | 09:15 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/431073 | 09:17 |
| *** jaganathan has joined #openstack-ironic | 09:18 | |
| *** nicodemos has quit IRC | 09:23 | |
| *** nicodemos has joined #openstack-ironic | 09:23 | |
| *** SerenaFeng has joined #openstack-ironic | 09:25 | |
| *** Nisha_agarwal has joined #openstack-ironic | 09:32 | |
| *** skazi has quit IRC | 09:36 | |
| *** lucas-afk is now known as lucasagomes | 09:38 | |
| lucasagomes | morning all | 09:38 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/431952 | 09:40 |
| openstackgerrit | Merged openstack/ironic master: Remove most unsupported drivers https://review.openstack.org/431205 | 09:41 |
| openstackgerrit | Merged openstack/ironic master: Fix object save after refresh failure https://review.openstack.org/354744 | 09:41 |
| *** faizy_ has joined #openstack-ironic | 09:42 | |
| *** sborkows has joined #openstack-ironic | 09:42 | |
| *** faizy has quit IRC | 09:45 | |
| Nisha_agarwal | lucasagomes, o/ | 09:46 |
| *** skazi has joined #openstack-ironic | 09:49 | |
| *** fragatina has joined #openstack-ironic | 09:49 | |
| *** mgoddard has quit IRC | 09:50 | |
| *** pmannidi has quit IRC | 09:50 | |
| *** kromanenko has joined #openstack-ironic | 09:52 | |
| *** jaganathan has quit IRC | 09:53 | |
| *** Nisha_agarwal has quit IRC | 09:54 | |
| *** fragatina has quit IRC | 09:55 | |
| vdrok | morning ironic, vsaienk0 lucasagomes | 09:55 |
| lucasagomes | o/ | 09:55 |
| vdrok | and morning nish | 09:55 |
| vsaienk0 | o/ | 09:55 |
| *** SerenaFeng has quit IRC | 09:55 | |
| vdrok | heh, tab fail | 09:55 |
| *** milan has joined #openstack-ironic | 09:57 | |
| milan | morning Ironic! :) | 09:57 |
| milan | how is the gate today? ;) | 09:57 |
| aarefiev | morning all! | 09:58 |
| milan | oh, and TGIF! :D | 09:58 |
| milan | morning aarefiev! :) | 09:58 |
| vdrok | morning milan and aarefiev | 09:58 |
| milan | morning vdrok! :) | 09:58 |
| vdrok | milan: gate seems fine :) | 09:58 |
| aarefiev | morning milan, vdrok :) | 09:58 |
| milan | ooooh :D | 09:58 |
| milan | aaah :D | 09:58 |
| * milan rechecks :D | 09:59 | |
| milan | vdrok, how was it corrected? | 09:59 |
| vdrok | milan: see mail from vasyl, basically needed to export one of the variables from our devstack so that it's visible in grenade | 10:00 |
| milan | vdrok, thx | 10:00 |
| * milan checks mail | 10:00 | |
| milan | vsaienk0, thanks for the fix! :) | 10:02 |
| milan | aarefiev, please, let me know wdyt about https://review.openstack.org/#/c/421285/15 I can make more updates if necessary | 10:03 |
| patchbot | patch 421285 - ironic-inspector - Find a node by multiple attributes | 10:03 |
| milan | * now that the gate works again ;) | 10:03 |
| aarefiev | milan: I'm looking | 10:08 |
| milan | aarefiev, thanks! | 10:08 |
| *** fxpester has joined #openstack-ironic | 10:08 | |
| milan | btw I've fired up rechecks on the patches in inspector that needed reverify | 10:09 |
| * milan hopes we're good now | 10:09 | |
| *** skazi has quit IRC | 10:12 | |
| *** jaganathan has joined #openstack-ironic | 10:13 | |
| *** openstackgerrit has quit IRC | 10:18 | |
| aarefiev | milan: could you check one thing regarding this patch | 10:19 |
| aarefiev | otherwise lgtm | 10:19 |
| aarefiev | milan: I'm not sure that it will work on mysql | 10:20 |
| milan | aarefiev, /me looks | 10:20 |
| *** hw_wutianwei has quit IRC | 10:21 | |
| *** alex_xu has quit IRC | 10:21 | |
| *** zhangjl has quit IRC | 10:21 | |
| *** amoralej has joined #openstack-ironic | 10:22 | |
| amoralej | dtantsure|afk any ETA for branch stable/ocata and ocata release? | 10:23 |
| amoralej | dtantsur|afk ^ | 10:23 |
| milan | aarefiev, I'll have to setup some env for it, but I'll check | 10:24 |
| milan | hopefully today | 10:24 |
| aarefiev | milan: thanks, I guess migration tests with mysql would be enough | 10:25 |
| *** mgoddard has joined #openstack-ironic | 10:26 | |
| milan | yeah, I'll run some MySQL on my laptop but gonna be trial&error O:-) | 10:26 |
| *** alex_xu has joined #openstack-ironic | 10:26 | |
| *** Nisha_agarwal has joined #openstack-ironic | 10:28 | |
| *** sambetts|afk is now known as sambetts | 10:30 | |
| *** openstackgerrit has joined #openstack-ironic | 10:32 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/432088 | 10:32 |
| *** skazi has joined #openstack-ironic | 10:32 | |
| *** udesale has quit IRC | 10:33 | |
| milan | gate merges \o/ :D | 10:33 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/virtualbmc master: Fix documentation title and remove bogus links https://review.openstack.org/432240 | 10:39 |
| *** owalsh_ is now known as owalsh | 10:43 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add functional tests for VIFs with ironicclient commands https://review.openstack.org/430881 | 10:43 |
| *** tuanluong has quit IRC | 10:44 | |
| *** hoangcx has quit IRC | 10:46 | |
| *** marlinc has quit IRC | 10:46 | |
| *** tiendc has quit IRC | 10:48 | |
| *** marlinc has joined #openstack-ironic | 10:50 | |
| lucasagomes | hey folks, if you have some time mind taking a look at https://review.openstack.org/#/q/status:open+project:openstack/virtualbmc,n,z ? I wanna release a new version and was thinking of merging the backlog first, the only patches standing are quite simple, just docs and requirements | 10:55 |
| lucasagomes | thanks | 10:55 |
| *** stendulker has quit IRC | 11:05 | |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic master: Add compatibility with Newton when creating a node https://review.openstack.org/420728 | 11:06 |
| *** dtantsur|afk is now known as dtantsur | 11:06 | |
| vdrok | lucasagomes: that patch removing module index and stuff. maybe better to fix that? I mean, seems like autodoc is not configured or something like that? | 11:07 |
| dtantsur | morning ironic | 11:07 |
| lucasagomes | vdrok, hm perhaps, lemme take a look | 11:07 |
| dtantsur | amoralej, we were supposed to do the release yesterday, but (as it usually happens) we had 2 days of grenade breakage | 11:07 |
| amoralej | ok | 11:07 |
| vdrok | lucasagomes: maybe we need sphinx.ext.viewcode too in the virtualbmc conf.py. thanks! | 11:08 |
| amoralej | dtantsur, do you know if you are using python-hardware package somewhere? | 11:08 |
| vdrok | morning dtantsur | 11:08 |
| amoralej | i don't see as dependency | 11:08 |
| dtantsur | amoralej, it's an optional dependency of IPA | 11:08 |
| amoralej | ok | 11:08 |
| amoralej | understood | 11:08 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: Ensure devstack does not try to delete a physical NIC https://review.openstack.org/432251 | 11:08 |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic master: Add compatibility with Newton when creating a node https://review.openstack.org/420728 | 11:08 |
| sambetts | dtantsur: super quick patch that fixes the blockage in my test environment: https://review.openstack.org/432251 | 11:09 |
| patchbot | patch 432251 - ironic-inspector - Ensure devstack does not try to delete a physical NIC | 11:09 |
| sambetts | aarefiev: ^^ | 11:09 |
| dtantsur | sure | 11:09 |
| dtantsur | how is our CI feeling today? the whiteboard says it's still down | 11:09 |
| sambetts | dtantsur: at least part of the fix merged | 11:10 |
| milan | morning dtantsur, lucasagomes sambetts ! :) | 11:10 |
| lucasagomes | o/ | 11:10 |
| *** crinkle_ has joined #openstack-ironic | 11:11 | |
| *** crinkle has quit IRC | 11:12 | |
| dtantsur | jroll, when you're here, mind checking https://review.openstack.org/#/c/430366/ please? | 11:12 |
| patchbot | patch 430366 - python-ironicclient (stable/ocata) - Bump the LAST_KNOWN_API_VERSION to 29 in OSC | 11:12 |
| vdrok | morning sambetts | 11:13 |
| vdrok | dtantsur: it's ok, will update the whiteboard | 11:13 |
| aarefiev | morning dtantsur, sambetts | 11:15 |
| *** alex_xu has quit IRC | 11:18 | |
| *** xavierr has quit IRC | 11:19 | |
| *** faizy_ has quit IRC | 11:20 | |
| aparnav | hey dtantsur, Is it possible for this doc patch to go in today https://review.openstack.org/#/c/408441/? | 11:22 |
| patchbot | patch 408441 - ironic - Doc for sanitize erase support in iLO drivers | 11:22 |
| *** alex_xu has joined #openstack-ironic | 11:22 | |
| dtantsur | aparnav, maybe. doc patches can be backported, so it's not critical for them to get in before a stable branch is cut. | 11:23 |
| *** mjura has quit IRC | 11:24 | |
| openstackgerrit | Merged openstack/virtualbmc master: Updated from global requirements https://review.openstack.org/432121 | 11:24 |
| aparnav | dtantsur, okay | 11:24 |
| pas-ha | TheJulia: seems upstream stable-2.1 Ansible branch is broken as most (all?) functional bifrost gates are currently failing - https://bugs.launchpad.net/bifrost/+bug/1663562 | 11:25 |
| openstack | Launchpad bug 1663562 in bifrost "task bifrost-ironic-install : Ensure guest user is removed from rabbitmq fails with "AttributeError: 'AnsibleModule' object has no attribute 'warn'"" [Undecided,New] | 11:25 |
| dtantsur | PTG "most meeting rooms don't have a projector" oh wonderful.. | 11:26 |
| pas-ha | dtantsur: may be a TV screen or smth? | 11:26 |
| dtantsur | hopefully | 11:26 |
| dtantsur | jroll, if we agree on some cross-project session with nova, we may want to reserve something https://ethercalc.openstack.org/Pike-PTG-Discussion-Rooms | 11:27 |
| openstackgerrit | Aparna proposed openstack/ironic master: Doc for sanitize erase support in iLO drivers https://review.openstack.org/408441 | 11:31 |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic master: [PoC] Add fake fields in Node object and RPC https://review.openstack.org/415462 | 11:31 |
| openstackgerrit | Merged openstack/virtualbmc master: Document some usage with examples https://review.openstack.org/428737 | 11:36 |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic master: Unit test for checking cross-version migrations compatibility https://review.openstack.org/419993 | 11:40 |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic master: Rolling upgrade procedure documentation https://review.openstack.org/416565 | 11:40 |
| *** jkilpatr has quit IRC | 11:42 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Update hardware inventory in docs https://review.openstack.org/432267 | 11:45 |
| *** tesseract has joined #openstack-ironic | 11:50 | |
| *** vsaienko has joined #openstack-ironic | 11:54 | |
| *** jkilpatr has joined #openstack-ironic | 11:56 | |
| pas-ha | yolanda: another question on OPNFV CI in ironic - are you pre-installing Ansible from source on those nodes where bifrost is started? | 11:56 |
| *** owalsh is now known as owalsh-afk | 11:57 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Update hardware inventory in docs https://review.openstack.org/432267 | 12:02 |
| yolanda | hi, pas-ha, this is still wip... that's the change that deals with OSA https://gerrit.opnfv.org/gerrit/#/c/28265 | 12:02 |
| lucasagomes | dtantsur, that "team and repository tag" was added later, Ironic also have it | 12:04 |
| lucasagomes | dtantsur, https://raw.githubusercontent.com/openstack/ironic/master/README.rst | 12:04 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/virtualbmc master: Fix documentation title and https://review.openstack.org/432240 | 12:06 |
| *** bfournie has quit IRC | 12:06 | |
| dtantsur | lucasagomes, yes, but it does not change the fact that it's wrong.. | 12:06 |
| *** bfournie has joined #openstack-ironic | 12:06 | |
| * lucasagomes inverts the order | 12:07 | |
| dtantsur | yes please (at least), though I don't see any value in this title at all | 12:08 |
| *** bfournie has quit IRC | 12:08 | |
| *** fragatina has joined #openstack-ironic | 12:10 | |
| *** Nisha_agarwal has quit IRC | 12:11 | |
| *** bfournie has joined #openstack-ironic | 12:13 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: When user executes "ironic node-show-states <node-name/node-uuid >", now it also displays information like name, uuid , maintenance and maintenance reason of the node. https://review.openstack.org/432277 | 12:14 |
| *** fragatina has quit IRC | 12:14 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/virtualbmc master: Fix documentation title and README https://review.openstack.org/432240 | 12:17 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/virtualbmc master: Fix documentation title and README https://review.openstack.org/432240 | 12:19 |
| *** cdearborn has joined #openstack-ironic | 12:24 | |
| *** faizy_ has joined #openstack-ironic | 12:31 | |
| *** milan has quit IRC | 12:31 | |
| *** milan has joined #openstack-ironic | 12:31 | |
| *** crinkle has joined #openstack-ironic | 12:33 | |
| *** crinkle_ has quit IRC | 12:34 | |
| *** vsaienko has quit IRC | 12:40 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/431952 | 12:47 |
| *** mgoddard has quit IRC | 12:53 | |
| TheJulia | pas-ha: Lovely. Making coffee and will login to take a look | 12:54 |
| openstackgerrit | Merged openstack/ironic-inspector master: Add node_info to some messages and clean up docstrings for lldp plugin https://review.openstack.org/430464 | 12:54 |
| openstackgerrit | Merged openstack/ironic-inspector master: Fix updating rows in d00d6e3f38c4 migration https://review.openstack.org/430295 | 12:54 |
| dtantsur | yay, working CI! | 12:54 |
| dtantsur | morning TheJulia | 12:55 |
| *** gcb has quit IRC | 12:55 | |
| *** faizy_ has quit IRC | 12:56 | |
| lucasagomes | TheJulia, morning | 12:57 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add basic tests for OSC plugin baremetal port commands https://review.openstack.org/365692 | 12:58 |
| openstackgerrit | Merged openstack/ironic master: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 12:59 |
| *** vsaienko has joined #openstack-ironic | 12:59 | |
| openstackgerrit | Merged openstack/ironic master: Add port groups configuration documentation https://review.openstack.org/429826 | 13:01 |
| openstackgerrit | Merged openstack/ironic master: Update api-ref for dynamic drivers https://review.openstack.org/428460 | 13:01 |
| *** aparnav has quit IRC | 13:04 | |
| *** owalsh-afk has quit IRC | 13:04 | |
| *** sdake has joined #openstack-ironic | 13:06 | |
| *** trown|outtypewww is now known as trown | 13:06 | |
| jroll | morning | 13:09 |
| dtantsur | morning jroll | 13:09 |
| *** mtanino has joined #openstack-ironic | 13:10 | |
| TheJulia | Good morning dtantsur, jroll | 13:12 |
| *** lucasagomes is now known as lucas-hungry | 13:12 | |
| lucas-hungry | jroll, morning! | 13:12 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Cleanup code associated with the ironic REST api https://review.openstack.org/432295 | 13:13 |
| *** owalsh-afk has joined #openstack-ironic | 13:15 | |
| *** owalsh-afk is now known as owalsh | 13:15 | |
| *** faizy has joined #openstack-ironic | 13:19 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Trivial: mock sleep in introspection tests https://review.openstack.org/430340 | 13:19 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Trivial: mock sleep in introspection tests https://review.openstack.org/430340 | 13:22 |
| *** ccamacho is now known as ccamacho|lunch | 13:23 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Switch to use test_utils.call_until_true https://review.openstack.org/431673 | 13:25 |
| aarefiev | lucasagomes: hey, do we have doc about root device hints in ironic, I didn't find one | 13:27 |
| dtantsur | aarefiev, http://docs.openstack.org/project-install-guide/baremetal/draft/advanced.html#specifying-the-disk-for-deployment-root-device-hints | 13:28 |
| *** Nisha_agarwal has joined #openstack-ironic | 13:28 | |
| aarefiev | dtantsur: aha, thanks | 13:29 |
| vdrok | morning TheJulia and jroll | 13:30 |
| *** dprince has joined #openstack-ironic | 13:32 | |
| *** mtanino has quit IRC | 13:35 | |
| *** baoli has joined #openstack-ironic | 13:37 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Update hardware inventory in docs https://review.openstack.org/432267 | 13:38 |
| milan | morning jroll, TheJulia! :) | 13:38 |
| *** baoli has quit IRC | 13:39 | |
| dtantsur | Nisha_agarwal, hi! there seems to be a unit test failure in your recent patch, when real proliantutils is used: https://trunk.rdoproject.org/centos7-ocata/2b/10/2b10dd8bfce789b50f29cff0d3f372dfdb56e19d_504339c8/rpmbuild.log | 13:39 |
| dtantsur | is it because of a proliantutils version mismatch or is it something we can fix in ironic? could you please take a look? | 13:39 |
| *** baoli has joined #openstack-ironic | 13:40 | |
| TheJulia | pas-ha: they occasionally like to release broken versions, we've also had to lock to specific commit sha until they've fixed whatever issue they created | 13:40 |
| TheJulia | that they | 13:40 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Update hardware inventory in docs https://review.openstack.org/432267 | 13:42 |
| openstackgerrit | Merged openstack/virtualbmc master: Fix documentation title and README https://review.openstack.org/432240 | 13:43 |
| Nisha_agarwal | vsaienk0, Hi | 13:44 |
| Nisha_agarwal | dtantsur, sure just looking at it | 13:44 |
| dtantsur | thnx | 13:44 |
| *** dtantsur is now known as dtantsur|brb | 13:45 | |
| Nisha_agarwal | dtantsur|brb, how can i trigger these tests again if i want to reproduce the failure | 13:48 |
| *** mgoddard has joined #openstack-ironic | 13:50 | |
| *** jcoufal has joined #openstack-ironic | 13:53 | |
| *** links has quit IRC | 13:55 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: [WiP] Add functional tests for VIFs in OSC plugin https://review.openstack.org/430904 | 13:55 |
| TheJulia | pas-ha: reviewed, we need to update the documentation and we are lacking a slight amount of direction since there are always people who don't read install documents or think that things are unchanged between releases :) | 13:56 |
| jroll | dtantsur|brb: are you seeing failures with real oneview libs, too? | 13:57 |
| jroll | and 60s unit tests | 13:57 |
| jroll | sigh | 13:57 |
| *** rpioso|afk has quit IRC | 13:58 | |
| milan | aarefiev, http://paste.openstack.org/show/598408/ | 13:58 |
| milan | aarefiev, how do I enforce the constraints checking on mariadb? | 13:58 |
| *** mjturek has joined #openstack-ironic | 13:59 | |
| aarefiev | milan: you need to install PyMySQL | 13:59 |
| milan | aarefiev, I had issues compiling that thing :-/ | 14:00 |
| milan | aarefiev, you know, my mac | 14:00 |
| milan | dtantsur|brb, ^ | 14:00 |
| aarefiev | milan: and https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/test/unit/test_migrations.py#L19 | 14:00 |
| milan | aarefiev, thx | 14:01 |
| * milan sets | 14:01 | |
| aarefiev | milan: :( btw did you check grenade | 14:01 |
| milan | aarefiev, I solved by using mysql-connector | 14:02 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Add a tox target for unit tests with driver libraries https://review.openstack.org/432317 | 14:02 |
| milan | aarefiev, nope, something odd in grenade? | 14:02 |
| jroll | dtantsur|brb: ^ you'll like this patch | 14:02 |
| *** sborkows has quit IRC | 14:02 | |
| aarefiev | milan: ok, I dunno but it fails on that patch | 14:02 |
| milan | :-/ | 14:03 |
| jroll | Nisha_agarwal: ^ if you run tox -e unit-with-driver-libs, with my patch there, it will trigger similar testing to dmitry's logs. beware, though, it's taking a long time because many oneview tests are running for 60 seconds and failing. | 14:03 |
| jroll | Nisha_agarwal: maybe use tox -e unit-with-driver-libs -- --regex ironic.tests.unit.drivers.modules.ilo | 14:03 |
| *** sasha2 has joined #openstack-ironic | 14:04 | |
| Nisha_agarwal | jroll thanks | 14:04 |
| milan | aarefiev, I need to check, but doesn't seem related to what I'm doing :-/ http://logs.openstack.org/85/421285/15/check/gate-grenade-dsvm-ironic-inspector-ubuntu-xenial/18c4a95/logs/grenade.sh.txt.gz#_2017-02-10_10_52_23_208 | 14:04 |
| Nisha_agarwal | i will post the patch soon | 14:04 |
| TheJulia | yolonda, If we retool on pypi based installs for bifrost, how horribly will it break your build model? | 14:04 |
| aarefiev | milan: ahh, ok, feel free to ping me if you need some help | 14:05 |
| milan | aarefiev, cool, thx. I'll try to run the test in the opportunistic env | 14:06 |
| milan | will let you know the result | 14:06 |
| aarefiev | thx | 14:06 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Add a tox target for unit tests with driver libraries https://review.openstack.org/432317 | 14:06 |
| Nisha_agarwal | jroll i get this "ERROR: unknown environment 'unit-with-driver-libs'" | 14:06 |
| jroll | Nisha_agarwal: did you apply my patch? | 14:07 |
| milan | aarefiev, /me just not sure why I have to have a special user&db for that | 14:07 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Use angular one-time-binding to improve performance https://review.openstack.org/432320 | 14:07 |
| Nisha_agarwal | can u paste link again | 14:07 |
| Nisha_agarwal | i think i missed the lnk | 14:07 |
| aarefiev | milan: oslo_db need admin access to db for run migrations | 14:08 |
| jroll | Nisha_agarwal: it's 7 lines up | 14:08 |
| jroll | Nisha_agarwal: https://review.openstack.org/432317 | 14:08 |
| patchbot | patch 432317 - ironic - Add a tox target for unit tests with driver libraries | 14:08 |
| Nisha_agarwal | jroll, thanks | 14:08 |
| milan | aarefiev, oslo_db, ack | 14:08 |
| Nisha_agarwal | jroll i didnt see that time...sorry, i will just test with ur pattch | 14:08 |
| *** vsaienko has quit IRC | 14:09 | |
| jroll | omg these oneview tests are killing me | 14:09 |
| Nisha_agarwal | :) | 14:09 |
| aarefiev | milan: yeah, it's hardcoded https://github.com/openstack/oslo.db/blob/accd272d15a142054b4d7e6c6c23743730a09bb2/oslo_db/sqlalchemy/test_base.py#L53 | 14:10 |
| milan | aarefiev, thx! I see | 14:10 |
| *** sayantani01 has joined #openstack-ironic | 14:10 | |
| *** fragatina has joined #openstack-ironic | 14:11 | |
| *** vsaienko has joined #openstack-ironic | 14:12 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Ensure devstack does not try to delete a physical NIC https://review.openstack.org/432251 | 14:16 |
| *** fragatina has quit IRC | 14:16 | |
| *** jcoufal has quit IRC | 14:16 | |
| *** lucas-hungry is now known as lucasagomes | 14:17 | |
| *** amoralej is now known as amoralej|lunch | 14:19 | |
| lucasagomes | vdrok, dtantsur|brb pas-ha thanks for the reviews, here's the patch for the release: https://review.openstack.org/#/c/431509/ | 14:20 |
| patchbot | patch 431509 - releases - Release virtualbmc 1.0.0 | 14:20 |
| lucasagomes | 1.0.0 o/! | 14:20 |
| jroll | \o/ | 14:20 |
| TheJulia | pas-ha: Just read your reply in the change. Let me know what I can do to help since it seems fixing it is a release blocker. | 14:21 |
| jroll | dtantsur|brb: oh, we should talk about liaison things sometime. I'm still release liaison at the moment. I also agreed with john/mrda that I should take over nova liaison if that's good with you (but in general we should look over all liaisons) | 14:21 |
| pas-ha | will work on this now. it won't pass OPNFV for sure with such changes - but hey, who cares about third-party CI when release is blocked ;) | 14:22 |
| vdrok | lucasagomes: nice :) | 14:23 |
| TheJulia | pas-ha: I do... kind of... :( | 14:23 |
| jroll | pas-ha: TheJulia: something blocking bifrost? | 14:23 |
| TheJulia | pas-ha: but, they have also been really fast to fix issues historically | 14:23 |
| pas-ha | well they'd need a couple of lines changed in their version of test-bifrost.sh | 14:23 |
| TheJulia | jroll: yeah, looks like ansible's stable release branch is broken | 14:23 |
| jroll | TheJulia: joy! | 14:24 |
| TheJulia | yup! | 14:24 |
| jroll | TheJulia: let me know if you need anything, I'd like to release that around the same time as ironic | 14:24 |
| jroll | which would hopefully be today but I have a feeling monday | 14:24 |
| TheJulia | jroll: 10-4, the fix is in pas-ha's court at the moment. I'm kind of hoping for the same, and have the same feeling. | 14:25 |
| jroll | TheJulia: cool, thanks | 14:25 |
| TheJulia | pas-ha: If you want, I'd be more than happy to take care of doc changes once you've got it re-worked | 14:25 |
| pas-ha | TheJulia: thanks, will ping you when done | 14:26 |
| TheJulia | Awesome | 14:26 |
| *** nicodemos has quit IRC | 14:27 | |
| *** nicodemos has joined #openstack-ironic | 14:27 | |
| pas-ha | OTOH seems not that straightforward - ansible launched from venv and accessing localhost is still lands inside that venv :( | 14:29 |
| *** dansmith is now known as superdan | 14:29 | |
| *** rbudden has joined #openstack-ironic | 14:29 | |
| TheJulia | what do you mean by is still lands inside that env? | 14:31 |
| TheJulia | venv | 14:31 |
| jroll | TheJulia: you think we'll get around to the nova side of BFV in pike? considering asking matt to approve the BP for pike | 14:33 |
| *** ccamacho|lunch has quit IRC | 14:33 | |
| TheJulia | jroll: I think so, that is along as the block device data is still passed along. | 14:34 |
| jroll | TheJulia: what do you mean by the last bit? | 14:34 |
| *** yogi has quit IRC | 14:35 | |
| TheJulia | There was talk of changing the block device mapping data around at the last summit, that info if memory serves is exactly what we need to populate for volumes in the volume connection information | 14:35 |
| *** sdake_ has joined #openstack-ironic | 14:36 | |
| jroll | TheJulia: oh, I assume it will still have what we need, virt should need it too | 14:37 |
| *** sdake has quit IRC | 14:39 | |
| *** jaganathan has quit IRC | 14:40 | |
| *** ccamacho has joined #openstack-ironic | 14:40 | |
| *** jcoufal has joined #openstack-ironic | 14:42 | |
| *** nmathew has quit IRC | 14:44 | |
| *** amoralej|lunch is now known as amoralej | 14:46 | |
| *** mtanino has joined #openstack-ironic | 14:47 | |
| jroll | omg, finally finished unit tests | 14:50 |
| jroll | - Failed: 121 | 14:50 |
| jroll | I see ucs and oneview, I assume ilo given dtantsur|brb's link | 14:51 |
| * jroll stepping out for about an hour | 14:51 | |
| *** jcoufal has quit IRC | 14:51 | |
| mariojv | i crossed off some of the things in the section under priorities on the whiteboard, hope that's useful to people | 14:53 |
| mariojv | only 2 things left around release notes :D | 14:53 |
| milan | aarefiev, so far i've got this: http://paste.openstack.org/show/598413/ | 14:55 |
| *** udesale has joined #openstack-ironic | 14:55 | |
| *** rpioso|afk has joined #openstack-ironic | 14:55 | |
| *** jcoufal has joined #openstack-ironic | 14:56 | |
| *** crushil has joined #openstack-ironic | 14:58 | |
| jroll | mariojv: ++, now I feel like the release blocker :/ | 14:59 |
| mariojv | jroll: nah, do you want some help with either of those? | 15:00 |
| mariojv | otherwise i'm just reviewing some client patches for driver comp + working on rescue today | 15:00 |
| jroll | mariojv: if you want to put a patch on top, sure, that would be helpful :) | 15:00 |
| jroll | 18:22:24 jroll | I left off around "The conductor process will refuse to start if a dynamic driver and a classic | 15:00 |
| patchbot | Error: No closing quotation | 15:00 |
| jroll | driver with the same name are both enabled" on http://docs.openstack.org/releasenotes/ironic/current-series.html | 15:00 |
| jroll | ^ from yesterday | 15:00 |
| mariojv | ok - that's for reno cleanup, not summary? | 15:01 |
| jroll | correct | 15:01 |
| mariojv | cool | 15:01 |
| jroll | it's all one chain | 15:01 |
| jroll | if you prefer to do one section e.g. "bug fixes" or "deprecations" or whatever, that's fine too | 15:02 |
| mariojv | i'll just work my way down and periodically update a wip patch | 15:02 |
| jroll | and if you look at my existing patch you'll see the sort of cleanup I've been focusing on | 15:02 |
| jroll | thanks :) | 15:02 |
| mariojv | np | 15:02 |
| * jroll actually goes now | 15:02 | |
| TheJulia | I can revise the summary | 15:03 |
| mariojv | going to grab a bit of brunch before starting on this | 15:04 |
| openstackgerrit | Miles Gould proposed openstack/ironic master: Allow setting UEFI boot devices in DRAC driver https://review.openstack.org/420107 | 15:05 |
| aarefiev | milan: so it doesn't run tests on maria for you | 15:06 |
| milan | aarefiev, but the tables get created for me ; do you have a log of a run that worked w/ mariadb? | 15:07 |
| aarefiev | milan: there should be same logs, but with Context impl MySQLImpl | 15:09 |
| aarefiev | let me try | 15:09 |
| patchbot | (let <variable> = <value> in <command>) -- Defines <variable> to be equal to <value> in the <command> and runs the <command>. '=' and 'in' can be omitted. | 15:09 |
| *** dtantsur|brb is now known as dtantsur | 15:09 | |
| milan | aarefiev, hmmm OK, thanks | 15:10 |
| *** jcoufal_ has joined #openstack-ironic | 15:10 | |
| dtantsur | jroll, I actually wanted to suggest you to take the nova liaison :) and it makes sense for you to finish at least the ocata releases, even if you don't want to stay the liaison there | 15:11 |
| *** jcoufal has quit IRC | 15:13 | |
| *** jcoufal__ has joined #openstack-ironic | 15:13 | |
| *** faizy_ has joined #openstack-ironic | 15:14 | |
| milan | aarefiev, found this one: http://git.openstack.org/cgit/openstack/tuning-box/tree/tox.ini#n12 | 15:15 |
| *** vsaienko has quit IRC | 15:16 | |
| *** jcoufal_ has quit IRC | 15:16 | |
| *** faizy has quit IRC | 15:17 | |
| aarefiev | milan: maybe wrong connection string https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/test/unit/test_migrations.py#L51 | 15:17 |
| aarefiev | hmm, but test creates tables | 15:18 |
| *** fxpester has quit IRC | 15:20 | |
| milan | aarefiev, got it: http://paste.openstack.org/show/598420/ | 15:20 |
| milan | I've not rebased on your fix though (hence the failure) | 15:21 |
| milan | I hope O:-) | 15:21 |
| aarefiev | milan: yeah, looks like it doesn't migrate on new revision | 15:21 |
| aarefiev | due this error | 15:21 |
| aarefiev | mysql works for me as well | 15:22 |
| milan | aarefiev, the mysqlconnector worked though | 15:22 |
| * milan tries rebasing | 15:22 | |
| *** sacharya has joined #openstack-ironic | 15:23 | |
| *** Goneri has joined #openstack-ironic | 15:25 | |
| milan | aarefiev, http://paste.openstack.org/show/598423/ | 15:26 |
| aarefiev | milan: yeah, same result for me | 15:27 |
| milan | aarefiev, it seems it's failing already in the _pre phase | 15:27 |
| *** sacharya has quit IRC | 15:27 | |
| *** rpioso|afk is now known as rpioso | 15:28 | |
| *** dims_ has quit IRC | 15:28 | |
| dtantsur | joanna, could you please provide any details on line 229 of https://etherpad.openstack.org/p/ironic-pike-ptg ? | 15:29 |
| aarefiev | milan: yeah, insert unexisting node uuid | 15:30 |
| milan | aarefiev, exactly | 15:31 |
| milan | I have to insert a node first | 15:31 |
| *** sacharya has joined #openstack-ironic | 15:33 | |
| *** dims has joined #openstack-ironic | 15:33 | |
| *** rama_y has joined #openstack-ironic | 15:36 | |
| *** yolanda has quit IRC | 15:38 | |
| *** yolanda has joined #openstack-ironic | 15:38 | |
| *** ralonsoh_ has joined #openstack-ironic | 15:40 | |
| *** jcoufal has joined #openstack-ironic | 15:42 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Pass session directly to swiftclient https://review.openstack.org/432353 | 15:42 |
| *** ralonsoh has quit IRC | 15:43 | |
| *** udesale has quit IRC | 15:44 | |
| *** jcoufal__ has quit IRC | 15:45 | |
| pas-ha | TheJulia: sorry, had to step away for a meetings... | 15:47 |
| pas-ha | TheJulia: what I meant was the following scenario: ansible is installed in a venv and runs against localhost (ansible_connection=local) - in this case the python env an ansible task has access to is the same venv unless the task is run as "become: yes" | 15:47 |
| pas-ha | but if I setup ssh keys to access localhost over ssh - everything works as usual... | 15:48 |
| TheJulia | pas-ha: ugh | 15:49 |
| TheJulia | that sounds rather broken if it won't work from with-in the env against localhost since it should just be executing commands | 15:49 |
| TheJulia | s/commands/steps in the playbook without ssh in between | 15:50 |
| NobodyCam | Good morning ironic'ers and...... | 15:50 |
| TheJulia | good morning NobodyCam | 15:50 |
| NobodyCam | TGIF | 15:50 |
| NobodyCam | Good morning TheJulia | 15:51 |
| dtantsur | morning NobodyCam | 15:51 |
| dtantsur | woohoo, TGIF! | 15:51 |
| *** Goneri has quit IRC | 15:51 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add a summary release note for ocata https://review.openstack.org/431187 | 15:52 |
| NobodyCam | Hey morning dtantsur, how goes the ship this morning... hehehe :) | 15:52 |
| *** crushil has quit IRC | 15:54 | |
| *** dims has quit IRC | 15:55 | |
| *** dims has joined #openstack-ironic | 15:55 | |
| *** jcoufal_ has joined #openstack-ironic | 15:55 | |
| dtantsur | heh, pretty fine :) | 15:55 |
| milan | aarefiev, you were right in the review: http://paste.openstack.org/show/598427/ | 15:56 |
| *** dims is now known as Guest12406 | 15:56 | |
| milan | aarefiev, what if I just keep the uuid and use it both as node_uuid and attribute uuid? https://review.openstack.org/#/c/421285/15/ironic_inspector/migrations/versions/882b2d84cb1b_attribute_constraints_relaxing.py@74 | 15:57 |
| patchbot | patch 421285 - ironic-inspector - Find a node by multiple attributes | 15:57 |
| *** ijw has joined #openstack-ironic | 15:58 | |
| aarefiev | milan: I was fine with just dropping all attributes as it's temp data :) | 15:58 |
| aarefiev | dtantsur ^ | 15:58 |
| milan | /o\ | 15:58 |
| *** jcoufal has quit IRC | 15:59 | |
| aarefiev | I mean milan's suggestion | 15:59 |
| *** rama_y has quit IRC | 15:59 | |
| milan | we'd have an odd situation w/ unforeseen consequences I guess http://sd.keepcalm-o-matic.co.uk/i/keep-calm-and-prepare-for-unforeseen-consequences.png | 16:00 |
| NobodyCam | :) | 16:00 |
| aarefiev | milan: I'm fine with both options :) | 16:01 |
| milan | aarefiev, you mean w/ the unforeseen consequences too? :D | 16:01 |
| * milan updates patch | 16:01 | |
| milan | NobodyCam, morning! TGIF! :D | 16:01 |
| aarefiev | :) | 16:02 |
| aarefiev | morning NobodyCam :) | 16:02 |
| NobodyCam | Morning milan and aarefiev :) | 16:02 |
| NobodyCam | happy Friday | 16:02 |
| dtantsur | milan, aarefiev, if we're ok with an upgrade with data loss... which is something we promised not to do when applies for supports-upgrade tag fyi | 16:02 |
| milan | dtantsur, yeah wdyt about keeping the uuids equal? | 16:03 |
| dtantsur | note that there is always an option to create a temporary table, populate it, then rename | 16:03 |
| dtantsur | milan, equal to what? :) | 16:03 |
| milan | node_uuid == attributes.uuid | 16:03 |
| milan | * attributes.node_uuid == attributes.uuid | 16:04 |
| milan | * == some nodes.uuid | 16:04 |
| dtantsur | it assumes that we don't have several attributes per node, which is not true | 16:04 |
| aarefiev | dtantsur: I don't think that this would the the biggest problem if you upgrade during running introspection | 16:04 |
| aarefiev | :) | 16:04 |
| *** Guest12406 has quit IRC | 16:05 | |
| milan | dtantsur, correct | 16:05 |
| aarefiev | ahh yeah, we have bmc_address and mac now ( | 16:05 |
| *** rcernin has quit IRC | 16:06 | |
| *** crushil has joined #openstack-ironic | 16:06 | |
| dtantsur | well, doing 'drop table' is not substantially easier than 'create table', 'insert from select', 'drop table', 'alter table rename' (or how is it?) | 16:06 |
| *** mnaser has joined #openstack-ironic | 16:07 | |
| dtantsur | I'm just a bit uneasy on giving up our commitments, even in such quite simple situations | 16:07 |
| aarefiev | create tmp table -> copy -> drop original -> rename | 16:08 |
| dtantsur | yep | 16:08 |
| aarefiev | the most easy way | 16:08 |
| aarefiev | ideally: drop constraint -> move data -> crate new constraint | 16:09 |
| milan | aarefiev, yeah | 16:10 |
| *** jcoufal has joined #openstack-ironic | 16:16 | |
| milan | aarefiev, the split messes w/ sqlite http://paste.openstack.org/show/598432/ :-/ | 16:17 |
| aarefiev | :( | 16:18 |
| milan | aarefiev, and MySQL doesn't mind | 16:18 |
| aarefiev | well, for mysql we could do this in 2 lines | 16:19 |
| *** jcoufal_ has quit IRC | 16:19 | |
| dtantsur | milan, connection.execute starts a transaction implicitly iirc | 16:19 |
| *** sayantani01 has quit IRC | 16:19 | |
| dtantsur | in sqlite | 16:19 |
| dtantsur | so you may want to put it in a transaction of its own, even though it's just a select | 16:19 |
| *** sayantani01 has joined #openstack-ironic | 16:20 | |
| openstackgerrit | Sofiia Andriichenko proposed openstack/ironic master: Add more tempest plugin API tests for driver: https://review.openstack.org/408531 | 16:20 |
| aarefiev | looks like it become more complicated | 16:20 |
| aarefiev | with hidden bugs | 16:21 |
| aarefiev | maybe lets go first way | 16:21 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Always install Ansible with pip in a VENV https://review.openstack.org/431614 | 16:22 |
| openstackgerrit | Sofiia Andriichenko proposed openstack/ironic master: Add more tempest plugin API tests for driver: https://review.openstack.org/408531 | 16:23 |
| *** dims_ has joined #openstack-ironic | 16:24 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Always install Ansible with pip in a VENV https://review.openstack.org/431614 | 16:27 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Find a node by multiple attributes https://review.openstack.org/421285 | 16:28 |
| * milan facepalm :D aarefiev, dtantsur I've got it ^ | 16:28 | |
| * dtantsur waits for CI | 16:28 | |
| *** rajinir_ has joined #openstack-ironic | 16:29 | |
| aarefiev | milan: nice, seems good at first glance | 16:29 |
| milan | aarefiev, dtantsur I've rebased it though :( | 16:29 |
| *** Syed__ has joined #openstack-ironic | 16:31 | |
| openstackgerrit | Merged openstack/ironic master: Fix missed '_' import https://review.openstack.org/432191 | 16:32 |
| *** afazekas has joined #openstack-ironic | 16:32 | |
| openstackgerrit | Merged openstack/ironic master: Switch to use test_utils.call_until_true https://review.openstack.org/431674 | 16:33 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Follow up patch for SNMPv3 support https://review.openstack.org/432375 | 16:33 |
| openstackgerrit | Merged openstack/ironic master: Adds network check in upgrade phase in devstack https://review.openstack.org/428043 | 16:33 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Always install Ansible with pip in a VENV https://review.openstack.org/431614 | 16:34 |
| Nisha_agarwal | dtantsur, ^^^ | 16:35 |
| dtantsur | already testing :) | 16:35 |
| Nisha_agarwal | dtantsur, let me know if u still see issue | 16:35 |
| dtantsur | Nisha_agarwal, left some comments, this is getting close | 16:37 |
| *** jcoufal_ has joined #openstack-ironic | 16:40 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types https://review.openstack.org/419274 | 16:40 |
| mariojv | do we capitalize "Ironic" in release notes? | 16:40 |
| * milan calls it a day, night Ironic! :) | 16:41 | |
| *** milan has quit IRC | 16:41 | |
| dtantsur | mariojv, we were recommended to not capitalize it | 16:41 |
| mariojv | npk | 16:41 |
| mariojv | *k | 16:41 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Always install Ansible with pip in a VENV https://review.openstack.org/431614 | 16:42 |
| *** jcoufal has quit IRC | 16:43 | |
| *** jcoufal__ has joined #openstack-ironic | 16:43 | |
| *** tesseract has quit IRC | 16:45 | |
| *** jcoufal_ has quit IRC | 16:45 | |
| *** rama_y has joined #openstack-ironic | 16:47 | |
| TheJulia | pas-ha: http://logs.openstack.org/14/431614/7/check/gate-bifrost-integration-tinyipa-ubuntu-xenial/cc460dd/console.html#_2017-02-10_16_47_07_710694 :( | 16:51 |
| * pas-ha bangs his head on keyboard... or the CI one... | 16:51 | |
| pas-ha | yep, https://github.com/openstack/bifrost/blob/master/scripts/install-deps.sh#L110 | 16:51 |
| pas-ha | a new patch is coming | 16:51 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Always install Ansible with pip in a VENV https://review.openstack.org/431614 | 16:52 |
| *** pcaruana has quit IRC | 16:53 | |
| mariojv | i think this release note is completely incorrect https://github.com/openstack/ironic/blob/f0108e05edd622ad45c2e5b844690733668993f6/releasenotes/notes/context-domain-id-name-deprecation-ae6e40718273be8d.yaml | 16:55 |
| mariojv | from looking at https://github.com/openstack/ironic/blob/f0108e05edd622ad45c2e5b844690733668993f6/ironic/common/context.py | 16:56 |
| *** ElCoyote_ has quit IRC | 16:56 | |
| mariojv | ah, never mind | 16:57 |
| mariojv | api does some translation it seems | 16:57 |
| *** sayantani01 has quit IRC | 17:00 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Follow up patch for SNMPv3 support https://review.openstack.org/432375 | 17:01 |
| mariojv | was 1 small error in the reno though | 17:01 |
| *** sayantani01 has joined #openstack-ironic | 17:01 | |
| mgoddard | hi folks, I have a quick question about the current state of networking in Ironic. Are multiple physical networks supported yet? I've been leafing through specs and source code and it appears not, due to https://bugs.launchpad.net/ironic/+bug/1405131 | 17:01 |
| openstack | Launchpad bug 1544169 in Ironic "duplicate for #1405131 [RFE] Advanced network configuration in ironic" [Wishlist,Confirmed] - Assigned to Vasyl Saienko (vsaienko) | 17:01 |
| sambetts | mgoddard: multiple physical networks means multiple neutron physnets or multiple vlans? | 17:02 |
| *** bnemec is now known as beekneemech | 17:02 | |
| mgoddard | sambetts: multiple physnets | 17:03 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: More release note cleanup for 7.0.0 https://review.openstack.org/432384 | 17:03 |
| mariojv | ^ that's still WIP | 17:03 |
| *** trown is now known as trown|lunch | 17:04 | |
| *** harlowja has joined #openstack-ironic | 17:05 | |
| *** harlowja has quit IRC | 17:05 | |
| *** harlowja has joined #openstack-ironic | 17:05 | |
| sambetts | mgoddard: currently we don't have support for that as there is no way (right now) to ensure the neutron ports end up on the right physical NICs | 17:05 |
| aslezil | dtantsur: once you get a chance, could you please review https://review.openstack.org/#/c/396406/ | 17:06 |
| patchbot | patch 396406 - ironic-python-agent - Collect NIC name given by BIOS | 17:06 |
| mgoddard | ok, thanks for confirming | 17:06 |
| *** ralonsoh_ has quit IRC | 17:07 | |
| dtantsur | aslezil, on my radar, but probably not for today, sorry :( | 17:07 |
| mgoddard | sambetts: are you aware of any (current) plans to resolve that? There are several related bugs that got marked as duplicates of https://bugs.launchpad.net/ironic/+bug/1544169, but I'm not sure I agree that it really covers this issue | 17:08 |
| openstack | Launchpad bug 1544169 in Ironic "[RFE] Advanced network configuration in ironic" [Wishlist,Confirmed] - Assigned to Vasyl Saienko (vsaienko) | 17:08 |
| pas-ha | TheJulia: sigh... installing Ansible in a venv does not work exactly for the reason I described (due to ansible_connection=local) - the non-becomed tasks are not seeing python packages that've been installed by become-d tasks before. So I will revert to the version that was actually working (by installing it systemwide by default), and we have to issue an | 17:08 |
| pas-ha | honest warning that if you have Ansible installed, just running "sudo scripts/env-setup.sh" might clobber it. | 17:08 |
| aslezil | dtantsur : no problem. Just have this patch on list :) | 17:09 |
| mgoddard | there is also the port mapping spec (https://review.openstack.org/279148), but it seems fairly stale | 17:09 |
| patchbot | patch 279148 - ironic-specs - Add neutron-ironic port mapping function | 17:09 |
| *** stevemar has left #openstack-ironic | 17:10 | |
| sambetts | mgoddard: yup there are plans to resolve that and we should be discussing that at the PTG, we need to resolve it to support routed networks in Neutron | 17:11 |
| mgoddard | ok great. I'll be at the PTG and am keen to finally knock it on the head :) | 17:11 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Always install Ansible with pip https://review.openstack.org/431614 | 17:13 |
| TheJulia | pas-ha: Ahh, that makes sense dependency wise :( What a migraine :( | 17:13 |
| pas-ha | so for docs I would put it this way - if you have appropriate ansible installed already - just run pip install -r bifrost/requirements.txt and then run the playbooks | 17:15 |
| pas-ha | if not - run the env-setup.sh, which honestly is more suitable for test environments | 17:16 |
| pas-ha | btw, I have to test how ansible installed outside venv can work with bifrost's dynamic inventory in a venv :) | 17:16 |
| mgoddard | sambetts: we're building on a prototype system for the SKA science data processor. It will be managed by OpenStack and will have two Ethernet NICs and an IB NIC per node | 17:17 |
| *** faizy_ has quit IRC | 17:17 | |
| *** kromanenko has quit IRC | 17:17 | |
| TheJulia | pas-ha: and that makes sense actually | 17:18 |
| sambetts | mgoddard: oh awesome, do you need the physnet support because of the IB network vs the Ethernet network | 17:18 |
| *** fragatina has joined #openstack-ironic | 17:19 | |
| *** aarefiev is now known as aarefiev_afk | 17:19 | |
| TheJulia | pas-ha: I'll likely get started on the doc updates in about an hour | 17:19 |
| pas-ha | TheJulia: especially since there is a patch to cut down on those packages in requirements.txt | 17:19 |
| pas-ha | https://review.openstack.org/#/c/431496/ | 17:20 |
| patchbot | patch 431496 - bifrost - Cleanup python requirements | 17:20 |
| mgoddard | IB will be outside of OpenStack's control for now. We need physnet support because the two Ethernet networks are physically separate | 17:20 |
| *** fragatina has quit IRC | 17:20 | |
| TheJulia | pas-ha: was thinking about that actually :) | 17:20 |
| *** fragatina has joined #openstack-ironic | 17:20 | |
| jroll | dtantsur: yeah, I'll finish ocata releases for sure, I don't mind handling them for pike if nobody else wants to. I'd like you to make the calls on when we want to release, if you like. full disclosure, I will push for lots of pike releases :) | 17:21 |
| jroll | TheJulia: thanks for updating the reno cleanup | 17:21 |
| TheJulia | jroll: it was just the summary | 17:21 |
| jroll | mariojv: and thank you for helping as well | 17:21 |
| jroll | TheJulia: er. yeah. that. :) | 17:22 |
| mariojv | np, more incoming later today | 17:22 |
| mariojv | i'm about down to "Fixes the issue that API service does not start if audit is enabled with default value of “ignore_req_list” config option. | 17:22 |
| patchbot | Error: No closing quotation | 17:22 |
| mariojv | going straight down the rendered page | 17:22 |
| mariojv | there are a lot of release notes... | 17:22 |
| pas-ha | i might be wrong about removing pymysql though - I have some tingling feeling it might break enable_venv as this one is not listed as python requirement for anything AFAIR | 17:22 |
| jroll | mariojv: cool, I have a haircut in 30-some minutes I forgot about, so I won't be doing any before that | 17:22 |
| dtantsur | jroll, full disclosure: me too :) | 17:22 |
| jroll | dtantsur: <3 | 17:23 |
| pas-ha | TheJulia: latest patch-set also failed, but with old good known "no ipv4 key" thing | 17:23 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Follow up patch for SNMPv3 support https://review.openstack.org/432375 | 17:24 |
| Nisha_agarwal | jroll, thanks for the tox patch | 17:24 |
| Nisha_agarwal | it helped a lot | 17:24 |
| jroll | Nisha_agarwal: awesome :) | 17:25 |
| jroll | we kinda need it anyway | 17:25 |
| Nisha_agarwal | Yes i also think so | 17:25 |
| jroll | dtantsur: did you see this, btw https://review.openstack.org/#/c/432317/ | 17:25 |
| patchbot | patch 432317 - ironic - Add a tox target for unit tests with driver libraries | 17:25 |
| dtantsur | jroll, yep, didn't have a chance to check it yet | 17:25 |
| Nisha_agarwal | dtantsur, works for me :) | 17:25 |
| jroll | ok, cool | 17:25 |
| jroll | dtantsur: skip oneview tests if you try it locally :) | 17:26 |
| dtantsur | ok :) | 17:26 |
| jroll | lots of missed mocks it seems, results in ~100 tests that take 60s to time out | 17:26 |
| TheJulia | pas-ha: I was _really_ hoping we got that fixed with not having the alias in defaults :( | 17:26 |
| *** aNuposic has joined #openstack-ironic | 17:27 | |
| TheJulia | pas-ha: it does seem to happen a lot less now, but still :( | 17:27 |
| dtantsur | jroll, ugh. mind filing bugs? | 17:30 |
| jroll | dtantsur: yeah, will do it later today or monday | 17:31 |
| * jroll is super backlogged but will get to it | 17:31 | |
| dtantsur | thanks | 17:32 |
| *** sayantani01 has quit IRC | 17:32 | |
| *** sayantani01 has joined #openstack-ironic | 17:33 | |
| jroll | this is a nice guide to getting used to the openstack workflow via a docs patch, if anyone has noobs incoming http://superuser.openstack.org/articles/new-openstack-contributor/ | 17:33 |
| * sambetts -> home | 17:36 | |
| sambetts | night all o/ | 17:36 |
| mgoddard | night sambetts, thanks for the answers | 17:37 |
| dtantsur | night all, see you on Monday | 17:38 |
| *** dtantsur is now known as dtantsur|afk | 17:38 | |
| lucasagomes | sambetts, dtantsur|afk see ya! | 17:39 |
| vdrok | night sambetts and dtantsur|afk | 17:40 |
| *** derekh has quit IRC | 17:40 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: More release note cleanup for 7.0.0 https://review.openstack.org/432384 | 17:43 |
| mariojv | have the 6.2.0 release notes already been cleaned up? not the 6.2.0-433 | 17:43 |
| mariojv | i went through all 6.2.0-433 after what jroll did in his patch | 17:44 |
| jroll | mariojv: yeah | 17:44 |
| jroll | oh wow, amazing | 17:44 |
| mariojv | great \o/ | 17:44 |
| mariojv | probably missed a few things but most should be ready | 17:44 |
| jroll | awesome, thank you | 17:44 |
| * jroll will update his own and review yours when he's back, see y'all in a bit | 17:44 | |
| *** sambetts is now known as sambetts|afk | 17:44 | |
| mariojv | np, hooray | 17:44 |
| mariojv | i'll review yours too in a bit also | 17:45 |
| pas-ha | yolanda: the reason for my earlier question re OPNFV with Ansible pre-installed on nodes is this http://artifacts.opnfv.org/cross-community-ci/openstack/bifrost/review.openstack.org/431614/9/openstack-bifrost-verify-centos7-virtual-master/build_log.txt | 17:45 |
| pas-ha | yolanda: it looks like it is already there - my patch is trying to install ansible from pip, does not install it as requirement ansible<2.2 is already satisfied, but the failure then is the same we've started seeing overnight today when bifrost installs ansible form stable-2.1 checkout | 17:47 |
| pas-ha | so it seems that Ansible 2.1.x is already installed on the node, and is installed from git checkout | 17:48 |
| *** nicodemos has quit IRC | 17:50 | |
| *** nicodemos has joined #openstack-ironic | 17:50 | |
| TheJulia | pas-ha: that actually kind of makes sense given their CI integration mission | 17:51 |
| pas-ha | to my taste this is strange a bit - we are not trying to develop configuration tools themselves, so why use a potentially unstable, unreleased versions? | 17:52 |
| TheJulia | pas-ha: because opnfv is attempting to integrate on as new code as possible, not code 6 months to a year old | 17:53 |
| pas-ha | why use Ansible 2.1.xx then? :) when 2.2.1 is already there | 17:54 |
| TheJulia | pas-ha: They are likely doing the same, stable-2.1 and haven't changed the branch tag to stable-2.2 | 17:54 |
| *** mgoddard has quit IRC | 18:00 | |
| *** amarao has joined #openstack-ironic | 18:01 | |
| *** jaosorior has joined #openstack-ironic | 18:03 | |
| *** lucasagomes is now known as lucas-afk | 18:06 | |
| lucas-afk | night all, enjoy the weekend! | 18:06 |
| *** Nisha_agarwal has quit IRC | 18:10 | |
| *** Nisha_agarwal has joined #openstack-ironic | 18:10 | |
| vdrok | good night lucas-afk | 18:10 |
| NobodyCam | night lucas-afk | 18:11 |
| *** yolanda has quit IRC | 18:14 | |
| *** yolanda has joined #openstack-ironic | 18:16 | |
| *** crushil has quit IRC | 18:20 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: DNM testing for bug 1660953 https://review.openstack.org/432416 | 18:23 |
| openstack | bug 1660953 in bifrost "transient gate failures on gate-bifrost-integration-tinyipa-ubuntu-xenial" [Undecided,New] https://launchpad.net/bugs/1660953 | 18:23 |
| *** crushil has joined #openstack-ironic | 18:37 | |
| *** vsaienko has joined #openstack-ironic | 18:37 | |
| *** mgoddard has joined #openstack-ironic | 18:41 | |
| TheJulia | pas-ha: I wonder if active == false is a result of STP... | 18:41 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Always install Ansible with pip https://review.openstack.org/431614 | 18:41 |
| TheJulia | pas-ha, NobodyCam, cinerama ^^^ | 18:41 |
| NobodyCam | TheJulia: is that bug worthy ? | 18:42 |
| TheJulia | heh, yes | 18:42 |
| * TheJulia goes and looks for the bug number | 18:43 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Always install Ansible with pip https://review.openstack.org/431614 | 18:44 |
| *** sdake_ has quit IRC | 18:48 | |
| pas-ha | TheJulia: about STP - does not look like - http://logs.openstack.org/16/432416/1/check/gate-bifrost-integration-dhcp-ubuntu-xenial-nv/ef9fd2b/console.html#_2017-02-10_18_33_21_621247 also has STP on, but does have IP address | 18:53 |
| *** Nisha_agarwal has quit IRC | 18:54 | |
| TheJulia | pas-ha: I think we're at the point where we need to grab a copy of the libvirtd log and see what it is really doing | 18:55 |
| TheJulia | pas-ha: but STP does dictate a hold down on interface initialization, kind of wondering if libvirt is not bringing up the address on the interface until it is "active" | 18:56 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Mock client initializations for irmc and oneview https://review.openstack.org/432424 | 18:56 |
| *** sdake has joined #openstack-ironic | 18:57 | |
| vdrok | jroll: ^^ fixes the timeouts when tests run with driver-requirements | 18:57 |
| jroll | vdrok: omg you rule | 18:58 |
| vdrok | :) | 18:58 |
| *** ricardoas has left #openstack-ironic | 18:59 | |
| *** jaosorior has quit IRC | 19:00 | |
| TheJulia | pas-ha: oh, hmm.... that one shows the interface as not active :\ | 19:02 |
| *** nicodemos has quit IRC | 19:05 | |
| *** jaosorior has joined #openstack-ironic | 19:05 | |
| *** owalsh is now known as owalsh-afk | 19:05 | |
| *** nicodemos has joined #openstack-ironic | 19:05 | |
| *** cdaniel has quit IRC | 19:05 | |
| *** aNuposic has quit IRC | 19:08 | |
| *** owalsh-afk has quit IRC | 19:09 | |
| *** nicodemos has quit IRC | 19:10 | |
| *** Sukhdev has joined #openstack-ironic | 19:17 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: WiP installing Ansible with pip install --user https://review.openstack.org/432428 | 19:18 |
| *** vsaienko has quit IRC | 19:18 | |
| vdrok | jroll: also an unrelated question, if you have time, as we decided not to go with merging that stuff with adding _lookup in chassis subcontroller, should we also revert that one patch that got in? https://github.com/openstack/ironic/commit/39e58151afc205fcf10f23256db123bc05007b8b As it causes another bug on client side - | 19:18 |
| vdrok | https://bugs.launchpad.net/python-ironicclient/+bug/1619471 | 19:18 |
| openstack | Launchpad bug 1619471 in python-ironicclient "Client should be able to process the pecan constructed exceptions in the response body" [Medium,In progress] - Assigned to Vladyslav Drok (vdrok) | 19:18 |
| jroll | ugh | 19:18 |
| TheJulia | pas-ha: so, I'm kind of wondering if we should exit 1 as part of the env-setup script | 19:18 |
| jroll | vdrok: it's an api change, idk about reverting it | 19:18 |
| jroll | I don't have the brain bandwidth for these things at the moment | 19:19 |
| *** trown|lunch is now known as trown|mtg | 19:19 | |
| *** trown|mtg is now known as trown | 19:19 | |
| vdrok | jroll: oh, or we should not, we've done the same with portgroups - https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/portgroup.py#L240 | 19:19 |
| vdrok | so yeah, will update that client fix then | 19:19 |
| jroll | vdrok: cool | 19:19 |
| pas-ha | TheJulia: why? to hard-notify external users like OPNFV folks? this whole workaround (lines above) was just so that they won't fail | 19:20 |
| pas-ha | ha! passed tests but failed reno :/ | 19:20 |
| TheJulia | heh | 19:20 |
| *** vsaienko has joined #openstack-ironic | 19:21 | |
| TheJulia | pas-ha: I was thinking that prior to the revision you just posted an update to, so disregard that thought | 19:21 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Always install Ansible with pip https://review.openstack.org/431614 | 19:23 |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent master: Collect NIC name given by BIOS https://review.openstack.org/396406 | 19:24 |
| TheJulia | pas-ha: revised the readme above, I guess I got lost in editing the text originally | 19:26 |
| *** sayantani01 has quit IRC | 19:27 | |
| *** sdake_ has joined #openstack-ironic | 19:29 | |
| pas-ha | I think I have a next candidate to not clobber system-wide Ansible - seems installing with --user to ${HOME}/.local/bin works :) waiting for more jobs to pass, but DHCP one is already passed :) | 19:31 |
| *** sdake has quit IRC | 19:32 | |
| TheJulia | \o/ | 19:32 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Print pecan exceptions properly https://review.openstack.org/364566 | 19:32 |
| vdrok | pretty easy one by the way ^^ | 19:33 |
| *** aNuposic has joined #openstack-ironic | 19:33 | |
| jroll | vdrok: one easy comment and then I'm +2 | 19:37 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Print pecan exceptions properly https://review.openstack.org/364566 | 19:38 |
| vdrok | jroll: done, thanks :) | 19:38 |
| jroll | vdrok: +2 there, +A on the unit test fixup | 19:39 |
| TheJulia | I tossed "or suppressed" on the comment, but *shrugs* | 19:39 |
| *** fragatina has quit IRC | 19:39 | |
| jroll | there's 2 +2s on the tox.ini change ahead of it, too, if someone wants to land that chain | 19:39 |
| jroll | TheJulia: that's probably a better word | 19:39 |
| * jroll good with either | 19:39 | |
| vdrok | :) can leave for the weekend with clear conscience | 19:39 |
| vdrok | I'll land that | 19:39 |
| jroll | cool | 19:40 |
| * jroll wonders if nisha got the ilo tests fixed up | 19:40 | |
| openstackgerrit | Julia Kreger proposed openstack/python-ironicclient master: Print pecan exceptions properly https://review.openstack.org/364566 | 19:40 |
| TheJulia | changed the word ^ | 19:40 |
| jroll | +A | 19:41 |
| vdrok | good night everyone! | 19:41 |
| jroll | happy weekend vdrok \o | 19:41 |
| * jroll tests this ilo patch then gets back to reno | 19:41 | |
| TheJulia | goodnight vdrok | 19:41 |
| *** mgoddard has quit IRC | 19:43 | |
| openstackgerrit | Merged openstack/ironic master: Add a tox target for unit tests with driver libraries https://review.openstack.org/432317 | 19:45 |
| openstackgerrit | Merged openstack/ironic master: Mock client initializations for irmc and oneview https://review.openstack.org/432424 | 19:47 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Add py35 to tox env list https://review.openstack.org/432438 | 19:51 |
| pas-ha | TheJulia: I've prepped another patch-set with pip install --user, and will try to fix reno too, so please do not push anything there for now ;) | 19:52 |
| TheJulia | pas-ha: awesome! | 19:53 |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent master: Collect NIC name given by BIOS https://review.openstack.org/396406 | 19:57 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Fix unit tests with UcsSdk installed https://review.openstack.org/432441 | 20:00 |
| jroll | ^ easy peasy | 20:00 |
| jroll | I also just approved the one fixing ilo tests, so that should be everything broken by installing real driver-requirements | 20:00 |
| * TheJulia looks at it and blinks | 20:03 | |
| jroll | right? :) | 20:05 |
| TheJulia | Looks fine to me, just kind of... unexpected | 20:07 |
| jroll | TheJulia: yeah, I think it's just a coding error. here's the commit that exception was introduced, so I'm confident it never had a node parameter: https://github.com/CiscoUcs/UcsPythonSDK/commit/0ae01cd2b34d2b7ca0e0fb5e717ba98d462bbe95#diff-8be92dd09a224e25e1bf6582dd32847aR38 | 20:09 |
| TheJulia | Awesome | 20:11 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Always install Ansible with pip https://review.openstack.org/431614 | 20:12 |
| pas-ha | TheJulia: done^. Now waiting for Jenkins... | 20:13 |
| TheJulia | \o/ | 20:13 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Clean up release notes for 7.0.0 https://review.openstack.org/431188 | 20:17 |
| *** Sukhdev has quit IRC | 20:17 | |
| jroll | mariojv: wondering if I should squash your patch into mine (reviewing it now) | 20:18 |
| jroll | just so it's clear about everything that's changing at once | 20:19 |
| mariojv | fine by me | 20:22 |
| mariojv | but then you can't +2 it :) | 20:22 |
| jroll | ¯\_(ツ)_/¯ | 20:22 |
| *** amoralej is now known as amoralej|off | 20:26 | |
| pas-ha | watching Jenkins online - it passed :) please proof-read my latest text changes to README and release note if you have time | 20:27 |
| pas-ha | TheJulia: ^ | 20:27 |
| TheJulia | Will do shortly | 20:28 |
| *** sdake_ has quit IRC | 20:31 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Print pecan exceptions properly https://review.openstack.org/364566 | 20:37 |
| *** baoli has quit IRC | 20:40 | |
| *** baoli has joined #openstack-ironic | 20:40 | |
| pas-ha | TheJulia: I'm calling it a day, there are some comments re typos etc already, feel free to fix those :) | 20:46 |
| pas-ha | have a nice weekend all :) | 20:47 |
| TheJulia | pas-ha: was just looking at them | 20:47 |
| jroll | later pas-ha | 20:47 |
| *** jcoufal__ has quit IRC | 20:49 | |
| *** ijw has quit IRC | 20:53 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Clean up release notes for 7.0.0 https://review.openstack.org/431188 | 20:53 |
| *** baoli has quit IRC | 20:53 | |
| * jroll takes a short break, that is almost ready | 20:54 | |
| *** baoli has joined #openstack-ironic | 20:54 | |
| *** fragatina has joined #openstack-ironic | 20:56 | |
| *** jcoufal has joined #openstack-ironic | 20:57 | |
| *** aNuposic has quit IRC | 20:59 | |
| *** racedo has quit IRC | 20:59 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Always install Ansible with pip https://review.openstack.org/431614 | 21:08 |
| *** crushil has quit IRC | 21:17 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent master: Collect NIC name given by BIOS https://review.openstack.org/396406 | 21:17 |
| *** crushil has joined #openstack-ironic | 21:18 | |
| *** sacharya has quit IRC | 21:18 | |
| *** sacharya has joined #openstack-ironic | 21:18 | |
| *** Sukhdev has joined #openstack-ironic | 21:21 | |
| *** crushil has quit IRC | 21:23 | |
| *** bfournie has quit IRC | 21:25 | |
| *** bfournie has joined #openstack-ironic | 21:25 | |
| *** bfournie has quit IRC | 21:25 | |
| *** dprince has quit IRC | 21:27 | |
| *** sdake has joined #openstack-ironic | 21:29 | |
| openstackgerrit | Merged openstack/ironic master: Follow up patch for SNMPv3 support https://review.openstack.org/432375 | 21:31 |
| *** sdake has quit IRC | 21:32 | |
| *** crushil has joined #openstack-ironic | 21:35 | |
| *** dsneddon_ has joined #openstack-ironic | 21:36 | |
| *** jcoufal has quit IRC | 21:41 | |
| *** crushil has quit IRC | 21:43 | |
| *** crushil has joined #openstack-ironic | 21:46 | |
| *** aNuposic has joined #openstack-ironic | 21:46 | |
| *** jcoufal has joined #openstack-ironic | 21:51 | |
| *** alexpilo_ has quit IRC | 21:51 | |
| *** vsaienko has quit IRC | 21:57 | |
| *** mgoddard has joined #openstack-ironic | 21:58 | |
| *** jcoufal_ has joined #openstack-ironic | 21:59 | |
| *** fragatina has quit IRC | 21:59 | |
| *** jcoufal__ has joined #openstack-ironic | 22:01 | |
| *** jcoufal has quit IRC | 22:02 | |
| *** jcoufal_ has quit IRC | 22:04 | |
| *** ijw has joined #openstack-ironic | 22:04 | |
| *** alexpilotti has joined #openstack-ironic | 22:07 | |
| *** jcoufal__ has quit IRC | 22:09 | |
| *** baoli has quit IRC | 22:11 | |
| *** crushil has quit IRC | 22:13 | |
| *** trown is now known as trown|outtypewww | 22:19 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Clean up release notes for 7.0.0 https://review.openstack.org/431188 | 22:23 |
| jroll | ^ finally done | 22:23 |
| *** aNuposic has quit IRC | 22:23 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Clean up release notes for 7.0.0 https://review.openstack.org/431188 | 22:24 |
| jroll | and rebased | 22:24 |
| *** causten has joined #openstack-ironic | 22:24 | |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic master: WIP/DNM: Baremetal Devstack Network Simulation Developer Doc https://review.openstack.org/422582 | 22:25 |
| *** baoli has joined #openstack-ironic | 22:31 | |
| *** dsneddon is now known as dsneddon_afk | 22:37 | |
| *** jcoufal has joined #openstack-ironic | 22:39 | |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic master: WIP/DNM: Baremetal Devstack Network Simulation Developer Doc https://review.openstack.org/422582 | 22:40 |
| *** jcoufal has quit IRC | 22:42 | |
| *** jtomasek has quit IRC | 22:43 | |
| *** rbudden has quit IRC | 22:52 | |
| *** mgoddard has quit IRC | 22:54 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Reapply doesn't update started_at time https://review.openstack.org/431053 | 23:03 |
| *** ijw has quit IRC | 23:06 | |
| *** ijw has joined #openstack-ironic | 23:07 | |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic master: WIP/DNM: Baremetal Devstack Network Simulation Developer Doc https://review.openstack.org/422582 | 23:11 |
| *** jtomasek has joined #openstack-ironic | 23:12 | |
| *** jaosorior has quit IRC | 23:15 | |
| *** cdearborn has quit IRC | 23:19 | |
| *** jkilpatr has quit IRC | 23:30 | |
| *** crushil has joined #openstack-ironic | 23:35 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent master: Collect NIC name given by BIOS https://review.openstack.org/396406 | 23:41 |
| *** rbudden has joined #openstack-ironic | 23:42 | |
| *** amarao has quit IRC | 23:45 | |
| *** adreznec has quit IRC | 23:47 | |
| *** adreznec has joined #openstack-ironic | 23:47 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
