| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-10-01 | |||
| 00:00:53 | melwitt | hehe | |
| 05:49:43 | opendevreview | melanie witt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/c/openstack/nova/+/712139 | |
| 05:49:43 | opendevreview | melanie witt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | |
| 05:49:44 | opendevreview | melanie witt proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/c/openstack/nova/+/712143 | |
| 05:49:44 | opendevreview | melanie witt proposed openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | |
| 05:49:45 | opendevreview | melanie witt proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749 | |
| 05:49:45 | opendevreview | melanie witt proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | |
| 05:49:46 | opendevreview | melanie witt proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| 05:49:46 | opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to placement unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 05:49:47 | opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |
| 05:49:47 | opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 06:53:45 | opendevreview | melanie witt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | |
| 06:53:46 | opendevreview | melanie witt proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/c/openstack/nova/+/712143 | |
| 06:53:46 | opendevreview | melanie witt proposed openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | |
| 06:53:47 | opendevreview | melanie witt proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749 | |
| 06:53:47 | opendevreview | melanie witt proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | |
| 06:53:48 | opendevreview | melanie witt proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| 06:53:48 | opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to placement unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 06:53:49 | opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |
| 06:53:49 | opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 06:56:32 | opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to placement unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 06:56:32 | opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |
| 06:56:33 | opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 07:21:12 | bauzas | good morning Nova | |
| 07:30:14 | gibi | bauzas: good Friday | |
| 07:30:20 | bauzas | indeed | |
| 07:48:52 | bauzas | gibi: looks like the nvme osbrick issue is now fixed | |
| 07:48:58 | gibi | bauzas: os-brick 5.0.1 with the nvme fix and global requirements has been bumped https://review.opendev.org/c/openstack/releases/+/812048/1/deliverables/xena/os-brick.yaml | |
| 07:49:01 | gibi | heh | |
| 07:49:05 | gibi | we looked at it at the same time :D | |
| 07:49:06 | bauzas | the xena backport is now merged https://review.opendev.org/c/openstack/os-brick/+/812010 | |
| 07:49:18 | bauzas | yeah | |
| 07:49:28 | bauzas | I call that "friends" :p | |
| 07:49:35 | gibi | :) | |
| 07:50:10 | bauzas | ok, so should we do something with our requirements ? | |
| 07:50:30 | bauzas | elodilles: thoughts on touching our reqs so close to GA ? | |
| 07:50:42 | gibi | upper constraints are coming from global | |
| 07:51:24 | gibi | an as far as I remember reqs repo hasn't been branched to xena yet | |
| 07:52:14 | gibi | so the merged global req bump is applied to Xena automatically | |
| 07:52:31 | gibi | the only question is, do we want to blacklist os-brick 5.0.0 | |
| 07:52:49 | bauzas | ah good point | |
| 07:52:54 | gibi | and I think we agreed with lyarwood that this is a narrow scoped issue so we don't need to blacklist | |
| 07:52:55 | bauzas | about the merge | |
| 07:53:04 | bauzas | gibi: yeah I think so | |
| 07:53:10 | gibi | then I think we are done | |
| 07:53:11 | bauzas | so let it be | |
| 07:53:17 | bauzas | yeah | |
| 07:53:26 | gibi | it was a nice bug as we needed to do nothing to fix it :D | |
| 07:53:34 | gibi | <3 cinder | |
| 07:53:52 | bauzas | ++ | |
| 08:59:26 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Add PCI VPD Capability Handling https://review.opendev.org/c/openstack/nova/+/808199 | |
| 08:59:27 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Support remote-managed SmartNIC DPU ports https://review.opendev.org/c/openstack/nova/+/812111 | |
| 09:34:57 | elodilles | bauzas: sorry, missed your question. but yes, as gibi said, it comes from openstack/requirements and RFE was granted for os-brick ( http://lists.openstack.org/pipermail/openstack-discuss/2021-September/025155.html ) | |
| 09:41:39 | lyarwood | bauzas: would you mind hitting https://review.opendev.org/c/openstack/nova/+/811713 and https://review.opendev.org/c/openstack/nova/+/811716 today? | |
| 09:47:26 | opendevreview | Elod Illes proposed openstack/nova stable/ussuri: [stable-only] Pin virtualenv and setuptools https://review.opendev.org/c/openstack/nova/+/810461 | |
| 09:57:16 | kashyap | dmitriis: Hi, from #virt here :) Thanks for reworking the patch and splitting it out. | |
| 09:58:20 | kashyap | Reviewer time is a bit limited these days, so please don't hesitate to re-ping here if you don't get any input in a bit | |
| 10:02:19 | dmitriis | kashyap: ack, will do. Appreciate that it's Xena release time so I'll circle back some time after it's out. | |
| 10:10:05 | opendevreview | Merged openstack/os-traits master: Change minversion of tox to 3.18.0 https://review.opendev.org/c/openstack/os-traits/+/791973 | |
| 10:24:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: Enable min pps tempest testing in nova-next https://review.opendev.org/c/openstack/nova/+/811748 | |
| 10:52:00 | lyarwood | tgif++ side_effect != side_effects | |
| 10:53:25 | gibi | +1 on Friday | |
| 11:32:30 | opendevreview | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1937084 https://review.opendev.org/c/openstack/nova/+/812126 | |
| 11:32:30 | opendevreview | Lee Yarwood proposed openstack/nova master: block_device: Ignore VolumeAttachmentNotFound during detach https://review.opendev.org/c/openstack/nova/+/812127 | |
| 12:04:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Enable min pps tempest testing in nova-next https://review.opendev.org/c/openstack/nova/+/811748 | |
| 13:09:03 | opendevreview | Takashi Natsume proposed openstack/nova master: Update min supported service version for Yoga https://review.opendev.org/c/openstack/nova/+/809932 | |
| 13:53:38 | gmann | melwitt: nova-grenade-multinode passing on stable/train with the devstack fix but devsdtack fix hitting the novnc failure in stable/stein https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_c3f/812092/1/check/tempest-full-py3/c3fde5e/testr_results.html | |
| 13:53:42 | gmann | are those known one? | |
| 14:17:41 | gmann | or gibi lyarwood elodilles ^^ you know about novnc issue ? | |
| 14:18:10 | gibi | does not ring a bell but let me grep a bit | |
| 14:19:01 | elodilles | haven't seen that before | |
| 14:19:58 | gmann | it seems circular deps, nova stable/stein hitting jsonschema issue which devstack fix but there we are hitting novnc | |
| 14:21:41 | opendevreview | Ghanshyam proposed openstack/nova stable/stein: DNM: testing jsonschema version fix https://review.opendev.org/c/openstack/nova/+/812142 | |
| 14:23:15 | gibi | gmann: I found on other hits recently about this | |
| 14:25:36 | gibi | gmann: I see that novncproxy says it is invalid / expired token https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_c3f/812092/1/check/tempest-full-py3/c3fde5e/controller/logs/screen-n-novnc-cell1.txt | |
| 14:26:27 | gibi | but I don't know why | |
| 14:26:54 | gibi | the token should not be expired based on the timestamps in the nova-api and novnc proxy logs | |
| 14:29:39 | gmann | k | |
| 14:31:48 | gibi | I don't see anything else | |
| 14:34:38 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Enable auto-generation of API DB migrations https://review.opendev.org/c/openstack/nova/+/812144 | |
| 14:34:39 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove unused build_requests columns https://review.opendev.org/c/openstack/nova/+/812145 | |
| 14:34:39 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove legacy placement models https://review.opendev.org/c/openstack/nova/+/812146 | |
| 14:34:40 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Stop querying the main DB for keypairs https://review.opendev.org/c/openstack/nova/+/812147 | |
| 14:34:40 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Remove 'bandwidth' fields from notifications https://review.opendev.org/c/openstack/nova/+/812148 | |
| 14:34:41 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove models that were moved to the API database https://review.opendev.org/c/openstack/nova/+/812149 | |
| 14:34:41 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove models for removed services, features https://review.opendev.org/c/openstack/nova/+/812150 | |
| 14:34:42 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove nova-network models https://review.opendev.org/c/openstack/nova/+/812151 | |
| 14:34:43 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: objects: Prepare for Instance, InstanceList v3 https://review.opendev.org/c/openstack/nova/+/812152 | |
| 14:34:43 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: objects: Add Instance, InstanceList v3 https://review.opendev.org/c/openstack/nova/+/812153 | |
| 14:42:35 | stephenfin | FYI, those patches are me stress testing the alembic infrastructure and fixing things I find along the way. Would be useful to merge early in Yoga (i.e. starting next week?) to give us loads of time to resolve any potential fallout :) | |
| 14:48:17 | gibi | stephenfin: ack, good point | |
| 15:06:48 | spatel | Folk, not sure what i should look for but we have same hardware on both cloud but local storage showing different result - https://paste.opendev.org/show/809727/ | |
| 15:09:06 | spatel | I don't have any option configure here on both clouddisk_cachemodes = | |
| 15:12:39 | artom_ | bauzas, around? Does https://bugs.launchpad.net/nova/+bug/1944947 and the fix at https://review.opendev.org/c/openstack/nova/+/810849 qualify as a review-priority bug? | |
| 15:12:55 | bauzas | still there | |
| 15:12:56 | artom_ | Seems pretty important to be able to restart hw:cpu_policy=dedicated instances after an upgrade :) | |
| 15:13:32 | bauzas | artom: any bug seems to me nice to review | |
| 15:13:39 | bauzas | so +1 on R-P labeling | |
| 15:13:48 | artom | priteau, btw, are you good to continue the patch above? ^^ It's basically just missing unit tests | |
| 15:13:57 | artom | Or would you want someone more experienced with Nova to take over? | |