| jlvillal | aNuposic: My rejected RFE: https://bugs.launchpad.net/bugs/1596421 | 00:00 |
|---|---|---|
| openstack | Launchpad bug 1596421 in Ironic "RFE: Increase size of data base entry for instance_info to allow configdrives larger than 64KiB" [Wishlist,Invalid] - Assigned to John L. Villalovos (happycamp) | 00:00 |
| jlvillal | aNuposic: What should be done instead: https://bugs.launchpad.net/ironic/+bug/1575935 | 00:00 |
| openstack | Launchpad bug 1575935 in Ironic "Rebuild should also accept a configdrive" [Wishlist,Triaged] | 00:00 |
| *** ijw has quit IRC | 00:01 | |
| *** ijw has joined #openstack-ironic | 00:02 | |
| *** ElCoyote_ has quit IRC | 00:02 | |
| aNuposic | JayF: jlvillal: I see. Good to know. | 00:03 |
| aNuposic | JayF: jlvillal: Don't know then if my RFE gets approved | 00:03 |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job https://review.openstack.org/403895 | 00:04 |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/408091 | 00:04 |
| *** [1]rpioso has quit IRC | 00:06 | |
| JayF | aNuposic: I suspect it will; rescue did and it's dependant on configdrive. John's was almost impossible to undo later which I think is why it got nix'd. | 00:07 |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job https://review.openstack.org/403895 | 00:09 |
| aNuposic | JayF: hmm looks like. | 00:10 |
| aNuposic | JayF: and also makes sense if configdrive is getting rebuild again | 00:13 |
| *** hw_wutianwei has joined #openstack-ironic | 00:13 | |
| JayF | aNuposic: I'm heading out for the day, you should talk to mario about seeing if we can get your rfe approved, I'm +1 on it now, we can try to get consensus before a larger group of cores | 00:13 |
| JayF | aNuposic: have a good day! see you in standup tomorrow o/ | 00:13 |
| aNuposic | JayF: Yeah, true; Have a great evening! See you in standup | 00:14 |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job https://review.openstack.org/403895 | 00:15 |
| *** ElCoyote_ has joined #openstack-ironic | 00:16 | |
| *** sambetts|afk has quit IRC | 00:18 | |
| *** sambetts_ has joined #openstack-ironic | 00:21 | |
| *** ijw has quit IRC | 00:22 | |
| *** ijw has joined #openstack-ironic | 00:22 | |
| *** aNuposic has quit IRC | 00:35 | |
| *** aNuposic has joined #openstack-ironic | 00:38 | |
| *** rama_y has quit IRC | 00:46 | |
| *** yufei has joined #openstack-ironic | 00:47 | |
| *** yufei has quit IRC | 00:48 | |
| *** SerenaFeng has joined #openstack-ironic | 00:52 | |
| *** hoangcx has joined #openstack-ironic | 00:54 | |
| *** devlaps has quit IRC | 00:59 | |
| *** jistr|mtgs has quit IRC | 01:00 | |
| *** jistr has joined #openstack-ironic | 01:00 | |
| *** devlaps has joined #openstack-ironic | 01:01 | |
| *** ijw has quit IRC | 01:02 | |
| *** yuanying has quit IRC | 01:05 | |
| *** fragatin_ has joined #openstack-ironic | 01:06 | |
| *** phuongnh has joined #openstack-ironic | 01:07 | |
| *** fragatina has quit IRC | 01:08 | |
| *** fragatin_ has quit IRC | 01:11 | |
| *** jkilpatr has joined #openstack-ironic | 01:12 | |
| *** tuanluong has joined #openstack-ironic | 01:12 | |
| *** ijw has joined #openstack-ironic | 01:12 | |
| *** yuanying has joined #openstack-ironic | 01:13 | |
| *** jschwarz has quit IRC | 01:25 | |
| *** jschwarz has joined #openstack-ironic | 01:27 | |
| *** zhangjl has joined #openstack-ironic | 01:28 | |
| *** mtanino has quit IRC | 01:41 | |
| *** Sukhdev has quit IRC | 01:42 | |
| *** rloo has quit IRC | 01:43 | |
| *** Syed__ has quit IRC | 01:47 | |
| *** SerenaFeng has quit IRC | 01:53 | |
| *** yufei has joined #openstack-ironic | 02:02 | |
| *** gcb has joined #openstack-ironic | 02:12 | |
| *** ijw has quit IRC | 02:17 | |
| *** aNuposic has quit IRC | 02:26 | |
| *** rbudden has joined #openstack-ironic | 02:33 | |
| *** yuanying has quit IRC | 02:35 | |
| *** baoli has joined #openstack-ironic | 02:36 | |
| *** ijw has joined #openstack-ironic | 02:37 | |
| *** mtanino has joined #openstack-ironic | 02:37 | |
| *** yuanying has joined #openstack-ironic | 02:38 | |
| *** ijw has quit IRC | 02:42 | |
| *** baoli has quit IRC | 02:50 | |
| *** ijw has joined #openstack-ironic | 02:55 | |
| *** rama_y has joined #openstack-ironic | 02:59 | |
| *** ijw has quit IRC | 03:01 | |
| *** mtanino has quit IRC | 03:02 | |
| *** ijw has joined #openstack-ironic | 03:10 | |
| *** SerenaFeng has joined #openstack-ironic | 03:15 | |
| *** rbudden has quit IRC | 03:28 | |
| *** vikrant has joined #openstack-ironic | 03:40 | |
| *** SerenaFeng has quit IRC | 03:42 | |
| *** absubram has joined #openstack-ironic | 03:48 | |
| *** absubram_ has joined #openstack-ironic | 03:50 | |
| *** SerenaFeng has joined #openstack-ironic | 03:51 | |
| *** absubram has quit IRC | 03:52 | |
| *** absubram_ is now known as absubram | 03:52 | |
| *** rama_y has quit IRC | 03:54 | |
| *** aparnav has joined #openstack-ironic | 03:55 | |
| *** aparna has joined #openstack-ironic | 03:58 | |
| *** aparnav has quit IRC | 03:59 | |
| *** aparna has quit IRC | 04:03 | |
| *** krtaylor has quit IRC | 04:04 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/ironic: Move baremetal tempest config seting from devstack https://review.openstack.org/408404 | 04:07 |
| *** aparnav has joined #openstack-ironic | 04:11 | |
| *** zhangjl has quit IRC | 04:12 | |
| *** zhangjl has joined #openstack-ironic | 04:12 | |
| *** alexpilo_ has quit IRC | 04:14 | |
| *** alexpilotti has joined #openstack-ironic | 04:15 | |
| *** aparnav has quit IRC | 04:15 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/ironic: Move baremetal tempest config setting from devstack https://review.openstack.org/408404 | 04:15 |
| *** zhangjl has quit IRC | 04:17 | |
| *** SerenaFeng has quit IRC | 04:25 | |
| *** mrtenio has quit IRC | 04:26 | |
| *** tuanluong has quit IRC | 04:27 | |
| *** yuanying has quit IRC | 04:28 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 04:33 |
| *** harlowja has joined #openstack-ironic | 04:37 | |
| *** mrtenio has joined #openstack-ironic | 04:38 | |
| *** krtaylor has joined #openstack-ironic | 04:38 | |
| *** nmathew has joined #openstack-ironic | 04:42 | |
| *** rbudden has joined #openstack-ironic | 05:03 | |
| *** links has joined #openstack-ironic | 05:05 | |
| *** devlaps has quit IRC | 05:13 | |
| *** fragatina has joined #openstack-ironic | 05:14 | |
| *** fragatina has quit IRC | 05:15 | |
| *** fragatina has joined #openstack-ironic | 05:15 | |
| *** rama_y has joined #openstack-ironic | 05:20 | |
| *** Sukhdev has joined #openstack-ironic | 05:26 | |
| *** yuanying has joined #openstack-ironic | 05:26 | |
| *** yuanying has quit IRC | 05:36 | |
| *** yuanying has joined #openstack-ironic | 05:36 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI https://review.openstack.org/348191 | 05:36 |
| *** SerenaFeng has joined #openstack-ironic | 05:40 | |
| *** harlowja has quit IRC | 05:46 | |
| *** pmannidi has quit IRC | 05:47 | |
| *** ijw has quit IRC | 05:53 | |
| *** moshele has joined #openstack-ironic | 05:55 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Foo https://review.openstack.org/408417 | 05:56 |
| *** jaosorior has joined #openstack-ironic | 06:05 | |
| *** rbudden has quit IRC | 06:08 | |
| *** moshele has quit IRC | 06:10 | |
| *** fragatina has quit IRC | 06:13 | |
| openstackgerrit | paresh sao proposed openstack/ironic: Boot from iSCSI volume https://review.openstack.org/308425 | 06:13 |
| *** ChubYann has quit IRC | 06:20 | |
| *** yufei_ has quit IRC | 06:20 | |
| *** alex_xu has quit IRC | 06:20 | |
| *** alex_xu has joined #openstack-ironic | 06:24 | |
| *** rama_y has quit IRC | 06:25 | |
| *** rama_y_ has joined #openstack-ironic | 06:25 | |
| *** fragatina has joined #openstack-ironic | 06:26 | |
| *** pmannidi has joined #openstack-ironic | 06:29 | |
| *** yufei_ has joined #openstack-ironic | 06:29 | |
| *** ijw has joined #openstack-ironic | 06:32 | |
| *** mjura has joined #openstack-ironic | 06:39 | |
| *** fragatina has quit IRC | 06:44 | |
| *** mjura has quit IRC | 06:45 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-inspector-specs: Collect Bios, Memory and Processor Info from Dmidecode https://review.openstack.org/406790 | 06:47 |
| *** jaosorior has quit IRC | 06:48 | |
| *** sacharya has quit IRC | 06:51 | |
| openstackgerrit | Aparna proposed openstack/ironic: Doc for sanitize erase support in iLO drivers https://review.openstack.org/408441 | 06:51 |
| *** sacharya has joined #openstack-ironic | 06:52 | |
| *** fxpester has joined #openstack-ironic | 06:56 | |
| *** rajinir has quit IRC | 06:56 | |
| *** sacharya has quit IRC | 06:57 | |
| *** zackf has quit IRC | 06:58 | |
| *** mjura has joined #openstack-ironic | 06:59 | |
| *** SerenaFeng has quit IRC | 07:01 | |
| *** moshele has joined #openstack-ironic | 07:11 | |
| *** SerenaFeng has joined #openstack-ironic | 07:17 | |
| *** SerenaFeng has quit IRC | 07:17 | |
| *** Sukhdev has quit IRC | 07:24 | |
| *** SerenaFeng has joined #openstack-ironic | 07:26 | |
| *** rcernin has joined #openstack-ironic | 07:26 | |
| *** vsaienko has joined #openstack-ironic | 07:29 | |
| *** magic has joined #openstack-ironic | 07:29 | |
| *** xiaoyang has quit IRC | 07:32 | |
| *** xiaoyang has joined #openstack-ironic | 07:32 | |
| *** magic has quit IRC | 07:33 | |
| *** rbartal has joined #openstack-ironic | 07:36 | |
| *** zhangjl has joined #openstack-ironic | 07:36 | |
| *** ijw has quit IRC | 07:40 | |
| *** sacharya has joined #openstack-ironic | 07:53 | |
| *** sacharya has quit IRC | 07:58 | |
| *** xiaoyang has quit IRC | 08:00 | |
| *** sacharya has joined #openstack-ironic | 08:00 | |
| *** sacharya has quit IRC | 08:06 | |
| *** ccamacho has quit IRC | 08:10 | |
| *** ccamacho has joined #openstack-ironic | 08:14 | |
| *** ralonsoh has joined #openstack-ironic | 08:18 | |
| *** mjura has quit IRC | 08:19 | |
| *** jpich has joined #openstack-ironic | 08:21 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Make _get_sensors_data concurrent https://review.openstack.org/407429 | 08:25 |
| *** vsaienko has quit IRC | 08:28 | |
| *** ohamada has joined #openstack-ironic | 08:32 | |
| *** yuanying has quit IRC | 08:35 | |
| *** yuanying has joined #openstack-ironic | 08:36 | |
| *** strigazi_AFK is now known as strigazi | 08:39 | |
| *** yuanying has quit IRC | 08:40 | |
| *** yufei_ has quit IRC | 08:53 | |
| *** pcaruana has joined #openstack-ironic | 08:58 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zackf has joined #openstack-ironic | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:00 | |
| *** yufei_ has joined #openstack-ironic | 09:02 | |
| *** Romanenko_K has joined #openstack-ironic | 09:03 | |
| *** zackf has quit IRC | 09:04 | |
| *** zhangjl has quit IRC | 09:10 | |
| *** zhangjl has joined #openstack-ironic | 09:11 | |
| *** zhangjl has quit IRC | 09:13 | |
| *** absubram_ has joined #openstack-ironic | 09:17 | |
| *** absubram has quit IRC | 09:19 | |
| *** absubram_ is now known as absubram | 09:19 | |
| *** priteau has joined #openstack-ironic | 09:20 | |
| *** derekh has joined #openstack-ironic | 09:26 | |
| *** pmannidi has quit IRC | 09:28 | |
| *** pmannidi has joined #openstack-ironic | 09:30 | |
| *** yuanying has joined #openstack-ironic | 09:35 | |
| *** lucas-afk is now known as lucasagomes | 09:36 | |
| *** pmannidi has quit IRC | 09:39 | |
| lucasagomes | morning all | 09:41 |
| *** pmannidi has joined #openstack-ironic | 09:41 | |
| *** jaosorior has joined #openstack-ironic | 09:41 | |
| *** hogepodge has quit IRC | 09:45 | |
| *** hogepodge has joined #openstack-ironic | 09:46 | |
| *** SerenaFeng has quit IRC | 09:46 | |
| *** SerenaFeng has joined #openstack-ironic | 09:47 | |
| *** pmannidi has quit IRC | 09:50 | |
| *** milan has joined #openstack-ironic | 09:51 | |
| *** SerenaFeng has quit IRC | 09:55 | |
| *** e0ne has joined #openstack-ironic | 10:01 | |
| milan | morning Ironic! :) | 10:01 |
| joanna | Morning! | 10:01 |
| milan | morning joanna! :) | 10:01 |
| openstackgerrit | Sofiia Andriichenko proposed openstack/ironic: Added api test test_properties_driver https://review.openstack.org/408531 | 10:03 |
| *** jpich has quit IRC | 10:05 | |
| *** jtomasek has joined #openstack-ironic | 10:07 | |
| *** bks has joined #openstack-ironic | 10:10 | |
| milan | guys, if I wanted to used a library not yet in global requirements, what would be the procedure? | 10:12 |
| * milan considers using http://construct.readthedocs.io/en/latest/index.html#example for declarative binary data parsing | 10:12 | |
| *** jpich has joined #openstack-ironic | 10:14 | |
| pas-ha | morning Ironicers :) | 10:14 |
| milan | morning pas-ha! :) | 10:14 |
| milan | aarefiev, sambetts_ , dtantsur|afk ^^^ if it was for Inspector | 10:16 |
| *** sambetts_ is now known as sambetts | 10:17 | |
| sambetts | JayF: sure, :) | 10:17 |
| milan | morning sambetts! :) | 10:18 |
| milan | sambetts, wdyt about http://construct.readthedocs.io/en/latest/index.html#example ;) | 10:18 |
| aarefiev | morning all! | 10:19 |
| milan | morning aarefiev! :) | 10:19 |
| aarefiev | milan: hey, sorry for delay, I was on sick leave, I'll take a look on state patch today | 10:20 |
| milan | aarefiev, thanks a ton! :) | 10:20 |
| milan | aarefiev, also wdyt about http://construct.readthedocs.io/en/latest/index.html#example | 10:21 |
| aarefiev | milan: as for requirements, I guess there is some info in readme about adding new libs | 10:21 |
| aarefiev | I didn't use it | 10:21 |
| milan | it's w/r https://review.openstack.org/#/c/406496/ | 10:21 |
| patchbot | patch 406496 - ironic-inspector - Add plugin to process basic management LLDP TLVs | 10:21 |
| milan | I thought it might be more convenient to use an existing parser here | 10:22 |
| milan | just thinking about it though | 10:22 |
| aarefiev | sure, reusing existing lib would be nice, I don't have experience with this one | 10:24 |
| milan | me neither actually, I've used other libs from this guy in the past though | 10:24 |
| milan | just knew it existed and didn't have any deps | 10:24 |
| milan | I'll try it out I think first localy | 10:25 |
| milan | w/ that patch | 10:25 |
| sambetts | milan: hmm, its certainly interesting, I'd be interested to see how it could be used with the TLVs I think at first glance it might be difficult because you have to parse the V differently depending on the T | 10:25 |
| milan | sambetts, it ought to support conditionals and enums | 10:26 |
| milan | like in C | 10:26 |
| milan | but yeah, I think it would require trying first whether it would actually make any difference | 10:26 |
| sambetts | yeah :) thanks for bringing up the idea, I never know these sort of libs exist until someone suggests it hehe | 10:27 |
| *** karts has quit IRC | 10:27 | |
| *** mgould|afk is now known as mgould | 10:29 | |
| milan | np :) | 10:29 |
| mgould | morning lucasagomes milan aarefiev pas-ha sambetts and everyone else | 10:30 |
| mgould | and welcome joanna, don't think I've seen you in here before :-) | 10:30 |
| lucasagomes | o/ | 10:30 |
| *** karthiks has joined #openstack-ironic | 10:30 | |
| *** athomas has joined #openstack-ironic | 10:34 | |
| mgould | milan: yep, looks like construct does what you want: http://construct.readthedocs.io/en/latest/misc.html#switch | 10:34 |
| *** sborkows has joined #openstack-ironic | 10:36 | |
| *** hoangcx has quit IRC | 10:36 | |
| sborkows | morning, all | 10:37 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-inspector: Use the device hints matching mechanism from ironic-lib https://review.openstack.org/408552 | 10:45 |
| lucasagomes | dtantsur|afk, ^ | 10:45 |
| lucasagomes | sborkows, morning | 10:45 |
| *** dtantsur|afk is now known as dtantsur | 10:50 | |
| dtantsur | morning ironic! wow, I woke up late today :) | 10:50 |
| *** yufei has quit IRC | 10:51 | |
| dtantsur | milan, https://github.com/openstack/requirements/blob/master/README.rst#proposing-changes | 10:51 |
| dtantsur | lucasagomes, thanks! | 10:51 |
| lucasagomes | dtantsur, will put a new patch-set in a min, just updating one exception message | 10:52 |
| dtantsur | sure, I'm not ready to review stuff just yet :) | 10:52 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-inspector: Use the device hints matching mechanism from ironic-lib https://review.openstack.org/408552 | 10:53 |
| lucasagomes | there you go | 10:53 |
| lucasagomes | heh it's small enough no worries :-) | 10:54 |
| dtantsur | cool :) | 10:54 |
| *** strigazi is now known as strigazi_AFK | 10:59 | |
| *** links has quit IRC | 11:00 | |
| sborkows | morning dtantsur | 11:01 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command online_data_migration https://review.openstack.org/408556 | 11:01 |
| *** hw_wutianwei has quit IRC | 11:01 | |
| *** hogepodge has quit IRC | 11:01 | |
| aarefiev | morning mgould, dtantsur | 11:11 |
| dtantsur | o/ | 11:11 |
| milan | morning lucasagomes, mgould, sborkows! :) | 11:15 |
| milan | dtantsur, thx | 11:15 |
| * milan looks | 11:15 | |
| *** links has joined #openstack-ironic | 11:17 | |
| *** hogepodge has joined #openstack-ironic | 11:21 | |
| *** nmathew has quit IRC | 11:23 | |
| *** yuanying has quit IRC | 11:24 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Remove upgrade from non-ironic setup https://review.openstack.org/408569 | 11:25 |
| *** yuanying has joined #openstack-ironic | 11:25 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Fix default value of "ignore_req_list" config option https://review.openstack.org/408572 | 11:26 |
| *** hogepodge has quit IRC | 11:26 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Remove default_params_with_timeout_values from tempest client https://review.openstack.org/408573 | 11:27 |
| *** hogepodge has joined #openstack-ironic | 11:38 | |
| *** jkilpatr has quit IRC | 11:39 | |
| sergek | Morning ALl :) | 11:42 |
| mrtenio | Morning sergek, morning all! | 11:44 |
| *** hogepodge has quit IRC | 11:48 | |
| *** moshele has quit IRC | 11:48 | |
| *** zackf has joined #openstack-ironic | 11:49 | |
| *** pester has joined #openstack-ironic | 11:49 | |
| *** fxpester has quit IRC | 11:53 | |
| *** fxpester|2 has joined #openstack-ironic | 11:53 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command online_data_migration https://review.openstack.org/408556 | 11:53 |
| *** zackf has quit IRC | 11:53 | |
| *** awiddersheim has quit IRC | 11:54 | |
| *** athomas has quit IRC | 11:54 | |
| *** fxpester has joined #openstack-ironic | 11:55 | |
| *** moshele has joined #openstack-ironic | 11:55 | |
| *** vikrant has quit IRC | 11:55 | |
| *** pester has quit IRC | 11:55 | |
| *** pester has joined #openstack-ironic | 11:57 | |
| *** fxpester|2 has quit IRC | 11:58 | |
| *** fxpester|2 has joined #openstack-ironic | 11:59 | |
| *** fxpester has quit IRC | 12:00 | |
| *** hogepodge has joined #openstack-ironic | 12:00 | |
| *** nmathew has joined #openstack-ironic | 12:01 | |
| *** nmathew has quit IRC | 12:01 | |
| *** sborkows_ has joined #openstack-ironic | 12:01 | |
| *** sborkows has quit IRC | 12:01 | |
| *** fxpester has joined #openstack-ironic | 12:01 | |
| *** pester has quit IRC | 12:02 | |
| *** sacharya has joined #openstack-ironic | 12:02 | |
| *** athomas has joined #openstack-ironic | 12:03 | |
| *** vikrant has joined #openstack-ironic | 12:03 | |
| *** fxpester|2 has quit IRC | 12:04 | |
| *** pester has joined #openstack-ironic | 12:06 | |
| *** sacharya has quit IRC | 12:07 | |
| *** fxpester|2 has joined #openstack-ironic | 12:08 | |
| *** vikrant has quit IRC | 12:09 | |
| *** fxpester has quit IRC | 12:09 | |
| dtantsur | lucasagomes, mind checking https://review.openstack.org/#/c/402052/ please? this seems to give us some troubles on OVB | 12:09 |
| patchbot | patch 402052 - ironic - [ipmitool] Do not try powering off nodes that are ... | 12:09 |
| lucasagomes | dtantsur, hi there... sure | 12:09 |
| lucasagomes | I will take a look in a few | 12:09 |
| *** jaosorior has quit IRC | 12:10 | |
| *** pester has quit IRC | 12:11 | |
| *** fxpester has joined #openstack-ironic | 12:11 | |
| *** pester has joined #openstack-ironic | 12:13 | |
| *** fxpester|2 has quit IRC | 12:13 | |
| dtantsur | lucasagomes, also, do you still plan on https://review.openstack.org/#/c/246033/ ? I see downstream complains about it | 12:14 |
| patchbot | patch 246033 - ironic - Conductor fails to do take_over for PXE drivers | 12:14 |
| dtantsur | without this change we're not really HA | 12:14 |
| *** jkilpatr has joined #openstack-ironic | 12:14 | |
| *** fxpester has quit IRC | 12:16 | |
| *** fxpester has joined #openstack-ironic | 12:17 | |
| *** bks has quit IRC | 12:18 | |
| *** pester has quit IRC | 12:20 | |
| *** pester has joined #openstack-ironic | 12:20 | |
| *** xavierr_ has joined #openstack-ironic | 12:22 | |
| *** fxpester|2 has joined #openstack-ironic | 12:23 | |
| *** fxpester has quit IRC | 12:23 | |
| pas-ha | dtantsur: lucasagomes I was not aware of this patch and made another one, that also fixed AgentDeploy to this regard https://review.openstack.org/#/c/404858/ | 12:23 |
| patchbot | patch 404858 - ironic - Fix takeover for AgentDeploy and PXEBoot | 12:23 |
| pas-ha | just needs a unit tests for new pxeboot functionality | 12:24 |
| lucasagomes | dtantsur, we probably can abandon that now in favor of ^ | 12:25 |
| *** pester has quit IRC | 12:26 | |
| pas-ha | although note I've made it dependent on some of my dynamic ipxe changes as those introduce some refactoring I'd like to reuse... but could change that if really needed | 12:26 |
| dtantsur | pas-ha, yeah, could you please change it? I'd like to backport it when it merges, if possible | 12:29 |
| *** vikrant has joined #openstack-ironic | 12:29 | |
| pas-ha | ok, will do | 12:29 |
| *** fxpester|2 has quit IRC | 12:31 | |
| * lucasagomes go grab some food before dealing with more RDO | 12:32 | |
| *** lucasagomes is now known as lucas-hungry | 12:32 | |
| *** sborkows_ has quit IRC | 12:35 | |
| *** jaosorior has joined #openstack-ironic | 12:37 | |
| *** vikrant has quit IRC | 12:38 | |
| *** sborkows_ has joined #openstack-ironic | 12:39 | |
| *** xavierr_ has quit IRC | 12:42 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic: Disable guessing content type from extension in node name. https://review.openstack.org/408597 | 12:45 |
| *** sborkows_ has quit IRC | 12:47 | |
| *** vinsh has quit IRC | 12:48 | |
| *** vikrant has joined #openstack-ironic | 12:52 | |
| *** links has quit IRC | 12:53 | |
| *** links has joined #openstack-ironic | 13:05 | |
| *** nmathew has joined #openstack-ironic | 13:09 | |
| *** boris-42 has quit IRC | 13:09 | |
| *** boris-42 has joined #openstack-ironic | 13:10 | |
| *** jheroux has joined #openstack-ironic | 13:11 | |
| TheJulia | xek: I just noticed https://review.openstack.org/#/c/408556 and I suspect that should be stacked in with the other rolling upgrade revisions. I also kind of suspect it might be a good idea to change the topic to match the one being used for rolling upgrades. | 13:12 |
| patchbot | patch 408556 - ironic - Add new dbsync command online_data_migration | 13:12 |
| *** vikrant has quit IRC | 13:15 | |
| *** dtantsur is now known as dtantsur|brb | 13:23 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Fix takeover for AgentDeploy and PXEBoot https://review.openstack.org/404858 | 13:23 |
| *** nmathew- has joined #openstack-ironic | 13:24 | |
| * TheJulia wonders what is up with gerrit this morning | 13:25 | |
| *** bfournie has quit IRC | 13:25 | |
| *** nmathew has quit IRC | 13:25 | |
| *** nmathew has joined #openstack-ironic | 13:26 | |
| pas-ha | dtantsur|brb: lucas-hungry ^^ | 13:28 |
| *** nmathew- has quit IRC | 13:30 | |
| openstackgerrit | Merged openstack/bifrost: README.rst: Add openSUSE Leap to list of supported distributions https://review.openstack.org/407970 | 13:30 |
| *** phuongnh has quit IRC | 13:36 | |
| *** lucas-hungry is now known as lucasagomes | 13:45 | |
| *** baoli has joined #openstack-ironic | 13:45 | |
| openstackgerrit | Sofiia Andriichenko proposed openstack/ironic: Added api tests for ironic driver https://review.openstack.org/408531 | 13:47 |
| *** baoli has quit IRC | 13:48 | |
| *** baoli has joined #openstack-ironic | 13:49 | |
| *** strigazi_AFK is now known as strigazi | 13:49 | |
| *** baoli has quit IRC | 13:50 | |
| *** baoli has joined #openstack-ironic | 13:50 | |
| *** absubram has quit IRC | 13:51 | |
| *** glonlas has joined #openstack-ironic | 13:55 | |
| openstackgerrit | Merged openstack/ironic-inspector: Remove default_params_with_timeout_values from tempest client https://review.openstack.org/408573 | 13:55 |
| *** glonlas has quit IRC | 13:56 | |
| *** bfournie has joined #openstack-ironic | 13:57 | |
| *** glonlas has joined #openstack-ironic | 13:59 | |
| *** glonlas has quit IRC | 14:00 | |
| *** hamzy has quit IRC | 14:00 | |
| *** glonlas has joined #openstack-ironic | 14:01 | |
| *** sborkows has joined #openstack-ironic | 14:02 | |
| *** sacharya has joined #openstack-ironic | 14:03 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic: Fail ironic startup if no protocol prefix in ironic api address https://review.openstack.org/404975 | 14:05 |
| jroll | morning y'all | 14:06 |
| sambetts | hey jroll | 14:06 |
| *** rloo has joined #openstack-ironic | 14:06 | |
| *** sacharya has quit IRC | 14:07 | |
| *** Goneri has joined #openstack-ironic | 14:08 | |
| *** lindycoder has joined #openstack-ironic | 14:12 | |
| lucasagomes | jroll, sambetts morning | 14:14 |
| *** rbudden has joined #openstack-ironic | 14:15 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic: Disable guessing content type from extension in node name. https://review.openstack.org/408597 | 14:16 |
| *** srobert has joined #openstack-ironic | 14:16 | |
| *** cdearborn has joined #openstack-ironic | 14:19 | |
| rloo | hi and good morning lucasagomes, jroll, sambetts | 14:23 |
| lucasagomes | rloo, hi there | 14:23 |
| sambetts | hey lucasagomes, rloo | 14:23 |
| sambetts | vsaienk0: ping | 14:23 |
| vsaienk0 | hello sambetts | 14:23 |
| sborkows | JayF, are you here? | 14:28 |
| rloo | TheJulia, xek: wrt https://review.openstack.org/#/c/408556, there is a separate (from rolling upgrades) RFE for it: https://bugs.launchpad.net/ironic/+bug/1585141 | 14:32 |
| openstack | Launchpad bug 1585141 in Ironic "[RFE] Add new ironic-dbsync command online-data-migration" [Wishlist,In progress] - Assigned to Grzegorz Grasza (xek) | 14:32 |
| patchbot | patch 408556 - ironic - Add new dbsync command online_data_migration | 14:32 |
| rloo | TheJulia, xek: When Tan Lin wrote the rolling upgrades spec, he separated it. i think cuz the rolling upgrades spec/work was already a lot. | 14:33 |
| rloo | TheJulia: and now, I just saw your comment in 408556. Am thinking... | 14:36 |
| TheJulia | rloo: good morning | 14:36 |
| rloo | TheJulia: yes, good morning to you too! | 14:37 |
| *** jcoufal has joined #openstack-ironic | 14:38 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Use identity api version 3 for OSC plugin tests https://review.openstack.org/408205 | 14:39 |
| *** dtantsur|brb is now known as dtantsur | 14:40 | |
| dtantsur | morning jroll | 14:40 |
| dtantsur | morning also rloo and TheJulia | 14:40 |
| rloo | hi dtantsur | 14:40 |
| TheJulia | good morning dtantsur | 14:42 |
| mariojv | jcook: i'm going to rebase your api/conductor patch if that's alright, since it's a dependency for one of mine | 14:44 |
| jcook | mariojv k | 14:45 |
| jcook | mariojv it does require an update to add in the password that JayF identified yesterday that I haven't got to yet | 14:45 |
| mariojv | i'll take a look and fix if it's trivial | 14:45 |
| jcook | mariojv it's not | 14:45 |
| mariojv | ah, ok then | 14:45 |
| jcook | mariojv just letting you know | 14:45 |
| mariojv | thx | 14:46 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: [WIP] Agent driver rescue implementation https://review.openstack.org/400437 | 14:47 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 14:47 |
| *** daemontool has joined #openstack-ironic | 14:49 | |
| joanna | mgould, I'm just shy ;) | 14:52 |
| mgould | joanna: heh :-) Hi anyway! | 14:59 |
| joanna | hi :) | 15:00 |
| *** sborkows has quit IRC | 15:02 | |
| rloo | mariojv: you might be interested in https://review.openstack.org/#/c/408556. I thought you had mentioned in the rolling upgrades spec that you might look into that. | 15:02 |
| patchbot | patch 408556 - ironic - Add new dbsync command online_data_migration | 15:02 |
| mariojv | rloo: thanks - i'll take a look. yup, i was actually going to work on that if xek didn't have time | 15:03 |
| mariojv | i will probably take a look later in the day, trying to meet a personal deadline for rescue implementation in agent driver | 15:03 |
| rloo | mariojv: i took a quick look. would be good to get your eyes/point of view on that command. | 15:03 |
| rloo | TheJulia: would you mind -2'ing https://review.openstack.org/#/c/408556. In case I'm not here when the -2 can be removed. | 15:04 |
| patchbot | patch 408556 - ironic - Add new dbsync command online_data_migration | 15:04 |
| TheJulia | rloo: will do, gerrit is making me login.... for like the 8th time today :( | 15:04 |
| jroll | rloo: idk that we need to -2 that one | 15:05 |
| mariojv | TheJulia: that's happened to me a lot when i'm logged in from a previous day session in one window and a new one in another window | 15:05 |
| jroll | I think it's useful on it's own | 15:05 |
| rloo | jroll: it is associated with an rfe that we haven't approved | 15:05 |
| mariojv | TheJulia: i've solved by closing all my gerrit tabs, then re-opening with the new auth cookie or whatever | 15:05 |
| jroll | rloo: oh heh | 15:05 |
| jroll | shall we just go approve that thing? | 15:06 |
| rloo | what's up with gerrit? so far (knock on wood) i am ok | 15:06 |
| rloo | jroll: no. cuz i just looked at that patch and i think the rfe should indicate more clearly what that command does. | 15:06 |
| jroll | rloo: okay | 15:06 |
| rloo | jroll: there is some online-migration thing that should have a list of functions, and the command can take some max objects to migrate. i think that should be documented w/o me having to figure out how it works by looking at the code. | 15:07 |
| TheJulia | mariojv: Same, actually about to close my browser and completely re-open... it has just never been this... grumpy. | 15:07 |
| jroll | rloo: sure, can you ask for xek to add more info there? | 15:08 |
| rloo | jroll: and i think (but not sure) that those functions will come from the rolling upgrades work. | 15:08 |
| rloo | jroll: yeah, sure. i asked the questions in the patch, and figured i'd update the rfe after i got answers :) | 15:08 |
| jroll | rloo: yeah, this RFE is around building the framework IMO, and the actual functions will come as needed | 15:08 |
| rloo | jroll: having a framework is good but... i don't know that we shoudl expose it to the user when it does nothing. | 15:09 |
| jroll | rloo: yeah, maybe we leave the setup.cfg entry point until we're actually using it? idk | 15:10 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Fix take over for ACTIVE nodes in PXEBoot https://review.openstack.org/404858 | 15:10 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Fix take over of ACTIVE nodes in AgentDeploy. https://review.openstack.org/408667 | 15:10 |
| rloo | jroll: there is no setup.cfg entry point. i don't think. don't you just do 'ironic-dbsync online-data-migration' | 15:11 |
| jroll | ohhhh | 15:11 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Fix take over of ACTIVE nodes in AgentDeploy https://review.openstack.org/408667 | 15:11 |
| rloo | jroll: hence, need more details in the rfe :) | 15:11 |
| *** glonlas has quit IRC | 15:11 | |
| jroll | indeed | 15:11 |
| jroll | rloo: I may be channeling nova here :P | 15:11 |
| rloo | jroll: would be good to know if/how different the ironic version is, to other projects | 15:12 |
| jroll | although, nova has a nova-manage command that all this goes under (migrations included) | 15:12 |
| rloo | jroll: ah, ok, so it is a 'subcommand' | 15:13 |
| rloo | jroll: and i'm guessing they make those subcommands available via setup.cfg | 15:13 |
| *** chlong has joined #openstack-ironic | 15:13 | |
| jroll | rloo: no, I was remembering wrong, setup.cfg exposes nova-manage, the rest is exposed from there | 15:13 |
| *** nmathew has quit IRC | 15:14 | |
| jroll | e.g. nova-manage db sync, nova-manage db online_data_migrations --limit <number> | 15:14 |
| jroll | they don't let you choose which online migrations to do though | 15:14 |
| rloo | jroll: oh. | 15:14 |
| rloo | jroll: we should use limit too then if nova does. the patch has max-count. | 15:14 |
| jroll | so ironic-dbsync online_data_migrations --limit <number> # makes sense to me | 15:14 |
| rloo | yup | 15:15 |
| * jroll comments on RFE | 15:16 | |
| *** glonlas has joined #openstack-ironic | 15:16 | |
| jroll | rloo: if we don't expose which commands to run, though, I'm fine with exposing and documenting online_data_migrations now | 15:16 |
| jroll | that means people will build their deployment tools to use it earlier | 15:16 |
| jroll | and thus more people do the right thing when we actually have one | 15:16 |
| rloo | jroll: so we put a 'this does nothing now' somewhere in that command? | 15:18 |
| jroll | rloo: no, it will just immediately return done (imo) | 15:18 |
| rloo | jroll: misleading. | 15:18 |
| TheJulia | I think that is extremely misleading | 15:18 |
| jroll | deployers shouldn't need to care what this does, only that they need to run it | 15:18 |
| NobodyCam | Good Morning Ironic'ers | 15:18 |
| jroll | (and release notes may tell them more, if they do care) | 15:19 |
| jroll | morning NobodyCam | 15:19 |
| TheJulia | jroll: But would they not still need to run the other command until we've switched over? | 15:19 |
| NobodyCam | Morning jroll :) | 15:19 |
| rloo | jroll, TheJulia: I think this is a juicy discussion to have at the weekly meeting. I'm fine punting on this for now, cuz who knows, everything may work at the 'same time'. | 15:19 |
| rloo | morning NobodyCam | 15:19 |
| jroll | rloo: TheJulia: I don't want our upgrade process to be "maybe run online_data_migrations, depending what release notes say" | 15:19 |
| NobodyCam | Morning rloo :) | 15:19 |
| jroll | TheJulia: regular migrations will always be run, this should always be run (but may be a noop) | 15:20 |
| rloo | jroll, TheJulia. Agree. I want our upgrade process to be complete *when* our upgrade process works :) | 15:20 |
| JayF | I have a philosophical question: What's the process for approving RFEs outside of the weekly meetings? Just 2x cores agreeing in chat? | 15:20 |
| jroll | TheJulia: maybe we return "nothing to do" instead of "done" | 15:20 |
| rloo | JayF: yes | 15:20 |
| jroll | yep | 15:20 |
| TheJulia | jroll: I agree with you there, but I think people are just going to run one command expecting it to do everything for them and potentially not know better | 15:20 |
| rloo | JayF: that's what has been done in the past. and documenting it of course in the rfe, who approves of it. | 15:20 |
| *** links has quit IRC | 15:20 | |
| jroll | TheJulia: two commands. | 15:20 |
| JayF | What do folks think about https://bugs.launchpad.net/ironic/+bug/1642719 ? I know aNuposic did some digging, we don't have any tempest tests even for the swift versino of this feature, so I think I'm onboard with approving it even w/o tempest tests. | 15:21 |
| openstack | Launchpad bug 1642719 in Ironic "[RFE] RADOSGW authentication mechanism for storing config drive in Ceph" [Wishlist,In progress] - Assigned to Anup (anup-d-navare) | 15:21 |
| TheJulia | jroll: I'm okay with letting it go, as long as we very explicitly document that both should be executed until x point to be defined in the future | 15:21 |
| rloo | JayF: i guess the no-two-from-same-company-blah-blah applies to that too | 15:21 |
| jroll | TheJulia: regular migrations, upgrade code (rolling if desired), then run online_data_migrations *any time before* the next upgrade | 15:21 |
| jroll | TheJulia: it will always be both | 15:21 |
| JayF | rloo: I mean, that's not an official rule, but yeah obviously conflicts of interest must be avoided :) | 15:21 |
| TheJulia | jroll: My perception was that it would only be one for some reason. Maybe not enough coffee this morning. | 15:22 |
| rloo | JayF: yes, not official but documented: https://wiki.openstack.org/wiki/Ironic/CoreTeam#Other_notes | 15:22 |
| jroll | TheJulia: it's one at upgrade time, and one at some point before the next upgrade to prepare for it :) | 15:23 |
| JayF | rloo: it warms my heart that we write everything down! | 15:23 |
| TheJulia | so silly question, how does one do that without upgrading a conductor or api node? *ducks* | 15:23 |
| jroll | TheJulia: the online migrations are included with release X to be performed before upgrading to X+1 :) | 15:24 |
| rloo | TheJulia: we document that. in the rolling upgrades patch, someone mentioned it. put it in a vm or something first :) | 15:24 |
| jroll | TheJulia: in the classic "move a column" case, we'd ship the new column addition and the online migrations, then in the next release delete the column | 15:24 |
| rloo | JayF: well, *some* things anyway :) | 15:24 |
| *** ijw has joined #openstack-ironic | 15:26 | |
| rloo | JayF: sorry, didn't answer your question about that rfe. I am fine w/o tempest tests but i'm not that hardcore on tests in the first place. | 15:34 |
| JayF | rloo: so are you +2 on that RFE being approved? | 15:34 |
| rloo | JayF: yeah i think. i haven't read it in detail. i just don't know/recall if we're tightening up our tempest tests, or if it only refers to 3rd party drivers. | 15:35 |
| rloo | JayF: give me a few minutes to grok that rfe | 15:35 |
| JayF | I think there's a general desire to improve our tempest test coverage | 15:35 |
| JayF | but until it's a priority we all know we aren't going to do it right :) | 15:35 |
| TheJulia | jroll: I guess my concerns regarding perceptions and velocity impact will just linger until we are knee deep into it. | 15:36 |
| JayF | and I'd feel gross if we denied the RFE for no tempest tests given the other supported backend for this feature doesn't | 15:36 |
| jroll | TheJulia: what are your concerns? | 15:36 |
| *** Syed__ has joined #openstack-ironic | 15:40 | |
| TheJulia | Reduction in our velocity to spread out changes such that we spread out development and implementation of any database impacting change to potentially multiple cycles, complication of upgrades for packagers. My perception from what you indicated earlier is x-1 will have schema additions for release, x or x-1 has the data migration for release x (my | 15:41 |
| TheJulia | perception is x would, but what I read earlier that you wrote seemed to suggest x-1 would carry it in anticipation for x), and x+1 we drop old columns | 15:41 |
| rloo | JayF: there are some things in that rfe that i don't understand. i'll comment. | 15:42 |
| *** aNuposic has joined #openstack-ironic | 15:42 | |
| *** hamzy has joined #openstack-ironic | 15:43 | |
| jroll | TheJulia: yeah, I hear you, but I think we've mostly agreed as a community that it's worth it for the deployers | 15:43 |
| JayF | rloo: Thanks! aNuposic is new at openstack so helping get to a good rfe is nice :) | 15:43 |
| JayF | aNuposic: rloo is about to comment on your RFE, you may want to take a look once she's done :) | 15:43 |
| openstackgerrit | Joanna Taryma proposed openstack/ironic: Fail ironic startup if no protocol prefix in ironic api address https://review.openstack.org/404975 | 15:43 |
| aNuposic | JayF: rloo: Sure, thanks :) | 15:44 |
| TheJulia | jroll: Indeed. Truthfully I'm kind of just wish we handled it all in the code for x and made ironic aware of how to manage it's own database. $STANDARD_INSANITY_CAVEAT ;) | 15:44 |
| TheJulia | s/wish/wishing/ | 15:45 |
| jroll | heh | 15:45 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Make _get_sensors_data concurrent https://review.openstack.org/407429 | 15:45 |
| lucasagomes | dtantsur, re: 402052 code-wise it looks good. But I wonder if we should do it, cause we are adding yet another request to BMCs (to check the current power state) and so far, all the real BMCs have no problem with issuing a power off command when it's already off. They just ignore it | 15:46 |
| lucasagomes | I feel like OVB should handle that better instead of ironic "looking before leaping" there | 15:47 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Make _get_sensors_data concurrent https://review.openstack.org/407429 | 15:47 |
| *** rama_y_ has quit IRC | 15:47 | |
| rloo | JayF, aNuposic: commented. fwiw, that was the next rfe i was going to bring up in next week's meeting :) | 15:48 |
| *** mtanino has joined #openstack-ironic | 15:49 | |
| JayF | Awesome, thanks Ruby! | 15:49 |
| *** jcoufal has quit IRC | 15:49 | |
| *** jcoufal has joined #openstack-ironic | 15:50 | |
| openstackgerrit | Aline Bousquet proposed openstack/ironic: Store agent driver’s internal fields in driver_internal_info https://review.openstack.org/408691 | 15:51 |
| *** jaosorior has quit IRC | 15:52 | |
| *** jaosorior has joined #openstack-ironic | 15:53 | |
| * TheJulia raises an eyebrow | 15:53 | |
| aNuposic | rloo: great! I will go ahead and respond to your comment :) | 15:54 |
| jlvillal | aNuposic: JayF rloo: I did ask in the RFE if it might be better to focus on fixing: https://bugs.launchpad.net/ironic/+bug/1575935 | 15:56 |
| openstack | Launchpad bug 1575935 in Ironic "Rebuild should also accept a configdrive" [Wishlist,Triaged] | 15:56 |
| jlvillal | I don't know if it would be or not. | 15:57 |
| openstackgerrit | Joanna Taryma proposed openstack/ironic: Fail ironic startup if no protocol prefix in ironic api address https://review.openstack.org/404975 | 15:58 |
| *** aNuposic has quit IRC | 15:58 | |
| rloo | jlvillal: better to focus on that than what? | 15:59 |
| rloo | jlvillal: oh, sorry, i see your comment. | 16:00 |
| jlvillal | rloo: Of course the rebuild thing could be a rat-hole and nobody is working on it yet... | 16:01 |
| mgould | qq: is the intended use case for molteniron "I am a hardware vendor and I wish to provide CI for Ironic, assigning baremetal to CI jobs in the same way nodepool assigns VMs"? | 16:02 |
| *** moshele has quit IRC | 16:03 | |
| *** sacharya has joined #openstack-ironic | 16:04 | |
| jlvillal | mgould: krtaylor can answer that. | 16:05 |
| *** daemontool has quit IRC | 16:05 | |
| jlvillal | mgould: I believe so. And I think sambetts also might have played with it. | 16:05 |
| mgould | jlvillal: thanks | 16:05 |
| *** yufei has joined #openstack-ironic | 16:06 | |
| mgould | I mentioned it over in #oooq as possibly relevant to their work on upstream CI, and then realised I couldn't actually explain it very well :-) | 16:06 |
| mgould | I don't think it *is* relevant until "test TripleO on real, possibly unusual, hardware" becomes a requirement | 16:06 |
| *** sacharya has quit IRC | 16:09 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Make _get_sensors_data concurrent https://review.openstack.org/407429 | 16:09 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Add spec for specific faults support https://review.openstack.org/334113 | 16:10 |
| xhku | Good morning everyone ! | 16:11 |
| *** jaosorior has quit IRC | 16:11 | |
| NobodyCam | Morning xhku :) | 16:12 |
| mgould | morning xhku | 16:12 |
| *** rpioso has joined #openstack-ironic | 16:13 | |
| *** aNuposic has joined #openstack-ironic | 16:14 | |
| *** zackf has joined #openstack-ironic | 16:16 | |
| lucasagomes | someone can take a quick look at https://review.openstack.org/#/c/406913/ please ? I want to enable Python3.5 jobs for virtualbmc in the gate | 16:17 |
| patchbot | patch 406913 - virtualbmc - Fix Python3 test errors | 16:17 |
| lucasagomes | thanks | 16:17 |
| *** jpich has quit IRC | 16:18 | |
| *** milan is now known as milan|afk | 16:18 | |
| *** pcaruana has quit IRC | 16:19 | |
| JayF | We have a gate-breaking bug for ironiccleint; https://bugs.launchpad.net/python-ironicclient/+bug/1648179 | 16:22 |
| openstack | Launchpad bug 1648179 in python-ironicclient "Fake node validation error: gate-ironicclient-dsvm-functional fails on test_node_validate" [Critical,New] | 16:22 |
| JayF | I'll update the whiteboard | 16:22 |
| JayF | but if anyone wants a thing to look at, that would be a extremely good choice | 16:22 |
| *** ijw has quit IRC | 16:25 | |
| *** baoli has quit IRC | 16:29 | |
| *** sacharya has joined #openstack-ironic | 16:30 | |
| rloo | lucasagomes: done | 16:31 |
| lucasagomes | rloo, thanks much! | 16:31 |
| *** rcernin has quit IRC | 16:31 | |
| * lucasagomes just submitted a patch to project-config to enable it | 16:31 | |
| lucasagomes | https://review.openstack.org/#/c/408697/ (for those interested) | 16:32 |
| patchbot | patch 408697 - openstack-infra/project-config - Enable python35-nv jobs for VirtualBMC | 16:32 |
| JayF | lucasagomes: isn't project-config still locked pending xenial migration completion? | 16:33 |
| jroll | yep | 16:33 |
| lucasagomes | JayF, yeah, well, the patch will hang there for a while :-) | 16:33 |
| *** krtaylor has quit IRC | 16:34 | |
| openstackgerrit | Merged openstack/virtualbmc: Fix Python3 test errors https://review.openstack.org/406913 | 16:35 |
| *** vinsh has joined #openstack-ironic | 16:35 | |
| *** Romanenko_K has quit IRC | 16:36 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic: Add REST API for volume connector and volume target operation https://review.openstack.org/214586 | 16:39 |
| *** srart has joined #openstack-ironic | 16:40 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/python-ironic-inspector-client: Pin functest-requirements to a specific SHA https://review.openstack.org/408712 | 16:41 |
| jroll | dtantsur: ^ | 16:42 |
| JayF | jroll: do we not do actual releases of ironic-inspector? | 16:43 |
| jroll | JayF: we do, but this needs to follow closer to master | 16:43 |
| jroll | client func tests import from inspector, and they often want to pull in updates from this | 16:44 |
| dtantsur | this ^^^ | 16:44 |
| jroll | so we'd have to release inspector for every feature that needs client support | 16:44 |
| JayF | it just feels a little gross to have a sha like that in a requirements file | 16:44 |
| JayF | that we're going to ship | 16:44 |
| JayF | if that's what we have to do, it's what we have to do, but it just seems gross to me :/ | 16:44 |
| jroll | it's functest requirements, /shrug | 16:45 |
| jroll | the other options are a bit grosser imo | 16:45 |
| JayF | I don't know the other options, and trust you all | 16:45 |
| dtantsur | yep, right | 16:45 |
| JayF | just sorta caught me by surprise when I opened it is all | 16:45 |
| jroll | JayF: the other options being: release inspector constantly, manually sync the base classes we're importing AND manually sync test-requirements | 16:45 |
| JayF | jroll: #1 should be easy, but I know nothing ever is | 16:46 |
| jroll | JayF: it's more about... when/what do we actually want to ship | 16:46 |
| JayF | oh, that makes sense | 16:46 |
| jroll | and then you're inherently shipping code that doesn't have client support yet | 16:46 |
| JayF | you want to release client support with the ser... yep | 16:46 |
| jroll | :) | 16:46 |
| JayF | man, that seems really crummy that there's not a better way to handle something like this | 16:46 |
| mgould | is anyone looking at the python-ironicclient issue yet? | 16:46 |
| dtantsur | also mind release team breaks, milestone firedrills, freezes, etc | 16:47 |
| jroll | not yet, was just reading scrollback | 16:47 |
| dtantsur | mgould, not me | 16:47 |
| JayF | mgould: not afaik, I was going to pick it up this afternoon if nobody had yet | 16:47 |
| * mgould assigns himself | 16:47 | |
| mgould | I'll let you know how I get on | 16:47 |
| *** jtomasek has quit IRC | 16:47 | |
| jroll | sounds like a config issue maybe | 16:47 |
| jroll | ironic config, to be clear | 16:47 |
| *** aNuposic has quit IRC | 16:58 | |
| mgould | jroll: cool, thanks | 16:58 |
| *** alex_xu has quit IRC | 17:00 | |
| *** ohamada has quit IRC | 17:00 | |
| *** alex_xu has joined #openstack-ironic | 17:02 | |
| *** rcernin has joined #openstack-ironic | 17:03 | |
| * jroll lunches | 17:04 | |
| *** ralonsoh_ has joined #openstack-ironic | 17:04 | |
| *** ralonsoh_ has quit IRC | 17:05 | |
| *** ralonsoh has quit IRC | 17:05 | |
| *** athomas has quit IRC | 17:09 | |
| *** krtaylor has joined #openstack-ironic | 17:10 | |
| *** ijw has joined #openstack-ironic | 17:15 | |
| *** ijw has quit IRC | 17:18 | |
| dtantsur | o/ | 17:21 |
| *** dtantsur is now known as dtantsur|afk | 17:21 | |
| *** aNuposic has joined #openstack-ironic | 17:24 | |
| mjturek | mgould: saw your question about MI. "I am a hardware vendor and I wish to provide CI for Ironic, assigning baremetal to CI jobs in the same way nodepool assigns VMs" is pretty good but just to be clear, it's about sharing a set of baremetal nodes between VMs that want to use the baremetal nodes as targets for testing. | 17:25 |
| mgould | mjturek: awesome, thanks | 17:27 |
| *** yufei has quit IRC | 17:27 | |
| mgould | so your ironic conductor lives on a VM, but you enroll BMs from molteniron as nodes? | 17:27 |
| sambetts | mgould: exaclty | 17:27 |
| mgould | sambetts: awesome, thanks | 17:28 |
| mgould | so interesting to TripleO CI people iff they start either testing (a) Ironic in overcloud with actual baremetal nodes or (b) overcloud is baremetal | 17:29 |
| mgould | whereas right now, I think they're happy with everything running on VMs | 17:30 |
| *** baoli has joined #openstack-ironic | 17:30 | |
| *** ijw has joined #openstack-ironic | 17:36 | |
| mgould | idiot question: what do people use as a server for running python-ironicclient functional tests against? | 17:39 |
| *** jkilpatr has quit IRC | 17:40 | |
| *** yufei has joined #openstack-ironic | 17:44 | |
| jroll | mgould: CI uses devstack, that's what I would use too if I wanted to do it locally | 17:44 |
| *** rama_y_ has joined #openstack-ironic | 17:45 | |
| *** e0ne has quit IRC | 17:45 | |
| mgould | jroll: OK, thanks | 17:46 |
| * mgould attempts to rescue his devstack VM from bitrot | 17:46 | |
| TheJulia | It looks like the ironic-ui launchpad page never had a bugs supervisor set. Can anyone with magical powers set that to the bugs team so we can triage/set priorities | 17:47 |
| sambetts | TheJulia: who's that project managed by us or the horizon team? | 17:50 |
| sambetts | s/managed/owned | 17:50 |
| TheJulia | It is owned by us | 17:50 |
| robcresswell | sambetts: Ironic | 17:50 |
| TheJulia | ironic-drivers team has rights on launchpad to update it | 17:51 |
| *** jkilpatr has joined #openstack-ironic | 17:51 | |
| sambetts | ohai robcresswell | 17:51 |
| robcresswell | sambetts: o/ Got a watcher on 'horizon'. I see all :p | 17:51 |
| TheJulia | ppiela: betherly: I finally created a bunch of bugs :) | 17:51 |
| *** derekh has quit IRC | 17:52 | |
| ppiela | I saw, thanks! I assigned some to myself that are already in progress. | 17:52 |
| *** ijw has quit IRC | 17:53 | |
| TheJulia | ppiela: awesome, I also marked them confirmed, but I couldn't suggest any priorities since nobody has access to right now | 17:54 |
| TheJulia | ppiela: I also kept the list of bugs to major functionality on the bottom half of the spreadsheet, aside from editing ports which one of the bugs does raise. | 17:55 |
| lucasagomes | folks I'm calling it aday | 17:55 |
| lucasagomes | I'm off tomorrow so I talk to you all on Monday | 17:56 |
| *** lucasagomes is now known as lucas-pto | 17:56 | |
| TheJulia | goodnight lucas-pto | 17:56 |
| TheJulia | have a good weekend! | 17:56 |
| lucas-pto | you too :-) | 17:56 |
| *** glonlas has quit IRC | 17:57 | |
| mgould | lucas-pto: have a good weekend! | 17:57 |
| *** ijw has joined #openstack-ironic | 17:59 | |
| jroll | TheJulia: on it | 17:59 |
| TheJulia | jroll: thank you! | 17:59 |
| jroll | and done | 18:00 |
| *** yufei has quit IRC | 18:00 | |
| *** moshele has joined #openstack-ironic | 18:00 | |
| TheJulia | \o/ | 18:01 |
| *** glonlas has joined #openstack-ironic | 18:02 | |
| *** UForgotten has quit IRC | 18:05 | |
| *** UForgotten has joined #openstack-ironic | 18:05 | |
| *** fragatina has joined #openstack-ironic | 18:06 | |
| *** ijw has quit IRC | 18:08 | |
| *** fragatina has quit IRC | 18:09 | |
| *** aNuposic has quit IRC | 18:09 | |
| *** aNuposic has joined #openstack-ironic | 18:09 | |
| *** fragatina has joined #openstack-ironic | 18:09 | |
| *** rm_work has quit IRC | 18:16 | |
| *** ijw has joined #openstack-ironic | 18:16 | |
| rloo | jroll, TheJulia, mgould, anyone interested. you all know how much I like to bikeshed... this is moving port.extra['vif_port_id'] to .internal_info['vif_port_id']. wouldn't .internal_info['tenant_vif_port_id'] be better wrt cleaning/provisioning: https://review.openstack.org/#/c/327046/28/ironic/common/network.py | 18:17 |
| patchbot | patch 327046 - ironic - Add Virtual Network Interface Driver APIs | 18:17 |
| *** harlowja has joined #openstack-ironic | 18:17 | |
| rloo | oh, but sambetts would kill me because that would mess up his unit tests | 18:18 |
| sambetts | hahaha | 18:18 |
| *** harlowja_ has joined #openstack-ironic | 18:20 | |
| *** aNuposic has quit IRC | 18:20 | |
| *** harlowja has quit IRC | 18:22 | |
| *** rm_work has joined #openstack-ironic | 18:23 | |
| *** ijw has quit IRC | 18:24 | |
| mgould | rloo: yeah, it would be consistent with cleaning_ and provisioning_ | 18:25 |
| mgould | sounds good to me | 18:25 |
| rloo | mgould: thx. i'll get you that beer now :) | 18:25 |
| *** rbudden has quit IRC | 18:27 | |
| mgould | :-) | 18:27 |
| *** MattMan_ has quit IRC | 18:28 | |
| *** MattMan has joined #openstack-ironic | 18:28 | |
| *** aweeks has joined #openstack-ironic | 18:35 | |
| openstackgerrit | Michael Turek proposed openstack/ironic-specs: Add spec for OpenBMC driver https://review.openstack.org/388218 | 18:35 |
| *** moshele has quit IRC | 18:35 | |
| mjturek | TheJulia: sorry for the delay on updating this! ^ | 18:35 |
| *** ijw_ has joined #openstack-ironic | 18:36 | |
| *** ijw_ has quit IRC | 18:36 | |
| *** ijw has joined #openstack-ironic | 18:36 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Agent driver rescue implementation https://review.openstack.org/400437 | 18:37 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 18:37 |
| mariojv | just another rebase on api/conductor, and marking agent driver rescue as not WIP even though it still needs unit tests and docs, since the rest of the code is ready for review | 18:37 |
| *** ijw has quit IRC | 18:39 | |
| JayF | mariojv: well, the api code still is -1 from me because it's not passing around rescue_password | 18:43 |
| JayF | mariojv: so I'm not sure anyone should be reviewing it yet ;) | 18:43 |
| TheJulia | rloo: doesn't the vif need to be set by the api and shouldn't internal_info be read only? | 18:44 |
| TheJulia | mjturek: No worries! :) | 18:45 |
| rloo | TheJulia: it is read only | 18:45 |
| mariojv | JayF: sure | 18:45 |
| rloo | TheJulia: and yes, it is/will be set via the API | 18:45 |
| TheJulia | rloo: I think that answers the question, at least for now then ;) | 18:46 |
| jroll | rloo: yeah, tenant_vif_port_id would make sense to me | 18:46 |
| rloo | sambetts: ^^ | 18:46 |
| rloo | thx TheJulia. and jroll | 18:47 |
| sambetts | rloo: ok thanks, I'll switch it | 18:47 |
| jroll | np | 18:47 |
| jroll | sambetts: sorry :/ | 18:47 |
| rloo | thx sambetts. sorry, i meant to ask folks sooner about it but forgot. | 18:48 |
| * mgould -> home; I've unassigned myself from | 18:49 | |
| jroll | mgould: get anywhere? | 18:50 |
| mgould | https://bugs.launchpad.net/python-ironicclient/+bug/1648179, if anyone else wants to take a look at it | 18:50 |
| openstack | Launchpad bug 1648179 in python-ironicclient "Fake node validation error: gate-ironicclient-dsvm-functional fails on test_node_validate" [Critical,Triaged] | 18:50 |
| mgould | jroll: not really, still trying to duplicate the problem locally :-/ | 18:50 |
| jroll | ok | 18:50 |
| mgould | added a couple of comments to the bug | 18:50 |
| jroll | cool, thanks | 18:50 |
| *** mgould is now known as mgould|afk | 18:53 | |
| * JayF wonders if that functional test job should be running against ironic too | 18:53 | |
| mgould|afk | JayF: if changes to ironic can break it, I think yes | 18:54 |
| * mgould|afk -> really home for real | 18:54 | |
| jroll | looking at that comment, seems maybe it's the keystone v3 change that broke it | 18:56 |
| *** ijw has joined #openstack-ironic | 18:56 | |
| *** ijw has quit IRC | 18:58 | |
| *** ChubYann has joined #openstack-ironic | 18:59 | |
| *** jkilpatr has quit IRC | 19:01 | |
| *** bfournie has quit IRC | 19:02 | |
| *** baoli has quit IRC | 19:05 | |
| *** baoli has joined #openstack-ironic | 19:05 | |
| *** Sukhdev has joined #openstack-ironic | 19:09 | |
| *** rbudden has joined #openstack-ironic | 19:10 | |
| *** ijw has joined #openstack-ironic | 19:11 | |
| *** ijw has quit IRC | 19:13 | |
| *** jkilpatr has joined #openstack-ironic | 19:17 | |
| *** ijw has joined #openstack-ironic | 19:17 | |
| *** ijw has quit IRC | 19:19 | |
| *** ijw has joined #openstack-ironic | 19:21 | |
| *** e0ne has joined #openstack-ironic | 19:25 | |
| *** strigazi is now known as strigazi_AFK | 19:26 | |
| *** ijw has quit IRC | 19:26 | |
| *** ijw has joined #openstack-ironic | 19:36 | |
| *** krtaylor has quit IRC | 19:37 | |
| *** rcernin has quit IRC | 19:43 | |
| JayF | For resuce, we're going to be adding a required argument of rescue_password when setting provision state to rescue | 19:45 |
| *** ijw_ has joined #openstack-ironic | 19:45 | |
| JayF | Should we enforce any guidelines on that password? minimum length? Character requirements? etc? | 19:45 |
| JayF | I'm leaning very heavily towards no, but am curious what the larger group thinks | 19:46 |
| JayF | jcook: mariojv ^ | 19:46 |
| JayF | jcook: also if you want I can update that API patch if you don't have time, I know it's a busy time for you | 19:46 |
| jcook | JayF I'm cool with that, I do need to get these reviews done | 19:48 |
| mariojv | JayF: i thought spec said it'd be randomly generated by nova | 19:48 |
| mariojv | and how it would be generated | 19:48 |
| mariojv | imbw though | 19:48 |
| JayF | mariojv: sure. Nova doesn't always exist though. | 19:48 |
| JayF | mariojv: we have to think about the API interface to Ironic separately from that, at least to a degree | 19:48 |
| *** ijw has quit IRC | 19:48 | |
| mariojv | sure | 19:49 |
| mariojv | we can check it in validate() | 19:49 |
| JayF | I'm thinking we shouldn't enforce any password requirements, to be clear | 19:49 |
| JayF | it just feels weird not to | 19:49 |
| JayF | and it wasn't mentioned one way or the other in the spec | 19:49 |
| JayF | hence the question | 19:49 |
| *** bfournie has joined #openstack-ironic | 19:55 | |
| *** rcernin has joined #openstack-ironic | 19:57 | |
| openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Documentation for Security Groups for baremetal servers https://review.openstack.org/393962 | 19:59 |
| *** fragatina has quit IRC | 20:03 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from OneView drivers https://review.openstack.org/397846 | 20:06 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove iBoot, WoL and AMT drivers https://review.openstack.org/397847 | 20:06 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from iLO drvers https://review.openstack.org/397845 | 20:06 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru completely https://review.openstack.org/397848 | 20:06 |
| *** fragatina has joined #openstack-ironic | 20:08 | |
| rloo | JayF: if you want a doc patch to review: https://review.openstack.org/#/c/393962/ | 20:09 |
| patchbot | patch 393962 - ironic - Documentation for Security Groups for baremetal se... | 20:09 |
| JayF | added to my queue, I'll take a look at it after lunch | 20:09 |
| rloo | JayF: thx, no hurry on that. just the last patch for an rfe. | 20:10 |
| JayF | ooh nice, I'll get it done and do the "feature done" busywork afterwards | 20:10 |
| *** ijw_ has quit IRC | 20:12 | |
| *** ijw has joined #openstack-ironic | 20:13 | |
| *** ijw has quit IRC | 20:14 | |
| *** absubram has joined #openstack-ironic | 20:16 | |
| jroll | anything I can help make progress on in the last hour or so I'm here? I'll be gone until next thursday, but will probably pop in to watch my tempest stuff | 20:16 |
| *** aNuposic has joined #openstack-ironic | 20:17 | |
| rloo | jroll: were you looking at https://bugs.launchpad.net/python-ironicclient/+bug/1648179? | 20:19 |
| openstack | Launchpad bug 1648179 in python-ironicclient "Fake node validation error: gate-ironicclient-dsvm-functional fails on test_node_validate" [Critical,Triaged] | 20:19 |
| jroll | I was not | 20:19 |
| jroll | I supposed I could | 20:19 |
| *** e0ne has quit IRC | 20:20 | |
| rloo | jroll: if no one else is, would be good if you could. until i think of something else to bug you with :) | 20:20 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru completely https://review.openstack.org/397848 | 20:24 |
| *** ijw has joined #openstack-ironic | 20:25 | |
| *** d0ugal has quit IRC | 20:29 | |
| jroll | rloo: looks like someone is on it: https://review.openstack.org/#/c/408205/ | 20:33 |
| patchbot | patch 408205 - python-ironicclient - Use identity api version 3 for OSC plugin tests | 20:33 |
| jroll | that takes care of the v3 errors | 20:33 |
| jroll | but isn't ready yet, and also needs a fix for the node validate error | 20:33 |
| rloo | jroll: isn't that the same person that reported that bug in the first place? | 20:34 |
| jroll | rloo: yes | 20:34 |
| rloo | jroll: guess there is more than problem... (I haven't actually been paying attention) | 20:34 |
| jroll | rloo: yes, there's https://bugs.launchpad.net/python-ironicclient/+bug/1648179 and https://bugs.launchpad.net/ironic/+bug/1646837 | 20:35 |
| openstack | Launchpad bug 1648179 in python-ironicclient "Fake node validation error: gate-ironicclient-dsvm-functional fails on test_node_validate" [Critical,Triaged] | 20:35 |
| *** d0ugal has joined #openstack-ironic | 20:35 | |
| openstack | Launchpad bug 1646837 in python-ironicclient "gate-ironicclient-dsvm-functional fails after change to Keystone v3" [Undecided,In progress] - Assigned to Kyrylo Romanenko (kromanenko) | 20:35 |
| *** cdearborn has quit IRC | 20:35 | |
| rloo | jroll: ok, so we have one possible fix. it is critical but not highly critical cuz most of us aren't touching the client...? | 20:36 |
| jroll | rloo: idk why a gate break wouldn't be highly critical :) | 20:37 |
| rloo | jroll: cuz i don't see people panic'ing about this :) | 20:37 |
| jroll | yeah, well | 20:37 |
| rloo | jroll: cuz most of us don't have patches in the client that are failing | 20:37 |
| jroll | people don't usually panic much about gate breakages either | 20:37 |
| JayF | ^ true | 20:38 |
| jroll | other than to say "is it fixed yet" | 20:38 |
| *** ijw has quit IRC | 20:38 | |
| JayF | there's a small portion of us who pay close attention to the gate | 20:38 |
| *** ijw has joined #openstack-ironic | 20:38 | |
| JayF | I'm going to point my brain in the direction of that gate failure when I'm done with lunch if nobody else it | 20:38 |
| JayF | *is | 20:38 |
| jroll | yeah I'm poking around now as well | 20:38 |
| jroll | but probably leaving by the time you eat | 20:39 |
| JayF | I'm mostly done eating, just on the 'break' part of the lunch break | 20:39 |
| JayF | (that's code for: watching the South Park episode that air'd last night, lol) | 20:39 |
| *** ijw has quit IRC | 20:43 | |
| *** aNuposic has quit IRC | 20:45 | |
| *** aNuposic has joined #openstack-ironic | 20:46 | |
| *** fragatina has quit IRC | 20:46 | |
| rloo | JayF, jroll: either of you good with this RFE? the code already has +2: https://bugs.launchpad.net/ironic/+bug/1648036 | 20:46 |
| openstack | Launchpad bug 1648036 in Ironic "[RFE] Add support for HCTL root device hint in Ironic" [Wishlist,In progress] - Assigned to Lucas Alvares Gomes (lucasagomes) | 20:46 |
| JayF | No problem with that at all | 20:47 |
| JayF | I'm going to mark it approved since Dmitry already has a +2 | 20:47 |
| rloo | thx JayF | 20:48 |
| JayF | nah I feel bad, I've read that IPA patch | 20:48 |
| JayF | I should've opened the bug and would've prevented the need to ask altogether :) | 20:48 |
| rloo | JayF: what, you should feel bad then! :) | 20:48 |
| *** aNuposic has quit IRC | 20:50 | |
| *** rcernin has quit IRC | 20:51 | |
| *** rcernin has joined #openstack-ironic | 20:51 | |
| *** e0ne has joined #openstack-ironic | 20:53 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface REST APIs https://review.openstack.org/404240 | 20:54 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface RPC APIs https://review.openstack.org/404239 | 20:54 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface Driver APIs https://review.openstack.org/327046 | 20:54 |
| sambetts | rloo: for you ^ :P | 20:54 |
| sambetts | now /me -> PTO until the new year | 20:54 |
| rloo | thanks Sam. Whoa, didn't realize this is your last day of work this year. | 20:55 |
| rloo | sambetts: have a great time! | 20:55 |
| sambetts | I'll be on my email and will check in when I can, vsaienk0 is going to look after my patches while I'm gone | 20:55 |
| JayF | have a good vacation sam! | 20:56 |
| rloo | sambetts: no worries, it'll all be done by the time you get back ;) | 20:56 |
| sambetts | we can only hope, hehe, thanks :D | 20:56 |
| *** mjturek has quit IRC | 20:56 | |
| *** sambetts is now known as sambetts|pto | 20:56 | |
| jroll | oh this is fun | 20:56 |
| jroll | so for https://bugs.launchpad.net/python-ironicclient/+bug/1648179? | 20:57 |
| openstack | Launchpad bug 1648179 in python-ironicclient "Fake node validation error: gate-ironicclient-dsvm-functional fails on test_node_validate" [Critical,Triaged] | 20:57 |
| jroll | we validate that the cleaning network exists, for fake networks | 20:57 |
| jroll | er, flat networks | 20:57 |
| jroll | since neutron isn't running for func tests, well | 20:57 |
| jroll | and noop isn't enabled by default, so this is a project-config change | 20:58 |
| jroll | wheeeeeeeeee | 20:58 |
| rloo | jroll: so what broke? or is that a new test? | 20:59 |
| jroll | rloo: we likely merged the code that validates that interface | 21:00 |
| jroll | couldn't add the test without it passing CI :) | 21:00 |
| rloo | jroll: ohh, ringing a bell. maybe. we did some changes for driver reform. | 21:00 |
| jroll | or it was broken and one of the driver composition patches fixed it | 21:00 |
| jroll | yeah | 21:00 |
| jroll | ¯\_(ツ)_/¯ | 21:00 |
| * jroll fixes | 21:01 | |
| JayF | jroll: nice find | 21:02 |
| JayF | jroll: do you think they'll let a gate-unblocking patch past the freeze | 21:02 |
| jroll | yeah, was a matter of couldn't reproduce, what's different? | 21:02 |
| jroll | JayF: probably, it has an extremely low probability of conflict | 21:03 |
| jroll | do you mind babysitting though? | 21:03 |
| JayF | I can | 21:03 |
| rloo | what freeze? | 21:03 |
| jroll | rloo: they're moving everything to xenial this week, project-config is frozen while that happens | 21:03 |
| rloo | jroll: OH. | 21:03 |
| rloo | jroll: can't this be fixed by changing the client's functional test? | 21:04 |
| rloo | or maybe you two are talking about something else :) | 21:04 |
| jroll | rloo: it's an ironic.conf thing | 21:04 |
| jroll | noop network driver isn't enabled, so I can't just use --network-interface when we create that node | 21:05 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Wire in storage interface attach/detach operations https://review.openstack.org/406290 | 21:05 |
| rloo | jroll: i was just thinking of changing base.assertNodeValidate() so that network interface is false :) | 21:05 |
| rloo | jroll: maybe that's cheating... | 21:06 |
| jroll | rloo: heh, we could, seems wrong | 21:06 |
| *** glonlas has quit IRC | 21:06 | |
| jroll | JayF: https://review.openstack.org/#/c/408816/ | 21:07 |
| patchbot | patch 408816 - openstack-infra/project-config - Set noop network driver for ironicclient func tests | 21:07 |
| *** rcernin has quit IRC | 21:09 | |
| jroll | JayF: given this doesn't actually fix the gate, due to https://bugs.launchpad.net/ironic/+bug/1646837 | 21:09 |
| openstack | Launchpad bug 1646837 in python-ironicclient "gate-ironicclient-dsvm-functional fails after change to Keystone v3" [Undecided,In progress] - Assigned to Kyrylo Romanenko (kromanenko) | 21:09 |
| jroll | idk how hard I would push until maybe tomorrow | 21:09 |
| jroll | I'll drop a comment in the patch for that bug though | 21:10 |
| *** chlong has quit IRC | 21:10 | |
| JayF | jroll: that should still pass though, with the patch that's up, right? | 21:10 |
| jroll | JayF: what should still pass? func tests? | 21:10 |
| openstackgerrit | Merged openstack/bifrost: Provide default to uwsgi_install_prefix https://review.openstack.org/407117 | 21:11 |
| jroll | it'll pass functional tests on https://review.openstack.org/#/c/408205/2 | 21:11 |
| patchbot | patch 408205 - python-ironicclient - Use identity api version 3 for OSC plugin tests | 21:11 |
| JayF | I thought there was a patch up to fix that bug, is what I'm saynig | 21:11 |
| jroll | but not pep8 :P | 21:11 |
| JayF | I mean, I'll fix that patch if it's otherwise fine | 21:11 |
| *** rcernin has joined #openstack-ironic | 21:11 | |
| JayF | no need to leave the gate busted just because someone is asleep | 21:11 |
| jroll | yeah idk how I feel about the fix, it's very dense so I haven't processed it in my brain | 21:12 |
| jroll | seems maybe hacky though? | 21:12 |
| JayF | I haven't even so much as glanced at it yet | 21:12 |
| jroll | oh ffs my tempest patch is conflicting | 21:13 |
| *** mjturek has joined #openstack-ironic | 21:18 | |
| jroll | okay, that's fixed up | 21:18 |
| jroll | I'm out for the night, will pop in tomorrow to try to get that tempest stuff merged, otherwise I'll be back next thursday | 21:19 |
| jroll | have a good week everyone \o | 21:19 |
| *** e0ne has quit IRC | 21:19 | |
| *** e0ne has joined #openstack-ironic | 21:20 | |
| TheJulia | you too jroll | 21:21 |
| jlvillal | jroll: Ciao! | 21:22 |
| rloo | have a great time jroll! | 21:23 |
| *** e0ne has quit IRC | 21:24 | |
| *** e0ne has joined #openstack-ironic | 21:28 | |
| JayF | joanna: https://review.openstack.org/#/c/408032/ | 21:28 |
| patchbot | patch 408032 - ironic-python-agent - Add HCTL to BlockDevices | 21:28 |
| *** trown is now known as trown|outtypewww | 21:30 | |
| JayF | https://review.openstack.org/#/c/386255/ is a good one to land, only impacts tests, if there's an extra core around with a land-something itch | 21:35 |
| patchbot | patch 386255 - ironic - Mock ironic-lib properly in test_deploy_utils | 21:35 |
| rloo | JayF: looking... | 21:36 |
| *** e0ne has quit IRC | 21:36 | |
| *** e0ne has joined #openstack-ironic | 21:37 | |
| *** mjturek has quit IRC | 21:39 | |
| *** hamzy has quit IRC | 21:41 | |
| mariojv | rloo: thanks for landing that, i'll follow up with the autospec tomorrow | 21:46 |
| *** Sukhdev has quit IRC | 21:47 | |
| rloo | yw. not a big deal, i think people forget the autospec a lot. | 21:47 |
| openstackgerrit | Merged openstack/ironic: Mock ironic-lib properly in test_deploy_utils https://review.openstack.org/386255 | 21:48 |
| rloo | JayF: can we +A https://review.openstack.org/#/c/393962/ ? | 21:53 |
| patchbot | patch 393962 - ironic - Documentation for Security Groups for baremetal se... | 21:53 |
| JayF | rloo: we're having an OSIC review party, I'm giving the other folks a chance to review it before I W+1 | 21:53 |
| JayF | rloo: once that's done I'll land it; I'm already marking the bug as fixed :) | 21:54 |
| rloo | JayF: ahhh, ok, have fun partying! | 21:54 |
| *** fragatina has joined #openstack-ironic | 21:58 | |
| *** e0ne has quit IRC | 21:59 | |
| *** ijw has joined #openstack-ironic | 22:01 | |
| *** jcoufal_ has joined #openstack-ironic | 22:03 | |
| openstackgerrit | Merged openstack/ironic: Documentation for Security Groups for baremetal servers https://review.openstack.org/393962 | 22:05 |
| *** jcoufal has quit IRC | 22:06 | |
| *** e0ne has joined #openstack-ironic | 22:06 | |
| *** jkilpatr has quit IRC | 22:08 | |
| *** e0ne has quit IRC | 22:14 | |
| *** jcoufal_ has quit IRC | 22:15 | |
| *** jheroux has quit IRC | 22:17 | |
| *** fragatina has quit IRC | 22:18 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Add HCTL to BlockDevices https://review.openstack.org/408032 | 22:32 |
| *** baoli has quit IRC | 22:36 | |
| *** srobert_ has joined #openstack-ironic | 22:37 | |
| *** jkilpatr has joined #openstack-ironic | 22:38 | |
| *** ijw has quit IRC | 22:38 | |
| *** ijw has joined #openstack-ironic | 22:39 | |
| *** srobert has quit IRC | 22:40 | |
| *** srobert_ has quit IRC | 22:41 | |
| *** fragatina has joined #openstack-ironic | 22:41 | |
| *** sacharya has quit IRC | 22:42 | |
| *** fragatin_ has joined #openstack-ironic | 22:42 | |
| *** slagle has quit IRC | 22:44 | |
| *** ijw has quit IRC | 22:44 | |
| *** rnoriega has quit IRC | 22:44 | |
| *** marios has quit IRC | 22:45 | |
| *** lhinds has quit IRC | 22:45 | |
| *** fragatina has quit IRC | 22:46 | |
| *** aNuposic has joined #openstack-ironic | 22:47 | |
| *** ijw has joined #openstack-ironic | 22:48 | |
| *** ijw has quit IRC | 22:53 | |
| *** lindycoder has quit IRC | 22:55 | |
| *** aNuposic has quit IRC | 22:56 | |
| *** aNuposic has joined #openstack-ironic | 22:57 | |
| *** Goneri has quit IRC | 22:58 | |
| *** pmannidi has joined #openstack-ironic | 23:02 | |
| *** david-lyle_ has joined #openstack-ironic | 23:05 | |
| *** Sukhdev has joined #openstack-ironic | 23:07 | |
| *** david-lyle has quit IRC | 23:07 | |
| *** rcernin has quit IRC | 23:11 | |
| openstackgerrit | Chris Krelle proposed openstack/bifrost: Adding staging driver support https://review.openstack.org/406401 | 23:13 |
| *** absubram has quit IRC | 23:20 | |
| *** pmannidi has quit IRC | 23:25 | |
| *** pmannidi has joined #openstack-ironic | 23:27 | |
| *** rbudden has quit IRC | 23:32 | |
| *** rcernin has joined #openstack-ironic | 23:33 | |
| openstackgerrit | Merged openstack/bifrost: Fix git_root default in README https://review.openstack.org/407122 | 23:33 |
| *** david-lyle_ is now known as david-lyle | 23:35 | |
| *** sacharya has joined #openstack-ironic | 23:42 | |
| *** sacharya has quit IRC | 23:47 | |
| *** rcernin has quit IRC | 23:49 | |
| *** ijw has joined #openstack-ironic | 23:49 | |
| *** ijw has quit IRC | 23:52 | |
| *** ijw has joined #openstack-ironic | 23:53 | |
| *** awiddersheim has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!