Wednesday, 2017-11-15

*** yangyapeng has quit IRC00:05
*** yangyapeng has joined #openstack-ironic00:19
*** yangyapeng has quit IRC00:23
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/51910500:23
*** bfournie has joined #openstack-ironic00:27
*** bfournie has quit IRC00:28
*** crushil_ has joined #openstack-ironic00:35
*** yangyapeng has joined #openstack-ironic00:38
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements  https://review.openstack.org/51980200:40
*** qsyqian has joined #openstack-ironic00:45
*** zshi has joined #openstack-ironic00:45
*** yangyapeng has quit IRC00:47
*** chas has joined #openstack-ironic00:49
*** priteau has joined #openstack-ironic00:49
*** zshi has quit IRC00:51
*** chas has quit IRC00:53
*** priteau has quit IRC00:54
*** zhangfei has joined #openstack-ironic00:57
*** phuongnh has joined #openstack-ironic01:03
*** zshi_ has joined #openstack-ironic01:04
*** yangyapeng has joined #openstack-ironic01:05
*** yangyapeng has quit IRC01:06
*** yangyapeng has joined #openstack-ironic01:06
*** rama_y has quit IRC01:07
*** gfhellma_ has quit IRC01:11
*** hw_wutianwei has joined #openstack-ironic01:12
*** phuongnh has quit IRC01:28
*** phuongnh has joined #openstack-ironic01:31
*** skazi has quit IRC01:35
*** crushil_ has quit IRC01:36
*** crushil_ has joined #openstack-ironic01:36
*** absubram has quit IRC01:37
*** chas has joined #openstack-ironic01:43
*** chas has quit IRC01:47
*** links has joined #openstack-ironic02:03
*** skazi has joined #openstack-ironic02:04
*** threestrands has joined #openstack-ironic02:07
*** ptx0 has quit IRC02:09
openstackgerritRuby Loo proposed openstack/python-ironicclient master: osc node power on & off commands  https://review.openstack.org/36444002:17
*** qsyqian_ has joined #openstack-ironic02:18
*** qsyqian_ has quit IRC02:18
*** AlexeyAbashkin has joined #openstack-ironic02:22
*** crushil_ has quit IRC02:23
*** AlexeyAbashkin has quit IRC02:26
*** qsyqian has quit IRC02:29
*** chas has joined #openstack-ironic02:37
*** chas has quit IRC02:42
*** qsyqian has joined #openstack-ironic02:44
openstackgerritRuby Loo proposed openstack/python-ironicclient master: osc node power on & off commands  https://review.openstack.org/36444002:44
*** ptx0 has joined #openstack-ironic02:48
*** priteau has joined #openstack-ironic02:50
*** crushil_ has joined #openstack-ironic02:50
*** Sukhdev has quit IRC02:51
*** priteau has quit IRC02:55
*** rloo has quit IRC02:55
*** bfournie has joined #openstack-ironic02:55
*** qsyqian has quit IRC02:56
-openstackstatus- NOTICE: Due to an unexpected outage with Zuul (1 hour), you'll need to recheck any jobs that were in progress. Sorry for the inconvenience.02:56
*** qsyqian has joined #openstack-ironic03:09
*** fragatina has quit IRC03:10
*** fragatina has joined #openstack-ironic03:13
*** fragatina has quit IRC03:17
*** chas has joined #openstack-ironic03:31
*** skazi has quit IRC03:34
*** rcernin has quit IRC03:34
*** zhangfei has quit IRC03:35
*** chas has quit IRC03:35
*** rcernin has joined #openstack-ironic03:36
*** udesale has joined #openstack-ironic03:41
*** zhangfei has joined #openstack-ironic03:47
*** qsyqian has quit IRC04:01
*** vmud213 has joined #openstack-ironic04:02
*** rama_y has joined #openstack-ironic04:03
*** qsyqian has joined #openstack-ironic04:13
*** rama_y has quit IRC04:16
*** chas has joined #openstack-ironic04:25
*** skazi has joined #openstack-ironic04:26
*** chas has quit IRC04:29
*** sasha2 has quit IRC04:30
*** etingof has quit IRC04:35
*** namnh has joined #openstack-ironic04:41
openstackgerritAnshul Jain proposed openstack/ironic-specs master: OOB RAID configuration for ilo5 based HPE Proliant Servers  https://review.openstack.org/51082704:41
*** Nisha_Agarwal has joined #openstack-ironic04:44
*** priteau has joined #openstack-ironic04:51
*** etingof has joined #openstack-ironic04:52
*** skazi has quit IRC04:54
*** skazi has joined #openstack-ironic04:54
*** priteau has quit IRC04:55
*** vmud213 has quit IRC05:04
Nisha_Agarwalpmannidi, hi05:13
*** jaganathan has joined #openstack-ironic05:13
*** yushiro has joined #openstack-ironic05:19
*** chas has joined #openstack-ironic05:19
*** AlexeyAbashkin has joined #openstack-ironic05:22
*** chas has quit IRC05:23
*** qsyqian has quit IRC05:26
*** AlexeyAbashkin has quit IRC05:26
*** qsyqian has joined #openstack-ironic05:26
*** qsyqian has quit IRC05:27
*** tzumainn has quit IRC05:31
*** skazi has quit IRC05:34
*** skazi has joined #openstack-ironic05:37
*** hshiina has joined #openstack-ironic05:48
*** charcol has quit IRC06:02
*** charcol has joined #openstack-ironic06:12
*** chas has joined #openstack-ironic06:13
*** skazi has quit IRC06:15
*** skazi has joined #openstack-ironic06:16
*** chas has quit IRC06:18
*** makowals has joined #openstack-ironic06:27
*** makowals_ has quit IRC06:28
*** arnewiebalck has quit IRC06:29
*** makowals has quit IRC06:29
*** amarao has joined #openstack-ironic06:29
*** arnewiebalck has joined #openstack-ironic06:29
*** makowals has joined #openstack-ironic06:30
*** namnh has quit IRC06:33
*** threestrands has quit IRC06:34
*** namnh has joined #openstack-ironic06:51
*** TuanLA has joined #openstack-ironic06:55
*** rbartal has joined #openstack-ironic06:58
*** marios has joined #openstack-ironic06:59
*** arnewiebalck_ has joined #openstack-ironic06:59
*** arnewiebalck has quit IRC07:00
*** arnewiebalck_ is now known as arnewiebalck07:00
*** mjura has joined #openstack-ironic07:01
*** charcol has quit IRC07:05
*** chas has joined #openstack-ironic07:07
*** dtantsur|afk is now known as dtantsur07:11
dtantsurmorning ironic07:11
*** chas has quit IRC07:12
*** spectr has joined #openstack-ironic07:13
Nisha_Agarwaldtantsur, morning07:15
Nisha_Agarwaldtantsur, i have patched a workaround fix for https://bugs.launchpad.net/proliantutils/+bug/1725204 but i have no way to test it. we couldnt reproduce the issue inhouse. I dont know if customer would be ok to test the patch to confirm if that works for them07:17
openstackLaunchpad bug 1725204 in proliantutils "iLO sometimes refuses power requests with Enclosure Busy" [Undecided,In progress] - Assigned to Nisha Agarwal (agarwalnisha1980)07:17
*** parlos has joined #openstack-ironic07:18
Nisha_Agarwaldtantsur, pmannidi is it possible to ask customer to test the fix as we cannot release/merge the fix till it is tested?07:19
*** rcernin has quit IRC07:20
*** zhangfei has quit IRC07:21
pas-hamorning ironic :)07:21
pas-hamorning dtantsur , how so early? :)07:22
Nisha_Agarwalpas-ha, morning07:26
*** vmud213 has joined #openstack-ironic07:28
*** hoonetorg has quit IRC07:30
*** priteau has joined #openstack-ironic07:36
*** zhangfei has joined #openstack-ironic07:37
*** spectr has quit IRC07:38
*** marios_ has joined #openstack-ironic07:42
*** chas has joined #openstack-ironic07:42
*** hoonetorg has joined #openstack-ironic07:43
*** marios has quit IRC07:43
*** dpawar has joined #openstack-ironic07:43
pas-hamorning Nisha_Agarwal07:44
*** e0ne has joined #openstack-ironic07:44
*** marios_ is now known as marios07:45
*** ccamacho has joined #openstack-ironic07:48
dtantsurNisha_Agarwal: hi. thank you very much! unfortunately, I don't have any contacts with the customer. Could you please update the bugzilla with your suggestion?07:49
dtantsurpas-ha: early meeting :(07:49
dtantsurmorning07:49
*** marios has quit IRC07:50
*** pcaruana has joined #openstack-ironic07:51
*** mgoddard has joined #openstack-ironic07:53
*** marios has joined #openstack-ironic07:54
Nisha_Agarwaldtantsur, sure.07:56
*** zhangfei has quit IRC07:57
*** mmedvede has quit IRC07:59
*** priteau has quit IRC08:00
*** priteau has joined #openstack-ironic08:01
*** priteau has quit IRC08:06
*** mmedvede has joined #openstack-ironic08:06
*** zhangfei has joined #openstack-ironic08:10
*** priteau has joined #openstack-ironic08:11
*** priteau has quit IRC08:15
*** tesseract has joined #openstack-ironic08:17
*** mgoddard has quit IRC08:33
*** hjensas has joined #openstack-ironic08:33
*** hjensas has quit IRC08:33
*** hjensas has joined #openstack-ironic08:33
*** zhangfei has quit IRC08:34
*** e0ne has quit IRC08:34
*** AlexeyAbashkin has joined #openstack-ironic08:37
openstackgerritDmitry Tantsur proposed openstack/ironic stable/pike: Add no-vendor interface to the idrac hardware types  https://review.openstack.org/52001008:44
*** zhangfei has joined #openstack-ironic08:47
*** priteau has joined #openstack-ironic08:55
*** priteau has quit IRC09:02
*** priteau has joined #openstack-ironic09:05
*** priteau has quit IRC09:10
*** karimb has joined #openstack-ironic09:10
*** hshiina has quit IRC09:11
*** mgoddard has joined #openstack-ironic09:12
mgoddardMorning Ironic. Can anyone tell me from where in the code provisioning ports get deleted when an instance is aborted during wait-callback?09:15
mgoddardI'm using pike. The only call to remove_provisioning_network I can see is in AgentDeployMixin.reboot_and_finish_deploy(), but that shouldn't be called when deployment is aborted09:17
*** lucas-afk is now known as lucasagomes09:17
mgoddardOne possibility I've considered is that the port is deleted as a cleaning network port, since my cleaning network is also my provisioning network. But then I don't have automated cleaning enabled, so I'm not sure how that would happen09:18
dtantsurmgoddard: good question. I'll defer it to vdrok or vsaienk0 (not online?) probably09:19
dtantsurbut I have a feeling we should add removing provisioning ports to https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent.py#L42009:20
*** ralonsoh has joined #openstack-ironic09:23
mgoddarddtantsur: +1. I think what I see is that the provisioning port remains after the node tear down, but something that happens after that cleans it up09:24
dtantsurmgoddard: ugh. could you please file a bug with your findings?09:30
dtantsurcan you catch when exactly it gets cleaned up? maybe something in our logs?09:31
*** serlex has joined #openstack-ironic09:31
*** derekh has joined #openstack-ironic09:34
yolandaTheJulia, no, it normally works for me on centos09:41
*** e0ne has joined #openstack-ironic09:42
*** yushiro has quit IRC09:42
*** jaianshu has joined #openstack-ironic09:43
jaianshuHi09:43
*** Nisha_Agarwal has quit IRC09:45
dtantsurhi jaianshu09:48
jaianshui'm getting the following error upon provisioning http://paste.openstack.org/show/626349/09:49
jaianshuHi dtantsur09:49
jaianshuany ideas..on the error?09:50
dtantsurhuh, I haven't seen it. do you have cleaning enabled?09:51
jaianshuno09:52
mgoddarddtantsur: I'll do some more digging today and will raise a bug with the results09:56
dtantsurmgoddard: thanks!09:57
dtantsurjaianshu: I highly recommend you keep cleaning enabled, it helps avoiding situations like this09:57
dtantsurjaianshu: is it reproducible? did you try rerunning deploy on the same node?09:57
jaianshudtantsur: yes..it was reproducible on same node...i will try the same with some other node and see.09:58
*** strigazi has quit IRC10:05
*** strigazi has joined #openstack-ironic10:06
*** makowals has quit IRC10:08
*** hw_wutianwei has quit IRC10:09
*** milan_ has joined #openstack-ironic10:16
*** phuongnh has quit IRC10:17
milan_morning Ironic! :)10:17
*** namnh has quit IRC10:17
*** TuanLA has quit IRC10:17
milan_#pixiesay -mflexing patch upda...yaaawn... time10:17
PixieBootsᕙʕ⇀ᴥ⇀ʔᕗ: patch upda...yaaawn... time10:17
*** TuanLA has joined #openstack-ironic10:18
*** zhangfei has quit IRC10:18
*** phuongnh has joined #openstack-ironic10:18
*** TuanLA has quit IRC10:19
*** mat128 has joined #openstack-ironic10:20
*** pester has joined #openstack-ironic10:20
*** phuongnh has quit IRC10:21
milan_dtantsur, morning, thanks for the review! https://review.openstack.org/#/c/466448/63/ironic_inspector/conf.py I'm wondering: how about non-mandatory config option (default empty string)? I'm wondering whether we need to require this from the deployer10:23
patchbotpatch 466448 - ironic-inspector - Introducing a dnsmasq PXE filter driver10:23
*** fxpester has quit IRC10:23
dtantsurmilan_: how will it work, the empty string?10:26
milan_dtantsur, it won't "start" the dnsmasq upon init filter10:27
milan_and won't stop dnsmasq upon an exception/shut-down10:27
milan_(reset filter)10:28
milan_what I'm wondering is should this behaviour be required or optional from the deployer perspective10:28
dtantsurmilan_: so, it will assume that dnsmasq is already started? works for me (just carefully document)10:29
*** phuongnh has joined #openstack-ironic10:29
*** crushil_ has quit IRC10:30
milan_dtantsur, ack I'm feeling the same10:30
*** jaganathan has quit IRC10:30
*** zhangfei has joined #openstack-ironic10:31
*** vmud213 has quit IRC10:37
*** vmud213 has joined #openstack-ironic10:38
*** phuongnh has quit IRC10:42
*** etingof has quit IRC10:53
*** etingof has joined #openstack-ironic10:55
*** yangyapeng has quit IRC11:01
*** yangyapeng has joined #openstack-ironic11:01
*** zhangfei has quit IRC11:02
*** yee37929 has quit IRC11:05
*** yee379 has joined #openstack-ironic11:05
*** yangyapeng has quit IRC11:09
mgoddarddtantsur: https://bugs.launchpad.net/ironic/+bug/173241211:14
openstackLaunchpad bug 1732412 in Ironic "Provisioning ports not deleted by ironic following deploy failure" [Undecided,New]11:14
*** mgoddard has quit IRC11:15
openstackgerritDao Cong Tien proposed openstack/python-ironicclient master: Add support for RESCUE and UNRESCUE provision states  https://review.openstack.org/40834111:19
*** yangyapeng has joined #openstack-ironic11:22
*** mgoddard has joined #openstack-ironic11:52
*** fxpester has joined #openstack-ironic11:57
*** vmud213 has quit IRC11:58
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements  https://review.openstack.org/51980212:00
*** pester has quit IRC12:01
*** lucasagomes is now known as lucas-hungry12:06
*** ianychoi has quit IRC12:07
*** racedo has quit IRC12:08
openstackgerritIlya Etingof proposed openstack/ironic-inspector-specs master: Enumerate Inspector errors  https://review.openstack.org/49453412:12
dtantsurmgoddard: thanks for the bug! do you plan to work on a fix or should someone else?12:15
openstackgerritVasyl Saienko proposed openstack/python-ironicclient master: Add session initiation timeout to API calls  https://review.openstack.org/52005612:17
*** stendulker has joined #openstack-ironic12:18
*** priteau has joined #openstack-ironic12:21
mgoddarddtantsur: just putting together a patch :)12:23
mgoddarddtantsur: basically, what you suggested12:23
dtantsurnice, thanks!12:24
stendulkerdtantsur: Hi, one query about default value for rescue interface12:26
dtantsursure12:27
stendulkerFor classic drivers we would always give NoRescue.12:27
stendulkerFor hardware types, Should it be Noop like we do it for Storage and Network?12:28
dtantsurstendulker: it has to be NoRescue too12:28
stendulkerOr it should be NoTRescue as we do for Inspect and Raid. Is there any convention about that?12:28
dtantsurwe use Noop when we don't want it to return an error12:29
dtantsursince rescue is a separate feature, we do want it to return an error, when used12:29
*** karthiks has quit IRC12:29
stendulkerNoRescue would return error and Noop would not12:29
openstackgerritMerged openstack/ironic master: update description for Change Node Power State  https://review.openstack.org/51981612:30
dtantsurstendulker: right, we need an error, I think12:30
openstackgerritMark Goddard proposed openstack/ironic master: Remove provisioning network ports during tear down  https://review.openstack.org/52006412:30
stendulkerBut reason for doing it dufferently for network and storage is that they get used during provisioning?12:30
dtantsurstendulker: yep, they're part of the default flow, so they cannot fail12:31
stendulkerthank you. that makes sense. in a patch i changed rescue default interface to Noop. Will revert it back.12:32
dtantsurack, thanks12:32
*** stendulker has quit IRC12:37
*** stendulker has joined #openstack-ironic12:44
*** karthiks has joined #openstack-ironic12:45
*** MattMan has quit IRC12:53
*** MattMan has joined #openstack-ironic12:53
openstackgerritVasyl Saienko proposed openstack/ironic master: DNR/DNM test patch to check our partition image  https://review.openstack.org/52006612:54
*** karthiks has quit IRC12:57
*** racedo has joined #openstack-ironic13:00
*** jlabarre has joined #openstack-ironic13:01
*** dprince has joined #openstack-ironic13:02
*** ianychoi has joined #openstack-ironic13:05
*** athomas has quit IRC13:07
*** lucas-hungry is now known as lucasagomes13:07
*** absubram has joined #openstack-ironic13:07
*** pester has joined #openstack-ironic13:07
*** zshi_ has quit IRC13:08
*** absubram_ has joined #openstack-ironic13:09
*** fxpester has quit IRC13:10
*** makowals has joined #openstack-ironic13:11
*** pester has quit IRC13:11
*** absubram has quit IRC13:11
*** absubram_ is now known as absubram13:11
*** jaianshu has quit IRC13:12
*** dprince has quit IRC13:12
*** dprince has joined #openstack-ironic13:12
*** karthiks has joined #openstack-ironic13:13
*** trown|outtypewww is now known as trown13:14
*** mat128 has quit IRC13:17
TheJuliaGood morning everyone13:17
openstackgerritIlya Etingof proposed openstack/ironic-inspector-specs master: Enumerate Inspector errors  https://review.openstack.org/49453413:19
*** links has quit IRC13:22
*** links has joined #openstack-ironic13:24
dtantsurmorning TheJulia13:25
*** tzumainn has joined #openstack-ironic13:26
*** stendulker has quit IRC13:28
*** stendulker has joined #openstack-ironic13:29
*** amarao1 has joined #openstack-ironic13:29
milan_morning TheJulia! :)13:31
*** athomas has joined #openstack-ironic13:31
openstackgerritJulia Kreger proposed openstack/bifrost stable/pike: WIP: Correct minor pike branch config issues  https://review.openstack.org/51981813:32
TheJuliaI just love it when logs are not picked up :(13:35
*** SerenaFeng has joined #openstack-ironic13:41
*** etingof has quit IRC13:48
*** hw_wutianwei has joined #openstack-ironic13:51
*** links has quit IRC13:53
*** rloo has joined #openstack-ironic13:54
*** udesale has quit IRC13:56
*** udesale has joined #openstack-ironic13:58
*** chas_ has joined #openstack-ironic14:02
*** sasha2 has joined #openstack-ironic14:02
*** jaypipes has joined #openstack-ironic14:04
*** chas has quit IRC14:05
*** SerenaFeng has quit IRC14:06
*** SerenaFeng has joined #openstack-ironic14:06
TheJuliarloo: Re: https://bugs.launchpad.net/ironic/+bug/1325407 your the last commenter, and I think we should just change it to wontfix.14:08
openstackLaunchpad bug 1325407 in Ironic ""Progress" field from Nova API for servers never updated" [Low,Triaged]14:08
TheJuliaWell, I just left a comment on it as well.14:08
rloomorning TheJulia, looking...14:08
TheJuliaoh, and Good morning!14:08
rloowow, from 2014... doesn't even ring a bell...14:08
dtantsurmorning rloo14:08
rloohi dtantsur14:08
rlooand hi and good morning to everyone else! :)14:08
*** Goneri has joined #openstack-ironic14:09
rlooTheJulia: is it 'won't fix', 'invalid' or 'incomplete'. I still dont' understand what they want14:11
*** fragatina has joined #openstack-ironic14:11
TheJuliaThey want there to be a progress bar14:12
TheJuliathink of it that way14:12
rlooTheJulia: where?14:12
TheJuliain nova14:12
TheJuliafor our virt driver14:12
rlooTheJulia: oh, so it is in the wrong project14:12
TheJuliano, it is actually corret14:12
TheJuliawe own the virt driver14:12
TheJuliait is that there is nothing we can do to really support it14:12
*** dpawar has quit IRC14:13
rlooTheJulia: the virt driver is in nova though. I think you mean that for the virt driver to have this info, it has to come from ironic?14:13
TheJuliaexactly14:13
rlooTheJulia: ok, i thought part (maybe) of this bug is that the prgress field never shows 0 when it is done :)14:14
*** parlos has quit IRC14:14
rlooTheJulia: I've seen others request a deploy status thing, so this is probably/partly a dup.14:14
*** SerenaFeng has quit IRC14:15
*** makowals has quit IRC14:15
TheJuliain that case, yes14:15
rlooTheJulia: it would be nice to do but... we need a diff status, like 'woudl like to have but don't see how'...14:15
TheJuliaIt would require a huge rewrite14:16
TheJuliawith that in mind...14:16
rlooTheJulia: so thinking about it more, i guess it is an rfe at the ironic level, maybe a bug at nova level14:16
*** fragatina has quit IRC14:16
TheJuliathen do we let it sit in our bug tracker forever?14:16
rlooTheJulia: there's a reason i don't volunteer to do bug triaging. i don't know what to do with this.14:17
TheJuliaI think it just clutters it up, and wontfix makes sense because it does not align14:17
* TheJulia ponders executive-ish decision on the bug and moving on 14:17
rlooTheJulia: if that is what we are doing with these kinds of bugs, then yes. Would be good to mention why wontfix though.14:18
*** dpawar has joined #openstack-ironic14:18
rloodtantsur: ^^ do you agree with wontfix?14:18
rlooTheJulia: can just link to this irc conversation, I can do it :)14:18
dtantsuryep14:18
*** krtaylor has joined #openstack-ironic14:19
TheJulia\o/ agreement14:22
rlooTheJulia, dtantsur: done14:22
TheJuliaanyone seen this before: Nov 15 13:49:14 debian ironic-python-agent[274]: 2017-11-15 13:49:14.751 274 ERROR root [-] Command failed: prepare_image, error: [Errno 2] No such file or directory: OSError: [Errno 2] No such file or directory14:22
TheJuliarloo: thanks!14:22
*** cdearborn has joined #openstack-ironic14:22
rlooTheJulia: Yolanda +2'd this, do you want me to +A, or did you want to wait/continue looking into it: https://review.openstack.org/#/c/516444/14:23
patchbotpatch 516444 - bifrost (stable/ocata) - Move legacy bifrost jobs in-tree14:23
dtantsurTheJulia: WUT14:23
dtantsurno, I haven't. looks like some wrong error reporting..14:23
TheJuliarloo: I guess you can land it14:24
* TheJulia goes and digs into the pike ipa branch14:24
dtantsurrloo: I was too lazy to review it, so please do land :)14:24
rlooTheJulia: +A'd14:25
*** dpawar has quit IRC14:25
rlooTheJulia, dtantsur: that bug and Julia's feedback from summit, caused me to think again, that we don't have a great way to keep track of asks & associated feedback/comments.14:26
rlooTheJulia: cuz I was wondering -- how do we keep track of all this great feedback from the community?14:26
TheJuliaerr, its an ironic-lib issue14:26
dtantsurrloo: certainly file RFEs/bugs for everything that looks like them14:27
rlooTheJulia: and cuz I don't really think that discussing (some of?) these issues at our weekly meeting is the best thing, if we are trying to get operators' feedback.14:27
TheJuliarloo: I merely was going to raise that it was posted, that people should go read it14:28
rlooTheJulia, dtantsur: do we need 'champions' for each thing? I can certainly file RFEs/whatever for each 'thing' in Julia's email14:28
TheJuliarloo: there were two items that really required discussions with vendors, and since they will should be present, might as well try and bring it up for discussion14:28
TheJuliaheh, ironic-lib bug14:29
dtantsurrloo: champions == assignees, or?14:29
*** udesale has quit IRC14:29
rloodtantsur: yeah. i mean, that's what i'd like the assignee to do, champion etc, but not many folks have risen to that. i know i shouldn't say that.14:29
dtantsurthen certainly let's bring it to the meeting14:30
TheJuliaerr... ugh14:30
* TheJulia goes and focuses on bug she is hitting14:30
dtantsurbut probably split into topics, otherwise folks will be like "hmm, I'll read it... later"14:30
openstackgerritMerged openstack/sushy master: zuul: clean up job definition  https://review.openstack.org/51971214:32
rloodtantsur: i guess i feel like, here's the feedback that came from the community, and we 'need' to address it. who is 'we'? or we can just leave that feedback in the email, and if any folks want to do something about any of it, they can, otherwise, nothing gets done.14:32
rloodtantsur: but it isn't just that feedback. we get feedback and we have rfes etc.14:33
rloodtantsur: i think i am just grumpy today. like it is a losing battle, so much i'd like to see done, but it looks like we get so little done14:33
dtantsuryeah, and assigning all the same people won't help. we need fresh folks to jump in14:34
rloodtantsur: when i think of all the ptgs/forums/summits/sessions where we talk/discuss things, and then how long it takes for anything to get done :-(14:34
dtantsurright14:35
rloodtantsur: yup, fresh folks... :-(14:35
dtantsur....14:35
TheJuliadtantsur: your wut is now -> https://bugs.launchpad.net/ironic-python-agent/+bug/173247014:35
openstackLaunchpad bug 1732470 in ironic-python-agent "Deployment can fail due to lack of hexdump" [Medium,Triaged]14:35
dtantsurTheJulia: well, it's not really a bug, I guess? do we expect it to work?14:36
TheJuliaeh....we raise an oserror directly up and expose it. We don't get any errors really, just failed deploy for an unknown reason14:36
TheJuliawell, expose it into the ipa log that gets shipped over14:36
*** dpawar has joined #openstack-ironic14:36
* TheJulia guesses dib built debian, at some point, stopped including hexdump14:37
rlooTheJulia: wrt the questions for hw vendors, eg 'Does the BMC your hardware you driver supports, support asserting keys into the TPM module?', we should have a list of vendor contact emails, send them that question, and put their answers *somewhere*. done.14:37
dtantsurI think the next question will be "please expose this feature" :)14:38
dtantsurI wonder at which point we should stop accepting random BMC-related features into ironic...14:38
rlooTheJulia: also thinking, we're we (community) going to start up a weekly meeting for hw vendors -- that's probably where these sorts of things can be discussed.14:38
rloodtantsur: 'random' ? :)14:39
openstackgerritJulia Kreger proposed openstack/bifrost stable/pike: WIP: Correct minor pike branch config issues  https://review.openstack.org/51981814:41
* TheJulia reads14:41
dtantsurrloo: asserting keys into TPM modules - is it within our scope?14:42
rloodtantsur: ironic is going to take over the world, so yes!14:42
TheJuliarloo:  I would prefer during the meeting, ideally yes we should have those details, but it changes periodically and if it lives on a wiki, then it likely won't be up to date.14:43
rloodtantsur: honestly, i dunno. once you/we provide something, it is inevitable that it should do 'the stuff' that folks want, and/or provide a way to seamlessly? hook into other things...14:43
rlooTheJulia: but what will we do with the info that we get at the meeting?14:43
TheJuliadtantsur: It is hardware setup in a sense, although right now if someone uses keys set into a tpm, it has to all be done by hand.14:43
openstackgerritMerged openstack/bifrost stable/ocata: Move legacy bifrost jobs in-tree  https://review.openstack.org/51644414:44
TheJuliarloo: gauge possibility of doing, see if we can push for it to be an feature at some point.14:44
rlooTheJulia: oh, that wasn't the question.14:44
rlooTheJulia: i mean, the question i saw in the meeting agenda14:44
*** rloo has quit IRC14:45
TheJuliayes14:45
*** udesale has joined #openstack-ironic14:45
*** rloo has joined #openstack-ironic14:45
TheJuliahmm14:45
TheJuliawb rloo14:45
rloowhat was that about :-(14:45
TheJuliarloo: if >1 bmc could possibly support something like out of band assertion, then my hope is to explore the possibility of somehow driving support for that forward, which could greatly improve operator quality of life if they are having to muck with the BMCs14:47
TheJuliaerr14:47
TheJulianot muck with the BMCs, but muck with TPM14:48
rlooTheJulia: yes, that makes sense. I just wonder if our meetings are an effective place to get that info/decide.14:48
rlooTheJulia: tell me to shut up. I wonder what folks think about our weekly meetings anyway. ok, i'll shut up now before i say something i shouldn't.14:49
TheJuliaIf there are vendors present yes, if not... we need to somehow get them to be present and involved14:49
*** crushil has joined #openstack-ironic14:49
TheJuliaI guess, I don't want it lost in an email abyss never to be heard from again if we even had up to date contact details for who is actively leading certain vendor drivers.14:50
rlooTheJulia: yup, i agree about email abyss. That's why as a start, we really should have email contacts for each vendor, and send email to them directly.14:51
*** yee379 has quit IRC14:51
TheJuliaexcept if we send it as a direct email, then it is not in the open14:51
rlooTheJulia: but that was to explicitly answer the question you had. if you want a discussion, the email approach isn't ideal.14:51
*** yee379 has joined #openstack-ironic14:51
TheJuliaI expect they will ask why14:52
TheJuliaor for more detail14:52
rlooTheJulia: the summary of that email should be in the open -- i envision it as 'gathered the info from these vendors, here it is', any other folks have comments.14:52
rlooTheJulia: cuz the meeting is at a particular date/time, so that excludes folks that might want to participate14:52
rlooTheJulia: anyway, i need to shut up before i stick my foot in my mouth.14:53
*** trown is now known as trown|brb14:53
*** serlex has left #openstack-ironic14:53
TheJuliadtantsur: w/r/t tpm data/config, I think the centralized amt management tools allows you to assert such data into the tpm.14:54
TheJuliaFine, I'll go ahead and remove the items, I'll directly contact the vendors, and get back to the community.. maybe... if they even reply.14:54
*** mat128 has joined #openstack-ironic14:55
*** etingof has joined #openstack-ironic14:56
TheJuliaGone14:56
* TheJulia checks on zuul before hunting down email addresses14:57
rlooTheJulia: thx, if what i suggested makes sense to you. If you wanted a discussion, it might not.14:57
*** trown|brb is now known as trown14:57
TheJuliaWell, without it being in the open, I expect zero traction or motivation by vendors14:57
rlooTheJulia: also, I don't see the urgency of this stuff, we could do it at midcycle, the hw vendors tend to show up there.14:58
rlooTheJulia: but gathering info before midcycle is useful too14:58
rlooTheJulia: sorry. I don't mean to pick on you/that. I just wonder how we prioritize everything and yet give attention to everything that others feel are importnat. so we got suggestions at the summit, from folks that were there, and we have lots of other suggestions in the form of rfes & specs, that we as a community are not acting 'fast' on.14:59
TheJuliarloo: the bottom line is that the purpose of having an agenda that can be set in advance is so items like those can be proposed, they can be reviewed, they can be resisted and shot down, or encouraged in advance of the meeting.15:00
TheJuliaso I don't feel like your picking on me15:00
TheJuliaYes, I'm irritated and grumpy because $omg_so_many_things, but not because of this.15:00
rlooTheJulia: yes, i think we have the same grumpy bug :(15:01
TheJuliaAnd my apologies if it comes out in my typing.15:01
rlooTheJulia: my apologies too. I'm just not happy with our pace and I don't know what to do about it.15:01
TheJuliaI'm also not happy with our pace15:02
*** etingof has quit IRC15:03
TheJuliaBut often we focus on the well defined, things we understand or like, and then we zoom in, and ignore things on the side. If we don't line up those things on the side moving forward and consider reprioritizing work as new information comes in, then we can't get traction later on15:04
TheJuliaThe lifecycle to get an idea into code in this community is really unhealthy, and largely it is bandwidth. Sacrificing a little bandwidth upfront for later planning is the only way we can fill the pipeline later.15:05
* TheJulia should stop ranting15:05
rlooTheJulia: ++15:05
rlooTheJulia: to some extent, that's what i mean by rfes. some of those aren't well defined, they are 'asks' for things15:06
TheJuliaindeed, and we've also learned we can't work just from asks, but we need to have those internal discussions as well. The more consensus we reach, the faster we do get things done15:08
rloo++. and I have thoughts about the coding/reviewing times too. about how to improve. but another discussion.15:11
*** links has joined #openstack-ironic15:11
*** hw_wutianwei has quit IRC15:12
openstackgerritVasyl Saienko proposed openstack/ironic master: Fix image type for partition-pxe_ipmitool-tinyipa-python3 job  https://review.openstack.org/52010415:13
*** hw_wutianwei has joined #openstack-ironic15:13
TheJulia++++++15:15
TheJuliaPossibly one where we all have whisky15:16
*** etingof has joined #openstack-ironic15:16
rlooTheJulia: guess i'll have to try to make it to dublin then :D15:17
TheJuliaI mean, why wouldn't you ;)15:18
rlooTheJulia: heh, we'll see...15:19
rlooYay, we did an ironic 9.2.0 release!15:21
TheJulia\o/15:22
*** grafoo has joined #openstack-ironic15:23
*** hjensas has quit IRC15:23
grafoohi! has anyone experienced errors from wsme.api saying something about Timed out waiting for a reply to message ID when setting a node to active?15:25
dtantsurrloo: indeed :)15:29
dtantsurTheJulia: whiskey++15:30
rloodtantsur: and the world moves on...15:30
dtantsurgrafoo: I've seen it from time to time.. never get enough patience to investigate15:31
dtantsurs/get/got/15:31
openstackgerritJulia Kreger proposed openstack/bifrost stable/pike: Correct minor pike branch config issues  https://review.openstack.org/51981815:33
*** stendulker has quit IRC15:33
grafoodtantsur: hm. this error is odd. everything worked fine and suddenly those timeouts started.15:33
dtantsurgrafoo: anything weird with rabbitmq?15:34
TheJuliayolanda: If you can take a look at https://review.openstack.org/519818 when you get a chance. On the most recent run before I made the last edit, it was unrelated dib failures from what I could tell off hand, and permissions issues on centos CI nodes. :(15:34
patchbotpatch 519818 - bifrost (stable/pike) - Correct minor pike branch config issues15:34
*** cfriesen has quit IRC15:34
grafoodtantsur: don't think so. but never used rabbitmq that much before.15:35
grafoodtantsur: all of the queues are empty.15:36
openstackgerritJulia Kreger proposed openstack/bifrost master: DNM: Test master branch CI  https://review.openstack.org/52011015:36
*** udesale has quit IRC15:37
TheJuliagrafoo: out of curiosity, what version of ironic-api/ironic-conductor are you using?15:38
TheJuliagrafoo: also, have you restarted the services?15:38
TheJuliaI'm not recommending that you do, just trying to understand15:39
grafooTheJulia: both are at version 9.1.2. it's a pike installation on centos7.15:40
TheJuliagrafoo: package based?15:40
grafooTheJulia: yeah i restarted the services but that didn't change anything.15:40
TheJuliahmmmm15:40
TheJuliaThat really sounds like rabbitmq went on vacation or something15:41
TheJuliaBack in the ?liberty? timeframe, there were some nasty oslo.messaging bugs, but things would clear up after a restart15:42
grafooTheJulia: jap package based (centos.org/centos/7/cloud/)15:42
*** cfriesen has joined #openstack-ironic15:42
TheJuliagrafoo: 40415:42
grafooTheJulia: sry. the full url is  http://mirror.centos.org/centos/7/cloud/x86_64/openstack-pike/15:44
grafooTheJulia: can you give me some pointers on how to investigate if the error is related to rabbitmq?15:45
TheJuliarpioso|afk: cdearborn: Going to send an email to dell folks with some questions to spur discussion in the next day or two, specifically in regards to bmc capabilities and firmware options would you both be good contacts? is there anyone else I should rope in?15:45
grafooTheJulia: already checked the logs and tried to connect/send messages but everything seems to be working fine.15:45
TheJuliagrafoo: what is your ironic-conductor log showing?15:46
cdearbornTheJulia: rpioso and I would be good contact.  Can't think of anyone else to add.15:46
*** cfriesen has quit IRC15:47
TheJuliacdearborn: thanks15:47
*** cfriesen has joined #openstack-ironic15:47
TheJulianicodemos: ricardoas: Regarding firmware and management of bmc capabilities questions, who from your team would be good to loop in?15:48
grafooTheJulia: that https://pastebin.com/X7yDd7Re timeout happend in the cause of setting a node to active.15:49
TheJuliagrafoo: is ironic-conductor running?15:50
grafooTheJulia: yes, the conductor seems to be working fine.15:51
TheJuliabut what is in the conductor log file?15:51
grafooTheJulia: at least it's running :D15:51
TheJuliabecause the conductor should be replying to that, and I'm guessing there is an exception in the logs15:51
TheJuliaor an error15:51
grafooTheJulia: the messages i posted in the pastebin are the last entries. there's nothing more.15:52
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver  https://review.openstack.org/46644815:53
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Allow concurrect updating of dnsmasq configuration  https://review.openstack.org/50443815:53
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack  https://review.openstack.org/46871215:53
TheJuliagrafoo: so, there should be lines that start with ironic-conductor. I would try stopping the service and try starting it manually so it logs to the screen15:54
TheJuliagrafoo: my guess, offhand, is the conductor is misconfigured and failing to actually start15:54
grafooTheJulia: ok. will do. just a sec.15:55
*** stendulker has joined #openstack-ironic15:55
grafooTheJulia: hm. i didn't change anything in the config when the timeouts started happening.15:56
TheJuliathen you may have found some awesome new issue!15:56
NobodyCamGood Morning Ironic'ers, its ☕️ time15:57
TheJuliaHmm.. I do need coffeeeeeeeeee15:58
NobodyCam:)15:58
*** serlex has joined #openstack-ironic15:58
milan_morning grafoo, NobodyCam, rloo, TheJulia and all! :)15:59
TheJuliastendulker: For firmware update/bmc capability discussion based on operator feedback at the summit, who would be good to contact in your org?15:59
NobodyCamGood Morning milan_ :)15:59
rloohi milan_ and NobodyCam15:59
grafooTheJulia: hooray ... <sarcasm>15:59
TheJuliawe <3 sarcasm here15:59
NobodyCamhey hey Morning rloo :)15:59
TheJuliaand cofffee15:59
NobodyCam:)16:00
grafooNobodyCam: morning!16:00
grafoomilan_: morning!16:00
milan_#pixiesay -mdead EINSUFFICIENTCAFFEINE16:00
PixieBootsʕxᴥxʔ: EINSUFFICIENTCAFFEINE16:00
TheJuliaAny OpenBMC folks here?16:00
stendulkerTheJulia: If its iLO driver specific, my team can take it forward16:00
NobodyCamhey grafoo I was just reading the back scroll16:00
*** rbartal has quit IRC16:00
* milan_ .mtg() -> unhandled exception16:01
stendulkerTheJulia: Is there any specific thing that is being asked?16:01
TheJuliastendulker: well, it may possibly involve adding a feature depending on the overall discussion, but if you can send me the email addresses I should email, that would be helpful16:01
* TheJulia is reminded that there are pixieboots photos from .au that need to appear16:02
stendulkerTheJulia: Let me get the alias16:02
TheJuliastendulker: thanks16:02
NobodyCamTheJulia: +++16:02
stendulkerTheJulia: Its ilo_drivers@groups.ext.hpe.com16:02
dtantsurmorning NobodyCam16:03
NobodyCamGood Morning dtantsur :)16:03
stendulkerGood morning NobodyCam :)16:04
NobodyCamGood Morning stendulker :)16:04
jlvillalGood morning Ironic :)16:10
NobodyCamGood morning jlvillal :)16:10
*** sasha2 has quit IRC16:11
jlvillal:)16:11
stendulkerGood morning jlvillal :)16:13
*** mjura has quit IRC16:13
*** sasha2 has joined #openstack-ironic16:17
grafooNobodyCam: hej. did you experience any similar timeouts?16:18
grafooTheJulia: no luck so far...16:18
NobodyCamonly when either rabbit was having issues, or the conductor wouldn't start16:19
*** amarao1 has quit IRC16:20
grafooNobodyCam: how did you trace back the issues to rabbit?16:20
grafooNobodyCam: conductor is starting just fine.16:21
NobodyCamI just check it when I see things like `MessagingTimeout: Timed out waiting for a reply to message ID` really I always check rabbit :p16:22
*** Arkady has joined #openstack-ironic16:28
*** Arkady has quit IRC16:33
NobodyCamTheJulia: were you looking at making the non-voting jobs pass with 519818?16:39
TheJuliaNobodyCam: kind of yeah16:40
NobodyCam:/16:40
TheJuliaNobodyCam: At the same time, some of them are not passing because dib16:40
* TheJulia thinks that is horrible, but not much that can be done16:40
*** hjensas has joined #openstack-ironic16:41
*** amarao has quit IRC16:46
TheJuliagrafoo: is the conductor getting the messages and replying though?  You should be able to see it picking up messages from the the message queue if there are no messages still sitting in rabbitmq16:48
TheJuliaPossibly, if you have a rabbitmq cluster... you cluster could be broken and the services could be talking to different hosts. The only real remedy that I'm aware for that is to reset the confused node and rejoin it to the rabbitmq cluster in those cases, hoping nothing is still present because the reset wipes out anything still on the message queues on that node.16:49
*** makowals has joined #openstack-ironic16:50
*** lucasagomes is now known as lucas-afk16:52
*** dprince has quit IRC16:59
*** links has quit IRC17:00
*** dpawar has quit IRC17:01
*** tesseract has quit IRC17:03
*** absubram has quit IRC17:03
*** milan_ has quit IRC17:06
*** milan has joined #openstack-ironic17:08
*** gfhellma has joined #openstack-ironic17:09
jlvillalHi stendulker :)17:10
stendulkerjlvillal: Hi :)17:10
jlvillalJust replying to your good morning :)17:11
stendulkerjlvillal: ok, np :)17:12
*** d0ugal has quit IRC17:12
TheJuliagood morning jlvillal17:13
*** hw_wutianwei has quit IRC17:13
jlvillalGood morning TheJulia :)17:13
*** masber has quit IRC17:15
*** milan has quit IRC17:17
*** mat128 has quit IRC17:17
*** crushil has quit IRC17:21
*** dpawar has joined #openstack-ironic17:23
*** dpawar has quit IRC17:24
*** dpawar has joined #openstack-ironic17:24
jlvillalvdrok: Question on Neutron event processing.17:27
vdrokgood morning ironic! and morning jlvillal , yup17:28
jlvillalvdrok: https://review.openstack.org/#/c/343684/10/specs/approved/synchronize-events-with-neutron.rst@5817:28
patchbotpatch 343684 - ironic-specs - Add synchronize-events-with-neutron spec17:28
jlvillalvdrok: Hi :)17:28
*** serlex has quit IRC17:28
*** makowals has quit IRC17:28
jlvillalvdrok: I'm confused a little bit. How does Neutron know to send things to Ironic? Is that documented in the spec?17:28
*** marios has quit IRC17:28
* jlvillal reads more...17:28
* vdrok is in GMT-8 for next several months17:28
vdrokjlvillal: it is kind of a neutron thing, it provides an interface for notifiers to implement17:29
*** AlexeyAbashkin has quit IRC17:29
jlvillalvdrok: Are you in California?17:29
vdrokyup17:29
jlvillalvdrok: Cool!17:29
jlvillalvdrok: Are there any pointers to this neutron thing in the document?17:30
vdrokso basically if you implement a method named smth like on_port_update, it will be called when update of port object is done17:30
jlvillalvdrok: GMT-8 = PDT :)17:30
jlvillalvdrok: Okay. So when we call Neutron, there is an argument we can pass that says call this function when you are done?17:31
vdrokjlvillal: yeah, I'll try to find something in neutron docs. I just don't remember anything like that there. would a link to a base class with the docstrings enough?17:31
jlvillalvdrok: Something would be good, IMHO17:31
vdrokjlvillal: no need to pass anything, as long as you inherit from base class and add an entrypoint to neutron.conf17:32
vdrokjlvillal: yeah, I'm not good with american tz naming yet :)17:32
jlvillalvdrok: Okay. I guess I think a few more of those details should be in the spec. Or at least pointers to docs so people can understand.17:32
*** crushil has joined #openstack-ironic17:32
jlvillalvdrok: My impression of a spec is that I can be given the spec and hopefully I would have a chance to be able to implement it.17:33
jlvillalvdrok: Realistically maybe not so much. But kind of a goal :)17:33
vdrokjlvillal: yeah. there is a patch that showing this stuff, in the links section. but I get what you mean. not sure about the docs but I'llk try to find something readable17:33
jlvillalvdrok: Awesome!17:33
openstackgerritMark Goddard proposed openstack/ironic master: Remove provisioning network ports during tear down  https://review.openstack.org/52006417:34
*** gfhellma_ has joined #openstack-ironic17:34
TheJuliavdrok: northern California or southern California?  Asking becuase I'm in the process of buying a place in southern california17:35
vdrokTheJulia: sunnyvale :)17:35
TheJuliaThat is a bit of a drive from where I'm buying  :(17:35
openstackgerritShivanand Tendulker proposed openstack/ironic master: Adds rescue_interface to base driver class  https://review.openstack.org/50933517:37
openstackgerritShivanand Tendulker proposed openstack/ironic master: Adds RPC calls for rescue interface  https://review.openstack.org/50933617:37
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add rescuewait timeout periodic task  https://review.openstack.org/35315617:37
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add rescue related methods to network interface  https://review.openstack.org/50934217:37
openstackgerritShivanand Tendulker proposed openstack/ironic master: Agent driver rescue implementation  https://review.openstack.org/40043717:37
*** gfhellma has quit IRC17:37
rlooTheJulia: are you somewhat 'up to speed' on the traits stuff? I still think we need this: https://review.openstack.org/#/c/412523/17:39
patchbotpatch 412523 - ironic-specs - Ironic deployment steps17:39
rlooTheJulia: but don't want to work on it unless others are convinced17:39
TheJuliaonly if we conjoin them and try to deliver them at once. We first need data storage as defined in the traits spec17:40
rlooTheJulia: you mean, to store the traits info17:40
TheJuliaexactly17:40
TheJuliaThat way nova can schedule, and moving forward off of the traits data, we can execute deployment steps. I'm not up to speed on the current state of deploy steps sadly :(17:41
rlooTheJulia: that deploy stuff isn't needed for the first traits spec. it is needed for doing anything 'configurable' at deploy time, wihc is the 2nd spec. https://review.openstack.org/#/c/504952/17:41
patchbotpatch 504952 - ironic-specs - Config Template traits17:41
rlooTheJulia: or maybe i should wait til that second spec ack's that the deploy steps work is needed...17:41
TheJuliahmmm17:42
rlooTheJulia: cuz that second spec refers to 'clean steps'...17:42
TheJuliaoh!17:42
* TheJulia thought you were talking about them in reverse somehow17:42
* TheJulia was confused for a moment17:42
rlooTheJulia: sorry. also, the first basic 'traits' spec was approved :)17:43
*** dpawar has quit IRC17:44
jlvillalvdrok: left a review on the Neutron spec. Thanks for updating it!17:45
*** racedo has quit IRC17:46
*** dtantsur is now known as dtantsur|afk17:46
dtantsur|afko/17:46
vdrokjlvillal: thanks!17:47
vdrokalso morning TheJulia and rloo and good night dtantsur|afk17:47
rlooah, changing of the guards. bye dtantsur|afk, hi vdrok!17:48
TheJuliarloo: Yeah, I know the first traits spec was approved, the second one seems like it is largely codifying expected use along with how that maps to templates, I just think the second one is kind of overriding deploy_steps concepts, or duplicating the overall concept.17:48
TheJuliaMaybe that would be a good discussion topic for monday if john is around?17:48
*** crushil has quit IRC17:49
rlooTheJulia: yeah, let's discuss with john first. i haven't seen him around lately.17:49
TheJuliarloo: I think the second one should almost be renamed "use of traits for deployment activities"17:49
*** stendulker has quit IRC17:50
rlooTheJulia: well... he is focused on 'templates' :) so yeah, probably worth discussing!17:50
TheJuliaand conceptually, there would need to be a way to define the templates, and that I don't really see/grok from that second traits spec.17:51
TheJuliayeah, double checked, it is kind of hand wavey on templates17:52
rlooTheJulia: yup, i am guessing some API to add templates to the DB, based on what i read.17:52
TheJuliaditto17:52
TheJulialikely not work that will land until sometime next cycle :(17:53
rlooTheJulia: or the one after that :-((17:53
TheJuliarloo: truthfully, I think a mechanics spec, and then a separate spec defining templates or step programming17:54
jlvillalTheJulia: By the way, thanks for the Summit summary. A lot of info for me to get through there :)17:54
rlooTheJulia: yeah. there are a lot of pieces to get it all working17:54
TheJuliathe trait mechanics that are covered in the second traits spec are really needed for us/drivers to maintain parity17:54
rlooTheJulia: Well, when john is around, we can look into another meeting to discuss.17:56
TheJuliaagreed17:59
*** derekh has quit IRC17:59
TheJuliahmm... no mjturek17:59
* TheJulia wonders if vacation18:00
*** fragatina has joined #openstack-ironic18:01
*** fragatina has quit IRC18:01
*** ralonsoh has quit IRC18:01
*** fragatina has joined #openstack-ironic18:02
*** dpawar has joined #openstack-ironic18:08
*** mgoddard has quit IRC18:09
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/51910518:10
*** dprince has joined #openstack-ironic18:11
*** pcaruana has quit IRC18:11
*** crushil has joined #openstack-ironic18:13
*** mat128 has joined #openstack-ironic18:13
*** fragatina has quit IRC18:15
*** samueldmq has quit IRC18:19
*** samueldmq has joined #openstack-ironic18:19
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: DNM experiment re-using devstack-tempest job  https://review.openstack.org/52016718:20
*** gfhellma__ has joined #openstack-ironic18:20
*** chas_ has quit IRC18:20
*** gfhellma_ has quit IRC18:24
*** dpawar has quit IRC18:25
*** trown is now known as trown|lunch18:26
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements  https://review.openstack.org/51980218:28
openstackgerritMark Goddard proposed openstack/ironic master: Remove provisioning network ports during tear down  https://review.openstack.org/52006418:32
*** fragatina has joined #openstack-ironic18:34
*** mgoddard has joined #openstack-ironic18:36
*** e0ne has quit IRC18:36
*** dpawar has joined #openstack-ironic18:39
mgoddard Nova bug related to ironic: https://bugs.launchpad.net/nova/+bug/1732506. Should I mark it as 'also affects ironic'? Or is there another way to ensure the ironic team remains aware of it?18:41
openstackLaunchpad bug 1732506 in OpenStack Compute (nova) "Baremetal instance stuck in BUILD state following ironic node tear down or delete" [Undecided,New]18:41
*** makowals has joined #openstack-ironic18:42
rloomgoddard: is it something that needs addressing/fixing in ironic too?18:42
rloomgoddard: i just tagged it with 'ironic'18:43
mgoddardrloo: nope, it's purely nova18:43
rloomgoddard: so, the ironic team, if they want to know about it, can look for the 'ironic' tag I think. we/ironic don't have any formal process, we used to have ironic liaisons to nova...18:44
*** dpawar has quit IRC18:45
rloomgoddard: and, you shouldn't do such nasty things using our software :D18:45
rloomgoddard: guess you were simulating some naive operator...18:46
*** chas_ has joined #openstack-ironic18:46
mgoddardrloo: good to know about the tags18:48
rloomgoddard: :)18:48
mgoddardrloo: noted. I will be sure to not veer from the beaten path in future :)18:48
rloomgoddard: ha ha18:49
mgoddardrloo: I think it can also happen if your token times out, which is a valid scenario18:50
rloomgoddard: yup, anyway, i believe we should handle all cases, cuz you know, power failures, acts of god, etc18:50
mgoddardrloo: nope forget that. I was simulating instance deploy failures to see how nova handles clean up with a timed out token. Spoiler alert: badly - use service tokens to avoid this18:52
mgoddardrloo: forget what I said, not what you said... +1 what you said18:53
rloomgoddard: heh, but good that you are testing!18:53
*** AlexeyAbashkin has joined #openstack-ironic19:13
*** AlexeyAbashkin has quit IRC19:17
*** d0ugal has joined #openstack-ironic19:20
*** trown|lunch is now known as trown19:21
*** AlexeyAbashkin has joined #openstack-ironic19:22
*** rwsu has quit IRC19:24
*** AlexeyAbashkin has quit IRC19:26
*** Sukhdev has joined #openstack-ironic19:28
*** crushil has quit IRC19:31
*** e0ne has joined #openstack-ironic19:39
*** dprince has quit IRC19:45
*** crushil has joined #openstack-ironic19:47
*** mgoddard has quit IRC19:50
*** ionutbalutoiu has quit IRC20:18
*** ionutbalutoiu has joined #openstack-ironic20:20
*** d0ugal_ has joined #openstack-ironic20:25
*** d0ugal has quit IRC20:26
*** rwsu has joined #openstack-ironic20:35
*** Sukhdev has quit IRC20:39
*** d0ugal_ has quit IRC20:40
*** d0ugal has joined #openstack-ironic20:40
*** d0ugal has quit IRC20:40
*** d0ugal has joined #openstack-ironic20:40
openstackgerritMerged openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/51910520:43
*** etingof has quit IRC20:46
*** d0ugal has quit IRC20:52
*** sasha2 has quit IRC20:55
rloo%#@# we did an ironic release and forgot to update the release mapping :-(20:55
rloo^^ which means unit test will fail :-(20:55
NobodyCamoh no!20:55
* rloo is working on a fix20:56
NobodyCam:)20:56
* NobodyCam slides a warm cup of tea towards rloo :)20:56
* rloo is thankful for the tea :)20:57
NobodyCam:)20:58
* TheJulia offers lots of tea to rloo 21:00
openstackgerritRuby Loo proposed openstack/ironic master: Add 9.2 to release mappings  https://review.openstack.org/52020321:02
rlooTheJulia, jlvillal, vdrok ^^ that should fix it. Not sure it needs a release note. We should have done that before we cut the release though.21:03
TheJulialooks correct to me21:03
TheJuliaIn theory, won't we need to cut 9.2.1?21:04
TheJuliaif so, then yes, we need to add a reno21:04
rlooTheJulia: am just thinking about what it means that we didn't do it before the release.21:04
TheJuliaI only think it would matter for those that need to pin21:05
rlooTheJulia: it means 1. you can still do a rolling upgrade from ocata to 9.2 cause you can pin 'ocata'.21:05
rlooTheJulia: 2. you cannot do a rolling upgrade from 9.2 to something21:05
rlooTheJulia: cuz '9.2' is not an avail choice.21:05
TheJuliabut you can with the next version, since that has to be present upon upgrade21:05
TheJuliaso I kind of wonder if it is moderately okay....21:05
*** Goneri has quit IRC21:06
TheJulialine 13121:06
rlooTheJulia: if you are using 9.2 and you want to upgrade to 9.3, then you are right, the 9.3 version will have '9.2' as a choice. so we're good. phew.21:06
TheJuliathe version mapping, but we've not shipped officially for this cycle so I think it is okay21:07
TheJuliathis is not end of cycle21:07
TheJuliaThe more I think about it, the more I'm okay with it21:07
rlooTheJulia: yup, I think we're good. I'll add a comment to that patch. thx for thinking it through with me :)21:07
*** livelace has quit IRC21:08
TheJuliano problem21:09
* TheJulia pokes google docs to see if it is working again21:09
TheJulianope....21:10
*** jtomasek has quit IRC21:21
*** livelace has joined #openstack-ironic21:21
*** sasha2 has joined #openstack-ironic21:26
*** sasha2 has quit IRC21:26
*** sasha2 has joined #openstack-ironic21:27
*** sasha2 has quit IRC21:27
*** sasha2 has joined #openstack-ironic21:27
*** threestrands has joined #openstack-ironic21:32
*** threestrands has quit IRC21:32
*** threestrands has joined #openstack-ironic21:32
openstackgerritJulia Kreger proposed openstack/bifrost stable/pike: Correct minor pike branch config issues  https://review.openstack.org/51981821:33
openstackgerritJulia Kreger proposed openstack/bifrost stable/pike: Change perms for the VM logs  https://review.openstack.org/52021021:33
TheJuliaNobodyCam: ^^^ that should clear up the pike branch21:39
TheJuliaminus dib21:40
NobodyCam:) will take a look after test have had a chance to run! :)21:40
*** trown is now known as trown|outtypewww21:41
*** rwsu has quit IRC21:47
*** linkmark has joined #openstack-ironic21:50
*** ccamacho has quit IRC22:02
*** makowals has quit IRC22:06
*** rwsu has joined #openstack-ironic22:08
*** e0ne has quit IRC22:09
*** e0ne has joined #openstack-ironic22:09
*** e0ne has quit IRC22:10
*** e0ne has joined #openstack-ironic22:11
*** e0ne has quit IRC22:11
*** e0ne has joined #openstack-ironic22:12
*** e0ne has quit IRC22:13
*** rwsu has quit IRC22:14
*** e0ne has joined #openstack-ironic22:14
*** e0ne has quit IRC22:14
*** e0ne has joined #openstack-ironic22:15
*** e0ne has quit IRC22:15
*** e0ne has joined #openstack-ironic22:15
*** e0ne has quit IRC22:16
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Catch OSError thrown when hexdump is missing  https://review.openstack.org/52022322:17
*** AlexeyAbashkin has joined #openstack-ironic22:21
openstackgerritJulia Kreger proposed openstack/ironic-lib master: Add raises note to disk_utils.get_disk_identifier  https://review.openstack.org/52022722:22
TheJuliaNobodyCam: done and the non dib ones passed22:23
* TheJulia dances briefly22:23
*** rcernin has joined #openstack-ironic22:23
*** rwsu has joined #openstack-ironic22:25
openstackgerritJulia Kreger proposed openstack/bifrost stable/ocata: Cleanup stable branch backport for CI  https://review.openstack.org/51644522:26
*** AlexeyAbashkin has quit IRC22:26
*** ionutbalutoiu has quit IRC22:27
openstackgerritJulia Kreger proposed openstack/ironic-ui master: Added support for injecting non-maskable interrupts  https://review.openstack.org/51614422:29
*** ionutbalutoiu has joined #openstack-ironic22:32
NobodyCamTheJulia: looking at 52021022:35
TheJuliaMerci22:35
NobodyCamcould that be a little over kill?22:35
NobodyCamhttps://www.irccloud.com/pastebin/FmOKnZvI/22:35
NobodyCamwith that we will chmod 755 /var22:36
NobodyCamI do understand that prepare_libvirt is meant for test env's22:37
*** priteau has quit IRC22:50
*** d0ugal has joined #openstack-ironic22:50
*** mat128 has quit IRC22:54
*** charcol has joined #openstack-ironic23:01
*** rwsu has quit IRC23:01
*** chas_ has quit IRC23:01
*** chas_ has joined #openstack-ironic23:03
rloojlvillal, vdrok, other cores out there, we need this approved t unblock our gate: https://review.openstack.org/#/c/520203/23:06
patchbotpatch 520203 - ironic - Add 9.2 to release mappings23:06
* jlvillal looks more closely this time23:07
jlvillalrloo: I didn't realize before it was blocking the gate. Sorry :(23:07
rloojlvillal:  no worries. i even updated our whiteboard :)23:07
jlvillalrloo: How did it break our gate? Is it because a release happened?23:08
*** chas_ has quit IRC23:08
rloojlvillal: yup, we released 9.2 today23:08
rloojlvillal: and forgot to update the table. there's a unit test that checks... :-(23:08
jlvillalrloo: Ah. Wonky that doing a release can then break our gate23:08
rloojlvillal: well, the unit test fails23:08
rloojlvillal: and it is only after the release, that the new version is available for the unit test23:09
rloojlvillal: which sees that the new version is not in the release_mappings...23:09
jlvillalrloo: Interesting unit test. I'm guessing it checks out the git versions or something.23:10
rloojlvillal: heh, vdrok beat you to it!23:10
jlvillalrloo: Damn vdrok for being on west coast time! ;)23:11
rloojlvillal: heh, i'm liking it so far :)23:11
* jlvillal thinks his thunder will be stolen often now 23:12
rloojlvillal: no worries, esp good since you'll be on vacation.23:12
jlvillalTrue :)23:12
vdrok:)23:12
*** cdearborn has quit IRC23:12
rloojlvillal: wrt the unit test, i am thinking there is an off-by-one error; maybe we don't need to test that the current version is in the table; we just need to test that the prev version is in the table. but I'm not even going to worry/think about that any more. dinner is calling...23:13
*** priteau has joined #openstack-ironic23:13
jlvillalrloo: Sounds good!23:14
openstackgerritMerged openstack/python-ironicclient master: Updated from global requirements  https://review.openstack.org/51980223:16
*** priteau has quit IRC23:17
*** AlexeyAbashkin has joined #openstack-ironic23:20
NobodyCamrloo: isn't master the current version?23:23
*** cfriesen has quit IRC23:24
*** cfriesen has joined #openstack-ironic23:24
rlooNobodyCam: i have already forgotten how the test works, but even with master, there is a version number. that number isn't incremented until after a release. so the master version is somewhat equal to the most recently released version.23:25
*** AlexeyAbashkin has quit IRC23:25
NobodyCamahh okay23:25
jlvillalrloo: NobodyCam: The test uses the pbr version23:25
* jlvillal doesn't know how that works, but stopped investigating when he got to pbr.someversionfunctioncall()23:26
NobodyCam:)23:26
*** cfriesen has quit IRC23:26
*** cfriesen_ has joined #openstack-ironic23:26
*** masber has joined #openstack-ironic23:30
*** cfriesen_ has quit IRC23:33
*** cfriesen_ has joined #openstack-ironic23:33
*** ianychoi has quit IRC23:37
*** etingof has joined #openstack-ironic23:48
*** etingof has quit IRC23:53
*** leitan has joined #openstack-ironic23:53
*** karimb has quit IRC23:54
leitanHi guys, dont know if theres someone out there , but i being playing with my ironic setup for a couple of weeks now, and i saw a driver called "networking_generic_switch", and i was wondering if that driver can be in the same deployment with OVS overlay networks for example, using ironic integrated with nova23:55

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