| arne_wiebalck | Good morning, Ironic! | 06:15 |
|---|---|---|
| *** tkajinam is now known as Guest162 | 06:33 | |
| opendevreview | Vanou Ishii proposed openstack/ironic-python-agent master: Fix failure of bind mount in _install_grub2 https://review.opendev.org/c/openstack/ironic-python-agent/+/857271 | 07:59 |
| *** mgoddard- is now known as mgoddard | 08:11 | |
| opendevreview | Mark Goddard proposed openstack/tenks master: Use Yoga branch of Kayobe for gating CI job https://review.opendev.org/c/openstack/tenks/+/857273 | 08:30 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/networking-baremetal/+/857311 | 09:07 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/networking-baremetal/+/857312 | 09:07 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal master: Update master for stable/zed https://review.opendev.org/c/openstack/networking-baremetal/+/857313 | 09:07 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/networking-baremetal/+/857314 | 09:07 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/857316 | 09:09 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/857317 | 09:09 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter master: Update master for stable/zed https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/857318 | 09:09 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/857319 | 09:09 |
| opendevreview | Merged openstack/bifrost master: Consolidate ubuntu/debian required_defaults https://review.opendev.org/c/openstack/bifrost/+/855805 | 10:19 |
| opendevreview | Merged openstack/ironic master: [config-doc] Fix help for default_boot_mode https://review.opendev.org/c/openstack/ironic/+/854771 | 10:31 |
| iurygregory | good morning Ironic | 11:26 |
| opendevreview | Jacob Anders proposed openstack/sushy master: [WIP] Improve resiliency of eTag handling https://review.opendev.org/c/openstack/sushy/+/856123 | 11:36 |
| opendevreview | Merged openstack/ironic master: Adds create_csr and add_https_certificate clean step https://review.opendev.org/c/openstack/ironic/+/804486 | 11:51 |
| iurygregory | dtantsur, sorry to bother you (but since riccardo is out you are the only one that can review the release patches I've pushed), can you check the cycle highlights again? o/ https://review.opendev.org/c/openstack/releases/+/856761 | 12:10 |
| dtantsur | iurygregory: no worries, lgtm | 12:28 |
| iurygregory | tks | 12:29 |
| kamlesh6808c | iurygregory, Need help for second review for patch https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/853621 | 12:35 |
| dtantsur | janders: I wild thought: do we need etag support in sushy-tools? | 12:52 |
| dtantsur | to bring the emulator closer to the reality? | 12:52 |
| dtantsur | having realistic bugs is optional :D | 12:53 |
| * TheJulia should good morning | 13:45 | |
| TheJulia | err | 13:45 |
| TheJulia | good morning | 13:45 |
| TheJulia | ugh, sqlalchemy | 14:22 |
| ajya | iurygregory: if you happen to find that patch, would be helpful | 14:33 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Add iDRAC Redfish firmware update test https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/853621 | 14:33 |
| dtantsur | morning TheJulia | 15:27 |
| TheJulia | I think jsonencodeddict are broken in sa 1.4 | 15:41 |
| dtantsur | ..... | 15:41 |
| TheJulia | since we have to unique the query result set since sa no longer does it in 2.0 nor in the new query format | 15:50 |
| TheJulia | orm quietly did it under the hood but it is going away | 15:51 |
| JayF | when you said the orm did it, do you mean in python or in DB? | 16:04 |
| JayF | Seems like if we want the unique, the DB should do it... right? | 16:04 |
| iurygregory | brb, lunch time | 16:13 |
| TheJulia | orm did (and still does, but orm querying is deprecated and throws warnings about 2.0 breakages) | 16:25 |
| TheJulia | db cannot do it when your doing joins | 16:25 |
| * TheJulia thought she changed it to be selectinload | 16:25 | |
| TheJulia | hmmmm orm relationship lazy flag might cure this | 16:30 |
| TheJulia | hmm, less failed tests | 16:33 |
| TheJulia | hmm, that might have done it | 16:34 |
| opendevreview | Merged openstack/ironic-prometheus-exporter stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/857316 | 18:01 |
| opendevreview | Merged openstack/ironic-prometheus-exporter stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/857317 | 18:02 |
| opendevreview | Merged openstack/networking-baremetal stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/networking-baremetal/+/857311 | 18:06 |
| opendevreview | Merged openstack/networking-baremetal stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/networking-baremetal/+/857312 | 18:06 |
| TheJulia | https://www.youtube.com/watch?v=x3rWRLOZX6U | 18:42 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Phase 1 - SQLAlchemy 2.0 Compatability https://review.opendev.org/c/openstack/ironic/+/856336 | 21:58 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: second phase sqlalchemy https://review.opendev.org/c/openstack/ironic/+/857516 | 21:58 |
| TheJulia | stevebaker[m]: phase1 above :) | 21:59 |
| stevebaker[m] | sweet | 21:59 |
| TheJulia | the updates/insert/deletes are next | 21:59 |
| TheJulia | Might have to do it in blocks so like deploy templates/steps as a unique block | 22:00 |
| TheJulia | but, tbd | 22:00 |
| TheJulia | and Likely best for tomorrow since I spent 13 hours on the first patch yesterday | 22:00 |
| TheJulia | well, 13 hours yesterday | 22:00 |
| TheJulia | anyway, going to go do some cleaning | 22:07 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!