| *** Goneri has quit IRC | 00:09 | |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Document http_basic auth strategy https://review.opendev.org/735482 | 01:05 |
|---|---|---|
| *** Qianbiao has joined #openstack-ironic | 02:20 | |
| *** jdandrea has joined #openstack-ironic | 02:21 | |
| *** mkrai has joined #openstack-ironic | 02:51 | |
| *** yaawang has joined #openstack-ironic | 02:52 | |
| *** k_mouza has joined #openstack-ironic | 03:11 | |
| *** k_mouza has quit IRC | 03:16 | |
| *** rh-jelabarre has joined #openstack-ironic | 03:28 | |
| *** rh-jelabarre has quit IRC | 03:28 | |
| *** rh-jelabarre has joined #openstack-ironic | 03:29 | |
| *** mkrai has quit IRC | 03:33 | |
| *** mkrai has joined #openstack-ironic | 03:35 | |
| *** ociuhandu has joined #openstack-ironic | 03:59 | |
| *** ociuhandu has quit IRC | 04:04 | |
| *** Habib-A52 has quit IRC | 04:10 | |
| *** jdandrea has quit IRC | 04:26 | |
| *** zaneb has quit IRC | 04:28 | |
| *** jdandrea has joined #openstack-ironic | 04:28 | |
| *** zaneb has joined #openstack-ironic | 04:31 | |
| *** jdandrea has quit IRC | 04:33 | |
| *** ricolin has joined #openstack-ironic | 04:42 | |
| *** jtomasek has joined #openstack-ironic | 04:49 | |
| *** rh-jelabarre has quit IRC | 04:57 | |
| *** diurnalist has joined #openstack-ironic | 06:04 | |
| *** irclogbot_3 has quit IRC | 06:05 | |
| *** irclogbot_1 has joined #openstack-ironic | 06:06 | |
| *** diurnalist has quit IRC | 06:08 | |
| *** k_mouza has joined #openstack-ironic | 06:12 | |
| arne_wiebalck | Good morning, ironic! | 06:13 |
| *** k_mouza has quit IRC | 06:16 | |
| *** priteau has joined #openstack-ironic | 06:34 | |
| *** iurygregory has joined #openstack-ironic | 07:11 | |
| iurygregory | good morning Ironic o/ | 07:12 |
| *** mkrai has quit IRC | 07:16 | |
| *** mkrai has joined #openstack-ironic | 07:27 | |
| *** yolanda has joined #openstack-ironic | 07:40 | |
| -openstackstatus- NOTICE: uWSGI made a new release that breaks devstack, please refrain from rechecking until a devstack fix is merged. | 07:43 | |
| *** dougsz has joined #openstack-ironic | 07:44 | |
| iurygregory | yay for mondays! | 07:44 |
| *** eliaswimmer has joined #openstack-ironic | 07:46 | |
| *** dtantsur|afk is now known as dtantsur | 07:51 | |
| dtantsur | morning ironic | 07:51 |
| dtantsur | oh, sweeeet! | 07:51 |
| iurygregory | morning dtantsur | 07:52 |
| iurygregory | hehehe | 07:52 |
| iurygregory | it's just to say "it's monday" | 07:52 |
| *** mkrai has quit IRC | 07:57 | |
| dtantsur | I was bored on Sunday, so I did this https://review.opendev.org/735461 and surprisingly it seems to work \o/ | 07:58 |
| patchbot | patch 735461 - bifrost - Migrate pip_install.yml to a new role bifrost-pip-... - 3 patch sets | 07:58 |
| arne_wiebalck | Hey dtantsur, totally agree. In fact, what I did in prod now is to reduce the cleanup_period to 3600. https://usercontent.irccloud-cdn.com/file/97eNrhYy/Screenshot%20from%202020-06-15%2008-54-03.png | 07:59 |
| dtantsur | that's your mitigation | 08:00 |
| dtantsur | :) | 08:00 |
| arne_wiebalck | The first reduction (10.06) is pagination, the second (12.06) is the change of cleanup_period. | 08:00 |
| *** rpittau|afk is now known as rpittau | 08:00 | |
| *** lucasagomes has joined #openstack-ironic | 08:00 | |
| rpittau | good morning ironic! o/ | 08:00 |
| dtantsur | and a real fix would probably be two-part: pagination + leader election | 08:00 |
| arne_wiebalck | dtantsur: right | 08:01 |
| dtantsur | luckily, we already have leader election code in inspector, only need to wire that in | 08:01 |
| arne_wiebalck | dtantsur: with 12 inspectors and 3600 the cleanup is around every 10 mins still | 08:01 |
| dtantsur | arne_wiebalck: could you maybe propose your pagination changes? and maybe document the scaling issue and the workaround? | 08:01 |
| arne_wiebalck | dtantsur: the pagination is max_limit in ironic.conf | 08:02 |
| dtantsur | ah | 08:02 |
| arne_wiebalck | dtantsur: we had a too high value, the Ironic default is good | 08:02 |
| arne_wiebalck | dtantsur: but we had 12 inspectors asking for all nodes in one chunk every 60 secs | 08:03 |
| dtantsur | that sounds crazy indeed, sorry for this design :) | 08:03 |
| arne_wiebalck | dtantsur: so, we paginated, then decreased the frequency | 08:03 |
| dtantsur | it was before we had notifications, maybe we should employ them.. | 08:03 |
| arne_wiebalck | :-D | 08:03 |
| dtantsur | maybe we need direct ironic->inspector notifications? | 08:03 |
| arne_wiebalck | it may be crazy to have 12 inspectors | 08:03 |
| iurygregory | good morning arne_wiebalck and rpittau o/ | 08:04 |
| dtantsur | it does sound a bit too much to me :) | 08:04 |
| rpittau | hey iurygregory :) | 08:04 |
| arne_wiebalck | but so far all my controllers are identical | 08:04 |
| dtantsur | but we should support all cases. with active node introspection you may actually need some scale. | 08:04 |
| arne_wiebalck | operationally, it is just easier to have controllers which look all the same | 08:04 |
| dtantsur | arne_wiebalck: as you now realize, you're the first to try ironic-inspector at really high scale :) | 08:04 |
| dtantsur | yeah, I see | 08:04 |
| arne_wiebalck | rather than splitting api, conductor, and inspector | 08:04 |
| arne_wiebalck | which I could do, of course | 08:05 |
| dtantsur | I think we need to do the leader election instead | 08:05 |
| arne_wiebalck | dtantsur: re inspector at scale, the fun fact is that this is an idle inspector | 08:05 |
| arne_wiebalck | dtantsur: this is just the regular cleanup bringing our DB down | 08:06 |
| dtantsur | well, yeah, idle, but with N000 nodes to track :) | 08:06 |
| arne_wiebalck | exactly | 08:06 |
| dtantsur | I certainly don't disagree we should fix that | 08:06 |
| * arne_wiebalck sent the DB colleqgues on a quest to fix the slowness of the Ironic DB | 08:06 | |
| dtantsur | I'm a bit careful about "just disable the task" as a solution | 08:06 |
| arne_wiebalck | I agree | 08:06 |
| dtantsur | I can look into the leader election, but I've been crazily busy recently. Help welcome, it shouldn't be a difficult patch. | 08:07 |
| arne_wiebalck | this is why I started by reducing the frequency and asked on the review for potential side effects | 08:07 |
| dtantsur | the side effect: be careful when recreating nodes | 08:07 |
| arne_wiebalck | no urgency, we can put this on our list and maybe gudrutis2 is interested :) | 08:08 |
| arne_wiebalck | recreating nodes? | 08:08 |
| dtantsur | well, in TripleO folks used to have a habit of deleting nodes and creating them again on any problem | 08:09 |
| arne_wiebalck | oh, I see | 08:09 |
| dtantsur | that could cause an issue because ironic-inspector would still assume the MAC address belonging to another node | 08:09 |
| arne_wiebalck | oh, yeah | 08:10 |
| *** ociuhandu has joined #openstack-ironic | 08:10 | |
| *** Lucas_Gray has joined #openstack-ironic | 08:11 | |
| *** mkrai has joined #openstack-ironic | 08:27 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: [WIP] Leader election for periodic sync with ironic https://review.opendev.org/735527 | 08:35 |
| dtantsur | arne_wiebalck: seems simpler than I thought ^^ | 08:35 |
| dtantsur | (completely untested, would be curious to try) | 08:35 |
| arne_wiebalck | wow, nice, we will try to test this, thanks dtantsur ! | 08:38 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: [WIP] Leader election for periodic sync with ironic https://review.opendev.org/735527 | 08:40 |
| dtantsur | arne_wiebalck: updated ^^. you'll need to configure a coordination backend if not already. | 08:40 |
| dtantsur | may I get another +2 on https://review.opendev.org/#/c/733111/ please? | 08:44 |
| patchbot | patch 733111 - bifrost - Get rid of the shade dependency - 5 patch sets | 08:44 |
| dtantsur | I know it's a bit involved, but shade has been deprecated for a while.. | 08:45 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Migrate pip_install.yml to a new role bifrost-pip-install https://review.opendev.org/735461 | 08:47 |
| *** Lucas_Gray has quit IRC | 08:47 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Disable automatic updates in systemd images https://review.opendev.org/735335 | 08:50 |
| iurygregory | dtantsur, looking now =) | 08:51 |
| *** uzumaki has joined #openstack-ironic | 08:54 | |
| arne_wiebalck | dtantsur: noted, thanks | 08:54 |
| uzumaki | hello ironic! o/ | 08:54 |
| dtantsur | hi uzumaki, how's your Monday? | 08:54 |
| uzumaki | very very slow.. I think this WFH is going to make me lazier than a snail | 08:55 |
| uzumaki | how about you? | 08:55 |
| iurygregory | hey uzumaki o/ | 08:55 |
| uzumaki | hey iurygregory \o/ | 08:55 |
| *** ociuhandu has quit IRC | 08:58 | |
| *** ociuhandu has joined #openstack-ironic | 08:59 | |
| openstackgerrit | Merged openstack/bifrost stable/ussuri: Install packages all at once instead of looping over them https://review.opendev.org/735456 | 09:00 |
| *** bonguardo has joined #openstack-ironic | 09:00 | |
| dtantsur | uzumaki: also very slow, although not so terrible (yet?). and I'm used to WFH :) | 09:00 |
| *** k_mouza has joined #openstack-ironic | 09:02 | |
| uzumaki | well, you're lucky | 09:02 |
| openstackgerrit | Merged openstack/bifrost master: Allow changing 'update' and 'force' when cloning repos https://review.opendev.org/735480 | 09:02 |
| dtantsur | probably :) | 09:02 |
| *** bonguardo has quit IRC | 09:10 | |
| *** priteau has quit IRC | 09:11 | |
| openstackgerrit | Verification of a change to openstack/ironic-lib failed: Use unittest.mock instead of mock https://review.opendev.org/734352 | 09:12 |
| *** ociuhandu has quit IRC | 09:13 | |
| openstackgerrit | Merged openstack/bifrost master: Remove the ansible_pkg_mgr hack https://review.opendev.org/735479 | 09:15 |
| openstackgerrit | Merged openstack/bifrost master: Update ansible versions in README of the roles https://review.opendev.org/735459 | 09:15 |
| openstackgerrit | Merged openstack/bifrost master: Remove the workaround for PEP517 issues https://review.opendev.org/735457 | 09:15 |
| *** Lucas_Gray has joined #openstack-ironic | 09:20 | |
| *** priteau has joined #openstack-ironic | 09:21 | |
| *** Wryhder has joined #openstack-ironic | 09:22 | |
| *** Lucas_Gray has quit IRC | 09:24 | |
| *** Wryhder is now known as Lucas_Gray | 09:24 | |
| openstackgerrit | Merged openstack/bifrost master: Get rid of the shade dependency https://review.opendev.org/733111 | 09:25 |
| openstackgerrit | Merged openstack/ironic master: Use assertCountEqual instead of assertItemsEqual https://review.opendev.org/735127 | 09:28 |
| *** ociuhandu has joined #openstack-ironic | 09:36 | |
| *** ociuhandu has quit IRC | 09:39 | |
| *** uzumaki has quit IRC | 09:39 | |
| *** ociuhandu has joined #openstack-ironic | 09:39 | |
| openstackgerrit | Merged openstack/bifrost master: Convert with_items to loop https://review.opendev.org/733847 | 09:40 |
| *** Lucas_Gray has quit IRC | 09:43 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: devstack: Use uwsgi binary from path https://review.opendev.org/735547 | 09:44 |
| dtantsur | This ^^^ presumably fixes the CI together with https://review.opendev.org/#/c/731159/ | 09:44 |
| patchbot | patch 731159 - ironic - devstack: Use uwsgi binary from path - 1 patch set | 09:44 |
| iurygregory | dtantsur, ack | 09:45 |
| *** Lucas_Gray has joined #openstack-ironic | 09:46 | |
| dtantsur | may I also get some reviews on https://review.opendev.org/733985 and https://review.opendev.org/#/c/716216/ please? they're blocking the rest of the deploy steps work | 09:49 |
| patchbot | patch 733985 - ironic - agent: poll long-running commands till completion - 3 patch sets | 09:49 |
| patchbot | patch 716216 - ironic-python-agent - Add a deploy step for writing an image - 10 patch sets | 09:49 |
| dtantsur | mgoddard: if you have time in the near future ^^ | 09:50 |
| *** uzumaki has joined #openstack-ironic | 09:51 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Disable automatic updates in dnf-based systems https://review.opendev.org/735335 | 09:53 |
| mgoddard | dtantsur: time? what's that? :) | 09:53 |
| iurygregory | it's something relative | 09:55 |
| uzumaki | oh, we're discussing time again \o/ | 09:55 |
| uzumaki | <enters the microwave> | 09:55 |
| iurygregory | steins;gate time | 09:56 |
| *** k_mouza has quit IRC | 09:59 | |
| *** rpittau is now known as rpittau|bbl | 10:03 | |
| dtantsur | :D | 10:06 |
| *** Lucas_Gray has quit IRC | 10:15 | |
| *** Lucas_Gray has joined #openstack-ironic | 10:17 | |
| *** k_mouza has joined #openstack-ironic | 10:19 | |
| *** ociuhandu has quit IRC | 10:27 | |
| *** ociuhandu has joined #openstack-ironic | 10:29 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: devstack: Use uwsgi binary from path https://review.opendev.org/735547 | 10:31 |
| *** mkrai has quit IRC | 10:33 | |
| *** mkrai_ has joined #openstack-ironic | 10:33 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Migrate pip_install.yml to a new role bifrost-pip-install https://review.opendev.org/735461 | 10:35 |
| *** derekh has joined #openstack-ironic | 10:35 | |
| *** ociuhandu has quit IRC | 10:41 | |
| *** rcernin has joined #openstack-ironic | 10:44 | |
| *** rcernin has quit IRC | 10:54 | |
| *** ociuhandu has joined #openstack-ironic | 10:55 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Only install proliantutils if ilo is enabled; support idrac https://review.opendev.org/735554 | 10:56 |
| *** rh-jelabarre has joined #openstack-ironic | 11:03 | |
| *** rh-jelabarre has quit IRC | 11:03 | |
| *** rh-jelabarre has joined #openstack-ironic | 11:13 | |
| *** ociuhandu has quit IRC | 11:14 | |
| *** ociuhandu has joined #openstack-ironic | 11:16 | |
| *** ociuhandu has quit IRC | 11:20 | |
| *** derekh has quit IRC | 11:31 | |
| *** derekh has joined #openstack-ironic | 11:37 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Only install proliantutils if ilo is enabled; support idrac https://review.opendev.org/735554 | 11:51 |
| *** ociuhandu has joined #openstack-ironic | 11:58 | |
| *** rpittau|bbl is now known as rpittau | 12:04 | |
| *** mkrai_ has quit IRC | 12:07 | |
| rpittau | oh wow all ci jobs are actually broken because of uwsgi :/ | 12:09 |
| dtantsur | yep | 12:10 |
| dtantsur | and we may need to temp-disable grenade to fix it.. | 12:11 |
| rpittau | yeah, I was reading on openstack-qa :/ | 12:11 |
| dtantsur | or cap uwsgi on stable branches | 12:11 |
| dtantsur | this ^^ may be easier | 12:12 |
| *** dougsz has quit IRC | 12:13 | |
| iurygregory | monday ... | 12:25 |
| iurygregory | things are crazy today | 12:25 |
| dtantsur | mmm, we cannot cap uwsgi, devstack is switching to installing it from system packages | 12:26 |
| dtantsur | jesus | 12:26 |
| dtantsur | I don't want to play the whole dance around disabling grenade on all branches | 12:26 |
| * dtantsur wants to switch most of the CI to bifrost now | 12:27 | |
| *** dougsz has joined #openstack-ironic | 12:28 | |
| *** derekh has quit IRC | 12:28 | |
| *** derekh has joined #openstack-ironic | 12:29 | |
| dtantsur | okay, if we backport the fix, and start landing it from the oldest branch, it may work | 12:29 |
| dtantsur | I mean, https://review.opendev.org/#/c/731159/ | 12:30 |
| patchbot | patch 731159 - ironic - devstack: Use uwsgi binary from path - 1 patch set | 12:30 |
| rpittau | if it stops failing for other reasons..... | 12:31 |
| iurygregory | ^ this | 12:32 |
| rpittau | oh that's actually grenade /facepalm | 12:32 |
| iurygregory | grenade <3 | 12:33 |
| dtantsur | rpittau: yep, I'm afraid grenade will not allow us to Just Merge It | 12:33 |
| rpittau | so I guess we'll have to disable grenade at least on master ;/ | 12:33 |
| * dtantsur updates the whiteboard | 12:33 | |
| iurygregory | "Red" | 12:33 |
| dtantsur | rpittau: we can try landing the fix from the oldest branch | 12:33 |
| rpittau | mmm ok | 12:34 |
| *** Wryhder has joined #openstack-ironic | 12:35 | |
| dtantsur | although.. it is blocked by fixing devstack | 12:36 |
| *** Lucas_Gray has quit IRC | 12:36 | |
| *** Wryhder is now known as Lucas_Gray | 12:36 | |
| dtantsur | so if we want to continue landing changes now, we do need to disable grenade | 12:36 |
| dtantsur | maybe only on master to limit the mess | 12:36 |
| iurygregory | ++ | 12:37 |
| rpittau | let's do it, unless we can give the devstack fix some more time | 12:37 |
| *** adrianc has quit IRC | 12:37 | |
| *** adrianc has joined #openstack-ironic | 12:38 | |
| dtantsur | k, updating | 12:38 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: devstack: Use uwsgi binary from path; temporary disable grenade https://review.opendev.org/731159 | 12:39 |
| dtantsur | here goes | 12:39 |
| *** Lucas_Gray has quit IRC | 12:41 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: CI: make grenade voting again https://review.opendev.org/735577 | 12:43 |
| *** Lucas_Gray has joined #openstack-ironic | 12:43 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: devstack: Use uwsgi binary from path; temporary disable grenade https://review.opendev.org/735547 | 12:45 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: CI: make grenade voting again https://review.opendev.org/735579 | 12:46 |
| *** Lucas_Gray has quit IRC | 12:50 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: CI: clean up non-votings jobs https://review.opendev.org/735581 | 12:51 |
| *** Lucas_Gray has joined #openstack-ironic | 12:52 | |
| *** priteau has quit IRC | 12:54 | |
| *** priteau has joined #openstack-ironic | 12:55 | |
| TheJulia | good morning | 13:07 |
| rpittau | good morning TheJulia :) | 13:07 |
| dtantsur | morning TheJulia | 13:08 |
| iurygregory | good morning TheJulia =) | 13:12 |
| *** rloo has joined #openstack-ironic | 13:29 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Agent power interface https://review.opendev.org/735591 | 13:38 |
| *** tzumainn has joined #openstack-ironic | 13:39 | |
| dtantsur | TheJulia: this is how I see it ^^ | 13:39 |
| *** Lucas_Gray has quit IRC | 13:39 | |
| *** kaiokmo has joined #openstack-ironic | 13:42 | |
| *** jdandrea has joined #openstack-ironic | 13:49 | |
| TheJulia | ok | 13:54 |
| TheJulia | I'll look when I'm not on meetings/calls | 13:54 |
| *** Goneri has joined #openstack-ironic | 13:57 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Allow setting other fields on discovery https://review.opendev.org/735594 | 14:07 |
| *** priteau has quit IRC | 14:12 | |
| openstackgerrit | James Denton proposed openstack/networking-generic-switch master: VLAN not tagged on every switchs https://review.opendev.org/557051 | 14:13 |
| TheJulia | Chris raised an interesting question on https://review.opendev.org/#/c/733716 re pxe_enabled. Do we have any thoughts on that? I'm hesitant since I know there was the whole initial thing about network interfaces being oem specific and optional in early stuffs | 14:19 |
| patchbot | patch 733716 - ironic - WIP Add a driver compatability matrix - 5 patch sets | 14:19 |
| *** Lucas_Gray has joined #openstack-ironic | 14:29 | |
| *** cdearborn has joined #openstack-ironic | 14:40 | |
| rpioso | Good morning, ironic. | 14:46 |
| rpittau | hey rpioso :) | 14:48 |
| rpioso | rpittau: o/ | 14:49 |
| *** diurnalist has joined #openstack-ironic | 14:50 | |
| dtantsur | TheJulia: commented. tl;dr it probably needs a separate feature | 14:52 |
| TheJulia | likely | 14:53 |
| dtantsur | folks, we need https://review.opendev.org/#/c/731159/ to unblock the CI (for now) | 14:55 |
| patchbot | patch 731159 - ironic - devstack: Use uwsgi binary from path; temporary di... - 2 patch sets | 14:55 |
| rpittau | dtantsur: done, I thought I put the +2 already | 14:56 |
| TheJulia | approved | 14:57 |
| TheJulia | *sigh* | 14:57 |
| dtantsur | yeah, le sigh | 14:57 |
| dtantsur | similarly, https://review.opendev.org/735547 | 14:57 |
| patchbot | patch 735547 - ironic-inspector - devstack: Use uwsgi binary from path; temporary di... - 3 patch sets | 14:57 |
| *** kaifeng has joined #openstack-ironic | 14:58 | |
| TheJulia | #startmeeting ironic | 15:00 |
| openstack | Meeting started Mon Jun 15 15:00:15 2020 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 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| TheJulia | Good morning everyone! | 15:00 |
| dtantsur | o/ | 15:00 |
| iurygregory | o/ | 15:00 |
| TheJulia | Time for another meeting full of Ironic! | 15:00 |
| ajya | \o | 15:00 |
| kaifeng | o/ | 15:00 |
| erbarr | o/ | 15:00 |
| arne_wiebalck | o/ | 15:00 |
| cdearborn | \o/ | 15:00 |
| rpittau | o/ | 15:00 |
| mgoddard | \o | 15:00 |
| rpioso | o/ | 15:00 |
| Qianbiao | o/ | 15:01 |
| TheJulia | Our agenda this week can be found on the wiki | 15:01 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| TheJulia | #topic Announcements / Reminder | 15:01 |
| *** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)" | 15:01 | |
| TheJulia | Looks like we have two items on the agenda | 15:01 |
| TheJulia | The first is the priorities for the Victoria cycle are up for review. Please review/comment as soon as reasonably possible. | 15:02 |
| TheJulia | #link https://review.opendev.org/#/c/720100/ | 15:02 |
| patchbot | patch 720100 - ironic-specs - Victoria Cycle Priorities - 8 patch sets | 15:02 |
| TheJulia | The second item is regarding a meeting for partitioning on Wednesday | 15:02 |
| TheJulia | Looks like dtantsur is hosting a call at 2 PM UTC on Wednesday | 15:02 |
| TheJulia | #link http://lists.openstack.org/pipermail/openstack-discuss/2020-June/015422.html | 15:02 |
| dtantsur | yep, note the s/July/June/ | 15:03 |
| TheJulia | Does anyone have anything to announce or remind us of? | 15:03 |
| TheJulia | dtantsur: No time machines available ? :) | 15:03 |
| dtantsur | two weeks till our first coordinates intermediate release? | 15:03 |
| dtantsur | * coordinated | 15:03 |
| dtantsur | or do I recall the dates incorrectly? | 15:03 |
| TheJulia | Yeah, roughly | 15:04 |
| TheJulia | Week of June 29th | 15:04 |
| rloo | o/ | 15:04 |
| TheJulia | Hence why I keep pushing for reviews to occur :) | 15:04 |
| dtantsur | aha, cool. if so, we may want to start slowing down the new feature stream (not that we have many) and concentrate on the sprint goals | 15:04 |
| dtantsur | and I need to make bifrost-based upgrades work.. | 15:05 |
| TheJulia | a good portion of that is basically code review at this point | 15:05 |
| rajinir | o/ | 15:05 |
| TheJulia | dtantsur: if that ends up being sprint 2, I don't think that specifically is the end of the world | 15:06 |
| TheJulia | \o rajinir | 15:06 |
| dtantsur | yep. but I like hacking on bifrost =^_^= | 15:06 |
| TheJulia | i think it is largely going to be input data based with maybe a slightly different scenario test. We'll figure it out :) | 15:07 |
| TheJulia | Anyway, we had no action items from our last meeting, so I believe since we've not ratified the priorites we can proceed to review priorites? | 15:08 |
| dtantsur | it would be good to have status updates on the whiteboard | 15:08 |
| dtantsur | even if there are no formally approved priorities | 15:08 |
| TheJulia | Seems reasonable | 15:09 |
| TheJulia | #topic Review subteam status reports | 15:09 |
| *** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:09 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:09 |
| TheJulia | Starting around line 253 | 15:09 |
| dtantsur | iurygregory: could you update the DHCP-less section? | 15:10 |
| iurygregory | dtantsur, doing now =) | 15:10 |
| dtantsur | great :) | 15:10 |
| TheJulia | arne_wiebalck: question for you on line 284 | 15:10 |
| arne_wiebalck | TheJulia: right, there is the OOM issue and the inspector issue | 15:11 |
| *** zaneb has quit IRC | 15:11 | |
| arne_wiebalck | TheJulia both are scaling issues | 15:11 |
| TheJulia | arne_wiebalck: yeah, we need to work those out | 15:11 |
| arne_wiebalck | TheJulia: shall we keep a "scaling" topic or address these individually? | 15:11 |
| TheJulia | I think we should scale back the topic | 15:12 |
| TheJulia | For next week :) | 15:12 |
| arne_wiebalck | 👍 | 15:12 |
| kaifeng | arne_wiebalck: looks like you are using non-standalone inspector already | 15:12 |
| TheJulia | Looks like we just need people to do a final review on sig whitepaper | 15:13 |
| arne_wiebalck | kaifeng: what is a non-standalone inspector? | 15:13 |
| TheJulia | arne_wiebalck: did you have a deadline date in mind? | 15:13 |
| arne_wiebalck | TheJulia: I was thinking end of this week | 15:13 |
| arne_wiebalck | TheJulia: to give everyone a chance to read once more | 15:13 |
| arne_wiebalck | TheJulia: and then file it with the foundation | 15:14 |
| dtantsur | arne_wiebalck: API/conductor split for inspector | 15:14 |
| arne_wiebalck | TheJulia: or is this too long? | 15:14 |
| TheJulia | arne_wiebalck: maybe thursday? | 15:14 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add L3 boot section to the docs https://review.opendev.org/689844 | 15:14 |
| arne_wiebalck | TheJulia: ok | 15:14 |
| dtantsur | which I guess you needed to have multi-node inspector | 15:14 |
| TheJulia | arne_wiebalck: just thinking that way you can send it on friday :) | 15:14 |
| arne_wiebalck | TheJulia: ok | 15:14 |
| arne_wiebalck | dtantsur: kaifeng : all our controllers run API/conductor/inspector | 15:14 |
| dtantsur | arne_wiebalck: oh, you don't split API/conductor for inspector? | 15:15 |
| TheJulia | And their config doesn't fundimentally need the sync process at all | 15:15 |
| TheJulia | the tl;dr is the inspector sync pounds ironic's API. | 15:15 |
| kaifeng | dtantsur: excelent! | 15:15 |
| dtantsur | well, we cannot expect everyone who uses inspector with many nodes to disable the sync | 15:15 |
| TheJulia | since they don't use it a pxe filter | 15:15 |
| arne_wiebalck | dtantsur: you mean running API and conductor on different hosts? | 15:15 |
| dtantsur | it's not about the PXE filter, it's about syncing the node list | 15:15 |
| TheJulia | dtantsur: agreed, but no filter basically means it is entirely redundant as well. So multiple things that should and can be fixed | 15:16 |
| dtantsur | arne_wiebalck: as different processes | 15:16 |
| dtantsur | TheJulia: I remember there was at least one problem with disabling it, cannot fully remember | 15:16 |
| TheJulia | dtantsur: the only downside i see having a knob is not deleting records from the inspector DB :\ since it will do the lookup if memory serves | 15:16 |
| dtantsur | anyway, some people may use the PXE filter, so we need to actually fix it | 15:16 |
| kaifeng | currently inspector doesn't have hashring to split nodes management | 15:16 |
| * dtantsur ponders attaching an inspector instance to a conductor group | 15:16 | |
| dtantsur | I mean, the biggest problem right now is that all 12 (?) inspector instances try to do the sync for all nodes | 15:17 |
| dtantsur | with the leader election in place it will be only one | 15:17 |
| dtantsur | which is 12x less load on ironic already | 15:17 |
| arne_wiebalck | dtantsur: yes | 15:17 |
| TheJulia | Lets have this discussion during open discussion | 15:17 |
| arne_wiebalck | dtantsur: I think this is a scalable solution | 15:17 |
| arne_wiebalck | ok | 15:18 |
| dtantsur | yep | 15:18 |
| TheJulia | Has anyone looked at neutron event processing recently? | 15:18 |
| TheJulia | hjensas, kaifeng, rpittau ^^^ | 15:18 |
| kaifeng | haven't got the time, sorry | 15:18 |
| rpittau | nope | 15:18 |
| hjensas | nope, I want to get back to it. but need to find time. | 15:19 |
| TheJulia | re: v6, I guess we never clicked the backport button? | 15:19 |
| * dtantsur dunno | 15:20 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/ussuri: Add IPv6 ci Job https://review.opendev.org/735614 | 15:20 |
| rpittau | no, it was not backported yet | 15:20 |
| TheJulia | #easy | 15:20 |
| TheJulia | iurygregory: re: grenade, can you elaborate on "related to ngs key" | 15:21 |
| dtantsur | note that the job seems red on master | 15:21 |
| iurygregory | TheJulia, yeah going to add a more info there =) | 15:21 |
| TheJulia | Okay, well seems good then. Everyone ready to proceed to priorities for the week? | 15:22 |
| dtantsur | I am | 15:22 |
| rpittau | let's | 15:22 |
| TheJulia | #topic Deciding on priorities for the coming week | 15:23 |
| *** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:23 | |
| TheJulia | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:23 |
| TheJulia | Starting at line 116! | 15:23 |
| TheJulia | Looks like we got a number of items merged last week. Thanks everyone who helped there! | 15:23 |
| TheJulia | As for things to add, dtantsur do you think it would be good to add your inspector leader election patch for at least initial feedback? | 15:24 |
| * TheJulia deletes merged items | 15:24 | |
| dtantsur | yep | 15:25 |
| iurygregory | TheJulia, added =) | 15:25 |
| rpittau | TheJulia: can we pkease add https://review.opendev.org/735335 | 15:25 |
| patchbot | patch 735335 - ironic-python-agent-builder - Disable automatic updates in dnf-based systems - 3 patch sets | 15:25 |
| TheJulia | rpittau: go ahead :) | 15:25 |
| rpittau | ok! | 15:25 |
| TheJulia | You guys don't need my permission to add items :) | 15:26 |
| kaifeng | dtantsur: i remember not all backend support leader election, maybe we need a more generic way? | 15:26 |
| * iurygregory added items without asking =X | 15:26 | |
| dtantsur | kaifeng: well, I'm not aware of a more generic way | 15:26 |
| dtantsur | anyway, if the backend returns NotImplemented, all nodes run the tasks | 15:27 |
| rpittau | TheJulia: oh it's just to see if there's place, based on priorities :) | 15:27 |
| kaifeng | dtantsur: hmm, better than nothing ;) | 15:28 |
| dtantsur | I've added a few things too | 15:28 |
| dtantsur | kaifeng: yeah, at least it does not make things worse :) | 15:28 |
| dtantsur | memcached and redis backends support leader election, etcd does not | 15:28 |
| Qianbiao | dtantsur why not use amqp to support task schedule | 15:28 |
| dtantsur | Qianbiao: we're trying to get rid of amqp :) and anyway, I'm not sure it's related | 15:29 |
| Qianbiao | any node which get the message will run the task. other nodes will just ignore | 15:29 |
| dtantsur | we don't have any entity to schedule the task, it's periodic | 15:29 |
| TheJulia | is there anything else people see that should be in the list for this week? | 15:29 |
| Qianbiao | ok if amqp will be removed | 15:29 |
| Qianbiao | dtantsur I find you are working on inband deploy steps, i got a question with inband steps: may it support restart bmc during steps? | 15:30 |
| dtantsur | not sure it will ever be removed, but I'd avoid a new dependency on it | 15:30 |
| TheJulia | we likely need to look at https://review.opendev.org/#/c/731644/ | 15:30 |
| patchbot | patch 731644 - ironic - Fix Redfish handle no continuous override boot src - 4 patch sets | 15:30 |
| dtantsur | Qianbiao: (let's talk later) it can be a deploy step | 15:30 |
| Qianbiao | ok. | 15:30 |
| dtantsur | TheJulia: good call | 15:30 |
| rpioso | TheJulia, dtantur: Please do :-) | 15:30 |
| rpioso | *dtantsur | 15:31 |
| * dtantsur wonders if next week we should only keep things that are bug fixes and sprint priorities | 15:31 | |
| TheJulia | also https://review.opendev.org/#/c/730366/ | 15:31 |
| patchbot | patch 730366 - ironic - Allow node lessee to see node's ports - 1 patch set | 15:31 |
| TheJulia | dtantsur: Likely | 15:31 |
| dtantsur | and somebody (me?) needs to talk to the release team on whether we actually need to change the release model | 15:32 |
| TheJulia | That looks good, I went through some of the items that got no review traffic it looks like | 15:32 |
| TheJulia | dtantsur: likely and I think we have to if we want to keep our stress level to manageable levels. | 15:33 |
| dtantsur | TheJulia: the wind is changing, we may be able to keep the model and the stress level | 15:33 |
| dtantsur | I'll talk to the folks | 15:33 |
| TheJulia | okay | 15:33 |
| * TheJulia doesn't want more stress though | 15:33 | |
| * TheJulia wants less stress | 15:33 | |
| * dtantsur wonders if he has ACL to do #action | 15:34 | |
| TheJulia | :) | 15:34 |
| dtantsur | heh | 15:34 |
| TheJulia | dtantsur: I can note one if you want | 15:34 |
| iurygregory | if you need any help let me know dtantsur =) | 15:34 |
| dtantsur | iurygregory: I will quite likely need help, but not on the talking stage | 15:34 |
| dtantsur | :) | 15:34 |
| iurygregory | yeah =) | 15:34 |
| dtantsur | there'll be an awkward moment when we figure out how the CI for new branches will work | 15:34 |
| dtantsur | (remove grenade, add more bifrost or whatever) | 15:34 |
| iurygregory | gotcha | 15:34 |
| TheJulia | #action dtantsur to go talk to release team about model and stuff | 15:35 |
| rpittau | dtantsur: I thought we needed to keep grenade for openstack.... needs | 15:35 |
| TheJulia | dtantsur: inherently broken out of the gate and then fix is likely okay in my book :) | 15:35 |
| dtantsur | rpittau: not for intermediate branches which are explicitly targeting standalone usage | 15:35 |
| TheJulia | since we can't anticipate everything | 15:35 |
| dtantsur | yep | 15:35 |
| dtantsur | the problem with grenade is, it expects to know the branches | 15:35 |
| TheJulia | That may also mean we need to get them to force merge or squash patches into one change set | 15:35 |
| rpittau | no, of course, I was thinking about major opesntack releases | 15:35 |
| dtantsur | if we have stable/17.0, it won't know what to upgrade from | 15:36 |
| dtantsur | hence the desire to have a bifrost-upgrade CI job | 15:36 |
| TheJulia | dtantsur: override variables likely needed in that case | 15:36 |
| TheJulia | ++ | 15:36 |
| iurygregory | this we can override the config .. | 15:36 |
| dtantsur | (which will also need to know the branches, but that's solvable) | 15:36 |
| dtantsur | yeah, all is possible, let's figure it out as we go | 15:36 |
| TheJulia | Anyway, sounds like everyone is good with the list of priorities | 15:36 |
| TheJulia | arne_wiebalck: was there anything else besdies the whitepaper which we've already touched upon a few times? | 15:37 |
| TheJulia | (specifically for the SIG) | 15:37 |
| arne_wiebalck | I don't think so. | 15:37 |
| TheJulia | Then in that case, lets proceed to Open Discussion! :) | 15:37 |
| dtantsur | ++ | 15:37 |
| TheJulia | #topic Open Discussion | 15:38 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:38 | |
| TheJulia | so, scaling of the ironic-inspector? | 15:39 |
| arne_wiebalck | I think the leader election should do it, no? | 15:39 |
| TheJulia | if configured properly, likely. | 15:40 |
| arne_wiebalck | Otherwise, we may need sth to describe how to set up services. | 15:40 |
| dtantsur | do you use memcached or redis already? | 15:40 |
| TheJulia | I... still think a global "don't do this" knob may be good in some configurations | 15:40 |
| dtantsur | agree with ^^^ | 15:40 |
| arne_wiebalck | Yes, for Cinder. | 15:41 |
| dtantsur | I just don't think we should call it a fix for this bug | 15:41 |
| kaifeng | do we have sync interval for this? | 15:41 |
| dtantsur | arne_wiebalck: okay, so an easy change for you | 15:41 |
| arne_wiebalck | heh | 15:41 |
| dtantsur | kaifeng: we don't handle interval==0, we should | 15:41 |
| TheJulia | dtantsur: oh yeah, I was never thinking it was a bug fix, more a "well, it doesn't really make sense" in this case change | 15:41 |
| arne_wiebalck | we only see an effect in prod | 15:41 |
| dtantsur | TheJulia: tss, lemme first lure arne_wiebalck into testing my leader election patch! | 15:41 |
| arne_wiebalck | due to the scale, but prod is ... prod :) | 15:41 |
| kaifeng | that's sad, it would be a fast solution if we have | 15:42 |
| TheJulia | interval==0 is not a bad idea, although it would be similar to the patch I already posted | 15:42 |
| TheJulia | dtantsur: hehe | 15:42 |
| arne_wiebalck | dtantsur: but, yeah, true should not be too complicated | 15:42 |
| dtantsur | TheJulia: yep, I commented on your patch that I'd prefer to reuse interval==0 rather than a new option | 15:42 |
| arne_wiebalck | dtantsur: I created an internal ticket to try | 15:42 |
| TheJulia | dtantsur: sounds good to me | 15:42 |
| dtantsur | (esp. since it's consistent with ironic) | 15:42 |
| arne_wiebalck | reducing the frequency to 3600 did the trick for us atm | 15:43 |
| arne_wiebalck | which is still <10 mins efeectively | 15:43 |
| arne_wiebalck | effectively | 15:43 |
| arne_wiebalck | and hence probably good enough | 15:43 |
| arne_wiebalck | dtantsur: don't worry, will still try your patch :-D | 15:44 |
| TheJulia | Anything else to discuss? | 15:46 |
| TheJulia | Do we need a "review jam" ? | 15:46 |
| dtantsur | mmm, maybe? | 15:46 |
| *** rloo has quit IRC | 15:47 | |
| dtantsur | not sure about a call, but maybe half-day to sit together on IRC discussing the same patches? | 15:47 |
| TheJulia | Would anyone be opposed to something on say ?thursday? | 15:47 |
| TheJulia | that could also work | 15:47 |
| dtantsur | Friday would be easier | 15:48 |
| arne_wiebalck | I'd prefer a call over IRC only tbh. Clarifying details/context would be quicker. | 15:49 |
| TheJulia | Yeah, just looking at down stream calendars and thinking that it is a lower key day | 15:49 |
| TheJulia | We could do a hybrid, both realistically | 15:50 |
| iurygregory | both sounds good to me | 15:50 |
| TheJulia | Do we want to schedule like an hour for such? | 15:50 |
| TheJulia | or even a half hour? | 15:51 |
| TheJulia | a half hour may be more than enough for a call if we've all had a chance to kind of get ideas/thoughts together | 15:51 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Switch Ironic to openstacksdk for Neutron https://review.opendev.org/734873 | 15:51 |
| *** rloo has joined #openstack-ironic | 15:52 | |
| TheJulia | Friday is also a heavy call day for me right now | 15:52 |
| rpittau | Thursday would probably be slightly better for me too | 15:53 |
| TheJulia | would 4PM UTC work that day? | 15:53 |
| rpittau | Thursday? good for me | 15:54 |
| TheJulia | I could also do before 3 PM UTC on thursday | 15:54 |
| iurygregory | good for me too | 15:54 |
| dtantsur | I have an API SIG meeting at 4pm, but it's usually silent | 15:54 |
| dtantsur | 3pm is our downstream meeting, no? | 15:54 |
| TheJulia | dtantsur: yeah, so after migh tbe besst from an interrupt management standpoint | 15:55 |
| dtantsur | ah, *before* 3pm | 15:55 |
| * dtantsur cannot read | 15:55 | |
| dtantsur | 2pm UTC would be ideal for me, 4pm works too | 15:55 |
| TheJulia | arne_wiebalck: does 4pm on next thursday work for you? | 15:55 |
| arne_wiebalck | yes, all fine for me :) | 15:55 |
| TheJulia | I can also get up for 2pm, I just can't promise full caffination | 15:56 |
| dtantsur | mmm, right, it's early for you. let's go for 4pm | 15:56 |
| TheJulia | sounds good | 15:56 |
| arne_wiebalck | +1 | 15:56 |
| TheJulia | I'll send out an email here in a little bit just for mailing list visibility | 15:56 |
| TheJulia | Do we have anything to discuss in the next 4 minutes? | 15:57 |
| TheJulia | Preferably centered around taking over the world using bare metal and bears with drum sticks? | 15:57 |
| arne_wiebalck | Please re-read the instructions aka the white paper! | 15:57 |
| * dtantsur still wants a sticker with a bear biting a redfish | 15:58 | |
| rpittau | dtantsur: https://thumbs.dreamstime.com/b/face-wild-bear-fish-close-up-concept-bears-hunting-salmon-spawning-185089098.jpg | 15:58 |
| dtantsur | exactly | 15:59 |
| iurygregory | LOL | 15:59 |
| * TheJulia thinks we need this sticker | 16:00 | |
| iurygregory | i ust noticed that in ironic-lib we do some blackmagic to allow the command to be executed without root =O | 16:00 |
| * dtantsur waits for iurygregory to realize ironic is made of black magic :) | 16:01 | |
| iurygregory | I knew that hehe | 16:01 |
| iurygregory | but I was surprised when I saw that we check the config and override run_as_root hehe | 16:02 |
| TheJulia | Maybe cast it as more "wizardry" or "incantations" | 16:02 |
| dtantsur | I guess we can #endmeeting, then discuss the black magic? | 16:02 |
| TheJulia | lol | 16:02 |
| TheJulia | Thanks everyone! Have a wonderful week! | 16:02 |
| TheJulia | #endmeeting | 16:02 |
| *** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list." | 16:02 | |
| openstack | Meeting ended Mon Jun 15 16:02:37 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:02 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-06-15-15.00.html | 16:02 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-06-15-15.00.txt | 16:02 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-06-15-15.00.log.html | 16:02 |
| arne_wiebalck | TheJulia: thanks | 16:02 |
| dtantsur | iurygregory: is it anything you need help with or just "WTF is wrong with us"? | 16:03 |
| iurygregory | basically even if the command has run_as_root set if the config is not set we override run_as_root https://github.com/openstack/ironic-lib/blob/e4e11ccc7df5cf566211e3b347067ca196691d54/ironic_lib/utils.py#L95 https://github.com/openstack/ironic-lib/blob/e4e11ccc7df5cf566211e3b347067ca196691d54/ironic_lib/disk_utils.py#L142-L143 | 16:03 |
| *** gyee has joined #openstack-ironic | 16:03 | |
| iurygregory | I'm thinking on how to deal with the privsep for this scenarios.. | 16:04 |
| *** Lucas_Gray has quit IRC | 16:04 | |
| dtantsur | I think root_helper is pretty much always set? | 16:04 |
| iurygregory | will try to talk with some oslo people to see if there is something we can use | 16:04 |
| dtantsur | I'd suggest you kill this code with fire? | 16:04 |
| rpittau | the root_helper is always 'on' | 16:04 |
| rpittau | that needs to be translated with privsep | 16:04 |
| iurygregory | hummm I would kill tests only basically.. | 16:04 |
| iurygregory | because we have some scenarios we want to make sure the commands are run with non-root | 16:05 |
| *** sshnaidm_ is now known as sshnaidm | 16:05 | |
| dtantsur | just pass run_as_root=False? | 16:05 |
| rpittau | you can set specific permissions with privsep, not necessarily just root | 16:05 |
| TheJulia | they are functionally privlegeless commands correct? | 16:06 |
| rpittau | and I'm pretty sure we need partprobe always running as root | 16:06 |
| TheJulia | As long as the file permissions are right, accessible? | 16:06 |
| TheJulia | or are they SUID bit binaries? | 16:06 |
| iurygregory | for more context the test Im talking https://github.com/openstack/ironic-lib/blob/e4e11ccc7df5cf566211e3b347067ca196691d54/ironic_lib/tests/test_disk_utils.py#L917 | 16:06 |
| iurygregory | yeah but privsep the idea is that is used by root | 16:07 |
| iurygregory | so we will have a root and non-root command right?\ | 16:07 |
| iurygregory | I saw nova have the priv command and non priv command in some cases and they use based on the config | 16:08 |
| rpittau | iurygregory: privsep allows running in privileged contexxts, that doesn't mean always as 'root' | 16:08 |
| *** lucasagomes has quit IRC | 16:09 | |
| TheJulia | I just don't grok why we would try to run that without root privs now... | 16:10 |
| rpittau | it's all about security, you don't need root, just the correct context | 16:10 |
| TheJulia | ++ | 16:10 |
| TheJulia | I think the confusion is we have these tests now and that doesn't make that much sense | 16:10 |
| rpittau | yeah | 16:10 |
| *** dsneddon has joined #openstack-ironic | 16:11 | |
| iurygregory | so so we can just ignore run_as_root in our execs? | 16:11 |
| iurygregory | and have the command wrapped by privsep | 16:11 |
| rpittau | now we have root/not-root but we need to think in a more open way, rainbow style :) | 16:11 |
| iurygregory | that would make sense? | 16:11 |
| TheJulia | protip: don't tell me how many people I might need to talk to when emailing me :( | 16:11 |
| rpittau | let's say that the root part needs to be translated into proper privsep contexts | 16:12 |
| iurygregory | rpittau, e.g https://review.opendev.org/#/c/728942/4/ironic/privsep/iscsi.py we could remove run_as_root=True in the calls | 16:12 |
| patchbot | patch 728942 - ironic - Migration from oslo.rootwrap to oslo.privsep - 4 patch sets | 16:12 |
| rpittau | yep | 16:12 |
| iurygregory | and have the correct privilege | 16:13 |
| rpittau | yes, that's the tricky part :) | 16:13 |
| iurygregory | that is based on the capabilities we set | 16:13 |
| iurygregory | ok I will go with the path "die run_as_root" XD | 16:15 |
| TheJulia | fyi, I just created https://meetpad.opendev.org/ironic | 16:15 |
| TheJulia | anyone can use that | 16:15 |
| iurygregory | will just confirm with the oslo people that is ok =) | 16:15 |
| rpittau | bye everyone! o/ | 16:17 |
| *** rpittau is now known as rpittau|afk | 16:17 | |
| TheJulia | goodnight! | 16:17 |
| rpittau|afk | tnx | 16:17 |
| * TheJulia goes back to emails | 16:18 | |
| *** Qianbiao has quit IRC | 16:19 | |
| iurygregory | bye everyone | 16:22 |
| dtantsur | furniture won't assemble itself, I have to go | 16:22 |
| *** dtantsur is now known as dtantsur|afk | 16:22 | |
| dtantsur|afk | o/ | 16:22 |
| * arne_wiebalck is off tmrw | 16:33 | |
| arne_wiebalck | bye everyone o/ | 16:33 |
| *** ricolin has quit IRC | 16:36 | |
| TheJulia | heh | 16:41 |
| *** k_mouza has quit IRC | 16:53 | |
| openstackgerrit | Merged openstack/ironic master: devstack: Use uwsgi binary from path; temporary disable grenade https://review.opendev.org/731159 | 16:55 |
| openstackgerrit | Merged openstack/ironic-inspector master: devstack: Use uwsgi binary from path; temporary disable grenade https://review.opendev.org/735547 | 16:55 |
| openstackgerrit | Merged openstack/ironic master: Make redfish boot interface to use `network_data` https://review.opendev.org/688272 | 17:02 |
| *** dougsz has quit IRC | 17:02 | |
| *** derekh has quit IRC | 17:08 | |
| *** zaneb has joined #openstack-ironic | 17:12 | |
| donnyd | TheJulia: I ended up figuring out how to get it all working with no keystone | 17:19 |
| *** uzumaki has quit IRC | 17:45 | |
| TheJulia | donnyd: what was it specifically? a quick doc patch would be appreciated | 17:47 |
| TheJulia | Also, is anyone interested in text for an ironic website? | 17:47 |
| donnyd | TheJulia: I will work one up | 17:49 |
| donnyd | auth_type needed to be set to noauth in keystone_authtoken | 17:49 |
| TheJulia | the keystone_authtoken changes all make my head hurt, so it definitely sounds like something to note in docs :\ | 18:10 |
| openstackgerrit | James Denton proposed openstack/networking-generic-switch master: VLAN not tagged on every switchs https://review.opendev.org/557051 | 18:19 |
| jamesdenton | ^^ would help to know if i'm on the right track with that particular unity test | 18:19 |
| jamesdenton | *unit | 18:19 |
| TheJulia | jamesdenton: sure, I can take a look a little later | 18:20 |
| jamesdenton | sure, whenever it's convenient. thanks! | 18:20 |
| TheJulia | So at some point, I guess we dropped autodoc from ironic... | 18:24 |
| TheJulia | replaced by apidoc | 18:26 |
| *** kaifeng has quit IRC | 18:27 | |
| TheJulia | yay for a typo | 18:27 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Fix apidoc build https://review.opendev.org/734712 | 18:30 |
| NobodyCam | Morning Ironic'er folks 👋 | 19:07 |
| *** dsneddon has quit IRC | 19:11 | |
| TheJulia | good morning NobodyCam | 19:13 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Switch Ironic to openstacksdk for Neutron https://review.opendev.org/734873 | 19:20 |
| *** dsneddon has joined #openstack-ironic | 19:27 | |
| *** tzumainn has quit IRC | 19:32 | |
| iurygregory | TheJulia, I'm interested (re text for ironic website) =) | 19:37 |
| iurygregory | morning NobodyCam | 19:37 |
| *** tzumainn has joined #openstack-ironic | 19:46 | |
| TheJulia | iurygregory: I'll send you the link | 20:00 |
| iurygregory | TheJulia, ack =) | 20:01 |
| TheJulia | So it looks like someone is not pleased with the python-dracclient docs on the mailing list | 20:19 |
| TheJulia | While... not exactly the way to send commentary or feedback, they raise a valid point that I published at least an internal to the package change. | 20:20 |
| TheJulia | See: http://lists.openstack.org/pipermail/openstack-discuss/2020-June/thread.html#15458 | 20:20 |
| TheJulia | cdearborn: rpioso: It looks like the docs that are being published are not actually being published but are from >4 years ago. The opendev infra folks can surely help on removing the old doc site. It looks like there is no documentation publishing job which would likely be helpful for people to be able to see the state of things and comment. | 20:23 |
| iurygregory | ouch | 20:25 |
| iurygregory | the end of the email is very unpleasant =( | 20:26 |
| TheJulia | Indeed | 20:26 |
| TheJulia | I suspect they did not realize they were emailing a mailing list?! | 20:26 |
| iurygregory | I hope so | 20:27 |
| iurygregory | we need to try to believe on the good side of people XD | 20:28 |
| TheJulia | Indeed | 20:29 |
| *** stevebaker has joined #openstack-ironic | 20:51 | |
| cdearborn | Hey TheJulia, iurygregory, yes, just saw the email and replied. I totally understand the frustration. We haven't been very diligent about keeping the dracclient docs up to date. Will take a look at the docs publishing. Thanks! | 20:53 |
| TheJulia | cdearborn: Thanks <3 | 20:55 |
| iurygregory | cdearborn,++ | 20:55 |
| TheJulia | Yeah, I totally get it too and there are a lot of plugins to make super pretty docs | 20:55 |
| TheJulia | And... sometimes we all have things that need to be fixed | 20:55 |
| TheJulia | such as https://review.opendev.org/#/c/734712/ | 20:56 |
| patchbot | patch 734712 - ironic-python-agent - Fix apidoc build - 7 patch sets | 20:56 |
| TheJulia | (which is a decent apidoc example) | 20:56 |
| stevebaker | morning | 20:57 |
| openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic master: Allow node vif attach to specify port_uuid or portgroup_uuid https://review.opendev.org/731780 | 20:59 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add a driver compatability matrix https://review.opendev.org/733716 | 20:59 |
| TheJulia | o/ stevebaker | 20:59 |
| TheJulia | stevebaker: any chance I could convince you to make a <5 minute basic authentication demo? | 20:59 |
| TheJulia | video that is | 20:59 |
| TheJulia | (bonus points for posting it to youtube or something, btw) | 21:00 |
| iurygregory | morning stevebaker o/ | 21:01 |
| stevebaker | TheJulia: yeah I could do that. Would the audience be folk who want to deploy ironic standalone? | 21:02 |
| TheJulia | cdearborn: great reply to the tread | 21:02 |
| TheJulia | stevebaker: I think so | 21:03 |
| cdearborn | TheJulia, thanks! Honestly, I'm excited to have a known user of dracclient (hopefully) | 21:03 |
| stevebaker | TheJulia: I might finish off json-rpc support first, so I can include that | 21:04 |
| stevebaker | iurygregory: hi! | 21:04 |
| TheJulia | stevebaker: if you record it with that still in flight, I think that is okay. I'm just thinking something for sprint demos, and posting to youtube would be kickass IMO | 21:04 |
| cdearborn | TheJulia, and thanks for the patch! | 21:05 |
| TheJulia | cdearborn: least I could do :) | 21:05 |
| * TheJulia goes back to populating the deploy interface matrix | 21:06 | |
| stevebaker | TheJulia: ack | 21:06 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add a driver compatability matrix https://review.opendev.org/733716 | 21:13 |
| * TheJulia wonders why her memory limit change hates her | 21:25 | |
| TheJulia | eek Stderr: 'Error setting Chassis Boot Parameter 5\n' | 21:30 |
| *** threestrands has joined #openstack-ironic | 21:36 | |
| *** gmann is now known as gmann_afk | 21:39 | |
| *** kaiokmo has quit IRC | 21:42 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Guard conductor from consuming all of the ram https://review.opendev.org/726483 | 21:59 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Iso booting via redfish virtual media https://review.opendev.org/727480 | 22:04 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: iPXE ISO Ramdisk booting https://review.opendev.org/729464 | 22:04 |
| openstackgerrit | James Denton proposed openstack/networking-generic-switch master: VLAN not tagged on every switchs https://review.opendev.org/557051 | 22:10 |
| jamesdenton | funny... you actually have to pass data to the test to make it work. smh | 22:15 |
| TheJulia | jamesdenton: heh | 22:18 |
| TheJulia | so... interestingly enough the item in the old launchpad records for networking-generic-switch doesn't map out to a story in storyboard | 22:18 |
| TheJulia | https://storyboard.openstack.org/#!/project/956 <-- Ideally we would map the change to something in that list (or create one) | 22:19 |
| TheJulia | jamesdenton: looks like the only thing your really missing, as far as I can tell is something involving the _is_port_binding method | 22:23 |
| TheJulia | aside from that, it looks good to me.. I'd stylistically do things a little bit differently but your doing it in the same style of the file so ++ | 22:23 |
| jamesdenton | i'm totally open to suggestions, so feel free to offer them if you'd like. And I didn't think that would be enough tests, just wanted to make sure i was on the right track | 22:31 |
| TheJulia | Your definitely on it | 22:32 |
| TheJulia | thanks! | 22:32 |
| jamesdenton | thanks! | 22:32 |
| TheJulia | and fwiw, I prefer in-file continuity above most else | 22:32 |
| TheJulia | Well, it looks like my grocery delivery is going to be here early | 22:33 |
| TheJulia | Which makes this a good time to wrap up the day. Have a wonderful day@ | 22:34 |
| TheJulia | s/@/!/ | 22:34 |
| *** rcernin has joined #openstack-ironic | 23:16 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
