opendevreview | Jacob Anders proposed openstack/ironic-python-agent master: Improve efficiency of storage cleaning in mixed media envs https://review.opendev.org/c/openstack/ironic-python-agent/+/818712 | 04:28 |
---|---|---|
opendevreview | Jacob Anders proposed openstack/ironic-python-agent master: Improve efficiency of storage cleaning in mixed media envs https://review.opendev.org/c/openstack/ironic-python-agent/+/818712 | 04:31 |
opendevreview | Jacob Anders proposed openstack/ironic-python-agent master: Improve efficiency of storage cleaning in mixed media envs https://review.opendev.org/c/openstack/ironic-python-agent/+/818712 | 04:49 |
*** pmannidi is now known as pmannidi|Away | 06:11 | |
*** pmannidi|Away is now known as pmannidi | 06:12 | |
arne_wiebalck | Good morning, janders and Ironic o/ | 06:14 |
* dtantsur slowly blinks | 07:10 | |
hgy_ | Good afternoon, Ironic! | 07:15 |
muellerbe | good morning ironic o/ | 07:25 |
rpittau | good morning ironic! o/ | 07:39 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on centos 9 stream https://review.opendev.org/c/openstack/bifrost/+/819058 | 08:09 |
rpittau | dtantsur: thanks, I also read the thread on openstack-discuss, to be honest I thought the centos 9 stream nodes were still not usable as the proper nodeset definition was missing | 08:10 |
rpittau | anyway, not sure how far we can go without epel pkgs :/ | 08:10 |
dtantsur | rpittau: I think bifrost mostly works without epel | 08:31 |
dtantsur | in the CI we only enable it when BUILD_IMAGE | 08:34 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on centos 9 stream https://review.opendev.org/c/openstack/bifrost/+/819058 | 08:51 |
janders | good morning arne_wiebalck dtantsur hgy_ muellerbe rpittau and Ironic o/ | 09:02 |
hgy_ | good morning danders | 09:03 |
muellerbe | good morning janders o/ | 09:07 |
hgy_ | Hello, Could I ask a question | 09:23 |
arne_wiebalck | hgy_: sure, you can always ask :) | 09:24 |
hgy_ | When I deploy Bare metal node by Ironic, I can capture the dhcp request package | 09:25 |
hgy_ | But neutron didn't return the response | 09:26 |
hgy_ | I use the default secutity group | 09:27 |
hgy_ | log like this : https://paste.opendev.org/show/811313/ | 09:29 |
hgy_ | And I can see that ip had been distributed to inst in horizon | 09:37 |
hgy_ | Does anyone know the cause of this question | 09:37 |
arne_wiebalck | hgy_: so, you see your baremetal node boot and send out the dhcp request (to boot into the IPA image)? | 09:39 |
hgy_ | arne_wiebalck: yes | 09:40 |
hgy_ | arne_wiebalck: I saw it in bmc and captured dhcp request package in controller node using tcpdump | 09:41 |
arne_wiebalck | hgy_: ok, so the dhcp server even got the request from the node? | 09:42 |
hgy_ | arne_wiebalck: em~~~, actually, the node which have the dhcp server got the requtst by tcpdump | 09:43 |
hgy_ | arne_wiebalck: But I don't sure that dhcp server also got it | 09:43 |
hgy_ | arne_wiebalck: /etc/neutron/dhcp_agent.ini [DEFAULT]dhcp_driver = neutron.agent.linux.dhcp.Dnsmasq | 09:45 |
hgy_ | tcpdump and network info and neutron configure: https://paste.opendev.org/show/811315/ | 09:48 |
muellerbe | hgy_ you look via tcpdump for the dhcp package on the interface of ironic server? have you also seen the dhcp answer packages? "tcpdump -nei <INTERFACE> udp port 67 or port 68 or port 69" and also look into bifrost dnsmasq log, and also the dnsmasq config for enabled mac address | 09:49 |
arne_wiebalck | hgy_: when you say "controller", you're referring to the Neutron controller (for dhcp) or to the Ironic controller (for PXE/tftp) ? From what I understand in standard deployments, Neutron would be used for DHCP and Ironic for tftp (as mentioned, our deployment is slightly different, so I am a little out of my depth ...). | 09:49 |
* arne_wiebalck has a meeting now | 09:49 | |
hgy_ | arne_wiebalck: "controller" is "Neutron controller", but I deploy Neutron controller and Ironic controller in one node actually | 09:51 |
hgy_ | arne_wiebalck: In my test environment, I deploy all of openstack in one server | 09:51 |
hgy_ | muellerbe: I have never seen the dhcp answer packages | 09:53 |
hgy_ | muellerbe: I am finding the log of dhsmasq and checking the config of dnsmasq | 09:56 |
hgy_ | muellerbe: hi, Thank you very very much. I found that my dnsmasq.service's status is inactive(dead) | 10:03 |
muellerbe | hgy_ very good, I hope this helps you | 10:04 |
hgy_ | muellerbe: I just configured "dhcp_driver = neutron.agent.linux.dhcp.Dnsmasq" in /etc/neutron/dhcp_agent.ini following the document | 10:05 |
hgy_ | muellerbe: no operation about dnsmasq | 10:05 |
hgy_ | muellerbe: Do I need to start it by handle? I never saw the information about it in https://docs.openstack.org/neutron/latest/install/controller-install-option2-rdo.html | 10:06 |
hgy_ | muellerbe: sorry for my poor English. Do I need to start dnsmasq on manually | 10:08 |
muellerbe | hgy_ I do not know which deployment framework you use, I have only experience with kolla and in kolla the dnsmasq will be started automatically | 10:27 |
hgy_ | muellerbe: I deploy it following https://docs.openstack.org/ironic/victoria/ | 10:31 |
hgy_ | muellerbe: one step by one step on hands | 10:32 |
hgy_ | muellerbe: If this, don't waste your time, I am trying it | 10:33 |
hgy_ | muellerbe: you can do yourself things | 10:33 |
muellerbe | hgy_ so you deployed neutron also like this, I think you have to debug your neutron installation, review logs etc, what is the output of this command "openstack network agent list" especially the "DHCP agent" | 10:39 |
hgy_ | muellerbe: It's like this: https://paste.opendev.org/show/811319/ | 10:42 |
hgy_ | muellerbe: dhcp agent is up | 10:42 |
muellerbe | ok, where do you see the dnsmasq inactive (dead) | 10:43 |
hgy_ | muellerbe: by "systemctl status dnsmasq.service" | 10:44 |
hgy_ | muellerbe: like this: https://paste.opendev.org/show/811322/ | 10:44 |
muellerbe | why says systemd inactive and openstack up? | 10:44 |
muellerbe | hgy_ I think this is a good point for debugging | 10:44 |
hgy_ | muellerbe: This point is also confused me | 10:45 |
hgy_ | muellerbe: so good, I will debug following this | 10:46 |
hgy_ | muellerbe: love you | 10:47 |
muellerbe | hgy_ you are welcome | 10:50 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] All-in-one Ironic service with a local RPC bus https://review.opendev.org/c/openstack/ironic/+/819620 | 11:10 |
dtantsur | TheJulia: this seems to work in basic local testing ^^^ | 11:10 |
janders | see you tomorrow Ironic o/ | 12:05 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on centos 9 stream https://review.opendev.org/c/openstack/bifrost/+/819058 | 13:17 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Terminate TLS on Nginx https://review.opendev.org/c/openstack/bifrost/+/819640 | 13:32 |
*** tosky is now known as Guest7142 | 13:37 | |
*** tosky_ is now known as tosky | 13:37 | |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Terminate TLS on Nginx https://review.opendev.org/c/openstack/bifrost/+/819640 | 13:40 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Terminate TLS on Nginx https://review.opendev.org/c/openstack/bifrost/+/819640 | 14:04 |
TheJulia | Good morning! | 14:17 |
dtantsur | morning TheJulia | 14:17 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Terminate TLS on Nginx https://review.opendev.org/c/openstack/bifrost/+/819640 | 14:17 |
rpittau | good morning TheJulia :) | 14:17 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] All-in-one Ironic service with a local RPC bus https://review.opendev.org/c/openstack/ironic/+/819620 | 14:25 |
TheJulia | dtantsur: impressive. I kind of like it | 14:33 |
TheJulia | hmm | 14:33 |
* TheJulia wonders which version she was just reviewing | 14:33 | |
TheJulia | Anyway! I'm awake! | 14:33 |
dtantsur | TheJulia: the version don't change much, I'm mostly polishing/debugging nuances now | 14:35 |
dtantsur | yay re aware :) | 14:35 |
TheJulia | hey, nobody said I was aware! | 14:35 |
TheJulia | ;) | 14:35 |
* TheJulia is really just an advanced AI | 14:35 | |
*** outbrito_ is now known as outbrito | 14:35 | |
TheJulia | Now, to convince the future AI overlords to deliver food/water ;) | 14:35 |
dtantsur | TheJulia: ask the feline overlord ;) | 14:36 |
TheJulia | heh | 14:37 |
TheJulia | ugh, I'm actually going to have to start to wear glasses while working on my desk. Or I need to get bigger monitors | 14:38 |
* dtantsur is trying to remember why we use TLS for JSON RPC in bifrost | 14:39 | |
TheJulia | tls all the thigns? | 14:40 |
dtantsur | one could argue that TLS on localhost doesn't make too much sense.. | 14:40 |
TheJulia | and one could also argue CAP_NET_ADMIN may be given in places it shouldn't be | 14:40 |
dtantsur | also true | 14:41 |
dtantsur | I'm trying to make nginx handle TLS | 14:41 |
TheJulia | pop neutron container container somehow, you may not be able to see ironic, but you might be able to sniff the loopback | 14:41 |
dtantsur | (i.e. https://review.opendev.org/c/openstack/bifrost/+/819640) | 14:41 |
dtantsur | even without root? | 14:41 |
TheJulia | all you need is cap_net_admin | 14:41 |
TheJulia | which root implies | 14:42 |
TheJulia | but you don't need uid=0 to launch or use tcpdump | 14:42 |
TheJulia | just cap_net_admin allows you to do the raw packet bind | 14:42 |
TheJulia | s/packet/socket/ | 14:42 |
TheJulia | having written tcp socket io code in C was super useful to my brain. | 14:43 |
TheJulia | </sarcasm mode=partial> | 14:43 |
dtantsur | cpdump: virbr0: You don't have permission to capture on that device | 14:43 |
dtantsur | well, at least a typical user does not seem to be able to do that | 14:43 |
TheJulia | lo | 14:43 |
TheJulia | yeah | 14:43 |
dtantsur | otherwise we need to start caring about not using TLS for mysql :) | 14:44 |
TheJulia | so mysql is a weird beast as it is sort of like smtp | 14:44 |
TheJulia | If I remember correctly | 14:44 |
TheJulia | ... or that might have only been 15 years ago | 14:45 |
dtantsur | no idea about the protocol itself | 14:45 |
dtantsur | I know that Metal3 had to write some configuration to use TLS with it | 14:45 |
TheJulia | smtp had to derivations, smtps which ran a an SSL socket on like port 463, and then smtp added "starttls" method on tcp/25 | 14:45 |
TheJulia | most everyone just used starttls when encrypted mail transport started to become a thing | 14:46 |
dtantsur | I'm not sure Bifrost's threat model includes people with CAP_NET_ADMIN on localhost | 14:46 |
dtantsur | yeah | 14:46 |
* TheJulia spent waaaayyyyy too much time helping law firms get smtp starttls support working. | 14:46 | |
dtantsur | hehe | 14:46 |
TheJulia | dtantsur: true | 14:46 |
dtantsur | it's not hard to add TLS support to MySQL bts: https://github.com/metal3-io/ironic-image/blob/master/scripts/runmariadb#L39-L41 | 14:47 |
TheJulia | dtantsur: in the threat model case, I think it comes down to what is easier to maintain and makes the most sense | 14:47 |
dtantsur | in metal3 we had a lot of issues with eventlet's TLS implementation | 14:47 |
TheJulia | :( | 14:48 |
TheJulia | it feels like https://review.opendev.org/c/openstack/ironic-specs/+/815308 just needs a bunch of underscores added | 14:49 |
dtantsur | ah, I think I remembered. JSON RPC uses host names, which in case of bifrost may not be localhost. | 14:50 |
TheJulia | yeah, this is true | 14:54 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Stop exposing JSON RPC to the whole network https://review.opendev.org/c/openstack/bifrost/+/819649 | 14:56 |
dtantsur | TheJulia: something like this ^^^ | 14:56 |
TheJulia | +2'ed | 14:57 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Terminate TLS on Nginx https://review.opendev.org/c/openstack/bifrost/+/819640 | 14:58 |
dtantsur | #startmeeting ironic | 15:00 |
opendevmeet | Meeting started Mon Nov 29 15:00:08 2021 UTC and is due to finish in 60 minutes. The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
rpittau | o/ | 15:00 |
dtantsur | hi folks | 15:00 |
TheJulia | o/ | 15:00 |
rpioso | \o | 15:00 |
ajya | o/ | 15:00 |
stendulker | o/ | 15:00 |
dtantsur | #link https://wiki.openstack.org/wiki/Meetings/Ironic our agenda for today | 15:01 |
* TheJulia wonders how light our december meetings may be | 15:01 | |
dtantsur | very light, I suspect :) | 15:01 |
dtantsur | #topic Announcements / Reminder | 15:01 |
dtantsur | #info Tested Runtimes for Yoga - Python 3.8 and 3.9 (https://governance.openstack.org/tc/reference/runtimes/yoga.html) | 15:02 |
dtantsur | This did cause a heated discussion on the ML since 3.6 is what CentOS/RHEL 8 has | 15:02 |
dtantsur | I have a discussion topic to return to this issue | 15:02 |
dtantsur | #info First bugfix branch should be created by the end of next week | 15:02 |
dtantsur | Who doesn't like a fresh warm release? :) | 15:02 |
dtantsur | let's make sure to finish whatever can be finished by EOW (this week) | 15:03 |
rloo | o/ | 15:03 |
dtantsur | #info PTL on PTO during whole December | 15:04 |
dtantsur | let's wish iurygregory great time :) | 15:04 |
TheJulia | ++ I'm glad he made it home | 15:04 |
dtantsur | yep! | 15:04 |
dtantsur | That's all we had, does anyone have any other announcements? | 15:04 |
TheJulia | I've got nothing annoucement wise | 15:05 |
TheJulia | we *should* try and see when we want to stop holding meetings for the rest of the year, during open discussion | 15:06 |
dtantsur | yep | 15:06 |
dtantsur | #topic Review action items from previous meeting | 15:06 |
dtantsur | iurygregory to create a doodle for the new time so people can vote + email to openstack-discuss | 15:06 |
dtantsur | I don't think this has actually happened? | 15:06 |
TheJulia | I did not see it | 15:06 |
TheJulia | He seemed excepptionally busy last week | 15:06 |
dtantsur | I can imagine | 15:06 |
dtantsur | was it about moving the meeting time? maybe rpittau could drive it since you're the most affected? | 15:07 |
TheJulia | Yeah | 15:07 |
* dtantsur pokes rpittau | 15:09 | |
rpittau | sounds good, I will take care of it (sorry, double meeting) | 15:09 |
dtantsur | ah, well, conflicting meetings. we'll get back to it. | 15:09 |
dtantsur | no worries, I obviously forgot *why* you were affected :D | 15:10 |
dtantsur | #topic Review subteam status reports | 15:10 |
dtantsur | hmm, first | 15:10 |
dtantsur | #action rpittau to create a doodle for the new meeting time so people can vote + email to openstack-discuss | 15:10 |
dtantsur | now, whiteboard | 15:10 |
dtantsur | #link https://etherpad.opendev.org/p/IronicWhiteBoard white board | 15:10 |
dtantsur | I don't think we've merged the Yoga topics | 15:10 |
TheJulia | We have not | 15:11 |
dtantsur | #link https://review.opendev.org/c/openstack/ironic-specs/+/815308/ Yoga themes | 15:11 |
dtantsur | still, I'd suggest we start reshaping the whiteboard according to it | 15:11 |
dtantsur | any volunteers? | 15:11 |
dtantsur | (I can do it, but I'm happy to delegate :) | 15:11 |
TheJulia | I might be able to this week. I've got a few past-due things to take care of today | 15:12 |
dtantsur | I can do it as well, just checking if anyone is really eager | 15:12 |
TheJulia | ++ | 15:12 |
dtantsur | #action dtantsur to update the whiteboard with the (draft) yoga themes | 15:12 |
dtantsur | #topic Deciding on priorities for the coming week | 15:13 |
dtantsur | first and foremost, please review the yoga themes | 15:13 |
dtantsur | I'd love to see https://review.opendev.org/c/openstack/sushy/+/817792 merged, but it's on the list already | 15:13 |
dtantsur | anything else? | 15:13 |
TheJulia | If I can crank out the client patch for node history events tomorrow, any objection for me to add it tomorrow? | 15:13 |
dtantsur | none from me | 15:14 |
dtantsur | okay, let's move on to discussions, we have a couple of things | 15:15 |
dtantsur | #topic Situation around Python 3.6 support | 15:15 |
rpittau | I Added a couple of patche sfor tempest | 15:15 |
rpittau | too late :( | 15:15 |
dtantsur | thanks rpittau | 15:15 |
TheJulia | rpittau: ack, I'll try to review them soon() | 15:15 |
rpittau | ack | 15:16 |
dtantsur | So, as I mentioned earlier, the tested runtimes no longer include python 3.6 | 15:16 |
dtantsur | which causes issues for Bifrost which supports Stream 8, which has 3.6 | 15:16 |
dtantsur | (it also supports Bionic, openSUSE Leap and Debian 10, all of which have <3.8 and will need to be deprecated) | 15:16 |
dtantsur | but Ubuntu and Debian have new versions which support 3.8+ | 15:17 |
dtantsur | CentOS has Stream 9, but I don't know how many folks will feel easy to use it for their production deployments | 15:17 |
dtantsur | any opinions or suggestions? | 15:17 |
TheJulia | I have this bad feeling it doesn't end up with anyone happy | 15:18 |
rpittau | about centos stream 9, I definitely do not suggest it in the current status | 15:18 |
rpittau | for the rest, if we want to keep testing on py3.6 we can, we "just" to add a py3.6 test | 15:18 |
dtantsur | rpittau: do you have any practical experience that makes you say so? re 9? | 15:18 |
dtantsur | we have a discussion on the ML where some people are basically going "9 is just as good as 8, what's the problem?" | 15:19 |
TheJulia | I feel like those saying it didn't get to feel pain during centos stream 8 | 15:20 |
dtantsur | I'd like us to keep 3.6 support for at least Yoga; similar voices are coming from Kolla and RDO teams | 15:21 |
dtantsur | but it's seems like the matter is already decided on the TC side | 15:22 |
TheJulia | I'm on board with that for now | 15:22 |
rloo | ++ keep 3.6 support | 15:22 |
rpittau | my experience so far is restricted to bifrost and metal3 | 15:22 |
rpittau | if it was an overlap phase where we had to temporarily support both centos8 and centos9 I would say no problem, but abruptly going from one to another is a bit tough | 15:22 |
dtantsur | at the very least we can add add 3.6 unit tests on Ironic and other our projects to make sure *we* don't break and serve as canary for broken dependencies | 15:23 |
TheJulia | This does feel like the community is trying to make a very hard cut | 15:23 |
TheJulia | but I think upfront we knew we would only be on 3.6 for 3-4 years tops. | 15:23 |
TheJulia | but it is hard for us to really comprehend or remember that in active memory from day to day when distros do long lived packaging | 15:23 |
rpittau | I'm ok to add unit tests for py3.6 for yoga, and work on centos9 at the same time | 15:24 |
dtantsur | yep | 15:24 |
dtantsur | and yep | 15:24 |
dtantsur | voting or non-voting? | 15:24 |
TheJulia | voting... at least for the time being | 15:24 |
rpittau | voting | 15:24 |
dtantsur | okay, sounds good, will do (help welcome) | 15:25 |
dtantsur | moving on? or any other opinions? | 15:25 |
TheJulia | I suspect we're safe to move on | 15:26 |
rpittau | I will help | 15:26 |
rpittau | one concern I have is for dependencies removing py3.6 support (oslo libraries for example), need to keep an eye on those | 15:26 |
TheJulia | yeah | 15:26 |
dtantsur | yeah, that's why I'm saying that the job will work as a canary | 15:26 |
TheJulia | if they start slipping in a minimum requirement of 3.8, that is pretty much shooting eveyrone in the foot | 15:26 |
dtantsur | at least we'll know when it's over :) | 15:26 |
rpittau | :) | 15:26 |
TheJulia | ++ | 15:26 |
dtantsur | #topic Baremetal SIG | 15:27 |
dtantsur | arne_wiebalck: anything from you? | 15:27 |
* TheJulia wonders if the sig is on holiday until January | 15:27 | |
dtantsur | well, I have a talk on the 14th :) | 15:28 |
* dtantsur is curious if anyone shows up | 15:28 | |
dtantsur | okay, I assume nothing new from the SIG at this point | 15:30 |
dtantsur | #topic RFE review | 15:30 |
TheJulia | dtantsur: what is the talk on? I've been under a rock | 15:30 |
dtantsur | #link https://storyboard.openstack.org/#!/story/2008723 Enhance Redfish firmware update user experience | 15:30 |
dtantsur | TheJulia: I'm going to talk about Metal3, maybe even show it | 15:30 |
dtantsur | meanwhile, ajya, you around? | 15:31 |
ajya | about RFE - it is updated after previous review - zip support removed and added download_source similar to images. Any feedback? | 15:31 |
TheJulia | Was zip support added by the ilo folks? | 15:32 |
dtantsur | I think we tried to figure that out and decided that rather no? | 15:33 |
TheJulia | Unfortunately, removing support would cause me to be worried about compatibility and messaging. | 15:33 |
dtantsur | I think ajya means it was removed from the RFE, not from the code | 15:33 |
ajya | as far as I recall, no it was not aded by ilo, they had different packages handled server side | 15:33 |
TheJulia | oh! | 15:33 |
TheJulia | okay | 15:33 |
ajya | yes, removed from RFE | 15:33 |
ajya | for redfish | 15:33 |
TheJulia | nevermind me! | 15:33 |
dtantsur | the RFE seems good to me | 15:34 |
ajya | thanks | 15:34 |
ajya | anyone else? | 15:34 |
* TheJulia looks | 15:35 | |
TheJulia | lgtm | 15:36 |
dtantsur | any objections? | 15:37 |
TheJulia | none at this time | 15:37 |
dtantsur | okay, marked approved | 15:38 |
dtantsur | #topic Open discussion | 15:38 |
dtantsur | the floor is open | 15:38 |
TheJulia | So, Typically we've taken a moment in these meetings in past years to identify when people were generally going to go *poof* for the year | 15:38 |
dtantsur | I'll be more or less present until ~ 22nd | 15:39 |
TheJulia | I personally can disappear after the 15th, but I can hang out until say the 22nd or just keep an eye on things | 15:39 |
rpittau | I will be present until the 24th | 15:39 |
TheJulia | so the next question, is how many more times do we want to meet this year? | 15:40 |
TheJulia | Typically we stop mid-month | 15:40 |
dtantsur | definitely not 27th :) unlikely Jan, 3rd | 15:40 |
rpittau | we could skip the 21st ? | 15:41 |
dtantsur | I'll probably be here on the 20th still | 15:41 |
rpittau | sorry, the 20th, yeah | 15:41 |
arne_wiebalck | o/ | 15:41 |
* arne_wiebalck was in another meeting | 15:41 | |
dtantsur | arne_wiebalck: anything from the SIG standpoint? | 15:41 |
dtantsur | arne_wiebalck: I wonder how many people will still be here on the 14th for the meeting.. | 15:42 |
arne_wiebalck | nothing for the SIG, except dtantsur's presentation on the 14th! | 15:42 |
arne_wiebalck | hmm | 15:42 |
TheJulia | I don't really have any strong opinions either way, If we want to skip the 27th and the 3rd, I think that works but I suspect the 20th will lack critical mass. Just speaking from experience | 15:42 |
arne_wiebalck | we can postpone to Jan if you prefer, but mid december should be ok, I guess | 15:42 |
arne_wiebalck | as you prefer | 15:42 |
dtantsur | I'll be here anyway | 15:43 |
arne_wiebalck | me, too | 15:43 |
dtantsur | maybe a pre-holiday presentation is actually nice :) | 15:43 |
TheJulia | :) | 15:43 |
dtantsur | TheJulia: I'm cool with skipping 20th | 15:43 |
arne_wiebalck | need some x-mas filter! | 15:43 |
dtantsur | yeah | 15:44 |
arne_wiebalck | rudolph circling the screen or sth | 15:44 |
TheJulia | then lets skip the 20th, 27th | 15:44 |
TheJulia | what about the third? | 15:44 |
dtantsur | I'll probably be still out | 15:45 |
arne_wiebalck | still holiday here | 15:45 |
TheJulia | Then lets resume January 10th | 15:45 |
arne_wiebalck | ++ | 15:45 |
rpittau | yep | 15:45 |
TheJulia | and hope all Ironic contributors are ready for plenty of code reviews :) | 15:45 |
dtantsur | who wants to send the announcement? | 15:45 |
dtantsur | yeah | 15:45 |
TheJulia | I'll send the email | 15:45 |
dtantsur | thanks! | 15:45 |
TheJulia | I guess I have a couple other minor items | 15:45 |
TheJulia | It is board nominations right now. You should have an email from the Open Infrastructure Foundation if you'd like to nominate anyone for board of directors. As a reminder, individuals need 10 nominees to appear on the ballot. Please use firefox since it is known to work. | 15:46 |
dtantsur | ++ | 15:47 |
dtantsur | anything else? | 15:47 |
TheJulia | Additionally, the nova patche to try and fix the lack of actual rebalance needs eyes | 15:47 |
TheJulia | https://review.opendev.org/c/openstack/nova/+/813897 | 15:47 |
sam_z | i've been workin on the implementation of the feature described here and would super appreciate some eyes on my work: https://review.opendev.org/c/openstack/ironic-specs/+/808804/ | 15:47 |
TheJulia | it changes the behavior, I'd <3 for operators to review it | 15:47 |
* TheJulia looks at rloo, zer0c00l, mnaser, arne_wiebalck ;) | 15:48 | |
dtantsur | sam_z: ah, redfish proxy. yeah, it fell off my radar, sorry | 15:48 |
arne_wiebalck | TheJulia: ++ | 15:48 |
dtantsur | I wonder how it plays together with the allinone (sorry TheJulia) ironic process.. will need to think | 15:49 |
TheJulia | dtantsur: sam_z ? | 15:49 |
TheJulia | or the nova stuffs? | 15:49 |
dtantsur | the redfish stuffs | 15:49 |
TheJulia | k, good question | 15:49 |
sam_z | also i remember TheJulia saying she had interest in migrating the ironic api to flask and since i'm workin' with flask here i just wanted to put my name in for that lol | 15:49 |
dtantsur | it's probably not a concern, unless sam_z wants to have a new executable | 15:49 |
TheJulia | yeah | 15:50 |
sam_z | allinone process? | 15:50 |
TheJulia | I think the consensus kind of lead to a new process for the proxy anyhow, at least for now | 15:50 |
dtantsur | sam_z: easier to show than to explain: https://review.opendev.org/c/openstack/ironic/+/819620 | 15:50 |
dtantsur | basically API and conductor in a single executable | 15:50 |
TheJulia | sam_z: the idea has been kicking around for the last two years | 15:50 |
dtantsur | I'd definitely want to see a possibility to use an all-in-one process | 15:51 |
dtantsur | even if the default is to use a new ironic-redfish-api executable | 15:51 |
TheJulia | one last item from me. It seems I'm doomed to do a lot more in tempest this cycle than I expected. I've been informed more comprehensive testing is expected, basically to ultimately remodel the plugin to be rbac aware at the level of tests above and beyond what client to use | 15:51 |
sam_z | would this feature working with allinone ironic imply it being integrated into the allinone executable? | 15:51 |
sam_z | cuz i *think* that actually might not be too bad actually | 15:52 |
dtantsur | sam_z: likely so. it's probably going to be trivial, just making sure you're aware that it's happening | 15:52 |
TheJulia | I've got a basic wip up https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/819165, since the desire is a mix of positive and negative testing, I'd appreciate any thoughts/feedback since it is going to be a large pile of work | 15:52 |
dtantsur | TheJulia: my condolences (re tempest) | 15:52 |
TheJulia | dtantsur: thanks | 15:52 |
TheJulia | not what I expected to spend december on | 15:52 |
dtantsur | I guess tempest is not the future we wanted, but the future we deserved | 15:53 |
TheJulia | maybe | 15:53 |
TheJulia | Our plugin has somewhat languished as well since it is often an after thought | 15:54 |
sam_z | i have already been brainstorming ways to launch the ironic api and this redfish proxy in the same service anyway and i think because my thingy is a separate wsgi app, in theory we could just use something like the werkzeug dispatcher to serve both | 15:54 |
TheJulia | or a minor thing creeps into being a more major thing | 15:54 |
TheJulia | sam_z: might work, although I think rpittau had an issue with werkzeug directly | 15:55 |
TheJulia | or I may be remembering something else | 15:55 |
dtantsur | sam_z: there are numerous possibilities, depending on whether you want to reuse the same port or not | 15:55 |
* TheJulia wonders if we should just let sam_z do what he wants and see what the outcome is | 15:55 | |
rpittau | mmmm not me or it was not recently | 15:55 |
TheJulia | rpittau: this felt recent to me | 15:55 |
dtantsur | I have some experience with werkzeug, but it mostly positive | 15:56 |
dtantsur | I think we use it in IPA directly? | 15:56 |
sam_z | yea, i've been meaning to look into trying to implement it and just seeing what happens lol | 15:56 |
TheJulia | hjensas: was there something w/r/t werkzeug recently? | 15:56 |
TheJulia | dtantsur: we do | 15:56 |
* TheJulia wonders if we're in the multiverse of python madness | 15:57 | |
TheJulia | (w/r/t my thinking of recent werkzeug issues | 15:57 |
TheJulia | ) | 15:57 |
dtantsur | werkzeug is the thing behind flask, it's not the worst piece of the puzzle | 15:57 |
TheJulia | true | 15:58 |
TheJulia | Anyway, I guess we're done for today? | 16:00 |
dtantsur | yep | 16:00 |
dtantsur | thanks folks! | 16:00 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Stop exposing JSON RPC to the whole network https://review.opendev.org/c/openstack/bifrost/+/819649 | 16:00 |
dtantsur | #endmeeting | 16:00 |
opendevmeet | Meeting ended Mon Nov 29 16:00:53 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:00 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-29-15.00.html | 16:00 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-29-15.00.txt | 16:00 |
opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-29-15.00.log.html | 16:00 |
arne_wiebalck | thanks dtantsur | 16:01 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Terminate TLS on Nginx https://review.opendev.org/c/openstack/bifrost/+/819640 | 16:01 |
*** sshnaidm is now known as sshnaidm|afk | 16:22 | |
hjensas | TheJulia: we ran into a IPv6 related bug in Werkzeug a couple of weeks back, https://github.com/pallets/werkzeug/issues/2262 | 16:25 |
TheJulia | fun! | 16:26 |
TheJulia | So I guess that should be relatively okay moving forward | 16:26 |
TheJulia | sam_z: ^^^ fyi | 16:27 |
rpittau | good night! o/ | 16:38 |
opendevreview | Merged openstack/ironic master: Remove isfile check from place_loaders_for_boot https://review.opendev.org/c/openstack/ironic/+/819378 | 17:29 |
opendevreview | Merged openstack/ironic-tempest-plugin master: Update pep8 requirements https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/819319 | 17:56 |
opendevreview | Julia Kreger proposed openstack/ironic stable/xena: Clarify driver load error message https://review.opendev.org/c/openstack/ironic/+/819587 | 17:56 |
opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: Clarify driver load error message https://review.opendev.org/c/openstack/ironic/+/819588 | 17:56 |
NobodyCam | Good Morning Ironic'ers! Happy Monday | 19:05 |
TheJulia | Good morning NobodyCam | 19:06 |
NobodyCam | o/ Good Morning TheJulia ... | 19:07 |
*** tosky is now known as Guest7167 | 20:17 | |
*** tosky_ is now known as tosky | 20:17 | |
*** tosky_ is now known as tosky | 21:57 | |
opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: An idea for rbac positive/negative testing https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/819165 | 23:03 |
TheJulia | stevebaker[m]: another look at ^^^ would be appreciated. I'm trying to figure out how to handle the empty bodies since we don't hand 403's back to everyone for everything, as it would be information leakage otherwise. | 23:04 |
TheJulia | some thoughts noted in test_nodes.py | 23:04 |
TheJulia | and kind of two different ideas | 23:05 |
stevebaker[m] | TheJulia: looking | 23:09 |
TheJulia | stevebaker[m]: no rush today | 23:09 |
stevebaker[m] | oki | 23:09 |
TheJulia | stevebaker[m]: I’m wrapping up since I started at 6 am local | 23:09 |
stevebaker[m] | yeah thats early | 23:10 |
TheJulia | I almost always start that early | 23:10 |
TheJulia | It yeah | 23:10 |
TheJulia | Err, but | 23:10 |
opendevreview | Steve Baker proposed openstack/ironic master: Write master grub config on startup https://review.opendev.org/c/openstack/ironic/+/815580 | 23:36 |
opendevreview | Steve Baker proposed openstack/ironic master: Capture [pxe]loader_file_paths for distros https://review.opendev.org/c/openstack/ironic/+/815392 | 23:36 |
opendevreview | Steve Baker proposed openstack/ironic master: Move place_loaders_for_boot to boot driver __init__ https://review.opendev.org/c/openstack/ironic/+/819729 | 23:36 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!