| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-05-01 | |||
| 11:19:21 | sean-k-mooney | speaking of finding someone tonyb: would you be able to take a look at two procedual patches .gitreview update https://review.opendev.org/#/c/724712/2 and tox constraits update https://review.opendev.org/#/c/724713/2 for os-vif | |
| 11:19:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use unittest.mock instead of third party mock https://review.opendev.org/714676 | |
| 11:19:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove future imports https://review.opendev.org/714675 | |
| 11:19:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hacking: force explicit import of python's mock https://review.opendev.org/708768 | |
| 11:21:32 | sean-k-mooney | oh im not on irc downstream my vpn is still down | |
| 11:34:11 | openstackgerrit | Merged openstack/nova stable/rocky: Lowercase ironic driver hash ring and ignore case in cache https://review.opendev.org/723050 | |
| 12:29:29 | openstackgerrit | Merged openstack/os-vif master: trivial: Remove some rules from flake8 ignore list https://review.opendev.org/716223 | |
| 13:10:42 | lyarwood | stephenfin: another trivial docs question for you, is there a way to reference the values of a given nova.objects.field in the api-ref? | |
| 13:11:21 | lyarwood | stephenfin: adding a validation of block_device_mapping_v2.disk_bus in the compute API and wanted to update the docs with the possible values without listing them out as a static list | |
| 13:11:34 | artom | Wait, we have the nova objects fields documented somewhere? | |
| 13:11:41 | stephenfin | Not easily. You'd have to write a custom extension to do it. That might be worth it if you plan to do it multiple times, but otherwise I'd just add a NOTE reminding people to update the docs if they change the o.vo | |
| 13:12:53 | lyarwood | artom: no this is in the api-ref for the actual request, I'm just using the fields to validate the input | |
| 13:13:21 | lyarwood | stephenfin: ack understood, I couldn't find one so just wanted to check that I wasn't missing anything | |
| 13:21:05 | openstackgerrit | tianmaofu proposed openstack/nova master: Delete queued image after deleting instance when creating image from instance https://review.opendev.org/724839 | |
| 13:40:46 | sean-k-mooney | lyarwood: out side of the notification object there should be non ovos at teh api level currently | |
| 13:41:02 | sean-k-mooney | we dont use ovo over any of the rest apis | |
| 13:41:23 | dansmith | aren't you guys on holiday today? | |
| 13:41:23 | sean-k-mooney | so i would not expect ovo to be mentioned in the api-ref at all | |
| 13:41:29 | sean-k-mooney | monday | |
| 13:41:34 | sean-k-mooney | in ireland | |
| 13:41:34 | dansmith | oh I thought both days | |
| 13:42:05 | sean-k-mooney | no ireland just moves the bank holiday to the first monday in may | |
| 13:42:24 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Validate a BDMs disk_bus when provided https://review.opendev.org/724842 | |
| 13:42:25 | dansmith | okay but is UK of today? | |
| 13:42:30 | sean-k-mooney | the uk normally doesd too but the have friday next week off instead as due to the 75 aniversry of ve day | |
| 13:42:58 | dansmith | okay I thought it was today and monday | |
| 13:43:04 | lyarwood | sean-k-mooney: ^ I'm not using the o.vo objects, just the associated field to validate the values in the compute API | |
| 13:43:07 | sean-k-mooney | dansmith: i think france is today | |
| 13:43:11 | dansmith | ahh | |
| 13:43:20 | dansmith | of course france | |
| 13:44:23 | sean-k-mooney | i assume the use is monday? | |
| 13:44:27 | sean-k-mooney | *us | |
| 13:44:42 | dansmith | nope, our next one is end of may | |
| 13:44:58 | sean-k-mooney | oh i tought ye had memorial day or something this weekend | |
| 13:45:15 | sean-k-mooney | oh that the end of may | |
| 13:45:18 | dansmith | that's end yeah | |
| 13:46:41 | sean-k-mooney | dansmith: you on nova-stable-maint right mind hitting these too os-vif stabel branch creation changes https://review.opendev.org/#/c/724712/2 and https://review.opendev.org/#/c/724713/2 | |
| 13:49:38 | artom | dansmith, hey, can you hit https://review.opendev.org/#/c/687404/ when you get a chance? If you're cool with it as-is, I can do all of the NUMA LM outstanding stuff in a single FUP | |
| 13:50:30 | dansmith | oh did I miss this? sorry | |
| 13:50:51 | sean-k-mooney | dansmith: thanks :) | |
| 13:50:56 | artom | dansmith, no worries, I think I linked you the bottom patch last time I pinged you. And thanks :) | |
| 13:51:10 | dansmith | I was really planing for all you people to be gone and me not have to work today | |
| 13:51:22 | dansmith | hmm, that sounds like the manifesto of a madma | |
| 13:51:24 | dansmith | *madman | |
| 13:51:26 | artom | Hey now, I'm only French in language | |
| 13:51:50 | artom | dansmith, or a serial killer congratulating himself on a job well done | |
| 13:51:52 | sean-k-mooney | dansmith: look forward to monday i guess | |
| 13:51:58 | dansmith | heh | |
| 13:52:31 | lyarwood | dansmith turning into our very own Milton | |
| 13:52:49 | dansmith | hey now, that's going a little far | |
| 13:53:41 | sean-k-mooney | dansmith: at least he is not confusting you with a boomer like stephenfin did | |
| 13:53:42 | lyarwood | apologies | |
| 13:54:29 | dansmith | stephenfin confused me with a boomer? | |
| 13:54:39 | stephenfin | yeah, what? | |
| 13:55:14 | sean-k-mooney | it was at the ptg in vancover and you quickly tried to gloss over it | |
| 13:55:17 | sean-k-mooney | maybe it worked | |
| 13:56:08 | stephenfin | say wha | |
| 13:56:36 | lyarwood | stephenfin: officespace, burns the office down. | |
| 13:56:42 | stephenfin | ahhhh | |
| 13:58:13 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Validate a BDMs disk_bus when provided https://review.opendev.org/724842 | |
| 13:59:00 | sean-k-mooney | stephenfin: by the way is https://review.opendev.org/#/c/724188/4 and https://review.opendev.org/#/c/724725/1 what you wanted | |
| 13:59:18 | sean-k-mooney | we already have the min version set to 10. something on master | |
| 14:00:36 | sean-k-mooney | the warning message is there from stien on and we cant bump the min version there so i kept the compat code in the first patch | |
| 14:00:45 | stephenfin | LGTM | |
| 14:00:59 | sean-k-mooney | cool | |
| 14:01:48 | sean-k-mooney | lyarwood: we are not merging any backport to stable/ussuri untill after the release right | |
| 14:01:54 | sean-k-mooney | well that are not blockers | |
| 14:01:58 | sean-k-mooney | or rc bugs | |
| 14:02:18 | lyarwood | AFAIK yeah | |
| 14:02:39 | sean-k-mooney | cool i wont cherry pick untill its merged and we are passed that point so | |
| 14:03:19 | lyarwood | sean-k-mooney: triage downstream btw | |
| 14:26:56 | openstack | bug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,Fix committed] https://launchpad.net/bugs/1845146 - Assigned to Dan Smith (danms) | |
| 14:26:56 | openstackgerrit | Merged openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404 | |
| 14:56:07 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: Lowercase ironic driver hash ring and ignore case in cache https://review.opendev.org/723054 | |
| 14:56:08 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: Include only required fields in ironic node cache https://review.opendev.org/724862 | |
| 15:13:57 | stephenfin | dansmith: I think this is ready to go now, if you have a chance to look at it again today https://review.opendev.org/#/c/724381/ | |
| 15:14:08 | stephenfin | Just going to strip the WIP tag now. Forgot to do that previously | |
| 15:14:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add migrate-on-load behavior for legacy NUMA objects https://review.opendev.org/724381 | |
| 15:16:55 | gmann | melwitt: yeah, there are lot of ways of doing the policy file things, hope we will provide a consistent single way after PTG discussion. | |
| 15:23:56 | melwitt | gmann: so are you saying that blank policy.json file or no policy.json file apply to option 3? I just wanted to understand that detail https://review.opendev.org/#/c/723645/15/nova/cmd/status.py@378 | |
| 15:25:54 | gmann | melwitt: yes and same for option2- file generated with oslo tool in yaml format ( which will have all rule commented out ) will use the defaults in code | |
| 15:26:08 | gmann | sorry option1 | |
| 15:27:00 | melwitt | thank you! | |
| 15:27:48 | gmann | melwitt: you want me to add in msg? anyways gate is failed so it has to run tests again. i can edit if you are ok with re+A ? | |
| 15:29:04 | melwitt | gmann: yeah I'll re+A | |
| 15:29:10 | openstackgerrit | Merged openstack/os-vif stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/724712 | |
| 15:29:11 | openstackgerrit | Merged openstack/os-vif stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/724713 | |
| 15:29:11 | gmann | ok | |
| 15:32:15 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add nova-status upgrade check and reno for policy new defaults https://review.opendev.org/723645 | |
| 15:32:26 | gmann | melwitt: updated ^^ | |
| 15:32:55 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add nova-status upgrade check and reno for policy new defaults https://review.opendev.org/723645 | |
| 15:33:05 | gmann | removed extra space in msg | |
| 15:36:04 | melwitt | gmann: +W thanks a lot | |
| 15:36:14 | gmann | melwitt: thanks | |
| 15:37:11 | openstack | Launchpad bug 1876330 in devstack-plugin-ceph "Rescue BFV instances feature not supported in ceph, Tempest new test failing" [Undecided,New] | |
| 15:37:11 | gmann | lyarwood: seems rescue from BFV servers failing on ceph backend. https://bugs.launchpad.net/devstack-plugin-ceph/+bug/1876330 | |
| 15:37:37 | gmann | I am going to skip the new tests to unblock the ceph job for now. | |
| 15:37:54 | lyarwood | gmann: looking | |
| 15:39:40 | lyarwood | gmann: that's unrelated to the feature at first glance and more a general rbd import failure of the import of the rescue image, ./me digs more. | |
| 15:42:24 | gmann | lyarwood: pushed the skip patch to unblock the gate where ceph job is voting. - https://review.opendev.org/#/c/724866/ | |