| cid | I can't just place how my change is breaking the wholedisk_image test which I believe might also be responsible for the state change waiting timeouts. | 03:07 |
|---|---|---|
| cid | I know it's there, somewhere, I could really use a hand here... https://review.opendev.org/c/openstack/ironic/+/917229. | 03:07 |
| iurygregory | good morning Ironic | 11:27 |
| dking | dtantsur: Would you be available this monring? | 12:22 |
| opendevreview | cid proposed openstack/ironic-tempest-plugin master: Patch to enforce json extension works in existing API behaviour https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/913926 | 12:31 |
| opendevreview | cid proposed openstack/ironic-tempest-plugin master: Testing aarch64 fake-bare-metal-vms https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918448 | 13:10 |
| opendevreview | cid proposed openstack/ironic master: wip: Provision aarch64 fake-bare-metal-vms https://review.opendev.org/c/openstack/ironic/+/915441 | 13:12 |
| TheJulia | o/ dking | 13:31 |
| TheJulia | dking: a specification document under development from discussions during our most recent ptg which you might find very interesting: https://review.opendev.org/c/openstack/ironic-specs/+/916126 | 13:33 |
| dtantsur | dking: only partly - nearly on PTO | 13:42 |
| dking | TheJulia: Thank you. I'll have to look at it. | 13:49 |
| opendevreview | cid proposed openstack/ironic-tempest-plugin master: Patch to enforce json extension works in existing API behaviour https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/913926 | 14:22 |
| opendevreview | cid proposed openstack/ironic-tempest-plugin master: Testing aarch64 fake-bare-metal-vms https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918448 | 14:27 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: Match publisher id https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/915170 | 14:27 |
| opendevreview | Kaifeng Wang proposed openstack/python-ironicclient master: Client support port name https://review.opendev.org/c/openstack/python-ironicclient/+/896067 | 14:29 |
| opendevreview | Maximilian Brandt proposed openstack/ironic-python-agent master: Add bmc_subnet information to introspection and Consolidated BMC information retrieval logic into a single function _get_bmc_info(). https://review.opendev.org/c/openstack/ironic-python-agent/+/918340 | 15:33 |
| opendevreview | Maximilian Brandt proposed openstack/ironic-python-agent master: Add bmc_subnet information to introspection https://review.opendev.org/c/openstack/ironic-python-agent/+/918340 | 15:44 |
| opendevreview | Verification of a change to openstack/ironic-python-agent stable/2023.1 failed: Unmount config drives https://review.opendev.org/c/openstack/ironic-python-agent/+/917852 | 15:45 |
| opendevreview | cid proposed openstack/ironic-tempest-plugin master: Patch to enforce json extension works in existing API behaviour https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/913926 | 15:49 |
| TheJulia | A quick and easy tempest change: https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/917277 | 15:53 |
| TheJulia | Another quick and easy tempest change: https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918001 | 15:54 |
| JayF | I was late but did it: https://bugs.launchpad.net/ironic-python-agent/+bug/2061362/comments/3 | 15:54 |
| JayF | Hey so I just wanted to let the larger community know, cid is going to be continuing to work with us upstream on behalf of GR-OSS even though his MLH fellowship is over. \o/ | 15:55 |
| cid | I'm honored :D | 15:56 |
| masghar | Congratulations and welcome, cid! | 16:07 |
| cid | Thank you | 16:09 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: reboot the node in basic ops tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918462 | 16:12 |
| TheJulia | Congrats! | 16:12 |
| opendevreview | cid proposed openstack/ironic-tempest-plugin master: Patch to enforce json extension works in existing API behaviour https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/913926 | 16:13 |
| cid | Thanks \o/ | 16:14 |
| dtantsur | Folks, I'll be on PTO until next Tuesday. The IRC bouncer will be off, so drop me an email if you need to contact. | 16:23 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fix spurious CI job failures around partition images https://review.opendev.org/c/openstack/ironic/+/918466 | 16:57 |
| TheJulia | dtantsur: enjoy! | 16:59 |
| iurygregory | enjoy the PTO dtantsur o/ | 17:17 |
| iurygregory | congratulations cid =) | 17:18 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: detail limitation of use of config drive + rebuild https://review.opendev.org/c/openstack/ironic/+/918469 | 17:31 |
| opendevreview | Merged openstack/ironic master: docs: detail limitation of use of config drive + rebuild https://review.opendev.org/c/openstack/ironic/+/918469 | 18:27 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: reboot the node in basic ops tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918462 | 18:38 |
| JayF | clarkb: I'd be very interested to see your script, and see if we can use it to spin up nodes quicker in the Ironic devstack plugin, even if capital-D-devstack doesn't like that approach | 18:53 |
| TheJulia | huh? | 18:58 |
| clarkb | TheJulia: related to the TC meeting | 18:59 |
| clarkb | JayF: its a devsatck change owned by me that may be abandoned or open but not merged | 18:59 |
| clarkb | I think if the thing dansmith is talking about works though that may be a better otpion as then you're still achieving the goal of using the standard tooling to set up and be an example | 18:59 |
| JayF | TheJulia: tl;dr clarkb has an old devstack change somewhere that uses a osdk script to do a lot of the stuff currently done in devstack via OSC CLI | 18:59 |
| JayF | and supposedly it sped a bunch of things up | 19:00 |
| JayF | I just thought "hey, we have a loop where we call a bunch of update nodes, I bet that'd be an easy win" but then later dan and clark may have found a cooler way to get the same win | 19:00 |
| TheJulia | ok | 19:00 |
| clarkb | https://review.opendev.org/c/openstack/devstack/+/673018/7 and child | 19:01 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: reboot the node in basic ops tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918462 | 19:04 |
| TheJulia | okay, that should clear the syntax error | 19:04 |
| TheJulia | JayF: fyi ^ | 19:10 |
| JayF | thanks for that! Wanna pick the color I use to mark it off my list :D | 19:11 |
| TheJulia | Hmm... https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_dd1/915441/20/check/ironic-standalone-aarch64/dd12919/controller/logs/libvirt/libvirt/qemu/node-2_log.txt we're going to need to find a way to observe whatever is failing | 19:15 |
| JayF | wow | 19:23 |
| JayF | getting to that log at all 👏 | 19:23 |
| TheJulia | much more an artifact of knowing where to look | 19:23 |
| clarkb | check syslog maybe? | 19:42 |
| clarkb | nope nothign in that fiel around those time stamps other than iptables filters | 19:43 |
| opendevreview | Maximilian Brandt proposed openstack/ironic-python-agent master: Add bmc_subnet information to introspection https://review.opendev.org/c/openstack/ironic-python-agent/+/918340 | 20:10 |
| TheJulia | found it | 20:17 |
| TheJulia | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_dd1/915441/20/check/ironic-standalone-aarch64/dd12919/controller/logs/screen-virtualbmc.txt | 20:17 |
| cid | TheJulia: I've seen the feedback, thanks. | 20:27 |
| cid | Good night \o | 20:27 |
| TheJulia | goodnight | 20:27 |
| clarkb | TheJulia: oh ya there is a devstack change to fix that. I thought it had merged though | 20:27 |
| clarkb | oh except I think it may be an override in lib/nova ratehr than a top level var reset | 20:28 |
| clarkb | so your custom virtual baremetal stuff may need to use a similar override (or bubble it up to top level) | 20:28 |
| opendevreview | Maximilian Brandt proposed openstack/ironic-python-agent master: Add bmc_subnet information to introspection https://review.opendev.org/c/openstack/ironic-python-agent/+/918340 | 20:28 |
| TheJulia | clarkb: for what specifically? | 20:28 |
| clarkb | TheJulia: for host-model cpu stuff | 20:29 |
| TheJulia | clarkb: we have an entirely custom xml payload | 20:30 |
| clarkb | ah ok | 20:30 |
| clarkb | in the regular devstack case it was trying to do a q6600 nehalem or whatever and then you can't do that on arm. This ist he other way around | 20:30 |
| opendevreview | Maximilian Brandt proposed openstack/ironic-python-agent master: Add bmc_subnet information to introspection https://review.opendev.org/c/openstack/ironic-python-agent/+/918340 | 20:31 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: reboot the node in basic ops tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918462 | 23:35 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!