Wednesday, 2025-03-26

TheJuliaJayF: physical_network I think needs to match the physical network name as recorded in neutron so networking-baremetal can reconcile it00:39
TheJuliarm_work[m]: JayF: https://github.com/openstack/networking-baremetal/blob/master/networking_baremetal/agent/ironic_neutron_agent.py#L238-L23901:22
JayFI will either review a change by somebody to update that doc with the specifics, or I'll take care of pushing the update in the morning if it hasn't been done01:33
rm_work[m]hmm, yeah that tracks but I feel like it also goes directly to some ovn mapping (which i don't see there? unless I am skimming badly)01:49
Amarachi_OGood morning Ironic!05:46
freemanboss[m]Good morning Amarachi_O: 06:42
ayo_Good morning Amarachi_O06:48
Amarachi_OPlease have anyone been able to successfully deploy nodes06:48
freemanboss[m]<Amarachi_O> "Please have anyone been able..." <- Yes07:22
Amarachi_OPlease how did you go about it07:24
freemanboss[m]where are you stuck at?07:24
rpittaugood morning ironic! o/07:44
rpittauHashaam[m]: hi! We're going to reply to you soon, in the meantime I suggest to have a look at bifrost documentation 07:46
Amarachi_Ofreemanboss[m]: I am not stuck really I just want to confirm if the process I went through was the right one, after enrolling the nodes, did you use  /bifrost-cli deploy / tmp/baremetal. json07:50
Amarachi_O-e @baremetal-install-env. json  or was there another process in between07:50
Amarachi_Ofreemanboss[m]: and what is the next thing to do after that I didn't see07:51
rpittauHashaam[m]: I can't actually find your email, did you send it to both masghar and myself?07:51
freemanboss[m]if the baremetal.json is present yes thats the file else use baremetal-inventorfy.json07:51
freemanboss[m]you are using ansible or cli?07:52
Amarachi_OCli07:52
freemanboss[m]alright 07:53
Amarachi_OAre you using cli or ansible07:53
freemanboss[m]I used both 07:53
freemanboss[m]The process is simple and direct it's just that it seems hard to get/understand and apply.07:54
Amarachi_OOkay I will try that,  so now that you have deployed the hardware what next are we meant to do ?07:55
freemanboss[m]First thing to avoid unnecessary issues is to create an ssh key pairs07:56
freemanboss[m]ssh-keygen -t rsa -b 4096 -f ~/.ssh/id_rsa07:56
Amarachi_OOh okay I ran into that issue initially but I was able to create the ssh key pairs07:57
freemanboss[m]Then prepare to run ironic in a test mode ./bifrost-cli testenv... (full message at <https://matrix.org/oftc/media/v1/media/download/ASjKlJ0e6dIIpze_Yqsx0hMVaOV9KN5jpOwkXDzSW9COANYZl0mWj6ou5KgMIV92zFc49CQ10Uh7Ahx4nXmz9aNCeWGY7ytgAG1hdHJpeC5vcmcvT2FxeEtDSHhrc1hQVkFIdktYT2dVYkRw>)07:58
freemanboss[m]Amarachi_O: Great good 07:58
freemanboss[m]Amarachi_O: Write up check your mail or the previous conversation.07:59
freemanboss[m]After the writeup rpittau: will tell us what next 07:59
Amarachi_OOkay no problem thank you so much!08:00
freemanboss[m]Amarachi_O: You're welcome08:00
rpittaufreemanboss[m]: don't forget to submit your bifrost experience as second contribution in the outreachy website, when you're ready :)08:00
freemanboss[m]w08:01
freemanboss[m]rpittau: yes I will do that as soon as possible08:01
Amarachi_O rpittau okay are we still to send that via email ? Or just to submit it on the website08:02
rpittauAmarachi_O: what really matters is the submission on the outreachy website08:03
freemanboss[m]adeolaadesina[m]: Yes it's doing the same thing. But I think the /tmp/baremental.json is only created when you use the script/setup.sh. 08:04
Amarachi_Orpittau: okay thank you08:04
freemanboss[m]When ./bifrost-cli is used 2 files got created in the working directory which are baremetal-nodes.json and baremetal-inventory.json and also ansible too has its created files too08:05
freemanboss[m]adeolaadesina[m]: Exactly08:06
opendevreviewMichal Nasiadka proposed openstack/bifrost unmaintained/2023.1: Switch to unmaintained/2023.1  https://review.opendev.org/c/openstack/bifrost/+/94554208:31
ayo_Amarachi_O: what are we using the key pair for ?10:18
freemanboss[m]ayo_: It's to establish the deployment. Ironic needs to connect with the host10:20
Amarachi_Oayo_: If you don't include it in the deployment phase you will run into an error10:22
ayo_Thank you freemanboss[m] Amarachi_O10:32
adavize_freemanboss[m] : I'm still stuck at after installing ironic. "./bifrost-cli install --testenv" runs fine but after that I can't seem to get any other thing to work.11:21
adavize_I've followed the documentation steps thoroughly and I don't know if I am missing anything. I'm trying to run Bifrost on Ubuntu wsl11:26
freemanboss[m]<freemanboss[m]> "Then prepare to run ironic in..." <- > <@freemanboss:matrix.org> Then prepare to run ironic in a test mode ./bifrost-cli testenv... (full message at <https://matrix.org/oftc/media/v1/media/download/AVHjEji28o_vzE9KrrZqUdO9OxDOOjGv_3UV0nkd2-U7ZGffPPyqklXN11-5kRUoodsBL_9hFiWin_EPEoe3aSFCeWGk9XogAG1hdHJpeC5vcmcvdFN6eVdEakNuZmNOSVpycnBOY05IUEZv>)11:28
freemanboss[m]<freemanboss[m]> "First thing to avoid unnecessary..." <- > <@freemanboss:matrix.org> First thing to avoid unnecessary issues is to create an ssh key pairs... (full message at <https://matrix.org/oftc/media/v1/media/download/AWBvpnKzEipS1FZJi9h7Q5xrtBwuqQabyDM8qhI0lTpSsmFam8jl4mzvFvrRW6_N4OVKaWJLwtv1RMcKIA5fFr1CeWGk_9ogAG1hdHJpeC5vcmcvYU5PY09sd1JBWm1zT0NpZ1RPU0R2eGlr>)11:29
Amarachi_Oadavize_: what error are you getting11:32
ps_adavizewhen i run enroll command "TASK [ironic-enroll-dynamic : Wait for the node to finish cleaning] ****************************************************FAILED - RETRYING: [testvm1 -> localhost]: Wait for the node to finish cleaning (45 retries left).11:32
ps_adavizeFAILED - RETRYING: [testvm2 -> localhost]: Wait for the node to finish cleaning (45 retries left)."11:32
opendevreviewcid proposed openstack/ironic master: docs: Clarify physical_network should be a name  https://review.opendev.org/c/openstack/ironic/+/94556211:33
Amarachi_Ops_adavize: what is the line of code you use to run it11:34
adavize_Amarachi_0 : "./bifrost-cli enroll baremental-inventory.json"11:35
freemanboss[m]ps_adavize: ./bifrost-cli node list 11:35
freemanboss[m]Send the results 11:35
ps_adavizefreemanboss[m] : https://postimg.cc/njFhZh5C  after running the enroll command.11:38
freemanboss[m]freemanboss[m]: > <@freemanboss:matrix.org> ./bifrost-cli node list 11:39
freemanboss[m]> Send the results 11:39
freemanboss[m]ps_adavize: 11:39
freemanboss[m]If you can just follow the processes I highlighted already I don't think you'll face any issue11:40
ps_adavize./bifrost-cli node list -->>>>>>>> "usage: Bifrost CLI [-h] [--debug] {testenv,install,enroll,deploy} ...11:40
ps_adavizeBifrost CLI: error: argument {testenv,install,enroll,deploy}: invalid choice: 'node' (choose from 'testenv', 'install', 'enroll', 'deploy')"11:40
ps_adavizefreemanboss[m] : https://postimg.cc/5jHMzzwf11:41
freemanboss[m]ps_adavize: Ohhh sorry 11:42
freemanboss[m]baremetal node list 11:42
ps_adavizefreemanboss[m] : I actually followed the steps you outlined in the former post11:42
freemanboss[m]ps_adavize: You started each process all over again?11:42
ps_adavizefreemanboss[m] : https://postimg.cc/NKBVd2r911:44
ps_adavizeyes I ran exactly that process11:44
ayo_Yep they enrolled ps_adavize11:45
ps_adavizeayo_ : what about the errors showing up? saying waiting for node to finish cleaning ?11:47
ayo_Cleaning up would most likely be for redeployed nodes11:47
freemanboss[m]ps_adavize: You can also validate it to see what's happening 11:48
ayo_Have you deployed before? ps_adavize ?11:49
freemanboss[m]ps_adavize: After the error that's when you change the state to manageable right? Just enroll again since you did that already 11:50
ps_adavizefreemanboss[m] : https://postimg.cc/vg0fwjPw   the validate output11:50
ps_adavizeayo_ i haven't deployed yet11:51
freemanboss[m]ps_adavize: Just restart from the process I highlighted 11:51
freemanboss[m]Make sure you follow step by step 11:51
ps_adavizefreemanboss[m]: ayo_ what system are you using?, I am using ubuntu on wsl11:53
ayo_Centos11:55
freemanboss[m]ps_adavize: That error I saw there I don't think it's system based you can try enrolling again if you think you want to tackle that issue first 11:56
freemanboss[m]I use Ubuntu 11:56
Amarachi_OI am using Ubuntu on a vm though but I don't think that would be the issue since you have gone this far11:57
freemanboss[m]ps_adavize: If it's system based error you shouldn't get into enrollment there should have been clear issue during installation 11:57
freemanboss[m]Amarachi_O: Exactly11:58
ps_adavizeokay 11:58
ps_adavizefreemanboss[m] : can you run baremetal driver list, let me see what shows up in yours11:58
freemanboss[m]ps_adavize: My system is off12:00
ps_adavizealso are you using the stable/2024.2 branch freemanboss[m] Amarachi_0 12:08
freemanboss[m]ps_adavize: I used master but that too will work fine12:09
freemanboss[m]Those are not real issue12:09
Amarachi_Ops_adavize: yes12:12
ps_adavizelet me try to delete everything and start again, although i've done that severally12:14
freemanboss[m]ps_adavize: Don't delete anything just start again and be cautious 12:17
freemanboss[m]What's the resources you're using? Ram? CPU?12:17
ps_adavizefreemanboss[m] that's what I have been doing for 2 days now, 24 gig ram, 2 cores. I also tried on a 16gig, 6core12:19
freemanboss[m]ps_adavize: Alright it's enough resources just follow the processes and make sure you've ssh key pairs12:20
ps_adavizeapart from git and python3 is there any other prerequisite that you installed manually?12:20
ps_adavizefreemanboss[m] at what point do i create the ssh key pairs12:20
Amarachi_OWhen you are deploying12:21
Amarachi_OBefore you deploy actually12:21
ps_adavizemy own problem is still at the enroll command12:22
freemanboss[m]First thing to avoid unnecessary issues is to create an ssh key pairs... (full message at <https://matrix.org/oftc/media/v1/media/download/AThkkQL7LbYNlbrkHvhvx1pmjuG2CuI5iKvm4hyTb2P_HCzVgPZ_rWK1dLLr4-SdvbExakFCLOieFaZUOoQ-qGZCeWGoBL2wAG1hdHJpeC5vcmcvU0psclhnY2lIa2NYaGNaY1RUU2JsZlV4>)12:22
freemanboss[m]After installing git and python3 then changing dir to bifrost12:22
freemanboss[m]Just follow these steps 12:22
Amarachi_Ofreemanboss[m]: thanks for this12:23
ps_adavizeThankyou freemanboss[m] I'm currently trying it out again will get back on the result12:27
freemanboss[m]ps_adavize: Alright no problem 12:28
Amarachi_O@freemanboss[m]: please what are the steps for using ansible aswell12:34
freemanboss[m]Amarachi_O: Get that done first or you faced any error?12:36
Amarachi_OGet what done first sorry?12:37
freemanboss[m]When using ansible just follow the docs and the enroll-dynamic.yaml and deploy-dynamic.yml is the substitute for enroll and deploy as in CLI12:37
freemanboss[m]Amarachi_O: Ohh sorry12:37
Amarachi_OOkay thanks I will try that12:39
TheJuliagood morning13:01
cid\o13:02
cidfreemanboss[m], could you record any instructions in the doc that you encountered didn't work and you had to do it a different way, as well as any omissions, in an etherpad https://etherpad.opendev.org/ so that you can just refer your peers, as well as to help us know how to update the docs later on. This also applies to anyone else that has encountered a problem and later fixed it, they can update the pad.13:02
freemanboss[m]cid: Yeah, I'll do that. 13:04
cidGreat, \o/. Thanks.13:05
freemanboss[m]cid: my pleasure13:08
cardoeJayF: definitely poke me on some of the network model stuff cause that's where we are battling as well. skrobul will be interested. there's someone else on my team that I'm poking to stay on IRC that's gonna be my point person.13:12
TheJuliaSo Jay, Myself, and johnthetubaguy had a long chat on ?Monday? and I started drafting a spec regarding some of the network issues specifically as it relates to nova and figuring out/managing how to sort portgroup and bonds, as well as influencing actual device selection. Does that happen to be relating to your interests cardoe ?13:17
cardoeYes13:17
cardoeactual device selection is the big item for us right now.13:18
TheJuliaCheckout https://etherpad.opendev.org/p/VNdieUWeejYChegDwbHh is the high level idea13:18
cardoeSo many irons in the fire.13:18
TheJuliaindeed13:18
cardoeI really need physical_network on vxlan or another type that's "l2vni". I don't know how the neutron folks want to go with that.13:18
TheJuliacardoe: cross-project session perhaps?13:19
cardoeyeah. except I need open minds / eyes at that session13:19
TheJuliaso the question is how do we breakdown that barrier13:20
cardoeI understand that the current view of vxlan in neutron is that it's an overlay on top of bare metal networks.13:20
TheJuliaif we invite them to us, I'm not sure we'll get traction but those interested in crossing their project's boundry would be the ones to talk to13:20
cardoeBut in my case, it's the underlay.13:20
cardoeSo roughly. I've got boxes that have 4 ports of attachment. There's two physically separate fabrics that those ports live on.13:21
cardoeI want all 4 ports to exist in Ironic (and Neutron) because I want to generate cloud-init metadata for them.13:22
cardoeAs discussed above (and in the networking doc), ironic selects the port based on physical_network. But vxlan doesn't have a physical_network field.13:23
TheJuliaoh13:23
TheJuliaheh13:23
cardoeAnd I'm not sure if my approach is 100% correct but I'll describe it real quick if it helps.13:24
cardoeor I've got a few to sync talk if that would be better and then I'll share the design write up.13:24
TheJuliaso, I think you should read the etherpad real quick13:25
TheJuliabut give me a couple minutes and I'll start a call so we can chat13:25
TheJuliahttps://meet.google.com/hrg-utbx-qqd13:28
TheJuliaI'll be back in ~30 seconds, grabbing coffee from the maker13:28
* TheJulia begins Round two of Caffination13:29
ps_adavizefreemanboss[m] this is the error I'm getting at the end when I try to run the enroll command https://pastebin.com/Na6EpNyF13:30
ps_adavizeso I just spin up an ec2 instance on aws and everything worked fine, I don't know if it has something to do with me using wsl? rpittau cid masghar13:33
ayo_Or you’ll try a vm?13:43
Amarachi_OI think a vm will work fine13:43
satoships_adavis: I don't know the root cause of this error. 13:43
satoshi"last_error": "During sync_power_state, max retries exceeded for node 878c3113-0035-5033-9f99-46520b89b56d, node state None does not match expected state 'power off'. Updating DB state to 'None' Switching node to maintenance mode. Error: IPMI call failed: power status.13:43
satoshiCan you check the power state of the node with something like baremetal node show node-name/node-id?13:43
ayo_I can’t say if there’s a dependency wsl might not be having ps_adavize13:44
rpittaups_adavize: it's probably wsl13:50
opendevreviewAnton proposed openstack/networking-baremetal master: Clean up baremetal agents on node delete  https://review.opendev.org/c/openstack/networking-baremetal/+/94550013:52
ps_adavizeso does it mean wsl can't run bifrost? or maybe it's a configuration i need to fix? rpittau13:52
rpittaups_adavize: I'm not sure how it is now, but I used a lot of wsl in the past, long time ago, I've never got a good experience with complex tasks/operations, always better to have a VM for that13:55
satoshiJust a thought for the bifrost documentation improvement. Adding system resource requirement(or link) to the Bifronst installation page would be nice as many of us are facing errors related to it. 13:55
ps_adavizeAlright thanks rpittau14:31
JayFps_adavize: wsl can probably run bifrost; absolutely cannot run bifrost testenv14:39
opendevreviewKaifeng Wang proposed openstack/ironic master: Fixes the issue of flat network provision when porgroup is configured  https://review.opendev.org/c/openstack/ironic/+/93443314:45
ps_adavizethat sounds plausible, since ironic got installed and running. JayF any particular reason why wsl can't run testenv?14:45
freemanboss[m]<ps_adavize> "Freeman Boss this is the error I..." <-  baremetal node manage testvm115:04
freemanboss[m]baremetal node provide testvm115:04
freemanboss[m]Same for testvm215:04
freemanboss[m]openstack baremetal node list15:05
freemanboss[m]Then send the result of this15:05
freemanboss[m]You're using ansible or cli?15:06
koperg[m]Hi, Looking for recommendations on how-to configure pin_release_version :15:08
koperg[m]https://docs.openstack.org/ironic/latest/configuration/config.html#DEFAULT.pin_release_version15:08
koperg[m]In preparation for a slurp upgrade (N+2)  in Epoxy release.15:08
hjensasSo, are we supposed to disconnect virtual media and still think the agent is accessible? snippet of logs, https://paste.opendev.org/show/beHv9TjxLPfxL4zqK4ea/ - It is disconnecting vmedia and then trying to run -> {'step': 'tear_down_agent'15:12
hjensasI am using sushy-emulator vmedia on openstack instance, so not like "normal" hardware. I guess the agent would still be running from memory on normal hardware.15:17
JayFadavize_ps: are you the same as ps_advize? If so, the answer to your question is that bifrost testenv uses virtual machines to pretend to be bare metal; I wouldn't trust that to work; or work performantly; inside the hyperv-vm that is a wsl instance15:33
adavize_psokay I think I understand what you mean JayF, thank you15:35
JayFlike, it's *possible* it could work, but it'd be slow and potentially need tweaking15:35
JayFnot a route I'd suggest for someone new to the community15:35
JayFWe do, however, try to ensure our unit tests will pass in WSL. If you find a unit test that won't pass in WSL; that would be a bug15:36
adavize_psalright, I'll just install ubuntu Os then so I don't have all that run around, JayF. And I dont think I saw any test fail, the error messages I could understand were mostly related to memory and hardware interaction15:38
satoshihttps://etherpad.opendev.org/p/XxXVAdF2rK9P_qLX_VYK 15:41
satoshiI'm working on a bug and not sure how to proceed. Would be great if someone can give me a pointer about this. The details are in the link.15:41
TheJuliakoperg[m]: so your on Epoxy-2 today, and I guess your intending to do a rolling upgrade over some period of time to newer version?15:55
TheJuliakoperg[m]: If the docs are not clear, you should only need to pin if you intend to have a slow rolling upgrade process and have records to create/change in a mixed upgrade state. If your doing the direct jump, you shouldn't need to leverage the pin16:05
TheJuliaEntirely depends on what exactly your doing and and that option offers a control to help sort of lock things so you can slowly roll instead of just jump, if that makes sense16:06
TheJuliahjensas: So... ISO contents should come from the ramdisk and operate from ram, yeah, that detach would break it I guess16:07
TheJuliasushy-tools does something under the hood, I just don't remember what16:07
koperg[m]TheJulia: Thank you. That makes more sense to me now! 16:07
TheJuliakoperg[m]: patches are always welcome to revise text strings and docs to make things more clear :)16:08
hjensasTheJulia: yeah, the sushy-emulator rebuilds the openstack instance on attach/detach - so the agent running in ram is gone as soon as that detach happens. This is metal3+fast-track enabled a well.16:13
hjensasTheJulia: anyhow, I'll try to dig into it more tomorrow. Might be an option to hold-off on sushy-emulator rebuild of the instance until a power cycle/reset/off action happens.16:15
TheJuliaWouldn't be a bad idea16:15
TheJuliaAnyway, have a great evening!16:15
*** iurygregory_ is now known as iurygregory17:26
iurygregoryTheJulia, hey o/, by any chance you know how can I check the privilege level of the user someone is using in the ipmitool command? 17:27
TheJuliaiurygregory: an API consumer, or ironic, or someone doing it externally/manually ?17:30
TheJuliaiurygregory: asking to help orient/understand the question17:32
iurygregoryI think in this case the customer created a custom user, when I checked the logs the ipmitool had ocpadmin as the user in the -U option17:32
iurygregoryso I'm thinking they might have provided wrong privilege level or something..17:33
iurygregorysince they cant retrieve the power status (even after providing different cipher suites -C 3 and -C 17)17:33
TheJuliahmmmm17:34
iurygregoryso I'm wondering if there is a command they can use to check that..17:34
iurygregoryor maybe the machine is weird and doesn't like normal cipher suites...17:35
TheJuliaso, if memory serves, there can be differnet privilege levels, we by default assume ADMINISTRATOR17:35
iurygregoryif you want I can share the thread on slack17:35
iurygregoryyeah, ironic will issue the command with ADMINISTRATOR 17:35
TheJuliathat is with -L17:35
iurygregoryyup17:36
TheJuliasure17:36
opendevreviewJulia Kreger proposed openstack/ironic-specs master: WIP: Trait based port selection and dynamic portgroups  https://review.opendev.org/c/openstack/ironic-specs/+/94564217:47
TheJuliaJayF: per discussion ^17:48
JayFWould be nice to get https://review.opendev.org/c/openstack/ironic-python-agent/+/941714 + https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/943348 landed for satoshi before his internship is over17:50
TheJuliaJayF: I think the dividing line along the networking around the task.driver.network interface allows us to delienate cleanly, just pure standalone users will need a network interface, perhaps mercury or something else17:58
TheJuliaJay: copyright header  is wrong  unless the year is 2015 and satoshi works for Rackspace   I’m stuck in traffic on my phone at the moment so can’t easily add the comment18:50
JayFlolsob18:55
JayFI went ahead and proxied that feedback18:57
JayFI was just looking through some code with cid that /actually was/ written at Rackspace in 2014/2015 lol19:01
* JayF was relieved to find that our api app uses real threads to do the work19:04
TheJuliaJayF: no need to lolsob, I just notice things like that19:05
JayFoh, it's just a funny thing to see generally19:06
JayFand one of those "of course the code is good but $openstack-paperwork-reasons means -1"  -- which frankly I've been on the giving side of more than once 19:07
TheJuliaYeah, sucks. Sometimes I’ve just fixed it19:08
JayFit's all part of what makes openstack openstack19:09
opendevreviewSatoshi Shirosaka proposed openstack/ironic-python-agent-builder master: Create Podman DIB Element for container-based cleaning  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/94334819:17
opendevreviewSatoshi Shirosaka proposed openstack/ironic-python-agent master: WIP Ironic Config Intergration for IPA  https://review.opendev.org/c/openstack/ironic-python-agent/+/94495119:19
opendevreviewSatoshi Shirosaka proposed openstack/ironic-python-agent master: Add ContainerHardwareManager  https://review.opendev.org/c/openstack/ironic-python-agent/+/94171419:21
opendevreviewSatoshi Shirosaka proposed openstack/ironic master: WIP Ironic Container Cofig for IPA Integration  https://review.opendev.org/c/openstack/ironic/+/94479619:25
satoshiTheJulia: Thanks for the review. I removed the line. 19:27
TheJuliasatoshi: another question on the change :)21:47
TheJuliaI guess I'm also trying to understand which change is first, and why are they not stacked?!21:49
JayFTheJulia: the IPA-B and ContainerHardwareManager are batch 1; the dynamic config from conductor is batch 222:05
JayFtrying to ensure he can fully deliver a thing before his fellowship is up, even if the other two are in review a bit longer 22:05
TheJuliaI get that part, but it looks like the two ipa patches should be stacked together22:05
JayFyeah, you're right I think22:06
JayFbut if we land the others it becomes moot, at least22:06
opendevreviewJay Faulkner proposed openstack/ironic-python-agent master: Remove eventlet greendns patching  https://review.opendev.org/c/openstack/ironic-python-agent/+/94504322:11
JayFhttps://review.opendev.org/q/topic:%22ironic-container-steps%22 I put all of satoshi's outstanding ContainerHWM + config integration patches in this topic22:12
TheJuliato help inform how often I should be looking, when does the session end?22:12
TheJuliasession may be the wrong word22:12
TheJuliahmm curious how hard it will be for us to remove eventlet monkey patching entirely from ipa22:15
JayFit breaks things to remove the one remaining bit22:15
JayFcid and I are working on IPA eventlet removal, now that better docs exist and it's not like wandering in the wildernessw22:15
JayFtargetting replacing the wsgi server as a required first step22:15
JayFthen maybe "really really hope we can just stop patching" is the only remaining step :D 22:15
TheJuliaYeah, I glanced at some of the stuff the other day and while a lot, it seems like good progress is being made overall22:16
JayFI can't find my doc on when the cohort ends for mlh :|22:16
TheJuliaJayF: can't we just ask satoshi ? ;)22:20
JayFsure; but I like knowing answers to questions I'm supposed to know lol22:21
JayFmy usual problem is happening: I know it's in my notes. Which notes? 🤷‍♂️22:22
TheJuliaIn stack 47A, under subheading 3 in row F, of building G22:22
JayFoh hell I left that building in my other laptop22:23
TheJuliawait?! *in*22:23
TheJuliaoh my...22:23
TheJuliawas that the VR building?!22:23
TheJuliaThere you go!22:23
JayFalice in kernelspace22:23
TheJuliaRing 0?22:23
TheJuliaSpeaking of changes, I'd <3 my network simulator stuff to get reviews, its just devstack stuffs for humans to do things22:32
satoshiMLH fellowship ends on April 18.22:32
TheJuliasatoshi: Thanks!22:32
TheJuliasatoshi: do you happen to understand what I mean by stacking the patches?22:32
satoshiI haven't done that before but I think I can try. 22:33
JayFbasically you want your branch to have both commits on it, in the right order22:34
JayFand when you run "git review" it'll ask if you mean to create/update two patches22:34
TheJuliaYes, and you can use "git review -x" to place one after the other22:34
TheJulia(if you haven't changed them)22:34
JayFthe easiest way to get to this point IME: `git review -d $firstchange`, `git review -x $secondchange`, git review22:34
* TheJulia confirms "this is the way"22:34
JayF(-d = download, -x = cherry pick FWIW)22:35
JayFthere's also -X which cherry picks, one puts a "Cherry picked from SHA" message in the commit22:35
JayFand I can never remember which is which without checking manpages22:35
TheJuliaI've done it enough that I know -X is with the commit message change22:36
JayFit's like a USB port for me22:37
JayFwhichever one I use the first time is not the one I wanted22:38
opendevreviewSatoshi Shirosaka proposed openstack/ironic-python-agent master: Ironic Config Intergration for IPA  https://review.opendev.org/c/openstack/ironic-python-agent/+/94495122:54
opendevreviewSatoshi Shirosaka proposed openstack/ironic master: Ironic Container Cofig for IPA Integration  https://review.opendev.org/c/openstack/ironic/+/94479623:00
satoshiI think I stacked right for IPA-B. I took off WIP from the second batch.23:02
TheJuliaIPA-B can't be stacked with the other patches23:04
TheJuliabut, I think you had a depends-on tag in the commit message which is good and serves a similar purpose for cross-repo items23:04
satoshiYeah, Jay asked me to add the tag. 23:06
satoshiI don't get this. Do I need to change something?23:06
satoshi> IPA-B can't be stacked with the other patches23:06
opendevreviewMerged openstack/ironic-python-agent-builder master: Create Podman DIB Element for container-based cleaning  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/94334823:06
TheJuliaWell, there you go, you don't need the tag now23:07
TheJulia:)23:07
TheJulianor need to stack it, focus on the IPA patches now, not IPA-B :)23:07
TheJuliayou can only stack patches inside of the same repository23:07
satoshiOh yes sorry I mean IPA.23:07
satoshiI mixed up IPA with IPA-B23:08
TheJuliafor things crossing outside of a single repository, we have a notation for commit messages "Depends-On" or "Needed-By"23:08
TheJuliaDepends-On is a firm requirement, needed-by is for humans23:08
TheJuliano worries!23:08

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!