Monday, 2019-07-08

*** rcernin has joined #openstack-ironic00:13
*** trident has joined #openstack-ironic00:33
*** yedongcan has joined #openstack-ironic01:14
*** altlogbot_2 has quit IRC01:28
*** altlogbot_3 has joined #openstack-ironic01:29
*** hwoarang has quit IRC02:36
*** hwoarang has joined #openstack-ironic02:38
*** rh-jelabarre has joined #openstack-ironic03:48
*** rh-jelabarre has quit IRC04:29
*** mkowalski has joined #openstack-ironic05:29
*** whoami-rajat has joined #openstack-ironic05:30
*** mkowalski has quit IRC05:44
*** baha has quit IRC05:45
*** ileixe has joined #openstack-ironic05:55
ileixeHi ironic~05:55
ileixeDoes anybody know the status of spec https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/L3-based-deployment.html?05:56
*** pcaruana has joined #openstack-ironic06:12
*** yaawang has quit IRC06:15
arne_wiebalckgood morning, ironic!06:38
*** yaawang has joined #openstack-ironic06:43
*** yaawang has quit IRC06:50
*** yaawang has joined #openstack-ironic06:55
*** mkowalski has joined #openstack-ironic07:01
*** iurygregory has joined #openstack-ironic07:07
*** rcernin has quit IRC07:12
iurygregorymorning all07:12
*** ricolin has joined #openstack-ironic07:18
*** tssurya has joined #openstack-ironic07:28
*** lucasagomes has joined #openstack-ironic07:39
dtantsurmorning ironic07:57
*** iurygregory has quit IRC07:57
arne_wiebalckeingof: for the IPMI cipher issue, there seems to be a patch to ipmitool to negotiate with the server side08:01
arne_wiebalcketingof ofc08:03
*** rpittau|afk is now known as rpittau08:03
rpittaugood morning ironic! o/08:03
*** etingof has joined #openstack-ironic08:13
*** iurygregory has joined #openstack-ironic08:24
*** S4ren has joined #openstack-ironic08:28
dtantsurTheJulia, rpittau, wdyt https://storyboard.openstack.org/#!/story/2006175 ?08:29
dtantsurjroll and mgoddard if you're around ^^^08:30
mgoddardbit early for jroll but I'll look08:31
rpittauhey dtantsur , reading08:31
mgoddarddtantsur: works well for http, but UUIDs in config files is bad for deployment tooling08:32
dtantsurmgoddard: yep, I guess it's more for the standalone case08:33
mgoddardyeah, fine for that08:33
dtantsurespecially when file links are used08:33
dtantsuras a side note, I'd support using image names, but that's another RFE08:33
rpittaudtantsur: it makes sense to me08:34
mgoddardper-node is nice for testing a new ramdisk, but I assume it would be compatible with a global setting08:34
dtantsurmgoddard, rpittau, would you mind commenting on the RFE itself? since we no longer have meetings, it's important to keep track of conversations in storyboard.08:34
mgoddardsure08:34
dtantsurmgoddard: yep, it's not removed or deprecated08:34
rpittaudtantsur: yup!08:34
dtantsurdriver_info still takes priority08:35
rpittaudtantsur: mmm so the note at the bottom confuses me a little, if we look for kernel/ramdisk in driver_info in the first step, are we excluding them if we have cpu_arch specified or we rely on driver_info anyway and the next 2 steps are skipped?08:39
dtantsurrpittau: driver_info always takes priority08:40
rpittaudtantsur: ok08:40
dtantsurunconditionally08:40
rpittauok, makese sense08:41
*** yaawang has quit IRC08:41
*** iurygregory has quit IRC08:51
openstackgerritVanou Ishii proposed openstack/ironic master: Add logic to determine Ironic node is HW or not into install_ironic  https://review.opendev.org/66961908:52
openstackgerritVanou Ishii proposed openstack/ironic master: Add logic to determine Ironic node is HW or not into configure_ironic_dirs  https://review.opendev.org/66962008:53
*** derekh has joined #openstack-ironic08:54
*** iurygregory has joined #openstack-ironic08:56
openstackgerritMerged openstack/metalsmith stable/stein: CI: increase the timeout for the СentOS job  https://review.opendev.org/66789109:00
*** Lucas_Gray has joined #openstack-ironic09:05
*** priteau has joined #openstack-ironic09:08
*** srini_ has joined #openstack-ironic09:12
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add Redfish Virtual Media Boot support  https://review.opendev.org/63845309:20
* etingof lost all his firefox credentials along with all hope in computers09:26
openstackgerritMerged openstack/ironic stable/stein: Remove ipxe tags when ipxe6 is in use.  https://review.opendev.org/66567210:21
*** priteau has quit IRC10:42
*** hjensas is now known as hjensas|lunch11:06
*** priteau has joined #openstack-ironic11:08
*** priteau has quit IRC11:16
*** tesseract has joined #openstack-ironic11:17
*** tesseract has quit IRC11:20
*** tesseract has joined #openstack-ironic11:21
*** rh-jelabarre has joined #openstack-ironic11:22
*** ricolin has quit IRC11:26
*** Lucas_Gray has quit IRC11:37
*** Lucas_Gray has joined #openstack-ironic11:40
jrolldtantsur: we've talked about that for years, I can't believe we've never done it11:41
dtantsurditto11:41
*** Lucas_Gray has quit IRC11:48
*** Lucas_Gray has joined #openstack-ironic11:52
*** henriqueof has joined #openstack-ironic11:59
*** srini_ has quit IRC12:00
xXraphXxhello :). Is there any known limitation about ironic and glance shared images ? I'm entcountering some issue with a specific driver (the ansible one), but when I look at ironic/common/glance_service/service_utils:def is_image_available, I was wondering if it was the only one concerned. The context at the moment of calling this function (continue_deploy) in the driver is not an admin one (triggered by the conductor heartbeat method)12:03
*** iurygregory has quit IRC12:04
dtantsurxXraphXx: hi. I think we require shared images for deploy_kernel/deploy_ramdisk12:07
dtantsurI'm not sure we require shared images for instances12:07
*** rachit7 has joined #openstack-ironic12:12
xXraphXxdtantsur, ok. I'm going to investigate a bit more, then create a story to give more details about the entcountered issue if needed12:15
*** rloo has joined #openstack-ironic12:28
*** ijw has joined #openstack-ironic12:30
*** ijw has quit IRC12:35
*** verma-varsha has joined #openstack-ironic12:35
*** edleafe has joined #openstack-ironic12:42
*** rachit7 has quit IRC12:48
*** hjensas|lunch is now known as hjensas12:54
openstackgerritIlya Etingof proposed openstack/sushy master: Disregard registry files of unsupported types  https://review.opendev.org/66893612:54
*** cdearborn has joined #openstack-ironic13:03
*** iurygregory has joined #openstack-ironic13:07
*** rachit7 has joined #openstack-ironic13:09
*** verma-varsha has quit IRC13:13
*** tesseract has quit IRC13:14
*** yolanda has quit IRC13:14
*** henriqueof has quit IRC13:15
*** tesseract has joined #openstack-ironic13:16
TheJuliagood morning everyone!13:16
dtantsurmorning TheJulia13:21
rpittauhi TheJulia :)13:21
*** gkadam has joined #openstack-ironic13:24
* TheJulia needs a coffee iv13:26
*** gkadam has quit IRC13:26
TheJuliadtantsur: seems there is general agreement it would be good, so I think that means rfe-approved13:28
dtantsur\o/13:29
TheJuliaileixe: implentation has not started on that feature. The original author of the spec became otherwise occupied. Oddly enough I do have an email from that author in my mailbox I need to read this morning13:29
xXraphXxHere are more details on the bug entcountered with shared images: https://storyboard.openstack.org/#!/story/2006178. Implementing a fix proposal asap13:37
TheJuliaxXraphXx: Fun! I guess the decline of the concept of admin users means that in your deployment ironic's service account just doesn't have rights in glance13:40
*** jcoufal has joined #openstack-ironic13:52
*** yedongcan has left #openstack-ironic13:57
*** priteau has joined #openstack-ironic14:02
*** baha has joined #openstack-ironic14:02
*** sthussey has joined #openstack-ironic14:04
*** FlorianFa has quit IRC14:05
xXraphXxTheJulia, yes it actually has special rights in glance :). And the image is correctly fetched from glance.14:07
xXraphXxBut ironic rightly drops it, thus interrupting the deployment, because at the point it checks the availabilty (in the callback), the method is_image_available has no way, given the context, to allow the image to be seen14:07
TheJuliaahh14:08
TheJuliaI didn't get that far, yeah, that is a nice bug14:08
dtantsurI think we have some code to compare image.project_id with the user's project_id, but it may well be broken..14:08
*** dmellado has quit IRC14:09
*** stevebaker has quit IRC14:09
*** dmellado has joined #openstack-ironic14:11
*** Lucas_Gray has quit IRC14:12
* TheJulia has not looked at the glance code in a very long time14:13
*** stevebaker has joined #openstack-ironic14:14
*** Lucas_Gray has joined #openstack-ironic14:14
*** yolanda has joined #openstack-ironic14:15
*** trandles has joined #openstack-ironic14:23
dtantsurdon't :)14:27
*** stevebaker has quit IRC14:30
*** dmellado has quit IRC14:30
*** dmellado has joined #openstack-ironic14:32
TheJuliawill I go blind?14:34
*** ricolin has joined #openstack-ironic14:35
*** stevebaker has joined #openstack-ironic14:37
*** bdodd has joined #openstack-ironic14:38
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: manual introspection trigger command  https://review.opendev.org/66619814:42
TheJuliarpittau: dtantsur ^^^14:42
dtantsurthx!14:43
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: manual introspection trigger command  https://review.opendev.org/66619814:44
TheJulianoticed one of kaifeng's comments14:44
TheJuliaokay, that should be better now14:44
TheJuliadtantsur: seems kaifeng's issue is with the pipe code to keep it in a loop forever which is exactly what IPA does14:44
TheJuliaopen to ideas14:45
dtantsurI haven't looked in a while, need to check now14:45
dtantsurwe don't have a meeting, do we?14:45
TheJuliano, office hours14:45
TheJuliaso! what chaos is everyone up to, and are people updating the ironic whiteboard?14:46
* TheJulia is going to make coffee and likely clean up reviewed stuffs... I believe once we got ironic-inspector's gate happy last week a number of things moved14:46
*** stevebaker has quit IRC14:49
*** dmellado has quit IRC14:49
rpittauI did update it last friday14:51
TheJulia\o/14:51
rpittauprobably forgot something :P14:52
*** dmellado has joined #openstack-ironic14:53
*** stendulker has joined #openstack-ironic14:55
*** iurygregory_ has joined #openstack-ironic14:57
*** iurygregory has quit IRC14:57
openstackgerritMerged openstack/ironic-inspector master: Use oslo_db.sqlalchemy.test_fixtures  https://review.opendev.org/66863314:57
*** kaifeng has joined #openstack-ironic14:58
*** stevebaker has joined #openstack-ironic15:01
*** jjohnson42 has joined #openstack-ironic15:03
*** trandles has quit IRC15:04
* TheJulia wonders if everyone has seen the mailing list :)15:05
dtantsurwhat is the mailing list? is it food? is it sweet?15:06
TheJuliaoffice hours kind of sweet? maybe?15:06
*** verma-varsha has joined #openstack-ironic15:09
dtantsurshould we start office hours officially with #startmeeting?15:09
TheJuliathis is a really good question15:09
rpittau#startofficehours ?15:09
rajiniro/15:10
TheJuliaI wonder if that should be a tag or a thing15:10
TheJuliaThe thing the TC ran into was a feeling of officiality in terms of special logs being generated15:11
* TheJulia gets out an "office hours" sign15:11
etingofshould we update this? -- https://wiki.openstack.org/wiki/Meetings/Ironic15:14
TheJuliaetingof: yeah, but ENOTIME15:15
*** gyee has joined #openstack-ironic15:17
* etingof lost that mailinglist message to link up15:19
* kaifeng didn't see the mail either15:22
TheJuliaoh15:23
TheJuliahmm :(15:23
jrollhttp://lists.openstack.org/pipermail/openstack-discuss/2019-July/007495.html15:23
jroll:)15:23
TheJuliahttp://lists.openstack.org/pipermail/openstack-discuss/2019-July/007495.html15:24
TheJulia\o/ for lag15:24
kaifengisn't that is a notification mail?15:24
TheJuliakaifeng: I replied to your comment on the ipa patch, looking for options since it is a repeated pattern already to cause select15:25
TheJuliakaifeng: yes15:25
kaifengI thought the ML meeting has started15:25
kaifengTheJulia yep, thanks for the clarification, I think it makes sense if you take is as a pattern15:26
kaifengs/is/it/15:26
TheJuliaI remember select without poll on a pipe, socket, or file is... weird15:26
kaifengselect just needs a file descriptor15:28
TheJuliayeah, which all of those things are15:29
kaifengactually sleep() is enough for the IPA patch15:29
TheJuliaI was thinking we could introduce jitter, but yeah, we could sleep. Break would still work15:29
kaifengwe can reuse the jitter code from the heartbeater later15:30
* TheJulia tries to ipa it's self could benefit from the select with the threads, this yeah, selep would just work too15:30
*** iurygregory has joined #openstack-ironic15:30
*** henriqueof has joined #openstack-ironic15:31
*** iurygregory_ has quit IRC15:32
*** mkowalski has quit IRC15:37
*** tesseract has quit IRC15:46
*** tesseract has joined #openstack-ironic15:48
*** kaifeng has quit IRC15:51
*** tssurya has quit IRC15:53
*** stendulker has quit IRC16:01
*** iurygregory has quit IRC16:07
*** dsneddon has joined #openstack-ironic16:08
*** lucasagomes has quit IRC16:09
*** whoami-rajat has quit IRC16:10
*** whoami-rajat has joined #openstack-ironic16:15
*** tesseract has quit IRC16:18
*** Lucas_Gray has quit IRC16:18
*** verma-varsha has quit IRC16:23
openstackgerritIlya Etingof proposed openstack/ironic master: Add Redfish Virtual Media Boot support  https://review.opendev.org/63845316:27
*** dtantsur is now known as dtantsur|afk16:34
dtantsur|afkg'night16:34
*** igordc has joined #openstack-ironic16:37
*** rachit7 has quit IRC16:47
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Using from_device_file from Devices  https://review.opendev.org/66972016:50
rpittaugoodnight! o/16:50
*** rpittau is now known as rpittau|afk16:50
dtantsur|afk/quit/quit16:51
*** derekh has quit IRC17:00
*** igordc has quit IRC17:10
*** henriqueof has quit IRC17:12
*** priteau has quit IRC17:21
TheJuliadtantsur|afk:  no chance of looking at https://review.opendev.org/#/c/666198/ ? :)17:25
patchbotpatch 666198 - ironic-python-agent - manual introspection trigger command - 8 patch sets17:25
*** henriqueof has joined #openstack-ironic17:32
*** ijw has joined #openstack-ironic17:39
*** ijw_ has joined #openstack-ironic17:40
*** igordc has joined #openstack-ironic17:41
*** igordc has quit IRC17:43
*** ijw has quit IRC17:44
*** S4ren has quit IRC17:44
*** SpamapS has quit IRC17:46
*** igordc has joined #openstack-ironic17:46
*** mkowalski has joined #openstack-ironic17:47
*** SpamapS has joined #openstack-ironic17:55
*** whoami-rajat has quit IRC18:20
*** priteau has joined #openstack-ironic18:22
*** mkowalski_ has joined #openstack-ironic18:22
*** mkowalski has quit IRC18:24
*** ijw_ has quit IRC18:43
*** ijw has joined #openstack-ironic18:45
*** rloo has quit IRC18:50
*** ijw has quit IRC18:52
*** mkowalski_ has quit IRC18:53
*** whoami-rajat has joined #openstack-ironic18:57
*** mkowalski_ has joined #openstack-ironic19:06
*** ricolin has quit IRC19:21
*** jtomasek has joined #openstack-ironic19:31
*** mkowalski_ has quit IRC19:39
*** ijw has joined #openstack-ironic19:56
*** jcoufal has quit IRC20:08
arne_wiebalckTheJulia: jroll: For the power sync between ironic and nova we're thinking to emit warnings when ironic cannot deliver the notifications. Specifically, this would be the case when in an ironic+nova deployment nova is not configured in ironic (e.g. all nova options commented out) but also when there is no nova, i.e. in the ironic stand-alone case. Does this sound like a sensible approach?20:09
arne_wiebalckTheJulia: jroll: This is in response to the points you made on the storyboard and the review for the ironic stand-alone use case.20:11
arne_wiebalckTheJulia: jroll: So far we've never tried ironic stand-alone, hence the question.20:12
*** Lucas_Gray has joined #openstack-ironic20:32
arne_wiebalckbye, everyone o/20:39
*** ianychoi has joined #openstack-ironic20:41
*** priteau has quit IRC20:44
*** priteau has joined #openstack-ironic20:49
*** pcaruana has quit IRC20:50
*** jjohnson42 has quit IRC20:58
*** whoami-rajat has quit IRC21:00
*** rloo has joined #openstack-ironic21:04
*** rloo has quit IRC21:08
*** rloo has joined #openstack-ironic21:09
jrollarne_wiebalck: that seems fine, INFO might even be loud enough21:22
*** priteau has quit IRC21:27
*** priteau has joined #openstack-ironic21:27
*** priteau has quit IRC21:29
*** henriqueof has quit IRC21:35
*** jtomasek has quit IRC21:36
*** rloo has quit IRC22:08
TheJuliajroll: arne_wiebalck: I kind of second info but would need to look tat the code again. In the grand scheme of things, I suspect that as long as we don't break the standalone case those users will not care22:15
*** Lucas_Gray has quit IRC22:37
*** Lucas_Gray has joined #openstack-ironic22:37
*** bdodd has quit IRC22:51
*** andreaf has quit IRC23:15
*** andreaf has joined #openstack-ironic23:15
*** hoonetorg has quit IRC23:32
*** rcernin has joined #openstack-ironic23:35
*** gyee has quit IRC23:42
*** hoonetorg has joined #openstack-ironic23:45
*** sthussey has quit IRC23:54
*** gyee has joined #openstack-ironic23:57

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