| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-05-01 | |||
| 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/ | |
| 15:45:08 | melwitt | gmann: extra space on L3! -1! | |
| 15:51:53 | lyarwood | sigh, what's with the weird encoding of the logs with this job? | |
| 15:52:00 | lyarwood | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_32a/723645/15/check/devstack-plugin-ceph-tempest-py3/32ae221/controller/logs/screen-n-cpu.txt for example | |
| 15:52:13 | lyarwood | I can't pull it down to review quickly | |
| 15:59:59 | gmann | melwitt: updated. thanks | |
| 16:01:42 | lyarwood | yeah this really looks like an issue either with the snapshot we've downloaded from glance *or* with the deployed ceph env that we are trying to import it into | |
| 18:20:33 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Follow-up for NUMA live migration functional tests https://review.opendev.org/724903 | |
| 18:24:25 | artom | I wonder if I can backport that at least to U... stephenfin churned the func tests too much in after T | |
| 18:24:29 | artom | s/churned/improved/ :) | |
| 19:56:20 | markguz_ | Hi Nova folks. I'm scratching my head just now over soft-anti-affinity. I have 4 compute nodes, and I want to be able to deploy servers with anti-afinity policy | |
| 19:57:01 | markguz_ | hard anti-affinity works perfectly. 4 vms 1 on each compute node and then fail on a 5th. | |
| 19:57:42 | markguz_ | soft-anti-affinity i thought meant that like hard anti-affinity it would put an instance on each node, but not fail on the 5th and just put on the next available one | |
| 19:58:24 | markguz_ | not so. it seems that when using soft-aa the system will stack instances on 1 node. i'm stumped. | |
| 19:59:06 | markguz_ | i have the default schedulers enabled. and i have modified any of the weight stuff from the defaults either. what's going on? | |
| 21:05:40 | melwitt | markguz_: I'm not an expert on this, but from what I see in these docs https://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/soft-affinity-for-server-group.html and https://docs.openstack.org/nova/latest/user/filter-scheduler.html#weights soft-anti-affinity is implemented via weights only, so you might have to tweak the weight multiplier to get a better result? | |
| 21:06:04 | melwitt | https://docs.openstack.org/nova/latest/configuration/config.html#filter_scheduler.soft_anti_affinity_weight_multiplier | |
| 21:07:10 | melwitt | and finally this might helpful https://specs.openstack.org/openstack/nova-specs/specs/rocky/implemented/complex-anti-affinity-policies.html | |
| 23:38:17 | markguz_ | melwitt: yeah i've been looking at the weights. was beginning to come to that conclusion too. Thanks for the pointers! | |
| #openstack-nova - 2020-05-02 | |||
| 01:08:29 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add nested resource providers limit for multi create https://review.opendev.org/723884 | |
| 01:10:35 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add nested resource providers limit for multi create https://review.opendev.org/723884 | |
| 03:44:16 | openstackgerrit | Merged openstack/nova master: Add nova-status upgrade check and reno for policy new defaults https://review.opendev.org/723645 | |
| 04:08:54 | openstackgerrit | Jie Li proposed openstack/nova master: replace the "hide_hypervisor_id" to "hw:hide_hypervisor_id" https://review.opendev.org/722187 | |
| 05:26:00 | openstackgerrit | Takashi Natsume proposed openstack/python-novaclient master: Remove mock in lower-constraints.txt https://review.opendev.org/724956 | |
| 06:24:18 | openstackgerrit | Takashi Natsume proposed openstack/nova master: doc: Fix list rendering in cli/nova-status.rst https://review.opendev.org/724958 | |
| 08:02:54 | openstackgerrit | Takashi Natsume proposed openstack/os-vif master: Use unittest.mock instead of third party mock https://review.opendev.org/724965 | |