| songwenping_ | sean-k-mooney: yeah, i will. | 01:26 |
|---|---|---|
| *** ministry is now known as __ministry | 02:13 | |
| opendevreview | Billy Olsen proposed openstack/nova master: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/838976 | 02:54 |
| *** bhagyashris is now known as bhagyashris|ruck | 05:33 | |
| opendevreview | Wenping Song proposed openstack/nova-specs master: Usage of new trait of OWNER_NOVA https://review.opendev.org/c/openstack/nova-specs/+/819510 | 06:32 |
| opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Add workaround to skip compareCPU() check on the destination https://review.opendev.org/c/openstack/nova/+/838926 | 07:04 |
| gibi | bauzas: that is sort of un unblocking change https://review.opendev.org/c/openstack/nova/+/838942 | 07:21 |
| gibi | and a simple fix | 07:21 |
| gibi | sean-k-mooney: thanks! | 07:21 |
| * bauzas clicks | 07:21 | |
| bauzas | gibi: do you know which specific change in os-brick added fs locks ? | 07:24 |
| * bauzas can't find one of them in https://github.com/openstack/os-brick/commits/master | 07:24 | |
| gibi | bauzas: here https://review.opendev.org/q/ecaf7f8962e12b43f9759ddc1b608f30eb9f5ebb | 07:26 |
| bauzas | oh this was very old | 07:27 |
| bauzas | gibi: I'm about to respin https://review.opendev.org/c/openstack/nova/+/838942 because we miss the osbrick version that creates our problem | 07:28 |
| bauzas | at least in the commit msg | 07:28 |
| bauzas | AFAICS, this was delivered in 5.1.0, right? | 07:29 |
| gibi | sure, feel free to respin. I'm not sure about the version | 07:30 |
| bauzas | comes from https://review.opendev.org/c/openstack/os-brick/+/814139 | 07:31 |
| bauzas | which was released by 5.1.0 | 07:31 |
| gibi | ahh I see now | 07:31 |
| gibi | yes | 07:31 |
| gibi | it is 5.1 then | 07:31 |
| opendevreview | Jorhson Deng proposed openstack/nova master: Reset the instance's task_state to None when the status is error https://review.opendev.org/c/openstack/nova/+/838991 | 07:32 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: enable locking test fixture https://review.opendev.org/c/openstack/nova/+/838942 | 07:36 |
| bauzas | gibi : ^ | 07:36 |
| gibi | thanks, +A | 07:41 |
| bauzas | gibi: I'm still confused by the urgency | 07:50 |
| bauzas | gibi: https://github.com/openstack/nova/blame/master/requirements.txt#L50 | 07:50 |
| bauzas | gibi: we use os-brick 5.2 for 4 months now | 07:50 |
| bauzas | gibi: and that's only then we upgraded from 4.3.1 https://github.com/openstack/nova/commit/b5e2128f3847d444a808a2b0f89e6f1e4ffb77fc | 07:52 |
| gibi | bauzas: there is a common confusion why it is not happening all the time for us upstream | 08:06 |
| gibi | but there was reports back from january that it happens locally | 08:06 |
| gibi | and I think now it happens downstream 100% | 08:06 |
| gibi | hence the urgency :) | 08:06 |
| gibi | honestly when it was first reported I was not able to reproduce it locally on my previous laptop | 08:07 |
| gibi | but with my new laptop I saw it once | 08:07 |
| gibi | so I guess it depends on some version caches | 08:07 |
| bauzas | gibi: ack thanks | 08:13 |
| kashyap | gibi: Getting out of the venv and recreating the 'tox' env resolved it. (I thought I tried this last night) | 08:14 |
| gibi | kashyap: great, then mistery resolved :) | 08:14 |
| kashyap | Yep | 08:15 |
| opendevreview | Merged openstack/nova master: Update python testing as per zed cycle teting runtime https://review.opendev.org/c/openstack/nova/+/838943 | 08:57 |
| opendevreview | Merged openstack/placement master: Make perfload jobs fail if write allocation fails https://review.opendev.org/c/openstack/placement/+/828438 | 08:58 |
| opendevreview | Merged openstack/nova master: enable locking test fixture https://review.opendev.org/c/openstack/nova/+/838942 | 08:58 |
| stephenfin | bauzas: Can you look at https://review.opendev.org/c/openstack/nova/+/837161/ at some point. Last patch left in that series that still needs a +W | 09:21 |
| opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Add workaround to skip compareCPU() check on the destination https://review.opendev.org/c/openstack/nova/+/838926 | 09:22 |
| bauzas | stephenfin: on a chat with rajesh sorry | 09:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Add Python 3.10 functional jobs https://review.opendev.org/c/openstack/nova/+/839029 | 09:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: db: Close connection on early return https://review.opendev.org/c/openstack/nova/+/839030 | 09:25 |
| stephenfin | gibi: ^ per your feedback | 09:26 |
| stephenfin | bauzas: all good, absolutely any time today would be great | 09:26 |
| gibi | stephenfin: thanks. looking | 09:50 |
| opendevreview | Jorhson Deng proposed openstack/nova master: Reset the instance's task_state to None when the status is error https://review.opendev.org/c/openstack/nova/+/838991 | 09:57 |
| opendevreview | Elod Illes proposed openstack/nova stable/ussuri: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838033 | 10:49 |
| sean-k-mooney | gibi: bauzas im +2 on https://review.opendev.org/c/openstack/nova-specs/+/819510/7 by the way | 13:04 |
| bauzas | sean-k-mooney: ack, I'll try to look at it | 13:08 |
| *** dasm|off is now known as dasm | 13:16 | |
| opendevreview | Elod Illes proposed openstack/nova stable/train: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838037 | 13:18 |
| sean-k-mooney | bauzas: refresh my memory am i allowd to singel core approve a reporpsoed previously apporved spec | 13:26 |
| bauzas | sean-k-mooney: for fast approvals, it depends whether it added or changed some stuff | 13:26 |
| sean-k-mooney | https://specs.openstack.org/openstack/nova-specs/readme.html#previously-approved-specifications does not say explcitly othen it follwo the normal process which woudl imply no | 13:26 |
| sean-k-mooney | bauzas: im just reviewing melwitt reposoed version fo the encyuption specs | 13:26 |
| sean-k-mooney | i think tehy are idential which is wht im checking now | 13:27 |
| sean-k-mooney | but wanted to knwo if i should +2w them or just +2 once i complete reviewing it | 13:27 |
| sean-k-mooney | bauzas: never mind the primary assine need to be updated | 13:31 |
| opendevreview | Merged openstack/nova-specs master: Make 'tox -e fast-specs' fail on sphinx warnings https://review.opendev.org/c/openstack/nova-specs/+/835930 | 13:33 |
| *** Guest2246 is now known as prometheanfire | 13:54 | |
| gibi | sean-k-mooney: I've started reading the fresh OWNER_NOVA spec but got distracted with bug triage, I will get back to it before end of today | 13:55 |
| opendevreview | Dan Smith proposed openstack/nova master: DNM: Run against performance.json patch https://review.opendev.org/c/openstack/nova/+/838934 | 14:00 |
| sean-k-mooney | gibi: no rush. i would have prefered the upgade impact workign in my comment as i incled that plamcent must be upgraded first but honestly i think we can jsut adress that in the code review of the release note adn docs | 14:01 |
| opendevreview | Merged openstack/nova-specs master: Usage of new trait of OWNER_NOVA https://review.opendev.org/c/openstack/nova-specs/+/819510 | 14:09 |
| bauzas | gibi: I gave my approval | 14:15 |
| gibi | bauzas: thanks, one less thing for me to do before the weekend :) | 14:18 |
| * bauzas has to quit for the weekend with this sound in mind https://www.youtube.com/watch?v=N3kkNfH4yco | 14:56 | |
| gibi | bauzas: have nice one | 15:10 |
| bauzas | thanks | 15:10 |
| * bauzas is not using the copier | 15:10 | |
| gibi | I don't need the details, please :D | 15:11 |
| *** akekane_ is now known as abhishekk | 15:15 | |
| opendevreview | Elod Illes proposed openstack/nova stable/stein: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838038 | 16:08 |
| *** amoralej is now known as amoralej|off | 16:14 | |
| opendevreview | Merged openstack/nova master: VMware: Early fail spawn if memory is not multiple of 4. https://review.opendev.org/c/openstack/nova/+/835739 | 16:29 |
| opendevreview | Dan Smith proposed openstack/nova master: DNM: Run against performance.json patch https://review.opendev.org/c/openstack/nova/+/838934 | 16:32 |
| opendevreview | Elod Illes proposed openstack/nova stable/rocky: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838041 | 16:36 |
| opendevreview | Dan Smith proposed openstack/nova master: DNM: Run against performance.json patch https://review.opendev.org/c/openstack/nova/+/838934 | 17:02 |
| opendevreview | Merged openstack/nova master: db: Remove inplicit coercion of SELECTs https://review.opendev.org/c/openstack/nova/+/837159 | 17:23 |
| opendevreview | Merged openstack/nova master: db: Replace use of Column.copy() method https://review.opendev.org/c/openstack/nova/+/837160 | 17:29 |
| opendevreview | Merged openstack/nova master: db: Don't rely on autocommit behavior https://review.opendev.org/c/openstack/nova/+/837161 | 17:29 |
| opendevreview | Artom Lifshitz proposed openstack/nova master: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/839091 | 18:28 |
| sean-k-mooney | artom_: does ^ actully repoduce it | 18:32 |
| opendevreview | Dan Smith proposed openstack/nova master: DNM: Run against performance.json patch https://review.opendev.org/c/openstack/nova/+/838934 | 18:41 |
| artom_ | sean-k-mooney, yeah, and because I frogot the FIXME assertRaises, it quite literally blows up in the same way as "real life" | 20:25 |
| opendevreview | Jakub Libosvar proposed openstack/os-vif master: Check for hybrid plugging in OVS https://review.opendev.org/c/openstack/os-vif/+/839102 | 21:09 |
| opendevreview | Artom Lifshitz proposed openstack/nova master: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/839091 | 21:44 |
| *** dasm is now known as dasm|off | 22:03 | |
| opendevreview | Jakub Libosvar proposed openstack/os-vif master: Check for hybrid plugging in OVS https://review.opendev.org/c/openstack/os-vif/+/839102 | 22:20 |
| opendevreview | melanie witt proposed openstack/nova master: scheduler: Add an ephemeral encryption pre filter https://review.opendev.org/c/openstack/nova/+/760456 | 22:24 |
| opendevreview | melanie witt proposed openstack/nova master: block_device: Add DriverImageBlockDevice to block_device_info https://review.opendev.org/c/openstack/nova/+/826527 | 22:24 |
| opendevreview | melanie witt proposed openstack/nova master: block_device: Add encryption attributes to image and ephemeral disks https://review.opendev.org/c/openstack/nova/+/826528 | 22:24 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!