Thursday, 2016-12-01

*** jcoufal has quit IRC00:00
*** bfournie has quit IRC00:01
jlvillalJayF: Trying to add comment. I'm thinking that the Gerrit restart thing happens at 4PM PST now :(00:03
JayFjlvillal: it's 00:00 UTC00:03
* jlvillal grumbles00:03
JayFjlvillal: which is 4p PST now that DST is changed00:03
*** rbudden has joined #openstack-ironic00:04
jlvillalYeah. I think I will be unlucky more often now. Until DST happens again.00:04
*** absubram has joined #openstack-ironic00:04
jlvillalJayF: Updated with comment and vote.00:05
NobodyCamwe need to get rid of DST!00:06
jlvillalNobodyCam: I could support that :)00:09
NobodyCam:)00:09
jlvillalNobodyCam: I liked the plan I saw where have only two time zones for the US. And only one hour apart.00:09
NobodyCamhmm not sure about that, I'm use to the three zones.. but would def vote to get rid DST00:10
NobodyCam:p00:10
jlvillalNobodyCam: There are 4 time zones in the lower 48.00:10
NobodyCams/three/four/00:10
jlvillalNobodyCam: http://qz.com/142199/the-us-needs-to-retire-daylight-savings-and-just-have-two-time-zones-one-hour-apart/00:11
NobodyCamhummm00:13
NobodyCamlol00:13
NobodyCamI would have to see how it would be implemented00:13
NobodyCamI fear that those of us in western states to have the largest adjustment in a plan like that00:15
*** hoangcx has joined #openstack-ironic00:28
*** vgadiraj has joined #openstack-ironic00:37
*** [1]rpioso has quit IRC00:43
*** rama_y has quit IRC00:49
*** soliosg has quit IRC00:52
*** SerenaFeng has joined #openstack-ironic00:55
*** baoli_ has quit IRC01:01
*** baoli has joined #openstack-ironic01:05
*** mtanino has quit IRC01:10
*** Goneri has quit IRC01:12
*** rajinir has quit IRC01:16
*** SerenaFeng has quit IRC01:17
*** SerenaFeng has joined #openstack-ironic01:26
*** SerenaFeng has quit IRC01:27
*** SerenaFeng has joined #openstack-ironic01:27
*** SerenaFeng has quit IRC01:28
*** penick has quit IRC01:29
*** hw_wutianwei has joined #openstack-ironic01:30
*** devlaps has quit IRC01:32
*** lindycoder has joined #openstack-ironic01:40
TheJuliaNobodyCam: just get rid of time zones completely01:43
*** zhangjl has joined #openstack-ironic01:51
*** rloo has quit IRC01:52
*** Nisha_ has joined #openstack-ironic01:54
*** pmannidi has quit IRC01:58
*** Nisha_ has quit IRC01:58
*** adu has joined #openstack-ironic01:59
*** pmannidi has joined #openstack-ironic02:00
*** adu has quit IRC02:12
joannathanks, JayF! :)02:17
*** Syed_ has quit IRC02:26
*** srobert has quit IRC02:26
*** SerenaFeng has joined #openstack-ironic02:26
openstackgerritjiang wei proposed openstack/ironic: Move root controller tests to functional dir  https://review.openstack.org/35618802:28
openstackgerritjiang wei proposed openstack/ironic: Move api driver tests to functional dir  https://review.openstack.org/35036502:28
openstackgerritjiang wei proposed openstack/ironic: Move port api tests to functional dir  https://review.openstack.org/35628702:28
openstackgerritjiang wei proposed openstack/ironic: Prepare for functional tests  https://review.openstack.org/34942102:28
*** gcb has joined #openstack-ironic02:34
*** srobert has joined #openstack-ironic02:43
*** srobert has quit IRC02:44
*** srobert has joined #openstack-ironic02:44
*** yuanying has quit IRC02:51
*** yuanying has joined #openstack-ironic02:51
*** aNuposic has quit IRC02:55
*** srobert has quit IRC02:58
openstackgerritjiang wei proposed openstack/python-ironicclient: Support i18n for baremetal node cmds  https://review.openstack.org/35763903:03
*** lindycoder has quit IRC03:08
openstackgerritJoanna Taryma proposed openstack/ironic: Fail IPA config if no protocol prefix in ironic api address  https://review.openstack.org/40497503:13
*** absubram has quit IRC03:21
*** Goneri has joined #openstack-ironic03:23
*** rbudden has quit IRC03:27
*** Nisha_Agarwal has joined #openstack-ironic03:28
*** bfournie has joined #openstack-ironic03:32
*** yuanying has quit IRC03:46
*** Goneri has quit IRC03:50
openstackgerritNisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers  https://review.openstack.org/37242803:53
openstackgerritAparna proposed openstack/ironic: Update the links in iLO documentation  https://review.openstack.org/40508403:56
*** tuanluong has joined #openstack-ironic04:06
*** SerenaFeng has quit IRC04:06
openstackgerritNisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers  https://review.openstack.org/37242804:08
*** SerenaFeng has joined #openstack-ironic04:09
openstackgerritNisha Agarwal proposed openstack/ironic: Update the links in iLO documentation  https://review.openstack.org/40508404:10
*** SerenaFeng has quit IRC04:12
*** absubram has joined #openstack-ironic04:21
*** absubram_ has joined #openstack-ironic04:25
*** absubram has quit IRC04:26
*** absubram_ is now known as absubram04:26
*** watanabe_isao has joined #openstack-ironic04:46
openstackgerritNisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers  https://review.openstack.org/37242804:47
*** yuanying has joined #openstack-ironic04:48
*** yuanying_ has joined #openstack-ironic04:55
*** yuanying has quit IRC04:57
*** links has joined #openstack-ironic05:00
*** vikrant has joined #openstack-ironic05:00
*** yuanying has joined #openstack-ironic05:02
*** yuanying_ has quit IRC05:04
*** yuanying_ has joined #openstack-ironic05:10
*** yuanying has quit IRC05:11
*** SerenaFeng has joined #openstack-ironic05:41
*** pmannidi_ has joined #openstack-ironic05:48
*** pmannidi has quit IRC05:48
*** pmannidi_ is now known as pmannidi05:48
*** aNuposic has joined #openstack-ironic05:55
*** SerenaFeng has quit IRC06:09
*** SerenaFeng has joined #openstack-ironic06:10
*** baoli has quit IRC06:11
*** yuanying_ has quit IRC06:18
*** jaosorior has joined #openstack-ironic06:19
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off  https://review.openstack.org/21673006:27
*** SerenaFeng has quit IRC06:30
*** watanabe_isao has quit IRC06:33
*** ChubYann has quit IRC06:36
*** moshele has joined #openstack-ironic06:38
*** mjura has joined #openstack-ironic06:46
*** SerenaFeng has joined #openstack-ironic06:51
*** mjura has quit IRC06:54
*** mjura has joined #openstack-ironic06:54
*** tuanluong_ has joined #openstack-ironic07:08
*** jaosorior has quit IRC07:09
*** jaosorior has joined #openstack-ironic07:10
openstackgerritjiang wei proposed openstack/python-ironicclient: Support i18n for baremetal node cmds  https://review.openstack.org/35763907:10
*** tuanluong has quit IRC07:11
*** zhangjl1 has joined #openstack-ironic07:13
*** zhangjl has quit IRC07:15
*** SerenaFeng has quit IRC07:32
*** e0ne has joined #openstack-ironic07:32
*** zhangjl1 has quit IRC07:33
*** zhangjl has joined #openstack-ironic07:33
openstackgerritMerged openstack/proliantutils: Change to disable target DHCP in iLO  https://review.openstack.org/40411107:35
*** SerenaFeng has joined #openstack-ironic07:39
*** vsaienko has joined #openstack-ironic07:42
*** ccamacho|afk is now known as ccamacho07:58
*** e0ne has quit IRC07:59
openstackgerritGhanshyam Mann proposed openstack/ironic: Keep CONF.baremetal.driver_enabled for Tempest plugin  https://review.openstack.org/40003208:01
*** slagle has quit IRC08:05
*** SerenaFeng has quit IRC08:05
*** SerenaFeng has joined #openstack-ironic08:06
*** tuanluong has joined #openstack-ironic08:08
*** vsaienko has quit IRC08:10
*** tuanluong_ has quit IRC08:10
*** lhinds has quit IRC08:13
*** ralonsoh has joined #openstack-ironic08:13
*** misty-cloud has joined #openstack-ironic08:14
*** dtantsur|afk is now known as dtantsur08:14
dtantsurmorning Ironic08:15
openstackgerritDmitry Tantsur proposed openstack/ironic: Move interface validation from API to conductor side  https://review.openstack.org/40131108:17
*** xiaoyang has joined #openstack-ironic08:22
*** wznoinsk has quit IRC08:26
*** marios has quit IRC08:26
*** yuanying has joined #openstack-ironic08:27
*** marios has joined #openstack-ironic08:28
*** ccamacho has quit IRC08:28
*** jcook has quit IRC08:28
*** wznoinsk has joined #openstack-ironic08:30
*** ccamacho has joined #openstack-ironic08:30
*** aNuposic has quit IRC08:31
*** jcook has joined #openstack-ironic08:34
*** rcernin has joined #openstack-ironic08:34
*** Guest66666 has quit IRC08:36
*** rcernin has quit IRC08:37
*** rcernin has joined #openstack-ironic08:37
*** Guest66666 has joined #openstack-ironic08:38
openstackgerritparesh sao proposed openstack/ironic: [WIP] Boot from iSCSI volume  https://review.openstack.org/30842508:39
*** aparnav has joined #openstack-ironic08:42
*** jpich has joined #openstack-ironic08:46
*** e0ne has joined #openstack-ironic08:50
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-ironic09:00
openstackgerritDmitry Tantsur proposed openstack/ironic: Move interface validation from API to conductor side  https://review.openstack.org/40131109:01
*** ohamada has joined #openstack-ironic09:01
*** fxpester has joined #openstack-ironic09:05
*** nizam037 has joined #openstack-ironic09:08
*** fxpester|2 has joined #openstack-ironic09:09
*** hogepodge has quit IRC09:11
*** fxpester has quit IRC09:11
*** hogepodge has joined #openstack-ironic09:12
*** fxpester has joined #openstack-ironic09:14
*** SerenaFeng has quit IRC09:14
*** Romanenko_K has joined #openstack-ironic09:15
*** SerenaFeng has joined #openstack-ironic09:16
*** fxpester|2 has quit IRC09:17
*** milan has joined #openstack-ironic09:17
milanmorning Ironic! :)09:17
dtantsurhey milan09:17
openstackgerritDmitry Tantsur proposed openstack/ironic: Support defining and loading hardware types  https://review.openstack.org/33662609:17
milanmorning dtantsur! :)09:17
*** fxpester|2 has joined #openstack-ironic09:19
aarefievmorning!09:20
*** igordcard has quit IRC09:21
*** sambetts|afk has quit IRC09:22
*** fxpester has quit IRC09:22
pas-hamorning all :)09:22
milanmorning aarefiev, pas-ha! :)09:23
aarefievdtantsur: morning, IIUC John's patch should fix inspector grenade, right ?09:23
aarefievmorning milan09:23
*** igordcard has joined #openstack-ironic09:23
dtantsurmorning aarefiev, dunno09:23
* dtantsur has not followed09:24
aarefievdtantsur: ok, also I have a suspicion, this patch could skip inspector tests on grenade https://review.openstack.org/#/c/404436/09:25
patchbotpatch 404436 - ironic - Move CONF.service_available.ironic to our plugin09:25
aarefievcould you take a look09:25
*** fxpester has joined #openstack-ironic09:25
aarefievmorning pas-ha09:26
*** dtantsur is now known as dtantsur|bbl09:27
*** sambetts_ has joined #openstack-ironic09:27
*** fxpester|3 has joined #openstack-ironic09:28
*** fxpester|2 has quit IRC09:28
*** rcernin has quit IRC09:29
*** fxpester|2 has joined #openstack-ironic09:30
*** rcernin has joined #openstack-ironic09:30
*** fxpester has quit IRC09:30
*** hogepodge has quit IRC09:31
*** fxpester has joined #openstack-ironic09:32
*** hogepodge has joined #openstack-ironic09:32
*** fxpester|3 has quit IRC09:33
openstackgerritDmitry Tantsur proposed openstack/ironic: Support names for {cleaning,provisioning}_network  https://review.openstack.org/35798309:33
*** SerenaFeng has quit IRC09:34
*** fxpester|2 has quit IRC09:35
*** fxpester|2 has joined #openstack-ironic09:35
*** lucas-afk is now known as lucasagomes09:36
lucasagomesmorning all09:36
*** derekh has joined #openstack-ironic09:36
aarefievmorning lucasagomes09:37
*** yuanying_ has joined #openstack-ironic09:38
*** fxpester has quit IRC09:38
*** fxpester has joined #openstack-ironic09:38
*** fxpester|3 has joined #openstack-ironic09:40
*** fxpester|2 has quit IRC09:41
*** yuanying has quit IRC09:41
*** fxpester|2 has joined #openstack-ironic09:42
*** fxpester has quit IRC09:43
*** SerenaFeng has joined #openstack-ironic09:43
*** fxpester has joined #openstack-ironic09:44
*** fxpester|3 has quit IRC09:45
*** fxpester|3 has joined #openstack-ironic09:46
*** fxpester|2 has quit IRC09:47
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Do not source old/localrc twise in grenade  https://review.openstack.org/40523309:47
*** SerenaFeng has quit IRC09:49
*** fxpester has quit IRC09:50
*** fxpester|2 has joined #openstack-ironic09:50
aarefievmilan: looks like grenade has the same problem as in ironic was09:51
milanaarefiev, I missed the discussion :-/09:51
* milan just looks into the traces and seems the gate is sick :-/09:51
*** fxpester|3 has quit IRC09:52
milanaarefiev, is it this the root cause: logs.openstack.org/43/348943/32/check/gate-grenade-dsvm-ironic-inspector/f3c0c6c/logs/grenade.sh.txt.gz#_2016-11-30_17_37_47_30309:52
*** fxpester has joined #openstack-ironic09:53
*** jralph_ has joined #openstack-ironic09:53
aarefievmilan: no, looks like this http://logs.openstack.org/43/348943/32/check/gate-grenade-dsvm-ironic-inspector/f3c0c6c/logs/grenade.sh.txt.gz#_2016-11-30_17_37_46_14409:55
*** fxpester|2 has quit IRC09:55
*** openstack has joined #openstack-ironic10:04
aarefievmilan: as you wish :)10:05
milanack10:05
*** rcernin has quit IRC10:06
*** rcernin has joined #openstack-ironic10:07
*** fxpester|2 has joined #openstack-ironic10:09
openstackgerritLucas Alvares Gomes proposed openstack/pyghmi: IpmiServer to allow using IP version 4  https://review.openstack.org/40483710:09
lucasagomesvdrok, o/10:09
*** fxpester|3 has joined #openstack-ironic10:11
*** hoangcx has quit IRC10:11
*** fxpester has quit IRC10:12
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Show team and repo badges on README  https://review.openstack.org/40249510:12
*** fxpester|2 has quit IRC10:14
*** fxpester has joined #openstack-ironic10:14
*** fxpester|2 has joined #openstack-ironic10:16
*** fxpester|3 has quit IRC10:17
milanaarefiev, https://bugs.launchpad.net/ironic-inspector/+bug/164643010:18
openstackLaunchpad bug 1646430 in Ironic Inspector "Plugin attempted to be enabled twice: ironic" [Undecided,New]10:18
milanmorning lucasagomes! :)10:19
*** fxpester has quit IRC10:19
aarefievmilan: thanks, waiting for ci10:19
*** fxpester has joined #openstack-ironic10:19
*** romcheg has quit IRC10:20
*** romcheg has joined #openstack-ironic10:21
*** romcheg has quit IRC10:21
milanaarefiev, roger that, ping me to review please ;)10:21
*** fxpester|2 has quit IRC10:22
aarefievmilan: sure, don't worry :)10:22
milan:)10:22
*** fxpester|2 has joined #openstack-ironic10:24
*** fxpester|3 has joined #openstack-ironic10:27
*** zhangjl1 has joined #openstack-ironic10:27
*** fxpester has quit IRC10:27
*** zhangjl has quit IRC10:29
*** fxpester|2 has quit IRC10:29
*** fxpester has joined #openstack-ironic10:31
*** zhangjl1 has quit IRC10:31
*** rbartal_ has joined #openstack-ironic10:31
*** anish has quit IRC10:34
*** fxpester|3 has quit IRC10:34
*** fxpester|2 has joined #openstack-ironic10:34
*** fxpester|3 has joined #openstack-ironic10:36
*** fxpester has quit IRC10:37
*** sambetts_ is now known as sambetts10:38
sambettsMorning all10:38
*** moshele has quit IRC10:39
*** vgadiraj has quit IRC10:39
*** rbartal has quit IRC10:39
*** rook has quit IRC10:39
*** patchbot has quit IRC10:39
*** tonyb has quit IRC10:39
*** dansmith has quit IRC10:39
*** fxpester|2 has quit IRC10:39
*** fxpester has joined #openstack-ironic10:40
*** fxpester|2 has joined #openstack-ironic10:43
*** moshele has joined #openstack-ironic10:43
*** vgadiraj has joined #openstack-ironic10:43
*** rook has joined #openstack-ironic10:43
*** patchbot has joined #openstack-ironic10:43
*** tonyb has joined #openstack-ironic10:43
*** dansmith has joined #openstack-ironic10:43
*** fxpester|3 has quit IRC10:43
*** fxpester has quit IRC10:45
*** fxpester has joined #openstack-ironic10:46
*** fxpester|3 has joined #openstack-ironic10:48
*** fxpester|2 has quit IRC10:48
*** fxpester|2 has joined #openstack-ironic10:50
*** fxpester has quit IRC10:51
*** fxpester has joined #openstack-ironic10:53
*** fxpester|3 has quit IRC10:53
*** fxpester|3 has joined #openstack-ironic10:55
*** fxpester|2 has quit IRC10:55
*** fxpester|2 has joined #openstack-ironic10:57
*** fxpester has quit IRC10:58
lucasagomessambetts, morning10:58
*** Guest2696 has quit IRC10:59
*** fxpester|3 has quit IRC11:00
*** fxpester has joined #openstack-ironic11:03
vdrokmorning sambetts11:03
*** fxpester|3 has joined #openstack-ironic11:05
*** fxpester|2 has quit IRC11:05
vsaienk0morning lucasagomes, sambetts, dtantsur, milan, aarefiev, vdrok, pas-ha and all ironic'ers!11:07
lucasagomeso/11:07
*** fxpester has quit IRC11:08
*** fxpester has joined #openstack-ironic11:09
*** misty-cloud has quit IRC11:10
*** bapalm has quit IRC11:10
aarefievmorning sambetts, vsaienk011:10
*** fxpester|3 has quit IRC11:12
aarefievvsaienk0: are you going to propose patch to ironic with removing upgrade from non-ironic  env?11:12
*** fxpester|2 has joined #openstack-ironic11:13
milanmorning vsaienk0! :)11:13
sambettso/ vsaienk0, vdrok, lucasagomes aarefiev milan11:13
vdrokmorning vsaienk011:13
milanmorning sambetts! :)11:13
vsaienk0aarefiev: I first want to know other opinion on this :)11:13
mgouldmorning sambetts vdrok vsaienk0 aarefiev lucasagomes and all other Ironicers11:14
aarefievmorning mgould11:14
*** fxpester has quit IRC11:15
*** bapalm has joined #openstack-ironic11:15
vdrokmorning mgould11:18
*** fxpester has joined #openstack-ironic11:19
*** fxpester|3 has joined #openstack-ironic11:21
*** karthiks has quit IRC11:21
*** fxpester|2 has quit IRC11:21
*** aparnav has quit IRC11:22
*** fxpester|2 has joined #openstack-ironic11:23
*** fxpester has quit IRC11:23
aarefievmilan: why do we have low number of achievements, there is no upgrade one (  https://review.openstack.org/#/c/402495/11:25
patchbotpatch 402495 - ironic-inspector - Show team and repo badges on README11:25
*** jaosorior_lunch is now known as jaosorior11:25
*** fxpester has joined #openstack-ironic11:26
*** fxpester|3 has quit IRC11:26
*** fxpester|2 has quit IRC11:28
*** fxpester|2 has joined #openstack-ironic11:29
*** karthiks has joined #openstack-ironic11:31
*** fxpester has quit IRC11:32
*** fxpester has joined #openstack-ironic11:32
*** athomas has joined #openstack-ironic11:34
openstackgerritHironori Shiina proposed openstack/ironic: Add REST API for volume connector and volume target operation  https://review.openstack.org/21458611:34
*** fxpester|2 has quit IRC11:35
*** fxpester|2 has joined #openstack-ironic11:36
*** fxpester has quit IRC11:39
jrollmorning y'all11:39
*** fxpester has joined #openstack-ironic11:39
* jroll chugs coffee11:39
sambettso/ jroll, that sounds like a good idea11:40
*** fxpester|3 has joined #openstack-ironic11:41
*** fxpester|2 has quit IRC11:42
openstackgerritVasyl Saienko proposed openstack/ironic: Cleanup adding Ironic to cluster on upgrade case  https://review.openstack.org/40529011:43
vsaienk0sambetts: are you planning to work on https://review.openstack.org/#/c/308348 I've seen this problem with ironic-multinode job recently11:44
patchbotpatch 308348 - nova - Eliminate resource tracker to host manager races (ABANDONED)11:44
*** fxpester has quit IRC11:44
*** fxpester has joined #openstack-ironic11:44
sambettsI need to revisit that patch, I spoke to one of the Nova team about it at the summit, they have been doing a lot of work on the scheduler and they wanted to see if it would be fixed by their changes first, but it doesn't seem like it11:45
*** fxpester|3 has quit IRC11:47
vdrokmorning jroll11:47
*** fxpester|2 has joined #openstack-ironic11:48
*** Nisha_Agarwal has quit IRC11:49
vsaienk0sambetts: yes, the problem still exist, we meet with it yesterday  http://logs.openstack.org/36/404436/4/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-multinode-nv/d13d19c/logs/screen-n-sch.txt.gz?level=ERROR11:49
*** fxpester|3 has joined #openstack-ironic11:50
vdroksambetts: yeah, I'm not sure if it will or not. if i understand the planned changes correctly, nova compute will not report any resources anymore in case of ironic11:50
*** jkilpatr has quit IRC11:50
vdrokvsaienk0: it can not be, we have not switched to resource providers yet :)11:50
*** fxpester has quit IRC11:51
*** fxpester|2 has quit IRC11:52
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add testcases for osc port group comands  https://review.openstack.org/38542711:53
vdroksambetts: actually, if the compute won't report resources, the problem should be gone. but i don't think it will happen this cycle11:54
*** fxpester has joined #openstack-ironic11:54
sambettsvdrok: yeah, I'll see if I can revive that patch and rebase it, its always a nasty rebase though ;)11:54
vdroksambetts: in nova, yup, I know that :)11:55
*** igordcard has quit IRC11:55
*** dprince has joined #openstack-ironic11:56
*** fxpester|2 has joined #openstack-ironic11:56
*** fxpester|3 has quit IRC11:57
*** fxpester|3 has joined #openstack-ironic11:58
*** fxpester has quit IRC11:59
*** awidders_ has quit IRC11:59
*** fxpester|2 has quit IRC12:01
*** fellypefca has joined #openstack-ironic12:01
*** karthiks has quit IRC12:10
*** slagle has joined #openstack-ironic12:10
TheJuliaGood morning everyone12:11
*** dprince has quit IRC12:11
aarefievmorning jroll, TheJulia12:13
*** fxpester has joined #openstack-ironic12:16
lucasagomesTheJulia, morning12:17
charlledanielmorning jroll, TheJulia, all12:17
*** fxpester|3 has quit IRC12:20
*** fxpester|2 has joined #openstack-ironic12:21
*** yufei has joined #openstack-ironic12:23
*** fxpester|3 has joined #openstack-ironic12:23
*** fxpester has quit IRC12:24
*** fxpester has joined #openstack-ironic12:25
*** karthiks has joined #openstack-ironic12:25
openstackgerritOksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV  https://review.openstack.org/39712612:26
*** fxpester|2 has quit IRC12:26
*** jkilpatr has joined #openstack-ironic12:27
*** fxpester|3 has quit IRC12:28
*** athomas has quit IRC12:28
*** fxpester|2 has joined #openstack-ironic12:30
*** fxpester has quit IRC12:33
*** fxpester has joined #openstack-ironic12:33
*** strigazi_AFK is now known as strigazi12:34
vsaienk0morning TheJulia12:34
*** lucasagomes is now known as lucas-hungry12:36
*** fxpester|2 has quit IRC12:36
*** fxpester|2 has joined #openstack-ironic12:36
openstackgerritGalyna Zholtkevych proposed openstack/ironic: 'updated_at' field value after node is updated  https://review.openstack.org/37160512:37
*** fxpester has quit IRC12:39
openstackgerritGalyna Zholtkevych proposed openstack/ironic: 'updated_at' field value after node is updated  https://review.openstack.org/37160512:41
openstackgerritStenio Araujo proposed openstack/ironic: Adds another validation step when using dynamic allocation  https://review.openstack.org/37707312:41
*** dprince has joined #openstack-ironic12:41
*** fxpester has joined #openstack-ironic12:41
*** fxpester|3 has joined #openstack-ironic12:43
*** fxpester|2 has quit IRC12:44
vdrokmorning TheJulia and charlledaniel12:45
*** fxpester|2 has joined #openstack-ironic12:46
*** fxpester has quit IRC12:47
*** fxpester|3 has quit IRC12:48
*** fxpester has joined #openstack-ironic12:49
mrteniomorning all12:49
*** fellypefca has quit IRC12:50
*** fxpester|2 has quit IRC12:51
*** bfournie has quit IRC12:53
*** fxpester|2 has joined #openstack-ironic12:54
mosheleaarefiev,  milan: hi  I fixed the comments on the usage.rst. I am not sure what to do about the comment on the node_cache in https://review.openstack.org/#/c/264257/12:55
patchbotpatch 264257 - ironic-inspector - Adding InfiniBand Support12:55
*** srobert has joined #openstack-ironic12:55
*** fxpester|3 has joined #openstack-ironic12:56
*** fxpester has quit IRC12:57
*** srobert has quit IRC12:59
*** fxpester|2 has quit IRC12:59
*** srobert has joined #openstack-ironic12:59
*** snecklifter has quit IRC13:00
*** fxpester has joined #openstack-ironic13:02
*** dtantsur|bbl is now known as dtantsur13:04
*** fxpester|2 has joined #openstack-ironic13:04
dtantsurmorning jroll, sambetts, TheJulia, vdrok and everyone I missed in the morning13:05
dtantsuraarefiev, we don't officially have supports-upgrade tag. I need to finish the upgrade documentation before claiming it.13:05
*** fxpester|3 has quit IRC13:05
*** fxpester has quit IRC13:07
*** fxpester has joined #openstack-ironic13:08
openstackgerritMerged openstack/ironic-inspector: Do not source old/localrc twise in grenade  https://review.openstack.org/40523313:08
*** ccamacho is now known as ccamacho|lunch13:09
dtantsuraarefiev, do we need stable backport for ^^^?13:09
*** fxpester|2 has quit IRC13:10
aarefievmoshele: thanks, I'll take a look later13:11
aarefievdtantsur: aha, I see, I guess we will need backport if don't delete it at all https://review.openstack.org/#/c/405290/113:12
patchbotpatch 405290 - ironic - Cleanup adding Ironic to cluster on upgrade case13:12
*** fxpester|2 has joined #openstack-ironic13:12
dtantsurI see13:12
*** trown|outtypewww is now known as trown13:13
*** fxpester has quit IRC13:15
*** fxpester has joined #openstack-ironic13:15
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Clean up retained code for grenade workaround  https://review.openstack.org/40536613:16
openstackgerritDmitry Tantsur proposed openstack/ironic: Support names for {cleaning,provisioning}_network  https://review.openstack.org/35798313:16
dtantsursambetts, mind taking a look please ^^^?13:16
sambettsdtantsur: I'll check it out when I get a minute :)13:16
dtantsurthanks, no hurry13:17
*** fxpester|2 has quit IRC13:18
*** fxpester|2 has joined #openstack-ironic13:19
*** mchasal_ has joined #openstack-ironic13:21
*** fxpester|3 has joined #openstack-ironic13:22
milanaarefiev, moshele sry guys just returned from the lunch13:22
*** fxpester has quit IRC13:22
* milan reads the scroll back13:22
*** mjturek1 has joined #openstack-ironic13:22
*** mjturek has quit IRC13:24
*** mchasal has quit IRC13:24
*** bfournie has joined #openstack-ironic13:24
*** fxpester|2 has quit IRC13:24
*** jrist has quit IRC13:25
aarefievmilan: I hope you had a great lunch :)13:25
mgouldaarefiev: we did, yes :-)13:27
milanaarefiev, thanks, yeah we had chicken korma with mgould :P13:27
mgouldthe first place we went to was closed, so we had to find somewhere else13:27
*** jtomasek has joined #openstack-ironic13:27
*** tuanluong has quit IRC13:29
aarefievnice :) I don't think I tried chicken korma13:30
mgould*chicken madras13:32
*** fxpester has joined #openstack-ironic13:32
mgouldkorma is nice, though: lots of coconut13:32
sambettshaha those are very different currys13:32
*** jtomasek has quit IRC13:33
*** lucas-hungry is now known as lucasagomes13:34
*** fxpester|2 has joined #openstack-ironic13:34
openstackgerritJarrod Johnson proposed openstack/pyghmi: IpmiServer to allow using IP version 4  https://review.openstack.org/40483713:35
*** fxpester|3 has quit IRC13:35
jjohnson2lucasagomes, sorry, in my haste I neglected to change 'if ipv6support' to 'elif'13:36
jjohnson2lucasagomes, I think this is the most 'proper' approach, if 'server', then use the address family and sockaddr from getaddrinfo call13:37
jjohnson2only when being a client to do ipv6 if possible, then fall back to ipv413:37
*** fxpester has quit IRC13:37
lucasagomesjjohnson2, hi there, no worries! I appreciate the review/update on that patch13:38
lucasagomesjjohnson2, right, yeah that would work too13:38
lucasagomeslemme update the patch to do it then13:39
jjohnson2lucasagomes, and when IPv8 inevitably happens, it may support that ;)13:39
*** fxpester has joined #openstack-ironic13:39
jjohnson2lucasagomes, well, I pushed patch set 613:39
lucasagomesjjohnson2, ah right, lemme test it out13:39
jjohnson2lucasagomes, thanks13:39
lucasagomesjjohnson2, hahah IPv8, IPv16 are ppl really talking about it already ?13:39
jjohnson2lucasagomes, nope13:40
jjohnson2but our intergalactic empire will need it13:40
lucasagomesindeed13:40
lucasagomesI was watching "the expanse" series few months ago13:41
lucasagomesthey def need it13:41
lucasagomes(it's a good series as well, worth watching)13:41
mgouldlucasagomes: saw the first episode, really liked it13:42
*** fxpester|2 has quit IRC13:42
mgouldlike Alien would have looked if they'd made it today13:42
lucasagomesjjohnson2, works13:43
*** snecklifter has joined #openstack-ironic13:43
lucasagomes+1'd13:43
lucasagomesmgould, yeah! Def watch the rest of the season13:44
*** fxpester|2 has joined #openstack-ironic13:44
*** jcoufal has joined #openstack-ironic13:45
mgouldlucasagomes: will do13:45
mgoulddo we have any HPE folks here?13:46
* mgould is trying to work out what to do in https://review.openstack.org/#/c/344214/13:46
patchbotpatch 344214 - diskimage-builder - Allow installing proliant tools from an OS package13:46
mgouldthe hpssacli tool has apparently been renamed, but only for some distros13:47
mgouldor something13:47
*** fxpester has quit IRC13:47
* mgould is confused, again13:47
*** fxpester has joined #openstack-ironic13:47
sambettsoh dear...13:48
mgouldyeah13:49
TheJuliamgould: ex-hpe :)  ssacli is likely the route to go... although it is not a pretty sight :(13:49
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Document upgrade procedure  https://review.openstack.org/39327513:49
dtantsurmilan, aarefiev, ^^^13:49
mgouldTheJulia: and install the RPM on .deb-based distros using alien?13:49
milanmorning TheJulia! :)13:49
milandtantsur, ack13:49
* milan looks13:49
*** fxpester|2 has quit IRC13:49
TheJuliamgould: maybe rpm2cpio....13:50
mgouldTheJulia: thanks13:51
mgouldthe code was using alien before my patch, so I should probably stick with that13:51
TheJuliamgould: you'll loose installer scripts, but that might be okay, dunno about how they package it13:51
* mgould has no frickin idea, sorry13:51
TheJuliano worries13:51
mgouldI guess I could look inside the RPM13:52
mgouldseems weird that they'd provide debs for v2.40 and then stop doing so for the next version13:52
TheJuliano comment regarding that :(13:52
dtantsurTheJulia, hey! I'm looking into future boot-from-volume support for TripleO. Is there anything special about Cinder installation for this purpose? What backend should be used? (sorry if it's already answered in the specs, I still can't completely digest it)13:55
TheJuliaThe intent is to be as agnostic as possible, so any backend should work as long as it can hand us an iscsi target and credentials back in the attach request.13:56
dtantsuraha13:57
* dtantsur should probably read something about cinder before jumping into this work :)13:57
TheJuliauhh... yes13:57
TheJuliayour brain might break though13:57
TheJuliabe warned ;)13:57
*** fxpester|2 has joined #openstack-ironic13:58
dtantsurthanks, this is exactly what I expect from learning yet another openstack service ;)13:58
*** cdearborn has joined #openstack-ironic13:58
TheJuliatl;dr, the attachments is why I drove forward storage_interface, so we could handle attachment data  with some grace13:58
*** lhinds has joined #openstack-ironic13:59
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817313:59
TheJuliadtantsur: feel free to ping me if you have any questions.  I realized last night that I forgot to write unit tests for some of the cinder interface stuff so that is on my list to do today, and then back to wiring it in14:00
*** snecklifter has quit IRC14:00
*** fxpester has quit IRC14:01
nizam037even though for port create some of the options are mandatory in help message those options are place under optional arguments.14:01
nizam037any reasons for this?14:01
nizam037please clarify me14:01
*** athomas has joined #openstack-ironic14:02
dtantsurTheJulia, thanks!14:05
TheJulianizam037: I think it would depend upon the protocol version your using with the client?14:05
TheJuliaoh, back to the kvm nested virt init failures.  I've spotted a second one on OVH. :(14:06
sambetts:(14:06
sambettsTheJulia: whats happening?14:06
TheJuliahttp://logs.openstack.org/73/403773/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial/1ecd1ac/logs/libvirt/qemu/node-0.txt.gz14:07
TheJuliatl;dr we changed our gate recently to try and use kvm paravirt if it appeared available.14:07
jrolldid we change that or was it a global devstack thing?14:07
TheJuliawe changed it for our vm creation14:08
jrollah14:08
TheJuliain our devstack library14:08
TheJuliaerr, plugin14:08
sambettsthere is a devstack flag we can set, if unset it tries to pick the KVM if avaible14:08
sambettselse14:08
sambettsqemu14:08
*** nicodemos has joined #openstack-ironic14:08
*** ccamacho|lunch is now known as ccamacho14:08
nicodemosgood morning, ironic14:10
*** athomas has quit IRC14:10
*** rcernin has quit IRC14:10
*** rcernin has joined #openstack-ironic14:11
*** rloo has joined #openstack-ironic14:12
*** fxpester has joined #openstack-ironic14:12
*** fxpester|2 has quit IRC14:13
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support  https://review.openstack.org/26425714:14
*** fxpester|2 has joined #openstack-ironic14:14
moshelemilan: fixed you comments https://review.openstack.org/#/c/264257/14:15
patchbotpatch 264257 - ironic-inspector - Adding InfiniBand Support14:15
*** fxpester has quit IRC14:17
*** fxpester has joined #openstack-ironic14:17
dtantsurmorning rloo :)14:17
*** lindycoder has joined #openstack-ironic14:18
rloohi dtantsur14:18
rloohello and good morning everyone!14:18
dtantsuranybody else wants to look at https://review.openstack.org/#/c/401311/ ? 2x +2 already14:18
patchbotpatch 401311 - ironic - Move interface validation from API to conductor side14:18
rloois anyone else planning on looking at https://review.openstack.org/#/c/401311/, or should I +A?14:18
patchbotpatch 401311 - ironic - Move interface validation from API to conductor side14:18
rloo^^ yeah, that14:18
dtantsurhaha, I was a bit faster ;)14:18
*** Goneri has joined #openstack-ironic14:18
*** srobert has quit IRC14:19
rloolucasagomes, vsaienk0, vdrok, yuriyz, ^^ ?14:19
lucasagomesrloo, morning, go ahead!14:20
*** fxpester|2 has quit IRC14:20
vsaienk0rloo I don't have anything to add set my vote14:20
rloohi lucasagomes, vsaienk0. thx!14:20
vdrokMorning rloo I was ok with it a couple revisions ago :) right now in a long meeting14:21
*** jrist has joined #openstack-ironic14:21
*** jtomasek has joined #openstack-ironic14:23
rloovdrok: thx14:23
*** athomas has joined #openstack-ironic14:24
rloo+A'd. ON to the next patch...14:24
dtantsurw00t14:24
*** fxpester|2 has joined #openstack-ironic14:25
*** trown is now known as trown|brb14:26
*** fxpester|3 has joined #openstack-ironic14:27
*** fxpester has quit IRC14:28
*** fxpester|2 has quit IRC14:30
*** vikrant has quit IRC14:32
openstackgerritFélix Bouliane proposed openstack/ironic: Add virtualpdu to ironic devstack plugin  https://review.openstack.org/38815414:37
*** ralonsoh_ has joined #openstack-ironic14:37
*** ralonsoh has quit IRC14:37
*** fxpester has joined #openstack-ironic14:38
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic-specs: Support rolling upgrades  https://review.openstack.org/29924514:39
*** ralonsoh_ has quit IRC14:39
*** ralonsoh_ has joined #openstack-ironic14:39
*** fxpester|3 has quit IRC14:41
jrollrloo: that volume target patch took much longer than I expected :P14:41
*** rnoriega has joined #openstack-ironic14:42
rloojroll: sorry, need to lower your expectations next time.14:44
jrollheh14:44
jrollrloo: it's close though14:44
rloojroll: i'll look/revise. Some of that stuff was there from orig author so I left them in. I'm going to have to rebase that anyway, cuz of an imminent patch that is landing...14:45
rloojroll: you can now look at the rolling upgrades spec :)14:46
jrollrloo: no sense in leaving things in just because it's from original author, just to fix it again later :)14:46
jrolloh I am14:46
jrollthanks for making those changes xek! :)14:46
milanmoshele, thx, will look into tomorrow14:46
moshelemilan: cool, thanks14:47
*** srobert has joined #openstack-ironic14:47
*** xhku_ has quit IRC14:49
*** xhku has joined #openstack-ironic14:49
*** fxpester|2 has joined #openstack-ironic14:50
*** jvgrant has quit IRC14:50
*** jvgrant has joined #openstack-ironic14:50
*** fxpester has quit IRC14:52
*** makowals has quit IRC14:53
*** numan_ has joined #openstack-ironic14:57
*** daemontool has joined #openstack-ironic15:00
*** anush has joined #openstack-ironic15:00
*** links has quit IRC15:01
*** devlaps has joined #openstack-ironic15:01
*** baoli has joined #openstack-ironic15:01
vsaienk0jlvillal can you please have look once have a time https://review.openstack.org/#/c/405290/15:03
patchbotpatch 405290 - ironic - Cleanup adding Ironic to cluster on upgrade case15:03
*** makowals has joined #openstack-ironic15:04
*** trown|brb is now known as trown15:06
*** fxpester has joined #openstack-ironic15:06
*** fxpester|2 has quit IRC15:08
*** jaosorior has quit IRC15:09
*** fxpester|2 has joined #openstack-ironic15:09
*** jaosorior has joined #openstack-ironic15:09
*** anush has quit IRC15:10
*** fxpester|2 has quit IRC15:10
*** fxpester has quit IRC15:11
*** absubram has quit IRC15:12
openstackgerritMerged openstack/pyghmi: IpmiServer to allow using IP version 4  https://review.openstack.org/40483715:13
openstackgerritJoanna Taryma proposed openstack/ironic: Fail IPA config if no protocol prefix in ironic api address  https://review.openstack.org/40497515:18
*** stevemar has joined #openstack-ironic15:21
*** rbudden has joined #openstack-ironic15:23
*** ppiela has joined #openstack-ironic15:24
openstackgerritRuby Loo proposed openstack/python-ironicclient: Update release note for fix to required args  https://review.openstack.org/40544515:25
NobodyCamGood Morning Ironic'ers15:25
dtantsurmorning NobodyCam15:25
rloomorning NobodyCam15:25
milanmorning NobodyCam, rloo! :)15:26
NobodyCammorning dtantsur & rloo :)15:26
rloohiya milan. how goes the brno midcycle?15:26
NobodyCammorning milan :)15:26
milanrloo, awesome :D15:26
lucasagomesNobodyCam, morning15:26
NobodyCammorning lucasagomes :)15:27
milanwe sort of getting tired of the beers though :D15:27
*** moshele has quit IRC15:27
rloojroll: what do you think of this documentation. i like it but it means manually updating with each release. https://review.openstack.org/#/c/404211/15:27
patchbotpatch 404211 - ironic - Add release names & numbers to API version history15:27
*** rajinir has joined #openstack-ironic15:27
TheJuliamilan: ciders or wines instead?15:27
dtantsuractually we had vodka with whiskey yesterday15:27
milanTheJulia, I'm not sure I want to answer that question :D but I cannot get enough coffee today ;)15:28
rloojroll: would you be ok with that, we can update the release mgr's notes about that.15:28
NobodyCamoh wow15:28
milanspeaking of which /me goes to make another one < mgould ? ;)15:28
rloomilan: sounds like the most successful midcycle ever :D15:28
milanrloo, +1000 :D15:28
*** stevemar_znc has joined #openstack-ironic15:29
*** Nisha_Agarwal has joined #openstack-ironic15:31
*** vikrant has joined #openstack-ironic15:31
mariojvJayF: what do you think about the updates to anup's RFE? https://bugs.launchpad.net/ironic/+bug/164271915:31
openstackmariojv: Error: Could not gather data from Launchpad for bug #1642719 (https://launchpad.net/bugs/1642719). The error has been logged15:31
mariojvalso, good morning15:31
*** rpioso has joined #openstack-ironic15:31
openstackgerritMerged openstack/ironic: Move interface validation from API to conductor side  https://review.openstack.org/40131115:32
dtantsur\o/15:32
NobodyCamcongrats15:33
*** mtanino has joined #openstack-ironic15:33
mariojvJayF: i'm not 100% certain, but i think the only other plan for ceph is to store inspection data in there at some point, which should probably be a separate RFE15:34
mariojvthe raw data, at least15:34
*** anush has joined #openstack-ironic15:36
TheJuliadtantsur: congrats, it begins! :)15:36
dtantsuryep :)15:38
*** jheroux has joined #openstack-ironic15:40
*** soliosg has joined #openstack-ironic15:43
*** rbartal_ has quit IRC15:43
*** athomas has quit IRC15:46
*** jvgrant has quit IRC15:46
*** dims has quit IRC15:46
*** jvgrant has joined #openstack-ironic15:47
*** links has joined #openstack-ironic15:48
jrollrloo: I'm cool with that, I'll -1 with a note to update those docs :)15:59
*** numan_ has quit IRC15:59
rloojroll: thx!15:59
*** dims has joined #openstack-ironic16:00
*** makowals has quit IRC16:01
*** Syed_ has joined #openstack-ironic16:02
rloojroll: wrt https://review.openstack.org/#/c/285218/, i had 2 questions. will update the patch after clarification from you :)16:02
patchbotpatch 285218 - ironic - Add volume_targets table to database16:02
*** makowals has joined #openstack-ironic16:02
* jroll clicks16:03
*** vgadiraj has quit IRC16:07
jrollrloo: ah you're right16:08
jrollapparently not enough coffee before reviewing that16:08
jrollrloo: just those few small things now, I guess16:09
openstackgerritDmitry Tantsur proposed openstack/ironic: Support defining and loading hardware types  https://review.openstack.org/33662616:09
rloojroll: thx!16:10
*** absubram has joined #openstack-ironic16:12
openstackgerritDmitry Tantsur proposed openstack/ironic: Support names for {cleaning,provisioning}_network  https://review.openstack.org/35798316:12
dtantsurwould appreciate reviews on ^^^ as well16:12
openstackgerritMerged openstack/ironic: Make all IronicExceptions RPC-serializable  https://review.openstack.org/39635516:12
*** stevemar_znc is now known as topol_16:12
*** rcernin has quit IRC16:13
*** topol_ is now known as topol16:20
rloodtantsur: what do you think of this rfe: https://bugs.launchpad.net/ironic/+bug/1645949.16:23
openstackLaunchpad bug 1645949 in Ironic "[RFE] Add iRMC drivers supporting the new socat console" [Undecided,In progress] - Assigned to Dao Cong Tien (tiendc)16:23
dtantsurrloo, fully agree with your comment16:23
rloodtantsur: thx, could you comment. just didn't want it to be just me.16:24
openstackgerritRuby Loo proposed openstack/ironic: Add volume_targets table to database  https://review.openstack.org/28521816:24
sambettsis devstack borked?16:24
sambettsI'm seeing version conflicts with keystone16:25
rloosambetts: hope not. some patches 'just' merged. what are you seeing (or not seeing)16:25
rloosambetts: just now?16:25
sambettsyeah, just did a reclone of everything, and I'm getting this error 2016-12-01 16:23:24.224 | ContextualVersionConflict: (passlib 1.6.5 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('passlib>=1.7.0'), set(['keystone']))16:26
jlvillalvsaienk0: I'll take a look at that patch today. Thanks.16:27
dtantsursambetts, was this locally pretty often16:27
dtantsursambetts, I think I just pip-installed required versions manually16:27
dtantsurs/was/saw/ of course16:28
jrollrloo: +2, thanks for the update16:28
rloothx jroll16:28
sambettsdtantsur: I have PIP_UPGRADE=true set in my local.conf which should *in theory* override any versions I have installed16:28
* dtantsur dunno16:29
dtantsurrloo, nullable=True for UUID field looks suspicious to me16:29
dtantsurditto for volume_id16:30
*** jaosorior has quit IRC16:30
sambettsdtantsur: this seems to be an issue with keystone conflicting with upper constraints16:31
sambettsdtantsur: eh who knows /me unpip installs it16:32
dtantsurrloo, huh, uuid seems nullable for all our models16:32
rloodtantsur: chassis, portgroups.. yeah, was just about to say that16:33
rloodtantsur: except for node.16:33
dtantsurok then.. just looks strange to me16:33
TheJuliadtantsur: w/r/t volume_id, the consumer of the data may not always be cinder and cinder's volume id, but at the same time if someone adds a volume without some sort of tracking ID, at least manually, they may be heading into into territory with warning signs for dragons.16:33
dtantsuryep16:33
rloodtantsur: i agree with you that it looks odd. it could very well be wrong and we're copy/pasting. dunno. we do handle it in code though, that it cannot be null.16:34
dtantsurrloo, ok, +A on this one16:35
rloothx dtantsur! :D16:35
*** e0ne has quit IRC16:37
*** mjura has quit IRC16:38
*** vgadiraj has joined #openstack-ironic16:40
*** aNuposic has joined #openstack-ironic16:42
*** strigazi is now known as strigazi_AFK16:44
aarefievdtantsur: thanks, left some comments16:44
rloodtantsur: could I get your +A again, looks like I rebased it: https://review.openstack.org/#/c/285218/16:46
patchbotpatch 285218 - ironic - Add volume_targets table to database16:46
*** aarefiev is now known as aarefiev_afk16:46
dtantsurrloo, done16:47
rpiosovdrok, I see that https://review.openstack.org/#/c/396355/12 landed. Congratulations!16:48
patchbotpatch 396355 - ironic - Make all IronicExceptions RPC-serializable (MERGED)16:48
rloodtantsur: thx. i was thinking about the db 'nullable=true'. if we want to fix/change that later, would it be a problem?16:48
rpiosovdrok, Question, ... I need that backported to stable/newton.16:48
rpiosovdrok, How is that done?16:49
vdrokhah thanks rpioso :)16:49
rpiosovdrok, You're most welcome.16:49
* jroll finds some lunch16:49
vdrokrpioso: http://docs.openstack.org/project-team-guide/stable-branches.html#proposing-fixes16:50
vdroksome good docs there16:50
*** rcernin has joined #openstack-ironic16:50
dtantsurrloo, it will, but as we have it everywhere... I guess it's fine.16:51
rloodtantsur: am just thinking that we should fix it, at least for future tables, but anyway, probably not worth spending time on now.16:52
dtantsuryep16:52
dtantsurafter all, it's just a database-level sanity check, nothing more16:52
*** links has quit IRC16:53
*** links has joined #openstack-ironic16:53
*** moshele has joined #openstack-ironic16:53
rloodtantsur: right.16:54
*** ohamada has quit IRC17:00
rpiosovdrok, ty for the doc pointer. Given my permissions (or perhaps lack of them :), should I backport it by myself using the Gerrit UI "Cherry Pick To" button?17:01
vdrokrpioso: there should not be any additional permissions required to propose a backport manually, but of course the easiest way is with button. worth trying the btton first, if there are no merge conflicts you won't have to do anything manually17:03
*** igordcard has joined #openstack-ironic17:03
*** nicodemos has quit IRC17:04
rpiosovdrok: Cool. I'll give it a try after lunch. Have a great evening.17:04
vdrokthanks!17:04
*** jpich has quit IRC17:07
*** nicodemos has joined #openstack-ironic17:07
*** rama_y has joined #openstack-ironic17:08
openstackgerritRuby Loo proposed openstack/ironic: Add RPCs to support volume target operations  https://review.openstack.org/28522017:11
* mgould -> away; good night, everyone!17:11
*** mgould is now known as mgould|afk17:11
*** nicodemos has quit IRC17:11
*** nicodemos has joined #openstack-ironic17:12
openstackgerritRichard G. Pioso proposed openstack/ironic: Make all IronicExceptions RPC-serializable  https://review.openstack.org/40553717:12
rpiosovdrok: Done.17:12
vdroknight mgould|afk17:13
jlvillalWould there be any security issue with logging the Ironic API URL in the Ironic logfile. I can't think of one.17:14
NobodyCammorning vdrok :)17:14
vdrokmorning NobodyCam and jlvillal :)17:14
NobodyCam:)17:14
jlvillalHi vdrok NobodyCam rloo jroll dtantsur TheJulia sambetts lucasagomes vsaienk0 and everyone else :)17:15
lucasagomesjlvillal, o/17:15
dtantsurhey jlvillal17:15
NobodyCammorning jlvillal lucasagomes :)17:15
jlvillalMy question on logging the API URL is in relation to this patch: https://review.openstack.org/#/c/404975/17:15
patchbotpatch 404975 - ironic - Fail IPA config if no protocol prefix in ironic ap...17:15
*** milan has quit IRC17:15
*** Nisha_Agarwal has quit IRC17:16
*** Nisha_Agarwal has joined #openstack-ironic17:16
rloomorning jlvillal17:17
*** ricardoas has quit IRC17:17
NobodyCammorning rloo17:18
vdrokjlvillal: yup, seems ok, ironic api url is passed as kernel parameter to ipa anyway I think17:18
rloohi NobodyCam!17:18
NobodyCam:)17:18
jlvillalvdrok: Thanks17:18
*** gabriel-bezerra has quit IRC17:19
*** vsaienko has joined #openstack-ironic17:20
*** derekh has quit IRC17:20
*** ricardoas has joined #openstack-ironic17:21
*** gabriel-bezerra has joined #openstack-ironic17:21
*** vikrant has quit IRC17:25
*** dprince has quit IRC17:27
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job  https://review.openstack.org/38926817:27
jlvillalvdrok: So my project-config change didn't break anything this time?17:28
vdrokjlvillal: which one? :) gates look OK today17:28
jlvillalvdrok: Great. The one to be able to disable the devstack plugin if needed.17:29
jlvillalvdrok: It was this patch: https://review.openstack.org/#/c/404479/17:29
patchbotpatch 404479 - openstack-infra/project-config - Disable devstack plugin for builder 'ironic-grenade' (MERGED)17:29
vdrokjlvillal: yup. tho I think enabling it second time was fixed by vsaienk0 's patch, but yeah, that stuff might be useful too17:31
jlvillalvdrok: The reason I was doing it was for the multi-tenant grenade. I need to update the grenade job to use the builder system like the rest of the jobs.17:32
jlvillalvdrok: I'm just scared I'll break something ;)17:33
vdrok:)17:33
lucasagomesfolks, calling it a day!17:34
lucasagomeshave a great evening all17:34
*** lucasagomes is now known as lucas-afk17:35
nicodemoslucasagomes o/17:35
openstackgerritAnnie Lezil proposed openstack/ironic-inspector-specs: Collect NIC name given by BIOS  https://review.openstack.org/40437217:35
*** penick has joined #openstack-ironic17:37
*** anush has quit IRC17:38
*** nizam037 has quit IRC17:38
*** soliosg has quit IRC17:39
jlvillalvsaienk0: This landed. https://review.openstack.org/383496  You might like it. I remember you having to hack around this issue before.17:40
patchbotpatch 383496 - openstack-infra/nodepool - Have an ending line-feed on the generated id_rsa.p... (MERGED)17:40
*** dtantsur is now known as dtantsur|afk17:40
dtantsur|afko/17:40
jlvillallucas-afk: Ciao17:40
jlvillaldtantsur|afk: Paka-paka17:40
*** soliosg has joined #openstack-ironic17:41
*** dprince has joined #openstack-ironic17:43
*** chlong has joined #openstack-ironic17:45
openstackgerritoctopuszhang proposed openstack/ironic-python-agent: Skip API releated work if no api url configured  https://review.openstack.org/39985717:46
openstackgerritAnnie Lezil proposed openstack/ironic-inspector-specs: Collect NIC name given by BIOS  https://review.openstack.org/40437217:46
*** yufei has quit IRC17:47
*** vgadiraj has quit IRC17:48
*** athomas has joined #openstack-ironic17:59
openstackgerritRuby Loo proposed openstack/ironic: Unneeded testing in DB migration of volume connector  https://review.openstack.org/40555318:03
mariojvrloo: logic behind that patch is just that it's testing the library, right? ^18:04
*** baoli has quit IRC18:05
rloomariojv: right.18:05
*** baoli has joined #openstack-ironic18:05
rloomariojv: unless you think there is value to have it. we don't have it for other tests, so i thought i'd remove it cuz of copy/paste'ism.18:05
mariojvi don't, +1ed it18:06
rloothx mariojv18:06
mariojvthank you18:06
mariojvi <3 reviewing code changes with net decrease in lines of code18:06
*** Romanenko_K has quit IRC18:07
*** ijw has joined #openstack-ironic18:08
rloomariojv: me too. and I love deleting :D18:09
*** ijw has quit IRC18:09
rlooTheJulia: wrt the volume connection stuff, at what point do we want to expose the REST API?18:09
*** ijw has joined #openstack-ironic18:10
TheJuliaI think we would want to expose the rest API right around when we have a working implementation to use the data18:10
rlooTheJulia: ok, so after some of the actual boot-from-volume stuff is ready?18:10
TheJuliayes18:10
rlooso i will start reviewing the boot from volume stuff. on monday i think.18:11
*** ralonsoh_ has quit IRC18:13
openstackgerritMerged openstack/pyghmi: Raise IpmiException on error checking power state  https://review.openstack.org/39862918:14
*** ijw has quit IRC18:15
TheJuliaokay, I realized I completely blanked on doing unit tests for some of it so that is what I'm focused on at the moment, I'll try and get stuff rebased on the updated driver comp stuff before then18:15
*** penick has quit IRC18:18
*** krtaylor has quit IRC18:20
*** penick has joined #openstack-ironic18:24
openstackgerritHugo Nicodemos proposed openstack/ironic: Fix return values in OneView deploy interface  https://review.openstack.org/39669818:26
openstackgerritMerged openstack/ironic-inspector: Clean up retained code for grenade workaround  https://review.openstack.org/40536618:27
*** ChubYann has joined #openstack-ironic18:40
rlooJayF, jroll: how are we handling RFE's for SNMP driver? Do we do 'won't fix' for now, until/if it is undeprecated?18:42
rlooJayF, jroll: wrt https://bugs.launchpad.net/ironic/+bug/164093218:42
openstackLaunchpad bug 1640932 in Ironic "[RFE] SNMP UDP timeout / retries are not configurable" [Wishlist,In progress] - Assigned to Félix Bouliane (fbouliane)18:42
rlooJayF, jroll: which reminds me, we should find out the status of where they are at with undeprecating snmp...18:43
*** ccamacho has quit IRC18:44
openstackgerritFélix Bouliane proposed openstack/ironic: Add virtualpdu to ironic devstack plugin  https://review.openstack.org/38815418:47
*** ijw has joined #openstack-ironic18:50
*** rama_y has quit IRC18:53
*** pester has quit IRC18:54
*** baoli has quit IRC18:54
*** pester has joined #openstack-ironic18:54
*** daemontool has quit IRC18:55
*** soliosg has quit IRC18:58
openstackgerritJoanna Taryma proposed openstack/ironic: Fail IPA config if no protocol prefix in ironic api address  https://review.openstack.org/40497518:58
*** soliosg has joined #openstack-ironic19:01
*** rama_y has joined #openstack-ironic19:01
*** trown is now known as trown|lunch19:04
*** Nisha_Agarwal has quit IRC19:05
openstackgerritChris Krelle proposed openstack/bifrost: Enable inspector by default  https://review.openstack.org/40558019:06
*** penick has quit IRC19:07
sambettsrloo: I don't really know how it happened but my RPC and REST API patch seemed to have swapped files...19:08
rloosambetts: heh.19:08
*** baoli has joined #openstack-ironic19:09
*** penick has joined #openstack-ironic19:09
* sambetts -> home 19:10
sambettsI'll fix it tomorrow I need dinner19:10
*** sambetts is now known as sambetts|afk19:10
openstackgerritMerged openstack/ironic: Add volume_targets table to database  https://review.openstack.org/28521819:10
*** vsaienko has quit IRC19:11
rloosambetts|afk: thx, i'll look at them next week19:12
*** soliosg has quit IRC19:12
*** soliosg has joined #openstack-ironic19:12
openstackgerritChris Krelle proposed openstack/bifrost: Enable inspector by default  https://review.openstack.org/40558019:18
openstackgerritRuby Loo proposed openstack/ironic: Unneeded testing in DB migration of volume connector  https://review.openstack.org/40555319:19
*** soliosg has quit IRC19:21
openstackgerritChris Krelle proposed openstack/bifrost: Enable inspector by default  https://review.openstack.org/40558019:25
*** david-lyle_ has joined #openstack-ironic19:28
openstackgerritAnup Navare proposed openstack/ironic: Allow logical name along with UUID in port creation  https://review.openstack.org/40558619:30
*** david-lyle has quit IRC19:31
*** openstackgerrit has quit IRC19:32
jrollrloo: yeah, I agree with wontfix for now19:33
rloojroll: ok19:33
*** openstackgerrit has joined #openstack-ironic19:33
jrollrloo: they're working on it, I know they have the virtualpdu repo up now, and saw some ironic/devstack stuff earlier this week19:34
rloojroll: great! i suppose we should make sure we get any of their patches for that, reviewed sooner rather than later.19:35
jrollrloo: yeah, they were kinda WIP, hoping they'll bug us when it's ready :)19:35
*** openstackgerrit has quit IRC19:36
rloojroll: heh, it occurred to me. even if i mark the RFE as won't fix, any time someone actually updates a patch related to it, I think the status will go to 'in progress'! At least I added a comment about it :)19:37
jrollha19:38
jrollthen you know which patches to -2 :P19:38
rloojroll: yeah, the patch itself is -2'd. just want to make it easier when looking at the rfe bugs, to know their status.19:38
*** openstackgerrit has joined #openstack-ironic19:39
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job  https://review.openstack.org/38926819:39
*** krtaylor has joined #openstack-ironic19:40
*** fxpester_home has joined #openstack-ironic19:42
*** openstackgerrit has quit IRC19:43
*** pester has quit IRC19:45
*** openstackgerrit has joined #openstack-ironic19:51
openstackgerritRuby Loo proposed openstack/ironic-specs: Support rolling upgrades  https://review.openstack.org/29924519:51
jroll\o/19:52
* jroll +2s that19:52
rloothx jroll! rolling, rolling ...19:53
jroll:D19:53
*** baoli has quit IRC19:54
*** baoli has joined #openstack-ironic20:00
*** david-lyle_ is now known as david-lyle20:01
*** links has quit IRC20:09
jrollwe have to be doing something wrong with our objects with all these nullable=True20:10
jroll:/20:10
rloojroll: yeah. i am wondering if it is too late to change the volume-table & volume-connector ones20:11
jrollrloo: idk, I think it's something odd under the hood of how we create things from the API or something20:12
jrollI think it's fixable for all of them, since we enforce it elsewhere, so it won't break users20:12
rloojroll: oh, you think (royal) we did that on purpose20:12
jrollnot on purpose20:12
jrollrloo: but like "ugh this isn't working just make it nullable and check somewhere else"20:12
rloojroll: ugh. that is on purpose then :)20:13
jrollrloo: heh. I think it was very early days :P20:13
rloojroll: yeah, it works, so move along folks :)20:13
rloojroll: have you been keeping an eye on the tooz/hashrings?20:14
rloojroll: i've been avoiding. wondering if i ought to look at it.20:14
*** dprince has quit IRC20:16
*** trown|lunch is now known as trown20:19
mrdaMorning Ironic20:23
jrollrloo: been watching, tooz patch is still in flight. I tested nova/ironic with it, both work fine :)20:24
*** soliosg has joined #openstack-ironic20:24
rloojroll: good to know. thx.20:25
rloomorning mrda!20:25
mrdaMorning rloo :)20:25
jrollrloo: or at least nova does. I thought ironic did, I may have broken something :P20:25
jrollit's almost a direct copy though, should be fine20:26
jroll\o mrda20:26
rloojroll: oops...20:26
mrdajroll: o/20:27
NobodyCammorning mrda :)20:30
mrdao/20:30
NobodyCam:)20:31
*** ijw_ has joined #openstack-ironic20:37
*** ijw has quit IRC20:41
* jroll steps away, may be back later20:42
*** openstack has joined #openstack-ironic20:44
*** penick has quit IRC21:01
*** soliosg has quit IRC21:01
*** vgadiraj has joined #openstack-ironic21:03
*** rpioso has quit IRC21:05
*** soliosg has joined #openstack-ironic21:05
*** penick has joined #openstack-ironic21:05
*** MattMan has quit IRC21:07
*** MattMan has joined #openstack-ironic21:07
xhkuHi all, I have a verified review which test the PXE_SNMP driver with an experimental gate. Can you please take a look and give some feedback ? https://review.openstack.org/#/c/388154/21:10
patchbotpatch 388154 - ironic - Add virtualpdu to ironic devstack plugin21:10
*** rcernin has quit IRC21:10
*** rpioso has joined #openstack-ironic21:11
*** vsaienko has joined #openstack-ironic21:11
lindycoder^ having this merge we can add the pxe_snmp as an official gate and keep the snmp power drive around for a while :)21:14
*** vsaienko has quit IRC21:17
*** vsaienko has joined #openstack-ironic21:18
*** hamzy has quit IRC21:23
*** e0ne has joined #openstack-ironic21:28
openstackgerritMartin Roy proposed openstack/ironic: Remove deprecation of snmp drivers  https://review.openstack.org/40563921:29
*** e0ne has quit IRC21:37
*** moshele has quit IRC21:37
openstackgerritOksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV  https://review.openstack.org/39712621:39
*** jkilpatr has quit IRC21:41
openstackgerritChris Krelle proposed openstack/bifrost: Enable inspector by default  https://review.openstack.org/40558021:47
*** jtomasek has quit IRC21:48
*** jrist has quit IRC21:49
*** trown is now known as trown|outtypewww21:51
*** baoli has quit IRC21:53
*** jcoufal has quit IRC21:54
*** krtaylor has quit IRC21:56
*** Goneri has quit IRC22:01
*** openstackgerrit has quit IRC22:03
*** aNuposic has quit IRC22:04
*** srart has joined #openstack-ironic22:06
*** cdearborn has quit IRC22:11
*** jheroux has quit IRC22:23
*** openstackgerrit has joined #openstack-ironic22:27
openstackgerritRuby Loo proposed openstack/ironic: Add VolumeTarget object  https://review.openstack.org/28521922:27
*** lindycoder has quit IRC22:35
*** baoli has joined #openstack-ironic22:40
*** baoli has quit IRC22:42
*** krtaylor has joined #openstack-ironic22:42
*** srobert_ has joined #openstack-ironic22:42
*** baoli has joined #openstack-ironic22:42
*** srobert has quit IRC22:46
*** rajinir has quit IRC22:46
*** srobert_ has quit IRC22:46
*** [1]rpioso has joined #openstack-ironic22:49
*** chlong has quit IRC22:50
*** rpioso has quit IRC22:51
*** bnemec has quit IRC22:57
*** athomas has quit IRC22:57
*** ijw_ has quit IRC23:01
*** soliosg has quit IRC23:03
*** soliosg has joined #openstack-ironic23:14
*** jkilpatr has joined #openstack-ironic23:18
*** absubram has quit IRC23:22
*** penick has quit IRC23:25
*** cdearborn has joined #openstack-ironic23:38
*** bfournie has quit IRC23:44
*** penick has joined #openstack-ironic23:45
*** penick has quit IRC23:59

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