cardoe | How do I generate the hash suffix for a releasenote? | 02:20 |
---|---|---|
opendevreview | Doug Goldstein proposed openstack/python-ironicclient master: support passing disable_ramdisk for clean state https://review.opendev.org/c/openstack/python-ironicclient/+/924895 | 02:37 |
cardoe | Well I tried to change the BIOS setting on those Dells with disable_ramdisk after Julia mentioned that. But the CLI didn't support sending that. So I believe the above will enable it in the library and the CLI. So I ended up sending with curl but alas the bios interface for redfish doesn't support setting settings without booting the ramdisk. Not really sure why. | 03:14 |
cardoe | I didn't make a hash suffix for my release note for the above change but happy to fix it. | 03:15 |
JayF | cardoe: `tox -e venv -- reno new short-description-of-change` creates the file. You can also just run reno from the root of the repo installed/accessed some other way (e.g. your own venv) | 03:30 |
cardoe | Thank you JayF | 03:36 |
JayF | np, thanks for putting in release notes :D | 03:38 |
opendevreview | Doug Goldstein proposed openstack/python-ironicclient master: support passing disable_ramdisk for clean state https://review.opendev.org/c/openstack/python-ironicclient/+/924895 | 03:39 |
opendevreview | Doug Goldstein proposed openstack/python-ironicclient master: support passing disable_ramdisk for clean state https://review.opendev.org/c/openstack/python-ironicclient/+/924895 | 03:54 |
cardoe | You've gotten me to install tox again. And made me remember why I dislike flake8... (if ya know the problem.... fix it!) | 04:39 |
opendevreview | Doug Goldstein proposed openstack/python-ironicclient master: support passing disable_ramdisk for clean state https://review.opendev.org/c/openstack/python-ironicclient/+/924895 | 04:44 |
rpittau | good morning ironic! o/ | 07:15 |
rpittau | looks like we're hitting the same issue with https://review.opendev.org/c/openstack/devstack/+/924867 :/ | 07:20 |
rpittau | frickler any advice? ^ | 07:21 |
frickler | rpittau: hmm, maybe ask some swift people, then. I'll also hold a node to look at the issue in place | 07:32 |
rpittau | ack | 07:33 |
frickler | actually, I'm seeing the same error in the kolla swift job, so it is likely something did change in osc or some lib | 07:33 |
rpittau | weird, I think we pin osc and it was not changed recently AFAICS | 07:36 |
rpittau | yep, as I checked yesterday, we use osc 6.6.0 | 07:37 |
frickler | yes, I just confirmed that in the logs, so library change then. sadly the pip-freeze output from devstack is useless, as it doesn't show the global_venv contents | 08:01 |
rpittau | I asked for help in the swift channel, fingers crossed | 08:04 |
frickler | hmm, I can reproduce this locally. just install osc into a venv and do "openstack help | grep object" => no swift commands, only some image metadef stuff | 08:43 |
frickler | "... install osc from pypi". osc installed from git works fine | 08:43 |
* frickler smells some weird python/virtualenv stuff happening | 08:44 | |
frickler | actually the issue is reproducible if I install the 6.6.0 tag from git, too. so I guess I can try to git bisect now | 08:49 |
frickler | but let's take this to #openstack-sdks then | 08:49 |
opendevreview | Jorhson Deng proposed openstack/ironic-inspector master: Use the ProcessLauncher to start the service https://review.opendev.org/c/openstack/ironic-inspector/+/924911 | 09:06 |
*** blady is now known as Guest1228 | 09:07 | |
rpittau | thanks frickler ! | 09:07 |
masghar | Thanks frickler and rpittau for taking a look! | 09:17 |
opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake system driver docs https://review.opendev.org/c/openstack/sushy-tools/+/924920 | 10:58 |
iurygregory | good morning ironic | 11:16 |
opendevreview | Himanshu Roy proposed openstack/ironic master: add virtual media GET api to fetch all details related to virtual media devices for a given node, add unit tests, updated api version https://review.opendev.org/c/openstack/ironic/+/921657 | 11:23 |
iurygregory | do we have a fix for the openstack-cli-server on our side or other repo? | 11:48 |
TheJulia | Good morning, before I head to the airport in a little bit | 11:57 |
iurygregory | good morning TheJulia, safe travels! | 12:02 |
mohammed | Can someone help with failure here on this tiny docs patch https://review.opendev.org/c/openstack/sushy-tools/+/924920 | 12:10 |
iurygregory | mohammed, is not related to your patch =( | 12:10 |
opendevreview | Stephen Finucane proposed openstack/python-ironicclient master: Remove use of distutils https://review.opendev.org/c/openstack/python-ironicclient/+/914523 | 12:10 |
iurygregory | we had this problem since yesterday | 12:10 |
frickler | iurygregory: no fix yet, but we're getting closer, current discussion in #-sdks | 12:11 |
iurygregory | frickler, tks! | 12:11 |
stephenfin | iurygregory: JayF: dtantsur: Can you folks merge that ^ when you get a chance. python-ironicclient is currently broken on Python 3.12, which is the default Python on Fedora 40 (and had been out since October 2023) | 12:11 |
mohammed | Thanks iurygregory, so no action I will be waiting for this to get solved | 12:12 |
iurygregory | mohammed, yeah, I will trigger a recheck once we get the issue fixed =) | 12:12 |
iurygregory | stephenfin, ack, will check | 12:12 |
stephenfin | ta | 12:13 |
rpittau | stephenfin: we need the fix for osc first :D | 12:13 |
iurygregory | rpittau, lol :D | 12:13 |
stephenfin | they're mutually exclusive ;) | 12:14 |
jamesdenton_alt | Good morning. Does Ironic support Neutron's routed networks (segments) functionality? And if so, is it as simple as ensuring the baremetal port(s) for computes in the respective racks define the proper physical_network? | 12:26 |
jamesdenton_alt | https://docs.openstack.org/ironic/latest/install/configure-networking.html seems to imply the baremetal ML2 driver is also needed. Will give that a go. | 12:30 |
*** blady is now known as Guest1235 | 12:43 | |
*** Guest1235 is now known as gryf | 12:43 | |
opendevreview | Merged openstack/python-ironicclient master: Remove use of distutils https://review.opendev.org/c/openstack/python-ironicclient/+/914523 | 13:52 |
dtantsur | stephenfin: ^^ | 13:58 |
stephenfin | dtantsur: thanks \o/ | 13:59 |
TheJulia | jamesdenton_alt: The networking-baremetal ml2 driver, and yes when running afaik it is just as simple as making the physical_network map | 14:14 |
TheJulia | networking-baremetal does the leg work with neutron | 14:14 |
rpittau | workaround for osc issue is working -> https://review.opendev.org/c/openstack/ironic/+/924868 | 14:15 |
rpittau | it's this if you're wondering https://review.opendev.org/c/openstack/devstack/+/924867 | 14:15 |
rpittau | it'll be merged soon | 14:15 |
rpittau | thanks frickler :) | 14:15 |
iurygregory | nice | 14:16 |
iurygregory | :D | 14:16 |
dtantsur | \o/ | 14:16 |
iurygregory | pip install -U ismplejson lol | 14:16 |
rpittau | yep | 14:16 |
rpittau | requirements was removed, but not dependency ehm.... | 14:16 |
iurygregory | it's fine | 14:17 |
jamesdenton_alt | Thanks TheJulia - was hoping you'd say that | 14:18 |
TheJulia | ... do I want to know what chaos is in progress? | 14:19 |
jamesdenton_alt | Just Ironic To The Max™ | 14:19 |
TheJulia | oh my | 14:22 |
opendevreview | Himanshu Roy proposed openstack/ironic master: add virtual media GET api to fetch all details related to virtual media devices for a given node, add unit tests, updated api version https://review.opendev.org/c/openstack/ironic/+/921657 | 14:23 |
opendevreview | Himanshu Roy proposed openstack/ironic master: add virtual media GET api https://review.opendev.org/c/openstack/ironic/+/921657 | 14:31 |
opendevreview | Doug Goldstein proposed openstack/ironic master: don't add empty mac address on redfish inspect https://review.opendev.org/c/openstack/ironic/+/924943 | 14:48 |
clarkb | rpittau: if simplejson is a hard dependency for making osc function why did we add the package to devstack installation process and not back into osc? | 15:53 |
clarkb | shouldn't we fix osc so taht the tool work for everyone and not just those using devstack? | 15:53 |
rpittau | clarkb: that's a workaround, the fix is already there but in an unreleased version, please check the discussion in #opensatck-sdks | 15:53 |
rpittau | errr or even #openstack-sdks | 15:54 |
JayF | rpittau: do you mind if I create the etherpad for next PTG? Just have some ideas I want to jot down so they aren't bouncing around in my head/in my notes | 15:54 |
rpittau | JayF: please go ahead, I was going to do it anyway this week and present it during the next meeting | 15:54 |
clarkb | rpittau: I see | 15:55 |
clarkb | rpittau: also is that a different issue to the openstack-cli-server object store command failure? or are they the same issue? | 15:56 |
rpittau | clarkb: it's the same issue | 15:56 |
clarkb | ack thanks | 15:56 |
rpittau | the root cause is the removal of simplejson from osc reqs | 15:56 |
rpittau | good night! o/ | 16:05 |
JayF | \o | 16:07 |
JayF | btw rpittau, that PTG page is up, in the whiteboard | 16:07 |
JayF | ^ linked | 16:07 |
cardoe | How do folks handle booting into IPA when the box doesn't have good time? Just hit a box creating an expired cert. I'm not seeing this anywhere in the docs. | 16:13 |
JayF | We support doing a time sync on start. I think if you send the DHCP opts we'll use it, or you can set ntp servers | 16:14 |
JayF | add ipa-ntp-server=ip-of-ntp-server to your kernel command line when booting IPA | 16:15 |
JayF | I think pxe_append_params is the config arg? | 16:16 |
JayF | ^^^ I can't find docs for this though, we should document it | 16:16 |
cardoe | okay. I'll tinker and submit something for the docs. | 16:20 |
cardoe | I was using the ipa based on debian bookworm and saw that timesyncd wasn't installed so I might have to install that via a DIB | 16:21 |
cardoe | It's "rc" when I did dpkg -l | 16:21 |
JayF | Might be something to ask zigo to put in those debian ramdisks by default | 16:26 |
zigo | We have chrony which much much better than the systemd implementation. | 16:28 |
zigo | Any valid reason for the switch ? | 16:29 |
zigo | Were you using our generic image ? | 16:31 |
cardoe | Where's the generic image? | 16:32 |
cardoe | No reason for the switch. I like chrony better. | 16:32 |
cardoe | https://tarballs.opendev.org/openstack/ironic-python-agent/ is where I had seen from docs which only had tinyipa which doesn't have enough drivers. | 16:33 |
*** iurygregory__ is now known as iurygregory | 16:35 | |
JayF | Oooh, you're using the *upstream debian* images | 16:35 |
JayF | I thought you were using the ones that debian ships | 16:36 |
JayF | my mistake | 16:36 |
cardoe | I used the ironic-python-agent-builder | 17:08 |
JayF | Yeah, then just adding an ntp element should be the trick. I'm slightly surprised we don't do one by default, but I haven't checked | 17:11 |
opendevreview | Himanshu Roy proposed openstack/ironic master: add virtual media GET api https://review.opendev.org/c/openstack/ironic/+/921657 | 17:35 |
*** iurygregory__ is now known as iurygregory | 17:57 | |
opendevreview | cid proposed openstack/python-ironicclient master: Add CLI support for runbooks https://review.opendev.org/c/openstack/python-ironicclient/+/924101 | 18:45 |
opendevreview | cid proposed openstack/ironic master: Self-Service via Runbooks https://review.opendev.org/c/openstack/ironic/+/922142 | 19:07 |
opendevreview | cid proposed openstack/ironic master: Self-Service via Runbooks https://review.opendev.org/c/openstack/ironic/+/922142 | 19:18 |
opendevreview | cid proposed openstack/ironic master: Self-Service via Runbooks https://review.opendev.org/c/openstack/ironic/+/922142 | 19:20 |
*** ex_tnode8 is now known as ex_tnode | 19:28 | |
JayF | Our devstack plugin doesn't seem to be very ... robust when unstacking/restacking to change code. Going to make some notes here: https://etherpad.opendev.org/p/ironic-devstack-issues | 20:20 |
opendevreview | cid proposed openstack/python-ironicclient master: Add CLI support for runbooks https://review.opendev.org/c/openstack/python-ironicclient/+/924101 | 20:23 |
opendevreview | cid proposed openstack/python-ironicclient master: Add CLI support for runbooks https://review.opendev.org/c/openstack/python-ironicclient/+/924101 | 20:27 |
JayF | hmm, looks like just that one dnsmasq thing, going to see if I can knock that out with a simple fix | 20:34 |
opendevreview | Jay Faulkner proposed openstack/ironic master: [devstack] Ensure downgrade_dnsmasq is idempotent https://review.opendev.org/c/openstack/ironic/+/924978 | 20:51 |
JayF | ^^ should be a fix for that issue I just ID'd | 20:51 |
JayF | automatic lessee tested working in devstack \o/ \o/ \o/ | 21:09 |
JayF | all the new metadata looks slick, too | 21:12 |
ashinclouds[m] | Awesome | 21:13 |
JayF | https://review.opendev.org/c/openstack/nova/+/923910 + https://review.opendev.org/c/openstack/ironic/+/924887 are tested working in devstack. Still have to write a small migration script and some tempest tests, but wanted to get general agreement on the shape of the change before going further. If you could have a look I'd appreciate it. | 21:13 |
JayF | you, in this case, == any ironic cores :D | 21:13 |
JayF | I have a couple of open questions in the automatic_lessee implementation. Wouldn't be surprised if you disagree with some of my choices there. | 21:14 |
cid | o/ | 21:26 |
JayF | \o | 21:30 |
TheJulia | I likely won’t have time to look tomorrow, will be playing it by ear. | 22:03 |
JayF | if you end up with a small amount of time to spare, just weighing in on the two questions I highlighted in the ironic side of the change would be useful (I put self-review comments there to highlight) | 22:03 |
TheJulia | Okay, I’m trying to mentally unwind atm, next flight in 3.5 hours | 22:05 |
JayF | ++ please do | 22:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!