Monday, 2017-12-11

*** fragatina has joined #openstack-ironic00:06
*** masber has joined #openstack-ironic00:07
*** jkilpatr has joined #openstack-ironic00:09
*** fragatina has quit IRC00:10
*** fragatina has joined #openstack-ironic00:11
*** fragatina has quit IRC00:12
*** yangyapeng has joined #openstack-ironic00:19
*** yangyapeng has quit IRC00:24
*** exodusftw has quit IRC00:36
*** exodusftw has joined #openstack-ironic00:45
openstackgerritRushil Chugh proposed openstack/ironic master: Add XClarity Driver  https://review.openstack.org/51942700:50
*** linkmark has quit IRC00:59
*** yangyapeng has joined #openstack-ironic01:07
*** phuongnh has joined #openstack-ironic01:08
*** zhangfei has joined #openstack-ironic01:15
*** r-daneel has joined #openstack-ironic01:21
*** liuzz has joined #openstack-ironic01:35
*** adreznec has joined #openstack-ironic01:43
*** asettle_ is now known as asettle01:45
*** Grenth has quit IRC02:07
*** Grenth has joined #openstack-ironic02:12
*** oanson has quit IRC02:19
*** oanson has joined #openstack-ironic02:21
*** kiseok7 has quit IRC02:46
*** namnh has joined #openstack-ironic03:02
openstackgerritzenghui.shi proposed openstack/ironic master: WIP: Add BIOSInterface for BIOS configuration  https://review.openstack.org/50779303:05
openstackgerritzenghui.shi proposed openstack/ironic master: Add db migration and model for node bios table  https://review.openstack.org/51116203:05
openstackgerritzenghui.shi proposed openstack/ironic master: Add db api layer for node bios operations  https://review.openstack.org/51140203:05
openstackgerritzenghui.shi proposed openstack/ironic master: Add node BIOS support - RPC object  https://review.openstack.org/51171403:05
openstackgerritzenghui.shi proposed openstack/ironic master: Add Node BIOS support - REST API  https://review.openstack.org/51257903:05
openstackgerritzenghui.shi proposed openstack/ironic master: Add Node BIOS support - REST API  https://review.openstack.org/51257903:14
*** hw_wutianwei has joined #openstack-ironic03:25
*** baoli has quit IRC03:34
*** qwebirc76456 has joined #openstack-ironic03:42
qwebirc76456join03:54
*** qwebirc76456 has left #openstack-ironic03:59
*** qwebirc57869 has joined #openstack-ironic04:01
*** links has joined #openstack-ironic04:07
*** links has quit IRC04:07
*** qwebirc57869 has quit IRC04:08
*** zhangfei has quit IRC04:16
*** rbudden has joined #openstack-ironic04:22
*** MattMan has quit IRC04:26
*** MattMan has joined #openstack-ironic04:26
*** udesale has joined #openstack-ironic04:27
*** zhangfei has joined #openstack-ironic04:29
*** masuberu has joined #openstack-ironic04:30
*** masber has quit IRC04:33
*** masuberu has quit IRC04:36
*** trungnv has joined #openstack-ironic05:01
*** jaganathan has joined #openstack-ironic05:09
*** fragatina has joined #openstack-ironic05:13
*** dpawar has joined #openstack-ironic05:16
*** rbudden has quit IRC05:16
*** udesale__ has joined #openstack-ironic05:34
*** udesale has quit IRC05:37
*** masber has joined #openstack-ironic05:42
*** fragatin_ has joined #openstack-ironic05:49
*** fragatina has quit IRC05:50
*** marios has joined #openstack-ironic05:54
*** ianychoi_ is now known as ianychoi06:12
*** karthiks has joined #openstack-ironic06:15
*** rbartal has joined #openstack-ironic06:15
*** threestrands has quit IRC06:17
*** hshiina has joined #openstack-ironic06:44
*** udesale has joined #openstack-ironic06:46
*** dpawar has quit IRC06:46
*** udesale__ has quit IRC06:48
*** dpawar has joined #openstack-ironic07:05
*** rbartal has quit IRC07:10
*** zhangfei has quit IRC07:10
*** hoonetorg has quit IRC07:18
openstackgerritzenghui.shi proposed openstack/ironic master: WIP: Add BIOSInterface for BIOS configuration  https://review.openstack.org/50779307:18
openstackgerritzenghui.shi proposed openstack/ironic master: Add db migration and model for node bios table  https://review.openstack.org/51116207:18
openstackgerritzenghui.shi proposed openstack/ironic master: Add db api layer for node bios operations  https://review.openstack.org/51140207:18
openstackgerritzenghui.shi proposed openstack/ironic master: Add node BIOS support - RPC object  https://review.openstack.org/51171407:18
openstackgerritzenghui.shi proposed openstack/ironic master: Add Node BIOS support - REST API  https://review.openstack.org/51257907:18
*** adisky_ has joined #openstack-ironic07:20
*** zhangfei has joined #openstack-ironic07:22
*** vsaienk0 has joined #openstack-ironic07:25
*** namnh has quit IRC07:29
*** namnh_ has joined #openstack-ironic07:29
*** hoonetorg has joined #openstack-ironic07:31
*** makowals has joined #openstack-ironic07:37
*** rcernin has quit IRC07:43
*** AlexeyAbashkin has joined #openstack-ironic07:46
*** dpawar has quit IRC07:58
openstackgerritMerged openstack/ironic-python-agent master: Update version of flake8-import-order package  https://review.openstack.org/52679308:02
*** dpawar has joined #openstack-ironic08:02
*** marios has quit IRC08:05
*** arnewiebalck_ has quit IRC08:12
*** phuongnh has quit IRC08:13
*** arnewiebalck has joined #openstack-ironic08:15
*** zhangfei has quit IRC08:24
*** zhangfei has joined #openstack-ironic08:36
*** stendulker has joined #openstack-ironic08:46
*** rcernin has joined #openstack-ironic08:47
*** charcol has joined #openstack-ironic08:55
*** hshiina has quit IRC08:57
*** marios has joined #openstack-ironic09:01
*** e0ne has joined #openstack-ironic09:02
*** mgoddard has joined #openstack-ironic09:08
openstackgerritparesh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO  https://review.openstack.org/46828809:16
*** xnox_ has quit IRC09:18
*** lucy has joined #openstack-ironic09:21
*** milan_ has joined #openstack-ironic09:21
*** vsaienk0 has quit IRC09:22
*** hjensas has joined #openstack-ironic09:22
*** crushil__ has quit IRC09:22
*** vsaienk0 has joined #openstack-ironic09:28
*** ccamacho has quit IRC09:29
*** ccamacho has joined #openstack-ironic09:30
milan_morning Ironic! :)09:31
*** lucas-afk is now known as lucasagomes09:33
pas-hamorning ironic :)09:33
pas-hamorning milan_09:33
milan_morning pas-ha! :)09:33
*** racedo has joined #openstack-ironic09:35
*** amarao has joined #openstack-ironic09:35
etingofmorning everyone! ;)09:38
openstackgerritMerged openstack/ironic-python-agent master: Installl OpenSSH by default  https://review.openstack.org/52552909:41
*** rbartal has joined #openstack-ironic09:42
*** serlex has joined #openstack-ironic09:44
*** serlex has quit IRC09:44
*** derekh has joined #openstack-ironic09:55
*** trungnv has quit IRC09:56
*** mrch has joined #openstack-ironic09:59
pas-hadtantsur|afk: re question on "if I copy deploy.yaml somewhere, modify only it" - it most probably won't wont work as this playbook has some includes and references roles that are in the same folder, so either copy-rename it in place or copy the whole playbooks folder..10:00
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Make discovery use dnsmasq dhcp filter  https://review.openstack.org/52568510:01
pas-hathe problem with roles might be solved by adding the path to the roles to ansible.cfg to search for roles there too, but including playbooks won't work (you might just copy-paste that though)10:01
milan_morning etingof! :)10:07
milan_etingof, I checked the bugs, there were just 2new (both from sambetts_ ) so I didn't ping you just set to confirmed as they made sense to me :) https://etherpad.openstack.org/p/ironic-bug-triage10:08
etingofthank you, milan_!10:09
milan_etingof, np! ;)10:09
*** zhangfei has quit IRC10:28
*** lucy has quit IRC10:36
*** sambetts_ is now known as sambetts10:36
*** lucy has joined #openstack-ironic10:40
*** adisky_ has quit IRC10:40
*** milan_ has quit IRC10:43
openstackgerritMerged openstack/networking-generic-switch stable/ocata: Add -dlm to the tempest job name  https://review.openstack.org/52608110:49
*** namnh_ has quit IRC11:03
*** vsaienk0 has quit IRC11:16
*** udesale has quit IRC11:19
*** AlexeyAbashkin has quit IRC11:25
openstackgerritzenghui.shi proposed openstack/ironic master: WIP: Add BIOSInterface for BIOS configuration  https://review.openstack.org/50779311:25
openstackgerritzenghui.shi proposed openstack/ironic master: Add db migration and model for node bios table  https://review.openstack.org/51116211:25
openstackgerritzenghui.shi proposed openstack/ironic master: Add db api layer for node bios operations  https://review.openstack.org/51140211:25
openstackgerritzenghui.shi proposed openstack/ironic master: Add node BIOS support - RPC object  https://review.openstack.org/51171411:25
openstackgerritzenghui.shi proposed openstack/ironic master: Add Node BIOS support - REST API  https://review.openstack.org/51257911:25
*** jkilpatr has quit IRC11:37
*** ianychoi has quit IRC11:37
*** vsaienk0 has joined #openstack-ironic11:39
*** dtantsur|afk is now known as dtantsur11:43
dtantsurmorning ironic11:43
dtantsurpas-ha: that would be nice to fix eventually, copying the whole playbooks dir is a bit of an overkill in many cases :) anyway, let's make sure it's clearly documented how exactly to customize the playbooks11:44
dtantsurof more boring things, vsaienk0 are you aware of https://bugs.launchpad.net/ironic/+bug/1737395 ?11:44
openstackLaunchpad bug 1737395 in Ironic "n-cpu fails to start in the multinode job: Conflicting resource provider name: <uuid> already exists" [High,Confirmed]11:44
dtantsurvdrok: ditto ^^11:44
*** AlexeyAbashkin has joined #openstack-ironic11:45
vsaienk0dtantsur: I wondered if it is related to hashring and how ironic nodes are spread across computes11:49
dtantsurI suspect it does11:49
dtantsur* is11:49
*** linkmark has joined #openstack-ironic11:49
*** ianychoi has joined #openstack-ironic11:50
pas-hadtantsur: I suspect this might be due to hashring in nova computes is being rebalanced when we stop/restart a compute - a given compute sees the "new" nodes (that were handled by a stopped one) and tries to register them in placement but they are already there as they were registered by the old compute..11:51
pas-habut this is w/o proper investigation yet, just an idea to check11:52
dtantsurit may be the case..11:52
dtantsurI'm not sure where a fix belongs then, and why it did not fail previously11:52
sambettsseems racy11:53
dtantsurit may be, but my patch failed 3 rechecks in a row11:54
sambetts:/11:54
dtantsurI don't see an immediate cause in recent nova commits11:54
dtantsurasking on #openstack-nova (though I guess it's too early)11:54
dtantsurpas-ha: I'm reading http://logs.openstack.org/01/525501/2/check/build-openstack-sphinx-docs/3a538a9/html/admin/drivers/ansible.html#extending-playbooks and it sounds like I can just change deploy.yaml.. also, IIRC the playbooks path cannot yet be overriden via driver_info11:56
dtantsurvsaienk0, vdrok, sambetts, cdent pointed me at https://review.openstack.org/#/c/524263/12:01
patchbotpatch 524263 - nova - Proper error handling by _ensure_resource_provider (MERGED)12:01
dtantsurso maybe we've had this problem forever, but it only started to be an issue recently12:01
sambettsdtantsur: quite likely :/12:01
*** nicodemos has joined #openstack-ironic12:03
sambettsdtantsur: https://github.com/openstack/nova/blob/ced5d0f323d1467c1fc2be602ded6bffb2c234eb/nova/scheduler/client/report.py#L53012:04
sambettsdtantsur: if my understanding of that code is correct it should check the placement API first and then if it doesn't exist create it12:04
dtantsursambetts: now cdent pointed me at https://review.openstack.org/#/c/508555/12:05
patchbotpatch 508555 - nova - Re-use existing ComputeNode on ironic rebalance12:05
dtantsurwithout looking into details, it may be a fix12:05
sambettsdtantsur: in the commit message it says it is12:06
sambettsdtantsur: "Without keeping the same ComputeNode.uuid placement errors out with a 40912:06
sambettsbecause we attempt to create a ResourceProvider that has the same name12:06
sambettsas an existing ResourceProvdier."12:06
dtantsurlet's move to #openstack-nova12:07
nicodemosgood morning.12:10
*** fellypefca has joined #openstack-ironic12:10
*** stendulker has quit IRC12:11
*** jkilpatr has joined #openstack-ironic12:11
vsaienk0dtantsur: we need to try to add multunode ironic job as -nv to nova12:11
dtantsurvsaienk0: would be nice12:12
*** ijw has joined #openstack-ironic12:12
*** ccamacho is now known as ccamacho|lunch12:15
fellypefcaGood morning, Ironic. :)12:18
*** lucasagomes is now known as lucas-hungry12:19
etingofjlvillal hey, just to ACK that I've added myself to the stackalytics as you've advised lately12:21
openstackgerritDmitry Tantsur proposed openstack/ironic master: [DO NOT MERGE] Testing gate fix  https://review.openstack.org/52707512:29
dtantsurvsaienk0, vdrok, sambetts, testing the rebased gate fix ^^^12:29
*** rbartal has quit IRC12:29
dtantsurvsaienk0: can we just replace the existing nova job with the multinode one? will we lose anything?12:30
*** jaypipes has joined #openstack-ironic12:31
*** bfournie has quit IRC12:33
*** bfournie has joined #openstack-ironic12:33
vsaienk0dtantsur: the difference between current nova job and our multinode is the tests we launched, on current job we run *ironic* tests but on multinode we run smoke12:35
vsaienk0so I think we are ok to do it12:35
dtantsurI suspect smoke tests cover us well enough..12:35
dtantsurI'd have another job with BFV, it may provide additional coverage12:36
*** bfournie has quit IRC12:37
*** hjensas has quit IRC12:37
*** hw_wutianwei has quit IRC12:45
*** dtantsur is now known as dtantsur|brb12:50
*** milan_ has joined #openstack-ironic12:51
*** ricardoas has quit IRC12:52
*** makowals has quit IRC12:58
*** jlabarre has joined #openstack-ironic12:59
*** makowals has joined #openstack-ironic12:59
*** lennyb has quit IRC13:00
*** ricardoas has joined #openstack-ironic13:01
*** lennyb has joined #openstack-ironic13:02
*** udesale has joined #openstack-ironic13:03
*** ccamacho|lunch is now known as ccamacho13:05
*** baoli has joined #openstack-ironic13:10
*** bfournie has joined #openstack-ironic13:15
*** serlex has joined #openstack-ironic13:16
*** yangyapeng has quit IRC13:19
*** yangyapeng has joined #openstack-ironic13:20
*** serlex has quit IRC13:20
*** dprince has joined #openstack-ironic13:20
*** serlex has joined #openstack-ironic13:20
*** jcoufal has joined #openstack-ironic13:20
TheJuliaGood morning everyone13:22
*** rcernin has quit IRC13:23
nicodemosmorning, TheJulia.13:24
*** lucas-hungry is now known as lucasagomes13:25
*** yangyapeng has quit IRC13:25
fellypefcaGood morning TheJulia13:25
*** fellypefca has quit IRC13:28
*** nicodemos has quit IRC13:28
*** nicodemos has joined #openstack-ironic13:29
*** hjensas has joined #openstack-ironic13:35
*** trown|outtypewww is now known as trown13:38
milan_morning nicodemos TheJulia and all! :13:38
milan_:)13:38
milan_faster Enter key13:39
*** ianychoi has quit IRC13:40
milan_folks, http://logs.openstack.org/85/525685/6/check/ironic-tempest-dsvm-ironic-inspector/d68ff59/logs/devstacklog.txt.gz#_2017-12-11_10_23_31_997 is puzzling me; I dunno what setting devstack uses but  http://paste.openstack.org/show/628617/ doesn't fail on me the same dvsm does :-/  any ideas?13:42
milan_https://review.openstack.org/#/c/525685/6/devstack/plugin.sh@20213:42
patchbotpatch 525685 - ironic-inspector - Make discovery use dnsmasq dhcp filter13:42
*** baoli has quit IRC13:43
* milan_ tries to make [[ "$driver" == "dnsmasq" ]] a function to avoid having to repeat it all over the place13:43
nicodemoshey, milan_ o/13:44
milan_o/ :)13:44
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: [WIP] Add baremetal neutron agent  https://review.openstack.org/45623513:46
*** dtantsur|brb is now known as dtantsur13:49
dtantsurmorning TheJulia, nicodemos, milan_13:49
milan_morning dtantsur! :)13:49
nicodemos:)13:49
*** makowals has quit IRC13:57
*** yangyapeng has joined #openstack-ironic13:57
*** vsaienk0 has quit IRC13:57
*** hjensas has quit IRC13:58
*** makowals has joined #openstack-ironic13:58
*** fellypefca has joined #openstack-ironic14:00
*** sasha2 has joined #openstack-ironic14:01
*** crushil__ has joined #openstack-ironic14:02
*** tzumainn has joined #openstack-ironic14:05
*** udesale has quit IRC14:05
*** vsaienk0 has joined #openstack-ironic14:06
*** baoli has joined #openstack-ironic14:12
*** weshay|MOD is now known as weshay|sickday14:13
fellypefcaTheJulia Did you had some time do take a look? https://review.openstack.org/#/c/523943/ :)14:16
patchbotpatch 523943 - ironic - Introduce hpOneView and ilorest to OneView14:16
*** rbartal has joined #openstack-ironic14:17
jrollmorning y'all14:17
TheJuliafellypefca: I did not, but I have time this morning! :)14:17
fellypefcaTheJulia Oh, thanks :D14:17
fellypefcaGood morning jroll14:18
milan_morning fellypefca, jroll! :)14:18
* milan_ greetasaservice 14:18
fellypefcaHey milan_ :D14:19
*** mjturek has joined #openstack-ironic14:19
milan_anyone knows why my unittests are mocking me?14:20
milan_I mean anyone besides dtantsur :P14:20
dtantsurmilan_: in Ironic world unit tests are mocking YOU14:21
dtantsurmorning jroll, fellypefca14:21
fellypefcaMorning dtantsur14:21
milan_ha, dtantsur, it's not my equipment then :D14:21
*** yangyapeng has quit IRC14:24
*** gcb has joined #openstack-ironic14:24
*** yangyapeng has joined #openstack-ironic14:25
dtantsuryay, https://review.openstack.org/#/c/508555/ helps the CI14:25
patchbotpatch 508555 - nova - Re-use existing ComputeNode on ironic rebalance14:25
*** makowals has quit IRC14:26
*** rloo has joined #openstack-ironic14:27
*** rbudden has joined #openstack-ironic14:27
*** makowals has joined #openstack-ironic14:29
*** Goneri has joined #openstack-ironic14:31
sambettsdtantsur: \o/14:34
*** crushil__ is now known as crushil14:35
*** hjensas has joined #openstack-ironic14:38
*** superdan is now known as dansmith14:42
*** hjensas has quit IRC14:49
TheJuliafellypefca: done14:51
*** dmellado has quit IRC14:53
*** jaganathan has quit IRC14:54
*** baoli has quit IRC14:57
*** lucy has quit IRC14:58
*** baoli has joined #openstack-ironic15:00
*** baoli has quit IRC15:00
*** baoli has joined #openstack-ironic15:04
*** dmellado has joined #openstack-ironic15:05
*** dmellado has quit IRC15:10
*** dmellado has joined #openstack-ironic15:12
*** dpawar has quit IRC15:13
fellypefcaTheJulia Nice :D Thanks again.15:15
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: [ansible] add defaults to config  https://review.openstack.org/52571015:17
openstackgerritMerged openstack/ironic-lib master: Update version of flake8-import-order package  https://review.openstack.org/52679215:17
*** r-daneel has quit IRC15:18
*** nicodemos has quit IRC15:20
*** racedo has quit IRC15:20
*** nicodemos has joined #openstack-ironic15:21
*** racedo has joined #openstack-ironic15:23
*** hw_wutianwei has joined #openstack-ironic15:27
openstackgerritJohn L. Villalovos proposed openstack/ironic-lib master: Fix issue with double mocking of utils.execute functions  https://review.openstack.org/52432915:28
*** dmellado has quit IRC15:29
openstackgerritRuby Loo proposed openstack/networking-generic-switch stable/ocata: zuul: add irrelevant-files section to job  https://review.openstack.org/52712915:29
openstackgerritJohn L. Villalovos proposed openstack/ironic-lib master: tox: Use the default version of Python 3 for tox tests  https://review.openstack.org/52713015:30
*** dmellado has joined #openstack-ironic15:30
openstackgerritJohn L. Villalovos proposed openstack/ironic master: tox: Use the default version of Python 3 for tox tests  https://review.openstack.org/52713115:30
*** nicodemos has quit IRC15:35
*** nicodemos has joined #openstack-ironic15:37
*** makowals has quit IRC15:39
openstackgerritFellype Cavalcante proposed openstack/ironic master: Migrate oneview hardware type to use python-hpOneView  https://review.openstack.org/52431015:39
*** bnemec has joined #openstack-ironic15:40
*** gcb has quit IRC15:44
*** baoli has quit IRC15:45
*** makowals has joined #openstack-ironic15:45
*** baoli has joined #openstack-ironic15:45
*** yangyapeng has quit IRC15:49
*** baoli has quit IRC15:50
*** amarao has quit IRC15:51
*** dmellado has quit IRC15:52
rlooGood Monday Morning ironickers including jlvillal, fellypefca, pas-ha, sambetts, TheJulia, dtantsur15:54
rloolooking forward to another productive week! :D15:54
pas-hamorning rloo :)15:55
*** dmellado has joined #openstack-ironic15:55
rlooTheJulia: if you or other bifrost folks have a chance, stable/ocata doesn't look happy: https://review.openstack.org/#/c/525300/15:55
patchbotpatch 525300 - bifrost (stable/ocata) - Clean up zuul files15:55
*** r-daneel has joined #openstack-ironic15:56
*** baoli has joined #openstack-ironic15:57
*** dmellado has quit IRC15:58
*** ccamacho is now known as ccamacho|brb16:00
dtantsurmorning rloo, our productive week starts with broken multinode :)16:01
*** dmellado has joined #openstack-ironic16:01
rloodtantsur: ugh, more broken than last week? i guess if it is not random, it'll be easier to fix... :(16:01
rloodtantsur: anything you want me to do or is it all under control? (guess i can look at whiteboard...)16:01
dtantsurrloo: we have a patch, just waiting for nova folks to get to it16:01
rloodtantsur: good! looks like the stable/ocata CIs are working... I hope...16:02
fellypefcaGood morning rloo :)16:08
rloodtantsur: i skimmed the nova/ironic bug etc. seems like it was causing issues before, do we know why it broke multinode "now" ?16:09
rloodtantsur: oh, forget it, i didn't look at the 'investigation' link...16:10
dtantsurrloo: something like a week before nova introduced realy validation for these 409 errors16:10
dtantsurbefore that they were usually ignored16:10
rloodtantsur: ah.16:10
rloodtantsur: makes me wonder if it better to try to validate sooner rather than later :)16:10
dtantsurheh16:12
openstackgerritJohn L. Villalovos proposed openstack/ironic master: tox: Use the default version of Python 3 for tox tests  https://review.openstack.org/52713116:13
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Add docs for ansible deploy interface  https://review.openstack.org/52550116:13
*** baoli has quit IRC16:15
TheJuliarloo: oh... joy :(16:17
openstackgerritJulia Kreger proposed openstack/bifrost master: DNM - Shade test  https://review.openstack.org/50176416:18
rlooTheJulia: i was hoping the fix for the rest of the ironic stable/ocata jobs, would have fixed the bifrost ones but doesn't seem like it. and i figure it'd be faster for a bifrost person to trouble-shoot. Sorry to be the bearer of not great news...16:18
TheJuliaIt is something to do with shade, going ahead and just running master branch to double check things16:18
rlooTheJulia: thx for looking into it16:19
*** hjensas has joined #openstack-ironic16:20
*** hjensas has quit IRC16:20
*** hjensas has joined #openstack-ironic16:20
openstackgerritSam Betts proposed openstack/ironic master: Receive and store agent version on heartbeat  https://review.openstack.org/36486116:26
*** baoli has joined #openstack-ironic16:27
*** dpawar has joined #openstack-ironic16:28
*** baoli has quit IRC16:29
*** dpawar has quit IRC16:29
*** baoli has joined #openstack-ironic16:29
*** rbudden has quit IRC16:32
*** marios has quit IRC16:33
*** dpawar has joined #openstack-ironic16:34
*** hshiina has joined #openstack-ironic16:39
*** hjensas has quit IRC16:40
*** bfournie1 has joined #openstack-ironic16:42
*** bfournie has quit IRC16:42
*** stendulker has joined #openstack-ironic16:43
*** AlexeyAbashkin has quit IRC16:43
*** cleong_ has joined #openstack-ironic16:44
*** cleong_ has quit IRC16:44
*** cleong has joined #openstack-ironic16:45
*** hjensas has joined #openstack-ironic16:45
*** hjensas has joined #openstack-ironic16:45
*** vsaienk0 has quit IRC16:51
*** stendulker has quit IRC16:52
*** penick has joined #openstack-ironic16:53
*** stendulker has joined #openstack-ironic16:54
jrollour meeting takes place here now, yes?16:55
dtantsuryes16:55
dtantsurwas just about to remind about it :)16:56
jrollcool16:56
jroll:D16:56
jrolllook I'm helping already16:56
jlvillalGood morning Ironic :)16:56
jlvillalmilan_, Because you are using 'mock'?  /me just reading the backscroll16:56
milan_jlvillal, hey :D16:57
milan_I guess dtantsur was right, it's cause in Ironic unittests mock YOU :D16:57
jlvillalheh16:57
vdrokgood morning all!16:58
fellypefcaGood morning vdrok and jlvillal16:58
vdroko/16:58
jlvillal\o16:58
*** rbartal has quit IRC16:58
*** baha has joined #openstack-ironic16:59
*** yangyapeng has joined #openstack-ironic16:59
dtantsurso, starting? :)17:00
jlvillal+117:00
etingofo/17:00
dtantsur#startmeeting ironic17:00
openstackMeeting started Mon Dec 11 17:00:18 2017 UTC and is due to finish in 60 minutes.  The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot.17:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:00
crushil\o17:00
*** openstack changes topic to " (Meeting topic: ironic)"17:00
TheJuliao/17:00
openstackThe meeting name has been set to 'ironic'17:00
jroll\o17:00
jlvillalo/17:00
rpiosoo/17:00
hshiinao/17:00
dtantsurhi all (again) :)17:00
fellypefcao/17:00
mjtureko/17:00
nicodemos\o17:00
etingofo/17:00
milan_o/17:00
dtantsur#link https://wiki.openstack.org/wiki/Meetings/Ironic17:00
*** stendulker_ has joined #openstack-ironic17:01
rlooo/17:01
dtantsurokay, looks like enough people remember that our meeting is here now :D17:01
sambettso/17:01
dtantsur#topic Announcements / Reminder17:02
*** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)"17:02
dtantsur#info Tempest plugin migration THIS WEEK Mon - Wed17:02
*** stendulker_ has quit IRC17:02
dtantsuranything else?17:03
rloodtantsur: so the inspector patches are failing in stable/pike? do they need to work first before migration?17:03
*** stendulker_ has joined #openstack-ironic17:03
rlooironic jobs seem to be working17:03
jlvillalGood question. Does inspector use the ironic tempest code?17:03
dtantsurI guess we should work it out in parallel..17:03
dtantsuryes, it does17:03
dtantsurmy main concern is that if we delay the migration again, it will slip to mid-Jan17:03
dtantsurwhich is a bit too late to destabilize the gate17:03
*** yangyapeng has quit IRC17:03
*** stendulker_ has quit IRC17:04
rlooi guess we also have to get the multinode/nova patch fixed too, before migration?17:04
jlvillalOkay. Not sure if we want to discuss now (annoucement section) or later...17:04
rloowe can discuss after meeting17:04
*** stendulker has quit IRC17:04
dtantsurrloo: totally, it has 1x +2 already17:04
dtantsurmaybe for open discussion..17:04
dtantsuranything else to remind/announce/...?17:04
*** dprince has quit IRC17:05
dtantsur#topic Review action items from previous meeting17:05
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)"17:05
dtantsur#link http://eavesdrop.openstack.org/meetings/ironic/2017/ironic.2017-12-04-17.00.html17:06
dtantsurwe only have bug triaging effort by etingof and milan_, how did it go? :)17:06
milan_dtantsur, 2 bus17:06
milan_updated the ether pad17:06
milan_I think both are good to implement17:06
* milan_ hopes nothing missed17:06
*** stendulker has joined #openstack-ironic17:07
dtantsurcool17:07
dtantsur#topic Review subteam status reports17:07
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"17:07
dtantsur#link https://etherpad.openstack.org/p/IronicWhiteBoard line 14817:08
*** penick has quit IRC17:08
rloodtantsur: remind me, are we trying to get the nova changes for version negotiation done, in queens?17:09
dtantsurrloo: ideally, because we bumped the hardcoded version again17:09
*** fragatin_ has quit IRC17:10
rloodtantsur: ok, cuz i think the nova feature freeze is late Jan. is it a nova bug or feature?17:10
*** charcol has quit IRC17:10
*** fragatina has joined #openstack-ironic17:10
TheJuliarloo: it would be a bug realistically17:10
dtantsurI see it more as a bug17:10
*** fragatina has quit IRC17:10
*** fragatina has joined #openstack-ironic17:11
*** stendulker has quit IRC17:11
rloodtantsur: let us know when you might want reviews on whatever then. i don't remember how/when the deadline is for bugs17:11
*** rama_y has joined #openstack-ironic17:11
dtantsurnever? :)17:11
rloomight be good to verify with nova. that it is a bug...17:11
dtantsurthe only problem with this "bug" is that it'll require an ironicclient version bump17:11
rloodtantsur: well, for critical, high. but others, who knows..17:11
jrolldid we file a bug with nova? (or a blueprint?)17:11
dtantsurso it cannot go to a stable branch17:11
dtantsurjroll: not that I'm aware of. TheJulia?17:12
*** lucasagomes is now known as lucas-afk17:12
*** penick has joined #openstack-ironic17:12
jrollthat's the first step, I seriously doubt nova takes that change without something in LP17:12
rloowould be good to make sure ironic & nova are on the same something on this...17:12
TheJuliajroll: it is more along the lines of "Hey ,you guys are shooting yourselves in the foot, please fix this"17:12
rloostill, they will want a bug or something. i suspect.17:13
jrollsure. I stand by what I said :)17:13
TheJuliaand really, the whole idea would have been to do it when we bumped a microversion in our virt driver code17:13
jrolljust needs to be written down / communicated17:13
jrollthis has been contentious before17:13
dtantsurTheJulia: I guess we can fix the configdrive thing that merged this cycle17:13
jroll(i.e. we tried to do this once already)17:13
*** stendulker has joined #openstack-ironic17:13
TheJuliajroll: yeah, things change though17:13
TheJuliadtantsur: That might be a feature then17:14
jrollTheJulia: agree. still 99% sure it needs something in LP, that's all I'm saying17:14
dtantsur"Fixes compatibility with pre-Queens Ironic" ;) <-- see, a bug fix!17:14
rloois there an action item here? :)17:14
jrollwhich will quickly answer the bug/feature question17:14
*** Nisha_Agarwal has joined #openstack-ironic17:14
dtantsurI can take an action of getting a bug and showing it to folks17:14
dtantsur#action dtantsur to create a bug against nova for not hardcoding ironic API version17:15
dtantsursomething like ^^17:15
rlooyay, candidate #1. dtantsur! thx!17:15
TheJuliaI feel like there is actually a bug someplace already, but I don't remember exactly17:15
dtantsurTheJulia: please throw it my direction if you find it17:15
* rajinir joined17:16
TheJuliadtantsur: I'll try to take a look later today, but no promises17:16
dtantsurthnx17:17
rloomgoddard, johnthetubaguy: any updated status related to traits work?17:17
dtantsurjroll likes traits, let's make him finish it :D17:17
mgoddardhi rloo, john is unlikely to be joining, as he has a new human to look after17:17
dtantsur\o/17:18
*** stendulker has quit IRC17:18
jroll>.>17:18
rloomgoddard: yay!17:18
TheJuliajroll: fyi we almost were blocked last cycle on the microversion bump because they wanted to see it more graceful then. We ended up shooting ourselves in the foot with the microversion pin due to grenade testing where they made some major changes, and nova-compute was failing to operate upon restart since ironic had not been updated yet17:18
*** stendulker has joined #openstack-ironic17:18
pas-ha:)17:18
rloomgoddard: are you planning on taking over?17:18
mgoddardI've not done any work on traits and John didn't mention having done any before he left17:18
rloomgoddard: ok17:18
jrolldtantsur: I know you're joking, but not sure I have time to pick up big things yet. would like to pick it up if I can find time but no promises17:19
dtantsurno problem man, I was joking indeed :)17:19
dtantsurI can pick it up, but not in Dec17:19
mgoddardIt's not currently lined up for me but I'm aware it is slipping so I'll talk to stig and see if I can get some time for it17:19
TheJuliahmmmm17:19
Grenthhey17:20
Grenthis inspection_dhcp_wait_timeout still a working kernel param during introspection? :x17:20
jrollTheJulia: gotcha17:20
dtantsurGrenth: it should look differently, but we have a meeting here right now :)17:20
dtantsurplease wait some more time17:20
*** stendulker_ has joined #openstack-ironic17:21
Grenthalright! sorry for bothering...17:21
*** rbudden has joined #openstack-ironic17:21
TheJuliaI keep retyping thoughts on the traits stuff, I'm not sure we have much of a choice other than just getting it done asap17:22
*** stendulker has quit IRC17:22
dtantsuryep, I'd not delay it forever17:22
dtantsurif not Queens, then early Rocky at the latest17:23
rlooTheJulia: i think we'd like the basic node traits in Queen's, right?17:23
dtantsuryes17:23
TheJuliawell, when exactly is nova planning on removing processing of properties?17:23
TheJuliafor scheduling purposes?17:23
dtantsurjaypipes: up for a quick question ^^^?17:24
rlooi don't think they've announced that.17:24
TheJuliaI understood it to be Rocky timeframe17:24
TheJuliabut, I don't think I've heard anything official either17:24
jrollI suspect it's waiting on us17:24
rlooi thought there was a comment in the traits spec, from tubafather, about that not going away until ... i forgot what.17:24
jaypipesTheJulia: yeah, Rocky is when the ComputeCapabilitiesFilter will be replaced with the traits processing in placement API.17:25
dtantsurthanks jaypipes17:25
jaypipesno problemo.17:25
dtantsurso we have a hard deadline of Rocky. I'd prefer the bits to be in place a bit earlier though..17:25
rlooyup.17:25
TheJuliaSo basically we need client api and rest api and storage done by what, week of January 22nd?17:26
rloothe node traits bit should be 'easy', just a matter of coding :)17:26
dtantsurTheJulia: ideally, yes17:26
TheJuliaI think this just became the top priority17:26
TheJuliabecause otherwise, we're not going tbe bale to deliver it17:27
*** stendulker_ has quit IRC17:27
TheJuliato be able, silly browser locking up for a moment17:27
dtantsurI'm all for it, if we find someone with free time to work on it17:27
mgoddardok, I've just got the go ahead for some time on traits17:27
dtantsurnice! mgoddard++17:27
rloomgoddard to the rescue! thx! :)17:27
dtantsurshould we vote on making this an essentialy priority for the cycle?17:27
* dtantsur knows rloo likes voting17:27
*** hjensas has quit IRC17:28
rlooit is definitely high, is it essential?17:28
*** stendulker has joined #openstack-ironic17:28
dtantsurit's already high, but I think TheJulia means it's essential17:28
rloowell, yeah, i suppose it is essential.17:28
TheJuliawithout it, things are likely goign to be a very hard breaking upgrade if someone goes to queens -> rocky17:28
jroll+117:28
rlooif they really are going to change filters in rocky.17:28
rloo(let's bet on whether the nova change happens in rocky... just joking)17:29
rloook, essential...17:29
rloolet's vote cuz i love that17:29
openstackgerritHugo Nicodemos proposed openstack/ironic master: Introduce hpOneView and ilorest to OneView  https://review.openstack.org/52394317:29
*** serlex has quit IRC17:29
dtantsur#startvote Should we raise the priority of the traits for to essential? Yes, No17:29
openstackBegin voting on: Should we raise the priority of the traits for to essential? Valid vote options are Yes, No.17:29
openstackVote using '#vote OPTION'. Only your last vote counts.17:29
dtantsur#vote yes17:29
sambetts#vote yes17:30
jroll#vote yes17:30
TheJulia#vote yes17:30
rloo#vote yes17:30
jrollmgoddard: fwiw I should be able to help with reviews on this, if not code17:30
jrollfeel free to ping17:30
jlvillal#vote yes17:30
vdrok#vote yes17:30
* dtantsur will review as well, me likes API additions :)17:30
mgoddardjroll: fantastic, thanks17:30
dtantsurany more votes?17:31
dtantsurc'mon, somebody vote no just for fun :D17:31
etingof#vote yes17:31
etingof;)17:31
NobodyCam#vote maybe (for dtantsur)17:31
dtantsurokay, okay17:31
openstackNobodyCam: maybe (for dtantsur) is not a valid option. Valid options are Yes, No.17:31
dtantsurlol17:31
TheJulialol17:31
NobodyCam:)17:31
*** stendulker_ has joined #openstack-ironic17:31
Nisha_Agarwal#vote yes17:31
rloo#vote no cuz I'm not convinced nova will actually make the switch in rocky17:31
openstackrloo: no cuz I'm not convinced nova will actually make the switch in rocky is not a valid option. Valid options are Yes, No.17:31
rlooi tried :)17:32
dtantsurthat bot is picky :D17:32
dtantsuranyway17:32
TheJuliaheh17:32
dtantsur#endvote17:32
openstackVoted on "Should we raise the priority of the traits for to essential?" Results are17:32
openstackYes (9): TheJulia, vdrok, rloo, jlvillal, jroll, Nisha_Agarwal, etingof, dtantsur, sambetts17:32
dtantsur#agreed The traits work priority is raised from High to Essential17:32
dtantsurfinished with the statuses now?17:32
rloo+1 finished, that was stressful... :)17:32
dtantsurit should be, it's hard work!17:33
dtantsur#topic Deciding on priorities for the coming week17:33
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"17:33
dtantsurlet's see where we are17:33
dtantsur#link https://etherpad.openstack.org/p/IronicWhiteBoard line 10917:33
rloothe tempest migration. do the rest of us need to do anything or it is in jlvillal and dtantsur ballcourt?17:34
dtantsurrloo: review things when needed17:34
jlvillalrloo, Well the inspector stuff needs to get fixed.17:34
dtantsuresp. since only you, TheJulia and me have core on stable branches17:34
rloook17:34
jlvillalI will work on the new tempest plugin repository when I get the word.17:34
*** stendulker has quit IRC17:35
rlooi don't see any changes to what's there for priorities then17:35
rloobut next week, i hope to see traits patches :)17:35
dtantsurI'm about to block any patches changing the tempest plugin after this meeting17:35
rloodtantsur: thx17:35
dtantsurso, how's the list looking? no new additions, except for the split17:35
*** nicodemos has quit IRC17:35
*** dpawar has quit IRC17:35
TheJuliastepping outside17:36
rloodtantsur: i think it is good17:36
dtantsurwe need a vote each time :D17:37
* dtantsur is kidding, as usual17:37
rlooyay, more votes!17:37
rlooboo17:37
dtantsurmoving on in 3..17:37
dtantsur2..17:37
dtantsur1..17:37
dtantsur#topic Appointing a bug triaging lead for the coming week17:37
*** openstack changes topic to "Appointing a bug triaging lead for the coming week (Meeting topic: ironic)"17:37
dtantsurwho's up?17:37
*** rbartal has joined #openstack-ironic17:37
*** nicodemos has joined #openstack-ironic17:37
*** stendulker has joined #openstack-ironic17:37
* dtantsur wants to appoint jroll :D but he's a kind PTL17:37
milan_crickets? :D17:38
jrolldon't make me quit already, I just started!17:38
dtantsurlol17:38
TheJulialol17:38
*** stendulker has quit IRC17:38
* milan_ can take it again17:38
dtantsurjroll: you're stronger than that, you survived being a PTL for 2 cycles17:38
dtantsurthanks milan_17:38
TheJuliaI can also do it17:38
dtantsur#action milan_ to lead bug triaging again17:38
*** stendulker has joined #openstack-ironic17:38
dtantsurmm17:38
dtantsur#undo17:38
openstackRemoving item from minutes: #action milan_ to lead bug triaging again17:38
dtantsur#action milan_ and TheJulia to lead bug triaging again17:38
milan_\0/ :D17:39
dtantsurmeh, "again"17:39
dtantsurokay, I won't fix it the 2nd time :)17:39
milan_lol17:39
dtantsur#topic Lets discuss a bug: Delete/Creation of ports on active nodes17:39
*** openstack changes topic to "Lets discuss a bug: Delete/Creation of ports on active nodes (Meeting topic: ironic)"17:39
dtantsur#link https://bugs.launchpad.net/ironic/+bug/173637317:39
openstackLaunchpad bug 1736373 in Ironic "succeed to delete port and create port when node is active" [High,In progress] - Assigned to wangzhengwei (wangsansui)17:39
dtantsurTheJulia: the mic is yours17:39
*** stendulker_ has quit IRC17:39
TheJuliaSo I ran across this bug in launchpad ?friday? and I'm thinking we need to discuss it17:39
*** dmellado has quit IRC17:40
TheJuliaspecifically should we be allowing port deletion/creation when a node is in an active state17:40
* TheJulia thoughts we had it blocked long ago, but couldn't remember17:40
dtantsurI read it as deletion is already allowed17:40
dtantsurbut update is not17:40
*** dmellado has joined #openstack-ironic17:40
rlooi think the bug is 'we currently allow create/delete of ports when a node is active; the bug is that we shouldn't17:41
sambetts+1 I agree we shouldn't (unless in maintenance)17:41
vdrokyeah, at least deleting should be forbidden17:41
rlooas an aside, i wonder what they were trying to update17:42
dtantsurgiven that it's a breaking change, it has to be at least microversioned..17:42
sambettsright I think that is the reason we never did it for delete/create17:42
rlooargh, love/hate microversion17:42
sambettsbecause it only applies from a new microversion onwarfd17:42
milan_nova allows that on an instance?17:42
dtantsurmilan_: nova operates on virtual interfaces, it's natural to create/delete them17:43
dtantsurmost hardware still does not allow creating/deleting interfaces on demand, though we're moving there..17:43
sambettsnova allows attach/detach of neutron ports, but this is for ironic phyical devices which means someone physically added a new NIC card17:43
jrollthat makes me wonder if this should depend on the network driver, since some hardware can do that thing, right?17:43
* jroll looks at sam17:43
TheJuliaMy concern was the fact that we would have to guard it with a microversion, and I'm not sure we really want to do this at this point17:43
sambettstrue17:43
* milan_ was just wondering, reconfiguring a switch might not be that hard to imagine17:43
dtantsureven if they can, doesn't it make sense to put a node in maintenance?17:43
*** dprince has joined #openstack-ironic17:43
dtantsurmilan_: you're confusing VIF attach/detach with port creation/deletion17:44
sambettsdtantsur: I have a case where on vif attach I'll want to create a port in Ironic (I think)17:44
milan_dtantsur, yeah maybe O:-)17:44
dtantsursambetts: this is from inside of ironic, right?17:44
dtantsurnot like an operator does it?17:45
sambettsdtantsur: yeah from the inside17:45
sambettsso not through the API17:45
dtantsurthat's different in my opinion. from inside we can make sure it happens in the right way and with the right hardware17:45
sambetts+1 I think that makes sense17:45
rloowhat about portgroups17:45
TheJuliaI would htink the vif would have to be detached, ports updated, or portgroup updated, then the vif reattached17:46
TheJuliain order to do it properly17:46
dtantsurdeleting an active port/port group may make a VIF ID to be lost.. not sure how bad it is17:46
TheJuliadtantsur: they are stored on the portgroup, not on the port17:46
dtantsurI think we at least should not allow deleting ports with a VIF attached17:46
dtantsurTheJulia: on either17:46
sambettsdtantsur: right but we can't block that on old API versions right?17:47
dtantsurdepending on what entity a VIF is attached to17:47
dtantsursambetts: well, it depends on how broken it is17:47
TheJuliadttrue, true for ports as well, I was just referring to only portgroups17:47
dtantsuryep17:47
dtantsurI think we can and should prevent deletion/update of attached ports/portgroups, even without a microversion17:48
sambettsunless in maintenance17:48
dtantsurI'm not sure it's worth bothering with non-attached ones at all17:48
dtantsurwell, yes. in maintenance you're free to screw up yourself :)17:48
*** yangyapeng has joined #openstack-ironic17:49
TheJuliaI can agree with this without microversion because it is kind of an edge case where someone is wanting to manipulate a node directly17:49
*** derekh has quit IRC17:50
TheJulia10 minute warning17:50
dtantsurare we in agreement? do we want a ML thread? a vote? :)17:50
*** penick has quit IRC17:50
sambettsyeah, I've just thought, do we block someone deleting a port/portgroup while a node is in deploying / other transition states? because that is a seriously breaking change because it might end up with different lists of ports at different point suring deployment17:50
dtantsurI hope we do...17:51
TheJuliaI feel like we are in agreement, but now is the time for someone to speak up if we are not17:51
rloomaybe before we vote/decide, someone could go through and just verify when we allow what to ports/portgroups...17:51
TheJuliasambetts: we do not, but interestingly enough the patch also looks at the presence of an instance_uuid17:51
TheJulias/patch/proposed patch/17:51
*** ijw has quit IRC17:51
rloothe instance uuid is only useful if using nova, or does bifrost also use that field?17:52
TheJuliarloo: it is settable via the ansible modules, it is not used, but bifrost's port support is extremely lacking right now, so I really wouldn't worry about it. It is on my todo list to update support for it all17:53
dtantsura standalone deployment does not have to use instance_uuid17:53
*** yangyapeng has quit IRC17:53
TheJulia^^^ that17:53
rlooso it makes me wonder about the submitted patch17:54
rlooit shouldn't be looking for presence of the instance uuid17:54
dtantsuryep17:54
TheJuliaI agree it likely shount, it is an easy way to know if nova is touching the node, but we should be using a list of states17:54
dtantsurand it should be looking at attached VIF IMO17:54
sambettsthey are also specifically looking at state == active and blocking it, instead of what we do in port update which is allow if its in manageable/enroll or maintenance17:54
TheJuliadtantsur: ++17:55
rlooso i think we need to decide/agree on behaviour. then look at how it is coded :)17:55
milan_submitted patch aside, bug it is, no microversion needed, all agreed? ;)17:55
milan_rloo,++17:55
* milan_ being productive :P17:55
rloothx milan_17:55
*** e0ne has quit IRC17:56
TheJuliaI feel like we're kind of agreeing given the riskof a port being deleted while a vif is attached17:56
*** dsneddon has joined #openstack-ironic17:56
rlooif i understand, we are relaxing the restriction on 'no updates to ports and possibly portgroups' to allowing if not attached17:56
dtantsurthis is a separate question :) we probably should, but I'm not sure17:56
rlooand we are restricting deletion of ports/portgroups, allowed only if not attached17:56
rlooor maintenance ^^17:56
rlooand creation of port/portgroups?17:56
TheJuliarloo: more thought required!17:56
sambettswe allow some port updates already, mac address works17:56
dtantsurcan someone please port to openstack-operators/oepnstack-dev?17:57
TheJulia3 Minute warning17:57
dtantsurand let's get back to this topic next time17:57
dtantsurOUCH17:57
rlooso that goes back to it being useful for me to know what we allow for ports & portgroups, and when17:57
dtantsur#info dtantsur is very likely to be out next Monday (and the Monday after that)17:57
dtantsurforgot ^^^17:57
rloodtantsur: only monday?17:57
dtantsurrloo: yep, next week only Monday17:57
dtantsurthen out of x-mas starting with 24th17:58
dtantsurs/of/for/17:58
rlooTheJulia: you avail to chair next Monday?17:58
TheJuliarloo: I am17:58
rlooTheJulia: thx17:58
rlooi expect the dec 25 meeting to be cancelled17:58
rloooh, and jan 1...17:58
TheJuliayup17:58
TheJuliaBoth fall on a monday17:58
jlvillalAgreed17:58
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for neutronclient  https://review.openstack.org/47617017:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Finalize migration to keystoneauth adapters  https://review.openstack.org/47882517:59
rloook, end of year party next monday then :D17:59
TheJuliaheh17:59
dtantsur#info TheJulia will chair the meeting on Dec 18th, the ones on Dec 25th and Jan 1st are cancelled17:59
jlvillalWell likely very little attendance from North America and Europe, I imagine17:59
jlvillalAnd probably South America too ;)17:59
pas-ha+117:59
dtantsuranything else before we wrap up?17:59
milan_rloo, crickets ;)17:59
*** dmellado has quit IRC17:59
TheJulianothing from me17:59
jlvillalAnd Oceania17:59
dtantsur#endmeeting18:00
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic"18:00
openstackMeeting ended Mon Dec 11 18:00:02 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)18:00
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2017/ironic.2017-12-11-17.00.html18:00
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2017/ironic.2017-12-11-17.00.txt18:00
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2017/ironic.2017-12-11-17.00.log.html18:00
dtantsuryay, now we can just continue talking :)18:00
rloomilan_: i was waiting for the hourly buzzer :)18:00
jlvillaldtantsur, milan_ Can you let me know when inspector is working on all the stable branches?18:00
milan_lol :D18:00
TheJuliadoes that talking include lunch appearing? :)18:00
rlooTheJulia: only if you have magic. i have to get up and make it...18:00
dtantsurof course :)18:00
*** baoli has quit IRC18:01
milan_jlvillal, I'd love to but I have to run, dtantsur will :P18:01
TheJuliarloo: oh, speaking of fun things, master works just fine for bifrost, stable/ocata is broken  and it is the same version of shade :(18:01
jlvillalmilan_, Thanks. I wasn't assuming it would be today...18:01
rloomy mind is going... we never decided the next steps (well, person to take next steps) for that port discussion18:02
* milan_ phew18:02
jlvillalI'm hoping it is :)18:02
dtantsurjlvillal: we should not get blocked on it..18:02
TheJuliarloo: I _think_ somewhere along the lines a dependency is getting downgraded :\18:02
rlooTheJulia: :-(18:02
TheJuliaactually, that must be it :\18:02
milan_k, let's see about the tests tomorrow18:02
* milan_ gone18:02
milan_night ya'll! :)18:02
rlooTheJulia: and those failures have nothing to do with ironic, so doesn't block the tempest migration, right?18:02
*** nicodemos has quit IRC18:02
*** milan_ has quit IRC18:03
*** trown is now known as trown|lunch18:03
TheJuliarloo:  yup, ironic's install is downloading keystoneauth1 :(18:04
*** baoli_ has joined #openstack-ironic18:04
TheJuliarloo: they do not, and we don't use tempest w/bifrost18:04
rlooTheJulia: whoa. ok, i think you're on your way to figuring it out18:04
dtantsurjlvillal: finished blocking patches touching tempest code. time for import!18:05
rloojlvillal: ^^ i think that the two blockers for tempest migration is fix for ironic multinode & inspector stable/pike. i don't know how much/far you can get, w/o those being fixed.18:06
*** mgoddard has quit IRC18:06
TheJuliarloo: oh, I know what it is, i might just swap install order around18:06
* sambetts --> home18:08
sambettsNight all18:08
*** sambetts is now known as sambetts|afk18:08
*** rbartal has quit IRC18:08
jlvillaldtantsur, Working on making the plugin repository now...18:09
*** dmellado has joined #openstack-ironic18:09
*** hshiina has quit IRC18:10
rloogood night sambetts|afk18:12
*** d0ugal has quit IRC18:13
*** amarao has joined #openstack-ironic18:15
* jlvillal realizes the transition script is not that fast after taking 7 minutes to run the first state. Luckily it is a one time deal18:16
*** ijw has joined #openstack-ironic18:17
rloojlvillal: unacceptable. you have to speed it up :D18:17
jlvillalrloo, :)  And darn it if this time it failed at the end. Now I have to debug :(18:18
Grenthdtantsur: so, meeting done? ipa-inspection-dhcp-wait-timeout is it still working? does not seem to take effect18:18
dtantsurit should be18:18
*** d0ugal has joined #openstack-ironic18:18
Grenthdtantsur: weird :/ though it might help with my DHCP timeouts coming in faster than the interface is properly brought up18:19
rloojlvillal: :-(18:19
*** ijw has quit IRC18:21
Grenthdtantsur: setting it as a kernel param ipa-inspection-dhcp-wait-timeout=300 should be sufficient, shouldn't it?18:22
*** ijw has joined #openstack-ironic18:22
*** amarao has quit IRC18:24
*** ijw has quit IRC18:27
*** AlexeyAbashkin has joined #openstack-ironic18:28
dtantsurI think so. Sorry, no time right now to dive into code..18:28
*** ijw has joined #openstack-ironic18:30
TheJuliahmmmm18:30
*** stendulker has quit IRC18:31
TheJuliaGrenth: out of curiosity, are the interfaces actually initializing on your inspection ramdisk?18:31
*** ijw has quit IRC18:32
GrenthTheJulia: calling dhclient -d eth0 works fine after a couple secs18:32
*** AlexeyAbashkin has quit IRC18:32
TheJuliaGrenth: but the interfaces are not initializing on their own?18:32
GrenthTheJulia: I believe they do, just DHCP timeout coming in too early18:33
TheJuliaGrenth: IPA giving up and saying that you have no interfaces? or the ramdisk attempting to dhcp independently of IPA?18:36
GrenthTheJulia: IPA being the introspector?18:37
TheJuliaIPA being ironic-python-agent18:37
GrenthTheJulia: RAMDISK's DHCP (after iPXE) before IPA but reporting failures18:38
TheJuliaGrenth: what are you using for the ramdisk?18:38
jlvillaldtantsur, https://github.com/JohnVillalovos/ironic-tempest-plugin18:39
jlvillaldtantsur, You can compare the contents of ironic_tempest_plugin/ directory there against the ironic one. For me they are identical18:40
*** fellypefca has quit IRC18:40
dtantsurjlvillal: w00t! I guess now the funny thing is to somehow get inspector there..18:40
*** baha has quit IRC18:40
jlvillal$ diff -Naur ironic_tempest_plugin/ ../ironic/ironic_tempest_plugin/; echo $?18:40
jlvillal018:40
jlvillaldtantsur, So now we need to ask infra to import that. Do we want to ask now?18:41
dtantsurjlvillal: wait please, what about inspector?18:41
jlvillaldtantsur, Can that be done later?18:41
dtantsurjlvillal: it's a lot of commits again, no?18:41
TheJuliaGrenth: fwiw, the tunable you referenced only causes IPA to try and wait for either an IP address to be on the interface that the ramdisk PXE loaded from, there is a "all interfaces" mode, but if the ramdisk is just not getting DHCP up and running or retrying to get it up on the interfaces upon boot-up, then that is going to be slightly problematic.18:41
jlvillaldtantsur, So is that a new directory in the plugin repo?18:41
dtantsurthat's what I'm thinking about right now...18:42
dtantsurI think pbr does not play really well with several top-level directories, but I may be wrong18:42
dtantsur*ideally* we should merge the directory structure, while keeping the files separate18:42
GrenthTheJulia: using OPNFV-apex... some CentOS distribution18:42
dtantsurI'm not sure it's possible without going insane18:42
jlvillaldtantsur, Yeah, I'm not sure...18:43
TheJuliadtantsur: I believe we are already insane18:43
dtantsurlol, true18:43
dtantsurjlvillal: I wonder if we can repeat the same for inspector starting with commit 0, then merge the two trees18:43
TheJuliayolanda: you around?18:43
jlvillaldtantsur, Maybe. So where is the inspector tempest plugin?18:44
TheJuliaAnyone ever built a centos IPA ramdisk?18:44
dtantsurjlvillal: https://github.com/openstack/ironic-inspector/tree/master/ironic_inspector/test/inspector_tempest_plugin18:44
jlvillaldtantsur, It isn't jumping out at me with a name like '.*tempest.*'18:44
openstackgerritIlya Etingof proposed openstack/ironic master: WIP: added boot mode support to ManagementInterface  https://review.openstack.org/52677318:44
dtantsurTheJulia: tripleo CI? :)18:44
jlvillaldtantsur, I already see file name conflicts.  config.py in both18:45
dtantsurjlvillal: and exception.py. I think they can be merely concatenated on merge18:45
TheJuliadtantsur: I guess that makes too much sense18:45
jlvillaldtantsur, And directories like services and tests...18:46
*** baoli has joined #openstack-ironic18:46
* jlvillal scared...18:46
*** mgoddard has joined #openstack-ironic18:47
*** baoli_ has quit IRC18:47
* jlvillal will be back in a about 15 minutes18:48
GrenthTheJulia: so just about no chance via kernel params? :/ meh18:49
*** yangyapeng has joined #openstack-ironic18:49
dtantsurjlvillal: if you prepare two trees, I can try doing the merge tomorrow morning18:49
TheJuliaGrenth: possibly not if the interfaces are not even getting started before IPA starts, you may want to try setting ipa-inspection-dhcp-wait-timeout=True on the kernel command line as well as setting a timeout to see if that helps, that is if the interfaces are eventually starting up18:49
dtantsur(while assembling yet another IKEA delivery)18:49
TheJuliaGrenth: starting up on their own that is18:49
TheJuliaIAaaS?18:50
TheJuliaIkea Assembly...18:50
GrenthTheJulia: ye, they just enter promicous and are stuck there, thus IPA fails18:50
dtantsurTheJulia: lol18:50
jlvillaldtantsur, What do you mean by two trees?18:51
dtantsurjlvillal: two git trees growing from the same first commit18:51
dtantsurI think it's  the easiest way to do it, but I may be wrong18:52
dtantsurand then merge them with pain and despair18:52
jlvillaldtantsur, Uh, okay. I'll see what I can do. I think I can do that today. I already have the ironic one.18:52
dtantsurcool!18:52
jlvillaldtantsur, Just need to create an inspector one18:52
TheJuliaGrenth: Interresting, without having anything set, are you getting https://github.com/openstack/ironic-python-agent/blob/f153a741e128501b4fe5ac30c763c670693766e5/ironic_python_agent/inspector.py#L154 ?18:52
*** yangyapeng has quit IRC18:54
GrenthTheJulia: gotta check a screencast, logs too fast18:55
*** mrch has quit IRC18:55
openstackgerritJulia Kreger proposed openstack/bifrost stable/ocata: Install shade after ironic  https://review.openstack.org/52721319:00
TheJuliarloo: ^^^ I think that should fix stable/ocata :(19:01
TheJuliathen again, it might implode in the most awesome way19:01
*** e0ne has joined #openstack-ironic19:01
* rloo watches 527213 to see if it implodes. gets some popcorn first...19:02
*** AlexeyAbashkin has joined #openstack-ironic19:03
*** baha has joined #openstack-ironic19:05
*** AlexeyAbashkin has quit IRC19:07
TheJuliaStepping away to go get some lunch19:08
*** ijw has joined #openstack-ironic19:10
*** trown|lunch is now known as trown19:11
*** penick has joined #openstack-ironic19:12
*** Goneri has quit IRC19:16
*** ijw has quit IRC19:17
*** jroll has quit IRC19:18
*** jroll has joined #openstack-ironic19:19
*** Nisha_Agarwal has quit IRC19:21
*** Nisha_ has joined #openstack-ironic19:21
*** dmellado has quit IRC19:23
*** dtantsur is now known as dtantsur|afk19:24
dtantsur|afko.19:24
dtantsur|afko/19:24
dtantsur|afksee you tomorrow19:25
*** dmellado has joined #openstack-ironic19:25
penicko/19:26
*** fragatina has quit IRC19:28
*** mgoddard has quit IRC19:31
*** e0ne has quit IRC19:38
*** e0ne has joined #openstack-ironic19:38
*** e0ne has quit IRC19:40
*** dmellado has quit IRC19:45
*** ijw has joined #openstack-ironic19:50
*** dmellado has joined #openstack-ironic19:51
*** dmellado has quit IRC19:51
*** ijw has quit IRC19:54
*** dmellado has joined #openstack-ironic19:57
*** dmellado has quit IRC19:57
*** e0ne has joined #openstack-ironic19:57
*** e0ne has quit IRC20:02
*** dmellado has joined #openstack-ironic20:03
*** dmellado has quit IRC20:04
*** rbudden has quit IRC20:11
*** dmellado has joined #openstack-ironic20:11
*** dmellado has quit IRC20:11
jlvillalvdrok: Will you be updating the Neutron Event Notification spec this week?20:12
*** dmellado has joined #openstack-ironic20:13
*** dmellado has quit IRC20:14
*** hw_wutianwei has quit IRC20:14
*** fragatina has joined #openstack-ironic20:22
*** fragatina has quit IRC20:23
*** fragatina has joined #openstack-ironic20:23
*** fragatina has quit IRC20:23
*** dmellado has joined #openstack-ironic20:24
*** fragatina has joined #openstack-ironic20:24
*** dmellado has quit IRC20:24
*** e0ne has joined #openstack-ironic20:28
*** Nisha_ has quit IRC20:33
*** dmellado has joined #openstack-ironic20:34
*** penick has quit IRC20:35
*** Goneri has joined #openstack-ironic20:41
mjturekTheJulia: saw your comment on https://review.openstack.org/#/c/511494/ So you think we should add another BFV job using an ubuntu image?20:50
patchbotpatch 511494 - diskimage-builder - Add iscsi-boot element20:50
openstackgerritJulia Kreger proposed openstack/bifrost stable/ocata: Install shade after ironic  https://review.openstack.org/52721320:50
*** dmellado has quit IRC20:52
TheJuliamjturek: yes, maybe experimental for ironic that we could then set as a default job on diskimage-builder20:52
mjturekokay sounds like a plan20:52
*** rbudden has joined #openstack-ironic20:53
*** trown is now known as trown|outtypewww20:58
*** jkilpatr has quit IRC20:59
*** hjensas has joined #openstack-ironic21:00
*** hjensas has quit IRC21:00
*** hjensas has joined #openstack-ironic21:00
*** bfournie1 has quit IRC21:05
*** jcoufal_ has joined #openstack-ironic21:11
openstackgerritMerged openstack/ironic-python-agent master: Rescue extension for CoreOS with DHCP tenant networks  https://review.openstack.org/42352121:14
*** jcoufal has quit IRC21:15
*** jcoufal_ has quit IRC21:16
*** dmellado has joined #openstack-ironic21:17
*** jkilpatr has joined #openstack-ironic21:24
*** dmellado has quit IRC21:28
rlooTheJulia: the voting jobs passed, but lots of non-voting fails. Are you good with this? https://review.openstack.org/#/c/527213/21:30
patchbotpatch 527213 - bifrost (stable/ocata) - Install shade after ironic21:30
TheJuliawheeeee21:31
*** dmellado has joined #openstack-ironic21:31
TheJuliaI'll take a look a little later, reading some other stuff at the moment. :(21:31
openstackgerritMerged openstack/ironic-specs master: Update 'Nova rescue/unrescue in Ironic'  https://review.openstack.org/50289121:39
*** threestrands has joined #openstack-ironic21:41
*** threestrands has quit IRC21:41
*** threestrands has joined #openstack-ironic21:41
*** r-daneel has quit IRC21:43
*** r-daneel has joined #openstack-ironic21:43
jlvillaldtantsur|afk: https://github.com/JohnVillalovos/ironic-tempest-plugin-inspector21:47
jlvillaldtantsur|afk: https://github.com/JohnVillalovos/ironic-tempest-plugin21:49
jlvillaldtantsur|afk: Those are the two repos21:49
*** chihhsin_ has joined #openstack-ironic21:49
*** crushil has quit IRC21:49
*** vsaienk0 has joined #openstack-ironic21:51
*** strigazi_ has joined #openstack-ironic21:51
*** tonyb_ has joined #openstack-ironic21:51
*** sasha2 has quit IRC21:54
*** ddurst- has joined #openstack-ironic21:55
*** bl0m1_ has joined #openstack-ironic21:55
*** EmilienM_ has joined #openstack-ironic21:55
*** aignatov2 has joined #openstack-ironic21:55
*** lhinds|out has joined #openstack-ironic21:55
*** homegrown_ has joined #openstack-ironic21:56
*** lennyb has quit IRC21:56
*** EmilienM has quit IRC21:56
*** tonyb has quit IRC21:56
*** strigazi has quit IRC21:56
*** chihhsin has quit IRC21:56
*** SpamapS has quit IRC21:56
*** lhinds has quit IRC21:56
*** homegrown has quit IRC21:56
*** bl0m1 has quit IRC21:56
*** rybridges has quit IRC21:56
*** ddurst has quit IRC21:56
*** Bico_Fino has quit IRC21:56
*** aignatov has quit IRC21:56
*** aignatov2 is now known as aignatov21:56
*** EmilienM_ is now known as EmilienM21:56
*** EmilienM has quit IRC21:56
*** EmilienM has joined #openstack-ironic21:56
*** Bico_Fino has joined #openstack-ironic21:57
*** cleong has quit IRC21:58
*** dmellado has quit IRC21:58
*** SpamapS has joined #openstack-ironic21:59
*** lennyb has joined #openstack-ironic22:00
*** rybridges has joined #openstack-ironic22:00
*** bl0m1_ has quit IRC22:00
*** vsaienk0 has quit IRC22:00
*** dprince has quit IRC22:01
*** bl0m1 has joined #openstack-ironic22:05
*** dmellado has joined #openstack-ironic22:08
*** yangyapeng has joined #openstack-ironic22:09
*** dmellado has quit IRC22:10
*** dmellado has joined #openstack-ironic22:10
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Catch OSError thrown when hexdump is missing  https://review.openstack.org/52022322:12
*** yangyapeng has quit IRC22:13
*** ijw has joined #openstack-ironic22:17
*** rcernin has joined #openstack-ironic22:19
*** ijw has quit IRC22:21
*** e0ne has quit IRC22:22
*** penick has joined #openstack-ironic22:23
*** baoli has quit IRC22:28
*** baoli has joined #openstack-ironic22:29
*** baha has quit IRC22:30
*** dmellado has quit IRC22:31
*** baoli has quit IRC22:33
*** dmellado has joined #openstack-ironic22:36
*** dmellado has quit IRC22:48
*** ccamacho|brb has quit IRC22:52
*** yangyapeng has joined #openstack-ironic22:58
*** yangyapeng has quit IRC23:03
*** jesusaur has quit IRC23:03
*** threestrands_ has joined #openstack-ironic23:04
*** threestrands_ has quit IRC23:04
*** threestrands_ has joined #openstack-ironic23:04
*** ijw has joined #openstack-ironic23:05
*** dmellado has joined #openstack-ironic23:06
*** threestrands has quit IRC23:06
*** ijw has quit IRC23:09
*** jesusaur has joined #openstack-ironic23:09
*** jappleii__ has joined #openstack-ironic23:20
*** jappleii__ has quit IRC23:21
*** jappleii__ has joined #openstack-ironic23:22
*** dmellado has quit IRC23:22
*** threestrands_ has quit IRC23:22
*** charcol has joined #openstack-ironic23:27
*** charcol has quit IRC23:28
*** dmellado has joined #openstack-ironic23:36
*** penick has quit IRC23:40
*** baoli has joined #openstack-ironic23:46
*** linkmark has quit IRC23:47
*** baoli_ has joined #openstack-ironic23:50
*** baoli has quit IRC23:50
*** yangyapeng has joined #openstack-ironic23:52
*** fdegir has quit IRC23:55
*** johnsom has quit IRC23:56
*** pas-ha has quit IRC23:56
*** yangyapeng has quit IRC23:56
*** TheJulia has quit IRC23:56
*** pas-ha has joined #openstack-ironic23:56
*** melwitt has quit IRC23:57
*** betherly has quit IRC23:57
*** portdirect has quit IRC23:57
*** johnsom has joined #openstack-ironic23:58
*** AlexeyAbashkin has joined #openstack-ironic23:58
*** TheJulia has joined #openstack-ironic23:58
*** absubram has joined #openstack-ironic23:59

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