| *** masber has joined #openstack-ironic | 00:03 | |
| *** masber has quit IRC | 00:03 | |
| *** masber has joined #openstack-ironic | 00:03 | |
| *** masber has quit IRC | 00:04 | |
| *** masber has joined #openstack-ironic | 00:04 | |
| *** masber has quit IRC | 00:04 | |
| *** masber has joined #openstack-ironic | 00:06 | |
| *** masber has quit IRC | 00:06 | |
| *** masber has joined #openstack-ironic | 00:08 | |
| *** etingof has quit IRC | 00:32 | |
| *** etingof has joined #openstack-ironic | 00:45 | |
| *** etingof has quit IRC | 00:53 | |
| *** hshiina has joined #openstack-ironic | 00:59 | |
| *** etingof has joined #openstack-ironic | 01:07 | |
| *** zhangfei has joined #openstack-ironic | 01:08 | |
| *** phuongnh has joined #openstack-ironic | 01:14 | |
| *** dprince has joined #openstack-ironic | 01:27 | |
| *** trungnv has joined #openstack-ironic | 01:33 | |
| *** tiendc has joined #openstack-ironic | 01:39 | |
| *** jesusaur has quit IRC | 01:45 | |
| *** baoli has joined #openstack-ironic | 01:45 | |
| *** jesusaur has joined #openstack-ironic | 01:53 | |
| *** zhangfei has quit IRC | 01:54 | |
| *** baoli has quit IRC | 01:59 | |
| *** baoli has joined #openstack-ironic | 02:00 | |
| *** baoli has quit IRC | 02:04 | |
| *** zhangfei has joined #openstack-ironic | 02:07 | |
| *** dprince has quit IRC | 02:10 | |
| *** baoli has joined #openstack-ironic | 02:17 | |
| *** baoli has quit IRC | 02:28 | |
| *** baoli has joined #openstack-ironic | 02:29 | |
| openstackgerrit | jiapei proposed openstack/ironic master: Add documentatin for XClarity Driver https://review.openstack.org/559960 | 02:32 |
|---|---|---|
| *** baoli has quit IRC | 02:34 | |
| *** ijw has joined #openstack-ironic | 02:38 | |
| *** ijw__ has quit IRC | 02:41 | |
| *** ijw_ has joined #openstack-ironic | 02:42 | |
| *** etingof has quit IRC | 02:42 | |
| *** ijw has quit IRC | 02:45 | |
| *** etingof has joined #openstack-ironic | 02:56 | |
| openstackgerrit | melissaml proposed openstack/python-ironic-inspector-client master: Trivial: Update pypi url to new url https://review.openstack.org/563327 | 03:07 |
| *** etingof has quit IRC | 03:22 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-python-agent master: Rescue bug: tinyipa fails to acquire IP in multitenant env https://review.openstack.org/566279 | 03:24 |
| *** etingof has joined #openstack-ironic | 03:35 | |
| *** etingof has quit IRC | 03:41 | |
| *** etingof has joined #openstack-ironic | 03:54 | |
| *** threestrands has joined #openstack-ironic | 03:59 | |
| *** liuzz_ has joined #openstack-ironic | 03:59 | |
| *** threestrands_ has joined #openstack-ironic | 04:00 | |
| *** threestrands_ has quit IRC | 04:01 | |
| *** threestrands_ has joined #openstack-ironic | 04:02 | |
| *** liuzz has quit IRC | 04:02 | |
| *** threestrands has quit IRC | 04:04 | |
| *** kevinbenton has joined #openstack-ironic | 04:05 | |
| *** hoonetorg has quit IRC | 04:21 | |
| *** hoonetorg has joined #openstack-ironic | 04:39 | |
| *** threestrands_ has quit IRC | 04:39 | |
| *** links has joined #openstack-ironic | 04:41 | |
| *** rbartal has joined #openstack-ironic | 04:41 | |
| *** fragatina has quit IRC | 04:57 | |
| *** zhangfei has quit IRC | 04:58 | |
| *** threestrands has joined #openstack-ironic | 05:04 | |
| *** fragatina has joined #openstack-ironic | 05:07 | |
| *** threestrands has quit IRC | 05:07 | |
| *** zhangfei has joined #openstack-ironic | 05:10 | |
| *** toabctl has joined #openstack-ironic | 05:12 | |
| openstackgerrit | Merged openstack/ironic master: Switch to the fake-hardware hardware type for API tests https://review.openstack.org/564519 | 05:16 |
| *** fragatina has quit IRC | 05:22 | |
| *** fragatina has joined #openstack-ironic | 05:22 | |
| *** hshiina2 has joined #openstack-ironic | 05:25 | |
| *** hshiina has quit IRC | 05:29 | |
| *** fragatina has quit IRC | 05:43 | |
| *** marios has joined #openstack-ironic | 05:46 | |
| openstackgerrit | zenghui.shi proposed openstack/ironic master: Indicate the continuation of cleaning steps in BIOS caching exception msg. https://review.openstack.org/568193 | 05:50 |
| *** zshi has quit IRC | 05:52 | |
| *** zshi has joined #openstack-ironic | 05:53 | |
| *** mjura has joined #openstack-ironic | 06:01 | |
| *** hshiina2 is now known as hshiina | 06:21 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Mark xclarity password as secret https://review.openstack.org/567637 | 06:26 |
| *** pcaruana has joined #openstack-ironic | 06:31 | |
| *** jtomasek has joined #openstack-ironic | 06:32 | |
| *** tesseract has joined #openstack-ironic | 07:10 | |
| *** ajya has joined #openstack-ironic | 07:12 | |
| *** jaganathan has joined #openstack-ironic | 07:17 | |
| *** mgoddard has joined #openstack-ironic | 07:27 | |
| *** jaganathan has quit IRC | 07:31 | |
| *** jaganathan has joined #openstack-ironic | 07:31 | |
| *** trungnv has quit IRC | 07:35 | |
| *** fragatina has joined #openstack-ironic | 07:35 | |
| *** trungnv has joined #openstack-ironic | 07:35 | |
| *** rcernin has quit IRC | 07:39 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:46 | |
| *** racedo has joined #openstack-ironic | 07:46 | |
| *** fragatina has quit IRC | 07:54 | |
| *** kevinbenton has quit IRC | 07:56 | |
| *** tiendc has quit IRC | 07:57 | |
| *** tiendc has joined #openstack-ironic | 07:58 | |
| *** lucas-afk is now known as lucasagomes | 07:59 | |
| *** athomas has joined #openstack-ironic | 08:06 | |
| *** erolg has joined #openstack-ironic | 08:12 | |
| *** dougsz has joined #openstack-ironic | 08:12 | |
| *** jaganathan_ has joined #openstack-ironic | 08:14 | |
| *** jaganathan has quit IRC | 08:14 | |
| *** dtantsur|afk is now known as dtantsur | 08:17 | |
| dtantsur | morning ironic, happy Monday | 08:18 |
| *** jaganathan_ has quit IRC | 08:18 | |
| *** racedo has quit IRC | 08:26 | |
| *** derekh has joined #openstack-ironic | 08:29 | |
| *** racedo has joined #openstack-ironic | 08:29 | |
| *** jaganathan_ has joined #openstack-ironic | 08:30 | |
| etingof | dtantsur, o/ though Fridays tend to be happier somehow | 08:31 |
| *** alex_xu has quit IRC | 08:32 | |
| dtantsur | :) | 08:32 |
| *** alex_xu has joined #openstack-ironic | 08:33 | |
| *** arnewiebalck has joined #openstack-ironic | 08:39 | |
| *** MattMan has quit IRC | 08:41 | |
| *** MattMan has joined #openstack-ironic | 08:41 | |
| *** rbartal has quit IRC | 09:00 | |
| *** milan_ has joined #openstack-ironic | 09:04 | |
| *** masber has quit IRC | 09:06 | |
| *** dtantsur is now known as dtantsur|brb | 09:08 | |
| *** rbartal has joined #openstack-ironic | 09:13 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Adds more `ipmitool` errors as retryable https://review.openstack.org/567217 | 09:31 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Update CI jobs for rescue mode https://review.openstack.org/528704 | 09:31 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: DNM: check rescue in multi-tenancy env https://review.openstack.org/566286 | 09:32 |
| *** mgoddard has quit IRC | 09:33 | |
| *** stendulker has joined #openstack-ironic | 09:37 | |
| *** zhangfei has quit IRC | 09:40 | |
| *** zhangfei has joined #openstack-ironic | 09:41 | |
| etingof | dtantsur|brb, there is still a chance to make Monday better by merging these immortal patches ;-) -- https://review.openstack.org/#/c/523879/ | 09:46 |
| patchbot | patch 523879 - sushy-tools - OpenStack cloud driver added | 09:46 |
| *** diga has joined #openstack-ironic | 09:53 | |
| *** tiendc has quit IRC | 09:54 | |
| *** sambetts|afk is now known as sambetts | 10:02 | |
| sambetts | Morning o/ | 10:02 |
| *** mgoddard has joined #openstack-ironic | 10:04 | |
| *** serlex has joined #openstack-ironic | 10:30 | |
| etingof | sambetts, \o | 10:30 |
| *** rbartal has quit IRC | 10:34 | |
| hjensas | morning o/ | 10:35 |
| *** racedo has quit IRC | 10:36 | |
| *** racedo has joined #openstack-ironic | 10:37 | |
| *** dtantsur|brb is now known as dtantsur | 10:38 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Fix tenant DeprecationWarning from oslo_context https://review.openstack.org/568257 | 10:48 |
| *** rbartal has joined #openstack-ironic | 10:49 | |
| *** dprince has joined #openstack-ironic | 10:49 | |
| sambetts | hey etingof, hjensas | 10:50 |
| *** e0ne has joined #openstack-ironic | 10:53 | |
| *** dprince has quit IRC | 10:55 | |
| *** dprince has joined #openstack-ironic | 10:56 | |
| *** arnewiebalck has quit IRC | 10:56 | |
| *** stendulker_ has joined #openstack-ironic | 11:02 | |
| *** links has quit IRC | 11:05 | |
| *** stendulker has quit IRC | 11:05 | |
| *** stendulker_ has quit IRC | 11:06 | |
| *** dprince has quit IRC | 11:11 | |
| *** zhangfei has quit IRC | 11:13 | |
| *** rbartal has quit IRC | 11:14 | |
| *** links has joined #openstack-ironic | 11:17 | |
| *** phuongnh has quit IRC | 11:18 | |
| *** needsleep is now known as TheJulia | 11:25 | |
| TheJulia | Good morning | 11:25 |
| sambetts | o/ TheJulia | 11:25 |
| *** rbartal has joined #openstack-ironic | 11:25 | |
| dtantsur | morning TheJulia | 11:31 |
| *** dougsz has quit IRC | 11:34 | |
| *** mgoddard has quit IRC | 11:38 | |
| *** diga has quit IRC | 11:40 | |
| * TheJulia needs lots of coffee | 11:47 | |
| *** dprince has joined #openstack-ironic | 11:49 | |
| *** livelace has joined #openstack-ironic | 11:50 | |
| *** links has quit IRC | 11:52 | |
| *** links has joined #openstack-ironic | 11:52 | |
| *** rh-jelabarre has joined #openstack-ironic | 11:53 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API https://review.openstack.org/512579 | 11:53 |
| *** hshiina is now known as hshiina|afk | 11:55 | |
| *** jcoufal has joined #openstack-ironic | 11:57 | |
| *** weshay is now known as weshay_interview | 11:59 | |
| *** dprince has quit IRC | 12:00 | |
| *** mgoddard has joined #openstack-ironic | 12:11 | |
| *** gyankum has joined #openstack-ironic | 12:11 | |
| *** dougsz has joined #openstack-ironic | 12:12 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Power fault recovery: db and rpc implementation https://review.openstack.org/555708 | 12:12 |
| openstackgerrit | Merged openstack/ironic master: Fix tenant DeprecationWarning from oslo_context https://review.openstack.org/567953 | 12:16 |
| openstackgerrit | Merged openstack/ironic master: Fix tenant DeprecationWarning from oslo_context https://review.openstack.org/568257 | 12:16 |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Power fault recovery: API implementation https://review.openstack.org/556015 | 12:19 |
| *** jaganathan_ has quit IRC | 12:19 | |
| *** jaganathan_ has joined #openstack-ironic | 12:19 | |
| TheJulia | Anyone know what Kaifeng's IRC handle is? | 12:20 |
| * dtantsur no idea | 12:21 | |
| *** trown|outtypewww is now known as trown | 12:25 | |
| *** dprince has joined #openstack-ironic | 12:29 | |
| TheJulia | hmm | 12:32 |
| TheJulia | okay | 12:32 |
| TheJulia | If anyone knows, please let me know | 12:32 |
| etingof | TheJulia, may be you can ask via gerrit code review... | 12:35 |
| TheJulia | I've sworn I've seen them on IRC at some point, of course I may just be loosing the remaining bits of my mind | 12:36 |
| dtantsur | yep, they've been here at least several times | 12:38 |
| dtantsur | the nick was more or less straightforward | 12:38 |
| TheJulia | That is what I was thinking | 12:38 |
| * dtantsur looks at ironic unit tests and wants to cry | 12:38 | |
| * dtantsur runs to the woods to live with other owls | 12:38 | |
| TheJulia | dtantsur: it is too early in the week to cry | 12:38 |
| dtantsur | it's just the right time for somebody looking at how deeply we rely on the 'fake' classic driver there | 12:39 |
| dtantsur | like, I need to rewrite maybe a few hundreds of tests | 12:39 |
| * dtantsur cries and runs, cries and runs | 12:39 | |
| * TheJulia slides booze over to dtantsur to hopefully improve things a little bit | 12:40 | |
| *** gyankum has quit IRC | 12:42 | |
| *** bfournie has joined #openstack-ironic | 12:43 | |
| dtantsur | TheJulia, sambetts, should we move on with https://review.openstack.org/#/c/305864/ ? it started more than 2 years ago, which is a bit too much for such a trivial change :D | 12:57 |
| patchbot | patch 305864 - ironic-specs - Boot management for in-band inspection | 12:57 |
| sambetts | dtantsur: +100 | 13:00 |
| TheJulia | dtantsur: yes, although wasn't there a reason on holding on that.... | 13:00 |
| *** rloo has joined #openstack-ironic | 13:00 | |
| dtantsur | TheJulia: I *think* 1. we kept arguing about the exact design, 2. there are/were some dreams of reworking inspector completely | 13:01 |
| sambetts | we talked about moving inspection stuff into ironic, but I don't see that happening for a while | 13:01 |
| *** jcoufal_ has joined #openstack-ironic | 13:01 | |
| sambetts | dtantsur++ | 13:01 |
| *** weshay_interview is now known as weshay | 13:01 | |
| TheJulia | We were also beginning to wade into drivercomp then | 13:02 |
| sambetts | I'm trying to find the gerrit patch with the conversation about changing the ironic config option | 13:02 |
| sambetts | for that work | 13:02 |
| dtantsur | I think the latest version reflects what we last discussed | 13:03 |
| *** mjturek has joined #openstack-ironic | 13:03 | |
| *** jcoufal has quit IRC | 13:03 | |
| sambetts | dtantsur: yeah looks right | 13:04 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API https://review.openstack.org/512579 | 13:05 |
| sambetts | dtantsur: merged | 13:07 |
| sambetts | thanks TheJulia | 13:07 |
| TheJulia | woot | 13:07 |
| dtantsur | that was fast, thanks :) | 13:07 |
| TheJulia | I concur, i re-read it and was like "oh, this is way simpler now, I'm onboard!" | 13:07 |
| sambetts | unfortunatly some of these things just get lost in the backlog :( | 13:07 |
| dtantsur | now I can update https://review.openstack.org/#/c/316801/ :) | 13:08 |
| patchbot | patch 316801 - ironic-inspector - Add manage_boot parameter to introspection API | 13:08 |
| dtantsur | though I wonder if it's faster to do it from scratch... >_< | 13:08 |
| TheJulia | dtantsur: Well, to be fair, you could have updated it before.... | 13:08 |
| dtantsur | right, I was swamped by other priority things, and so decided to wait for the resolution | 13:09 |
| *** rbudden has joined #openstack-ironic | 13:11 | |
| * TheJulia nods :) | 13:13 | |
| *** Goneri has joined #openstack-ironic | 13:15 | |
| dtantsur | OMG merge conflicts in EVERY SINGLE FILE | 13:15 |
| *** ElCoyote_ has joined #openstack-ironic | 13:16 | |
| openstackgerrit | Merged openstack/ironic-specs master: Boot management for in-band inspection https://review.openstack.org/305864 | 13:18 |
| rloo | Happy Monday morning ironickers, including dtantsur, TheJulia, sambetts | 13:22 |
| rloo | TheJulia: it is (or was) kaifeng | 13:22 |
| openstackgerrit | Merged openstack/ironic master: Add more parameter explanation when create a node https://review.openstack.org/562452 | 13:23 |
| TheJulia | rloo: thanks | 13:23 |
| * TheJulia wonders if owls like laphroaig.... | 13:24 | |
| sambetts | rloo o/ | 13:28 |
| *** baoli has joined #openstack-ironic | 13:30 | |
| *** superdan is now known as dansmith | 13:33 | |
| *** baoli has quit IRC | 13:34 | |
| *** baoli has joined #openstack-ironic | 13:34 | |
| openstackgerrit | Merged openstack/ironic-ui stable/queens: Imported Translations from Zanata https://review.openstack.org/564402 | 13:34 |
| dtantsur | TheJulia: a lot! :) | 13:35 |
| dtantsur | morning rloo | 13:35 |
| rloo | :) | 13:35 |
| rloo | TheJulia: I was just reviewing this, but did I miss something? The RFE isn't approved yet: https://review.openstack.org/#/c/556758 | 13:38 |
| patchbot | patch 556758 - ironic - Power fault recovery: applying maintenance_type | 13:38 |
| dtantsur | I thought it has a spec | 13:39 |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/564403 | 13:39 |
| dtantsur | rloo: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/power-fault-recovery.html we probably forgot to update the RFE | 13:39 |
| TheJulia | yeah, spec landed, tag just never got updated | 13:40 |
| openstackgerrit | Merged openstack/ironic-ui master: Updated from global requirements https://review.openstack.org/551490 | 13:40 |
| rloo | OUCH. Cuz... we have a usecase where we want both maintenance reasons: 1. from ironic/internally (power failure, clean failure... although the maintenance reason itself should indicate why it failed), and 2. manual via API. | 13:41 |
| rloo | I thought there was another RFE about it but honestly, i am having a hard time navigating via storyboard | 13:42 |
| TheJulia | I'm not sure I grok the disconnect, although I've not looked at the api patches yet | 13:42 |
| dtantsur | rloo: there was a huge RFE that never got any progress | 13:42 |
| rloo | TheJulia: this particular solution only allows for 1 maintenace-reason at any given time. | 13:42 |
| dtantsur | so we ended up with a smaller RFE that just fixes that awful problem of nodes silently becoming non-operable | 13:43 |
| dtantsur | rloo: I think if you want 2 maintenance_reasons, you need a new RFE | 13:43 |
| dtantsur | it was not possible previously as well, so this RFE is clearly a way forward | 13:43 |
| rloo | dtantsur: true, except that for 2 maintenance reasons, one for manual and one for internal-ironic, the internal node fields would be diff than adding this new 'maintenance_type' field. | 13:44 |
| rloo | dtantsur: which means problems upgrading if we were ever to support two diff maintenance_reasons | 13:44 |
| dtantsur | I'm not quite convinced we should | 13:44 |
| TheJulia | Uhhh.. The reason tag indicates when a failure has occured and we have last_error | 13:45 |
| rloo | TheJulia: the 'maintenance_reason' tag? or do you mean some other 'reason' tag? | 13:45 |
| TheJulia | maintenance_reason has two failure cases which are set upon those failures being encountered in the code base, at that point we also update last_error | 13:46 |
| dtantsur | if we allow two reasons, we should allow three and more. and this brings us back to that huge specific fauls spec that nobody reviewed | 13:46 |
| TheJulia | so we have then stored the error, and taken the node out of service | 13:46 |
| dtantsur | which makes me think that we should finish this and then see if we have somebody to work on more | 13:46 |
| TheJulia | iteration++ | 13:46 |
| rloo | I don't think this solution lends itself to iterating w/o issues regarding upgrades | 13:47 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: [WIP] Add manage_boot parameter to introspection API https://review.openstack.org/316801 | 13:48 |
| rloo | sec, /me will actually read the spec/problem first... | 13:48 |
| TheJulia | Well, it may never need iteration, it is the needful behavior. | 13:48 |
| dtantsur | rloo: I don't think it's fair to block a ready-to-land work based on something that is not obvious will even happen | 13:48 |
| dtantsur | or do you mean that you're going to start working on it around today? ;) | 13:48 |
| rloo | dtantsur: heh, this story didn't get updated/tagged and there's no info there that there even was a spec. that isn't fair either. | 13:49 |
| rloo | dtantsur: can you at least give me a few minutes to wrap my head around it? | 13:49 |
| rloo | dtantsur: it was fair for me to put a -2 when i did, and i am reaching out/asking about it nwo. | 13:49 |
| dtantsur | I don't think we -2 patches because some core hasn't read the spec, but sure :) | 13:50 |
| *** jaypipes_ is now known as jaypipes | 13:51 | |
| rloo | dtantsur: the -2 was cuz https://storyboard.openstack.org/#!/story/1596107 is not tagged with rfe-approved. i had no idea even that there was a spec cuz it wasn't obvious to me that a spec had been approved. | 13:51 |
| * TheJulia hates the "rfe" process | 13:51 | |
| dtantsur | okay, I'm fixing this one | 13:51 |
| TheJulia | I already fixed the tag | 13:51 |
| rloo | TheJulia: then please propose something else. | 13:51 |
| dtantsur | okay, so now it's approved according to our process | 13:51 |
| TheJulia | This community doesn't like change, I can't change that at this point | 13:52 |
| dtantsur | note that I don't question your right to -2 patches | 13:52 |
| dtantsur | we should just be fair to the contributor who is not present on why we do it | 13:52 |
| TheJulia | I just don't like the way we use it to block value adding work that is in a ready state to move forward | 13:52 |
| dtantsur | s/be/should be/ | 13:52 |
| *** rbartal has quit IRC | 13:53 | |
| *** jgwentworth is now known as melwitt | 13:57 | |
| TheJulia | I am with ruby, she had every right to block it until her concerns are understood in thinking that something hasn't been approved to be a thing. Design/overall effort changing is inevitable with every feature, and the smaller features are naturally going to get more traction and land faster than epics or epics of epics. | 13:58 |
| *** beekneemech is now known as bnemec | 13:59 | |
| *** baha has joined #openstack-ironic | 14:00 | |
| jroll | morning | 14:02 |
| *** links has quit IRC | 14:03 | |
| dtantsur | TheJulia: I don't disagree with that, my point goes to the further discussion re "Should we delay the much needed work for the sake of potential yet-to-be-implemented-maybe-some-day bigger work" | 14:04 |
| dtantsur | morning jroll | 14:04 |
| jroll | \o | 14:04 |
| jroll | some perspective | 14:04 |
| jroll | we have an internal patch that allows for both an ironic-set reason and a user-set reason | 14:05 |
| jroll | it is from the before times, so we don't (yet) have full context on the use case | 14:05 |
| *** r-daneel has joined #openstack-ironic | 14:05 | |
| *** baoli has quit IRC | 14:05 | |
| *** jiapei has joined #openstack-ironic | 14:06 | |
| *** baoli has joined #openstack-ironic | 14:06 | |
| jroll | so I suspect ruby is wondering if what we're doing covers the needs of large operators, and if the incremental step we're doing now will clobber any further progress | 14:06 |
| *** rbartal has joined #openstack-ironic | 14:06 | |
| rloo | not 'clobber', but make it a bit more difficult to move forward in a coherent fashion. | 14:07 |
| jroll | sure | 14:08 |
| dtantsur | well, the input from fields I have in my hands is that Ironic is doing a horrible job with the power faults | 14:08 |
| dtantsur | and the patch we have in our hands solves that in a way | 14:08 |
| dtantsur | I wish your downstream patch was proposed upstream (long ago, ideally) | 14:08 |
| jroll | me too, trust me :) | 14:09 |
| rloo | dtantsur: the downstream patch would not have directly addressed this particular use case. If I understand, you want *some field* to encode the reason so you can search on it. | 14:09 |
| jroll | but I also think we should give cores more than 30 minutes of irc discussion to think through the effects of landing a patch :) | 14:09 |
| dtantsur | rloo: not really | 14:09 |
| rloo | cuz otherwise, you coudl have just prefixed the mainteance string with 'CLEAN FAILURE: ...' | 14:10 |
| dtantsur | jroll: will till end of the cycle work? ;) | 14:10 |
| rloo | jroll: to be honest, the thinking is during the spec review, which i missed. | 14:10 |
| dtantsur | rloo: right, the goal here is to be able to retry the failed power sync | 14:10 |
| rloo | jroll: but having said that, i believe that i am usually open during the coding phase, to rethink things, so I hope we are too. | 14:10 |
| jroll | rloo: sure, but we also can't say "this matches the spec, cores may no longer question it" | 14:10 |
| jroll | yes | 14:11 |
| dtantsur | which requires knowing whether maintenance was caused by an operator or by ironic | 14:11 |
| dtantsur | jroll: oh, no, we cannot. that's certainly not my intention here. | 14:11 |
| jroll | dtantsur: I know you're joking about end of cycle, but I hope you get what I mean. I want to speed up progress as much as anyone here | 14:11 |
| rloo | dtantsur: right, but you REALLY want, some encoded reason. Not 'operator' vs 'user'. but CLEAN_FAILURE, RESCUE_FAILURE, ... MANUAL | 14:11 |
| jroll | dtantsur: yes, I'm exaggerating for effect :) | 14:11 |
| dtantsur | rloo: this is a side effect, I guess. I personally only care about the maintenance going away if the BMC behaves again/network returned. | 14:12 |
| *** baoli has quit IRC | 14:12 | |
| dtantsur | * ironic-induced maintenance only - hence the database change | 14:12 |
| sambetts | recoverable mainteiance vs unrecoverable maintenace | 14:13 |
| dtantsur | right, something like that | 14:13 |
| sambetts | man I can't spell today.. | 14:13 |
| dtantsur | Monday | 14:14 |
| rloo | dtantsur: so i think it is for our internal/coding/speed purposes, that we have a new separate maintenance_type field. Cuz like i mentioned above, we could prepend the mainteancne_reason strings with eg 'CLEANFAILURE', 'RESCUEFAILURE', etc and use those as indicators of ironic-induced maintenance. | 14:14 |
| *** baoli has joined #openstack-ironic | 14:14 | |
| sambetts | I mean this personally feels like a boolean | 14:14 |
| dtantsur | rloo: it is slow and not overly i18n-friendly | 14:14 |
| dtantsur | I'm fine with a boolean, but I'm not sure it solves rloo's concerns | 14:14 |
| sambetts | maintenance: true/false, maintenance_reason: ..., ironic_induced: true/false | 14:15 |
| rloo | you mean maintenance_type being a boolean? | 14:15 |
| rloo | i don't think so. you need the type, so that your (our) periodic task to try to sync power, can know which nodes are in maint due to power failure. | 14:15 |
| rloo | (note, I haven't actually looked at the code to address this; i just read the spec) | 14:16 |
| * dtantsur W-1 https://review.openstack.org/#/c/555708/ until we agree | 14:16 | |
| patchbot | patch 555708 - ironic - Power fault recovery: db and rpc implementation | 14:16 |
| dtantsur | ah, correct | 14:16 |
| dtantsur | we cannot retry a cleaning failure | 14:16 |
| * dtantsur cannot think today | 14:16 | |
| rloo | ok, tell you what. let me think about it, will get back to you later today. and i promise, i'll review this patch (series) in return :) | 14:17 |
| dtantsur | another approach would be to untangle maintenance and faults (guarding by an API version) | 14:17 |
| rloo | (although i really, really, hate to promise anything) | 14:17 |
| rloo | dtantsur: do you know offhand, the 'faults' big spec? | 14:17 |
| rloo | cuz i honestly haven't read it | 14:17 |
| *** rpioso|afk is now known as rpioso | 14:18 | |
| * jroll can find it | 14:18 | |
| rpioso | Good morning | 14:18 |
| dtantsur | rloo: it proposed a whole new generic mechanism for faults | 14:18 |
| dtantsur | with a new API family, new drivers API, etc | 14:18 |
| rloo | dtantsur: and this one fits into the big picture? | 14:18 |
| jroll | rloo: https://review.openstack.org/#/c/334113/ | 14:18 |
| patchbot | patch 334113 - ironic-specs - Add spec for specific faults support | 14:18 |
| dtantsur | rloo: not quite. I'm just not convinced we should do specific faults at all | 14:19 |
| TheJulia | \o/ discussion /o\ downstream meeting consuming brain | 14:19 |
| rloo | thx jroll. | 14:19 |
| jroll | np | 14:19 |
| rloo | dtantsur: ah, so you're meh on 334113? | 14:19 |
| rloo | dtantsur: or, er, you require convincing :) | 14:19 |
| jroll | I'm meh on 334113 simply because I think it's eating the whole pie in one bite | 14:20 |
| rloo | dtantsur: i'll take a look to see if it convinces me ... | 14:20 |
| jroll | and we don't have two rackspace employees dedicated to implementing it anymore | 14:20 |
| *** cjloader has joined #openstack-ironic | 14:20 | |
| *** cjloader has quit IRC | 14:20 | |
| *** zhangfei has joined #openstack-ironic | 14:20 | |
| rloo | i like it when people dream big | 14:20 |
| * jroll grumbles "Oh, So It's Cancelled?" | 14:20 | |
| *** cjloader has joined #openstack-ironic | 14:21 | |
| TheJulia | dreaming big and then breaking it into small consumable bites is always the best | 14:21 |
| TheJulia | Kind of like huge pizzas | 14:21 |
| * TheJulia now wants izza | 14:21 | |
| rloo | ++ | 14:21 |
| TheJulia | pizza | 14:21 |
| rloo | ++ to both! | 14:21 |
| * rloo removes ++ to pizza. too early for me to eat that. | 14:22 | |
| dtantsur | rloo: I'm with jroll on it | 14:22 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Stop double json encoding the error message responses https://review.openstack.org/567837 | 14:22 |
| * jroll has a fridge full of pulled pork for the week \o/ | 14:23 | |
| * sambetts so jelly | 14:23 | |
| TheJulia | mmmmm pulled pork | 14:23 |
| TheJulia | I've not made pulled pork in ages | 14:24 |
| jroll | https://imgur.com/3rsJroz.jpg for those not already hungry | 14:25 |
| * rloo votes for a pizza & pulled pork & porridge & plum pudding ironic meeting today | 14:25 | |
| TheJulia | porridge and plum pudding? | 14:25 |
| rloo | the two other p* foods that i could think of in 1 sec. although popcorn comes to mind now. | 14:26 |
| olivierb- | Hello Ironicers/Bifrosters. I posted the following on openstack-dib IRC but may be this is more related to ironic/bifrost: | 14:26 |
| jroll | don't forget potato salad | 14:27 |
| olivierb- | just another question on gpt generated dib image which is then provisioned to a machine using bifrost/ironic. When I launch fdisk on the disk where the image is written, I get: GPT PMBR size mismatch (125829119 != 209715199) will be corrected by w(rite).(message is repeated twice). However when I launch a KVM/libvirt instance using the qcow2 directly I do not get any error/warning. Does this ring a bell to someone ??? Should I | 14:27 |
| olivierb- | worry about this errror ? My gut feeling tells me I should ;-) | 14:27 |
| * TheJulia makes a sad face at popcorn | 14:27 | |
| TheJulia | olivierb-: that seems familar, I'm on a meeting right now and can't fully contex switch | 14:29 |
| rloo | jroll: do you know if that faults stuff (the solution advocated in the spec) was discussed with operators? I seem to recall that there had been some discussion, but I don't know if it was generic 'yeah we need to deal with faults' and/or whether it was this particular proposal | 14:30 |
| rloo | jroll: also wondering -- is there a usecase for there being more than one fault occurring at any given time? | 14:30 |
| olivierb- | TheJulia no pb, no urgency there | 14:30 |
| jroll | rloo: I think we had some ops in a summit session or soemthing about it, not sure when/where/who though. it was driven by our operational needs with onmetal | 14:31 |
| jroll | I'd ask penick if he remembers discussing the specific proposal | 14:31 |
| rloo | and my other thought (for now): after putting in place whatever 'fault' mechanism. would it make sense to allow an operator to put that node into that fault -- to get the/wahtever self-healing to do the work? | 14:31 |
| TheJulia | olivierb-: so, your going to get the geometry mismatch because the drive written to is larger | 14:31 |
| *** zhangfei has quit IRC | 14:31 | |
| TheJulia | olivierb-: that being said, I thought we had code to catch and address that in IPA | 14:31 |
| jroll | rloo: not sure about more than one fault, if it's in maintenance nothing else should come up | 14:31 |
| rloo | jroll: yeah, that's what I would think. although the spec talks about multiple faults. | 14:32 |
| jroll | rloo: I'm also not sure about operator putting the node into a fault mode. maybe it makes sense. I could see that being useful for a user to say "hey this disk died" and the operator being able to mark that | 14:32 |
| olivierb- | TheJulia checking with parted I get Warning: Not all of the space available to /dev/vda appears to be used, you can fix the GPT to use all of the space (an extra 83886080 blocks) or continue so that may be exactly what you suggested | 14:33 |
| jroll | rloo: I wasn't super involved with that spec and I've blocked most of it from my memory | 14:33 |
| TheJulia | oh you know what, I bet it is in ironic-lib.... | 14:33 |
| rloo | jroll: totally understandable :) | 14:33 |
| *** chihhsin has quit IRC | 14:34 | |
| TheJulia | olivierb-: are you writing a configuration drive out? | 14:34 |
| *** gyankum has joined #openstack-ironic | 14:34 | |
| *** gyankum has quit IRC | 14:34 | |
| olivierb- | TheJulia sorry what do you mean exactly by that ? | 14:35 |
| *** chihhsin has joined #openstack-ironic | 14:35 | |
| openstackgerrit | Sam Betts proposed openstack/python-ironicclient master: Stop double json decoding API error messages https://review.openstack.org/567836 | 14:35 |
| TheJulia | olivierb-: when you boot up, do you have a config-2 partition? | 14:35 |
| olivierb- | nope, I have only two as follows: | 14:36 |
| TheJulia | olivierb-: are you deploying to baremetal using a playbook? | 14:36 |
| olivierb- | Number Start End Size File system Name Flags | 14:36 |
| olivierb- | 1 1049kB 9437kB 8389kB "BSP" bios_grub | 14:36 |
| olivierb- | 2 9437kB 60.0GB 60.0GB "root" lvm | 14:36 |
| olivierb- | and my disk is 100Gb wide | 14:36 |
| olivierb- | however when I try to create a new partition I also get: | 14:37 |
| olivierb- | (parted) mkpart | 14:37 |
| olivierb- | Partition name? []? test | 14:37 |
| olivierb- | File system type? [ext2]? ext4 | 14:37 |
| olivierb- | Start? 60Gb | 14:37 |
| olivierb- | End? 100% | 14:37 |
| olivierb- | Warning: You requested a partition from 60.0GB to 107GB (sectors 117187500..209715199). | 14:37 |
| olivierb- | The closest location we can manage is 60.0GB to 64.4GB (sectors 117204992..125829086). | 14:37 |
| olivierb- | Is this still acceptable to you? | 14:37 |
| olivierb- | Yes/No? | 14:37 |
| sambetts | olivierb-: please use paste.openstack.org for outputs like that | 14:38 |
| olivierb- | sambetts , oops sorry about this | 14:38 |
| TheJulia | so we have code to correct that, but only when we are writinga configuration drive out to the disk | 14:39 |
| olivierb- | TheJulia could you please point me to this code so that I have a look ? | 14:39 |
| *** rajinir has joined #openstack-ironic | 14:40 | |
| olivierb- | http://paste.openstack.org/show/720926/, sorry again for the former copy/paste in IRC | 14:40 |
| TheJulia | it looks like we would need to make a method in ironic-lib a first class citizen instead of being private, and calling that from IPA in general if it is GPT | 14:40 |
| TheJulia | https://github.com/openstack/ironic-python-agent/blob/0912a24b384518e402394e63fc1ad94579d38b76/ironic_python_agent/extensions/standby.py#L472-L479 <-- IPA deciding what to do and to call ironic-lib | 14:40 |
| TheJulia | https://github.com/openstack/ironic-lib/blob/300e171546ed6832106596861bd3ff50fa061dce/ironic_lib/disk_utils.py#L761-L762 | 14:41 |
| TheJulia | olivierb-: seems like it shouldn't be too hard of a bug to fix, if your up for it | 14:42 |
| TheJulia | We would need to cut an ironic-lib release to make the method public though | 14:42 |
| olivierb- | thanks for the pointer, I'll try to understand the code better and shoot for a fix | 14:42 |
| TheJulia | but that is not too big of a deal | 14:42 |
| TheJulia | olivierb-: it will be two, one for ironic-lib and one for ironic-python-agent. The IPA fix likely won't work until we merge/release ironic-lib, For what it is worth | 14:43 |
| *** zhangfei has joined #openstack-ironic | 14:44 | |
| olivierb- | got it thanks TheJulia | 14:44 |
| TheJulia | Everyone updating whiteboard? :) | 14:46 |
| *** rbudden has quit IRC | 14:48 | |
| *** stendulker has joined #openstack-ironic | 14:53 | |
| *** rbudden has joined #openstack-ironic | 14:53 | |
| *** hshiina has joined #openstack-ironic | 14:55 | |
| *** marlinc has quit IRC | 14:59 | |
| *** mjura has quit IRC | 14:59 | |
| *** ijw_ has quit IRC | 14:59 | |
| *** ijw has joined #openstack-ironic | 14:59 | |
| TheJulia | #startmeeting ironic | 15:00 |
| openstack | Meeting started Mon May 14 15:00:03 2018 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| TheJulia | o/ | 15:00 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| mraineri | o/ | 15:00 |
| TheJulia | Good morning everyone! | 15:00 |
| vdrok | o/ | 15:00 |
| rpioso | o/ | 15:00 |
| jroll | \o | 15:00 |
| rloo | o/ | 15:00 |
| hshiina | o/ | 15:00 |
| jiapei | o/ | 15:00 |
| TheJulia | It is time for our weekly ironic meeting! (Our project name is never going to get old....) :) | 15:00 |
| stendulker | o/ | 15:01 |
| TheJulia | Our agenda is on the wiki. | 15:01 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic | 15:01 |
| mjturek | \o | 15:01 |
| *** gcb has joined #openstack-ironic | 15:01 | |
| jroll | nice and light :) | 15:01 |
| TheJulia | indeed! | 15:01 |
| TheJulia | #topic Announcements / Reminders | 15:01 |
| *** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)" | 15:01 | |
| dtantsur | o/ | 15:02 |
| TheJulia | #info Our meeting is cancelled next week due to the Summit. | 15:02 |
| hjensas | o/ | 15:02 |
| *** pcaruana has quit IRC | 15:02 | |
| TheJulia | That is all I have right now for announcements/reminders. Does anyone else have anything they would like to raise? | 15:03 |
| rloo | the rocky-2 milestone is coming up (sort of) June 7th week | 15:03 |
| rloo | that is nova-spec freeze | 15:03 |
| rloo | are there any from us, that we need to help ? | 15:04 |
| TheJulia | #info Rocky-2 milestone approaching - June 7th | 15:04 |
| TheJulia | rloo: We should follow up on rescue I believe. I think that is the only thing we need to worry about right now. | 15:05 |
| rloo | TheJulia: ok. (or any other non-ironic project specs. i guess i shouldn't just focus on nova.) | 15:05 |
| hshiina | bp for rescue was approved for rocky | 15:05 |
| hshiina | in nova | 15:05 |
| rloo | hshiina: awesome! | 15:06 |
| rloo | so we're good for now :) | 15:06 |
| *** milan_ has quit IRC | 15:06 | |
| TheJulia | Anyway, if there is nothing else, we can proceed to reviewing action items. | 15:06 |
| TheJulia | hshiina: excellent to hear | 15:06 |
| TheJulia | #topic Reviewing action items from last week | 15:07 |
| *** openstack changes topic to "Reviewing action items from last week (Meeting topic: ironic)" | 15:07 | |
| *** S4ren has joined #openstack-ironic | 15:07 | |
| TheJulia | #link http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-05-07-15.00.txt | 15:07 |
| *** marlinc has joined #openstack-ironic | 15:08 | |
| TheJulia | I was super busy last week and fell down on the action items some last week. I did clean-up the whiteboard some, I forgot to send out an email w/r/t cancelling the meeting next week, and I didn't get a chance to look through what other projects are doing re prioritization in storyboard. | 15:08 |
| rloo | Congrats to TheJulia for having the most (and all) action items! | 15:08 |
| TheJulia | i know! I should throw a party! | 15:08 |
| TheJulia | #action TheJulia to send meeting re-meeting cancellation for next week. | 15:09 |
| TheJulia | Re: priorities in storyboard, I think I'll try and get with folks in vancouver and wrap my brain around it then. | 15:09 |
| TheJulia | One last item, the status update email discussion is ongoing on the list, I know etingof replied if memory serves, but I've not had a chance to read it. | 15:10 |
| TheJulia | With that being said, if there are no objections, we can move on. | 15:10 |
| TheJulia | #topic Review subteam status reports | 15:11 |
| *** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:12 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:12 |
| TheJulia | Starting around line 89 | 15:12 |
| TheJulia | err | 15:12 |
| TheJulia | Line 131! | 15:12 |
| TheJulia | err | 15:12 |
| TheJulia | Line 145 | 15:12 |
| TheJulia | Clearly I need lots of coffee | 15:13 |
| TheJulia | and a vacation. | 15:13 |
| jroll | +1000 | 15:13 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Clean up templates from items that cannot be implemented for libvirt https://review.openstack.org/538268 | 15:13 |
| jroll | oh my, bios is close \o/ | 15:13 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Support EthernetInterface resource in the emulator https://review.openstack.org/538338 | 15:14 |
| dtantsur | yep :) | 15:14 |
| TheJulia | Very very close \o/ | 15:14 |
| rpioso | Awesome | 15:14 |
| jroll | classic driver removal also nearing the finish line | 15:15 |
| TheJulia | I'm going to put a few specs that relate to items that are outstanding on this list on the priorities for this week | 15:15 |
| jroll | sounds like a good idea to me | 15:15 |
| TheJulia | That way we get some eyes on them | 15:16 |
| rloo | dtantsur: wrt the classic driver removal. are you OK if I delete the stuff that has already been done? hard to see what is left to do | 15:16 |
| dtantsur | rloo: yes please | 15:16 |
| dtantsur | I did not update the list, because IIRC TheJulia planned to move it to storyboard | 15:16 |
| dtantsur | or maybe I should do it? :) | 15:16 |
| TheJulia | dtantsur: you know it better and I've been crazy busy recently | 15:16 |
| dtantsur | actually, it's a good question: should we track it on the whiteboard or storyboard? | 15:16 |
| rloo | dtantsur: in the classic driver removal, there is a question @ L244ish | 15:17 |
| rloo | dtantsur: kill it :) | 15:17 |
| TheJulia | The preference we seem to have as a community is to track efforts on etherpads, changing to just using storyboard for that might feel a bit painful | 15:17 |
| *** derekh has quit IRC | 15:17 | |
| dtantsur | rloo: I think I even did it already :) | 15:17 |
| jroll | I'd prefer storyboard, with whiteboard as the "what got done last week / needs help this week" | 15:18 |
| rloo | dtantsur: please update the whiteboard then :D | 15:18 |
| jroll | but open to whatever | 15:18 |
| TheJulia | I remember dtantsur lightly singed the wiki | 15:18 |
| TheJulia | There may have been some smoldering afterwards :) | 15:18 |
| rloo | ++ if we are using storyboard, we might want to use it for what it was meant/good at. | 15:18 |
| rloo | dtantsur: 'fake' -> 'fake_hardware' hardware type? :) | 15:19 |
| TheJulia | Which is the effort tracking, I've personally been trying to break things up and use multiple tasks which is the way it was intended, but yeah... One step at a time | 15:19 |
| dtantsur | rloo: yep, unfortunately this is MUCH harder than just sed | 15:20 |
| TheJulia | Is everyone good to move on to priorities? | 15:20 |
| *** derekh has joined #openstack-ironic | 15:20 | |
| *** derekh has quit IRC | 15:20 | |
| dtantsur | yep | 15:20 |
| rloo | + good | 15:20 |
| jiapei | ++ | 15:20 |
| *** zhangfei has quit IRC | 15:21 | |
| TheJulia | #topic Priorities for the next week | 15:21 |
| *** openstack changes topic to "Priorities for the next week (Meeting topic: ironic)" | 15:21 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:21 |
| *** derekh has joined #openstack-ironic | 15:21 | |
| TheJulia | Line 92 | 15:21 |
| rloo | what about adding this to priorities: https://review.openstack.org/#/c/507067 | 15:21 |
| patchbot | patch 507067 - ironic - [devstack] Switch ironic to uWSGI | 15:21 |
| TheJulia | rloo: I actually have that on the list I was about to paste in.... | 15:21 |
| rloo | TheJulia: :) | 15:21 |
| *** gcb has quit IRC | 15:22 | |
| TheJulia | dtantsur: is the hw-types topic unblocked? | 15:23 |
| TheJulia | dtantsur: or are there more api-jobs that need changing? | 15:23 |
| olivierb- | TheJulia & all, I made the changes in ironic-lib and ironic-python-agent code in my own repos ? what is the (best) way to take this into account ? Just git clone/checkout properly on bifrost master or change some of the requirements.txt files. Sorry for the freshman developper questions ;-) | 15:23 |
| dtantsur | yep, but no patches open right this second | 15:23 |
| dtantsur | I mean, yes, unblocked | 15:23 |
| rloo | olivierb-: we're in the weekly ironic meeting now (you're welcome to join). Shoudl be done in 30-ish minutes. | 15:24 |
| olivierb- | will head back to home/gym and will connect later to read your advices, cheers | 15:24 |
| TheJulia | 30-ish minutes if my cat does not find the router, and swipe it from the edge of the earth into the universe.... (so far during the meeting five things have been swiped onto the floor....) | 15:25 |
| dtantsur | ENEEDATREAT | 15:25 |
| rpioso | TheJulia: Sounds like your cat doesn't like meetings :-) | 15:26 |
| TheJulia | pretty much.... | 15:26 |
| jroll | good cat | 15:26 |
| TheJulia | Anyway, I've added the power fault stuff to the list since it is in solid shape as far as I've seen thus far and does improve things, I know there was discussion earlier, so we can remove it or not. I've also placed some specs at the beginning of the list that could use some eyeballs. The virtual media/dhcp-less boot deploy interface nokia is interested in needs a minor revision, but I also feel it is in good shape. | 15:27 |
| * TheJulia feels bad for not having read the buffer from when she was on a call. | 15:27 | |
| jroll | seems like a decent list, though far too long to complete :) | 15:28 |
| TheJulia | Well, no meeting next week :) | 15:28 |
| jroll | probably not much review work either :P | 15:28 |
| rloo | TheJulia is giving us homework. heh! | 15:30 |
| TheJulia | lol | 15:30 |
| TheJulia | But we can't call it homework ;) | 15:30 |
| rloo | s/homework/ironi-ng/ | 15:31 |
| TheJulia | lol | 15:31 |
| TheJulia | I like that! | 15:31 |
| * rloo thinks we're ready to move on | 15:31 | |
| rloo | :) | 15:32 |
| TheJulia | Yup | 15:32 |
| * TheJulia lost connectivity for a minute :\ | 15:33 | |
| TheJulia | And not at the paws of the cat | 15:33 |
| TheJulia | So since everyone seems good with the list and we have no specific RFEs or discusison topics, lets jump to Open Discussion | 15:34 |
| TheJulia | #topic Open Discussion | 15:34 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:34 | |
| jroll | I have a quick thing here | 15:35 |
| jroll | this discussion happened a while back: | 15:35 |
| jroll | #link http://lists.openstack.org/pipermail/openstack-dev/2018-April/129216.html | 15:35 |
| jroll | and again today | 15:35 |
| jroll | #link http://lists.openstack.org/pipermail/openstack-dev/2018-May/130423.html | 15:35 |
| TheJulia | olivierb-: Best way for you to test your changes is to craft a custom IPA image or patch it. If your booting form pxe, then it is easy with taking apart the ramdisk and patching it | 15:35 |
| openstackgerrit | Merged openstack/ironic-inspector master: Raise KeyboardInterrupt on SIGTERM - Workaround https://review.openstack.org/563335 | 15:35 |
| jroll | tl;dr people want to get pycrypto out of global-requirements in rocky, some projects are blocking that, including pyghmi | 15:35 |
| jroll | is anyone available to switch to the cryptography library in pyghmi and coordinate that with jarrod? | 15:36 |
| *** srart has quit IRC | 15:36 | |
| TheJulia | I had someone reach out to me first thing and ask for jarrod's contact info to discuss exactly that | 15:36 |
| jroll | that was me, see the april link for his plan :) | 15:37 |
| TheJulia | if it was you this morning, my brain is a blur | 15:37 |
| jroll | oh, not this morning, no | 15:37 |
| jroll | so maybe we'll wait to see if this person and/or jarrod makes progress soon | 15:38 |
| jroll | I just want to keep the requirements team unblocked :) | 15:38 |
| TheJulia | The person that asked me seems to not be around at the moment | 15:39 |
| TheJulia | #action TheJulia to follow-up w/r/t pycrypto in pyghmi - to determine if we have someone to take that on. | 15:39 |
| dtantsur | TheJulia: it was etingof? | 15:39 |
| jroll | thanks TheJulia | 15:39 |
| TheJulia | dtantsur: well, I wasn't going to name him because I hadn't gotten confirmation back yet, but I'll follow-up | 15:39 |
| rloo | action item, action item, action item, ... :) | 15:40 |
| TheJulia | Speaking of following up things, there was the ceph tempurl thing last week, who is picking that patch up? | 15:40 |
| rloo | (for etingof I mean, cuz he isn't here) | 15:40 |
| * TheJulia is asking now so it does not become yet another action item | 15:40 | |
| TheJulia | rloo: Excellent idea! | 15:40 |
| jroll | TheJulia: it's on my list | 15:40 |
| *** erolg has quit IRC | 15:40 | |
| TheJulia | jroll: awesome! Thanks! | 15:41 |
| rloo | thx jroll! | 15:41 |
| jroll | np | 15:41 |
| mgoddard | olivierb-, TheJulia: in kayobe we use environment variables to configure the IPA repo URL & version to use. In bifrost these could be set in dib_env_vars. I'm not sure how you could use a custom ironic-lib, perhaps via requirements.txt like you say. Take a look at the ironic-agent DIB element: https://github.com/openstack/diskimage-builder/tree/master/diskimage_builder/elements/ironic-agent | 15:42 |
| TheJulia | I'd really just rip the cpio file apart and rebuild it, but I'm a little old fashioned | 15:42 |
| mgoddard | rloo: I was wondering what the status of deploy steps is. Are you planning to implement it? | 15:42 |
| rloo | mgoddard: yes, still planning on it. unless someone else wants to do it. | 15:43 |
| rloo | mgoddard: hoping to get around to it next month. | 15:43 |
| mgoddard | ok great. I'm up for helping on the coding and/or reviewing side - whatever works for you | 15:44 |
| rloo | thx mgoddard! | 15:44 |
| TheJulia | Awesome, well does anyone have anything else to bring up/discuss today? | 15:45 |
| mgoddard | zshi: how is the BIOS work going? Are there any tempest tests and documentation in progress? | 15:45 |
| mgoddard | Would be good to see tempest tests before merging the API | 15:46 |
| *** ajya has quit IRC | 15:46 | |
| *** ajya2 has joined #openstack-ironic | 15:46 | |
| mgoddard | yolanda_: ^^ | 15:46 |
| mgoddard | we can take this offline if everyone else is done here | 15:47 |
| TheJulia | mgoddard: tempest api read tests, I guess could exist easily... otherwise we don't explicitly test cleaning in the gate, although maybe this is a good route to get there | 15:47 |
| TheJulia | redfish support for bios could help us get there I guess | 15:48 |
| TheJulia | Just thinking outloud | 15:48 |
| mgoddard | TheJulia: yeah, the fake driver was built in a way that much of the BIOS code could be tested via tempest | 15:48 |
| mgoddard | i.e., just writes any requested options to the DB | 15:49 |
| TheJulia | True, and there are two sides to it | 15:49 |
| TheJulia | Well, if there is nothing else, I guess we can all get ten minutes back :) | 15:50 |
| TheJulia | well, more like 9 minutes now | 15:50 |
| TheJulia | :) | 15:50 |
| jroll | \o/ | 15:50 |
| dtantsur | :) | 15:51 |
| TheJulia | Awesome! | 15:51 |
| *** srart has joined #openstack-ironic | 15:51 | |
| TheJulia | Thanks everyone! | 15:51 |
| jroll | thanks! | 15:51 |
| dtantsur | thanks | 15:51 |
| TheJulia | #endmeeting | 15:51 |
| *** 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" | 15:51 | |
| *** marios has quit IRC | 15:51 | |
| openstack | Meeting ended Mon May 14 15:51:27 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:51 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-05-14-15.00.html | 15:51 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-05-14-15.00.txt | 15:51 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-05-14-15.00.log.html | 15:51 |
| * dtantsur can boast grabbing ID 1000 in storyboard for his pet project :D https://storyboard.openstack.org/#!/project/1000 | 15:51 | |
| TheJulia | dtantsur: nice! | 15:52 |
| *** stendulker has quit IRC | 15:59 | |
| *** hshiina has quit IRC | 16:00 | |
| *** trown is now known as trown|lunch | 16:01 | |
| *** mgoddard has quit IRC | 16:03 | |
| *** ajya2 has quit IRC | 16:06 | |
| *** ajya has joined #openstack-ironic | 16:06 | |
| *** ijw has quit IRC | 16:08 | |
| *** lucasagomes is now known as lucas-afk | 16:10 | |
| *** gyee has joined #openstack-ironic | 16:13 | |
| *** baha has quit IRC | 16:17 | |
| TheJulia | Running into town to get some things to fix a water leak, I should be back in an hour or so | 16:19 |
| *** pcaruana has joined #openstack-ironic | 16:20 | |
| etingof | TheJulia, dtantsur rloo sorry folks, I got a minor hiccup right before the meeting | 16:30 |
| rloo | etingof: no worries. Are you OK to work on the pycrypto/pyghmi thing? | 16:30 |
| * sambetts ----> home | 16:36 | |
| sambetts | night all o/ | 16:36 |
| *** sambetts is now known as sambetts|afk | 16:36 | |
| etingof | rloo, sure, I am delighted ;) | 16:40 |
| rpioso | sambetts|afk: Good night | 16:40 |
| rloo | etingof: thank you so much. And I am so glad we can make you feel delighted! :) | 16:41 |
| *** dprince has quit IRC | 16:44 | |
| *** rbartal has quit IRC | 16:49 | |
| *** baha has joined #openstack-ironic | 16:55 | |
| *** dougsz has quit IRC | 16:57 | |
| *** AlexeyAbashkin has quit IRC | 16:57 | |
| *** derekh has quit IRC | 17:00 | |
| *** trown|lunch is now known as trown | 17:00 | |
| *** serlex has quit IRC | 17:05 | |
| *** dprince has joined #openstack-ironic | 17:06 | |
| *** hemna_ has joined #openstack-ironic | 17:07 | |
| *** hemna_ has quit IRC | 17:07 | |
| *** hemna_ has joined #openstack-ironic | 17:07 | |
| *** e0ne has quit IRC | 17:10 | |
| *** tesseract has quit IRC | 17:11 | |
| *** dtantsur is now known as dtantsur|afk | 17:14 | |
| dtantsur|afk | see you | 17:14 |
| jroll | night dtantsur|afk :) | 17:16 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: SNMPv3 security features added to the `snmp` driver https://review.openstack.org/547999 | 17:17 |
| *** efried has quit IRC | 17:19 | |
| *** efried has joined #openstack-ironic | 17:19 | |
| *** baoli has quit IRC | 17:21 | |
| *** etingof is now known as etingof|afk | 17:21 | |
| *** baoli has joined #openstack-ironic | 17:26 | |
| *** fragatina has joined #openstack-ironic | 17:45 | |
| *** fragatina has quit IRC | 17:46 | |
| *** fragatina has joined #openstack-ironic | 17:47 | |
| *** fragatina has quit IRC | 17:47 | |
| *** fragatina has joined #openstack-ironic | 17:48 | |
| *** S4ren has quit IRC | 17:48 | |
| *** r-daneel_ has joined #openstack-ironic | 18:00 | |
| *** r-daneel has quit IRC | 18:01 | |
| *** r-daneel_ is now known as r-daneel | 18:01 | |
| TheJulia | etingof|afk: awesome, thanks! Have a wonderful evening | 18:02 |
| *** anupn has joined #openstack-ironic | 18:08 | |
| *** harlowja has joined #openstack-ironic | 18:12 | |
| *** e0ne has joined #openstack-ironic | 18:18 | |
| *** ajya has quit IRC | 18:23 | |
| olivierb- | TheJulia, like you I like to work directly on gzipped,cpioed initramfs, however I just discovered that what is inside the ipa ramdisk are .pyo files not .py :-( would it be an issue replacing the .pyo by its patched .py version without anything else ? | 18:30 |
| *** hamzy has quit IRC | 18:30 | |
| TheJulia | olivierb-: doh :( | 18:30 |
| TheJulia | olivierb-: maybe use a coreos ramdisk? | 18:30 |
| TheJulia | we don't strip the objects out of those | 18:30 |
| *** hamzy has joined #openstack-ironic | 18:33 | |
| anupn | oliverb-: you can get the .py files by setting export PYOPTIMIZE_TINYIPA=false | 18:37 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client master: Support passing manage_boot argument in Python API https://review.openstack.org/339479 | 18:37 |
| anupn | olivierb-: you can get the .py files by setting export PYOPTIMIZE_TINYIPA=false | 18:37 |
| olivierb- | anupn thx however I am not rebuillding the ramdisk, i am decompressing it and un-archiving it to patch it and reconstruct | 18:39 |
| *** mgoddard has joined #openstack-ironic | 18:40 | |
| *** etingof|afk is now known as etingof | 18:40 | |
| *** Guest16323 is now known as mgagne | 18:40 | |
| *** mgagne has joined #openstack-ironic | 18:40 | |
| *** mgoddard has quit IRC | 18:48 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix E501 errors https://review.openstack.org/567028 | 18:54 |
| *** trown is now known as trown|brb | 18:54 | |
| *** r-daneel has quit IRC | 18:59 | |
| *** r-daneel_ has joined #openstack-ironic | 18:59 | |
| *** r-daneel_ is now known as r-daneel | 19:01 | |
| *** mrunge has quit IRC | 19:21 | |
| *** Goneri has quit IRC | 19:21 | |
| openstackgerrit | chandra shekar proposed openstack/ironic-specs master: Added new spec for L3 based Ironic deployment https://review.openstack.org/543936 | 19:22 |
| *** dprince has quit IRC | 19:23 | |
| *** mgoddard has joined #openstack-ironic | 19:25 | |
| olivierb- | are there some checksum mechanism implemented because after regenerating the ipa.initramfs the boot process is stuck in booting the kernel ? I just gunzipped + cpio extract, cpio archive + gzip without changing any contents and same issue | 19:28 |
| olivierb- | (just to try without any changeds) | 19:29 |
| *** trown|brb is now known as trown | 19:31 | |
| olivierb- | may be TheJulia knows ^^^ | 19:36 |
| olivierb- | https://gist.github.com/obourdon/3c1208853c03316b041a48e37ec9df65 is what I used without being able to get a successfull boot | 19:38 |
| *** dprince has joined #openstack-ironic | 19:39 | |
| TheJulia | olivierb-: add "--format='newc'" to your cpio command | 19:40 |
| *** mrunge has joined #openstack-ironic | 19:41 | |
| olivierb- | TheJulia thx just rebuilt ipa and restarted provisioning to see | 19:49 |
| olivierb- | boot in progress | 19:50 |
| TheJulia | no problem, it is slightly obscure knowledge | 19:50 |
| TheJulia | :) | 19:50 |
| olivierb- | horray, working now thanks | 19:50 |
| olivierb- | final question for today, are the console logs of 2nd IPA pass (writing qcow2 image to HDD) retrieved+stored somewhere on bifrost master node ? | 19:58 |
| openstackgerrit | Julia Kreger proposed openstack/python-ironicclient master: Wire in header microversion into negotiation https://review.openstack.org/558027 | 19:58 |
| *** athomas has quit IRC | 20:00 | |
| *** dprince has quit IRC | 20:09 | |
| *** efried has quit IRC | 20:17 | |
| *** mgoddard has quit IRC | 20:26 | |
| *** racedo has quit IRC | 20:27 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added https://review.openstack.org/523879 | 20:32 |
| *** baoli has quit IRC | 20:37 | |
| *** baoli has joined #openstack-ironic | 20:37 | |
| *** baoli has quit IRC | 20:41 | |
| *** pcaruana has quit IRC | 20:43 | |
| *** jcoufal_ has quit IRC | 20:43 | |
| *** livelace has quit IRC | 20:54 | |
| *** Sukhdev has joined #openstack-ironic | 20:55 | |
| *** racedo has joined #openstack-ironic | 20:55 | |
| TheJulia | olivierb-: console, no. ironic ships its logging data up to the conductor... /var/lib/ironic... I think | 20:56 |
| TheJulia | olivierb-: did your patch work? | 20:57 |
| olivierb- | nope the call to the new code is made but then the machine fails to restart, I think there is an uncaught exception but need to debug it further. will do this tomorrow early morning, time to go to sleep now ;-) thanks for everything TheJulia | 20:59 |
| *** trown is now known as trown|outtypewww | 21:00 | |
| *** hemna_ has quit IRC | 21:04 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Switch to none auth for standalone mode https://review.openstack.org/359061 | 21:06 |
| *** r-daneel has quit IRC | 21:07 | |
| *** r-daneel has joined #openstack-ironic | 21:07 | |
| *** e0ne has quit IRC | 21:16 | |
| *** bnemec has quit IRC | 21:17 | |
| *** bnemec has joined #openstack-ironic | 21:18 | |
| *** baha has quit IRC | 21:31 | |
| *** mjturek has quit IRC | 21:40 | |
| *** cjloader has quit IRC | 21:50 | |
| *** cjloader_ has joined #openstack-ironic | 21:50 | |
| *** rcernin has joined #openstack-ironic | 21:53 | |
| *** cjloader has joined #openstack-ironic | 21:55 | |
| *** cjloader_ has quit IRC | 21:55 | |
| *** harlowja has quit IRC | 21:56 | |
| *** cjloader has quit IRC | 21:59 | |
| *** rh-jelabarre has quit IRC | 22:03 | |
| *** bfournie has quit IRC | 22:06 | |
| *** jistr has quit IRC | 22:07 | |
| *** etingof has quit IRC | 22:08 | |
| *** jistr has joined #openstack-ironic | 22:10 | |
| *** threestrands has joined #openstack-ironic | 22:16 | |
| *** etingof has joined #openstack-ironic | 22:21 | |
| *** hemna_ has joined #openstack-ironic | 22:29 | |
| *** bfournie has joined #openstack-ironic | 22:46 | |
| *** r-daneel has quit IRC | 22:54 | |
| *** ElCoyote_ has quit IRC | 22:59 | |
| *** baoli has joined #openstack-ironic | 23:00 | |
| *** baoli has quit IRC | 23:04 | |
| *** rajinir has quit IRC | 23:19 | |
| *** rpioso is now known as rpioso|afk | 23:26 | |
| *** oanson has quit IRC | 23:37 | |
| *** oanson has joined #openstack-ironic | 23:48 | |
| *** srart has quit IRC | 23:52 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
