| TheJulia | tl;dr it is a session is never exiting from sqlalchemy... I thought it was the read, I thought it was a db deadlock, just... not sure. reaching that, I know where to look next, just it was painful to get there :( | 00:04 |
|---|---|---|
| TheJulia | anyway, pizza() | 00:04 |
| opendevreview | Steve Baker proposed openstack/metalsmith master: Limit openstacksdk install to <0.99 https://review.opendev.org/c/openstack/metalsmith/+/860885 | 00:10 |
| opendevreview | Steve Baker proposed openstack/metalsmith master: WIP Revert "CI: Force ansible to version 5" https://review.opendev.org/c/openstack/metalsmith/+/860886 | 00:10 |
| vanou | good morning ironic | 01:55 |
| opendevreview | SONG SHUKUN proposed openstack/ironic stable/yoga: Add SNMPv3 authentication functionality https://review.opendev.org/c/openstack/ironic/+/850553 | 03:01 |
| rpittau | good morning ironic! o/ | 06:50 |
| opendevreview | Riccardo Pittau proposed openstack/metalsmith stable/zed: CI: Force ansible to version 5 https://review.opendev.org/c/openstack/metalsmith/+/860903 | 07:41 |
| rpittau | considering that ansible 6 is stuck with openstack.cloud collection 1.9.1 (that requires openstacksdk <0.99) we need a new release of ansible-collections-openstack first, then the change in ansible 6 | 07:45 |
| rpittau | otherwise a workaround to limit it as it's not limited in global-requirements, or add the upper-limit | 07:45 |
| rpittau | wondering if someone else is having the same issue now that zull defaults to ansible 6 | 07:45 |
| iurygregory | good morning Ironic | 11:50 |
| opendevreview | Jakob Meng proposed openstack/metalsmith master: Allow to use Ansible OpenStack Collection 1.x.x with openstacksdk >=0.99.0 https://review.opendev.org/c/openstack/metalsmith/+/860943 | 12:40 |
| opendevreview | Jakob Meng proposed openstack/metalsmith master: Allow to use Ansible OpenStack Col. 1.x.x with openstacksdk >=0.99.0 https://review.opendev.org/c/openstack/metalsmith/+/860943 | 12:41 |
| opendevreview | Jakob Meng proposed openstack/metalsmith master: Allow to use Ansible OpenStack Col. 1.x.x with openstacksdk >=0.99.0 https://review.opendev.org/c/openstack/metalsmith/+/860943 | 12:42 |
| opendevreview | Riccardo Pittau proposed openstack/metalsmith master: WIP Revert "CI: Force ansible to version 5" https://review.opendev.org/c/openstack/metalsmith/+/860886 | 12:43 |
| opendevreview | Jakob Meng proposed openstack/metalsmith master: WIP Revert "CI: Force ansible to version 5" https://review.opendev.org/c/openstack/metalsmith/+/860886 | 12:46 |
| arozman | Hi Ironic! | 12:56 |
| TheJulia | good morning | 13:13 |
| opendevreview | Riccardo Pittau proposed openstack/metalsmith master: Revert "CI: Force ansible to version 5" https://review.opendev.org/c/openstack/metalsmith/+/860886 | 14:33 |
| rpittau | TheJulia, JayF, stevebaker[m], iurygregory: workaorund for ansible 6 for metalsmith -> https://review.opendev.org/c/openstack/metalsmith/+/860943 | 14:34 |
| rpittau | we should backport it to stable/zed once it's merged | 14:34 |
| rpittau | also updated the revert patch | 14:34 |
| * TheJulia is convinced sqllite was a mistake | 14:35 | |
| rpittau | lol | 14:35 |
| JayF | TheJulia: pretending it's the same as mysql is the mistake (our tests make) | 14:35 |
| JayF | rpittau: this works on ansible 5 and 6, right? I think gate is validating that but wanted to check explicitly | 14:35 |
| rpittau | JayF: yes works on both versions | 14:35 |
| arozman | iurygregory, rpittau: I have fixed your and dtantsur's suggestions and the change have passed the CI, please check it when you have time, https://review.opendev.org/c/openstack/ironic-python-agent/+/855866 | 14:35 |
| TheJulia | the update_node action in inspector's unit tests stupidly easily | 14:36 |
| TheJulia | I think in part how the state machine works too | 14:36 |
| JayF | rpittau: going to let others review that too; I +2'd but it'd be better if someone more ansible'd looked too | 14:37 |
| kubajj | TheJulia: could we do the walkthrough at some point? | 15:04 |
| TheJulia | kubajj: when would be good? | 15:25 |
| rpittau | bye everyone! o/ | 15:26 |
| kubajj | TheJulia: I can rearrange most stuff, so I am quite flexible this week | 15:28 |
| TheJulia | kubajj: Friday would likely be best | 15:39 |
| TheJulia | I only have a single meeting on Friday, which helps :) | 15:39 |
| TheJulia | any thoughts on just nuking inspector's "functional" tests from high orbit | 15:41 |
| TheJulia | like... obliterating it?! | 15:41 |
| kubajj | TheJulia: I could do any time before 2 pm or after 5 pm UTC on Friday | 15:42 |
| TheJulia | after 5pm utc should work well | 15:43 |
| * TheJulia hopes functional tests exit soon.... | 15:45 | |
| TheJulia | now to figure out why automaton's process event just hangs | 15:47 |
| kubajj | TheJulia: ok, I will note 5 pm down. How do we do it? | 16:48 |
| TheJulia | kubajj: does google meet work for you? | 16:50 |
| kubajj | TheJulia: sure | 16:51 |
| TheJulia | kubajj: do you want me to send you a calendar invite? | 16:51 |
| kubajj | TheJulia: I was just going to suggest that and create it. | 16:53 |
| TheJulia | then why don't you send me an invite then :) | 16:53 |
| kubajj | TheJulia: to the email you use on gerrit? | 16:54 |
| opendevreview | Merged openstack/metalsmith master: Allow to use Ansible OpenStack Col. 1.x.x with openstacksdk >=0.99.0 https://review.opendev.org/c/openstack/metalsmith/+/860943 | 17:04 |
| opendevreview | Julia Kreger proposed openstack/ironic-inspector master: WIP SQLAlchemy 2.0 prep https://review.opendev.org/c/openstack/ironic-inspector/+/860731 | 17:25 |
| opendevreview | Harald Jensås proposed openstack/ironic stable/wallaby: [Wallaby only] - Fix bashate E002/E003 https://review.opendev.org/c/openstack/ironic/+/860971 | 18:16 |
| opendevreview | Merged openstack/metalsmith master: Revert "CI: Force ansible to version 5" https://review.opendev.org/c/openstack/metalsmith/+/860886 | 18:30 |
| opendevreview | Ruby Loo proposed openstack/ironic stable/wallaby: bashate 2.1.1 reveals errors https://review.opendev.org/c/openstack/ironic/+/860990 | 19:09 |
| iurygregory | JayF, I've fwd an email about the PTG to you o/ | 19:37 |
| rloo | hi JayF, wrt PTG next week. Monday at 14 UTC, I see nova-placement scheduled in bexar. Is that the time slot where ironic will discuss the ironic driver with nova? | 19:37 |
| iurygregory | rloo, correct | 19:37 |
| JayF | Yes | 19:37 |
| rloo | thx for confirming! | 19:37 |
| JayF | I'm excited that you might be there! | 19:37 |
| rloo | (Well, i don't know yet, i have meetings so trying to do the utc xlation etc. but will be there if i am avail!) | 19:38 |
| JayF | iurygregory: thanks for the forward (and thank you, explicitly, for forwarding it to both my personal and gross email) | 19:39 |
| iurygregory | yeah, just to make sure that you would receive :D | 19:39 |
| JayF | Yeah; I don't have a good split yet and having it both places is legitimately helpful | 19:40 |
| JayF | Also my personal email is on office365; it doesn't do mailing lists well so I'm on os-discuss from work email | 19:40 |
| opendevreview | Merged openstack/ironic stable/wallaby: [Wallaby only] - Fix bashate E002/E003 https://review.opendev.org/c/openstack/ironic/+/860971 | 21:02 |
| opendevreview | Jay Faulkner proposed openstack/ironic-inspector bugfix/10.7: CI: Various required fixes https://review.opendev.org/c/openstack/ironic-inspector/+/860171 | 22:24 |
| JayF | https://review.opendev.org/c/openstack/ironic-inspector/+/860164 anyone have any ideas what's going on with the grenade job there? | 22:31 |
| JayF | the actual bug is virtualbmc not installing due to python 3.6.9 | 22:32 |
| TheJulia | pin the version | 22:33 |
| JayF | So in ironic, we just don't have a grenade job | 22:33 |
| TheJulia | its trying to install virtualbmc master | 22:33 |
| JayF | why would we have one in inspector? | 22:33 |
| JayF | (in ironic stable/victoria) | 22:33 |
| TheJulia | oh! | 22:33 |
| JayF | I suspect dropping the job is probably the better fix? | 22:33 |
| TheJulia | remove it! | 22:34 |
| * JayF kicks it | 22:34 | |
| TheJulia | if it is that old, it is out of the window we care about | 22:34 |
| TheJulia | that is why ironic no longer has one | 22:34 |
| TheJulia | on stable/victoria | 22:34 |
| opendevreview | Jay Faulkner proposed openstack/ironic-inspector stable/victoria: CI: Various fixes https://review.opendev.org/c/openstack/ironic-inspector/+/860164 | 22:35 |
| opendevreview | Jay Faulkner proposed openstack/ironic-inspector bugfix/10.9: CI: Various fixes https://review.opendev.org/c/openstack/ironic-inspector/+/860170 | 22:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!