haozhou | Hi, I have authored an RFE https://bugs.launchpad.net/ironic/+bug/2040236 concerning the implementation of proposal #1395(https://github.com/openshift/enhancements/pull/1395) in Ironic. | 06:14 |
---|---|---|
haozhou | Your insights and opinions would be greatly appreciated, feel free to review it. | 06:14 |
rpittau | good morning ironic! o/ | 06:48 |
masghar | Good morning! | 08:19 |
*** jroll05 is now known as jroll0 | 08:57 | |
iurygregory | good morning | 12:04 |
iurygregory | haozhou, tks for creating the RFE, I think it would be good to add it to RFE Review in our Weekly meeting https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting (next meeting is Oct 30, 15:00 UTC here in the irc) | 12:08 |
TheJulia | good morning | 12:36 |
iurygregory | good morning TheJulia =) | 12:37 |
TheJulia | haozhou: Additionally, we *really* cannot accept patches through bug reports without the CLA having been signed. In part this is enforced by our code review system. If you could upload a change into gerrit, that would help ensure that requirement is satisfied. | 12:37 |
iurygregory | JayF, it would be good to update https://ptg.opendev.org/ptg.html with what we did and the next topic we will start =) | 15:49 |
* iurygregory goes for lunch | 15:49 | |
JayF | https://bugs.launchpad.net/ironic-python-agent/+bug/1751077 https://bugs.launchpad.net/ironic-python-agent/+bug/1817347 are the only two I have questions about | 15:50 |
JayF | others I was able to triage myself | 15:50 |
JayF | after those are done, IPA is done | 15:50 |
TheJulia | That latter one I have questions myself | 15:54 |
TheJulia | like... how did they cause that to happen?! | 15:54 |
JayF | my hunch? bad image | 15:55 |
JayF | like probably some kind of LVM setup | 15:55 |
JayF | with what's the word | 15:55 |
JayF | sparse | 15:55 |
JayF | sparse LVM pvs or something | 15:55 |
JayF | and confused whatever made the image, perhaps? | 15:55 |
JayF | Ironic'ers: I've been asked to do a group photo at some point in PTG | 17:16 |
JayF | I will likely do so tomorrow | 17:16 |
JayF | so please put on your good webcam :D | 17:17 |
JayF | or at least an extra-funny zoom avatar :P | 17:17 |
iurygregory | JayF, ack | 17:34 |
iurygregory | Interesting, seems like sqlite is mad with Firmware Interface https://github.com/metal3-io/ironic-image/pull/438#issuecomment-1777498656 O.o | 17:35 |
iurygregory | but in our CI the change was green O.o (in metal3-integration...) | 17:36 |
JayF | Do we have a place to post OpenStack jobs? | 17:37 |
JayF | G-R is hiring for Openstack engys in Dallas, curious if there's an openinfra or openstack job board I can point them at to get more spread | 17:38 |
iurygregory | JayF, probably kendal or alisson know the palce for it | 17:38 |
iurygregory | if I remember there is a place for it | 17:39 |
iurygregory | JayF, https://www.openstack.org/community/jobs/ | 17:39 |
JayF | ack; I'll google around a bit more | 17:39 |
clarkb | I suspect helena manages that these days | 17:39 |
clarkb | JayF: let me ask | 17:39 |
iurygregory | seems like you just need to provide the information after clicking in Post Job Now | 17:40 |
clarkb | oh cool its all automatic even better | 17:40 |
clarkb | the process is you submit them there then Helena will review and approve things | 17:41 |
JayF | wonderful, thank you I've passed that on | 17:47 |
JayF | if you know anyone, feel free to point them in my direction; don't worry though, they won't be subjected to working directly with me :P | 17:47 |
opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Get numa_node info when collecting pci devices info https://review.opendev.org/c/openstack/ironic-python-agent/+/369245 | 17:56 |
JayF | https://review.opendev.org/c/openstack/ironic-python-agent/+/369245 bwahaha that change was so old JENKINS voted on it | 17:57 |
JayF | and it might end up merging after a super trivial rebase | 17:57 |
JayF | that's great | 17:57 |
TheJulia | brraaaains | 18:03 |
JayF | that needed a lot more than a rebase but it's been a nice distraction | 20:51 |
JayF | pretty straightforward and I've not done a lot in inspector | 20:51 |
JayF | Toss a thank you over to boushra, because I did this on my WSL, which can run IPA tests because of her bugfix \o/ | 21:27 |
opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Get numa_node info when collecting pci devices info https://review.opendev.org/c/openstack/ironic-python-agent/+/369245 | 21:27 |
iurygregory | ok, people are trying to use sushy-tools in aarch64... | 21:34 |
JayF | oh, that's cool | 21:34 |
JayF | like running it in aarch64? | 21:34 |
JayF | I mean, I don't think we've ever tried it, I don't know a reason it won't work, but I bet *you do* now! LOL | 21:34 |
iurygregory | seems like it | 21:35 |
iurygregory | byt it's not working | 21:35 |
iurygregory | but*4 | 21:35 |
iurygregory | I don't think we have support to provide vms in aarch64 (not sure if we would need to change something on it tbh) | 21:36 |
iurygregory | ok, seems like we do have configs on it in libvirt driver | 21:37 |
samcat116 | For NGS, does adding a new switch to the ml2 conf require a restart of neutron-server? | 21:37 |
iurygregory | samcat116, I would assume it would require a restart on services that look at the config file | 21:42 |
TheJulia | iurygregory: if created, I would think it would work, but... yeah. the different emulated vm type is weird on x86 but that would be different if you were running on an aarch64 machine | 21:43 |
iurygregory | depending if the confg option is mutable or not (like if you can just override values etc) | 21:43 |
TheJulia | I'm thinking the same thing | 21:43 |
TheJulia | does not appear mutable | 21:44 |
iurygregory | TheJulia, yeah, I'm trying to understand if they want a virtual BM with x86 and sushy-tools is running on a aarch64 HW (hub spoke depoyment) | 21:44 |
TheJulia | patches welcome! | 21:44 |
iurygregory | if that's the case this would be a crazy thing =X | 21:44 |
TheJulia | crazy is all relative | 21:44 |
iurygregory | well, openshift is not supporting multi-arch deployments afaik atm | 21:45 |
TheJulia | did the scheduler ever get support to handle the difference? | 21:45 |
JayF | sushy-tools, and upstream openstack, supports more than that; and there is upstream user call for ARM/x86 deployed by x86 servers, at least | 21:45 |
JayF | the other way around ... doesn't make much sense to me | 21:46 |
JayF | but I guess is more and more likely as time goes by (ARM openstack provisioning x86 ironic) | 21:46 |
iurygregory | yeah | 21:46 |
JayF | btw https://review.opendev.org/c/openstack/ironic-python-agent/+/369245 should be reviewable now, trivial, passing unit and flake tests locally | 21:48 |
JayF | if we wanna land the oldest change ever | 21:48 |
JayF | send email to the most fun set of openstack alum :D | 21:48 |
TheJulia | heh | 21:54 |
opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Remove standby.cache_image support https://review.opendev.org/c/openstack/ironic-python-agent/+/899214 | 21:57 |
TheJulia | funky wrapping https://review.opendev.org/c/openstack/ironic-python-agent/+/899214/1/releasenotes/notes/cache-image-removal-3b5a80a6038a320b.yaml | 21:58 |
JayF | egad | 22:01 |
JayF | I bet that's windows line endings | 22:01 |
JayF | and idea did not do me a solid | 22:01 |
* TheJulia feels frightened | 22:02 | |
JayF | my "line" in that IDE is not configured and apparently defaults to 120, so that's the actual issue | 22:02 |
opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Remove standby.cache_image support https://review.opendev.org/c/openstack/ironic-python-agent/+/899214 | 22:02 |
JayF | basically been using PTG as an excuse to get some kinda "useful for small stuff" dev going in my WSL on my gaming desktop | 22:03 |
JayF | since I've been windowsing for the ptg, anyway | 22:03 |
JayF | getting up from the pc for the day, see you in the morn o/ | 22:04 |
TheJulia | goodnight | 22:09 |
TheJulia | cool cool | 22:09 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: WIP: Add Glean into TinyIPA image https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/898242 | 23:16 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: WIP/DNM: Get rid of simple-init (but keep glean) https://review.opendev.org/c/openstack/ironic-python-agent/+/895519 | 23:54 |
TheJulia | I'm going to have to work on that a bit more, I change the logic to more aggressively undo the running state of any older glean in the flow and stepped on my own toes | 23:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!