| opendevreview | melanie witt proposed openstack/nova master: Prevent deletion of a compute node belonging to another host https://review.opendev.org/c/openstack/nova/+/694802 | 00:49 |
|---|---|---|
| opendevreview | melanie witt proposed openstack/nova master: Fix inactive session error in compute node creation https://review.opendev.org/c/openstack/nova/+/695189 | 00:49 |
| *** redrobot4 is now known as redrobot | 04:52 | |
| opendevreview | ruishawn proposed openstack/nova master: Add armv6l to arch list https://review.opendev.org/c/openstack/nova/+/799964 | 06:56 |
| *** rpittau|afk is now known as rpittau | 06:59 | |
| opendevreview | ruishawn proposed openstack/nova master: Add armv6l to arch list https://review.opendev.org/c/openstack/nova/+/799964 | 07:00 |
| opendevreview | Lee Yarwood proposed openstack/nova master: DNM - Test tempest-integrated-compute-centos-8-stream https://review.opendev.org/c/openstack/nova/+/799996 | 09:21 |
| opendevreview | Ye Haiyang proposed openstack/nova master: Replace `split(' ')` with `split()` https://review.opendev.org/c/openstack/nova/+/800004 | 10:10 |
| bauzas | oh man, would we get some mats if we were having a non-virtual PTG ? :D | 10:18 |
| bauzas | context : look at the Y name | 10:19 |
| lyarwood | If we end up having an in-person PTG and they replace beer with yoga I'm going to be very very upset | 10:21 |
| stephenfin | but very very flexible? | 10:22 |
| stephenfin | jaysus, working on this alembic stuff is like pulling teeth... | 10:24 |
| sean-k-mooney | stephenfin: aren't you glad you elected to do it :) | 10:40 |
| stephenfin | ugh | 10:40 |
| sean-k-mooney | stephenfin: thank you in anycases | 10:40 |
| sean-k-mooney | are you still having issue with the last patch | 10:41 |
| sean-k-mooney | i saw you ask dansmith about it a few days ago | 10:41 |
| stephenfin | nope, zzzeek++ helped me out massively. Just fixing up the issues he pointed out and addressing bugs that I hadn't seen yet because I'd only been running the alembic stuff against SQLite without foreign key constraints enabled /o\ | 10:42 |
| bauzas | lyarwood: I'm pretty sure you can do both | 10:43 |
| bauzas | lyarwood: ie. taking beer on a yoga mat | 10:43 |
| bauzas | I could even take a picture if wanted | 10:43 |
| bauzas | but I'm not sure I could be helpful in giving some yoga practicing, I'm not a streched person :) | 10:43 |
| gibi | I vote for the beer | 10:44 |
| bauzas | people creativity always amazes me | 10:44 |
| bauzas | https://www.youtube.com/watch?v=dgPgN7CB06Y | 10:44 |
| gibi | bauzas: you have the solution | 10:45 |
| bauzas | man, I need to find a class teaching this https://www.youtube.com/watch?v=KoDvXcTgTl4 | 10:46 |
| stephenfin | \o/ it works | 10:46 |
| * stephenfin finishes fixing up unit tests | 10:47 | |
| gibi | "without alcohol I might feel a bit shy. Alcohol makes me brave and enjoy joga." that is a good summary how to convince the fundation to provide free beer | 10:48 |
| gibi | stephenfin: you will be our sqlA and sqlite expert. :) | 10:49 |
| lyarwood | stephenfin: ship it | 10:50 |
| opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: Add regression test for bug #1909120 https://review.opendev.org/c/openstack/nova/+/773320 | 10:56 |
| opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: api: Reject requests to detach a volume when the compute is down https://review.opendev.org/c/openstack/nova/+/773321 | 10:56 |
| *** slaweq_ is now known as slaweq | 11:42 | |
| *** gc is now known as grandchild | 11:52 | |
| mnaser | does anyone of the top of their mind know what rescue behaviour will be if the vm is bfv? | 12:15 |
| gibi | mnaser: I think normally we don't allow resuce of bfv, but we have stable device rescue that allows it https://docs.openstack.org/nova/latest/user/rescue.html#stable-device-instance-rescue | 12:19 |
| sean-k-mooney | i think it was added recently | 12:21 |
| sean-k-mooney | mnaser: we used to not supprot it at all at one point but i think it was added around train | 12:21 |
| sean-k-mooney | https://specs.openstack.org/openstack/nova-specs/specs/ussuri/implemented/virt-bfv-instance-rescue.html | 12:22 |
| sean-k-mooney | so ussuri | 12:22 |
| sean-k-mooney | mnaser: so no i think it shoudl work like a normal guest | 12:22 |
| sean-k-mooney | it does depend on the stable device rescue too | 12:24 |
| mnaser | oh, stable device rescue is very neat | 12:28 |
| opendevreview | Merged openstack/nova stable/victoria: Honor [neutron]http_retries in the manual client https://review.opendev.org/c/openstack/nova/+/795175 | 12:36 |
| *** rpittau is now known as rpittau|afk | 12:45 | |
| sean-k-mooney | stephenfin: how do we deprecate commands in osc | 12:52 |
| sean-k-mooney | im asking because we shoudl proably start issueing warning whenever someone uses "openstack host list" or similar | 12:52 |
| stephenfin | I'm not sure. I guess with an update to the help string. Using Python's 'warning' lib might be a bit too noisy | 12:53 |
| sean-k-mooney | can we have it output a warnign to std:error and remove it in the future | 12:53 |
| sean-k-mooney | i would like it to shot at people when they use it so they know not too idealy | 12:54 |
| stephenfin | you need to support old clouds though | 12:54 |
| stephenfin | so we don't remove things from OSC, typically | 12:54 |
| sean-k-mooney | ya we can issue the warning based on micorverison | 12:55 |
| gibi | jinx, I wanted to say the same | 12:55 |
| sean-k-mooney | https://docs.openstack.org/api-ref/compute/?expanded=create-image-createimage-action-detail#hosts-os-hosts-deprecated | 12:55 |
| sean-k-mooney | it was depercated in 2.43 and removed in later microverions | 12:55 |
| sean-k-mooney | most of this api was only used by zen i think too | 12:56 |
| lyarwood | mnaser: sorry was on a call earlier, did stable device rescue work for your bfv instance? | 13:36 |
| opendevreview | Merged openstack/nova stable/train: Error anti-affinity violation on migrations https://review.opendev.org/c/openstack/nova/+/798717 | 13:55 |
| mnaser | lyarwood: haven’t tried it yet! | 14:08 |
| lyarwood | kk let me know if you run into any issues :) | 14:09 |
| *** frickler is now known as frickler_pto | 15:03 | |
| opendevreview | Stephen Finucane proposed openstack/nova master: db: Add initial alembic migration for main DB https://review.opendev.org/c/openstack/nova/+/799527 | 16:15 |
| opendevreview | Stephen Finucane proposed openstack/nova master: db: Add initial alembic migration for API DB https://review.opendev.org/c/openstack/nova/+/799528 | 16:15 |
| opendevreview | Stephen Finucane proposed openstack/nova master: db: Trivial style changes https://review.opendev.org/c/openstack/nova/+/799529 | 16:15 |
| opendevreview | Stephen Finucane proposed openstack/nova master: db: Normalize migrations tests https://review.opendev.org/c/openstack/nova/+/799684 | 16:15 |
| opendevreview | Stephen Finucane proposed openstack/nova master: db: Integrate alembic https://review.opendev.org/c/openstack/nova/+/799530 | 16:15 |
| opendevreview | Stephen Finucane proposed openstack/nova master: db: Avoid use of ALTER in initial migration https://review.opendev.org/c/openstack/nova/+/800076 | 16:15 |
| opendevreview | Stephen Finucane proposed openstack/nova master: db: Enable auto-generation of migrations https://review.opendev.org/c/openstack/nova/+/800077 | 16:15 |
| opendevreview | Stephen Finucane proposed openstack/nova master: docs: Add documentation on database migrations https://review.opendev.org/c/openstack/nova/+/800078 | 16:15 |
| opendevreview | Stephen Finucane proposed openstack/nova master: WIP db: Add our first alembic migration https://review.opendev.org/c/openstack/nova/+/800079 | 16:15 |
| stephenfin | gibi: lyarwood: bauzas: I think that DB series is basically good to go now. I need to wrap up docs and work on that initial migration to identify the test coverage gap that let it through, but it's otherwise in good shape IMO ^ | 16:19 |
| bauzas | stephenfin: I won't be able to review it until end of next week | 16:19 |
| lyarwood | stephenfin: I'll queue it up for tomorrow | 16:20 |
| bauzas | gibi: stephenfin: fwiw, /me goes on PTO tonight till Wed July 14th (incl.) | 16:20 |
| stephenfin | bauzas: ack, no rush | 16:20 |
| stephenfin | well, there is a little rush but nothing more important than PTO :P | 16:20 |
| * stephenfin goes to throw a ball about in the rain o/ | 16:20 | |
| gibi | stephenfin: ack, I will added to my queue. I try to wrap up the pps code series in the coming day so I think we can traid reviews there :) | 16:21 |
| stephenfin | gibi: yup, I've been keeping an eye on it and am simply waiting for the TODOs in the base patches to disappear before I take a look :) | 16:21 |
| * bauzas disappears for the next 6 days then | 16:22 | |
| gibi | stephenfin: good decision. I'm constantly reorganizing the series to make it easyier to review | 16:22 |
| gibi | bauzas: have a nice PTO | 16:22 |
| gibi | stephenfin: I will let you know when it make sense to start reviewing | 16:22 |
| stephenfin | ack | 16:22 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | 17:06 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | 17:06 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Parse extended resource request from the port https://review.opendev.org/c/openstack/nova/+/800085 | 17:06 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | 17:09 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | 17:09 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | 17:09 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test]Refactor interface attach with qos https://review.opendev.org/c/openstack/nova/+/800088 | 17:17 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support interaface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | 17:19 |
| opendevreview | melanie witt proposed openstack/nova stable/train: [CI] Fix gate by using zuulv3 live migration and grenade jobs https://review.opendev.org/c/openstack/nova/+/795435 | 19:22 |
| opendevreview | melanie witt proposed openstack/nova master: Add func test for nova-manage db archive_deleted_rows --before https://review.opendev.org/c/openstack/nova/+/796744 | 19:34 |
| opendevreview | melanie witt proposed openstack/nova stable/ussuri: Initialize global data separately and run_once in WSGI app init https://review.opendev.org/c/openstack/nova/+/785061 | 19:44 |
| opendevreview | melanie witt proposed openstack/nova stable/train: Initialize global data separately and run_once in WSGI app init https://review.opendev.org/c/openstack/nova/+/785064 | 19:59 |
| *** mgoddard- is now known as mgoddard | 20:50 | |
| opendevreview | melanie witt proposed openstack/nova stable/train: Initialize global data separately and run_once in WSGI app init https://review.opendev.org/c/openstack/nova/+/785064 | 21:01 |
| opendevreview | Rodrigo Barbieri proposed openstack/nova stable/stein: Error anti-affinity violation on migrations https://review.opendev.org/c/openstack/nova/+/800114 | 21:09 |
| opendevreview | Merged openstack/nova stable/wallaby: Move 'check-cherry-picks' test to gate, n-v check https://review.opendev.org/c/openstack/nova/+/797039 | 21:23 |
| *** rm_work is now known as rm_work| | 23:59 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!