*** aNuposic has joined #openstack-ironic | 00:00 | |
konetzed | anyone tell me how to get a node back to 'available' from 'clean failed' Provisioning State? | 00:03 |
---|---|---|
konetzed | ironic node-set-provision-state <id> manage | 00:07 |
konetzed | got it figured out! :D google was failing me earlier | 00:07 |
*** yogi has joined #openstack-ironic | 00:07 | |
*** bfournie has joined #openstack-ironic | 00:14 | |
*** soliosg has quit IRC | 00:21 | |
*** soliosg has joined #openstack-ironic | 00:22 | |
*** soliosg has quit IRC | 00:23 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-specs: Support rolling upgrades https://review.openstack.org/299245 | 00:30 |
*** hoangcx has joined #openstack-ironic | 00:41 | |
*** Goneri has joined #openstack-ironic | 00:42 | |
*** cdearborn has quit IRC | 00:45 | |
*** awiddersheim has joined #openstack-ironic | 00:46 | |
*** awiddersheim has quit IRC | 00:50 | |
*** SerenaFeng has joined #openstack-ironic | 00:57 | |
*** vgadiraj has quit IRC | 00:59 | |
*** [1]rpioso has quit IRC | 01:00 | |
*** rbudden has quit IRC | 01:05 | |
*** rama_y has quit IRC | 01:05 | |
*** zhangjl has joined #openstack-ironic | 01:08 | |
*** ijw has joined #openstack-ironic | 01:17 | |
*** ijw has quit IRC | 01:17 | |
*** ijw has joined #openstack-ironic | 01:18 | |
*** jrist has joined #openstack-ironic | 01:18 | |
*** SerenaFeng has quit IRC | 01:25 | |
*** rbudden has joined #openstack-ironic | 01:28 | |
*** rbudden has quit IRC | 01:29 | |
*** SerenaFeng has joined #openstack-ironic | 01:30 | |
*** aNuposic has quit IRC | 01:36 | |
*** aNuposic has joined #openstack-ironic | 01:37 | |
*** aNuposic has quit IRC | 01:38 | |
*** aNuposic has joined #openstack-ironic | 01:39 | |
*** mtanino has quit IRC | 01:40 | |
*** pramodrj07 has joined #openstack-ironic | 01:45 | |
*** PramodJ has joined #openstack-ironic | 01:50 | |
*** awiddersheim has joined #openstack-ironic | 02:02 | |
*** Syed_ has quit IRC | 02:06 | |
*** persia has quit IRC | 02:06 | |
*** persia has joined #openstack-ironic | 02:09 | |
openstackgerrit | zhangyanxian proposed openstack/ironic: Fix import method to follow community guideline https://review.openstack.org/405730 | 02:11 |
openstackgerrit | zhangyanxian proposed openstack/ironic: Fix import method to follow community guideline https://review.openstack.org/405730 | 02:12 |
openstackgerrit | gengchc2 proposed openstack/ironic-lib: Correct reraising of exception https://review.openstack.org/405734 | 02:15 |
*** ijw has quit IRC | 02:18 | |
*** ijw has joined #openstack-ironic | 02:19 | |
*** aNuposic has quit IRC | 02:19 | |
*** ijw_ has joined #openstack-ironic | 02:21 | |
*** ijw has quit IRC | 02:23 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 02:23 |
*** rloo has quit IRC | 02:28 | |
*** cdearborn has joined #openstack-ironic | 02:43 | |
*** yuanying_ has quit IRC | 02:48 | |
*** yuanying has joined #openstack-ironic | 02:52 | |
openstackgerrit | Ghanshyam Mann proposed openstack/ironic: Keep CONF.baremetal.driver_enabled for Tempest plugin https://review.openstack.org/400032 | 02:53 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off https://review.openstack.org/216738 | 02:54 |
*** absubram has joined #openstack-ironic | 02:54 | |
*** absubram_ has joined #openstack-ironic | 02:56 | |
*** absubram has quit IRC | 02:59 | |
*** absubram_ is now known as absubram | 02:59 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Add RPCs to support volume target operations https://review.openstack.org/285220 | 03:04 |
openstackgerrit | Ruby Loo proposed openstack/ironic: Add RPCs to support volume target operations https://review.openstack.org/285220 | 03:06 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off https://review.openstack.org/216743 | 03:06 |
*** cdearborn has quit IRC | 03:20 | |
*** mtanino has joined #openstack-ironic | 03:27 | |
*** SerenaFeng has quit IRC | 03:43 | |
*** SerenaFeng has joined #openstack-ironic | 03:43 | |
*** yuanying has quit IRC | 03:44 | |
*** Goneri has quit IRC | 03:49 | |
*** links has joined #openstack-ironic | 03:52 | |
*** SerenaFeng has quit IRC | 03:56 | |
*** zhangjl has quit IRC | 04:02 | |
*** baoli has quit IRC | 04:04 | |
*** rama_y has joined #openstack-ironic | 04:05 | |
*** SerenaFeng has joined #openstack-ironic | 04:06 | |
*** baoli has joined #openstack-ironic | 04:08 | |
*** SerenaFeng has quit IRC | 04:15 | |
*** baoli has quit IRC | 04:15 | |
*** zhangjl has joined #openstack-ironic | 04:24 | |
*** mtanino has quit IRC | 04:26 | |
*** misty-cloud has joined #openstack-ironic | 04:38 | |
*** aNuposic has joined #openstack-ironic | 04:49 | |
*** rbudden has joined #openstack-ironic | 04:50 | |
*** zackf has quit IRC | 04:53 | |
*** aNuposic has quit IRC | 05:03 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/405812 | 05:05 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/405813 | 05:05 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/405814 | 05:05 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/405886 | 05:13 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/405887 | 05:13 |
*** vmud213 has joined #openstack-ironic | 05:15 | |
*** vikrant has joined #openstack-ironic | 05:16 | |
*** fragatina has joined #openstack-ironic | 05:21 | |
*** ijw_ has quit IRC | 05:22 | |
*** zackf has joined #openstack-ironic | 05:22 | |
*** zackf has quit IRC | 05:27 | |
*** penick has joined #openstack-ironic | 05:27 | |
*** fragatina has quit IRC | 05:33 | |
*** fragatina has joined #openstack-ironic | 05:33 | |
*** fragatina has quit IRC | 05:34 | |
*** fragatina has joined #openstack-ironic | 05:38 | |
*** fragatina has quit IRC | 05:38 | |
*** fragatina has joined #openstack-ironic | 05:38 | |
*** fragatina has quit IRC | 05:38 | |
*** fragatina has joined #openstack-ironic | 05:39 | |
*** fragatina has quit IRC | 05:40 | |
*** fragatina has joined #openstack-ironic | 05:40 | |
*** fragatina has quit IRC | 05:42 | |
*** fragatina has joined #openstack-ironic | 05:43 | |
*** rbudden has quit IRC | 05:48 | |
*** bnemec has joined #openstack-ironic | 05:49 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Clean nodes stuck in CLEANING state when ir-cond restarts https://review.openstack.org/349971 | 05:49 |
*** rama_y has quit IRC | 05:51 | |
*** PramodJ has quit IRC | 05:55 | |
*** pramodrj07 has quit IRC | 05:55 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers https://review.openstack.org/325230 | 06:03 |
*** aNuposic has joined #openstack-ironic | 06:04 | |
*** jaosorior has joined #openstack-ironic | 06:06 | |
openstackgerrit | Ghanshyam Mann proposed openstack/ironic: Keep CONF.baremetal.driver_enabled for Tempest plugin https://review.openstack.org/400032 | 06:10 |
*** bnemec has quit IRC | 06:11 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers https://review.openstack.org/325230 | 06:16 |
*** moshele has joined #openstack-ironic | 06:17 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers https://review.openstack.org/325230 | 06:18 |
*** yufei has joined #openstack-ironic | 06:20 | |
*** moshele has quit IRC | 06:24 | |
*** e0ne has joined #openstack-ironic | 06:26 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Update the existing APIs due to adding get_supported_power_states https://review.openstack.org/382194 | 06:26 |
*** aNuposic has quit IRC | 06:29 | |
*** ChubYann has quit IRC | 06:31 | |
*** openstackgerrit has quit IRC | 06:33 | |
*** openstackgerrit has joined #openstack-ironic | 06:38 | |
openstackgerrit | Hironori Shiina proposed openstack/ironic: Add REST API for volume connector and volume target operation https://review.openstack.org/214586 | 06:38 |
*** penick has quit IRC | 06:39 | |
*** zackf has joined #openstack-ironic | 06:41 | |
*** zackf has quit IRC | 06:45 | |
*** e0ne has quit IRC | 06:46 | |
*** SerenaFeng has joined #openstack-ironic | 07:00 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI https://review.openstack.org/348191 | 07:02 |
*** fxpester has joined #openstack-ironic | 07:09 | |
*** jaosorior has quit IRC | 07:10 | |
*** jaosorior has joined #openstack-ironic | 07:11 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool management driver for Inject NMI https://review.openstack.org/348721 | 07:11 |
*** pester has joined #openstack-ironic | 07:13 | |
*** jrist has quit IRC | 07:14 | |
*** fxpester|2 has joined #openstack-ironic | 07:15 | |
*** jrist has joined #openstack-ironic | 07:15 | |
*** fxpester has quit IRC | 07:15 | |
*** pester has quit IRC | 07:17 | |
*** fxpester has joined #openstack-ironic | 07:18 | |
*** e0ne has joined #openstack-ironic | 07:19 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC management driver for Inject NMI https://review.openstack.org/348732 | 07:20 |
*** pester has joined #openstack-ironic | 07:20 | |
*** fxpester|2 has quit IRC | 07:20 | |
*** fxpester has quit IRC | 07:23 | |
*** fxpester has joined #openstack-ironic | 07:24 | |
*** pester has quit IRC | 07:26 | |
*** pester has joined #openstack-ironic | 07:26 | |
*** fxpester has quit IRC | 07:29 | |
*** fxpester has joined #openstack-ironic | 07:29 | |
*** pester has quit IRC | 07:32 | |
*** pester has joined #openstack-ironic | 07:32 | |
*** fxpester|2 has joined #openstack-ironic | 07:34 | |
*** ohamada has joined #openstack-ironic | 07:34 | |
*** fxpester has quit IRC | 07:34 | |
*** fxpester has joined #openstack-ironic | 07:36 | |
*** aparnav has joined #openstack-ironic | 07:36 | |
*** pester has quit IRC | 07:37 | |
*** fxpester|2 has quit IRC | 07:39 | |
*** pester has joined #openstack-ironic | 07:39 | |
*** rcernin has joined #openstack-ironic | 07:40 | |
*** fxpester has quit IRC | 07:42 | |
*** fxpester has joined #openstack-ironic | 07:42 | |
*** fxpester|2 has joined #openstack-ironic | 07:45 | |
*** pester has quit IRC | 07:45 | |
*** fxpester has quit IRC | 07:47 | |
*** fxpester has joined #openstack-ironic | 07:47 | |
*** parlos has joined #openstack-ironic | 07:48 | |
*** pester has joined #openstack-ironic | 07:49 | |
*** srart has quit IRC | 07:50 | |
*** fxpester|2 has quit IRC | 07:50 | |
*** absubram has quit IRC | 07:50 | |
*** fxpester|2 has joined #openstack-ironic | 07:52 | |
*** fxpester has quit IRC | 07:52 | |
*** pester has quit IRC | 07:55 | |
*** fxpester has joined #openstack-ironic | 07:55 | |
*** pmannidi has quit IRC | 07:56 | |
*** ccamacho has joined #openstack-ironic | 07:56 | |
*** ralonsoh has joined #openstack-ironic | 07:57 | |
*** fxpester|2 has quit IRC | 07:58 | |
*** pester has joined #openstack-ironic | 08:00 | |
*** fxpester|2 has joined #openstack-ironic | 08:03 | |
*** fxpester has quit IRC | 08:03 | |
*** openstackgerrit has quit IRC | 08:03 | |
*** pester has quit IRC | 08:05 | |
*** srart has joined #openstack-ironic | 08:06 | |
*** fxpester has joined #openstack-ironic | 08:08 | |
*** SerenaFeng has quit IRC | 08:09 | |
*** fxpester|2 has quit IRC | 08:11 | |
*** pester has joined #openstack-ironic | 08:11 | |
*** fxpester has quit IRC | 08:14 | |
*** fxpester has joined #openstack-ironic | 08:15 | |
*** fxpester|2 has joined #openstack-ironic | 08:18 | |
*** pester has quit IRC | 08:18 | |
*** fxpester has quit IRC | 08:20 | |
*** fxpester has joined #openstack-ironic | 08:21 | |
*** pester has joined #openstack-ironic | 08:23 | |
*** fxpester|2 has quit IRC | 08:23 | |
*** fxpester|2 has joined #openstack-ironic | 08:25 | |
*** fxpester has quit IRC | 08:25 | |
*** fxpester has joined #openstack-ironic | 08:27 | |
*** pester has quit IRC | 08:28 | |
*** pester has joined #openstack-ironic | 08:30 | |
*** fxpester|2 has quit IRC | 08:30 | |
*** fxpester has quit IRC | 08:33 | |
*** fxpester has joined #openstack-ironic | 08:33 | |
*** fxpester|2 has joined #openstack-ironic | 08:35 | |
*** openstackgerrit has joined #openstack-ironic | 08:36 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 08:36 |
*** pester has quit IRC | 08:36 | |
*** pester has joined #openstack-ironic | 08:37 | |
*** fxpester has quit IRC | 08:38 | |
*** fxpester|2 has quit IRC | 08:40 | |
*** e0ne has quit IRC | 08:40 | |
*** fxpester has joined #openstack-ironic | 08:41 | |
*** fxpester|2 has joined #openstack-ironic | 08:43 | |
*** pester has quit IRC | 08:43 | |
*** SerenaFeng has joined #openstack-ironic | 08:44 | |
*** pester has joined #openstack-ironic | 08:45 | |
*** jpich has joined #openstack-ironic | 08:45 | |
*** fxpester has quit IRC | 08:45 | |
*** fxpester|2 has quit IRC | 08:48 | |
*** fxpester has joined #openstack-ironic | 08:48 | |
*** romcheg has joined #openstack-ironic | 08:48 | |
*** fxpester|2 has joined #openstack-ironic | 08:50 | |
*** pester has quit IRC | 08:51 | |
*** betherly has quit IRC | 08:51 | |
*** romcheg16 has joined #openstack-ironic | 08:52 | |
*** romcheg16 has quit IRC | 08:52 | |
*** fxpester has quit IRC | 08:53 | |
*** fxpester has joined #openstack-ironic | 08:54 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add ironic resources CRUD notifications https://review.openstack.org/356541 | 08:55 |
*** Romanenko_K has joined #openstack-ironic | 08:56 | |
*** pester has joined #openstack-ironic | 08:56 | |
*** fxpester|2 has quit IRC | 08:57 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node maintenance notifications https://review.openstack.org/396239 | 08:59 |
*** zzzeek has quit IRC | 09:00 | |
*** fxpester has quit IRC | 09:00 | |
*** aarefiev_afk is now known as aarefiev | 09:00 | |
*** zzzeek has joined #openstack-ironic | 09:01 | |
*** fxpester has joined #openstack-ironic | 09:02 | |
*** openstackgerrit has quit IRC | 09:03 | |
*** openstackgerrit has joined #openstack-ironic | 09:03 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node console notifications https://review.openstack.org/397812 | 09:03 |
*** pester has quit IRC | 09:04 | |
*** pester has joined #openstack-ironic | 09:05 | |
*** fxpester has quit IRC | 09:07 | |
*** vsaienko has quit IRC | 09:09 | |
*** fxpester has joined #openstack-ironic | 09:10 | |
*** pester has quit IRC | 09:13 | |
*** pester has joined #openstack-ironic | 09:13 | |
*** yuriyz has quit IRC | 09:13 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers https://review.openstack.org/325230 | 09:15 |
*** fxpester|2 has joined #openstack-ironic | 09:15 | |
*** zhangjl has left #openstack-ironic | 09:15 | |
*** zhugaoxiao has joined #openstack-ironic | 09:16 | |
*** fxpester has quit IRC | 09:16 | |
*** pester has quit IRC | 09:18 | |
*** fxpester has joined #openstack-ironic | 09:18 | |
*** romcheg22 has joined #openstack-ironic | 09:19 | |
*** romcheg has quit IRC | 09:19 | |
*** romcheg22 is now known as romcheg | 09:19 | |
*** romcheg97 has joined #openstack-ironic | 09:21 | |
*** fxpester|2 has quit IRC | 09:21 | |
*** pester has joined #openstack-ironic | 09:21 | |
*** romcheg97 has left #openstack-ironic | 09:22 | |
*** fxpester|2 has joined #openstack-ironic | 09:24 | |
*** fxpester has quit IRC | 09:24 | |
*** pester has quit IRC | 09:27 | |
*** fxpester has joined #openstack-ironic | 09:32 | |
*** SerenaFeng has quit IRC | 09:33 | |
*** pester has joined #openstack-ironic | 09:34 | |
*** fxpester|2 has quit IRC | 09:34 | |
*** lucas-afk is now known as lucasagomes | 09:35 | |
lucasagomes | morning all | 09:35 |
*** fxpester has quit IRC | 09:36 | |
*** fxpester has joined #openstack-ironic | 09:37 | |
*** pester has quit IRC | 09:40 | |
*** pester has joined #openstack-ironic | 09:44 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow use *_ipmitool with vbmc on multinode https://review.openstack.org/406020 | 09:44 |
*** ccamacho is now known as ccamacho|afk | 09:45 | |
*** fxpester|2 has joined #openstack-ironic | 09:46 | |
*** yuriyz has joined #openstack-ironic | 09:46 | |
*** fxpester has quit IRC | 09:46 | |
openstackgerrit | paresh sao proposed openstack/ironic: [WIP] Boot from iSCSI volume https://review.openstack.org/308425 | 09:47 |
*** amoralej has joined #openstack-ironic | 09:47 | |
amoralej | hi, i reported bug https://bugs.launchpad.net/ironic/+bug/1646477 about false callback timeouts | 09:48 |
openstack | Launchpad bug 1646477 in Ironic "Callback false timeouts" [Undecided,New] | 09:48 |
amoralej | i can provide logs for several hits | 09:49 |
*** rcernin has quit IRC | 09:49 | |
*** pester has quit IRC | 09:49 | |
*** fxpester|2 has quit IRC | 09:50 | |
*** rcernin has joined #openstack-ironic | 09:50 | |
*** zhugaoxiao has quit IRC | 09:54 | |
openstackgerrit | Merged openstack/ironic-ui: Incorporate driver-validation into node-detail panels https://review.openstack.org/399695 | 09:57 |
*** e0ne has joined #openstack-ironic | 09:58 | |
vdrok | good morning ironic! | 10:01 |
*** e0ne has quit IRC | 10:02 | |
*** e0ne has joined #openstack-ironic | 10:03 | |
*** abalfour has quit IRC | 10:03 | |
*** priteau has joined #openstack-ironic | 10:03 | |
*** abalfour has joined #openstack-ironic | 10:03 | |
*** strigazi_AFK is now known as strigazi | 10:06 | |
pas-ha | morning Ironic :) TGIF :) | 10:10 |
aarefiev | morning all! | 10:17 |
*** ccamacho|afk is now known as ccamacho | 10:24 | |
*** xiaoyang has quit IRC | 10:25 | |
*** tesseract has joined #openstack-ironic | 10:27 | |
*** tesseract is now known as Guest79986 | 10:27 | |
*** slagle has quit IRC | 10:28 | |
*** e0ne has quit IRC | 10:28 | |
*** e0ne has joined #openstack-ironic | 10:29 | |
*** slagle has joined #openstack-ironic | 10:29 | |
*** dtantsur|afk is now known as dtantsur | 10:33 | |
dtantsur | morning Ironic, aarefiev, pas-ha, vdrok, lucasagomes | 10:33 |
*** milan has joined #openstack-ironic | 10:39 | |
milan | morning Ironic! :) | 10:41 |
milan | TGIF! | 10:41 |
*** mgould|afk is now known as mgould | 10:47 | |
mgould | morning dtantsur aarefiev pas-ha vdrok lucasagomes milan and everyone else! | 10:48 |
mgould | and TGIF :-) | 10:48 |
lucasagomes | o/ | 10:48 |
milan | lucasagomes, morning! :) | 10:48 |
milan | mgould, morning! :) | 10:48 |
dtantsur | hey mgould, milan! how was zastavka? | 10:49 |
milan | morning dtantsur! :) very good! But we missed you | 10:49 |
dtantsur | yeah, sorry :) | 10:51 |
*** hoangcx has quit IRC | 10:52 | |
mgould | dtantsur: I really enjoyed it | 10:52 |
mgould | how are you feeling today? | 10:52 |
dtantsur | so-so | 10:53 |
milan | :-/ | 10:54 |
milan | midcycle pleague | 10:54 |
milan | spreading in Brno | 10:54 |
mgould | dtantsur: get well soon! | 10:54 |
dtantsur | thanks! | 10:55 |
*** e0ne has quit IRC | 10:55 | |
*** e0ne has joined #openstack-ironic | 10:55 | |
vdrok | good morning lucasagomes pas-ha aarefiev dtantsur milan and mgould ! :) | 10:56 |
aarefiev | morning dtantsur, milan, lucasagomes, mgould, vdrok | 10:57 |
sambetts|afk | Morning all /me is on PTO today so I won't be around much | 10:57 |
dtantsur | hey sambetts|afk, enjoy it | 10:58 |
milan | morning aarefiev, vdrok! :) | 10:58 |
milan | morning sambetts|afk! :) | 10:58 |
milan | sambetts|afk, TGIF, no bother on PTO ;) | 10:58 |
sambetts|afk | hehe definatly TGIF | 10:59 |
sambetts|afk | definitely* | 10:59 |
sambetts|afk | I never get that spelling right | 10:59 |
milan | :) | 10:59 |
*** e0ne has quit IRC | 11:00 | |
*** ohamada has quit IRC | 11:00 | |
*** ohamada_ has joined #openstack-ironic | 11:01 | |
*** e0ne has joined #openstack-ironic | 11:01 | |
*** ohamada_ has quit IRC | 11:01 | |
*** ohamada_ has joined #openstack-ironic | 11:01 | |
*** ohamada__ has joined #openstack-ironic | 11:01 | |
*** ohamada_ has quit IRC | 11:01 | |
vsaienk0 | morning dtantsur, sambbets, vdrok, milan, pas-ha, lucasagomes and all Ironic'ers! | 11:02 |
lucasagomes | o/ | 11:03 |
sambetts|afk | o/ | 11:03 |
milan | morning vsaienk0! :) | 11:03 |
dtantsur | hey vsaienk0 | 11:03 |
openstackgerrit | Merged openstack/ironic-inspector: Document upgrade procedure https://review.openstack.org/393275 | 11:06 |
openstackgerrit | Merged openstack/ironic: Unneeded testing in DB migration of volume connector https://review.openstack.org/405553 | 11:07 |
*** vsaienko has joined #openstack-ironic | 11:08 | |
dtantsur | 11 files changed, 795 insertions(+), 66 deletions(-) | 11:08 |
dtantsur | my next patch grows too big :( | 11:08 |
mgould | sambetts|afk: what are you doing in here at all? shoo, shoo, go and have fun :-) | 11:15 |
sambetts|afk | mgould: ahaha I'm here because dtantsur's patch that merged yesterday landed me with a giant merge conflict which is playing on my mind XD | 11:15 |
sambetts|afk | mgould: blames all the dtantsur | 11:15 |
sambetts|afk | ;) | 11:16 |
dtantsur | \o/ | 11:16 |
vdrok | morning sambetts|afk :) | 11:17 |
sambetts|afk | hey vdrok | 11:17 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Support defining and loading hardware types https://review.openstack.org/336626 | 11:18 |
dtantsur | folks, can I get some reviews on https://review.openstack.org/357983 please? this blocks some of my work on tripleo (namely, supporting cleaning there :) | 11:23 |
patchbot | patch 357983 - ironic - Support names for {cleaning,provisioning}_network | 11:23 |
*** yufei has quit IRC | 11:27 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/405886 | 11:27 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Show team and repo badges on README https://review.openstack.org/402496 | 11:33 |
openstackgerrit | Merged openstack/ironic-inspector: Show team and repo badges on README https://review.openstack.org/402495 | 11:33 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Rework handling requirements for functional tests https://review.openstack.org/406077 | 11:38 |
dtantsur | milan, aarefiev please check ^^^ | 11:38 |
* milan checks | 11:38 | |
*** ohamada__ has quit IRC | 11:39 | |
*** ohamada__ has joined #openstack-ironic | 11:39 | |
*** gcb has quit IRC | 11:41 | |
*** misty-cloud has quit IRC | 11:41 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector-specs: Show team and repo badges on README https://review.openstack.org/402494 | 11:50 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/405812 | 11:54 |
*** fxpester has joined #openstack-ironic | 11:56 | |
*** pester has joined #openstack-ironic | 12:00 | |
amoralej | hi, i reported bug https://bugs.launchpad.net/ironic/+bug/1646477 about false callback timeouts | 12:01 |
openstack | Launchpad bug 1646477 in Ironic "Callback false timeouts" [Undecided,New] | 12:01 |
amoralej | i think i'm seeing it in newton also | 12:01 |
amoralej | dtantsur ^ | 12:01 |
*** gcb has joined #openstack-ironic | 12:01 | |
dtantsur | on my radar, just don't have time yet | 12:01 |
dtantsur | lucasagomes, ^^^ | 12:01 |
* lucasagomes checks | 12:02 | |
*** fxpester has quit IRC | 12:03 | |
*** fxpester has joined #openstack-ironic | 12:03 | |
*** cdearborn has joined #openstack-ironic | 12:06 | |
vsaienk0 | lucasagomes: could you please review 1 line fix for vbm ipmi_address, needed for ironic multinode job https://review.openstack.org/#/c/406020/ | 12:06 |
patchbot | patch 406020 - ironic - Allow use *_ipmitool with vbmc on multinode | 12:06 |
*** pester has quit IRC | 12:06 | |
lucasagomes | vsaienk0, hi there... sure 1 sec | 12:06 |
*** pester has joined #openstack-ironic | 12:06 | |
lucasagomes | dtantsur, I will investigate it | 12:07 |
openstackgerrit | jiang wei proposed openstack/python-ironicclient: Support i18n for baremetal port cmds https://review.openstack.org/375971 | 12:07 |
sambetts|afk | vsaienk0: literally hit and fixed the same issue in my patch while fixing it yesterday | 12:08 |
*** fxpester has quit IRC | 12:09 | |
*** fxpester has joined #openstack-ironic | 12:09 | |
*** jkilpatr has quit IRC | 12:10 | |
*** [1]cdearborn has joined #openstack-ironic | 12:10 | |
*** derekh has joined #openstack-ironic | 12:10 | |
jroll | morning y'all | 12:11 |
dtantsur | morning jroll | 12:11 |
jroll | aarefiev: not sure I understand your comment here https://review.openstack.org/#/c/404436/ | 12:11 |
patchbot | patch 404436 - ironic - Move CONF.service_available.ironic to our plugin | 12:11 |
jroll | aarefiev: there's grenade failures here, I think because we need to add that config to newton devstack plugin: https://review.openstack.org/#/c/397775/ | 12:12 |
patchbot | patch 397775 - ironic - DNM: Testing 355586 | 12:12 |
*** pester has quit IRC | 12:12 | |
*** rcernin has quit IRC | 12:12 | |
*** ohamada__ has quit IRC | 12:12 | |
*** ohamada__ has joined #openstack-ironic | 12:12 | |
*** pester has joined #openstack-ironic | 12:15 | |
*** rcernin has joined #openstack-ironic | 12:15 | |
aarefiev | jroll: the problem is grenade doesn't call ironic to configure tempest on new, updated setup, I mean ironic_configure_tempest method | 12:17 |
*** fxpester has quit IRC | 12:17 | |
*** fxpester has joined #openstack-ironic | 12:18 | |
jroll | aarefiev: right, it should use the config from the old side, right? | 12:18 |
jroll | so we need that config in stable/newton branch | 12:18 |
jroll | and even mitaka I guess | 12:18 |
aarefiev | jroll: it doesn't use old config | 12:19 |
jroll | that seems... wrong | 12:19 |
aarefiev | it generate new on and configure it | 12:19 |
aarefiev | if you remember we had issue with this, when inspector test was skipped | 12:20 |
*** pester has quit IRC | 12:21 | |
*** pester has joined #openstack-ironic | 12:21 | |
charlledaniel | morning all | 12:23 |
jroll | hrm, I thought that was fixed, else how do we configure flavors and images? | 12:23 |
*** fxpester has quit IRC | 12:24 | |
*** cdearborn has quit IRC | 12:24 | |
sambetts|afk | just a thought, literally in the last 2 days a patch landed to move the tempest devstack plugin from early in the devstack run to much later in the devstack run so we don't need to do the overriding like we do today | 12:24 |
aarefiev | jroll: I guess tempest configure all nova staff as nova isn't plugin | 12:24 |
sambetts|afk | but it also will likly overwrite any changes we make today | 12:25 |
jroll | aarefiev: that config is done in ironic_configure_tempest, though | 12:25 |
*** aparnav has quit IRC | 12:25 | |
aarefiev | jroll: we doesn't use any ironic tests in grenade | 12:25 |
jroll | aarefiev: it still needs to use a baremetal flavor to boot a node, though | 12:26 |
sambetts|afk | aarefiev: we still have to set the default flvor though | 12:26 |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Show team and repo badges on README https://review.openstack.org/402496 | 12:26 |
sambetts|afk | otherwise a node wouldn't be scheduled | 12:26 |
*** [1]cdearborn has quit IRC | 12:26 | |
vdrok | morning jroll | 12:28 |
aarefiev | hard to say where grenade configure this, maybe during tempest configuration , let me take a look | 12:29 |
jroll | if you look here, it's pretty clear that ironic config gets into tempest on the new side http://logs.openstack.org/36/404436/4/check/gate-grenade-dsvm-ironic/bed9c8c/logs/new/tempest_conf.txt.gz | 12:29 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Support names for {cleaning,provisioning}_network https://review.openstack.org/357983 | 12:29 |
dtantsur | vdrok, updated the release note ^^^ | 12:29 |
vdrok | dtantsur: thx | 12:30 |
dtantsur | jroll, this is from devstack, I think. our config runs before tempest.conf is generated | 12:30 |
sambetts|afk | it does now, it didn't use to | 12:31 |
sambetts|afk | that behaviour changed like 2 days ago | 12:31 |
jroll | I'm just even more confused now | 12:31 |
sambetts|afk | they added a test-config phase to the devstack plugins | 12:31 |
sambetts|afk | so that tempest could run late in the devstack run | 12:31 |
sambetts|afk | it used to run early | 12:32 |
dtantsur | cool, but right now it's not run at all :) | 12:32 |
*** fragatina has quit IRC | 12:32 | |
*** moshele has joined #openstack-ironic | 12:32 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Turn off tempest's multitenant network tests https://review.openstack.org/406094 | 12:32 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Move CONF.service_available.ironic to our plugin https://review.openstack.org/406095 | 12:32 |
dtantsur | I see it only in the old devstack run: http://logs.openstack.org/36/404436/4/check/gate-grenade-dsvm-ironic/bed9c8c/logs/grenade.sh.txt.gz#_2016-11-30_23_27_25_585 | 12:32 |
* jroll backports to newton and rechecks 397775, if that grenade job passes we will see | 12:32 | |
*** fragatina has joined #openstack-ironic | 12:32 | |
jroll | dtantsur: aha, yeah maybe you're right | 12:33 |
aarefiev | we are talking about grenade it doesn't run any devstack phase on update | 12:33 |
sambetts|afk | dtantsur: https://review.openstack.org/#/c/293954/ I wonder if thats a branch issue | 12:33 |
patchbot | patch 293954 - openstack-dev/devstack - Ensure testing configuration can run as late as po... (MERGED) | 12:33 |
sambetts|afk | aarefiev: if grenade doesn't run any devstack phases, how does the tempest config get removed/changed from the original one? | 12:33 |
*** fxpester has joined #openstack-ironic | 12:34 | |
aarefiev | sambetts|afk: https://github.com/openstack-dev/grenade/blob/master/upgrade-tempest#L81 | 12:35 |
aarefiev | after all services was updated | 12:35 |
jroll | sambetts|afk: which code do we duplicate in our tempest plugin that you mention in 293954? | 12:35 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add testcases for osc port group comands https://review.openstack.org/385427 | 12:35 |
aarefiev | it's the last stage | 12:35 |
vdrok | btw, another thing has merged allowing changing tempest configuration after test-config: https://review.openstack.org/405446 | 12:36 |
patchbot | patch 405446 - openstack-dev/devstack - Add test-config local.conf processing (MERGED) | 12:36 |
sambetts|afk | jroll: all of this https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L1622 | 12:36 |
mrtenio | Morning Ironic | 12:37 |
jroll | sambetts|afk: okay, I read it as our tempest plugin, sorry | 12:37 |
*** pester has quit IRC | 12:37 | |
sambetts|afk | jroll: we used to and now can go back to just overriding the devstack varibles for all those things | 12:37 |
sambetts|afk | jroll: instead of editing the file ourselves | 12:37 |
jroll | sambetts|afk: except for the tempest configs we define :P | 12:38 |
sambetts|afk | jroll: except those :) | 12:38 |
jroll | good stuff | 12:38 |
sambetts|afk | jroll: but we should set those in the new test-config phase in the devstack plugins | 12:38 |
sambetts|afk | jroll: instead of at extra or whereever it is today | 12:38 |
*** hw_wutianwei has quit IRC | 12:39 | |
jroll | sambetts|afk: yeah that's where it's called today :) https://github.com/openstack/ironic/blob/master/devstack/plugin.sh#L63 | 12:39 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal driver commands https://review.openstack.org/367359 | 12:39 |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Rework handling requirements for functional tests https://review.openstack.org/406077 | 12:40 |
*** pester has joined #openstack-ironic | 12:41 | |
aarefiev | sambetts|afk: but we configure tempest in test-config phase | 12:42 |
dtantsur | jroll, https://review.openstack.org/406100 | 12:42 |
patchbot | patch 406100 - governance - Claim supports-upgrade and follows-standard-deprec... | 12:42 |
*** fxpester|2 has joined #openstack-ironic | 12:44 | |
*** fxpester has quit IRC | 12:44 | |
*** pester has quit IRC | 12:46 | |
*** fxpester has joined #openstack-ironic | 12:46 | |
*** lucasagomes is now known as lucas-hungry | 12:47 | |
*** pester has joined #openstack-ironic | 12:49 | |
*** fxpester|2 has quit IRC | 12:49 | |
*** bfournie has quit IRC | 12:50 | |
*** fxpester|2 has joined #openstack-ironic | 12:51 | |
*** fxpester has quit IRC | 12:51 | |
*** jkilpatr has joined #openstack-ironic | 12:51 | |
aarefiev | jroll: sambetts|afk: thats how flavor is configured https://github.com/openstack-dev/devstack/blob/master/lib/tempest#L190 | 12:52 |
aarefiev | DEFAULT_INSTANCE_TYPE=baremetal | 12:52 |
*** bfournie has joined #openstack-ironic | 12:53 | |
*** fxpester has joined #openstack-ironic | 12:53 | |
*** pester has quit IRC | 12:53 | |
*** bfournie has quit IRC | 12:54 | |
*** hamzy has joined #openstack-ironic | 12:55 | |
*** pester has joined #openstack-ironic | 12:55 | |
*** fxpester|2 has quit IRC | 12:56 | |
*** fxpester|2 has joined #openstack-ironic | 12:57 | |
*** gcb has quit IRC | 12:58 | |
*** fxpester has quit IRC | 12:58 | |
* milan gone | 12:58 | |
milan | bye everyone! | 12:59 |
*** jrist has quit IRC | 12:59 | |
* mgould likewise goes; have a good weekend everyone! | 13:00 | |
*** mgould is now known as mgould|afk | 13:00 | |
*** pester has quit IRC | 13:00 | |
*** fxpester has joined #openstack-ironic | 13:00 | |
aarefiev | milan, mgould Have a nice weekend | 13:00 |
*** mrtenio has quit IRC | 13:01 | |
mgould|afk | aarefiev: thanks, will do! | 13:01 |
mgould|afk | we're going to visit ifarkas in Budapest | 13:01 |
milan | aarefiev, thx! :) | 13:01 |
mgould|afk | o/ | 13:01 |
*** milan has quit IRC | 13:01 | |
aarefiev | I'm sure you will :) | 13:01 |
*** fxpester|2 has quit IRC | 13:03 | |
openstackgerrit | Merged openstack/ironic-inspector-specs: Show team and repo badges on README https://review.openstack.org/402494 | 13:03 |
*** pester has joined #openstack-ironic | 13:04 | |
sambetts|afk | aarefiev: we used to do it that way, but then the order the plugins were loaded caused problems so we started overriding it here https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L1611 | 13:05 |
*** fxpester|2 has joined #openstack-ironic | 13:06 | |
*** fxpester has quit IRC | 13:06 | |
*** rcernin has quit IRC | 13:06 | |
sambetts|afk | aarefiev: now we can go back to using the DEFAULT_INSTANCE_TYPE again because the order of the plugins has been fixed by the new test-config phase | 13:06 |
aarefiev | sambetts|afk: maybe we are talking about diff problems | 13:06 |
aarefiev | grenade doesn't call ironic_configure_tempest | 13:07 |
sambetts|afk | aarefiev: correct | 13:07 |
aarefiev | so possible misconfiguration | 13:08 |
*** rcernin has joined #openstack-ironic | 13:08 | |
*** fxpester has joined #openstack-ironic | 13:09 | |
*** pester has quit IRC | 13:09 | |
*** pester has joined #openstack-ironic | 13:11 | |
*** fxpester|2 has quit IRC | 13:11 | |
sambetts|afk | aarefiev: what I'm trying to say is the configuration options that tempest knows about and will be setting, we should no longer be setting in ironic_configure_tempest, because the bug that was the reason we set them in there has been fixed | 13:13 |
*** fxpester|2 has joined #openstack-ironic | 13:13 | |
*** fxpester has quit IRC | 13:13 | |
sambetts|afk | aarefiev: now if configure_tempest also deletes all other configurations that already existed, the we'll still need to call ironic_configure_tempest in grenade at some point to configure the ironic specifc tempest configurations | 13:14 |
openstackgerrit | Merged openstack/ironic: Allow use *_ipmitool with vbmc on multinode https://review.openstack.org/406020 | 13:15 |
*** pester has quit IRC | 13:16 | |
*** fxpester has joined #openstack-ironic | 13:16 | |
*** links has quit IRC | 13:17 | |
*** fxpester|2 has quit IRC | 13:19 | |
*** pester has joined #openstack-ironic | 13:19 | |
openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 13:21 |
*** srobert has joined #openstack-ironic | 13:21 | |
*** fxpester has quit IRC | 13:21 | |
aarefiev | sambetts|afk: completely agree with that, as for [service-available]ironic , it's configured in tempest right now, so ironic option doesn't take effect | 13:21 |
*** srobert has quit IRC | 13:22 | |
*** jtomasek has joined #openstack-ironic | 13:22 | |
*** srobert has joined #openstack-ironic | 13:22 | |
*** pester has quit IRC | 13:23 | |
aarefiev | or at least when we are relying on devstack enable_services | 13:23 |
*** moshele has quit IRC | 13:23 | |
*** jaosorior is now known as jaosorior_brb | 13:24 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: POC: ETAG supporting to enhance API evolution https://review.openstack.org/392213 | 13:24 |
sambetts|afk | aarefiev: if tempest isn't setting the option anymore does the option get removed/overriding from the tempest.conf by grenade calling configure_tempest? | 13:25 |
sambetts|afk | aarefiev: does it nuke the whole file? | 13:25 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/405812 | 13:25 |
sambetts|afk | aarefiev: or only override stuff it knows about | 13:25 |
*** bfournie has joined #openstack-ironic | 13:26 | |
aarefiev | sambetts|afk: option doesn't get removed, configure_tempest only override it, also it is new config | 13:27 |
aarefiev | so if we set some option in ironic on old config, it could be missing in new one | 13:28 |
sambetts|afk | aarefiev: so because ironic set the option in the first devstack run in ironic_config_tempest, it should be there in the new tempest run, because tempest won't override it | 13:29 |
jroll | https://bugs.launchpad.net/ironic/+bug/1646477 | 13:29 |
openstack | Launchpad bug 1646477 in Ironic "Callback false timeouts" [Undecided,New] - Assigned to Lucas Alvares Gomes (lucasagomes) | 13:29 |
*** [1]cdearborn has joined #openstack-ironic | 13:29 | |
jroll | lucas-hungry: need any help on that one | 13:29 |
jroll | ? | 13:29 |
*** dtantsur is now known as dtantsur|brb | 13:30 | |
*** sambetts|afk is now known as sambetts|pto | 13:30 | |
aarefiev | sambetts|pto: imo, as you said before, grenade should call ironic to configure it | 13:30 |
sambetts|pto | aarefiev: yeah, we should handle that in our grenade plugin | 13:31 |
*** krtaylor has quit IRC | 13:31 | |
sambetts|pto | aarefiev: somewhere here problably https://github.com/openstack/ironic/blob/master/devstack/upgrade/upgrade.sh#L134 | 13:31 |
aarefiev | sambetts|pto: we can't do this in ironic grenade plugin becase there is no new tempest | 13:32 |
aarefiev | yet | 13:32 |
sambetts|pto | aarefiev: if we aren't setting anything that tempest is going to override, then it doesn't amtter | 13:32 |
*** vmud213 has quit IRC | 13:32 | |
sambetts|pto | isn't* | 13:32 |
sambetts|pto | no is | 13:33 |
sambetts|pto | I was right the first time X | 13:33 |
sambetts|pto | DXD | 13:33 |
*** dprince has joined #openstack-ironic | 13:33 | |
aarefiev | the problem we can't set it, and not that tempest can to override it :) | 13:34 |
aarefiev | because there are no config yet | 13:34 |
sambetts|pto | you said tempest doesn't nuke the config | 13:34 |
sambetts|pto | it just overrides stuff | 13:34 |
sambetts|pto | so the old config will still be there | 13:35 |
sambetts|pto | we can just write to it | 13:35 |
sambetts|pto | the same as we always do | 13:35 |
aarefiev | sambetts|pto: no, there will be new config | 13:36 |
aarefiev | https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L302-L313 | 13:37 |
aarefiev | but probably we can copy it | 13:39 |
sambetts|pto | tbh I'm not sure why grenade is even touching tempest | 13:40 |
sambetts|pto | tempest is branch less | 13:40 |
Romanenko_K | Hi! | 13:40 |
sambetts|pto | so is our tempest plugin | 13:40 |
jroll | sure hope these centos ci boxes are ephemeral >.> | 13:42 |
Romanenko_K | From yesterday gate-ironicclient-dsvm-functional job fails on reviews for ironicclient. | 13:42 |
aarefiev | sambetts|pto: me either, we are running only smoke | 13:42 |
Romanenko_K | https://bugs.launchpad.net/ironic/+bug/1646837 | 13:43 |
openstack | Launchpad bug 1646837 in Ironic "gate-ironicclient-dsvm-functional fails after change to Keystone v3" [Undecided,New] | 13:43 |
*** zackf has joined #openstack-ironic | 13:44 | |
*** baoli has joined #openstack-ironic | 13:55 | |
aarefiev | Romanenko_K: looks like tempest cli client doesn't support keystone v3 | 13:56 |
aarefiev | you need to pass domain info to client object | 13:57 |
jroll | the first two patches in this chain look ready to me, if someone has time https://review.openstack.org/#/c/285219/43 | 13:57 |
patchbot | patch 285219 - ironic - Add VolumeTarget object | 13:57 |
*** baoli has quit IRC | 13:57 | |
*** baoli has joined #openstack-ironic | 13:57 | |
*** lucas-hungry is now known as lucasagomes | 14:01 | |
*** makowals has quit IRC | 14:02 | |
lucasagomes | jroll, hi there, on that bug ? So I've to confirm, but I think that when the callback is invoked at https://github.com/openstack/ironic/blob/master/ironic/conductor/base_manager.py#L359 | 14:02 |
lucasagomes | the lock gets released at the end of the execution scope from _fail_if_in_state() so the task passed to the callback doesn't hold an exclusive lock anymore | 14:03 |
lucasagomes | which might explain why the node gets to deploying from deploywait in the meantime | 14:04 |
jroll | lucasagomes: how'd we reach a timeout, though? | 14:04 |
lucasagomes | jroll, there's a periodic task running checking the wait states | 14:05 |
lucasagomes | but still, I need to dig more to see what happened cause the times seems too short | 14:06 |
jroll | yeah, that's what bugs me, it doesn't seem like it should have timed out | 14:06 |
jroll | unless you think it timed out and we got the callback at the same time | 14:06 |
sambetts|pto | seems like a race between the task changing the node to active and the periodic task processing the timeout | 14:07 |
jroll | lucasagomes: also of note, I don't think the heartbeat gets an exclusive lock anymore | 14:07 |
jroll | but changing state should | 14:07 |
jroll | so idk | 14:07 |
lucasagomes | yeah | 14:07 |
*** dtantsur|brb is now known as dtantsur | 14:08 | |
*** bnemec has joined #openstack-ironic | 14:09 | |
*** jcoufal has joined #openstack-ironic | 14:10 | |
*** makowals has joined #openstack-ironic | 14:10 | |
*** makowals has quit IRC | 14:11 | |
TheJulia | good morning | 14:12 |
*** lindycoder has joined #openstack-ironic | 14:15 | |
*** athomas has joined #openstack-ironic | 14:16 | |
nicodemos | morning TheJulia and ironic'ers | 14:17 |
*** makowals has joined #openstack-ironic | 14:17 | |
*** gcb has joined #openstack-ironic | 14:18 | |
dtantsur | morning TheJulia, nicodemos | 14:18 |
lucasagomes | TheJulia, nicodemos morning | 14:18 |
vdrok | morning TheJulia and nicodemos | 14:21 |
*** athomas has quit IRC | 14:24 | |
*** athomas has joined #openstack-ironic | 14:25 | |
lucasagomes | jroll, this series of events is also strange: http://paste.openstack.org/show/591242/ | 14:25 |
lucasagomes | jroll, basically the node was in deploying state and somehow heartbeat() could get an exclusive lock | 14:25 |
lindycoder | JayF, if you have a couple minutes to chat about my comment on https://review.openstack.org/#/c/387657/ , it could be nice to find a solution | 14:25 |
patchbot | patch 387657 - ironic - Devstack will die if default driver is not enabled | 14:25 |
lucasagomes | and released it after | 14:25 |
lucasagomes | the next state change is moving from deploying to deploy fail | 14:26 |
*** vikrant has quit IRC | 14:26 | |
jroll | hrm | 14:26 |
* lucasagomes checks upgrade_lock() | 14:26 | |
*** vikrant has joined #openstack-ironic | 14:28 | |
*** rcernin has quit IRC | 14:33 | |
*** jheroux has joined #openstack-ironic | 14:35 | |
*** amoralej is now known as amoralej|lunch | 14:36 | |
*** vikrant has quit IRC | 14:38 | |
*** rcernin has joined #openstack-ironic | 14:41 | |
*** vgadiraj has joined #openstack-ironic | 14:41 | |
*** yufei has joined #openstack-ironic | 14:42 | |
*** rbudden has joined #openstack-ironic | 14:45 | |
*** Goneri has joined #openstack-ironic | 14:45 | |
*** absubram has joined #openstack-ironic | 14:48 | |
openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Skip API releated work if no api url configured https://review.openstack.org/399857 | 14:50 |
*** absubram_ has joined #openstack-ironic | 14:50 | |
*** absubram has quit IRC | 14:53 | |
*** absubram_ is now known as absubram | 14:53 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Config drive spec uses network_data.json https://review.openstack.org/315862 | 14:53 |
*** mtanino has joined #openstack-ironic | 14:54 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch unknown exceptions in validate driver ifaces https://review.openstack.org/406157 | 14:54 |
yufei | hello, ironicers, does someone has successful use ironic to deploy fedora-24-kubernetes on baremetal with the image builded by diskimage-builder ? | 14:54 |
*** rcernin has quit IRC | 14:55 | |
lucasagomes | yufei, I've deployed fedora24 (no kubernetes tho). What error are you seem when deploying it ? | 14:57 |
yufei | I’m testing this recently, for some nodes, it works well, but for others, ironic deploy the os successfuly, but the node stuck at ‘A start job is running for dev-disk(10s/no limit)’ when the os starts | 14:57 |
*** mjturek1 has left #openstack-ironic | 14:58 | |
*** mjturek1 has joined #openstack-ironic | 14:58 | |
*** rcernin has joined #openstack-ironic | 14:58 | |
lucasagomes | yufei, sure it's not something in the image with that hardware in specific ? Cause, at that point Ironic doesn't have much to do with it | 14:59 |
lucasagomes | at that point I mean, after the image has been written to the disk | 14:59 |
*** dansmith is now known as superdan | 14:59 | |
dtantsur | lucasagomes, when you have a minute, could you review https://review.openstack.org/357983 please? this is the last big thing before we can finally start bringing cleaning in tripleo | 14:59 |
patchbot | patch 357983 - ironic - Support names for {cleaning,provisioning}_network | 14:59 |
lucasagomes | dtantsur, adding to the list, cause I'm still debugging the tripleo thing with node states. But will take a look | 15:00 |
dtantsur | (which is also required for RAID to work fwiw) | 15:00 |
dtantsur | lucasagomes, sure, thanks | 15:00 |
*** mjturek1 is now known as mjturek | 15:02 | |
*** krtaylor has joined #openstack-ironic | 15:02 | |
yufei | hi, lucasagomes, I try to delpoy those nodes with ubuntu and centos, it all works well, but for fedora23 -k8s, fedora24-k8s, fedora-24-without k8s, ironic could deploy it successfuly, but os can’t starts. Can you share your fedora image to me, I want to test it. | 15:02 |
*** amoralej|lunch is now known as amoralej | 15:03 | |
*** rama_y has joined #openstack-ironic | 15:03 | |
*** jhesketh has quit IRC | 15:04 | |
lucasagomes | yufei, I don't have one handy, cause I just built it for testing purposes when working on stuff (I work for rh, so I test things on fedora/centos/rhel etc...) :-( | 15:05 |
*** jaosorior_brb is now known as jaosorior | 15:06 | |
*** jhesketh has joined #openstack-ironic | 15:06 | |
lucasagomes | yufei, so hmmm quickly googled about similar errors... Saw a couple of people reporting that the boot can get stuck at that point because it's waiting for a partition with a given uuid in fstab to appear | 15:07 |
lucasagomes | yufei, is it possible to check the /etc/fstab in your image ? | 15:08 |
lucasagomes | also, is it a partition image (just the partition filesystem, no partition table) or a whole disk image (an imagine with partition table already) ? | 15:09 |
yufei | I checket that, it’s the same as the fstab in node which os starts successfuly | 15:10 |
yufei | I have three types of servers, one supports fedora, the other two never works with fedora, but all the three types works well with ubuntu and centos | 15:11 |
*** jaosorior has quit IRC | 15:11 | |
*** jaosorior has joined #openstack-ironic | 15:12 | |
*** yogi has quit IRC | 15:12 | |
*** jaosorior has quit IRC | 15:13 | |
*** jaosorior has joined #openstack-ironic | 15:13 | |
TheJulia | yufei: Are you trying to image deployed machines and then deploy that image out? | 15:14 |
openstackgerrit | Merged openstack/python-ironicclient: Update release note for fix to required args https://review.openstack.org/405445 | 15:18 |
*** Syed__ has joined #openstack-ironic | 15:18 | |
yufei | hi, TheJulia, Do you mean that dd the deployed disk to other machines? | 15:19 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Serve boot configs from Ironic API https://review.openstack.org/369438 | 15:20 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Implement Driver API for dynamic boot config https://review.openstack.org/394398 | 15:20 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get the boot config https://review.openstack.org/394399 | 15:20 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Allow not writing pxe configs to disk https://review.openstack.org/394397 | 15:20 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot config to proper Jinja template https://review.openstack.org/393462 | 15:20 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE https://review.openstack.org/391943 | 15:20 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: DNM testing stateless conductor https://review.openstack.org/402387 | 15:20 |
*** jkilpatr has quit IRC | 15:20 | |
*** vsaienko has quit IRC | 15:21 | |
*** jkilpatr has joined #openstack-ironic | 15:21 | |
yufei | hi, lucasagomes, I test three kinds of fedora partion image, I haven’t test whole disk image yet. | 15:23 |
yufei | which kind of image did you use? | 15:24 |
lucasagomes | yufei, right, I built one using the "fedora" element from DIB | 15:26 |
lucasagomes | e.g: diskimage-builder -o fedora-image fedora baremetal | 15:26 |
*** [1]cdearborn has quit IRC | 15:27 | |
yufei | how do you install the diskimage-builder, for source or pip? | 15:27 |
lucasagomes | yufei, either pip or packages (from RDO) | 15:28 |
lucasagomes | yufei, btw can you try booting with "systemd.log_level=debug" ? | 15:30 |
lucasagomes | or even add a "emergency" in the kernel cmd line so you drop in the emergency shell | 15:30 |
openstackgerrit | Merged openstack/ironic: Add VolumeTarget object https://review.openstack.org/285219 | 15:30 |
yufei | I didn’t do that before, should the be appened to pxe_append_params? | 15:30 |
lucasagomes | yufei, yeah | 15:30 |
lucasagomes | yufei, well... are you using local boot ? | 15:31 |
lucasagomes | if so that won't have any effect really :-/ | 15:31 |
yufei | no… | 15:31 |
yufei | Thanks you, let me have a try, try to build a whole disk image from pip insatlled builder, and add the debug to the parm | 15:32 |
yufei | You've given me a lot of inspiration :D | 15:32 |
*** openstackgerrit has quit IRC | 15:33 | |
lucasagomes | yufei, :D | 15:33 |
*** openstackgerrit has joined #openstack-ironic | 15:34 | |
openstackgerrit | Merged openstack/ironic: Cleanup adding Ironic to cluster on upgrade case https://review.openstack.org/405290 | 15:34 |
*** parlos has quit IRC | 15:34 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session glance and swift https://review.openstack.org/352289 | 15:35 |
*** vgadiraj has quit IRC | 15:36 | |
*** alexpilotti has quit IRC | 15:43 | |
*** srobert has quit IRC | 15:45 | |
openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/405814 | 15:47 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Config drive spec uses network_data.json https://review.openstack.org/315862 | 15:47 |
*** chlong has joined #openstack-ironic | 15:48 | |
openstackgerrit | Peter Piela proposed openstack/ironic-ui: Consolidate node last_error processing https://review.openstack.org/406184 | 15:48 |
*** Guest79986 has quit IRC | 15:54 | |
*** ralonsoh has quit IRC | 15:54 | |
NobodyCam | Good Morning Ironic'ers .. and (ofc) ... | 15:56 |
NobodyCam | TGIF! | 15:56 |
nicodemos | hey, NobodyCam. TGIF! =D | 15:56 |
NobodyCam | morning nicodemus :) | 15:57 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Config drive spec uses network_data.json https://review.openstack.org/315862 | 16:00 |
dtantsur | morning NobodyCam | 16:02 |
*** makowals has quit IRC | 16:02 | |
dtantsur | joanna, hey! have you had a chance to check my comment on https://review.openstack.org/#/c/336626/ ? we can chat here if this code still confuses you | 16:03 |
patchbot | patch 336626 - ironic - Support defining and loading hardware types | 16:03 |
TheJulia | Good morning nicodemos | 16:03 |
TheJulia | err | 16:03 |
TheJulia | NobodyCam: | 16:03 |
* TheJulia ponders going back to irssi | 16:03 | |
*** absubram has quit IRC | 16:05 | |
*** rcernin has quit IRC | 16:05 | |
*** ohamada__ has quit IRC | 16:10 | |
NobodyCam | Good Morning dtantsur and TheJulia :) | 16:10 |
*** srobert has joined #openstack-ironic | 16:11 | |
NobodyCam | TheJulia: it took a while for me but I believe I am good with irc cloud. I love cross device access | 16:11 |
*** ohamada has joined #openstack-ironic | 16:11 | |
*** srobert has quit IRC | 16:14 | |
*** srobert has joined #openstack-ironic | 16:15 | |
*** rpioso has joined #openstack-ironic | 16:17 | |
*** david-lyle_ has joined #openstack-ironic | 16:17 | |
openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Skip API releated work if no api url configured https://review.openstack.org/399857 | 16:17 |
*** soliosg has joined #openstack-ironic | 16:18 | |
*** david-lyle has quit IRC | 16:19 | |
*** chlong has quit IRC | 16:20 | |
*** gcb has quit IRC | 16:20 | |
*** chlong has joined #openstack-ironic | 16:21 | |
TheJulia | NobodyCam: just web browsers trying to autocorrect things like handles that have been populated since... this is a text field after all... is frustrating to no end. | 16:22 |
NobodyCam | TheJulia: ++ I have (mostly) trained my browser to understand most folks handles :p | 16:24 |
TheJulia | I've noticed my browser has gotten a little better about some, but it is still frustrating | 16:25 |
NobodyCam | hey hey TheJulia any opposition to me adding support to install staging drivers? | 16:25 |
TheJulia | NobodyCam: Absolutely none. That might save us from HAVING to implement virutalbmc real real quick, although I still want to do that | 16:26 |
TheJulia | I just haven't had time to look at either | 16:26 |
NobodyCam | +++ | 16:27 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Renaming audit map conf sample file https://review.openstack.org/350399 | 16:27 |
NobodyCam | awesome :) | 16:27 |
openstackgerrit | Merged openstack/ironic: Add RPCs to support volume target operations https://review.openstack.org/285220 | 16:28 |
TheJulia | woot \o/ | 16:28 |
TheJulia | time to rebase! | 16:28 |
NobodyCam | :p | 16:28 |
*** [1]cdearborn has joined #openstack-ironic | 16:30 | |
*** rama_y has quit IRC | 16:30 | |
vdrok | morning NobodyCam, tgif :) | 16:31 |
NobodyCam | morning vdrok :) | 16:31 |
*** soliosg has quit IRC | 16:32 | |
jlvillal | vsaienk0, vdrok: How would you feel if I proposed a change to break the multi-node job? Basically remove the TEMPEST_REGEX so it runs the tempest "smoke" test. https://review.openstack.org/#/c/406214/ | 16:32 |
patchbot | patch 406214 - openstack-infra/project-config - ironic: Have multinode job not use DEVSTACK_GATE_T... | 16:32 |
*** jistr is now known as jistr|biab | 16:33 | |
vdrok | morning jlvillal ! hah, would you fix it then? :) | 16:34 |
jlvillal | vdrok: heh :) Well we need to fix it. I will try of course. | 16:35 |
openstackgerrit | Merged openstack/ironic-lib: Updated from global requirements https://review.openstack.org/405813 | 16:35 |
vdrok | jlvillal: I agree, it's more logical to have smoke run on multinode, as the next step is grenade | 16:35 |
jlvillal | vdrok: In order to run Grenade we need to get smoke running. | 16:35 |
jlvillal | vdrok: Exactly | 16:35 |
vdrok | jlvillal: not sure how vsaienk0 would feel about ut | 16:35 |
jlvillal | vdrok: Is he around today? | 16:35 |
vdrok | jlvillal: he was, not anymore | 16:36 |
jlvillal | vdrok: Okay. Yeah it is getting late there I think. | 16:36 |
rpioso | I submitted a cherry-pick patch for stable/newton, https://review.openstack.org/#/c/405537/. It needs one more +2 and a Workflow+1. Poke :) If someone has some time to put eyes on it, I would appreciate it. | 16:37 |
patchbot | patch 405537 - ironic (stable/newton) - Make all IronicExceptions RPC-serializable | 16:37 |
JayF | rpioso: looking | 16:37 |
vdrok | jlvillal: do you know which tests are failing already? maybe I can start looking into that too | 16:37 |
rpioso | JayF: ty | 16:37 |
vdrok | morning rpioso and JayF :) | 16:38 |
rpioso | vdrok: Good afternoon :) | 16:38 |
jlvillal | vdrok: Uh. I changed the "multitenant" job locally to run with 7 VMs and deleted the REGEX. | 16:38 |
jlvillal | vdrok: Would those log files work? | 16:38 |
*** jaosorior has quit IRC | 16:38 | |
vdrok | jlvillal: I'd be fine even with just test names for start :) | 16:39 |
JayF | rpioso: +2A | 16:39 |
lindycoder | Hi JayF | 16:39 |
lindycoder | Sorry to bother have you had time to check out my comment https://review.openstack.org/#/c/387657/ | 16:40 |
patchbot | patch 387657 - ironic - Devstack will die if default driver is not enabled | 16:40 |
JayF | lindycoder: I saw it; I think to add support for VirtualPDU will end up requiring a patch to devstack_gate | 16:40 |
rpioso | JayF: \o/ Thanks, again! | 16:40 |
lindycoder | JayF, well i'll get right on that then :) | 16:41 |
jlvillal | vdrok: Let me find something | 16:41 |
JayF | lindycoder: all those gate configs are split between devstack-gate, defaults in the ironic plugin, and project-config | 16:43 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/405812 | 16:44 |
*** jlvillal has quit IRC | 16:45 | |
*** rcernin has joined #openstack-ironic | 16:45 | |
lindycoder | JayF, here's devstack-gatse https://review.openstack.org/#/c/406221/ should we add it to the defaults also in ironic? (in devstack/lib/ironic) | 16:45 |
patchbot | patch 406221 - openstack-infra/devstack-gate - Add pxe_snmp to enabled drivers | 16:46 |
JayF | lindycoder: probably need it on L414 as well | 16:47 |
lindycoder | JayF, therE's no agent_snmp driver atm | 16:48 |
*** Romanenko_K has quit IRC | 16:48 | |
JayF | lindycoder: oh, TIL :) | 16:48 |
lindycoder | JayF, thanks for the help, we'll get that to merge and update our change! | 16:49 |
dtantsur | TheJulia, do you plan (in the near future) to support anything but Cinder (directly iscsi, dunno) in your boot-from-volume work? | 16:49 |
*** strigazi is now known as strigazi_AFK | 16:50 | |
TheJulia | dtantsur: as in something where cinder is not involved at all? Because direct iscsi should work once we have attach/detach wired in with skipping deploy (which is next on my list after rebasing the entire set upon master right now) | 16:51 |
TheJulia | well attach/detach with cinder | 16:51 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/405887 | 16:52 |
TheJulia | dtantsur: not sure what your getting at in terms of management of the data and storage either, but it sounds like HBAs are what your expecting hardware side? | 16:52 |
dtantsur | TheJulia, yeah, I assume some folks don't want cinder (still figuring out why) | 16:52 |
* dtantsur is quite confused by what people want tbh | 16:52 | |
TheJulia | dtantsur: if you want to setup a discussion, more than happy to participate | 16:52 |
dtantsur | cool, thanks! I'll collect more background myself first | 16:53 |
openstackgerrit | Merged openstack/ironic: Support names for {cleaning,provisioning}_network https://review.openstack.org/357983 | 16:54 |
dtantsur | w00t | 16:54 |
*** e0ne has quit IRC | 16:54 | |
openstackgerrit | Merged openstack/ironic: Update the links in iLO documentation https://review.openstack.org/405084 | 16:55 |
*** aNuposic has joined #openstack-ironic | 16:56 | |
*** jlvillal has joined #openstack-ironic | 16:57 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/405812 | 16:57 |
openstackgerrit | Merged openstack/bifrost: Add support to update keystone for bifrost https://review.openstack.org/380226 | 16:57 |
*** jlvillal has quit IRC | 16:58 | |
*** jlvillal has joined #openstack-ironic | 16:59 | |
openstackgerrit | Merged openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 17:00 |
openstackgerrit | Merged openstack/bifrost: Add keystone install and clouds.yaml to testing https://review.openstack.org/380228 | 17:00 |
openstackgerrit | Merged openstack/bifrost: Convert ironic.conf to template generated https://review.openstack.org/305634 | 17:00 |
openstackgerrit | Merged openstack/bifrost: Add Keystone support to ironic.conf https://review.openstack.org/382469 | 17:00 |
*** chlong has quit IRC | 17:02 | |
*** penick has joined #openstack-ironic | 17:03 | |
openstackgerrit | Peter Piela proposed openstack/ironic-ui: Consolidate node last_error processing https://review.openstack.org/406184 | 17:04 |
vdrok | ok, time to leave, good night everyone! | 17:06 |
dtantsur | g'night vdrok | 17:07 |
nicodemos | bye, vdrok | 17:07 |
*** jlvillal has quit IRC | 17:08 | |
*** jlvillal has joined #openstack-ironic | 17:08 | |
*** fxpester_home has quit IRC | 17:11 | |
*** aNuposic has quit IRC | 17:13 | |
*** aNuposic has joined #openstack-ironic | 17:14 | |
JayF | would most folks judge this as an RFE or an actual bug? https://bugs.launchpad.net/ironic/+bug/1635390 my guy says it's probably an RFE | 17:15 |
openstack | Launchpad bug 1635390 in Ironic "Replace usage of 'retrying' with 'tenacity'" [Undecided,In progress] - Assigned to Boden R (boden) | 17:15 |
*** rama_y has joined #openstack-ironic | 17:17 | |
dtantsur | JayF, it does not seem to fix a problem.. | 17:17 |
jlvillal | The joy of rebooting your ZNC instance and then figuring out that the firewall blocks all the ZNC ports... | 17:18 |
jlvillal | 30 minutes wasted :( | 17:18 |
*** zackf has quit IRC | 17:18 | |
dtantsur | ugh | 17:18 |
JayF | How many billions of dollars of productivity have been lost | 17:19 |
JayF | to overly restrictive corporate firewalls? | 17:19 |
jlvillal | Well in this case it was the firewall on my VM. | 17:20 |
jlvillal | But the proxy server at work is a huge waste of time/money for our group. | 17:21 |
*** soliosg has joined #openstack-ironic | 17:24 | |
openstackgerrit | Merged openstack/bifrost: Config drive spec uses network_data.json https://review.openstack.org/315862 | 17:24 |
dtantsur | have a good weekend folks, see you on Monday | 17:26 |
jlvillal | dtantsur: Ciao | 17:26 |
jlvillal | vdrok: Are you still around? | 17:26 |
*** dtantsur is now known as dtantsur|afk | 17:26 | |
*** jkilpatr has quit IRC | 17:29 | |
openstackgerrit | Merged openstack/ironic: Increase verbosity of devstack/lib/ironic https://review.openstack.org/370492 | 17:30 |
*** fragatina has quit IRC | 17:32 | |
*** fragatina has joined #openstack-ironic | 17:34 | |
JayF | Do we have stable branches/releases for python-ironicclient? | 17:34 |
openstackgerrit | Chris Krelle proposed openstack/bifrost: follow up to Change-Id: I4150f86fbc141dcd6fd12094acab1ca43d76910c https://review.openstack.org/406261 | 17:34 |
JayF | it appears so... | 17:35 |
jroll | yerp | 17:37 |
JayF | aight, I just found a bug that needs backports for client and for ipa, and probably ironic once done | 17:37 |
JayF | hooray | 17:37 |
*** gmmaha has quit IRC | 17:38 | |
JayF | hmm. Actually probably can't be backported, because it adds new requirements (oslo.serialization) | 17:38 |
*** fragatina has quit IRC | 17:39 | |
*** athomas has quit IRC | 17:39 | |
JayF | Well, our stable policy doesn't say anything about that... | 17:39 |
jlvillal | Has stable/liberty disappeared yet? | 17:40 |
*** gmmaha has joined #openstack-ironic | 17:40 | |
JayF | jroll: ^ WDYT? https://bugs.launchpad.net/ironic/+bug/1629068 should be backported to stable/newton in ipa/ironicclient (and probably ironic once that patch lands), but it adds a requirement for oslo.serialization that we didn't have in newton ... but since it's oslo, I imagine packages for it must already exist | 17:40 |
openstack | Launchpad bug 1629068 in Ironic "python3 incompatibility using the python-ironicclient" [Undecided,In progress] - Assigned to Nguyen Hung Phuong (phuongnh) | 17:40 |
openstackgerrit | Stephanie Miller proposed openstack/bifrost: follow up to Change-Id: I4150f86fbc141dcd6fd12094acab1ca43d76910c https://review.openstack.org/406261 | 17:40 |
jroll | JayF: g-r has stable branches, stable of projects have to match | 17:40 |
openstackgerrit | Merged openstack/ironic: Make all IronicExceptions RPC-serializable https://review.openstack.org/405537 | 17:40 |
JayF | so as long as the requirement added matches the g-r for the stable branch it's OK? | 17:41 |
jroll | yes | 17:41 |
JayF | jlvillal: mitaka+liberty are in security fixes only mode | 17:41 |
jroll | there's ofc rules around changing g-r for stable | 17:41 |
jroll | probably a tony b question | 17:41 |
*** chlong has joined #openstack-ironic | 17:41 | |
jroll | liberty is about to die, don't bother | 17:41 |
JayF | I don't think g-r needs to change | 17:41 |
JayF | just a new requirement for ironicclient | 17:41 |
openstackgerrit | Chris Krelle proposed openstack/bifrost: follow up to Change-Id: I4150f86fbc141dcd6fd12094acab1ca43d76910c https://review.openstack.org/406261 | 17:42 |
jroll | ah yep | 17:42 |
jroll | https://github.com/openstack/requirements/blob/stable/newton/global-requirements.txt#L115 | 17:42 |
jroll | nice | 17:42 |
jroll | I'm still not sure the policy with those but should be fine | 17:42 |
openstackgerrit | Jay Faulkner proposed openstack/python-ironicclient: Fix python3 compatibility when HTTP Error are returned https://review.openstack.org/406263 | 17:42 |
JayF | ^ is the patch in question, at least on the ironicclient side | 17:43 |
jroll | thanks | 17:43 |
*** ohamada has quit IRC | 17:43 | |
* jroll pokes a person | 17:43 | |
JayF | Hmm. | 17:44 |
JayF | python-ironicclient doesn't list "Newton" as a series you can target a bug at | 17:44 |
lucasagomes | jroll, I built an env using tripleo-quickstart and that minimal_pacemaker template to try to reproduce the problem, no dice so far but I will continue on monday | 17:45 |
lucasagomes | jroll, lemme know if you find something before (not sure if you are investigating it or not) | 17:45 |
jroll | lucasagomes: right on, I gave up for a bit but will let you know if I look at it | 17:45 |
lucasagomes | cool | 17:45 |
*** rcernin has quit IRC | 17:45 | |
*** jcoufal has quit IRC | 17:46 | |
lucasagomes | I'm calling it a day | 17:46 |
lucasagomes | have a great evening all, enjoy the weekend | 17:46 |
*** rcernin has joined #openstack-ironic | 17:46 | |
*** jtomasek has quit IRC | 17:46 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Python 3 Compatible JSON https://review.openstack.org/406266 | 17:46 |
*** lucasagomes is now known as lucas-afk | 17:46 | |
*** bnemec has quit IRC | 17:47 | |
jroll | later lucas-afk | 17:48 |
openstackgerrit | Stephanie Miller proposed openstack/bifrost: follow up to Change-Id: I4150f86fbc141dcd6fd12094acab1ca43d76910c https://review.openstack.org/406261 | 17:48 |
jroll | JayF: we stopped tracking series in launchpad but maybe that's a good reason to keep doing it | 17:49 |
*** jkilpatr has joined #openstack-ironic | 17:49 | |
JayF | *shrug* if I don't have to target the bugs to stable, I can not do that too | 17:49 |
JayF | that works for me | 17:49 |
JayF | lol | 17:49 |
*** jpich has quit IRC | 17:49 | |
*** derekh has quit IRC | 17:49 | |
jroll | ¯\_(ツ)_/¯ | 17:49 |
JayF | exactly | 17:49 |
*** anish has joined #openstack-ironic | 17:49 | |
JayF | Anyone have any comments on this RFE? I'm OK with approving it as it sits: https://bugs.launchpad.net/ironic-python-agent/+bug/1642515 | 17:50 |
openstack | Launchpad bug 1642515 in ironic-python-agent "[RFE] Allow IPA to skip SSL certs validation" [Undecided,In progress] - Assigned to Pavlo Shchelokovskyy (pshchelo) | 17:50 |
*** jistr|biab is now known as jistr | 17:53 | |
*** Sukhdev has joined #openstack-ironic | 17:54 | |
TheJulia | hmmm | 17:54 |
TheJulia | JayF: LGTM | 17:55 |
*** ElCoyote_ has left #openstack-ironic | 17:55 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers https://review.openstack.org/325230 | 17:56 |
openstackgerrit | Félix Bouliane proposed openstack/ironic: Add virtualpdu to ironic devstack plugin https://review.openstack.org/388154 | 17:58 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Adding keystone support to inspector config https://review.openstack.org/383425 | 18:00 |
*** ElCoyote_ has joined #openstack-ironic | 18:00 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add test script for Bifrost using keystone https://review.openstack.org/383426 | 18:00 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Update the task label for inspection https://review.openstack.org/383427 | 18:00 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone capability to base install https://review.openstack.org/385040 | 18:01 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add initial keystone use documentation https://review.openstack.org/385041 | 18:01 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Serve boot configs from Ironic API https://review.openstack.org/369438 | 18:04 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get the boot config https://review.openstack.org/394399 | 18:04 |
*** fragatina has joined #openstack-ironic | 18:04 | |
*** priteau has quit IRC | 18:05 | |
*** david-lyle_ is now known as david-lyle | 18:08 | |
openstackgerrit | octopuszhang proposed openstack/ironic: Update multitenancy docs https://review.openstack.org/397517 | 18:08 |
jlvillal | JayF: I think tonyb is going to be removing stable/liberty soon. If I understood the email that went out. | 18:13 |
JayF | jlvillal: ah, okay. I was just going by what I read on releases.openstack.org this morning. | 18:13 |
jlvillal | JayF: http://lists.openstack.org/pipermail/openstack-dev/2016-November/107717.html | 18:13 |
jlvillal | JayF: Though I'm not sure if EOL means deleting the branch or not. | 18:14 |
*** vgadiraj has joined #openstack-ironic | 18:14 | |
openstackgerrit | Merged openstack/bifrost: Enable inspector by default https://review.openstack.org/405580 | 18:14 |
JayF | https://bugs.launchpad.net/python-ironicclient/+bug/1644833 I don't think this is an RFE, I think it's a bug-bug | 18:15 |
openstack | Launchpad bug 1644833 in python-ironicclient "[RFE] Write tests for help messages for Ironic OSC plugin CLI" [Undecided,In progress] - Assigned to Kyrylo Romanenko (kromanenko) | 18:15 |
JayF | If no objections going to mark it that way | 18:15 |
yufei | Could someone help take look at this patch, https://review.openstack.org/#/c/397517/ Add a note to remind user not to use provision network for instance spawning. | 18:18 |
patchbot | patch 397517 - ironic - Update multitenancy docs | 18:18 |
yufei | If it’s unnecessary now, I shall abandon it. | 18:18 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/405812 | 18:20 |
*** jcoufal has joined #openstack-ironic | 18:21 | |
JayF | yufei: it's a useful thing, I think. Got a few comments about the wording I'll put inline. | 18:21 |
yufei | I see your comments now, thanks. Also thank you for review at the patch ‘Skip API releated work if no api url configured’ | 18:24 |
yufei | I will modify it tomorrow. | 18:24 |
JayF | no problem | 18:25 |
JayF | thanks for contributing | 18:25 |
yufei | :) | 18:25 |
*** pramodrj07 has joined #openstack-ironic | 18:25 | |
*** PramodJ has joined #openstack-ironic | 18:25 | |
*** fragatina has quit IRC | 18:27 | |
*** fragatina has joined #openstack-ironic | 18:27 | |
openstackgerrit | Chris Krelle proposed openstack/bifrost: Add initial keystone use documentation https://review.openstack.org/385041 | 18:31 |
*** daemontool has joined #openstack-ironic | 18:32 | |
openstackgerrit | octopuszhang proposed openstack/ironic: Update multitenancy docs https://review.openstack.org/397517 | 18:32 |
openstackgerrit | Chris Krelle proposed openstack/bifrost: Add test script for Bifrost using keystone https://review.openstack.org/383426 | 18:33 |
*** e0ne has joined #openstack-ironic | 18:33 | |
openstackgerrit | Chris Krelle proposed openstack/bifrost: Update the task label for inspection https://review.openstack.org/383427 | 18:34 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone capability to base install https://review.openstack.org/385040 | 18:36 |
*** e0ne has quit IRC | 18:36 | |
*** daemontool has quit IRC | 18:38 | |
*** karts has joined #openstack-ironic | 18:39 | |
*** karthiks has quit IRC | 18:39 | |
*** yufei has quit IRC | 18:39 | |
lindycoder | Hello people with devstack knowledge, the IRONIC_ENABLED_DRIVERS's default are currently fake,pxe_ssh,pxe_ipmitool, i was speaking with people of devstack-gatse, in which this is override by either this same set or an "agent_*", so in gates this default is not used. | 18:39 |
lindycoder | so i'm wondering what is the goal of this, is it to list all drivers that will work with a default devstack or all drivers that could work without outside specials components? | 18:40 |
lindycoder | if it is the latter could we have the IRONIC_ENABLED_DRIVERS's default = fake,agent_ssh,agent_ipmitool,pxe_ssh,pxe_ipmitool,pxe_snmp ? | 18:41 |
JayF | lindycoder: basically asking: is there harm in loading more drivers than we need there | 18:41 |
JayF | lindycoder: is what it sounds like it boils down to | 18:41 |
openstackgerrit | Merged openstack/bifrost: Add initial keystone use documentation https://review.openstack.org/385041 | 18:41 |
lindycoder | JayF, ok, the gate is being given a driver to run tests against, maybe the gate should just set the list of enabled drivers to one and only one driver, the one we use ? | 18:43 |
*** jheroux has quit IRC | 18:44 | |
*** karts has quit IRC | 18:44 | |
JayF | I honestly don't know, I follow your logic though: it shouldn't do harm to enable all drivers we can use in devstack | 18:44 |
JayF | and just toggle the default driver based on which one is being tested | 18:44 |
JayF | but I'm curious what others would thing | 18:45 |
JayF | *think | 18:45 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add Cinder storage driver https://review.openstack.org/366197 | 18:45 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add base cinder common interface https://review.openstack.org/355625 | 18:45 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connectors and volume_targets to task https://review.openstack.org/366196 | 18:45 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add storage_interface DB field and object https://review.openstack.org/348005 | 18:45 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add storage_interface to base driver class https://review.openstack.org/348006 | 18:45 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Wire in storage interface attach/detach operations https://review.openstack.org/406290 | 18:45 |
lindycoder | Well i don't know what is the cost of loading all the drivers | 18:46 |
*** vsaienko has joined #openstack-ironic | 18:46 | |
lindycoder | do you know if gates usually use more than one driver? cause if not, let's just say in the gate that IRONIC_ENABLED_DRIVERS = IRONIC_DEPLOY_DRIVER and we good | 18:46 |
*** e0ne has joined #openstack-ironic | 18:47 | |
JayF | lindycoder: today; no, but I know in the future we want to do that | 18:47 |
JayF | lindycoder: in order to have more than one deploy method tested in a given job | 18:48 |
JayF | lindycoder: afaict the only cost in loading more drivers is conductor ram usage | 18:48 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Serve boot configs from Ironic API https://review.openstack.org/369438 | 18:48 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Implement Driver API for dynamic boot config https://review.openstack.org/394398 | 18:48 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get the boot config https://review.openstack.org/394399 | 18:48 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Allow not writing pxe configs to disk https://review.openstack.org/394397 | 18:48 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot config to proper Jinja template https://review.openstack.org/393462 | 18:48 |
*** e0ne has quit IRC | 18:48 | |
*** karts has joined #openstack-ironic | 18:49 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE https://review.openstack.org/391943 | 18:49 |
*** soliosg has quit IRC | 18:49 | |
lindycoder | JayF, then maybe the gate could simply append the IRONIC_DEPLOY_DRIVER to IRONIC_ENABLED_DRIVERS and it wouldn't break anything and let new drivers make their way more easily | 18:51 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 18:51 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 18:51 |
JayF | lindycoder: I think that's a pretty solid idea, or removing the IRONIC_ENABLED_DRIVERS entirely from devstack_gate and have the default in devstack be sufficient | 18:52 |
*** soliosg has joined #openstack-ironic | 18:52 | |
lindycoder | JayF, that would require agent_ssh,agent_ipmitool to be added permanently, and we don't know if that's a problem, any idea who would know? | 18:52 |
JayF | lindycoder: I'm OK with that addition :) | 18:53 |
JayF | lindycoder: and I'm a core reviewer, so I guess I count as someone who knows :) | 18:53 |
JayF | sometimes this stuff existed beforehand and the logic behind why it's arranged how it is gets lost over time | 18:53 |
lindycoder | JayF, yeah of course you do, you were just wondering what other would think earlier, i referred to that :P | 18:53 |
JayF | Yeah, the whole devstack-gate/project-config/default-settings-in-devstack configurations are ... strange, and I don't know always why things go where they go | 18:54 |
JayF | jlvillal: ^ any insight? | 18:54 |
jlvillal | JayF: At the top of my head...no. I am in the middle of my own gate problems at the moment... Let me read some backscroll. | 18:56 |
JayF | jlvillal: well you can go back to the gate, it's not an urgent question :) | 18:57 |
*** mrtenio has joined #openstack-ironic | 18:57 | |
jlvillal | JayF: Okay. Will do... Thanks :) | 18:57 |
*** PramodJ has quit IRC | 18:58 | |
*** pramodrj07 has quit IRC | 18:58 | |
*** ChubYann has joined #openstack-ironic | 19:02 | |
lindycoder | JayF, i'm having second thoughts... actually that seems to just push the problem away for a while, if i want to make a gate with a new driver i'll need to include it in ironic's defaults and force it on any devstack user... that's probably why it is currently defined in devstack gate. | 19:02 |
lindycoder | What about ironic's devstack script support an "Additional enabled drivers" that would be set by project-config, and appended to whatever is given as a list of enabled drivers. | 19:02 |
*** vsaienko has quit IRC | 19:02 | |
lindycoder | This would let destack-gate get rid of their override as they wish, will still work with third party gates (which can switch to the new flag) and we wouldn't need to update devstack to support all drivers we want to tests in a gate | 19:02 |
JayF | lindycoder: aiui a lot of those settings being in devstack-gate is for legacy reasons | 19:03 |
JayF | lindycoder: which is why I don't think it's bad to move it to devstack defaults | 19:04 |
openstackgerrit | Merged openstack/bifrost: follow up to Change-Id: I4150f86fbc141dcd6fd12094acab1ca43d76910c https://review.openstack.org/406261 | 19:04 |
lindycoder | JayF, i agree just seems counter intuitive to have more and more drivers loaded by default everytime we add a gate for a new driver, but if you're okay with that, i am :D | 19:05 |
JayF | lindycoder: well, I think it's more that it's unlikely we'll have too many more community CI'd drivers | 19:06 |
lindycoder | well xkhu and i are adding pxe_snmp... so that's one :P | 19:06 |
jlvillal | lindycoder: Hopefully you are aware of the 3rd Party CI requirement for new drivers. | 19:08 |
openstackgerrit | Merged openstack/ironic: Renaming audit map conf sample file https://review.openstack.org/350399 | 19:10 |
lindycoder | jlvillal, i'm afraid i am not, but pxe_snmp is already in ironic we're just adding a gate for it | 19:10 |
jlvillal | lindycoder: Ah okay. I got confused as thought a new driver was being added. Please ignore me :) | 19:11 |
lindycoder | jlvillal, no problem :D | 19:12 |
*** ijw has joined #openstack-ironic | 19:14 | |
openstackgerrit | Martin Roy proposed openstack/ironic: Add all drivers used in gates as devstack defaults https://review.openstack.org/406313 | 19:14 |
lindycoder | JayF, here it is ^ :) | 19:15 |
*** ijw has quit IRC | 19:15 | |
*** ijw has joined #openstack-ironic | 19:15 | |
*** Sukhdev has quit IRC | 19:21 | |
*** zackf has joined #openstack-ironic | 19:22 | |
*** soliosg has quit IRC | 19:24 | |
*** bdeschenes has joined #openstack-ironic | 19:25 | |
*** bdeschenes has quit IRC | 19:26 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Allow installing Ansible from PyPI https://review.openstack.org/406321 | 19:27 |
*** lindycoder has quit IRC | 19:29 | |
openstackgerrit | Merged openstack/bifrost: Add test script for Bifrost using keystone https://review.openstack.org/383426 | 19:32 |
openstackgerrit | Merged openstack/bifrost: Update the task label for inspection https://review.openstack.org/383427 | 19:32 |
openstackgerrit | Merged openstack/bifrost: Add keystone capability to base install https://review.openstack.org/385040 | 19:33 |
*** jkilpatr has quit IRC | 19:34 | |
*** serverascode has quit IRC | 19:34 | |
*** NobodyCam has quit IRC | 19:34 | |
*** tdurakov has quit IRC | 19:34 | |
*** vdrok has quit IRC | 19:34 | |
*** francoblanco has quit IRC | 19:34 | |
*** TheJulia has quit IRC | 19:34 | |
*** jkilpatr has joined #openstack-ironic | 19:35 | |
*** lindycoder has joined #openstack-ironic | 19:36 | |
*** dprince has quit IRC | 19:36 | |
*** moshele has joined #openstack-ironic | 19:38 | |
*** lindycoder has quit IRC | 19:39 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Adding keystone support to inspector config https://review.openstack.org/383425 | 19:39 |
*** ijw has quit IRC | 19:48 | |
*** serverascode has joined #openstack-ironic | 19:50 | |
openstackgerrit | Jesse J. Cook proposed openstack/ironic: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 19:58 |
*** ijw has joined #openstack-ironic | 20:00 | |
*** clenimar has quit IRC | 20:11 | |
*** vgadiraj has quit IRC | 20:12 | |
*** tdurakov has joined #openstack-ironic | 20:15 | |
*** vdrok has joined #openstack-ironic | 20:15 | |
*** NobodyCam has joined #openstack-ironic | 20:16 | |
*** francoblanco has joined #openstack-ironic | 20:17 | |
*** TheJulia has joined #openstack-ironic | 20:19 | |
*** vgadiraj has joined #openstack-ironic | 20:19 | |
*** vsaienko has joined #openstack-ironic | 20:22 | |
*** vsaienko has quit IRC | 20:26 | |
*** ijw has quit IRC | 20:31 | |
*** zackf has quit IRC | 20:32 | |
*** soliosg has joined #openstack-ironic | 20:41 | |
*** chlong has quit IRC | 20:42 | |
openstackgerrit | Joanna Taryma proposed openstack/ironic: Fail IPA config if no protocol prefix in ironic api address https://review.openstack.org/404975 | 20:49 |
*** amoralej is now known as amoralej|off | 20:51 | |
*** chlong has joined #openstack-ironic | 20:56 | |
*** zackf has joined #openstack-ironic | 20:56 | |
openstackgerrit | Merged openstack/bifrost: Adding keystone support to inspector config https://review.openstack.org/383425 | 20:57 |
*** devlaps has quit IRC | 20:57 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/405812 | 21:03 |
*** karts has quit IRC | 21:06 | |
*** MattMan has quit IRC | 21:07 | |
*** MattMan has joined #openstack-ironic | 21:07 | |
*** bfournie has quit IRC | 21:08 | |
*** lindycoder has joined #openstack-ironic | 21:09 | |
*** karts has joined #openstack-ironic | 21:09 | |
*** karts has quit IRC | 21:13 | |
*** srart has quit IRC | 21:17 | |
*** srobert_ has joined #openstack-ironic | 21:26 | |
*** karts has joined #openstack-ironic | 21:28 | |
*** srobert has quit IRC | 21:30 | |
*** srobert_ has quit IRC | 21:31 | |
*** ijw has joined #openstack-ironic | 21:32 | |
*** Sukhdev has joined #openstack-ironic | 21:33 | |
*** srart has joined #openstack-ironic | 21:34 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: multi-TENANT with tempest "smoke" test helper https://review.openstack.org/406368 | 21:37 |
*** ijw has quit IRC | 21:38 | |
*** srart has quit IRC | 21:38 | |
lindycoder | JayF, if we go on with having all the drivers, when we add pxe_snmp to the list, ironic conductor fails to load because that driver require the presence of pysnmp. | 21:43 |
lindycoder | it works in our working change, because when the gate uses pxe_snmp we install virtualpdu that installs pysnmp so devstack is happy | 21:44 |
lindycoder | i think having the gate control the enabled drivers is more extensible, i'll propose that instead | 21:44 |
JayF | lindycoder: cool, makes sense :) | 21:44 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: multi-TENANT with tempest "smoke" test helper https://review.openstack.org/406368 | 21:45 |
*** absubram has joined #openstack-ironic | 21:46 | |
*** bfournie has joined #openstack-ironic | 21:48 | |
*** jcoufal has quit IRC | 21:51 | |
*** moshele has quit IRC | 21:53 | |
*** jkilpatr has quit IRC | 21:53 | |
*** ijw has joined #openstack-ironic | 21:55 | |
*** srart has joined #openstack-ironic | 21:55 | |
*** jcoufal has joined #openstack-ironic | 22:01 | |
*** daemontool has joined #openstack-ironic | 22:04 | |
*** jcoufal_ has joined #openstack-ironic | 22:05 | |
*** jcoufal has quit IRC | 22:07 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: multi-TENANT with tempest "smoke" test helper https://review.openstack.org/406368 | 22:10 |
*** jcoufal_ has quit IRC | 22:10 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing tempest setting... https://review.openstack.org/406376 | 22:12 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: multi-TENANT with tempest "smoke" test helper https://review.openstack.org/406368 | 22:13 |
*** ElCoyote_ has left #openstack-ironic | 22:13 | |
*** Sukhdev has quit IRC | 22:14 | |
*** ijw has quit IRC | 22:17 | |
*** vsaienko has joined #openstack-ironic | 22:20 | |
openstackgerrit | Martin Roy proposed openstack/ironic: Add a flag to auto-enable the deploy driver https://review.openstack.org/406381 | 22:27 |
*** vgadiraj has quit IRC | 22:31 | |
lindycoder | JayF, what do you think about this one? https://review.openstack.org/#/c/406381/ and the corresponding change in d-g https://review.openstack.org/#/c/406384/ | 22:32 |
patchbot | patch 406381 - ironic - Add a flag to auto-enable the deploy driver | 22:32 |
patchbot | patch 406384 - openstack-infra/devstack-gate - Use the auto-enable flag instead of changin enable... | 22:32 |
*** srart has quit IRC | 22:33 | |
openstackgerrit | Annie Lezil proposed openstack/ironic-inspector-specs: Collect NIC name given by BIOS https://review.openstack.org/404372 | 22:36 |
jlvillal | lindycoder: Why is it called "auto-enable"? Isn't it just enabling the driver? | 22:36 |
jlvillal | lindycoder: Ah, it is a boolean. I thought it was a driver name. | 22:38 |
JayF | lindycoder: TBH I'd just make that the default behavior | 22:38 |
JayF | lindycoder: if IRONIC_DEPLOY_DRIVER not in IRONIC_ENABLED_DRIVERS, enable it | 22:38 |
JayF | lindycoder: and maybe print a log message or something | 22:38 |
lindycoder | jlvillal, oh yeah, i wanted it to be addional_enabled_drivers at first but as i was affecting it i figured it would look stupid in devstack gate to put the driver name on 2 configs | 22:38 |
jroll | hrm | 22:39 |
jroll | why don't we just make devstack harder to misconfigure and always do that | 22:39 |
lindycoder | JayF, making that the default behaviour goes against your review (https://review.openstack.org/#/c/387657/) no? | 22:39 |
jroll | I think that's what JayF just said, ignore me :P | 22:39 |
patchbot | patch 387657 - ironic - Devstack will die if default driver is not enabled | 22:39 |
JayF | lindycoder: if you implement it this way | 22:39 |
JayF | lindycoder: I'll hit abandon on tha tpatch :) | 22:39 |
lindycoder | All right let's go that :D | 22:40 |
jcook | for rescuewait timeout should it use deploy_callback_timeout or a new rescue timeout? | 22:40 |
JayF | jcook: I'd suggest making it a new option/config that defaults to the same value | 22:40 |
jroll | jcook: probably the same thing cleaning does | 22:40 |
JayF | jcook: we do have a separate one for cleaning ,right? | 22:40 |
jroll | (which I think is the latter | 22:40 |
jroll | ) | 22:41 |
jcook | kk | 22:41 |
jcook | I code so infrequently it takes me 5 minutes to find the stoopid checkout this gerrit commit line to run | 22:41 |
jroll | -d | 22:42 |
jroll | always -d | 22:42 |
jroll | -x for cherry pick | 22:42 |
*** ppiela has quit IRC | 22:43 | |
JayF | feel free to ask stupid questions instead of wasting 5 minutes trying to fnid their answers | 22:43 |
jcook | oh no kidding | 22:43 |
JayF | I keep that one in L1 cache | 22:43 |
JayF | lol | 22:43 |
jroll | time to book mark http://docs.openstack.org/infra/git-review/usage.html | 22:43 |
jcook | I just man git review and was like o_o, that's useful | 22:43 |
jroll | lol | 22:43 |
jcook | I mostly just git review when done making a change and don't use it otherwise | 22:43 |
*** Sukhdev has joined #openstack-ironic | 22:43 | |
jroll | were you doing the copy line from the web page thing? | 22:43 |
jcook | think that's gonna change | 22:43 |
jcook | yeah | 22:43 |
jroll | yeah git-review is awesome | 22:44 |
jcook | it's hard to find now in the webpage | 22:44 |
JayF | Gerrit is sorta strange when you first start working with it | 22:44 |
jroll | yeah always a pain, I have to grab it sometime for faking out the gate | 22:44 |
*** chlong has quit IRC | 22:44 | |
JayF | but once you get used to it, and git-review, holy crap it's powerful | 22:44 |
jcook | well I'll definitely be using -d going forward | 22:45 |
*** alexpilotti has joined #openstack-ironic | 22:46 | |
JayF | lindycoder: your patch will partial-bug this: https://bugs.launchpad.net/ironic/+bug/1633567 | 22:47 |
openstack | Launchpad bug 1633567 in Ironic "Devstack fails silently when provided an invalid driver" [Low,In progress] - Assigned to Jay Faulkner (jason-oldos) | 22:47 |
lindycoder | JayF, all right i'll include it | 22:48 |
*** soliosg has quit IRC | 22:48 | |
openstackgerrit | Annie Lezil proposed openstack/ironic-inspector-specs: Collect NIC name given by BIOS https://review.openstack.org/404372 | 22:50 |
*** alexpilotti has quit IRC | 22:50 | |
*** alexpilotti has joined #openstack-ironic | 22:54 | |
openstackgerrit | Martin Roy proposed openstack/ironic: Add a flag to auto-enable the deploy driver https://review.openstack.org/406381 | 22:58 |
*** alexpilotti has quit IRC | 22:58 | |
openstackgerrit | Martin Roy proposed openstack/ironic: Auto enable the deploy driver https://review.openstack.org/406381 | 23:00 |
lindycoder | JayF, here it is ^ i'm off for today, thanks a lot for the help! | 23:01 |
*** alexpilotti has joined #openstack-ironic | 23:02 | |
*** vgadiraj has joined #openstack-ironic | 23:08 | |
*** lindycoder has quit IRC | 23:08 | |
*** daemontool has quit IRC | 23:13 | |
*** ijw has joined #openstack-ironic | 23:15 | |
openstackgerrit | Annie Lezil proposed openstack/ironic-inspector-specs: Collect NIC name given by BIOS https://review.openstack.org/404372 | 23:16 |
*** ijw has quit IRC | 23:16 | |
openstackgerrit | Félix Bouliane proposed openstack/ironic: Add virtualpdu to ironic devstack plugin https://review.openstack.org/388154 | 23:17 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing tempest setting... https://review.openstack.org/406376 | 23:22 |
*** Sukhdev has quit IRC | 23:28 | |
*** vgadiraj has quit IRC | 23:28 | |
*** jcoufal has joined #openstack-ironic | 23:28 | |
*** jcoufal has quit IRC | 23:29 | |
*** rbudden has quit IRC | 23:36 | |
*** rama_y has quit IRC | 23:38 | |
*** [1]cdearborn has quit IRC | 23:38 | |
*** vgadiraj has joined #openstack-ironic | 23:39 | |
openstackgerrit | Chris Krelle proposed openstack/bifrost: Initial work on adding staging driver support https://review.openstack.org/406401 | 23:43 |
*** vgadiraj has quit IRC | 23:55 | |
*** ijw has joined #openstack-ironic | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!