| opendevreview | Merged openstack/sushy-tools master: Handle BIOS Firmware Update the same way as real hardware https://review.opendev.org/c/openstack/sushy-tools/+/990166 | 06:35 |
|---|---|---|
| opendevreview | Merged openstack/ironic master: Correct the change number linked in docs for is_root_volume https://review.opendev.org/c/openstack/ironic/+/990085 | 07:43 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix send_sensor_data periodic with oslo.service spawn workers https://review.opendev.org/c/openstack/ironic/+/990251 | 07:51 |
| opendevreview | Merged openstack/ironic-python-agent master: Update pre-commit bandit version to 1.9.4 https://review.opendev.org/c/openstack/ironic-python-agent/+/990196 | 09:11 |
| estherd[m] | Hi everyone. I'm working on the node history API. My first patch (989994) is failing the test_models_sync test with KeyError: 'state'. I've added state, entry_at, exit_at columns to NodeHistory model and created a migration. Does anyone know where test_models_sync is defined, or what I need to update? | 10:32 |
| estherd[m] | This is the link to the error log: https://zuul.opendev.org/t/openstack/build/1b16ab5214cc44e9a482219dcdc9275a | 10:32 |
| estherd[m] | Any help would be appreciated. Thanks! | 10:32 |
| hjensas | TheJulia: cardoe: Hit a snag with NETCONF, I created: https://bugs.launchpad.net/networking-generic-switch/+bug/2154349 - it is an RFE that would mitigate. And I think it'd be a general improvement for netmiko devices as well? | 11:22 |
| dtantsur | estherd[m]: hi! You can always check the full name of a test to see where it's located, e.g. in this case ironic.tests.unit.db.sqlalchemy.test_migrations.ModelsMigrationsSyncMysql.test_models_sync - just follow the modules. | 11:43 |
| dtantsur | estherd[m]: however, in the job you've linked, this test has *not* failed | 11:43 |
| dtantsur | estherd[m]: the "Artifacts" section of any job run will give you a link to the summary file that looks like this: https://6e19fc6a550f8706f583-0bc86d5c9d8beccf8b3e541f45742cd9.ssl.cf5.rackcdn.com/openstack/1b16ab5214cc44e9a482219dcdc9275a/testr_results.html | 11:44 |
| dtantsur | If you expand a test result, you'll see the traceback of the KeyError in question, it may give you enough hints to understand | 11:45 |
| dtantsur | to run a subset of tests locally, you can use regular expressions like $ tox -epy3 -- TestNodeHistoryObject | 11:45 |
| dtantsur | (TestNodeHistoryObject is a trivial regular expression in this case) | 11:45 |
| iurygregory | good morning ironic | 12:22 |
| TheJulia | Braaains | 13:10 |
| TheJulia | hjensas: not a bad idea.. we’ve sort of talked about doing as much before as well. | 13:12 |
| TheJulia | The only downside is the switch physnet would need to be declared. | 13:14 |
| TheJulia | Since they are more a human infrastructure management context, that may be okay. | 13:15 |
| TheJulia | The downside is say there are 500 or a thousand vlans in a physnet. | 13:15 |
| hjensas | TheJulia: yeah, it's a bit strange that we allow multiple physnets on one switch. If two physnets use the same vlan range that would be chaos. | 13:19 |
| hjensas | But, I'll see if I can make this work as a "required for NETCONF", and follow up with only applies if physnet is defined for netmiko drivers. | 13:20 |
| hjensas | ... and yeah may want to have something that build 10,12,20-27 type single commands for netmiko devices for that lot's of networks case. | 13:22 |
| TheJulia | oh, yeah, absolutely. Like, CVE territory chaos rooted in a misconfiguration we can't defend folks from potentially making | 13:25 |
| TheJulia | I do really like that last bit of an idea though ;) | 13:25 |
| cardoe | TheJulia, hjensas: yep doing all the VLANs at once makes sense. That’s what our mechanism does. Slow convergence. | 13:51 |
| cardoe | We also have a hard rule of one physnet per switch. | 13:51 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: docs: Clean up the diagram to be more explicit https://review.opendev.org/c/openstack/networking-baremetal/+/990294 | 13:52 |
| TheJulia | cardoe: you might just want to mash +2+A on that | 13:53 |
| cardoe | I can workflow it. | 14:13 |
| cardoe | It now looks like the one from the doc I tried to land in neutron. | 14:13 |
| cardoe | Maybe on the plane I’ll work on docs. | 14:13 |
| cardoe | Got 2 layovers today. DTW and SEA | 14:13 |
| * TheJulia senses Island Time will soon arrive | 14:15 | |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: docs: fix docs build https://review.opendev.org/c/openstack/networking-baremetal/+/990304 | 14:34 |
| TheJulia | cardoe: that ^^^ should fix the docs build.. | 14:34 |
| hjensas | TheJulia: ngs as well - https://review.opendev.org/c/openstack/networking-generic-switch/+/990088/1 (locally, I have to add pyroute2 to test-requirements as well for unittests, but that works in CI) | 15:00 |
| TheJulia | ugh | 15:03 |
| opendevreview | Merged openstack/ironic stable/2025.1: security: move file url validation up into deploy_utils main path https://review.opendev.org/c/openstack/ironic/+/988357 | 15:14 |
| opendevreview | Merged openstack/networking-generic-switch master: Add pyroute2 to doc requirements https://review.opendev.org/c/openstack/networking-generic-switch/+/990088 | 15:20 |
| estherd[m] | dtantsur: Got it. Thank you! I'll look at the test report and check as explained. Really appreciate the help. | 15:30 |
| cardoe | TheJulia: docs build no happy | 16:25 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: docs: fix docs build https://review.opendev.org/c/openstack/networking-baremetal/+/990304 | 16:28 |
| TheJulia | cardoe: okay, hopefully that is happier | 16:51 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: docs: fix docs build https://review.opendev.org/c/openstack/networking-baremetal/+/990304 | 16:52 |
| * TheJulia tosses a wet cat at CI | 16:52 | |
| opendevreview | Merged openstack/ironic unmaintained/2024.1: Shell-quote console command passed to `socat` https://review.opendev.org/c/openstack/ironic/+/986417 | 17:24 |
| opendevreview | cid proposed openstack/ironic master: Honor `add_ports=disabled` in OOB port creation https://review.opendev.org/c/openstack/ironic/+/990340 | 17:29 |
| cid | cardoe, would you have 20 to 25 minutes to chat about the maintenance mode feature? Some time this week or next. | 17:44 |
| cid | I already am getting actionable feedback on the draft spec https://review.opendev.org/c/openstack/ironic-specs/+/974894 | 17:44 |
| cid | Will appreciate an operator perspective | 17:45 |
| TheJulia | I believe cardoe is on PTO for the next week, although obviously today is a travel day for him and he is keeping his mind busy while on the metal tubes | 17:47 |
| cardoe | Yeah. I’m in the airport right now and about to get some lunch. | 17:47 |
| TheJulia | heh | 17:48 |
| cardoe | https://6411f27413aeaeed863a-624a66b127421bb56fbada761a704171.ssl.cf5.rackcdn.com/openstack/39b481c9403c4fd9b8ad3199ab5313ad/docs/ | 17:50 |
| cardoe | What’s the rendered name? | 17:50 |
| cardoe | Oh it’s not linked anywhere. | 17:51 |
| TheJulia | yeah, that ;) | 17:51 |
| cardoe | You wanna Google meet? I’ll grab my laptop. | 17:51 |
| cardoe | Don’t I have a bunch of patches pending to move specs to completed releases? | 17:52 |
| TheJulia | ... I think I merged one | 17:53 |
| cardoe | cid: when do you have time? | 17:56 |
| cid | cardoe, I am availble now (in 3minutes) | 17:57 |
| cid | Sorry for disturbing your PTO :-) | 17:57 |
| cid | Google meet works. | 17:58 |
| cid | https://meet.google.com/saf-fopy-mqx | 18:05 |
| opendevreview | cid proposed openstack/ironic-specs master: An extensible node maintenance data model https://review.opendev.org/c/openstack/ironic-specs/+/974894 | 18:26 |
| cid | Thanks for your time, cardoe. | 18:37 |
| cid | I just added the symlink. | 18:37 |
| cid | I will update the spec within the week with all the review feedback in mind. | 18:37 |
| opendevreview | Bartosz Bezak proposed openstack/tenks master: DNM: test aarch64 Tenks CI https://review.opendev.org/c/openstack/tenks/+/989768 | 18:55 |
| opendevreview | Bartosz Bezak proposed openstack/tenks master: CI: add aarch64 jobs https://review.opendev.org/c/openstack/tenks/+/989768 | 19:01 |
| opendevreview | Merged openstack/ironic master: Notes can be silently dropped from the rendered HTML https://review.opendev.org/c/openstack/ironic/+/989959 | 21:51 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!