| opendevreview | Merged openstack/ironic stable/xena: Stable only: Factor out addition of packaging lib https://review.opendev.org/c/openstack/ironic/+/860142 | 01:29 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/ironic master: Add configurable delays to the fake drivers https://review.opendev.org/c/openstack/ironic/+/861127 | 01:42 |
| stevebaker[m] | TheJulia: I think ^^ will be useful for various manual performance testing things, but my immediate plan for it is to dig into graceful conductor shutdown behaviour | 01:47 |
| opendevreview | Steve Baker proposed openstack/ironic master: Add configurable delays to the fake drivers https://review.opendev.org/c/openstack/ironic/+/861127 | 02:01 |
| arne_wiebalck | Good morning, Ironic! | 06:27 |
| opendevreview | Riccardo Pittau proposed openstack/metalsmith stable/yoga: Allow to use Ansible OpenStack Col. 1.x.x with openstacksdk >=0.99.0 https://review.opendev.org/c/openstack/metalsmith/+/861054 | 07:05 |
| opendevreview | Riccardo Pittau proposed openstack/metalsmith stable/xena: Allow to use Ansible OpenStack Col. 1.x.x with openstacksdk >=0.99.0 https://review.opendev.org/c/openstack/metalsmith/+/861055 | 07:06 |
| opendevreview | Riccardo Pittau proposed openstack/metalsmith stable/wallaby: Allow to use Ansible OpenStack Col. 1.x.x with openstacksdk >=0.99.0 https://review.opendev.org/c/openstack/metalsmith/+/861056 | 07:06 |
| rpittau | good morning ironic! o/ | 07:06 |
| rpittau | if anyone has a moment please review https://review.opendev.org/c/openstack/bifrost/+/859075 thanks! | 07:14 |
| rpittau | and hum first https://review.opendev.org/c/openstack/bifrost/+/848828 :D | 07:15 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Bump min version of testtools https://review.opendev.org/c/openstack/ironic/+/861144 | 07:32 |
| rpittau | TheJulia, JayF re ironic-inspector tests in bugfix/10.9: updating the jobs is not that easy because they're template, so you need to create the templates like we did for ironic in other bugfix branches | 07:38 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.9: CI: Various fixes https://review.opendev.org/c/openstack/ironic-inspector/+/860170 | 07:43 |
| rpittau | ^ I've hijacked JayF patch with, should work now | 07:45 |
| rpittau | wondering if we should keep all of those in master branch in ironic for ease of access and to avoid duplicating stuff | 07:45 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.9: CI: Various fixes https://review.opendev.org/c/openstack/ironic-inspector/+/860170 | 08:08 |
| opendevreview | Merged openstack/bifrost master: Explicitly fetch baremetal ports and use id on openstacksdk>=0.99.0 https://review.opendev.org/c/openstack/bifrost/+/859430 | 08:36 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/20.2: devstack: use CentOS 9 for DIB IPA builds https://review.opendev.org/c/openstack/ironic/+/861058 | 09:42 |
| rpittau | ^ bugfix/20/2 being zed requires this for compatibility with constraints | 09:44 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/9.0: Pin bugfix 9.0 ci jobs to zed https://review.opendev.org/c/openstack/ironic-python-agent/+/859255 | 09:45 |
| kubajj | Good afternoon Ironic! Does anybody has any suggestions on what OS images to try to deploy devstack on? I've tried couple Ubuntu versions (20.04 and 22.04 in desktop and server versions ), but no luck so far. | 11:02 |
| iurygregory | good morning Ironic | 11:30 |
| rpittau | kubajj: both ubuntu 20.04 and 22.04 should work, I think currently we're still running mostly 20.04 | 12:17 |
| rpittau | hey iurygregory :) | 12:17 |
| kubajj | rpittau: can you run then in something like VirtualBox or similar? | 12:18 |
| kubajj | Is 8GB of RAM for it enough? | 12:19 |
| kubajj | (I am just trying to figure out what I'm doing wrong) | 12:19 |
| dtantsur | kubajj: 8G may not suffice for bare metal testing by default. let me check our defaults.. | 12:22 |
| dtantsur | kubajj: nested VMs use 3G by default: https://opendev.org/openstack/ironic/src/branch/master/devstack/lib/ironic#L146 | 12:23 |
| dtantsur | if you intend to use tinyipa as a ramdisk, you can lower that to 1G I think | 12:23 |
| rpittau | kubajj: you can run that in a virtualized environment, probably 8GB RAM is a bit low, depends which services you run on top of that, for bare metal you need more | 12:23 |
| dtantsur | IRONIC_RAMDISK_TYPE=tinyipa | 12:23 |
| kubajj | rpittau: would the stack.sh script fail if it doesn't have enough RAM? | 12:24 |
| kubajj | dtantsur: I'll try to specify the tinyipa, that might help | 12:25 |
| rpittau | kubajj: stack.sh may fail for lack of memory, yes | 12:28 |
| TheJulia | good morning | 13:06 |
| rpittau | good morning TheJulia :) | 13:26 |
| *** knikolla[m] is now known as knikolla | 13:26 | |
| * TheJulia did not order a migraine this morning | 13:28 | |
| dtantsur | this is always a surprise gift.. | 14:01 |
| opendevreview | Merged openstack/ironic bugfix/20.2: devstack: use CentOS 9 for DIB IPA builds https://review.opendev.org/c/openstack/ironic/+/861058 | 14:21 |
| rpittau | \o/ | 14:21 |
| TheJulia | dtantsur: it feels a bit like a fly in my soup | 14:21 |
| rpittau | mmmm JayF TheJulia what do we want to do about functional-tests in -> https://review.opendev.org/c/openstack/ironic-inspector/+/860170 error seems related to sqlite | 14:24 |
| JayF | rpittau: that needs the sqla2.0 workaround then | 14:24 |
| rpittau | ah ok | 14:24 |
| TheJulia | yeah, it is pulling in oslodb 12.1.0 which defaults autocommit to False | 14:25 |
| * TheJulia suspects that should have been oslodb 13.0.0 | 14:26 | |
| TheJulia | anyway! | 14:26 |
| opendevreview | Merged openstack/ironic bugfix/20.2: Pin bugfix 20.2 ci jobs to zed https://review.opendev.org/c/openstack/ironic/+/859250 | 15:14 |
| JayF | Hey; just a reminder to folks: https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures please do not `recheck` patches without putting a reason afterwards | 15:17 |
| JayF | this is being tracked by the TC and I don't want Ironic to end up on the naughty list :) | 15:18 |
| dtantsur | File that under "things everyone should be doing, noone will and everyone knows why" :) | 15:18 |
| JayF | Honestly, I usually shrug at these but I saw like, 4 bare rechecks for Ironic stuff in my inbox from overnight | 15:19 |
| JayF | and I think our gate is getting flakier where rechecking over and over isn't going to cut it :( | 15:20 |
| dtantsur | I know, I used to be just upset as you. And then I got always tired, and stop being able to spend half an hour dragging myself through devstack logs. | 15:20 |
| JayF | I'm trying to ... utilize the exuberant motivation while it exists | 15:20 |
| JayF | I know it won't be here forever :) | 15:20 |
| dtantsur | \o/ | 15:20 |
| dtantsur | I wonder how many of the issues will disappear if we ever finish the neutron sync work | 15:21 |
| JayF | if it seems like I'm being spastic since being elected; it's literally just me trying to get every ounce of motivation out of the situation lol | 15:21 |
| dtantsur | fair :D | 15:21 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/xena: Use utf-16-le if BOM not present https://review.opendev.org/c/openstack/ironic-python-agent/+/861197 | 15:31 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent bugfix/8.1: Use utf-16-le if BOM not present https://review.opendev.org/c/openstack/ironic-python-agent/+/861062 | 15:31 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/wallaby: Use utf-16-le if BOM not present https://review.opendev.org/c/openstack/ironic-python-agent/+/861063 | 15:32 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/victoria: Use utf-16-le if BOM not present https://review.opendev.org/c/openstack/ironic-python-agent/+/861064 | 15:32 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Use utf-16-le if BOM not present https://review.opendev.org/c/openstack/ironic-python-agent/+/861065 | 15:32 |
| *** tkajinam is now known as Guest2971 | 15:43 | |
| opendevreview | Merged openstack/ironic-python-agent bugfix/9.0: Pin bugfix 9.0 ci jobs to zed https://review.opendev.org/c/openstack/ironic-python-agent/+/859255 | 15:45 |
| kubajj | Anybody has any idea what is devstack trying to tell me? (This might not be the problem, I am just trying to figure out why it failed) HttpException: 500: Server Error for url: http://10.0.2.15:9696/networking/v2.0/subnets, Failed to allocate subnet: Cannot allocate requested subnet from the available set of prefixes. | 15:53 |
| JayF | So that's an error talking to Neutron | 15:53 |
| JayF | you likely can look at neutron api logs and see why it 500 errored | 15:53 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/train: Use utf-16-le if BOM not present https://review.opendev.org/c/openstack/ironic-python-agent/+/861200 | 16:02 |
| rpittau | bye everyone! o/ | 16:05 |
| TheJulia | ugh, I thought we changed devstack to permit warnings to be okay on pre-upgrade check | 16:58 |
| TheJulia | .. maybe I did that elsewhere | 17:00 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add upgrade check warning for allocations db https://review.opendev.org/c/openstack/ironic/+/861107 | 17:07 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP Add upgrade check warning for allocations db https://review.opendev.org/c/openstack/ironic/+/861107 | 17:12 |
| TheJulia | something is afoot in what I'm getting back from the db in devstack :\ | 17:15 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP Add upgrade check warning for allocations db https://review.opendev.org/c/openstack/ironic/+/861107 | 17:30 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add upgrade check warning for allocations db https://review.opendev.org/c/openstack/ironic/+/861107 | 17:55 |
| TheJulia | Interestingly enough, on devstack, that fixed check doesn't error since it is reporting utf8mb3 | 18:01 |
| TheJulia | and innodb... | 18:01 |
| TheJulia | *rage* | 18:01 |
| JayF | underlying OS differences? | 18:15 |
| opendevreview | Merged openstack/ironic-python-agent bugfix/8.6: Pin bugfix 8.6 ci jobs to zed https://review.opendev.org/c/openstack/ironic-python-agent/+/859253 | 18:46 |
| TheJulia | JayF: distribution defaults most likely | 20:20 |
| JayF | I'm doing some office hours @ youtube.com/jayofdoom if anyone wants to jump in | 20:42 |
| JayF | working on PTG schedule | 20:53 |
| JayF | (it's late, but better late than never) | 20:53 |
| JayF | Put up a tentative PTG schedule here: https://etherpad.opendev.org/p/ironic-antelope-ptg -- please let me know if any of the times are uncomfortably late/early or overlapping for anyone and I will try to accomodate | 21:02 |
| JayF | sorry this didn't happen sooner, I legit didn't realize it was something I had to do until literally today :) | 21:02 |
| JayF | TheJulia: ^ can you sanity check? | 21:03 |
| JayF | TheJulia: if you have time, I know you're heads down | 21:03 |
| TheJulia | JayF: at a quick look, it looks good to me | 21:20 |
| JayF | ty | 21:20 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fix allocations default table type https://review.opendev.org/c/openstack/ironic/+/860198 | 21:21 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Phase 1 - SQLAlchemy 2.0 Compatability https://review.opendev.org/c/openstack/ironic/+/856336 | 21:21 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Phase 2 - SQLAlchemy 2.0 Compatability https://review.opendev.org/c/openstack/ironic/+/857516 | 21:24 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Phase 3 - SQLAlchemy 2.0 Compatability https://review.opendev.org/c/openstack/ironic/+/857932 | 21:24 |
| TheJulia | stevebaker[m]: if you can review the rest of the chain ^^ it would be appreciated. If you have any questions, I would be happy to go over/review various aspects | 21:24 |
| stevebaker[m] | TheJulia: ok, I'll take a look now | 21:25 |
| TheJulia | okay, I'm going to step away for a little bit, going to try and dig into the rest of the inspector stuff tomorrow and *hopefully* move that forward with a clear mind | 21:26 |
| JayF | Heads up: I'm going to be out-ish tomorrow. I'll have PTO but if you need me urgently, feel free to send a DM or ping me in here and I'll get around to it. | 21:50 |
| janders | good morning Ironic o/ | 21:56 |
| TheJulia | JayF: have a good weekend! | 21:58 |
| TheJulia | janders: good morning! | 21:58 |
| janders | hey TheJulia o/ | 21:59 |
| stevebaker[m] | TheJulia: looks good, just a question on https://review.opendev.org/c/openstack/ironic/+/857516 | 22:36 |
| TheJulia | stevebaker[m]: possibly…. The entire query would need to be rewritten but I don’t think there is any reason not to | 22:53 |
| stevebaker[m] | It could be a follow up | 23:02 |
| ashinclouds[m] | At, op should be query though. Realistically minor | 23:03 |
| opendevreview | Merged openstack/ironic master: Add upgrade check warning for allocations db https://review.opendev.org/c/openstack/ironic/+/861107 | 23:20 |
| TheJulia | janders: eol'ing the ipa train branch as well? | 23:53 |
| TheJulia | err | 23:53 |
| TheJulia | JayF: | 23:53 |
| TheJulia | JayF: Eol'ing the train api branch that is? | 23:54 |
| JayF | TheJulia: Not the plan anymore; if I accidentally a patch in the wrong branch it's my bad | 23:54 |
| TheJulia | JayF: ack, unabandnoing | 23:54 |
| TheJulia | I'll fix tomorrow since I need to get stuff for train sorted downstream | 23:55 |
| TheJulia | hmm... likely ipa-b needs to be pinned on that branch | 23:55 |
| TheJulia | I can look in the morning | 23:55 |
| JayF | oh! I bet that's what's breaking ironic-lib victoria, too | 23:57 |
| JayF | I was looking at that earlier | 23:57 |
| TheJulia | most likely | 23:57 |
| JayF | I'm going to go comment that on the patch lol | 23:57 |
| TheJulia | k | 23:57 |
| TheJulia | hmmm | 23:59 |
| TheJulia | nope, already pinned | 23:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!