| *** amotoki has quit IRC | 00:01 | |
| *** baoli has joined #openstack-ironic | 00:16 | |
| *** mtanino has joined #openstack-ironic | 00:19 | |
| *** mtanino has quit IRC | 00:26 | |
| *** baoli has quit IRC | 00:29 | |
| *** jkilpatr has quit IRC | 00:31 | |
| *** jkilpatr has joined #openstack-ironic | 00:31 | |
| *** oomichi has quit IRC | 00:39 | |
| *** oomichi has joined #openstack-ironic | 00:42 | |
| *** SerenaFeng has joined #openstack-ironic | 00:57 | |
| *** amotoki has joined #openstack-ironic | 01:04 | |
| *** yuanying_ has joined #openstack-ironic | 01:04 | |
| *** yuanying has quit IRC | 01:07 | |
| *** phuongnh has joined #openstack-ironic | 01:13 | |
| *** gcb has joined #openstack-ironic | 01:15 | |
| *** phuongnh has quit IRC | 01:19 | |
| *** amotoki has quit IRC | 01:27 | |
| *** tuanluong has joined #openstack-ironic | 01:37 | |
| *** tovin07 has joined #openstack-ironic | 01:41 | |
| *** sbezverk has quit IRC | 01:53 | |
| *** oomichi has quit IRC | 02:08 | |
| *** oomichi has joined #openstack-ironic | 02:12 | |
| *** hw_wutianwei has joined #openstack-ironic | 02:23 | |
| *** yuanying has joined #openstack-ironic | 02:30 | |
| *** yuanying_ has quit IRC | 02:33 | |
| openstackgerrit | Ma Xinjian proposed openstack/python-ironicclient master: Fix or Add parameter comments https://review.openstack.org/448992 | 02:39 |
|---|---|---|
| *** amotoki has joined #openstack-ironic | 02:42 | |
| *** amotoki has quit IRC | 02:53 | |
| *** cdearborn has quit IRC | 02:59 | |
| *** amotoki has joined #openstack-ironic | 03:06 | |
| *** links has joined #openstack-ironic | 03:09 | |
| *** Syed__ has quit IRC | 03:25 | |
| *** amotoki has quit IRC | 03:31 | |
| *** gcb has quit IRC | 03:33 | |
| *** joanna has quit IRC | 03:37 | |
| *** mpotdar has quit IRC | 03:38 | |
| openstackgerrit | Julian Edwards proposed openstack/ironic master: Don't retry power status if power action fails https://review.openstack.org/449454 | 03:40 |
| *** mpotdar has joined #openstack-ironic | 03:43 | |
| *** SerenaFeng has quit IRC | 03:47 | |
| *** sbezverk has joined #openstack-ironic | 03:50 | |
| *** amotoki has joined #openstack-ironic | 03:52 | |
| *** gcb has joined #openstack-ironic | 03:58 | |
| *** udesale has joined #openstack-ironic | 04:09 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:11 | |
| openstackgerrit | Julian Edwards proposed openstack/ironic master: Don't retry power status if power action fails https://review.openstack.org/449454 | 04:13 |
| *** vmud213 has joined #openstack-ironic | 04:14 | |
| *** udesale has quit IRC | 04:17 | |
| *** udesale has joined #openstack-ironic | 04:18 | |
| ianw | if anyone owns "Dell Ironic CI" it seems to be fairly unhappy -> see merge comments on https://review.openstack.org/#/c/450049/ | 04:25 |
| *** rbudden has quit IRC | 04:26 | |
| openstackgerrit | Tovin Seven proposed openstack/ironic master: Enable OSProfiler support in Ironic https://review.openstack.org/358142 | 04:28 |
| *** dsneddon has quit IRC | 04:46 | |
| *** causten has joined #openstack-ironic | 04:48 | |
| *** causten has quit IRC | 04:48 | |
| *** causten has joined #openstack-ironic | 04:48 | |
| *** dsneddon has joined #openstack-ironic | 04:51 | |
| *** crushil has quit IRC | 05:03 | |
| *** yogi has joined #openstack-ironic | 05:07 | |
| *** rcernin has joined #openstack-ironic | 05:26 | |
| *** ChubYann has quit IRC | 05:32 | |
| *** SerenaFeng has joined #openstack-ironic | 05:36 | |
| *** mjura has joined #openstack-ironic | 05:43 | |
| *** causten has quit IRC | 05:49 | |
| *** causten has joined #openstack-ironic | 05:51 | |
| *** oomichi has quit IRC | 05:58 | |
| *** oomichi has joined #openstack-ironic | 06:01 | |
| *** MarginHu has joined #openstack-ironic | 06:03 | |
| *** jaosorior has joined #openstack-ironic | 06:13 | |
| *** moshele has joined #openstack-ironic | 06:19 | |
| *** oomichi has quit IRC | 06:28 | |
| *** oomichi has joined #openstack-ironic | 06:32 | |
| *** oomichi has quit IRC | 06:39 | |
| *** pmannidi has quit IRC | 06:41 | |
| *** oomichi has joined #openstack-ironic | 06:43 | |
| *** tuanluong has quit IRC | 06:50 | |
| *** pmannidi has joined #openstack-ironic | 06:54 | |
| *** ralonsoh has joined #openstack-ironic | 06:55 | |
| *** jaosorior has quit IRC | 06:58 | |
| *** oomichi has quit IRC | 06:58 | |
| *** oomichi has joined #openstack-ironic | 07:03 | |
| *** vikrant has joined #openstack-ironic | 07:04 | |
| *** fxpester has joined #openstack-ironic | 07:06 | |
| *** SerenaFeng has quit IRC | 07:08 | |
| *** oomichi has quit IRC | 07:08 | |
| MarginHu | hi, I don't know how to layout the Bare Metal service when openstack is a multi-nodes envioment. | 07:10 |
| *** oomichi has joined #openstack-ironic | 07:13 | |
| *** jaosorior has joined #openstack-ironic | 07:18 | |
| *** pcaruana has joined #openstack-ironic | 07:20 | |
| *** fragatina has quit IRC | 07:26 | |
| *** fragatina has joined #openstack-ironic | 07:27 | |
| *** tuanluong has joined #openstack-ironic | 07:31 | |
| *** moshele has quit IRC | 07:32 | |
| pas-ha | MarginHu: what exactly is your problem? | 07:32 |
| pas-ha | and good morning ironic :) | 07:33 |
| MarginHu | hi pas-ha , good morning. | 07:33 |
| MarginHu | My question is that how to layout the baremetal service on a multi-node openstack. | 07:34 |
| MarginHu | which is best-practise? | 07:34 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Fail deploy if no pxe_enabled ports exist https://review.openstack.org/450104 | 07:35 |
| *** oomichi has quit IRC | 07:38 | |
| *** chas has joined #openstack-ironic | 07:41 | |
| *** oomichi has joined #openstack-ironic | 07:43 | |
| *** tesseract has joined #openstack-ironic | 07:44 | |
| *** amarao has joined #openstack-ironic | 07:44 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Add /baremetal path instead of port 6385 https://review.openstack.org/444337 | 07:44 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi https://review.openstack.org/437299 | 07:45 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Add new transaction starting -> error on timeout https://review.openstack.org/437377 | 07:47 |
| *** aarefiev_afk is now known as aarefiev | 07:47 | |
| aarefiev | morning all! | 07:47 |
| *** Serlex has joined #openstack-ironic | 07:47 | |
| *** pmannidi has quit IRC | 07:52 | |
| pas-ha | MarginHu: depends on other things.. the layout I think best is: 1) ironic-api services together with other APIs; 2) ironic-conductors on separate hosts, to minimize number of hosts plugged into the out-of-band/ipmi network; 3) might consider putting a separate pool of ironic-apis to serve the requests from nodes on provisioning/cleaning network (set policies | 07:52 |
| pas-ha | on them to only allow these endpoints, and configure ironic-conductors appropriately, block the lookup/heartbeat endpoints in the "public"-facing ironic-api services) | 07:52 |
| pas-ha | actually jroll promised to devise a 'reference' deployment diagram for ironic :) | 07:53 |
| MarginHu | :), thanks, usually I have 3 controllers which run all kinds of service except nova-compute | 07:55 |
| *** amoralej|off is now known as amoralej | 07:55 | |
| *** moshele has joined #openstack-ironic | 07:56 | |
| MarginHu | ironic-api service are put on those controllers, and ironic-conductors are put on compute nodes | 07:56 |
| MarginHu | is it ok? | 07:57 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:01 | |
| *** makowals has quit IRC | 08:03 | |
| *** makowals has joined #openstack-ironic | 08:04 | |
| *** kromanenko has joined #openstack-ironic | 08:06 | |
| *** dimak has joined #openstack-ironic | 08:09 | |
| *** oanson has joined #openstack-ironic | 08:10 | |
| oanson | Hi. is 'Dell Ironic CI' somehow related to ironic? | 08:11 |
| *** athomas has joined #openstack-ironic | 08:16 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector stable/ocata: DNM test gates with disabled *_ssh drivers https://review.openstack.org/450114 | 08:18 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector stable/newton: DNM test gates with disabled *_ssh drivers https://review.openstack.org/450116 | 08:19 |
| *** moshele has quit IRC | 08:25 | |
| *** stendulker has joined #openstack-ironic | 08:29 | |
| *** lucas-afk is now known as lucasagomes | 08:32 | |
| lucasagomes | morning all | 08:32 |
| *** openstackgerrit has quit IRC | 08:33 | |
| stendulker | morning lucasagomes o/ | 08:34 |
| oanson | Morning | 08:35 |
| oanson | Is 'Dell Ironic CI' somehow related to ironic? | 08:35 |
| oanson | Or is it unrelated? | 08:35 |
| *** racedo has joined #openstack-ironic | 08:36 | |
| *** derekh has joined #openstack-ironic | 08:37 | |
| lucasagomes | oanson, it is, it's a 3rd party CI running the integration tests using the dell drivers (*_drac) | 08:37 |
| oanson | I see. It's failing on in Dragonflow patches. | 08:38 |
| oanson | e.g. here: https://review.openstack.org/#/c/449208/ on PS4 | 08:38 |
| lucasagomes | oanson, yeah I think it's not working for some time now :-/ | 08:39 |
| oanson | There are no logs, so I don't have any more info | 08:39 |
| oanson | Just wanted to make sure you guys know | 08:39 |
| lucasagomes | oanson, thanks yeah, this is run by dell so we need them to fix it | 08:39 |
| lucasagomes | as far as I remember from the last upstream meeting they are aware of it | 08:40 |
| lucasagomes | oanson, so no worries about it on ur patch for now | 08:40 |
| oanson | All right. Thanks. | 08:40 |
| *** chas has quit IRC | 08:44 | |
| *** ccamacho has joined #openstack-ironic | 08:44 | |
| *** openstackgerrit has joined #openstack-ironic | 08:50 | |
| openstackgerrit | Ji Chengke proposed openstack/ironic master: Use HostAddressOpt for opts that accept IP and hostnames https://review.openstack.org/448002 | 08:50 |
| *** mgoddard has joined #openstack-ironic | 08:50 | |
| tuanluong | Good Morning lucasagomes, stendulker, oanson and Ironic's members | 09:05 |
| stendulker | Good Morning tuanluong o/ | 09:06 |
| vdrok | good morning all, pas-ha aarefiev lucasagomes stendulker | 09:07 |
| stendulker | Good morning vdrok o/ | 09:07 |
| tuanluong | Hi stendulker, I plan to implement to support Secure Boot for baremental | 09:08 |
| *** SerenaFeng has joined #openstack-ironic | 09:08 | |
| stendulker | tuanluong: ok. For which driver? | 09:09 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add DB layer support for node tags filters https://review.openstack.org/272624 | 09:10 |
| vdrok | and morning tuanluong | 09:10 |
| lucasagomes | o/ | 09:11 |
| openstackgerrit | Ji Chengke proposed openstack/ironic master: Use HostAddressOpt for opts that accept IP and hostnames https://review.openstack.org/448002 | 09:23 |
| tuanluong | morning vdrok | 09:26 |
| *** udesale__ has joined #openstack-ironic | 09:28 | |
| *** udesale has quit IRC | 09:28 | |
| *** udesale__ has quit IRC | 09:36 | |
| *** udesale has joined #openstack-ironic | 09:36 | |
| *** bcornec has joined #openstack-ironic | 09:36 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Add /baremetal path instead of port 6385 https://review.openstack.org/444337 | 09:43 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi https://review.openstack.org/437299 | 09:43 |
| *** chas has joined #openstack-ironic | 09:44 | |
| *** MarginHu has quit IRC | 09:45 | |
| *** MarginHu has joined #openstack-ironic | 09:45 | |
| *** MarginHu has quit IRC | 09:47 | |
| *** chas has quit IRC | 09:49 | |
| *** sambetts_ is now known as sambetts | 09:51 | |
| sambetts | Morning all | 09:51 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Call clean_up_instance() during node teardown for Agent deploy https://review.openstack.org/450146 | 09:51 |
| sambetts | I've been gone for a while traveling and then on PTO I hope I've not missed too much | 09:52 |
| *** SerenaFeng has quit IRC | 09:56 | |
| vdrok | morning sambetts , welcome back :) | 09:56 |
| *** Nisha_Agarwal has quit IRC | 09:59 | |
| lucasagomes | sambetts, wb! | 10:02 |
| sambetts | Hey vdrok, lucasagomes o/! | 10:02 |
| *** Nisha_Agarwal has joined #openstack-ironic | 10:12 | |
| *** MattMan has quit IRC | 10:13 | |
| *** MattMan has joined #openstack-ironic | 10:14 | |
| vsaienk0 | morning Ironic'ers! | 10:17 |
| sambetts | hey vsaienk0 | 10:18 |
| mgoddard | morning vsaienko & other ironic folk | 10:18 |
| thomas_oneill | morning all - bifrost folk, is there a way to do a big deployment of lots of nodes all at once? I notice BIFROST_INVENTORY_SOURCE seems to have to point to a source for a single node | 10:20 |
| sambetts | thomas_oneill: the bifrost inventory can have multiple nodes in in | 10:21 |
| sambetts | it | 10:21 |
| openstackgerrit | Mark Goddard proposed openstack/bifrost master: Allow undionly.kpxe boot option to be overridden https://review.openstack.org/449085 | 10:21 |
| thomas_oneill | oh cool | 10:21 |
| openstackgerrit | Mark Goddard proposed openstack/bifrost master: Allow undionly.kpxe boot option to be overridden https://review.openstack.org/449085 | 10:22 |
| *** tovin07 has quit IRC | 10:27 | |
| *** milan has joined #openstack-ironic | 10:29 | |
| milan | morning Ironic! :) | 10:30 |
| *** amotoki has quit IRC | 10:30 | |
| *** udesale has quit IRC | 10:31 | |
| *** moshele has joined #openstack-ironic | 10:32 | |
| *** yatinkarel has quit IRC | 10:34 | |
| *** jjung has joined #openstack-ironic | 10:35 | |
| *** deadnull has quit IRC | 10:37 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector stable/newton: Switch to use test_utils.call_until_true https://review.openstack.org/450158 | 10:47 |
| *** yatinkarel has joined #openstack-ironic | 10:47 | |
| *** zackf has joined #openstack-ironic | 10:49 | |
| *** e0ne has joined #openstack-ironic | 10:50 | |
| *** nmathew has joined #openstack-ironic | 10:50 | |
| *** yatinkarel has quit IRC | 10:53 | |
| *** Nisha_Agarwal has quit IRC | 10:55 | |
| *** mat128 has joined #openstack-ironic | 10:58 | |
| openstackgerrit | Merged openstack/sushy master: Sushy to adhere to the resource identifier portion of the spec https://review.openstack.org/448539 | 11:00 |
| *** lucasagomes is now known as lucas-hungry | 11:00 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: WIP: SSL TEST https://review.openstack.org/442402 | 11:02 |
| *** ccamacho is now known as ccamacho|lunch | 11:02 | |
| *** yatinkarel has joined #openstack-ironic | 11:05 | |
| *** bcornec has quit IRC | 11:10 | |
| *** moshele has quit IRC | 11:11 | |
| *** nmathew has quit IRC | 11:12 | |
| *** garyk has joined #openstack-ironic | 11:20 | |
| garyk | EmilienM: i am trying to get triplo up and running and am hitting issues with pxe_shh having a timeout. Any idea how i can debug this? | 11:21 |
| *** yatinkarel has quit IRC | 11:24 | |
| *** zackf has quit IRC | 11:24 | |
| *** gcb has quit IRC | 11:25 | |
| *** stendulker has quit IRC | 11:26 | |
| *** vmud213 has quit IRC | 11:27 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add nodes tagging support - API https://review.openstack.org/250478 | 11:27 |
| *** bcornec has joined #openstack-ironic | 11:28 | |
| *** zhurong has joined #openstack-ironic | 11:32 | |
| *** andreykurilin has joined #openstack-ironic | 11:36 | |
| *** yatinkarel has joined #openstack-ironic | 11:36 | |
| *** snecklifter has quit IRC | 11:37 | |
| andreykurilin | rajinir, cdearborn: hi | 11:37 |
| andreykurilin | rajinir, cdearborn: `Dell Ironic CI` started to push messages to openstack/rally project for every change. | 11:38 |
| andreykurilin | "Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset." | 11:38 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Add /baremetal path instead of port 6385 https://review.openstack.org/444337 | 11:38 |
| andreykurilin | I can assume that it is wrong behaviour :) | 11:39 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi https://review.openstack.org/437299 | 11:39 |
| *** bfournie has quit IRC | 11:44 | |
| *** amotoki has joined #openstack-ironic | 11:46 | |
| ricardoas | morning! | 11:46 |
| *** mgould|afk is now known as mgould | 11:49 | |
| mgould | morning everyone! | 11:49 |
| mgould | garyk: pxe_ssh is deprecated on the Ironic side, but I'm not sure what the state of support for the alternatives is in tripleo: https://blueprints.launchpad.net/tripleo/+spec/switch-to-virtualbmc | 11:50 |
| mgould | oh wait, Status: Complete | 11:51 |
| mgould | so I think "don't use pxe_ssh" is the answer here | 11:51 |
| * mgould can read, honest | 11:51 | |
| garyk | mgould: ok, thanks! so what should i use to try and get the virtual interface to work | 11:51 |
| EmilienM | garyk: I would check with mgould or dtantsur on this one | 11:52 |
| garyk | ok, thanks. | 11:54 |
| mgould | garyk: are you using tripleo-quickstart? | 11:54 |
| *** jaypipes has joined #openstack-ironic | 11:55 | |
| garyk | mgould: i am using director - I am not sure if that is triple-quickstart | 11:55 |
| garyk | i am pretty new to tripleo so i am learning the ropes at the moment | 11:55 |
| *** tuanluong has quit IRC | 11:56 | |
| mgould | there are a lot of ropes to learn | 11:57 |
| mgould | garyk: which version? | 11:57 |
| *** kromanenko has quit IRC | 11:58 | |
| garyk | mgould: 10 (i think) | 11:58 |
| mgould | OK, I think that's based on quickstart | 12:01 |
| *** rbudden has joined #openstack-ironic | 12:02 | |
| garyk | ok, thanks | 12:02 |
| mgould | garyk: do you have a link to the instructions you're using? | 12:02 |
| *** lucas-hungry is now known as lucasagomes | 12:03 | |
| garyk | i have director installed on one instance and would like it to deploy images on another instance that is running libvirt (my understanding is that the deployment should work with libvirt). is that correct? | 12:03 |
| garyk | that is whay I asked about the pxe_ssh | 12:03 |
| mgould | right, OK | 12:03 |
| mgould | that should work, I think | 12:03 |
| mgould | lucasagomes ^^^ | 12:03 |
| * lucasagomes reads | 12:05 | |
| lucasagomes | garyk, mgould hi... yeah that should be fine | 12:06 |
| mgould | garyk: did you tell it to use pxe_ssh, or was that the default? | 12:06 |
| garyk | mgould: i think i told it to. I am trying again | 12:07 |
| mgould | if there's an option to use pxe_ipmitool+virtualbmc, try that instead | 12:07 |
| garyk | what do I need to set for the power_manager? | 12:07 |
| mgould | what are the options? | 12:07 |
| garyk | ok, will try that | 12:08 |
| *** yatinkarel has quit IRC | 12:08 | |
| garyk | mgould: not sure how i can get the options. i saw that written in one online doc - https://keithtenzer.com/2015/10/14/howto-openstack-deployment-using-tripleo-and-the-red-hat-openstack-director/ | 12:08 |
| garyk | that may be a little outdated | 12:08 |
| mgould | Oct 2015? yeah, probably | 12:09 |
| mgould | hmmm, I can't see anything about virtualbmc in the OSPd 10 docs | 12:09 |
| *** trown|outtypewww is now known as trown | 12:10 | |
| *** dprince has joined #openstack-ironic | 12:10 | |
| mgould | hi trown | 12:10 |
| mgould | trown: garyk is trying to install OpenStack using OSPd 10, and having trouble with pxe_ssh timeouts | 12:11 |
| mgould | should pxe_ssh work in that context? | 12:11 |
| mgould | garyk: I'd check that you can ssh to your libvirt host with the credentials you supply and run `virsh list` | 12:12 |
| mgould | because that's basically what pxe_ssh does | 12:12 |
| trown | mgould: quickstart is not part of any downstream product (at least not yet) | 12:12 |
| *** zhurong has quit IRC | 12:12 | |
| mgould | trown: OK, thanks! | 12:13 |
| garyk | ok, will try with the visrh list. thanks | 12:13 |
| trown | mgould: but afaik pxe_ssh should work on any of the downstream versions ... is it even removed from Ironic now? | 12:13 |
| mgould | no, but it is deprecated | 12:13 |
| trown | ya, thats what I though | 12:13 |
| trown | t | 12:13 |
| mgould | I was looking at https://blueprints.launchpad.net/tripleo/+spec/switch-to-virtualbmc, and support for virtualbmc seems to depend on oooq | 12:14 |
| trown | quickstart switched to vbmc thanks to lucasagomes | 12:14 |
| mgould | garyk: OK, forget everything I said about quickstart and virtualbmc | 12:14 |
| trown | ya, nobody plans to add it to instack-virt-setup which is also deprecated | 12:14 |
| garyk | mgould: ok, thanks | 12:14 |
| mgould | check the ssh + virsh thing, and if that works please paste your pxe_ssh error to paste.openstack.org | 12:15 |
| mgould | trown: is there a better channel for garyk to ask in? | 12:15 |
| *** amoralej is now known as amoralej|lunch | 12:16 | |
| *** ccamacho|lunch is now known as ccamacho | 12:16 | |
| garyk | mgould: thanks - at the moment i am in the process of getting my KVM host up so that I can run the overcloud. Will send you the data when I have it | 12:17 |
| mgould | garyk: good luck! | 12:17 |
| trown | mgould: here or #tripleo make sense to me... it is a bit tough because nobody is really "supporting" pxe_ssh upstream | 12:17 |
| garyk | gracias | 12:17 |
| mgould | trown: thanks! | 12:20 |
| *** bfournie has joined #openstack-ironic | 12:20 | |
| *** yatinkarel has joined #openstack-ironic | 12:20 | |
| lucasagomes | hi all, anyone else wants to take a look at https://review.openstack.org/#/c/445478/ ? It already has 2+2 and the work in sushy is already being done to accommodate the changes | 12:24 |
| pas-ha | garyk: also, if you are running 'virtual' baremetal, you could consider libvirt power/mgmt driver interfaces from ironic-staging-drivers, scales much better than *_ssh ones | 12:24 |
| lucasagomes | thanks :D | 12:24 |
| garyk | pas-ha: thanks. do you have doc or reference that i can look at? | 12:25 |
| pas-ha | garyk: http://ironic-staging-drivers.readthedocs.io/en/latest/drivers/libvirt.html | 12:26 |
| *** causten has joined #openstack-ironic | 12:27 | |
| *** yatinkarel has quit IRC | 12:28 | |
| *** crushil has joined #openstack-ironic | 12:33 | |
| openstackgerrit | Miles Gould proposed openstack/ironic master: devstack: install python-dracclient if DRAC enabled https://review.openstack.org/425326 | 12:38 |
| *** links has quit IRC | 12:40 | |
| *** Goneri has joined #openstack-ironic | 12:41 | |
| *** yatinkarel has joined #openstack-ironic | 12:42 | |
| *** chlong has joined #openstack-ironic | 12:51 | |
| amarao | Hi guys. Yet another bug with config-drive. Now with pxe_* drivers: https://bugs.launchpad.net/ironic/+bug/1676422 | 12:59 |
| openstack | Launchpad bug 1676422 in Ironic "Node failed at deploy. Exception: int() argument must be a string or a number, not 'tuple'" [Undecided,New] | 12:59 |
| *** amoralej|lunch is now known as amoralej | 13:00 | |
| *** baoli has joined #openstack-ironic | 13:03 | |
| *** sasha2 has joined #openstack-ironic | 13:03 | |
| *** jcoufal has joined #openstack-ironic | 13:05 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add manual clean step ironic standalone test https://review.openstack.org/429770 | 13:06 |
| aarefiev | milan: hey, when you have time https://review.openstack.org/#/c/444910/1 we need to decide something with api in unit tests | 13:07 |
| milan | aarefiev, hi | 13:07 |
| * milan looks | 13:07 | |
| aarefiev | milan: I know looks ugly, maybe you have some ideas | 13:08 |
| jroll | morning y'all | 13:09 |
| milan | morning jroll! :) | 13:09 |
| aarefiev | morning jroll | 13:09 |
| milan | aarefiev, I know I might have a decorator-abuse condition, but it feels like one ;) | 13:10 |
| *** rloo has joined #openstack-ironic | 13:11 | |
| *** cdearborn has joined #openstack-ironic | 13:12 | |
| crushil | morning jroll | 13:12 |
| aarefiev | milan: something like http://paste.openstack.org/show/604297/ | 13:13 |
| milan | aarefiev, yeah, if we want to have an explicit version | 13:13 |
| milan | aarefiev, otherwise default to highest in the most generic method | 13:14 |
| milan | s/highest/current | 13:14 |
| garyk | mgould: pas-ha: trown: EmilienM: any idea what is wrong here - https://paste.fedoraproject.org/paste/zKOYF0zpkuCrQyn9Dj7BBl5M1UNdIGYhyRLivL9gydE= | 13:16 |
| aarefiev | milan: yeah, I guess it would be the highest in most cases | 13:16 |
| EmilienM | garyk: in a mtg, i'll look shortly | 13:17 |
| milan | aarefiev, didn't we already have similar discussion over a previous patch or is just my deja vu? | 13:17 |
| garyk | EmilienM: ok, thanks | 13:18 |
| EmilienM | garyk: but asking on #tripleo was a great idea, hopefuly someone from the team can help more quickly than me | 13:18 |
| aarefiev | milan: yeah, it was in deprecating finished key patch, so I'm trying to solve this issue | 13:18 |
| milan | right | 13:18 |
| garyk | EmilienM: ok, thanks | 13:18 |
| aarefiev | milan: ok, let me try to add decorator to version specific tests to see how it would look | 13:21 |
| milan | aarefiev, in my proposal to that patch I had a working decorator: https://review.openstack.org/#/c/436415/12..17/ironic_inspector/test/functional.py | 13:23 |
| milan | but someone thought that the micro version was data input to the test which I sort of disagree with | 13:24 |
| milan | and my solution was replaced w/ ddt | 13:24 |
| *** gcb has joined #openstack-ironic | 13:25 | |
| milan | IMO we're configuring a client here so the client's internal state should change | 13:25 |
| aarefiev | milan: you are pointing to functional tests | 13:25 |
| milan | yup but the idea is the same, no? | 13:25 |
| aarefiev | we agreed always use latest microversion for unit tests | 13:26 |
| aarefiev | by default | 13:26 |
| milan | aarefiev, ah, so maybe I don't follow why we need a custom versions then | 13:27 |
| milan | aarefiev, you mean that some unittest should use different micro version? | 13:27 |
| milan | like that boo test you pasted? | 13:27 |
| aarefiev | the problem we have version specific tests | 13:27 |
| aarefiev | maybe we should move them functional | 13:27 |
| aarefiev | to functional | 13:28 |
| aarefiev | milan: https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/test/unit/test_main.py#L76 | 13:28 |
| milan | aarefiev, I see. The problem might be more generic, i.e how to assert that we always maintain backwards compatibility | 13:31 |
| milan | aarefiev, if we agree that the unittests are always version agnostic (use latest) then this sort of tests should be integration-tested w/ tempest | 13:33 |
| aarefiev | milan: yeah, and I'm thinking now that I like this lightwess tests more | 13:35 |
| aarefiev | milan: the idea was use latest version by default, but allow to specify needed version anyway, with this removed `finished` key we don't need to add "if version < 1.9 do ... else ... | 13:41 |
| aarefiev | and use states on every new microversion | 13:41 |
| milan | aarefiev, sort of how to assert a feature was deprecated test case for tempest while always test just the latest in unittest&functional? | 13:42 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Rework VMs connection logic https://review.openstack.org/440719 | 13:46 |
| milan | aarefiev, I've got issues to even express how to test these things O:-) | 13:46 |
| *** garyk has quit IRC | 13:46 | |
| aarefiev | milan: it depends on feature, like this https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/test/unit/test_main.py#L74 | 13:46 |
| aarefiev | I don't want to write integration test for this | 13:47 |
| aarefiev | it's simple api verification | 13:47 |
| milan | hmmm | 13:47 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 13:47 |
| mariojv | good morning ironic | 13:48 |
| aarefiev | milan: if it would be something more complicated I'd create more complex functional test | 13:48 |
| TheJulia | Good morning everyone | 13:49 |
| aarefiev | morning mariojv, TheJulia | 13:49 |
| milan | morning mariojv, TheJulia! :) | 13:49 |
| vdrok | morning to everyone I've missed today, milan TheJulia mariojv jroll :) | 13:50 |
| milan | aarefiev, I see, but that unittest seems to be version dependant, but maybe it's OK | 13:50 |
| milan | morning vdrok! :) | 13:50 |
| aarefiev | milan; it's what we already have :) | 13:51 |
| rloo | hi and good morning milan, vdrok, aarefiev, TheJulia, mariojv | 13:51 |
| aarefiev | morning rloo | 13:51 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 13:51 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Rework VMs connection logic https://review.openstack.org/440719 | 13:51 |
| rloo | lucasagomes: any reason not to merge https://review.openstack.org/#/c/445478/? | 13:52 |
| milan | aarefiev, somehow I'd like to have the coverage that if I try use an unsupported API micro version I get a 4xx response in some test "flavours" (functional/integration/unit?) | 13:52 |
| vdrok | morning rloo :) | 13:52 |
| lucasagomes | rloo, hi checking | 13:52 |
| milan | morning rloo! :) | 13:52 |
| lucasagomes | rloo, oh that's the spec... well, no reason that I can point out | 13:52 |
| *** dyasny has joined #openstack-ironic | 13:52 | |
| lucasagomes | I've merged today the work in sushy to accomodate these changes | 13:52 |
| rloo | lucasagomes: ok, will +A then. | 13:53 |
| lucasagomes | I need to push a new patch set to the driver in Ironic which I'm waiting on that spec change | 13:53 |
| lucasagomes | rloo, thank you! | 13:53 |
| rloo | lucasagomes: thank YOU! :) | 13:53 |
| lucasagomes | btw folks, I haven't seem this error before: http://logs.openstack.org/77/449577/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-ubuntu-xenial/ee71558/logs/screen-ir-cond.txt.gz#_2017-03-24_17_11_42_387 | 13:53 |
| lucasagomes | looks unrelated to the patch... anyone has any idea ? | 13:53 |
| *** andreykurilin has left #openstack-ironic | 13:53 | |
| lucasagomes | derekh, ^ it's ur patch btw... it doesn't look related at all but the error is a bit surprising, I haven't seem that in the past AFAICT | 13:54 |
| vdrok | lucasagomes: yup, vsaienk0 has a patch iirc | 13:54 |
| *** gcb has quit IRC | 13:54 | |
| jroll | hrm, maybe the iscsi device doesn't exist? I feel like vsaienk0 was looking at this | 13:54 |
| vdrok | lemme find a link | 13:54 |
| lucasagomes | vdrok, oh right | 13:54 |
| jroll | ah yeah | 13:54 |
| aarefiev | milan: yeah, and looks like we are doing it only in unit tests | 13:54 |
| aarefiev | hmm | 13:54 |
| *** gcb has joined #openstack-ironic | 13:54 | |
| milan | aarefiev, my intuition would be to bump these assertions into integration tests | 13:55 |
| vsaienk0 | lucasagomes: https://review.openstack.org/#/c/446950/ this should solve the problem but we need to release ironic-lib I think | 13:55 |
| derekh | lucasagomes: ack, thanks | 13:55 |
| vdrok | yup, just needs a release I guess | 13:55 |
| aarefiev | milan: you mean this existing tests | 13:56 |
| vsaienk0 | I wondered why in master we still install released version of ironic-lib | 13:56 |
| openstackgerrit | Merged openstack/ironic-specs master: Follow-up patch for the Redfish spec https://review.openstack.org/445478 | 13:56 |
| *** mtanino has joined #openstack-ironic | 13:57 | |
| aarefiev | milan: or only new one | 13:57 |
| milan | aarefiev, hmm maybe as they miss the coverage of the negative condition? | 13:57 |
| milan | but unsure | 13:57 |
| lucasagomes | vsaienk0, vdrok jroll awesome! Thanks a lot so we just need a release now | 13:57 |
| jroll | oh cool, it merged | 13:58 |
| jroll | I was going to do some other releases this week too, I'll do them all this afternoon | 13:58 |
| milan | negative w/r the: "I run 1.8 request agains 1.9 api and I should get 4xx" test or some sort of @deprecated_in(1.9) | 13:58 |
| milan | aarefiev, ^ | 13:58 |
| aarefiev | milan: I guess that what we have now, no | 13:59 |
| milan | btw that sort of test cases should pass w/ a deprecation in 1.9 but fail w/ server API 1.10 | 14:00 |
| milan | aarefiev, if we do then never mind me | 14:00 |
| *** nmathew has joined #openstack-ironic | 14:00 | |
| *** Syed__ has joined #openstack-ironic | 14:01 | |
| *** SerenaFeng has joined #openstack-ironic | 14:01 | |
| *** nmathew has quit IRC | 14:01 | |
| milan | aarefiev, I still think that it shouldn't be a matter of unittest though but wouldn't -1 on it O-) | 14:01 |
| aarefiev | milan: you can't -1 because they are already landed :) | 14:03 |
| *** yufei has joined #openstack-ironic | 14:03 | |
| milan | aarefiev, OK :D | 14:03 |
| aarefiev | jroll: please, take a look on fixes for stable branches when you have time https://review.openstack.org/#/c/443878/1 https://review.openstack.org/#/c/425122/1 | 14:06 |
| * jroll looks | 14:07 | |
| aarefiev | thanks | 14:08 |
| jroll | and done | 14:08 |
| jroll | JayF: this needs landing when you're around https://review.openstack.org/#/c/425122/1 | 14:08 |
| *** yufei has quit IRC | 14:09 | |
| milan | aarefiev, I'm also wondering what's the point in running e.g https://review.openstack.org/#/c/444910/1/ironic_inspector/test/unit/test_main.py@85 in the future say 1.11 unless we declare unlimited backwards compatibility? | 14:09 |
| *** chlong has quit IRC | 14:09 | |
| milan | aarefiev, maybe we should at least mark those for removal? # TODO: remove me in 1.11? | 14:10 |
| *** zhurong has joined #openstack-ironic | 14:11 | |
| aarefiev | milan: it's not unlimited, but we should support it anyway till we say that min supported version 1.9 | 14:12 |
| *** rpioso has joined #openstack-ironic | 14:12 | |
| aarefiev | milan: the problem now is latest microversion differs from the default one | 14:12 |
| aarefiev | as result hell in tests :( | 14:13 |
| milan | aarefiev, right, basically we need different set of tests for different API versions w/ different configuration of the client (in this case just headers w/ proper data) | 14:16 |
| milan | aarefiev, moreover, we should assert that some tests expect a certain failure in certain version for the sake of coverage | 14:17 |
| milan | aarefiev, but I may be overthinking it :) | 14:17 |
| aarefiev | milan: right, but it also have impact on existing tests | 14:17 |
| milan | aarefiev, in my ideal world with lot's of happy decorators, I'd have a decorator that would care about those three points :) | 14:18 |
| milan | @api_version(introduced='1.8', deprecated='1.9', skipped='>1.9') | 14:19 |
| milan | aarefiev, but if there's a better way than a decorator, I won't -1 it ;) | 14:20 |
| milan | or will I? }:-) | 14:20 |
| aarefiev | milan: why do you need to skip | 14:20 |
| milan | not to run 1.9 test in 1.11? | 14:21 |
| aarefiev | you need only request needed version to server | 14:21 |
| milan | aarefiev, but if the code is removed from the server, should we remove the test case too? Oh it's unittests, so maybe we should, dunno | 14:22 |
| aarefiev | milan: if code is removed we should drop all related tests anyway | 14:23 |
| milan | aarefiev, functional and unittest true, maybe not so true for integration? | 14:23 |
| *** ricardoas has left #openstack-ironic | 14:23 | |
| milan | but yeah, we're talking unittests here | 14:24 |
| aarefiev | milan: yeah, tempest should cover previous releases | 14:25 |
| *** ccamacho is now known as ccamacho|brb | 14:25 | |
| milan | aarefiev, so @api_version(introduced=1.8, deprecated=1.9)? | 14:25 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: WIP: SSL TEST https://review.openstack.org/442402 | 14:27 |
| aarefiev | milan: something like that, but we don't have version specific tempest tests yet | 14:27 |
| *** vikrant has quit IRC | 14:27 | |
| aarefiev | milan: also we don't have mechanism for dropping old microversions, I guess jroll is working on this. | 14:29 |
| jroll | yeah, that's a ways off still | 14:30 |
| milan | aarefiev, http://paste.openstack.org/show/604319/ is my idea more or less | 14:30 |
| milan | ah, OK | 14:30 |
| *** athomas has quit IRC | 14:32 | |
| milan | aarefiev, http://paste.openstack.org/show/604321/ rather (missed the removed state) | 14:33 |
| aarefiev | milan: so, we are talking about unit tests now, right? | 14:35 |
| vdrok | would be cool to have a thing like that in our api too :) as right now it's really hard to read the api code. maybe will do an rfe | 14:35 |
| milan | aarefiev, right unittests | 14:35 |
| aarefiev | milan: I see how dtantsur -2 on this :) | 14:36 |
| milan | aarefiev, lol, what's so wrong about test decorator generating tests? ;) | 14:37 |
| *** superdan is now known as dansmith | 14:38 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add tempest plugin API tests for node https://review.openstack.org/411197 | 14:38 |
| *** athomas has joined #openstack-ironic | 14:39 | |
| *** zhurong has quit IRC | 14:41 | |
| openstackgerrit | Merged openstack/ironic-specs master: [RFE] Add spec for physical network awareness https://review.openstack.org/435781 | 14:42 |
| aarefiev | milan: we could always test default api version, and fix all unit tests once we bump version | 14:45 |
| *** trown is now known as trown|brb | 14:45 | |
| milan | aarefiev, yeah, lot's of type 3 fun though ;) | 14:46 |
| openstackgerrit | Lin Yang proposed openstack/sushy master: Parse generic attributions for redfish resource https://review.openstack.org/448834 | 14:46 |
| milan | I guess that's what we've been doing so far | 14:46 |
| aarefiev | yeah | 14:46 |
| *** trown|brb is now known as trown | 14:49 | |
| vsaienk0 | sambetts: could you please add to your review queue https://review.openstack.org/#/c/415003/ | 14:49 |
| aarefiev | milan: it would be fun for the one who bumps microversion :) | 14:49 |
| sambetts | vsaienk0: will do :) | 14:49 |
| *** fxpester has quit IRC | 14:49 | |
| milan | aarefiev, oh really? :P | 14:50 |
| vsaienk0 | sambetts: thanks! | 14:52 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client stable/ocata: Updated from global requirements https://review.openstack.org/443878 | 14:53 |
| *** udesale has joined #openstack-ironic | 14:54 | |
| aarefiev | milan: what if we add new functional tests for testing microversion and keep existing tests with default microversion | 14:55 |
| milan | aarefiev, it's OK just that I think those are somehow linked together w/ what conditions we expect them to work, be skipped & be failing | 14:56 |
| milan | we'd be loosing the link | 14:56 |
| milan | but if it's just the link we're loosing, maybe it's OK (save for the typing effort) ;) | 14:57 |
| *** zackf has joined #openstack-ironic | 14:57 | |
| *** Goneri has quit IRC | 14:57 | |
| openstackgerrit | Merged openstack/bifrost stable/ocata: Change dib element to enable serial console https://review.openstack.org/449547 | 14:59 |
| aarefiev | milan: we don't need to skip functional, ok, I'll try to play with this | 15:01 |
| *** Serlex has quit IRC | 15:03 | |
| milan | aarefiev, alright, we can bump it to some testing refactoring patch if it's too much to worry about right now | 15:03 |
| *** vgadiraj_ has joined #openstack-ironic | 15:04 | |
| *** ijw has joined #openstack-ironic | 15:05 | |
| * milan relocates -> CU later Ironic | 15:05 | |
| *** ijw_ has joined #openstack-ironic | 15:06 | |
| *** rcernin has quit IRC | 15:07 | |
| *** ijw has quit IRC | 15:09 | |
| *** joanna has joined #openstack-ironic | 15:10 | |
| *** milan has quit IRC | 15:10 | |
| *** Goneri has joined #openstack-ironic | 15:13 | |
| vgadiraj_ | Was taking a look at the spec for pike priorities and the hyper links and details for a couple of the essential priorities and the Redfish driver are missing. Is this by design/is there a reason for this? http://specs.openstack.org/openstack/ironic-specs/priorities/pike-priorities.html | 15:21 |
| NobodyCam | Good Morning Ironic'ers | 15:22 |
| lucasagomes | NobodyCam, morning | 15:23 |
| *** bcornec has quit IRC | 15:24 | |
| lucasagomes | vgadiraj, hi there, yeah it wasn't included in the original patch maybe because it's kinda obvious what it needs to be done there (it's just a basic driver for now) | 15:24 |
| lucasagomes | vgadiraj, but, I don't think that it would have any problem to put up another patch adding some text about it there | 15:24 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 15:25 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Agent driver rescue implementation https://review.openstack.org/400437 | 15:25 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 15:25 |
| NobodyCam | god morning lucasagomes | 15:25 |
| NobodyCam | good even :p | 15:25 |
| *** chlong has joined #openstack-ironic | 15:25 | |
| * NobodyCam goes back to coffee | 15:25 | |
| *** baha has joined #openstack-ironic | 15:26 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic master: Add redfish driver https://review.openstack.org/438982 | 15:26 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic master: Add documentation for the redfish driver https://review.openstack.org/442537 | 15:26 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic master: DevStack: Setup a Redfish environment https://review.openstack.org/445529 | 15:26 |
| *** dyasny has quit IRC | 15:28 | |
| openstackgerrit | Merged openstack/ironic-specs master: Remove exposing hardware capabilities backlog spec https://review.openstack.org/437674 | 15:32 |
| mgould | morning NobodyCam | 15:32 |
| * TheJulia wonders where the coffee IVs are located | 15:33 | |
| NobodyCam | Morning mogul and TheJulia :) | 15:33 |
| NobodyCam | gah mgould Even | 15:34 |
| NobodyCam | hehehe | 15:34 |
| mgould | :-) | 15:35 |
| NobodyCam | :p | 15:35 |
| *** chas has joined #openstack-ironic | 15:38 | |
| *** jjung has quit IRC | 15:38 | |
| *** chas_ has joined #openstack-ironic | 15:40 | |
| *** jjung has joined #openstack-ironic | 15:41 | |
| *** chas has quit IRC | 15:42 | |
| *** chas_ has quit IRC | 15:44 | |
| *** e0ne has quit IRC | 15:44 | |
| *** chas has joined #openstack-ironic | 15:46 | |
| *** udesale has quit IRC | 15:50 | |
| *** Serlex has joined #openstack-ironic | 15:52 | |
| *** yogi has quit IRC | 15:55 | |
| *** amotoki has quit IRC | 15:57 | |
| joanna | morning NobodyCam, mgould, lucasagomes | 15:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Rework VMs connection logic https://review.openstack.org/440719 | 15:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 16:00 |
| *** amarao has quit IRC | 16:02 | |
| mgould | morning joanna | 16:02 |
| *** vgadiraj_ has quit IRC | 16:02 | |
| *** milan has joined #openstack-ironic | 16:03 | |
| lucasagomes | joanna, morning | 16:03 |
| *** vgadiraj_ has joined #openstack-ironic | 16:03 | |
| *** aarefiev is now known as aarefiev_afk | 16:05 | |
| *** udesale has joined #openstack-ironic | 16:06 | |
| *** causten has quit IRC | 16:06 | |
| *** MapspaM is now known as SpamapS | 16:08 | |
| openstackgerrit | Merged openstack/bifrost master: Allow undionly.kpxe boot option to be overridden https://review.openstack.org/449085 | 16:10 |
| NobodyCam | morning joanna :) | 16:12 |
| *** cinerama has joined #openstack-ironic | 16:12 | |
| *** SerenaFeng has quit IRC | 16:12 | |
| jlvillal | So I don't know the answer to this question. Though I probably should. Are the Ironic nova driver and traditional nova-hypervisors supported in the same region? | 16:18 |
| vdrok | morning NobodyCam joanna mgould jlvillal ! | 16:20 |
| NobodyCam | morning vdrok :) | 16:20 |
| joanna | morning vdrok :) | 16:20 |
| jlvillal | Hi vdrok NobodyCam joanna lucasagomes mgould milan sambetts vsaienk0 jroll TheJulia dtantsur rloo and everyone else :) | 16:22 |
| vsaienk0 | morning jlvillal | 16:22 |
| jlvillal | Anyone know the answer to my question about? :) | 16:23 |
| rloo | Good Monday morning jlvillal, joanna, NobodyCam, vsaienk0 | 16:23 |
| milan | morning jlvillal, joanna, NobodyCam! :) | 16:23 |
| vdrok | jlvillal: ah, yeah, they are supported, aggregates can be used for that | 16:23 |
| lucasagomes | jlvillal, o/ morning | 16:23 |
| NobodyCam | morning jlvillal and roll :) | 16:23 |
| joanna | morning rloo, milan | 16:23 |
| NobodyCam | morning milan | 16:23 |
| vsaienk0 | jlvillal: yeah, they are supported | 16:23 |
| vdrok | jlvillal: https://docs.openstack.org/developer/nova/aggregates.html | 16:23 |
| jlvillal | vsaienk0: Thanks! | 16:23 |
| vdrok | though hopefully, in this release we wont need them anymore | 16:24 |
| jroll | jlvillal: this is an interesting read as well, fwiw, about the problems with having both in a single region https://etherpad.openstack.org/p/ironic-and-virt-mixed-limitations | 16:24 |
| * jroll is working on tackling some of this | 16:24 | |
| jroll | or trying to work on, at least | 16:25 |
| jlvillal | Thanks jroll ! | 16:25 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic-specs master: Change CLI default API version https://review.openstack.org/448661 | 16:25 |
| rloo | JayF: wondering what you think of this bug. My first thought is, wouldn't cleaning have fixed this problem? https://bugs.launchpad.net/ironic-python-agent/+bug/1670239 | 16:26 |
| openstack | Launchpad bug 1670239 in ironic-lib "create_config_drive_partition failed because ironic-lib call cmd "partprobe /dev/sda" failed" [Undecided,In progress] - Assigned to Hao Li (lihaosz) | 16:26 |
| sambetts | hey rloo, jroll, jlvillal, NobodyCam o/ | 16:27 |
| rloo | hi sambetts (stranger!) | 16:27 |
| NobodyCam | morning sambetts | 16:28 |
| sambetts | rloo: IKR! I was traveling and then I had some PTO, while I was traveling I was only able to log in during late PST time and nobody was around by then :( | 16:29 |
| rloo | sambetts: glad you're well rested now :) | 16:29 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Rework VMs connection logic https://review.openstack.org/440719 | 16:29 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 16:30 |
| sambetts | rloo: thanks :) | 16:32 |
| milan | hey sambetts! :) fyi https://review.openstack.org/#/c/439862/ was merged, but if you have some concerns I can do a follow-up (me procrastinated so still need to update the POC ;) | 16:36 |
| sambetts | milan: \o/ | 16:37 |
| *** rcernin has joined #openstack-ironic | 16:37 | |
| sambetts | milan: other than my original comments about the defaults I think everything else made sense to me :) | 16:38 |
| *** Goneri has quit IRC | 16:38 | |
| milan | sambetts, great! :) | 16:38 |
| *** srart has joined #openstack-ironic | 16:39 | |
| milan | sambetts, though the api changed from update_filter to whitelist_node_ids, blacklist_node_ids and remove_node_ids | 16:39 |
| sambetts | sounds good to me :) | 16:40 |
| * milan glad ;) | 16:40 | |
| *** jaosorior has quit IRC | 16:41 | |
| rloo | subteam responsible people -- please update the statuses before meeting in 12 minutes time :) | 16:48 |
| *** stendulker has joined #openstack-ironic | 16:48 | |
| *** derekh has quit IRC | 16:49 | |
| *** stendulker has quit IRC | 16:49 | |
| milan | rloo, oh, right, thx :) | 16:49 |
| *** stendulker has joined #openstack-ironic | 16:49 | |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Copy and append to static lists https://review.openstack.org/449577 | 16:50 |
| openstackgerrit | Thomas O'Neill proposed openstack/bifrost master: Trivial spacing fix. https://review.openstack.org/450338 | 16:53 |
| *** udesale has quit IRC | 16:54 | |
| TheJulia | jroll: are you chairing today? | 16:57 |
| jroll | TheJulia: oh, idk, did I say I would? heh | 16:57 |
| jroll | (if so, then yes) | 16:58 |
| TheJulia | No, you didn't ;) | 16:58 |
| jroll | lol | 16:58 |
| *** vgadiraj_ has quit IRC | 16:58 | |
| TheJulia | no worries, I can do it | 16:58 |
| * TheJulia runs to grab some water | 16:58 | |
| jroll | if you're not in the mood, I don't mind :) | 16:58 |
| * jroll will make sure it's short | 16:58 | |
| *** amotoki has joined #openstack-ironic | 16:58 | |
| TheJulia | jroll: short ++ | 16:58 |
| *** mgoddard has quit IRC | 16:59 | |
| TheJulia | jroll: not terribly in the mood today, juggling a few things at the moment :( | 16:59 |
| TheJulia | but more than happy to if needed | 16:59 |
| jroll | TheJulia: yeah no worries, I got it | 16:59 |
| TheJulia | thank you | 16:59 |
| jroll | np | 16:59 |
| jroll | let's do this | 16:59 |
| *** tesseract has quit IRC | 16:59 | |
| openstackgerrit | Merged openstack/ironic-inspector stable/newton: Grenade: don't enable tempest plugins in settings https://review.openstack.org/425122 | 17:01 |
| *** amotoki has quit IRC | 17:03 | |
| *** ricardoas has joined #openstack-ironic | 17:03 | |
| *** trown is now known as trown|compassing | 17:05 | |
| *** rcernin has quit IRC | 17:08 | |
| *** pcaruana has quit IRC | 17:12 | |
| *** harlowja has quit IRC | 17:18 | |
| *** chlong has quit IRC | 17:22 | |
| *** mgould is now known as mgould|afk | 17:22 | |
| openstackgerrit | Thomas O'Neill proposed openstack/bifrost master: Abstraction of distribution variables and cleanup role. https://review.openstack.org/450347 | 17:22 |
| *** harlowja has joined #openstack-ironic | 17:24 | |
| thomas_oneill | TheJulia: and any other central bifrost kinda people, could you possibly take a look at this WIP I just posted and let me know what you think? ^^ | 17:24 |
| *** amoralej is now known as amoralej|off | 17:27 | |
| thomas_oneill | there's stuff missing but I thought I'd post it up for you guys to look at at this stage, cuz I don't wanna waste time doing fine-tuning stuff if it's conceptually wrong | 17:27 |
| NobodyCam | thomas_oneill: I'll take a look after the meeting | 17:28 |
| thomas_oneill | okay cool... end of the working day for me anyway so will catch back up with this tomorrow, cya!! :) | 17:28 |
| NobodyCam | have a good night :) | 17:28 |
| mjturek | rajinir: could you link me to an example failure of your current failure? Wondering if we're seeing the same thing | 17:29 |
| rajinir | mjturek: we fixed a bunch, here is the latest that I'm working on - http://lists.openstack.org/pipermail/openstack-infra/2017-March/005263.html | 17:30 |
| mjturek | rajinir: do you have links to the logs? | 17:31 |
| rajinir | mjturek: The logs and details are in launchpad https://bugs.launchpad.net/neutron/+bug/1675151 | 17:32 |
| openstack | Launchpad bug 1675151 in neutron "Switch to haproxy fails dhcp request" [Undecided,New] | 17:32 |
| mjturek | ahhh okay rajinir our failure is different, we're stuck at wait_for_resource during stacking | 17:33 |
| mjturek | thx! | 17:33 |
| rajinir | mjturek: may be we will hit that issue | 17:35 |
| rloo | vdrok: do you have time to talk about multinode grenade? | 17:36 |
| *** Goneri has joined #openstack-ironic | 17:36 | |
| rloo | vdrok: and rolling upgrades? | 17:36 |
| *** vgadiraj_ has joined #openstack-ironic | 17:36 | |
| vdrok | yup, I have a bit | 17:36 |
| TheJulia | thomas_oneill: Is it okay if I take a look in about an hour? I need to eat some lunch and get some coffee :) | 17:36 |
| rloo | vdrok: so i don't understand how multinode grenade works, what it is testing | 17:36 |
| rloo | vdrok: when you say things break it, could you elaborate? | 17:36 |
| mjturek | rajinir: I hope not! But if you do let me know, maybe we can brainstorm through it | 17:36 |
| rajinir | mjturek: sure | 17:36 |
| NobodyCam | good meeting .. .thank you all | 17:36 |
| rloo | vdrok: i want to make sure i understand, and that rolling upgrades will address it... | 17:37 |
| vdrok | rloo: right now what it does is, upgrades the primary node, but leaves ironic conductor on a subnode untouched | 17:37 |
| thomas_oneill | TheJulia: yeah ofc lol I'm going now, back tomorrow, enjoy!! | 17:37 |
| TheJulia | thomas_oneill: Have a good evening! | 17:37 |
| rloo | vdrok: so is it running ocata, then master in primary node? | 17:37 |
| vdrok | yup | 17:37 |
| aNuposic | Hi rpioso, are you going to take any other driver module for removing translation logs? | 17:37 |
| rloo | vdrok: oh. so when master has a version bump... | 17:37 |
| *** chlong has joined #openstack-ironic | 17:38 | |
| vdrok | rloo: yup. that's why we need to pin ocata | 17:38 |
| rloo | vdrok: is it doing a cold upgrade? must be. | 17:38 |
| vdrok | rloo: yes | 17:38 |
| vdrok | rloo: but, the communication between new api and old conductor breaks | 17:38 |
| *** milan has quit IRC | 17:38 | |
| rloo | vdrok: so basically, it runs all the tests using ocata. then upgrades primary node (is that API only, or API + conductor) to master, then runs all tests again? | 17:38 |
| vdrok | I think primary is both api and conductor, and yes, that seems to be accurate | 17:39 |
| jroll | hrm | 17:39 |
| jroll | the spec says the process is to update all conductors first | 17:39 |
| jroll | part of that was for this reason | 17:39 |
| jroll | iirc | 17:39 |
| *** milan has joined #openstack-ironic | 17:39 | |
| rloo | vdrok, jroll: seems like a partial upgrade. that mutlinode test. | 17:40 |
| jroll | so why are we running new conductor with old api? :/ | 17:40 |
| vdrok | jroll: yeah, but grenade does not support upgrading subnode iiuc | 17:40 |
| jroll | rloo: sure | 17:40 |
| rloo | vdrok, jroll: seems like if it is doing cold upgrade, the subnode with conductor should be upgraded to master too? | 17:40 |
| jroll | rloo: it isn't intended to be a cold upgrade, it's intended to be partial so we test the mixed versions of services | 17:41 |
| rloo | jroll: OH. do we support mixed versions of services? | 17:41 |
| vdrok | jroll: for now this just tests communication ,so only adding pinning would be enough for now I guess | 17:41 |
| *** e0ne has joined #openstack-ironic | 17:41 | |
| rloo | jroll: I mean, I don't think or didn't think we did. | 17:41 |
| jroll | vdrok: so we just changed the upgrade process without updating the spec or anything :/ | 17:41 |
| jroll | rloo: that's the idea behind rolling upgrades, make mixed versions work so you can upgrade one thing at a time | 17:42 |
| rloo | jroll: right, once we have the code to support that. we don't yet? | 17:42 |
| rloo | jroll: or maybe we do or think we do. sec, let me see. | 17:43 |
| jroll | rloo: sure, the discussion in the meeting was "without rolling upgrades, we break rolling upgrades CI constantly, so let's just finish rolling upgrades. | 17:43 |
| vdrok | jroll: yeah, seems so. will need to talk to vsaienk0 to see what changes we need on the grenade side | 17:43 |
| *** mgoddard has joined #openstack-ironic | 17:43 | |
| jroll | rloo: which lead us to "why are these RPC version bumps breaking that CI, they should not" | 17:43 |
| rloo | jroll: ah. ok, i missed that part. so that multinode test IS to test rolling upgrades, got it. | 17:43 |
| jroll | rloo: yes. | 17:43 |
| *** vgadiraj_ has quit IRC | 17:44 | |
| *** mtanino has quit IRC | 17:44 | |
| rloo | jroll: hey, in that case, the rolling upgrades patches work :) I think... :) | 17:44 |
| jroll | vdrok: we should be following http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/support-rolling-upgrade.html#testing | 17:44 |
| lucasagomes | calling it a day! Have a great evening all | 17:44 |
| rloo | bye lucasagomes! | 17:45 |
| *** lucasagomes is now known as lucas-afk | 17:45 | |
| jroll | rloo: probably, but it seems the test is not implemented the way we intended, in that it isn't following the upgrade process | 17:45 |
| jroll | night lucas-afk | 17:45 |
| lucas-afk | o/ | 17:45 |
| *** vgadiraj_ has joined #openstack-ironic | 17:45 | |
| vdrok | jroll: aha, so we need to move api to subnode | 17:45 |
| rloo | jroll: oh yeah. ding. light bulb flickered. you're right. | 17:45 |
| jroll | vdrok: indeed | 17:45 |
| *** vgadiraj_ has quit IRC | 17:46 | |
| *** ccamacho|brb is now known as ccamacho | 17:46 | |
| *** causten has joined #openstack-ironic | 17:46 | |
| jroll | rloo: :) | 17:46 |
| rloo | vdrok: i'm happier now cuz i thought i'd have to add the grenade tests for rolling upgrades. | 17:46 |
| rloo | vdrok: the spec describes two tests... | 17:47 |
| jroll | rloo: the first bullet point is what we have with grenade today | 17:48 |
| vdrok | ok, so we'll work on that. and iiuc, it's not a blocker then | 17:48 |
| jroll | rloo: the third is "we could", but I don't think we plan on doing that yet | 17:48 |
| jroll | vdrok: awesome, thanks! I'll leave the priorities alone, I'd like that feature done anyway :) | 17:48 |
| rloo | jroll: ok, so for rolling upgrades work to be accepted, we need the second bullet. | 17:48 |
| vdrok | jroll: sure, thanks | 17:48 |
| jroll | rloo: correct, and that's what the multinode grenade job we have is intended to do | 17:49 |
| rloo | and vdrok: that multinode test is to test that second bullet, right? | 17:49 |
| *** baoli has quit IRC | 17:49 | |
| vdrok | rloo: yup | 17:49 |
| rloo | vdrok: so to be clear, are you going to make the changes to the multinode grenade test? | 17:49 |
| *** fragatina has quit IRC | 17:49 | |
| vdrok | as for multinode doing the second bullet | 17:49 |
| rloo | vdrok: or vsaienk0, or jlvillal or find a volunteer. (like moi)? | 17:49 |
| vdrok | rloo: yes, these are changes to grenade plugin and/or grenade I suppose | 17:49 |
| jroll | should just be changing ENABLED_SERVICES for each node :) | 17:50 |
| vdrok | hopefully yes :) | 17:50 |
| rloo | jroll: 'should' :) vdrok, ok, I'll assume you're going to be making the change then. | 17:50 |
| jroll | I chose my word carefully :) | 17:51 |
| rloo | vdrok: thx! | 17:51 |
| rloo | jroll: as you 'should' :) | 17:51 |
| vdrok | np :) | 17:51 |
| vdrok | time to leave for /me, good night! | 17:52 |
| rloo | good night vdrok! | 17:52 |
| NobodyCam | night vdrok | 17:53 |
| openstackgerrit | Merged openstack/ironic-inspector master: Remove translation of log messages Part-1 https://review.openstack.org/447884 | 17:53 |
| *** stendulker has quit IRC | 17:54 | |
| *** ChubYann has joined #openstack-ironic | 17:57 | |
| vdrok | ah, one last thing, it seems we have api running on both primary and subnode, we just need to disable api on primary. this will likely lead to more hacks, related to need to wait for ironic api on subnode so that nova compute can start properly. anyway, will look at this closer tomorrow | 17:57 |
| *** amotoki has joined #openstack-ironic | 17:59 | |
| dsneddon | Has anyone seen this error? | 17:59 |
| dsneddon | )$ openstack baremetal introspection bulk status | 17:59 |
| dsneddon | Could not find node 06919852-544b-4369-8e2a-1e52d05d12ef in cache | 17:59 |
| *** baoli has joined #openstack-ironic | 17:59 | |
| openstackgerrit | Merged openstack/ironic-ui master: Adds Ironic-UI Devstack plugin https://review.openstack.org/446298 | 18:00 |
| dsneddon | The node shows up fine if I run "ironic node-list": http://paste.openstack.org/show/604361/ | 18:01 |
| *** causten has quit IRC | 18:01 | |
| dsneddon | The same "openstack baremetal introspection bulk status" command works with the same version of Ironic on a different environment. | 18:01 |
| *** causten has joined #openstack-ironic | 18:01 | |
| *** amotoki has quit IRC | 18:03 | |
| *** jjung has quit IRC | 18:06 | |
| *** causten has quit IRC | 18:06 | |
| *** e0ne has quit IRC | 18:07 | |
| TheJulia | dsneddon: Sorry, I've not seen anything like that before. :( | 18:08 |
| *** milan has quit IRC | 18:08 | |
| *** milan has joined #openstack-ironic | 18:10 | |
| *** ralonsoh has quit IRC | 18:11 | |
| dsneddon | TheJulia, Thanks, I just figured it out. I had upgraded this Ironic Inspector, but had previously introspected nodes with a version that didn't cache the introspection data. | 18:12 |
| *** mtanino has joined #openstack-ironic | 18:12 | |
| dsneddon | TheJulia, The same command works on an environment with recently introspected nodes. :-) | 18:12 |
| *** jjung has joined #openstack-ironic | 18:18 | |
| *** sayantan_ has joined #openstack-ironic | 18:20 | |
| *** baoli_ has joined #openstack-ironic | 18:24 | |
| *** [1]rpioso has joined #openstack-ironic | 18:24 | |
| *** baoli has quit IRC | 18:25 | |
| *** e0ne has joined #openstack-ironic | 18:27 | |
| *** rpioso has quit IRC | 18:27 | |
| *** e0ne has quit IRC | 18:28 | |
| *** d0ugal has quit IRC | 18:31 | |
| *** Goneri has quit IRC | 18:34 | |
| rloo | vdrok: thx, will touch base with you tomorrow about the multinode test! | 18:36 |
| *** fragatina has joined #openstack-ironic | 18:37 | |
| *** causten has joined #openstack-ironic | 18:38 | |
| *** chas has quit IRC | 18:39 | |
| *** milan has quit IRC | 18:39 | |
| *** milan has joined #openstack-ironic | 18:40 | |
| * milan gone, good night Ironic! :) | 18:45 | |
| *** milan has quit IRC | 18:46 | |
| joanna | gnight, milan! | 18:47 |
| *** d0ugal has joined #openstack-ironic | 18:47 | |
| *** racedo has quit IRC | 18:47 | |
| *** mgoddard has quit IRC | 18:47 | |
| *** e0ne has joined #openstack-ironic | 18:51 | |
| *** vgadiraj_ has joined #openstack-ironic | 18:53 | |
| *** trown|compassing is now known as trown | 18:53 | |
| *** e0ne has quit IRC | 18:56 | |
| *** vgadiraj_ has quit IRC | 18:58 | |
| openstackgerrit | Merged openstack/bifrost stable/ocata: Pin proliantutils version https://review.openstack.org/449518 | 18:59 |
| *** amotoki has joined #openstack-ironic | 19:00 | |
| *** beekneemech is now known as bnemec | 19:00 | |
| *** amotoki has quit IRC | 19:05 | |
| *** vills has joined #openstack-ironic | 19:05 | |
| *** jkilpatr has quit IRC | 19:11 | |
| *** g3ek has quit IRC | 19:12 | |
| *** sambetts is now known as sambetts|afk | 19:13 | |
| sambetts|afk | night all | 19:13 |
| *** g3ek has joined #openstack-ironic | 19:15 | |
| *** jjung has quit IRC | 19:17 | |
| *** jjung has joined #openstack-ironic | 19:20 | |
| *** jkilpatr has joined #openstack-ironic | 19:22 | |
| *** samueldmq has quit IRC | 19:23 | |
| *** samueldmq has joined #openstack-ironic | 19:24 | |
| *** causten has quit IRC | 19:30 | |
| *** causten has joined #openstack-ironic | 19:30 | |
| *** mjura has quit IRC | 19:31 | |
| *** baoli_ has quit IRC | 19:32 | |
| *** baoli has joined #openstack-ironic | 19:39 | |
| *** crushil has quit IRC | 19:44 | |
| *** vills has quit IRC | 19:45 | |
| *** ionutbalutoiu has quit IRC | 19:45 | |
| *** joanna has quit IRC | 19:49 | |
| *** ionutbalutoiu has joined #openstack-ironic | 19:49 | |
| mariojv | hey, random question: what's the best way to ensure that an ironic gate job up for review is pulling down + building the latest IPA code from a particular IPA patch set? | 19:49 |
| mariojv | it's not immediately clear to me poking around whether just adding Depends-On will do that somehow | 19:50 |
| jroll | mariojv: that would be in devstacklog.txt, ctrl-f tinyipa or something | 19:50 |
| jroll | I don't believe depends-on works from ironic->ipa | 19:50 |
| mariojv | ah ok, that's what i feared | 19:50 |
| jroll | since it's a separate devstack variable to decide if it builds | 19:51 |
| mariojv | yup | 19:51 |
| jroll | test patch to flip that is easy though :) | 19:51 |
| jroll | patch on ironic to set the build thing to true, depends-on from there to ipa | 19:51 |
| mariojv | yeah, this is mainly for rescue CI - i just didn't want to keep switching the IPA branch var for the test job every time the IPA patch is updated, but if it's needed, it's fine | 19:52 |
| mariojv | will science when i'm a bit closer to having all the pieces ready | 19:52 |
| jroll | mariojv: just have the one that sets "build ipa" to true depend on your IPA and your ironic patches, you shouldn't need to flip the branch variable | 19:53 |
| mariojv | ah, ok ^ that'd be super easy | 19:54 |
| jroll | :) | 19:54 |
| mariojv | so dependencies would look like: ironic + IPA rescue code -> devstack-gate DNM patch that sets "build IPA" var appropriately -> tests | 19:55 |
| mariojv | something like that, i think | 19:55 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Rework VMs connection logic https://review.openstack.org/440719 | 19:57 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 19:57 |
| jroll | mariojv: yeah, but you can even just make it an ironic patch that changes the default or whatever of that variable | 19:58 |
| jroll | in the devstack plugin | 19:58 |
| mariojv | oh sure, then there's no need to modify devstack-gate as long as the defaults aren't overwritten by the gate jobs | 19:59 |
| *** causten has quit IRC | 20:01 | |
| mariojv | the less patches the better :D | 20:01 |
| *** causten has joined #openstack-ironic | 20:01 | |
| jroll | oh they might be, hm, just change the value later then :P | 20:01 |
| jroll | IPA -> ironic rescue code -> ironic test patch that sets the variable | 20:02 |
| *** baoli has quit IRC | 20:02 | |
| *** baoli has joined #openstack-ironic | 20:02 | |
| TheJulia | cinerama: NobodyCam: If you two get a chance, can you take a look at 450347. It is a WIP but conceptual cleanup of role variables and inclusion of the beginnings of basic installation cleanup. I've left my thoughts on it, but I'd appreciate both of you taking a look. | 20:09 |
| *** vgadiraj_ has joined #openstack-ironic | 20:10 | |
| *** vgadiraj_ has quit IRC | 20:12 | |
| TheJulia | ppiela: looks like you somehow reverted 437419 to patch set 3 with patch set 8 :( | 20:15 |
| ppiela | TheJulia: ugh | 20:16 |
| TheJulia | ppiela: Oh, I see | 20:17 |
| TheJulia | you have that as the parent for another revision | 20:17 |
| *** baoli has quit IRC | 20:17 | |
| TheJulia | I bet the other rev got updated and *boom* | 20:17 |
| ppiela | TheJulia: I have been working on other projects last few days, and have not had a chance to fix the conflicts | 20:18 |
| TheJulia | ppiela: no worries :) | 20:18 |
| *** baoli has joined #openstack-ironic | 20:23 | |
| *** Goneri has joined #openstack-ironic | 20:29 | |
| *** rpioso has joined #openstack-ironic | 20:33 | |
| *** [1]rpioso has quit IRC | 20:37 | |
| rloo | jroll: sorry, forgot to get back to you on ironic releases, wrt the driver comp. client patch: https://review.openstack.org/#/c/419274/. I don't know when that will be approved so if you wanted to release today/tomorrow, don't wait for it. | 20:39 |
| jroll | rloo: oh yeah, I also forgot. I wasn't sure if you were talking about that or the driver properties API discussion that I still haven't looped around to | 20:40 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-inspector-specs master: Collect Bios, Memory and Processor Info from Dmidecode https://review.openstack.org/406790 | 20:40 |
| rloo | jroll: the driver properties API discussion will take waaaaayyyyyy longer :) | 20:41 |
| rloo | jroll: that client patch might be easy or not; i can't recall now, how easy it is to get access to the microversion, in the driver_shell code... | 20:42 |
| jroll | rloo: indeed, that's why I got confused | 20:42 |
| rloo | jroll: heh. do you think it is worth getting 419274 in? | 20:42 |
| jroll | rloo: we can just release again when we land it | 20:43 |
| *** baoli has quit IRC | 20:43 | |
| rloo | jroll: i'm fine with that too. | 20:43 |
| TheJulia | jroll: speaking of releases, bifrost would be good to cut an updated stable/ocata minor bug fix release if your cutting other stuff. | 20:44 |
| jroll | TheJulia: cool, I'm thinking tomorrow | 20:44 |
| TheJulia | Indeed, it is approaching beer-o'clock | 20:45 |
| rloo | jroll: you are, of course, going to cause me to have to update the rolling upgrades patches after the release, but that is a good exercise i guess... | 20:45 |
| jroll | rloo: eh? why? | 20:45 |
| rloo | jroll: new release; gotta add it to the release_mappings :) | 20:45 |
| jroll | oh we put middle release in yeah that | 20:46 |
| rloo | jroll: but then i can test my notes on what to do when we push out a release :) | 20:46 |
| jroll | rloo: is it possible to just land the mappings now or not? | 20:46 |
| rloo | jroll: no, cuz that code hasn't landed yet :) | 20:46 |
| rloo | jroll: oh, no, we shouldn't land that patch. i don't know that it makes sense w/o the whole thing. | 20:47 |
| jroll | okay | 20:47 |
| *** causten has quit IRC | 20:47 | |
| *** causten has joined #openstack-ironic | 20:47 | |
| *** jcoufal has quit IRC | 20:48 | |
| *** joanna has joined #openstack-ironic | 20:49 | |
| *** causten has quit IRC | 20:50 | |
| *** causten_ has joined #openstack-ironic | 20:50 | |
| *** baoli has joined #openstack-ironic | 20:51 | |
| *** causten_ has quit IRC | 20:56 | |
| *** causten has joined #openstack-ironic | 20:56 | |
| *** dprince has quit IRC | 20:58 | |
| *** causten has quit IRC | 21:01 | |
| *** trown is now known as trown|outtypewww | 21:01 | |
| *** baoli has quit IRC | 21:02 | |
| *** Sukhdev has joined #openstack-ironic | 21:03 | |
| *** causten has joined #openstack-ironic | 21:05 | |
| *** causten has quit IRC | 21:10 | |
| *** Serlex has quit IRC | 21:11 | |
| *** ccamacho has quit IRC | 21:12 | |
| *** causten has joined #openstack-ironic | 21:13 | |
| *** fragatin_ has joined #openstack-ironic | 21:13 | |
| *** jjung has quit IRC | 21:14 | |
| *** fragatina has quit IRC | 21:16 | |
| *** mat128 has quit IRC | 21:21 | |
| *** causten has quit IRC | 21:23 | |
| *** causten has joined #openstack-ironic | 21:24 | |
| openstackgerrit | Mario Villaplana proposed openstack/python-ironicclient master: Log warning when API version unspecified https://review.openstack.org/442153 | 21:26 |
| *** causten_ has joined #openstack-ironic | 21:26 | |
| *** causten has quit IRC | 21:28 | |
| *** fragatin_ has quit IRC | 21:29 | |
| *** fragatina has joined #openstack-ironic | 21:29 | |
| *** causten_ has quit IRC | 21:30 | |
| *** causten has joined #openstack-ironic | 21:31 | |
| *** causten_ has joined #openstack-ironic | 21:33 | |
| *** causten has quit IRC | 21:36 | |
| *** MarginHu has joined #openstack-ironic | 21:42 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Install networking-baremetal ML2 plugin https://review.openstack.org/450467 | 21:43 |
| *** ijw_ is now known as ijw | 21:46 | |
| *** bfournie has quit IRC | 21:52 | |
| *** baha has quit IRC | 21:59 | |
| *** amotoki has joined #openstack-ironic | 22:02 | |
| *** chlong has quit IRC | 22:03 | |
| *** causten_ has quit IRC | 22:05 | |
| *** causten has joined #openstack-ironic | 22:06 | |
| *** jkilpatr has quit IRC | 22:07 | |
| *** cdearborn has quit IRC | 22:09 | |
| *** causten has quit IRC | 22:10 | |
| *** amotoki has quit IRC | 22:12 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic master: Add base cinder common interface https://review.openstack.org/355625 | 22:13 |
| *** rpioso has quit IRC | 22:13 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Fix some reST field lists in docstrings https://review.openstack.org/449629 | 22:17 |
| openstackgerrit | Joanna Taryma proposed openstack/ironic master: Add Cinder storage driver https://review.openstack.org/366197 | 22:21 |
| *** bfournie has joined #openstack-ironic | 22:25 | |
| *** Syed__ has quit IRC | 22:25 | |
| *** jkilpatr has joined #openstack-ironic | 22:35 | |
| *** Goneri has quit IRC | 22:37 | |
| *** rbudden has quit IRC | 22:39 | |
| *** sasha2 has quit IRC | 22:41 | |
| *** Sukhdev has quit IRC | 23:06 | |
| openstackgerrit | Julian Edwards proposed openstack/ironic master: Don't retry power status if power action fails https://review.openstack.org/449454 | 23:12 |
| *** jaypipes has quit IRC | 23:24 | |
| *** MarginHu has quit IRC | 23:31 | |
| *** crushil has joined #openstack-ironic | 23:36 | |
| *** pmannidi has joined #openstack-ironic | 23:36 | |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Config drive support for Ceph https://review.openstack.org/407639 | 23:40 |
| aNuposic | jlvillal, vdrok joanna soliosg Addressed your comments could you please add ^ to your review list | 23:46 |
| *** joanna has quit IRC | 23:48 | |
| *** jjung has joined #openstack-ironic | 23:54 | |
| *** fragatin_ has joined #openstack-ironic | 23:55 | |
| *** fragatina has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!