Friday, 2016-12-02

*** aNuposic has joined #openstack-ironic00:00
konetzedanyone tell me how to get a node back to 'available' from 'clean failed' Provisioning State?00:03
konetzedironic node-set-provision-state <id> manage00:07
konetzedgot it figured out! :D google was failing me earlier00:07
*** yogi has joined #openstack-ironic00:07
*** bfournie has joined #openstack-ironic00:14
*** soliosg has quit IRC00:21
*** soliosg has joined #openstack-ironic00:22
*** soliosg has quit IRC00:23
openstackgerritRuby Loo proposed openstack/ironic-specs: Support rolling upgrades  https://review.openstack.org/29924500:30
*** hoangcx has joined #openstack-ironic00:41
*** Goneri has joined #openstack-ironic00:42
*** cdearborn has quit IRC00:45
*** awiddersheim has joined #openstack-ironic00:46
*** awiddersheim has quit IRC00:50
*** SerenaFeng has joined #openstack-ironic00:57
*** vgadiraj has quit IRC00:59
*** [1]rpioso has quit IRC01:00
*** rbudden has quit IRC01:05
*** rama_y has quit IRC01:05
*** zhangjl has joined #openstack-ironic01:08
*** ijw has joined #openstack-ironic01:17
*** ijw has quit IRC01:17
*** ijw has joined #openstack-ironic01:18
*** jrist has joined #openstack-ironic01:18
*** SerenaFeng has quit IRC01:25
*** rbudden has joined #openstack-ironic01:28
*** rbudden has quit IRC01:29
*** SerenaFeng has joined #openstack-ironic01:30
*** aNuposic has quit IRC01:36
*** aNuposic has joined #openstack-ironic01:37
*** aNuposic has quit IRC01:38
*** aNuposic has joined #openstack-ironic01:39
*** mtanino has quit IRC01:40
*** pramodrj07 has joined #openstack-ironic01:45
*** PramodJ has joined #openstack-ironic01:50
*** awiddersheim has joined #openstack-ironic02:02
*** Syed_ has quit IRC02:06
*** persia has quit IRC02:06
*** persia has joined #openstack-ironic02:09
openstackgerritzhangyanxian proposed openstack/ironic: Fix import method to follow community guideline  https://review.openstack.org/40573002:11
openstackgerritzhangyanxian proposed openstack/ironic: Fix import method to follow community guideline  https://review.openstack.org/40573002:12
openstackgerritgengchc2 proposed openstack/ironic-lib: Correct reraising of exception  https://review.openstack.org/40573402:15
*** ijw has quit IRC02:18
*** ijw has joined #openstack-ironic02:19
*** aNuposic has quit IRC02:19
*** ijw_ has joined #openstack-ironic02:21
*** ijw has quit IRC02:23
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off  https://review.openstack.org/21673002:23
*** rloo has quit IRC02:28
*** cdearborn has joined #openstack-ironic02:43
*** yuanying_ has quit IRC02:48
*** yuanying has joined #openstack-ironic02:52
openstackgerritGhanshyam Mann proposed openstack/ironic: Keep CONF.baremetal.driver_enabled for Tempest plugin  https://review.openstack.org/40003202:53
openstackgerritNaohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off  https://review.openstack.org/21673802:54
*** absubram has joined #openstack-ironic02:54
*** absubram_ has joined #openstack-ironic02:56
*** absubram has quit IRC02:59
*** absubram_ is now known as absubram02:59
openstackgerritRuby Loo proposed openstack/ironic: Add RPCs to support volume target operations  https://review.openstack.org/28522003:04
openstackgerritRuby Loo proposed openstack/ironic: Add RPCs to support volume target operations  https://review.openstack.org/28522003:06
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off  https://review.openstack.org/21674303:06
*** cdearborn has quit IRC03:20
*** mtanino has joined #openstack-ironic03:27
*** SerenaFeng has quit IRC03:43
*** SerenaFeng has joined #openstack-ironic03:43
*** yuanying has quit IRC03:44
*** Goneri has quit IRC03:49
*** links has joined #openstack-ironic03:52
*** SerenaFeng has quit IRC03:56
*** zhangjl has quit IRC04:02
*** baoli has quit IRC04:04
*** rama_y has joined #openstack-ironic04:05
*** SerenaFeng has joined #openstack-ironic04:06
*** baoli has joined #openstack-ironic04:08
*** SerenaFeng has quit IRC04:15
*** baoli has quit IRC04:15
*** zhangjl has joined #openstack-ironic04:24
*** mtanino has quit IRC04:26
*** misty-cloud has joined #openstack-ironic04:38
*** aNuposic has joined #openstack-ironic04:49
*** rbudden has joined #openstack-ironic04:50
*** zackf has quit IRC04:53
*** aNuposic has quit IRC05:03
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/40581205:05
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/40581305:05
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/40581405:05
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/40588605:13
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/40588705:13
*** vmud213 has joined #openstack-ironic05:15
*** vikrant has joined #openstack-ironic05:16
*** fragatina has joined #openstack-ironic05:21
*** ijw_ has quit IRC05:22
*** zackf has joined #openstack-ironic05:22
*** zackf has quit IRC05:27
*** penick has joined #openstack-ironic05:27
*** fragatina has quit IRC05:33
*** fragatina has joined #openstack-ironic05:33
*** fragatina has quit IRC05:34
*** fragatina has joined #openstack-ironic05:38
*** fragatina has quit IRC05:38
*** fragatina has joined #openstack-ironic05:38
*** fragatina has quit IRC05:38
*** fragatina has joined #openstack-ironic05:39
*** fragatina has quit IRC05:40
*** fragatina has joined #openstack-ironic05:40
*** fragatina has quit IRC05:42
*** fragatina has joined #openstack-ironic05:43
*** rbudden has quit IRC05:48
*** bnemec has joined #openstack-ironic05:49
openstackgerritZhenguo Niu proposed openstack/ironic: Clean nodes stuck in CLEANING state when ir-cond restarts  https://review.openstack.org/34997105:49
*** rama_y has quit IRC05:51
*** PramodJ has quit IRC05:55
*** pramodrj07 has quit IRC05:55
openstackgerritNisha Agarwal proposed openstack/ironic: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers  https://review.openstack.org/32523006:03
*** aNuposic has joined #openstack-ironic06:04
*** jaosorior has joined #openstack-ironic06:06
openstackgerritGhanshyam Mann proposed openstack/ironic: Keep CONF.baremetal.driver_enabled for Tempest plugin  https://review.openstack.org/40003206:10
*** bnemec has quit IRC06:11
openstackgerritNisha Agarwal proposed openstack/ironic: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers  https://review.openstack.org/32523006:16
*** moshele has joined #openstack-ironic06:17
openstackgerritNisha Agarwal proposed openstack/ironic: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers  https://review.openstack.org/32523006:18
*** yufei has joined #openstack-ironic06:20
*** moshele has quit IRC06:24
*** e0ne has joined #openstack-ironic06:26
openstackgerritNaohiro Tamura proposed openstack/ironic: Update the existing APIs due to adding get_supported_power_states  https://review.openstack.org/38219406:26
*** aNuposic has quit IRC06:29
*** ChubYann has quit IRC06:31
*** openstackgerrit has quit IRC06:33
*** openstackgerrit has joined #openstack-ironic06:38
openstackgerritHironori Shiina proposed openstack/ironic: Add REST API for volume connector and volume target operation  https://review.openstack.org/21458606:38
*** penick has quit IRC06:39
*** zackf has joined #openstack-ironic06:41
*** zackf has quit IRC06:45
*** e0ne has quit IRC06:46
*** SerenaFeng has joined #openstack-ironic07:00
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI  https://review.openstack.org/34819107:02
*** fxpester has joined #openstack-ironic07:09
*** jaosorior has quit IRC07:10
*** jaosorior has joined #openstack-ironic07:11
openstackgerritNaohiro Tamura proposed openstack/ironic: Ipmitool management driver for Inject NMI  https://review.openstack.org/34872107:11
*** pester has joined #openstack-ironic07:13
*** jrist has quit IRC07:14
*** fxpester|2 has joined #openstack-ironic07:15
*** jrist has joined #openstack-ironic07:15
*** fxpester has quit IRC07:15
*** pester has quit IRC07:17
*** fxpester has joined #openstack-ironic07:18
*** e0ne has joined #openstack-ironic07:19
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC management driver for Inject NMI  https://review.openstack.org/34873207:20
*** pester has joined #openstack-ironic07:20
*** fxpester|2 has quit IRC07:20
*** fxpester has quit IRC07:23
*** fxpester has joined #openstack-ironic07:24
*** pester has quit IRC07:26
*** pester has joined #openstack-ironic07:26
*** fxpester has quit IRC07:29
*** fxpester has joined #openstack-ironic07:29
*** pester has quit IRC07:32
*** pester has joined #openstack-ironic07:32
*** fxpester|2 has joined #openstack-ironic07:34
*** ohamada has joined #openstack-ironic07:34
*** fxpester has quit IRC07:34
*** fxpester has joined #openstack-ironic07:36
*** aparnav has joined #openstack-ironic07:36
*** pester has quit IRC07:37
*** fxpester|2 has quit IRC07:39
*** pester has joined #openstack-ironic07:39
*** rcernin has joined #openstack-ironic07:40
*** fxpester has quit IRC07:42
*** fxpester has joined #openstack-ironic07:42
*** fxpester|2 has joined #openstack-ironic07:45
*** pester has quit IRC07:45
*** fxpester has quit IRC07:47
*** fxpester has joined #openstack-ironic07:47
*** parlos has joined #openstack-ironic07:48
*** pester has joined #openstack-ironic07:49
*** srart has quit IRC07:50
*** fxpester|2 has quit IRC07:50
*** absubram has quit IRC07:50
*** fxpester|2 has joined #openstack-ironic07:52
*** fxpester has quit IRC07:52
*** pester has quit IRC07:55
*** fxpester has joined #openstack-ironic07:55
*** pmannidi has quit IRC07:56
*** ccamacho has joined #openstack-ironic07:56
*** ralonsoh has joined #openstack-ironic07:57
*** fxpester|2 has quit IRC07:58
*** pester has joined #openstack-ironic08:00
*** fxpester|2 has joined #openstack-ironic08:03
*** fxpester has quit IRC08:03
*** openstackgerrit has quit IRC08:03
*** pester has quit IRC08:05
*** srart has joined #openstack-ironic08:06
*** fxpester has joined #openstack-ironic08:08
*** SerenaFeng has quit IRC08:09
*** fxpester|2 has quit IRC08:11
*** pester has joined #openstack-ironic08:11
*** fxpester has quit IRC08:14
*** fxpester has joined #openstack-ironic08:15
*** fxpester|2 has joined #openstack-ironic08:18
*** pester has quit IRC08:18
*** fxpester has quit IRC08:20
*** fxpester has joined #openstack-ironic08:21
*** pester has joined #openstack-ironic08:23
*** fxpester|2 has quit IRC08:23
*** fxpester|2 has joined #openstack-ironic08:25
*** fxpester has quit IRC08:25
*** fxpester has joined #openstack-ironic08:27
*** pester has quit IRC08:28
*** pester has joined #openstack-ironic08:30
*** fxpester|2 has quit IRC08:30
*** fxpester has quit IRC08:33
*** fxpester has joined #openstack-ironic08:33
*** fxpester|2 has joined #openstack-ironic08:35
*** openstackgerrit has joined #openstack-ironic08:36
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817308:36
*** pester has quit IRC08:36
*** pester has joined #openstack-ironic08:37
*** fxpester has quit IRC08:38
*** fxpester|2 has quit IRC08:40
*** e0ne has quit IRC08:40
*** fxpester has joined #openstack-ironic08:41
*** fxpester|2 has joined #openstack-ironic08:43
*** pester has quit IRC08:43
*** SerenaFeng has joined #openstack-ironic08:44
*** pester has joined #openstack-ironic08:45
*** jpich has joined #openstack-ironic08:45
*** fxpester has quit IRC08:45
*** fxpester|2 has quit IRC08:48
*** fxpester has joined #openstack-ironic08:48
*** romcheg has joined #openstack-ironic08:48
*** fxpester|2 has joined #openstack-ironic08:50
*** pester has quit IRC08:51
*** betherly has quit IRC08:51
*** romcheg16 has joined #openstack-ironic08:52
*** romcheg16 has quit IRC08:52
*** fxpester has quit IRC08:53
*** fxpester has joined #openstack-ironic08:54
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add ironic resources CRUD notifications  https://review.openstack.org/35654108:55
*** Romanenko_K has joined #openstack-ironic08:56
*** pester has joined #openstack-ironic08:56
*** fxpester|2 has quit IRC08:57
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node maintenance notifications  https://review.openstack.org/39623908:59
*** zzzeek has quit IRC09:00
*** fxpester has quit IRC09:00
*** aarefiev_afk is now known as aarefiev09:00
*** zzzeek has joined #openstack-ironic09:01
*** fxpester has joined #openstack-ironic09:02
*** openstackgerrit has quit IRC09:03
*** openstackgerrit has joined #openstack-ironic09:03
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node console notifications  https://review.openstack.org/39781209:03
*** pester has quit IRC09:04
*** pester has joined #openstack-ironic09:05
*** fxpester has quit IRC09:07
*** vsaienko has quit IRC09:09
*** fxpester has joined #openstack-ironic09:10
*** pester has quit IRC09:13
*** pester has joined #openstack-ironic09:13
*** yuriyz has quit IRC09:13
openstackgerritNisha Agarwal proposed openstack/ironic: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers  https://review.openstack.org/32523009:15
*** fxpester|2 has joined #openstack-ironic09:15
*** zhangjl has left #openstack-ironic09:15
*** zhugaoxiao has joined #openstack-ironic09:16
*** fxpester has quit IRC09:16
*** pester has quit IRC09:18
*** fxpester has joined #openstack-ironic09:18
*** romcheg22 has joined #openstack-ironic09:19
*** romcheg has quit IRC09:19
*** romcheg22 is now known as romcheg09:19
*** romcheg97 has joined #openstack-ironic09:21
*** fxpester|2 has quit IRC09:21
*** pester has joined #openstack-ironic09:21
*** romcheg97 has left #openstack-ironic09:22
*** fxpester|2 has joined #openstack-ironic09:24
*** fxpester has quit IRC09:24
*** pester has quit IRC09:27
*** fxpester has joined #openstack-ironic09:32
*** SerenaFeng has quit IRC09:33
*** pester has joined #openstack-ironic09:34
*** fxpester|2 has quit IRC09:34
*** lucas-afk is now known as lucasagomes09:35
lucasagomesmorning all09:35
*** fxpester has quit IRC09:36
*** fxpester has joined #openstack-ironic09:37
*** pester has quit IRC09:40
*** pester has joined #openstack-ironic09:44
openstackgerritVasyl Saienko proposed openstack/ironic: Allow use *_ipmitool with vbmc on multinode  https://review.openstack.org/40602009:44
*** ccamacho is now known as ccamacho|afk09:45
*** fxpester|2 has joined #openstack-ironic09:46
*** yuriyz has joined #openstack-ironic09:46
*** fxpester has quit IRC09:46
openstackgerritparesh sao proposed openstack/ironic: [WIP] Boot from iSCSI volume  https://review.openstack.org/30842509:47
*** amoralej has joined #openstack-ironic09:47
amoralejhi, i reported bug https://bugs.launchpad.net/ironic/+bug/1646477 about false callback timeouts09:48
openstackLaunchpad bug 1646477 in Ironic "Callback false timeouts" [Undecided,New]09:48
amoraleji can provide logs for several hits09:49
*** rcernin has quit IRC09:49
*** pester has quit IRC09:49
*** fxpester|2 has quit IRC09:50
*** rcernin has joined #openstack-ironic09:50
*** zhugaoxiao has quit IRC09:54
openstackgerritMerged openstack/ironic-ui: Incorporate driver-validation into node-detail panels  https://review.openstack.org/39969509:57
*** e0ne has joined #openstack-ironic09:58
vdrokgood morning ironic!10:01
*** e0ne has quit IRC10:02
*** e0ne has joined #openstack-ironic10:03
*** abalfour has quit IRC10:03
*** priteau has joined #openstack-ironic10:03
*** abalfour has joined #openstack-ironic10:03
*** strigazi_AFK is now known as strigazi10:06
pas-hamorning Ironic :) TGIF :)10:10
aarefievmorning all!10:17
*** ccamacho|afk is now known as ccamacho10:24
*** xiaoyang has quit IRC10:25
*** tesseract has joined #openstack-ironic10:27
*** tesseract is now known as Guest7998610:27
*** slagle has quit IRC10:28
*** e0ne has quit IRC10:28
*** e0ne has joined #openstack-ironic10:29
*** slagle has joined #openstack-ironic10:29
*** dtantsur|afk is now known as dtantsur10:33
dtantsurmorning Ironic, aarefiev, pas-ha, vdrok, lucasagomes10:33
*** milan has joined #openstack-ironic10:39
milanmorning Ironic! :)10:41
milanTGIF!10:41
*** mgould|afk is now known as mgould10:47
mgouldmorning dtantsur aarefiev pas-ha vdrok lucasagomes milan and everyone else!10:48
mgouldand TGIF :-)10:48
lucasagomeso/10:48
milanlucasagomes, morning! :)10:48
milanmgould, morning! :)10:48
dtantsurhey mgould, milan! how was zastavka?10:49
milanmorning dtantsur! :) very good! But we missed you10:49
dtantsuryeah, sorry :)10:51
*** hoangcx has quit IRC10:52
mgoulddtantsur: I really enjoyed it10:52
mgouldhow are you feeling today?10:52
dtantsurso-so10:53
milan:-/10:54
milanmidcycle pleague10:54
milanspreading in Brno10:54
mgoulddtantsur: get well soon!10:54
dtantsurthanks!10:55
*** e0ne has quit IRC10:55
*** e0ne has joined #openstack-ironic10:55
vdrokgood morning lucasagomes pas-ha aarefiev dtantsur milan and mgould ! :)10:56
aarefievmorning dtantsur, milan, lucasagomes, mgould, vdrok10:57
sambetts|afkMorning all /me is on PTO today so I won't be around much10:57
dtantsurhey sambetts|afk, enjoy it10:58
milanmorning aarefiev, vdrok! :)10:58
milanmorning sambetts|afk! :)10:58
milansambetts|afk, TGIF, no bother on PTO ;)10:58
sambetts|afkhehe definatly TGIF10:59
sambetts|afkdefinitely*10:59
sambetts|afkI never get that spelling right10:59
milan:)10:59
*** e0ne has quit IRC11:00
*** ohamada has quit IRC11:00
*** ohamada_ has joined #openstack-ironic11:01
*** e0ne has joined #openstack-ironic11:01
*** ohamada_ has quit IRC11:01
*** ohamada_ has joined #openstack-ironic11:01
*** ohamada__ has joined #openstack-ironic11:01
*** ohamada_ has quit IRC11:01
vsaienk0morning dtantsur, sambbets, vdrok, milan, pas-ha, lucasagomes and all Ironic'ers!11:02
lucasagomeso/11:03
sambetts|afko/11:03
milanmorning vsaienk0! :)11:03
dtantsurhey vsaienk011:03
openstackgerritMerged openstack/ironic-inspector: Document upgrade procedure  https://review.openstack.org/39327511:06
openstackgerritMerged openstack/ironic: Unneeded testing in DB migration of volume connector  https://review.openstack.org/40555311:07
*** vsaienko has joined #openstack-ironic11:08
dtantsur11 files changed, 795 insertions(+), 66 deletions(-)11:08
dtantsurmy next patch grows too big :(11:08
mgouldsambetts|afk: what are you doing in here at all? shoo, shoo, go and have fun :-)11:15
sambetts|afkmgould: ahaha I'm here because dtantsur's patch that merged yesterday landed me with a giant merge conflict which is playing on my mind XD11:15
sambetts|afkmgould: blames all the dtantsur11:15
sambetts|afk;)11:16
dtantsur\o/11:16
vdrokmorning sambetts|afk :)11:17
sambetts|afkhey vdrok11:17
openstackgerritDmitry Tantsur proposed openstack/ironic: Support defining and loading hardware types  https://review.openstack.org/33662611:18
dtantsurfolks, 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
patchbotpatch 357983 - ironic - Support names for {cleaning,provisioning}_network11:23
*** yufei has quit IRC11:27
openstackgerritMerged openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/40588611:27
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Show team and repo badges on README  https://review.openstack.org/40249611:33
openstackgerritMerged openstack/ironic-inspector: Show team and repo badges on README  https://review.openstack.org/40249511:33
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Rework handling requirements for functional tests  https://review.openstack.org/40607711:38
dtantsurmilan, aarefiev please check ^^^11:38
* milan checks11:38
*** ohamada__ has quit IRC11:39
*** ohamada__ has joined #openstack-ironic11:39
*** gcb has quit IRC11:41
*** misty-cloud has quit IRC11:41
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector-specs: Show team and repo badges on README  https://review.openstack.org/40249411:50
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/40581211:54
*** fxpester has joined #openstack-ironic11:56
*** pester has joined #openstack-ironic12:00
amoralejhi, i reported bug https://bugs.launchpad.net/ironic/+bug/1646477 about false callback timeouts12:01
openstackLaunchpad bug 1646477 in Ironic "Callback false timeouts" [Undecided,New]12:01
amoraleji think i'm seeing it in newton also12:01
amoralejdtantsur ^12:01
*** gcb has joined #openstack-ironic12:01
dtantsuron my radar, just don't have time yet12:01
dtantsurlucasagomes, ^^^12:01
* lucasagomes checks12:02
*** fxpester has quit IRC12:03
*** fxpester has joined #openstack-ironic12:03
*** cdearborn has joined #openstack-ironic12:06
vsaienk0lucasagomes: could you please review 1 line fix for vbm ipmi_address, needed for ironic multinode job https://review.openstack.org/#/c/406020/12:06
patchbotpatch 406020 - ironic - Allow use *_ipmitool with vbmc on multinode12:06
*** pester has quit IRC12:06
lucasagomesvsaienk0, hi there... sure 1 sec12:06
*** pester has joined #openstack-ironic12:06
lucasagomesdtantsur, I will investigate it12:07
openstackgerritjiang wei proposed openstack/python-ironicclient: Support i18n for baremetal port cmds  https://review.openstack.org/37597112:07
sambetts|afkvsaienk0: literally hit and fixed the same issue in my patch while fixing it yesterday12:08
*** fxpester has quit IRC12:09
*** fxpester has joined #openstack-ironic12:09
*** jkilpatr has quit IRC12:10
*** [1]cdearborn has joined #openstack-ironic12:10
*** derekh has joined #openstack-ironic12:10
jrollmorning y'all12:11
dtantsurmorning jroll12:11
jrollaarefiev: not sure I understand your comment here https://review.openstack.org/#/c/404436/12:11
patchbotpatch 404436 - ironic - Move CONF.service_available.ironic to our plugin12:11
jrollaarefiev: 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
patchbotpatch 397775 - ironic - DNM: Testing 35558612:12
*** pester has quit IRC12:12
*** rcernin has quit IRC12:12
*** ohamada__ has quit IRC12:12
*** ohamada__ has joined #openstack-ironic12:12
*** pester has joined #openstack-ironic12:15
*** rcernin has joined #openstack-ironic12:15
aarefievjroll: the problem is grenade doesn't call ironic to configure tempest on new, updated setup, I mean ironic_configure_tempest method12:17
*** fxpester has quit IRC12:17
*** fxpester has joined #openstack-ironic12:18
jrollaarefiev: right, it should use the config from the old side, right?12:18
jrollso we need that config in stable/newton branch12:18
jrolland even mitaka I guess12:18
aarefievjroll: it doesn't use old config12:19
jrollthat seems... wrong12:19
aarefievit generate new on and configure it12:19
aarefievif you remember we had issue with this, when inspector test was skipped12:20
*** pester has quit IRC12:21
*** pester has joined #openstack-ironic12:21
charlledanielmorning all12:23
jrollhrm, I thought that was fixed, else how do we configure flavors and images?12:23
*** fxpester has quit IRC12:24
*** cdearborn has quit IRC12:24
sambetts|afkjust 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 today12:24
aarefievjroll: I guess tempest configure all nova staff  as nova isn't  plugin12:24
sambetts|afkbut it also will likly overwrite any changes we make today12:25
jrollaarefiev: that config is done in ironic_configure_tempest, though12:25
*** aparnav has quit IRC12:25
aarefievjroll: we doesn't use any ironic tests in grenade12:25
jrollaarefiev: it still needs to use a baremetal flavor to boot a node, though12:26
sambetts|afkaarefiev: we still have to set the default flvor though12:26
openstackgerritMerged openstack/python-ironic-inspector-client: Show team and repo badges on README  https://review.openstack.org/40249612:26
sambetts|afkotherwise a node wouldn't be scheduled12:26
*** [1]cdearborn has quit IRC12:26
vdrokmorning jroll12:28
aarefievhard to say where grenade configure this, maybe during tempest configuration , let me take a look12:29
jrollif 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.gz12:29
openstackgerritDmitry Tantsur proposed openstack/ironic: Support names for {cleaning,provisioning}_network  https://review.openstack.org/35798312:29
dtantsurvdrok, updated the release note ^^^12:29
vdrokdtantsur: thx12:30
dtantsurjroll, this is from devstack, I think. our config runs before tempest.conf is generated12:30
sambetts|afkit does now, it didn't use to12:31
sambetts|afkthat behaviour changed like 2 days ago12:31
jrollI'm just even more confused now12:31
sambetts|afkthey added a test-config phase to the devstack plugins12:31
sambetts|afkso that tempest could run late in the devstack run12:31
sambetts|afkit used to run early12:32
dtantsurcool, but right now it's not run at all :)12:32
*** fragatina has quit IRC12:32
*** moshele has joined #openstack-ironic12:32
openstackgerritJim Rollenhagen proposed openstack/ironic: Turn off tempest's multitenant network tests  https://review.openstack.org/40609412:32
openstackgerritJim Rollenhagen proposed openstack/ironic: Move CONF.service_available.ironic to our plugin  https://review.openstack.org/40609512:32
dtantsurI 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_58512:32
* jroll backports to newton and rechecks 397775, if that grenade job passes we will see12:32
*** fragatina has joined #openstack-ironic12:32
jrolldtantsur: aha, yeah maybe you're right12:33
aarefievwe are talking about grenade it doesn't run any devstack phase on update12:33
sambetts|afkdtantsur: https://review.openstack.org/#/c/293954/ I wonder if thats a branch issue12:33
patchbotpatch 293954 - openstack-dev/devstack - Ensure testing configuration can run as late as po... (MERGED)12:33
sambetts|afkaarefiev: 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-ironic12:34
aarefievsambetts|afk: https://github.com/openstack-dev/grenade/blob/master/upgrade-tempest#L8112:35
aarefievafter all services was updated12:35
jrollsambetts|afk: which code do we duplicate in our tempest plugin that you mention in 293954?12:35
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add testcases for osc port group comands  https://review.openstack.org/38542712:35
aarefievit's the last stage12:35
vdrokbtw, another thing has merged allowing changing tempest configuration after test-config: https://review.openstack.org/40544612:36
patchbotpatch 405446 - openstack-dev/devstack - Add test-config local.conf processing (MERGED)12:36
sambetts|afkjroll: all of this  https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L162212:36
mrtenioMorning Ironic12:37
jrollsambetts|afk: okay, I read it as our tempest plugin, sorry12:37
*** pester has quit IRC12:37
sambetts|afkjroll: we used to and now can go back to just overriding the devstack varibles for all those things12:37
sambetts|afkjroll: instead of editing the file ourselves12:37
jrollsambetts|afk: except for the tempest configs we define :P12:38
sambetts|afkjroll: except those :)12:38
jrollgood stuff12:38
sambetts|afkjroll: but we should set those in the new test-config phase in the devstack plugins12:38
sambetts|afkjroll: instead of at extra or whereever it is today12:38
*** hw_wutianwei has quit IRC12:39
jrollsambetts|afk: yeah that's where it's called today :) https://github.com/openstack/ironic/blob/master/devstack/plugin.sh#L6312:39
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal driver commands  https://review.openstack.org/36735912:39
openstackgerritMerged openstack/python-ironic-inspector-client: Rework handling requirements for functional tests  https://review.openstack.org/40607712:40
*** pester has joined #openstack-ironic12:41
aarefievsambetts|afk: but we configure tempest in test-config phase12:42
dtantsurjroll, https://review.openstack.org/40610012:42
patchbotpatch 406100 - governance - Claim supports-upgrade and follows-standard-deprec...12:42
*** fxpester|2 has joined #openstack-ironic12:44
*** fxpester has quit IRC12:44
*** pester has quit IRC12:46
*** fxpester has joined #openstack-ironic12:46
*** lucasagomes is now known as lucas-hungry12:47
*** pester has joined #openstack-ironic12:49
*** fxpester|2 has quit IRC12:49
*** bfournie has quit IRC12:50
*** fxpester|2 has joined #openstack-ironic12:51
*** fxpester has quit IRC12:51
*** jkilpatr has joined #openstack-ironic12:51
aarefievjroll: sambetts|afk: thats how flavor is configured  https://github.com/openstack-dev/devstack/blob/master/lib/tempest#L19012:52
aarefievDEFAULT_INSTANCE_TYPE=baremetal12:52
*** bfournie has joined #openstack-ironic12:53
*** fxpester has joined #openstack-ironic12:53
*** pester has quit IRC12:53
*** bfournie has quit IRC12:54
*** hamzy has joined #openstack-ironic12:55
*** pester has joined #openstack-ironic12:55
*** fxpester|2 has quit IRC12:56
*** fxpester|2 has joined #openstack-ironic12:57
*** gcb has quit IRC12:58
*** fxpester has quit IRC12:58
* milan gone12:58
milanbye everyone!12:59
*** jrist has quit IRC12:59
* mgould likewise goes; have a good weekend everyone!13:00
*** mgould is now known as mgould|afk13:00
*** pester has quit IRC13:00
*** fxpester has joined #openstack-ironic13:00
aarefievmilan, mgould Have a nice weekend13:00
*** mrtenio has quit IRC13:01
mgould|afkaarefiev: thanks, will do!13:01
mgould|afkwe're going to visit ifarkas in Budapest13:01
milanaarefiev, thx! :)13:01
mgould|afko/13:01
*** milan has quit IRC13:01
aarefievI'm sure you will :)13:01
*** fxpester|2 has quit IRC13:03
openstackgerritMerged openstack/ironic-inspector-specs: Show team and repo badges on README  https://review.openstack.org/40249413:03
*** pester has joined #openstack-ironic13:04
sambetts|afkaarefiev: 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#L161113:05
*** fxpester|2 has joined #openstack-ironic13:06
*** fxpester has quit IRC13:06
*** rcernin has quit IRC13:06
sambetts|afkaarefiev: 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 phase13:06
aarefievsambetts|afk: maybe we are talking about diff problems13:06
aarefievgrenade doesn't call ironic_configure_tempest13:07
sambetts|afkaarefiev: correct13:07
aarefievso possible misconfiguration13:08
*** rcernin has joined #openstack-ironic13:08
*** fxpester has joined #openstack-ironic13:09
*** pester has quit IRC13:09
*** pester has joined #openstack-ironic13:11
*** fxpester|2 has quit IRC13:11
sambetts|afkaarefiev: 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 fixed13:13
*** fxpester|2 has joined #openstack-ironic13:13
*** fxpester has quit IRC13:13
sambetts|afkaarefiev: 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 configurations13:14
openstackgerritMerged openstack/ironic: Allow use *_ipmitool with vbmc on multinode  https://review.openstack.org/40602013:15
*** pester has quit IRC13:16
*** fxpester has joined #openstack-ironic13:16
*** links has quit IRC13:17
*** fxpester|2 has quit IRC13:19
*** pester has joined #openstack-ironic13:19
openstackgerritSam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation  https://review.openstack.org/39295913:21
*** srobert has joined #openstack-ironic13:21
*** fxpester has quit IRC13:21
aarefievsambetts|afk: completely agree with that, as for [service-available]ironic , it's configured in tempest  right now, so ironic option  doesn't take effect13:21
*** srobert has quit IRC13:22
*** jtomasek has joined #openstack-ironic13:22
*** srobert has joined #openstack-ironic13:22
*** pester has quit IRC13:23
aarefievor at least when we are relying on devstack enable_services13:23
*** moshele has quit IRC13:23
*** jaosorior is now known as jaosorior_brb13:24
openstackgerritGalyna Zholtkevych proposed openstack/ironic: POC: ETAG supporting to enhance API evolution  https://review.openstack.org/39221313:24
sambetts|afkaarefiev: 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|afkaarefiev: does it nuke the whole file?13:25
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/40581213:25
sambetts|afkaarefiev: or only override stuff it knows about13:25
*** bfournie has joined #openstack-ironic13:26
aarefievsambetts|afk: option doesn't get removed, configure_tempest only override it, also it is new config13:27
aarefievso if we set some option in ironic on old config, it could be missing in new one13:28
sambetts|afkaarefiev: 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 it13:29
jrollhttps://bugs.launchpad.net/ironic/+bug/164647713:29
openstackLaunchpad bug 1646477 in Ironic "Callback false timeouts" [Undecided,New] - Assigned to Lucas Alvares Gomes (lucasagomes)13:29
*** [1]cdearborn has joined #openstack-ironic13:29
jrolllucas-hungry: need any help on that one13:29
jroll?13:29
*** dtantsur is now known as dtantsur|brb13:30
*** sambetts|afk is now known as sambetts|pto13:30
aarefievsambetts|pto: imo, as you said before, grenade should call ironic to configure it13:30
sambetts|ptoaarefiev: yeah, we should handle that in our grenade plugin13:31
*** krtaylor has quit IRC13:31
sambetts|ptoaarefiev: somewhere here problably https://github.com/openstack/ironic/blob/master/devstack/upgrade/upgrade.sh#L13413:31
aarefievsambetts|pto: we can't do this in ironic grenade plugin becase there is no new tempest13:32
aarefievyet13:32
sambetts|ptoaarefiev: if we aren't setting anything that tempest is going to override, then it doesn't amtter13:32
*** vmud213 has quit IRC13:32
sambetts|ptoisn't*13:32
sambetts|ptono is13:33
sambetts|ptoI was right the first time X13:33
sambetts|ptoDXD13:33
*** dprince has joined #openstack-ironic13:33
aarefievthe problem we can't set it, and not that tempest can to override it :)13:34
aarefievbecause there are no config yet13:34
sambetts|ptoyou said tempest doesn't nuke the config13:34
sambetts|ptoit just overrides stuff13:34
sambetts|ptoso the old config will still be there13:35
sambetts|ptowe can just write to it13:35
sambetts|ptothe same as we always do13:35
aarefievsambetts|pto: no, there will be new config13:36
aarefievhttps://github.com/openstack-dev/grenade/blob/master/grenade.sh#L302-L31313:37
aarefievbut probably we can copy it13:39
sambetts|ptotbh I'm not sure why grenade is even touching tempest13:40
sambetts|ptotempest is branch less13:40
Romanenko_KHi!13:40
sambetts|ptoso is our tempest plugin13:40
jrollsure hope these centos ci boxes are ephemeral >.>13:42
Romanenko_KFrom yesterday gate-ironicclient-dsvm-functional job fails on reviews for ironicclient.13:42
aarefievsambetts|pto: me either, we are running only smoke13:42
Romanenko_Khttps://bugs.launchpad.net/ironic/+bug/164683713:43
openstackLaunchpad bug 1646837 in Ironic "gate-ironicclient-dsvm-functional fails after change to Keystone v3" [Undecided,New]13:43
*** zackf has joined #openstack-ironic13:44
*** baoli has joined #openstack-ironic13:55
aarefievRomanenko_K: looks like tempest cli client doesn't support keystone v313:56
aarefievyou need to pass domain info to client object13:57
jrollthe first two patches in this chain look ready to me, if someone has time https://review.openstack.org/#/c/285219/4313:57
patchbotpatch 285219 - ironic - Add VolumeTarget object13:57
*** baoli has quit IRC13:57
*** baoli has joined #openstack-ironic13:57
*** lucas-hungry is now known as lucasagomes14:01
*** makowals has quit IRC14:02
lucasagomesjroll, 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#L35914:02
lucasagomesthe 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 anymore14:03
lucasagomeswhich might explain why the node gets to deploying from deploywait in the meantime14:04
jrolllucasagomes: how'd we reach a timeout, though?14:04
lucasagomesjroll, there's a periodic task running checking the wait states14:05
lucasagomesbut still, I need to dig more to see what happened cause the times seems too short14:06
jrollyeah, that's what bugs me, it doesn't seem like it should have timed out14:06
jrollunless you think it timed out and we got the callback at the same time14:06
sambetts|ptoseems like a race between the task changing the node to active and the periodic task processing the timeout14:07
jrolllucasagomes: also of note, I don't think the heartbeat gets an exclusive lock anymore14:07
jrollbut changing state should14:07
jrollso idk14:07
lucasagomesyeah14:07
*** dtantsur|brb is now known as dtantsur14:08
*** bnemec has joined #openstack-ironic14:09
*** jcoufal has joined #openstack-ironic14:10
*** makowals has joined #openstack-ironic14:10
*** makowals has quit IRC14:11
TheJuliagood morning14:12
*** lindycoder has joined #openstack-ironic14:15
*** athomas has joined #openstack-ironic14:16
nicodemosmorning TheJulia and ironic'ers14:17
*** makowals has joined #openstack-ironic14:17
*** gcb has joined #openstack-ironic14:18
dtantsurmorning TheJulia, nicodemos14:18
lucasagomesTheJulia, nicodemos morning14:18
vdrokmorning TheJulia and nicodemos14:21
*** athomas has quit IRC14:24
*** athomas has joined #openstack-ironic14:25
lucasagomesjroll, this series of events is also strange: http://paste.openstack.org/show/591242/14:25
lucasagomesjroll, basically the node was in deploying state and somehow heartbeat() could get an exclusive lock14:25
lindycoderJayF, 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 solution14:25
patchbotpatch 387657 - ironic - Devstack will die if default driver is not enabled14:25
lucasagomesand released it after14:25
lucasagomesthe next state change is moving from deploying to deploy fail14:26
*** vikrant has quit IRC14:26
jrollhrm14:26
* lucasagomes checks upgrade_lock()14:26
*** vikrant has joined #openstack-ironic14:28
*** rcernin has quit IRC14:33
*** jheroux has joined #openstack-ironic14:35
*** amoralej is now known as amoralej|lunch14:36
*** vikrant has quit IRC14:38
*** rcernin has joined #openstack-ironic14:41
*** vgadiraj has joined #openstack-ironic14:41
*** yufei has joined #openstack-ironic14:42
*** rbudden has joined #openstack-ironic14:45
*** Goneri has joined #openstack-ironic14:45
*** absubram has joined #openstack-ironic14:48
openstackgerritoctopuszhang proposed openstack/ironic-python-agent: Skip API releated work if no api url configured  https://review.openstack.org/39985714:50
*** absubram_ has joined #openstack-ironic14:50
*** absubram has quit IRC14:53
*** absubram_ is now known as absubram14:53
openstackgerritJulia Kreger proposed openstack/bifrost: Config drive spec uses network_data.json  https://review.openstack.org/31586214:53
*** mtanino has joined #openstack-ironic14:54
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Catch unknown exceptions in validate driver ifaces  https://review.openstack.org/40615714:54
yufeihello, 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 IRC14:55
lucasagomesyufei, I've deployed fedora24 (no kubernetes tho). What error are you seem when deploying it ?14:57
yufeiI’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 starts14:57
*** mjturek1 has left #openstack-ironic14:58
*** mjturek1 has joined #openstack-ironic14:58
*** rcernin has joined #openstack-ironic14:58
lucasagomesyufei, 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 it14:59
lucasagomesat that point I mean, after the image has been written to the disk14:59
*** dansmith is now known as superdan14:59
dtantsurlucasagomes, 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 tripleo14:59
patchbotpatch 357983 - ironic - Support names for {cleaning,provisioning}_network14:59
lucasagomesdtantsur, adding to the list, cause I'm still debugging the tripleo thing with node states. But will take a look15:00
dtantsur(which is also required for RAID to work fwiw)15:00
dtantsurlucasagomes, sure, thanks15:00
*** mjturek1 is now known as mjturek15:02
*** krtaylor has joined #openstack-ironic15:02
yufeihi, 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 amoralej15:03
*** rama_y has joined #openstack-ironic15:03
*** jhesketh has quit IRC15:04
lucasagomesyufei, 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 jaosorior15:06
*** jhesketh has joined #openstack-ironic15:06
lucasagomesyufei, 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 appear15:07
lucasagomesyufei, is it possible to check the /etc/fstab in your image ?15:08
lucasagomesalso, 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
yufeiI checket that, it’s the same as the fstab in node which os starts successfuly15:10
yufeiI 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 centos15:11
*** jaosorior has quit IRC15:11
*** jaosorior has joined #openstack-ironic15:12
*** yogi has quit IRC15:12
*** jaosorior has quit IRC15:13
*** jaosorior has joined #openstack-ironic15:13
TheJuliayufei: Are you trying to image deployed machines and then deploy that image out?15:14
openstackgerritMerged openstack/python-ironicclient: Update release note for fix to required args  https://review.openstack.org/40544515:18
*** Syed__ has joined #openstack-ironic15:18
yufeihi, TheJulia, Do you mean that dd the deployed disk to other machines?15:19
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Serve boot configs from Ironic API  https://review.openstack.org/36943815:20
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Implement Driver API for dynamic boot config  https://review.openstack.org/39439815:20
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get the boot config  https://review.openstack.org/39439915:20
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Allow not writing pxe configs to disk  https://review.openstack.org/39439715:20
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot config to proper Jinja template  https://review.openstack.org/39346215:20
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE  https://review.openstack.org/39194315:20
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: DNM testing stateless conductor  https://review.openstack.org/40238715:20
*** jkilpatr has quit IRC15:20
*** vsaienko has quit IRC15:21
*** jkilpatr has joined #openstack-ironic15:21
yufeihi, lucasagomes, I test three kinds of fedora partion image, I haven’t test whole disk image yet.15:23
yufeiwhich kind of image did you use?15:24
lucasagomesyufei, right, I built one using the "fedora" element from DIB15:26
lucasagomese.g: diskimage-builder -o fedora-image fedora baremetal15:26
*** [1]cdearborn has quit IRC15:27
yufeihow do you install the diskimage-builder, for source or pip?15:27
lucasagomesyufei, either pip or packages (from RDO)15:28
lucasagomesyufei, btw can you try booting with "systemd.log_level=debug" ?15:30
lucasagomesor even add a "emergency" in the kernel cmd line so you drop in the emergency shell15:30
openstackgerritMerged openstack/ironic: Add VolumeTarget object  https://review.openstack.org/28521915:30
yufeiI didn’t do that before, should the be appened to pxe_append_params?15:30
lucasagomesyufei, yeah15:30
lucasagomesyufei, well... are you using local boot ?15:31
lucasagomesif so that won't have any effect really :-/15:31
yufeino…15:31
yufeiThanks you, let me have a try, try to build a whole disk image from pip insatlled builder, and add the debug to the parm15:32
yufeiYou've given me a lot of inspiration :D15:32
*** openstackgerrit has quit IRC15:33
lucasagomesyufei, :D15:33
*** openstackgerrit has joined #openstack-ironic15:34
openstackgerritMerged openstack/ironic: Cleanup adding Ironic to cluster on upgrade case  https://review.openstack.org/40529015:34
*** parlos has quit IRC15:34
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to keystone session glance and swift  https://review.openstack.org/35228915:35
*** vgadiraj has quit IRC15:36
*** alexpilotti has quit IRC15:43
*** srobert has quit IRC15:45
openstackgerritMerged openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/40581415:47
openstackgerritJulia Kreger proposed openstack/bifrost: Config drive spec uses network_data.json  https://review.openstack.org/31586215:47
*** chlong has joined #openstack-ironic15:48
openstackgerritPeter Piela proposed openstack/ironic-ui: Consolidate node last_error processing  https://review.openstack.org/40618415:48
*** Guest79986 has quit IRC15:54
*** ralonsoh has quit IRC15:54
NobodyCamGood Morning Ironic'ers .. and (ofc) ...15:56
NobodyCamTGIF!15:56
nicodemoshey, NobodyCam. TGIF! =D15:56
NobodyCammorning nicodemus :)15:57
openstackgerritJulia Kreger proposed openstack/bifrost: Config drive spec uses network_data.json  https://review.openstack.org/31586216:00
dtantsurmorning NobodyCam16:02
*** makowals has quit IRC16:02
dtantsurjoanna, 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 you16:03
patchbotpatch 336626 - ironic - Support defining and loading hardware types16:03
TheJuliaGood morning nicodemos16:03
TheJuliaerr16:03
TheJuliaNobodyCam:16:03
* TheJulia ponders going back to irssi16:03
*** absubram has quit IRC16:05
*** rcernin has quit IRC16:05
*** ohamada__ has quit IRC16:10
NobodyCamGood Morning dtantsur and TheJulia :)16:10
*** srobert has joined #openstack-ironic16:11
NobodyCamTheJulia: it took a while for me but I believe I am good with irc cloud. I love cross device access16:11
*** ohamada has joined #openstack-ironic16:11
*** srobert has quit IRC16:14
*** srobert has joined #openstack-ironic16:15
*** rpioso has joined #openstack-ironic16:17
*** david-lyle_ has joined #openstack-ironic16:17
openstackgerritoctopuszhang proposed openstack/ironic-python-agent: Skip API releated work if no api url configured  https://review.openstack.org/39985716:17
*** soliosg has joined #openstack-ironic16:18
*** david-lyle has quit IRC16:19
*** chlong has quit IRC16:20
*** gcb has quit IRC16:20
*** chlong has joined #openstack-ironic16:21
TheJuliaNobodyCam: 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
NobodyCamTheJulia: ++ I have (mostly) trained my browser to understand most folks handles :p16:24
TheJuliaI've noticed my browser has gotten a little better about some, but it is still frustrating16:25
NobodyCamhey hey TheJulia any opposition to me adding support to install staging drivers?16:25
TheJuliaNobodyCam: Absolutely none.  That might save us from HAVING to implement virutalbmc real real quick, although I still want to do that16:26
TheJuliaI just haven't had time to look at either16:26
NobodyCam+++16:27
openstackgerritVladyslav Drok proposed openstack/ironic: Renaming audit map conf sample file  https://review.openstack.org/35039916:27
NobodyCamawesome :)16:27
openstackgerritMerged openstack/ironic: Add RPCs to support volume target operations  https://review.openstack.org/28522016:28
TheJuliawoot \o/16:28
TheJuliatime to rebase!16:28
NobodyCam:p16:28
*** [1]cdearborn has joined #openstack-ironic16:30
*** rama_y has quit IRC16:30
vdrokmorning NobodyCam, tgif :)16:31
NobodyCammorning vdrok :)16:31
*** soliosg has quit IRC16:32
jlvillalvsaienk0, 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
patchbotpatch 406214 - openstack-infra/project-config - ironic: Have multinode job not use DEVSTACK_GATE_T...16:32
*** jistr is now known as jistr|biab16:33
vdrokmorning jlvillal ! hah, would you fix it then? :)16:34
jlvillalvdrok: heh :) Well we need to fix it. I will try of course.16:35
openstackgerritMerged openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/40581316:35
vdrokjlvillal: I agree, it's more logical to have smoke run on multinode, as the next step is grenade16:35
jlvillalvdrok: In order to run Grenade we need to get smoke running.16:35
jlvillalvdrok: Exactly16:35
vdrokjlvillal: not sure how vsaienk0 would feel about ut16:35
jlvillalvdrok: Is he around today?16:35
vdrokjlvillal: he was, not anymore16:36
jlvillalvdrok: Okay. Yeah it is getting late there I think.16:36
rpiosoI 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
patchbotpatch 405537 - ironic (stable/newton) - Make all IronicExceptions RPC-serializable16:37
JayFrpioso: looking16:37
vdrokjlvillal: do you know which tests are failing already? maybe I can start looking into that too16:37
rpiosoJayF: ty16:37
vdrokmorning rpioso and JayF :)16:38
rpiosovdrok: Good afternoon :)16:38
jlvillalvdrok: Uh. I changed the "multitenant" job locally to run with 7 VMs and deleted the REGEX.16:38
jlvillalvdrok: Would those log files work?16:38
*** jaosorior has quit IRC16:38
vdrokjlvillal: I'd be fine even with just test names for start :)16:39
JayFrpioso: +2A16:39
lindycoderHi JayF16:39
lindycoderSorry to bother have you had time to check out my comment https://review.openstack.org/#/c/387657/16:40
patchbotpatch 387657 - ironic - Devstack will die if default driver is not enabled16:40
JayFlindycoder: I saw it; I think to add support for VirtualPDU will end up requiring a patch to devstack_gate16:40
rpiosoJayF: \o/ Thanks, again!16:40
lindycoderJayF, well i'll get right on that then :)16:41
jlvillalvdrok: Let me find something16:41
JayFlindycoder: all those gate configs are split between devstack-gate, defaults in the ironic plugin, and project-config16:43
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/40581216:44
*** jlvillal has quit IRC16:45
*** rcernin has joined #openstack-ironic16:45
lindycoderJayF, 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
patchbotpatch 406221 - openstack-infra/devstack-gate - Add pxe_snmp to enabled drivers16:46
JayFlindycoder: probably need it on L414 as well16:47
lindycoderJayF, therE's no agent_snmp driver atm16:48
*** Romanenko_K has quit IRC16:48
JayFlindycoder: oh, TIL :)16:48
lindycoderJayF, thanks for the help, we'll get that to merge and update our change!16:49
dtantsurTheJulia, 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_AFK16:50
TheJuliadtantsur: 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
TheJuliawell attach/detach with cinder16:51
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/40588716:52
TheJuliadtantsur: 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
dtantsurTheJulia, yeah, I assume some folks don't want cinder (still figuring out why)16:52
* dtantsur is quite confused by what people want tbh16:52
TheJuliadtantsur: if you want to setup a discussion, more than happy to participate16:52
dtantsurcool, thanks! I'll collect more background myself first16:53
openstackgerritMerged openstack/ironic: Support names for {cleaning,provisioning}_network  https://review.openstack.org/35798316:54
dtantsurw00t16:54
*** e0ne has quit IRC16:54
openstackgerritMerged openstack/ironic: Update the links in iLO documentation  https://review.openstack.org/40508416:55
*** aNuposic has joined #openstack-ironic16:56
*** jlvillal has joined #openstack-ironic16:57
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/40581216:57
openstackgerritMerged openstack/bifrost: Add support to update keystone for bifrost  https://review.openstack.org/38022616:57
*** jlvillal has quit IRC16:58
*** jlvillal has joined #openstack-ironic16:59
openstackgerritMerged openstack/bifrost: Add ability to write clouds.yaml  https://review.openstack.org/38022717:00
openstackgerritMerged openstack/bifrost: Add keystone install and clouds.yaml to testing  https://review.openstack.org/38022817:00
openstackgerritMerged openstack/bifrost: Convert ironic.conf to template generated  https://review.openstack.org/30563417:00
openstackgerritMerged openstack/bifrost: Add Keystone support to ironic.conf  https://review.openstack.org/38246917:00
*** chlong has quit IRC17:02
*** penick has joined #openstack-ironic17:03
openstackgerritPeter Piela proposed openstack/ironic-ui: Consolidate node last_error processing  https://review.openstack.org/40618417:04
vdrokok, time to leave, good night everyone!17:06
dtantsurg'night vdrok17:07
nicodemosbye, vdrok17:07
*** jlvillal has quit IRC17:08
*** jlvillal has joined #openstack-ironic17:08
*** fxpester_home has quit IRC17:11
*** aNuposic has quit IRC17:13
*** aNuposic has joined #openstack-ironic17:14
JayFwould 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 RFE17:15
openstackLaunchpad 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-ironic17:17
dtantsurJayF, it does not seem to fix a problem..17:17
jlvillalThe joy of rebooting your ZNC instance and then figuring out that the firewall blocks all the ZNC ports...17:18
jlvillal30 minutes wasted :(17:18
*** zackf has quit IRC17:18
dtantsurugh17:18
JayFHow many billions of dollars of productivity have been lost17:19
JayFto overly restrictive corporate firewalls?17:19
jlvillalWell in this case it was the firewall on my VM.17:20
jlvillalBut the proxy server at work is a huge waste of time/money for our group.17:21
*** soliosg has joined #openstack-ironic17:24
openstackgerritMerged openstack/bifrost: Config drive spec uses network_data.json  https://review.openstack.org/31586217:24
dtantsurhave a good weekend folks, see you on Monday17:26
jlvillaldtantsur: Ciao17:26
jlvillalvdrok: Are you still around?17:26
*** dtantsur is now known as dtantsur|afk17:26
*** jkilpatr has quit IRC17:29
openstackgerritMerged openstack/ironic: Increase verbosity of devstack/lib/ironic  https://review.openstack.org/37049217:30
*** fragatina has quit IRC17:32
*** fragatina has joined #openstack-ironic17:34
JayFDo we have stable branches/releases for python-ironicclient?17:34
openstackgerritChris Krelle proposed openstack/bifrost: follow up to Change-Id: I4150f86fbc141dcd6fd12094acab1ca43d76910c  https://review.openstack.org/40626117:34
JayFit appears so...17:35
jrollyerp17:37
JayFaight, I just found a bug that needs backports for client and for ipa, and probably ironic once done17:37
JayFhooray17:37
*** gmmaha has quit IRC17:38
JayFhmm. Actually probably can't be backported, because it adds new requirements (oslo.serialization)17:38
*** fragatina has quit IRC17:39
*** athomas has quit IRC17:39
JayFWell, our stable policy doesn't say anything about that...17:39
jlvillalHas stable/liberty disappeared yet?17:40
*** gmmaha has joined #openstack-ironic17:40
JayFjroll: ^ 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 exist17:40
openstackLaunchpad bug 1629068 in Ironic "python3 incompatibility using the python-ironicclient" [Undecided,In progress] - Assigned to Nguyen Hung Phuong (phuongnh)17:40
openstackgerritStephanie Miller proposed openstack/bifrost: follow up to Change-Id: I4150f86fbc141dcd6fd12094acab1ca43d76910c  https://review.openstack.org/40626117:40
jrollJayF: g-r has stable branches, stable of projects have to match17:40
openstackgerritMerged openstack/ironic: Make all IronicExceptions RPC-serializable  https://review.openstack.org/40553717:40
JayFso as long as the requirement added matches the g-r for the stable branch it's OK?17:41
jrollyes17:41
JayFjlvillal: mitaka+liberty are in security fixes only mode17:41
jrollthere's ofc rules around changing g-r for stable17:41
jrollprobably a tony b question17:41
*** chlong has joined #openstack-ironic17:41
jrollliberty is about to die, don't bother17:41
JayFI don't think g-r needs to change17:41
JayFjust a new requirement for ironicclient17:41
openstackgerritChris Krelle proposed openstack/bifrost: follow up to Change-Id: I4150f86fbc141dcd6fd12094acab1ca43d76910c  https://review.openstack.org/40626117:42
jrollah yep17:42
jrollhttps://github.com/openstack/requirements/blob/stable/newton/global-requirements.txt#L11517:42
jrollnice17:42
jrollI'm still not sure the policy with those but should be fine17:42
openstackgerritJay Faulkner proposed openstack/python-ironicclient: Fix python3 compatibility when HTTP Error are returned  https://review.openstack.org/40626317:42
JayF^ is the patch in question, at least on the ironicclient side17:43
jrollthanks17:43
*** ohamada has quit IRC17:43
* jroll pokes a person17:43
JayFHmm.17:44
JayFpython-ironicclient doesn't list "Newton" as a series you can target a bug at17:44
lucasagomesjroll, 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 monday17:45
lucasagomesjroll, lemme know if you find something before (not sure if you are investigating it or not)17:45
jrolllucasagomes: right on, I gave up for a bit but will let you know if I look at it17:45
lucasagomescool17:45
*** rcernin has quit IRC17:45
*** jcoufal has quit IRC17:46
lucasagomesI'm calling it a day17:46
lucasagomeshave a great evening all, enjoy the weekend17:46
*** rcernin has joined #openstack-ironic17:46
*** jtomasek has quit IRC17:46
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Python 3 Compatible JSON  https://review.openstack.org/40626617:46
*** lucasagomes is now known as lucas-afk17:46
*** bnemec has quit IRC17:47
jrolllater lucas-afk17:48
openstackgerritStephanie Miller proposed openstack/bifrost: follow up to Change-Id: I4150f86fbc141dcd6fd12094acab1ca43d76910c  https://review.openstack.org/40626117:48
jrollJayF: we stopped tracking series in launchpad but maybe that's a good reason to keep doing it17:49
*** jkilpatr has joined #openstack-ironic17:49
JayF*shrug* if I don't have to target the bugs to stable, I can not do that too17:49
JayFthat works for me17:49
JayFlol17:49
*** jpich has quit IRC17:49
*** derekh has quit IRC17:49
jroll¯\_(ツ)_/¯17:49
JayFexactly17:49
*** anish has joined #openstack-ironic17:49
JayFAnyone have any comments on this RFE? I'm OK with approving it as it sits: https://bugs.launchpad.net/ironic-python-agent/+bug/164251517:50
openstackLaunchpad 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 jistr17:53
*** Sukhdev has joined #openstack-ironic17:54
TheJuliahmmm17:54
TheJuliaJayF: LGTM17:55
*** ElCoyote_ has left #openstack-ironic17:55
openstackgerritNisha Agarwal proposed openstack/ironic: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers  https://review.openstack.org/32523017:56
openstackgerritFélix Bouliane proposed openstack/ironic: Add virtualpdu to ironic devstack plugin  https://review.openstack.org/38815417:58
openstackgerritJulia Kreger proposed openstack/bifrost: Adding keystone support to inspector config  https://review.openstack.org/38342518:00
*** ElCoyote_ has joined #openstack-ironic18:00
openstackgerritJulia Kreger proposed openstack/bifrost: Add test script for Bifrost using keystone  https://review.openstack.org/38342618:00
openstackgerritJulia Kreger proposed openstack/bifrost: Update the task label for inspection  https://review.openstack.org/38342718:00
openstackgerritJulia Kreger proposed openstack/bifrost: Add keystone capability to base install  https://review.openstack.org/38504018:01
openstackgerritJulia Kreger proposed openstack/bifrost: Add initial keystone use documentation  https://review.openstack.org/38504118:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Serve boot configs from Ironic API  https://review.openstack.org/36943818:04
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get the boot config  https://review.openstack.org/39439918:04
*** fragatina has joined #openstack-ironic18:04
*** priteau has quit IRC18:05
*** david-lyle_ is now known as david-lyle18:08
openstackgerritoctopuszhang proposed openstack/ironic: Update multitenancy docs  https://review.openstack.org/39751718:08
jlvillalJayF: I think tonyb is going to be removing stable/liberty soon. If I understood the email that went out.18:13
JayFjlvillal: ah, okay. I was just going by what I read on releases.openstack.org this morning.18:13
jlvillalJayF: http://lists.openstack.org/pipermail/openstack-dev/2016-November/107717.html18:13
jlvillalJayF: Though I'm not sure if EOL means deleting the branch or not.18:14
*** vgadiraj has joined #openstack-ironic18:14
openstackgerritMerged openstack/bifrost: Enable inspector by default  https://review.openstack.org/40558018:14
JayFhttps://bugs.launchpad.net/python-ironicclient/+bug/1644833 I don't think this is an RFE, I think it's a bug-bug18:15
openstackLaunchpad 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
JayFIf no objections going to mark it that way18:15
yufeiCould 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
patchbotpatch 397517 - ironic - Update multitenancy docs18:18
yufeiIf it’s unnecessary now, I shall abandon it.18:18
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/40581218:20
*** jcoufal has joined #openstack-ironic18:21
JayFyufei: it's a useful thing, I think. Got a few comments about the wording I'll put inline.18:21
yufeiI see your comments now, thanks.  Also thank you for review at the patch ‘Skip API releated work if no api url configured’18:24
yufeiI will modify it tomorrow.18:24
JayFno problem18:25
JayFthanks for contributing18:25
yufei:)18:25
*** pramodrj07 has joined #openstack-ironic18:25
*** PramodJ has joined #openstack-ironic18:25
*** fragatina has quit IRC18:27
*** fragatina has joined #openstack-ironic18:27
openstackgerritChris Krelle proposed openstack/bifrost: Add initial keystone use documentation  https://review.openstack.org/38504118:31
*** daemontool has joined #openstack-ironic18:32
openstackgerritoctopuszhang proposed openstack/ironic: Update multitenancy docs  https://review.openstack.org/39751718:32
openstackgerritChris Krelle proposed openstack/bifrost: Add test script for Bifrost using keystone  https://review.openstack.org/38342618:33
*** e0ne has joined #openstack-ironic18:33
openstackgerritChris Krelle proposed openstack/bifrost: Update the task label for inspection  https://review.openstack.org/38342718:34
openstackgerritJulia Kreger proposed openstack/bifrost: Add keystone capability to base install  https://review.openstack.org/38504018:36
*** e0ne has quit IRC18:36
*** daemontool has quit IRC18:38
*** karts has joined #openstack-ironic18:39
*** karthiks has quit IRC18:39
*** yufei has quit IRC18:39
lindycoderHello 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
lindycoderso 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
lindycoderif 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
JayFlindycoder: basically asking: is there harm in loading more drivers than we need there18:41
JayFlindycoder: is what it sounds like it boils down to18:41
openstackgerritMerged openstack/bifrost: Add initial keystone use documentation  https://review.openstack.org/38504118:41
lindycoderJayF, 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 IRC18:44
*** karts has quit IRC18:44
JayFI honestly don't know, I follow your logic though: it shouldn't do harm to enable all drivers we can use in devstack18:44
JayFand just toggle the default driver based on which one is being tested18:44
JayFbut I'm curious what others would thing18:45
JayF*think18:45
openstackgerritJulia Kreger proposed openstack/ironic: Add Cinder storage driver  https://review.openstack.org/36619718:45
openstackgerritJulia Kreger proposed openstack/ironic: Add base cinder common interface  https://review.openstack.org/35562518:45
openstackgerritJulia Kreger proposed openstack/ironic: Add volume_connectors and volume_targets to task  https://review.openstack.org/36619618:45
openstackgerritJulia Kreger proposed openstack/ironic: Add storage_interface DB field and object  https://review.openstack.org/34800518:45
openstackgerritJulia Kreger proposed openstack/ironic: Add storage_interface to base driver class  https://review.openstack.org/34800618:45
openstackgerritJulia Kreger proposed openstack/ironic: Wire in storage interface attach/detach operations  https://review.openstack.org/40629018:45
lindycoderWell i don't know what is the cost of loading all the drivers18:46
*** vsaienko has joined #openstack-ironic18:46
lindycoderdo 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 good18:46
*** e0ne has joined #openstack-ironic18:47
JayFlindycoder: today; no, but I know in the future we want to do that18:47
JayFlindycoder: in order to have more than one deploy method tested in a given job18:48
JayFlindycoder: afaict the only cost in loading more drivers is conductor ram usage18:48
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Serve boot configs from Ironic API  https://review.openstack.org/36943818:48
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Implement Driver API for dynamic boot config  https://review.openstack.org/39439818:48
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get the boot config  https://review.openstack.org/39439918:48
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Allow not writing pxe configs to disk  https://review.openstack.org/39439718:48
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot config to proper Jinja template  https://review.openstack.org/39346218:48
*** e0ne has quit IRC18:48
*** karts has joined #openstack-ironic18:49
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE  https://review.openstack.org/39194318:49
*** soliosg has quit IRC18:49
lindycoderJayF, 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 easily18:51
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817318:51
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817318:51
JayFlindycoder: 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 sufficient18:52
*** soliosg has joined #openstack-ironic18:52
lindycoderJayF, 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
JayFlindycoder: I'm OK with that addition :)18:53
JayFlindycoder: and I'm a core reviewer, so I guess I count as someone who knows :)18:53
JayFsometimes this stuff existed beforehand and the logic behind why it's arranged how it is gets lost over time18:53
lindycoderJayF, yeah of course you do, you were just wondering what other would think earlier, i referred to that :P18:53
JayFYeah, the whole devstack-gate/project-config/default-settings-in-devstack configurations are ... strange, and I don't know always why things go where they go18:54
JayFjlvillal: ^ any insight?18:54
jlvillalJayF: 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
JayFjlvillal: well you can go back to the gate, it's not an urgent question :)18:57
*** mrtenio has joined #openstack-ironic18:57
jlvillalJayF: Okay. Will do...  Thanks :)18:57
*** PramodJ has quit IRC18:58
*** pramodrj07 has quit IRC18:58
*** ChubYann has joined #openstack-ironic19:02
lindycoderJayF, 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
lindycoderWhat 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 IRC19:02
lindycoderThis 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 gate19:02
JayFlindycoder: aiui a lot of those settings being in devstack-gate is for legacy reasons19:03
JayFlindycoder: which is why I don't think it's bad to move it to devstack defaults19:04
openstackgerritMerged openstack/bifrost: follow up to Change-Id: I4150f86fbc141dcd6fd12094acab1ca43d76910c  https://review.openstack.org/40626119:04
lindycoderJayF, 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 :D19:05
JayFlindycoder: well, I think it's more that it's unlikely we'll have too many more community CI'd drivers19:06
lindycoderwell xkhu and i are adding pxe_snmp... so that's one :P19:06
jlvillallindycoder: Hopefully you are aware of the 3rd Party CI requirement for new drivers.19:08
openstackgerritMerged openstack/ironic: Renaming audit map conf sample file  https://review.openstack.org/35039919:10
lindycoderjlvillal, i'm afraid i am not, but pxe_snmp is already in ironic we're just adding a gate for it19:10
jlvillallindycoder: Ah okay. I got confused as thought a new driver was being added. Please ignore me :)19:11
lindycoderjlvillal, no problem :D19:12
*** ijw has joined #openstack-ironic19:14
openstackgerritMartin Roy proposed openstack/ironic: Add all drivers used in gates as devstack defaults  https://review.openstack.org/40631319:14
lindycoderJayF, here it is ^ :)19:15
*** ijw has quit IRC19:15
*** ijw has joined #openstack-ironic19:15
*** Sukhdev has quit IRC19:21
*** zackf has joined #openstack-ironic19:22
*** soliosg has quit IRC19:24
*** bdeschenes has joined #openstack-ironic19:25
*** bdeschenes has quit IRC19:26
openstackgerritPavlo Shchelokovskyy proposed openstack/bifrost: Allow installing Ansible from PyPI  https://review.openstack.org/40632119:27
*** lindycoder has quit IRC19:29
openstackgerritMerged openstack/bifrost: Add test script for Bifrost using keystone  https://review.openstack.org/38342619:32
openstackgerritMerged openstack/bifrost: Update the task label for inspection  https://review.openstack.org/38342719:32
openstackgerritMerged openstack/bifrost: Add keystone capability to base install  https://review.openstack.org/38504019:33
*** jkilpatr has quit IRC19:34
*** serverascode has quit IRC19:34
*** NobodyCam has quit IRC19:34
*** tdurakov has quit IRC19:34
*** vdrok has quit IRC19:34
*** francoblanco has quit IRC19:34
*** TheJulia has quit IRC19:34
*** jkilpatr has joined #openstack-ironic19:35
*** lindycoder has joined #openstack-ironic19:36
*** dprince has quit IRC19:36
*** moshele has joined #openstack-ironic19:38
*** lindycoder has quit IRC19:39
openstackgerritJulia Kreger proposed openstack/bifrost: Adding keystone support to inspector config  https://review.openstack.org/38342519:39
*** ijw has quit IRC19:48
*** serverascode has joined #openstack-ironic19:50
openstackgerritJesse J. Cook proposed openstack/ironic: Add API and conductor methods for [un]rescue  https://review.openstack.org/35083119:58
*** ijw has joined #openstack-ironic20:00
*** clenimar has quit IRC20:11
*** vgadiraj has quit IRC20:12
*** tdurakov has joined #openstack-ironic20:15
*** vdrok has joined #openstack-ironic20:15
*** NobodyCam has joined #openstack-ironic20:16
*** francoblanco has joined #openstack-ironic20:17
*** TheJulia has joined #openstack-ironic20:19
*** vgadiraj has joined #openstack-ironic20:19
*** vsaienko has joined #openstack-ironic20:22
*** vsaienko has quit IRC20:26
*** ijw has quit IRC20:31
*** zackf has quit IRC20:32
*** soliosg has joined #openstack-ironic20:41
*** chlong has quit IRC20:42
openstackgerritJoanna Taryma proposed openstack/ironic: Fail IPA config if no protocol prefix in ironic api address  https://review.openstack.org/40497520:49
*** amoralej is now known as amoralej|off20:51
*** chlong has joined #openstack-ironic20:56
*** zackf has joined #openstack-ironic20:56
openstackgerritMerged openstack/bifrost: Adding keystone support to inspector config  https://review.openstack.org/38342520:57
*** devlaps has quit IRC20:57
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/40581221:03
*** karts has quit IRC21:06
*** MattMan has quit IRC21:07
*** MattMan has joined #openstack-ironic21:07
*** bfournie has quit IRC21:08
*** lindycoder has joined #openstack-ironic21:09
*** karts has joined #openstack-ironic21:09
*** karts has quit IRC21:13
*** srart has quit IRC21:17
*** srobert_ has joined #openstack-ironic21:26
*** karts has joined #openstack-ironic21:28
*** srobert has quit IRC21:30
*** srobert_ has quit IRC21:31
*** ijw has joined #openstack-ironic21:32
*** Sukhdev has joined #openstack-ironic21:33
*** srart has joined #openstack-ironic21:34
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM: multi-TENANT with tempest "smoke" test helper  https://review.openstack.org/40636821:37
*** ijw has quit IRC21:38
*** srart has quit IRC21:38
lindycoderJayF, 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
lindycoderit works in our working change, because when the gate uses pxe_snmp we install virtualpdu that installs pysnmp so devstack is happy21:44
lindycoderi think having the gate control the enabled drivers is more extensible, i'll propose that instead21:44
JayFlindycoder: cool, makes sense :)21:44
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM: multi-TENANT with tempest "smoke" test helper  https://review.openstack.org/40636821:45
*** absubram has joined #openstack-ironic21:46
*** bfournie has joined #openstack-ironic21:48
*** jcoufal has quit IRC21:51
*** moshele has quit IRC21:53
*** jkilpatr has quit IRC21:53
*** ijw has joined #openstack-ironic21:55
*** srart has joined #openstack-ironic21:55
*** jcoufal has joined #openstack-ironic22:01
*** daemontool has joined #openstack-ironic22:04
*** jcoufal_ has joined #openstack-ironic22:05
*** jcoufal has quit IRC22:07
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM: multi-TENANT with tempest "smoke" test helper  https://review.openstack.org/40636822:10
*** jcoufal_ has quit IRC22:10
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM: Testing tempest setting...  https://review.openstack.org/40637622:12
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM: multi-TENANT with tempest "smoke" test helper  https://review.openstack.org/40636822:13
*** ElCoyote_ has left #openstack-ironic22:13
*** Sukhdev has quit IRC22:14
*** ijw has quit IRC22:17
*** vsaienko has joined #openstack-ironic22:20
openstackgerritMartin Roy proposed openstack/ironic: Add a flag to auto-enable the deploy driver  https://review.openstack.org/40638122:27
*** vgadiraj has quit IRC22:31
lindycoderJayF, 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
patchbotpatch 406381 - ironic - Add a flag to auto-enable the deploy driver22:32
patchbotpatch 406384 - openstack-infra/devstack-gate - Use the auto-enable flag instead of changin enable...22:32
*** srart has quit IRC22:33
openstackgerritAnnie Lezil proposed openstack/ironic-inspector-specs: Collect NIC name given by BIOS  https://review.openstack.org/40437222:36
jlvillallindycoder: Why is it called "auto-enable"?  Isn't it just enabling the driver?22:36
jlvillallindycoder: Ah, it is a boolean. I thought it was a driver name.22:38
JayFlindycoder: TBH I'd just make that the default behavior22:38
JayFlindycoder: if IRONIC_DEPLOY_DRIVER not in IRONIC_ENABLED_DRIVERS, enable it22:38
JayFlindycoder: and maybe print a log message or something22:38
lindycoderjlvillal, 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 configs22:38
jrollhrm22:39
jrollwhy don't we just make devstack harder to misconfigure and always do that22:39
lindycoderJayF, making that the default behaviour goes against your review (https://review.openstack.org/#/c/387657/) no?22:39
jrollI think that's what JayF just said, ignore me :P22:39
patchbotpatch 387657 - ironic - Devstack will die if default driver is not enabled22:39
JayFlindycoder: if you implement it this way22:39
JayFlindycoder: I'll hit abandon on tha tpatch :)22:39
lindycoderAll right let's go that :D22:40
jcookfor rescuewait timeout should it use deploy_callback_timeout or a new rescue timeout?22:40
JayFjcook: I'd suggest making it a new option/config that defaults to the same value22:40
jrolljcook: probably the same thing cleaning does22:40
JayFjcook: we do have a separate one for cleaning ,right?22:40
jroll(which I think is the latter22:40
jroll)22:41
jcookkk22:41
jcookI code so infrequently it takes me 5 minutes to find the stoopid checkout this gerrit commit line to run22:41
jroll-d22:42
jrollalways -d22:42
jroll-x for cherry pick22:42
*** ppiela has quit IRC22:43
JayFfeel free to ask stupid questions instead of wasting 5 minutes trying to fnid their answers22:43
jcookoh no kidding22:43
JayFI keep that one in L1 cache22:43
JayFlol22:43
jrolltime to book mark http://docs.openstack.org/infra/git-review/usage.html22:43
jcookI just man git review and was like o_o, that's useful22:43
jrolllol22:43
jcookI mostly just git review when done making a change and don't use it otherwise22:43
*** Sukhdev has joined #openstack-ironic22:43
jrollwere you doing the copy line from the web page thing?22:43
jcookthink that's gonna change22:43
jcookyeah22:43
jrollyeah git-review is awesome22:44
jcookit's hard to find now in the webpage22:44
JayFGerrit is sorta strange when you first start working with it22:44
jrollyeah always a pain, I have to grab it sometime for faking out the gate22:44
*** chlong has quit IRC22:44
JayFbut once you get used to it, and git-review, holy crap it's powerful22:44
jcookwell I'll definitely be using -d going forward22:45
*** alexpilotti has joined #openstack-ironic22:46
JayFlindycoder: your patch will partial-bug this: https://bugs.launchpad.net/ironic/+bug/163356722:47
openstackLaunchpad bug 1633567 in Ironic "Devstack fails silently when provided an invalid driver" [Low,In progress] - Assigned to Jay Faulkner (jason-oldos)22:47
lindycoderJayF, all right i'll include it22:48
*** soliosg has quit IRC22:48
openstackgerritAnnie Lezil proposed openstack/ironic-inspector-specs: Collect NIC name given by BIOS  https://review.openstack.org/40437222:50
*** alexpilotti has quit IRC22:50
*** alexpilotti has joined #openstack-ironic22:54
openstackgerritMartin Roy proposed openstack/ironic: Add a flag to auto-enable the deploy driver  https://review.openstack.org/40638122:58
*** alexpilotti has quit IRC22:58
openstackgerritMartin Roy proposed openstack/ironic: Auto enable the deploy driver  https://review.openstack.org/40638123:00
lindycoderJayF, here it is ^ i'm off for today, thanks a lot for the help!23:01
*** alexpilotti has joined #openstack-ironic23:02
*** vgadiraj has joined #openstack-ironic23:08
*** lindycoder has quit IRC23:08
*** daemontool has quit IRC23:13
*** ijw has joined #openstack-ironic23:15
openstackgerritAnnie Lezil proposed openstack/ironic-inspector-specs: Collect NIC name given by BIOS  https://review.openstack.org/40437223:16
*** ijw has quit IRC23:16
openstackgerritFélix Bouliane proposed openstack/ironic: Add virtualpdu to ironic devstack plugin  https://review.openstack.org/38815423:17
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM: Testing tempest setting...  https://review.openstack.org/40637623:22
*** Sukhdev has quit IRC23:28
*** vgadiraj has quit IRC23:28
*** jcoufal has joined #openstack-ironic23:28
*** jcoufal has quit IRC23:29
*** rbudden has quit IRC23:36
*** rama_y has quit IRC23:38
*** [1]cdearborn has quit IRC23:38
*** vgadiraj has joined #openstack-ironic23:39
openstackgerritChris Krelle proposed openstack/bifrost: Initial work on adding staging driver support  https://review.openstack.org/40640123:43
*** vgadiraj has quit IRC23:55
*** ijw has joined #openstack-ironic23:59

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