| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-05-01 | |||
| 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 | openstackgerrit | Merged openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404 | |
| 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: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 | gmann | ok | |
| 15:29:11 | openstackgerrit | Merged openstack/os-vif stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/724713 | |
| 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 | gmann | lyarwood: seems rescue from BFV servers failing on ceph backend. https://bugs.launchpad.net/devstack-plugin-ceph/+bug/1876330 | |
| 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: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 | |
| 09:29:12 | openstackgerrit | Takashi Natsume proposed openstack/os-vif master: Remove six https://review.opendev.org/724966 | |
| 12:11:51 | openstackgerrit | Takashi Natsume proposed openstack/os-vif master: Remove Babel requirement https://review.opendev.org/724974 | |
| 12:50:37 | openstackgerrit | Sean McGinnis proposed openstack/nova master: Add py38 package metadata https://review.opendev.org/724990 | |
| 17:32:36 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Validate a BDMs disk_bus when provided https://review.opendev.org/724842 | |
| 23:28:21 | openstackgerrit | Thomas Goirand proposed openstack/nova master: Add a /healthcheck URL https://review.opendev.org/724684 | |
| #openstack-nova - 2020-05-04 | |||
| 00:27:23 | openstackgerrit | Ghanshyam Mann proposed openstack/nova stable/ussuri: Add nova-status upgrade check and reno for policy new defaults https://review.opendev.org/725146 | |
| 00:57:10 | openstackgerrit | Ghanshyam Mann proposed openstack/nova stable/train: DNM: testing what all grenade jobs running https://review.opendev.org/725147 | |
| 02:33:07 | alex_xu | sean-k-mooney: gibi fyi, we will have someone working on provider config xml. | |
| 05:08:03 | openstackgerrit | Takashi Natsume proposed openstack/python-novaclient master: [WIP] Switch legacy Zuul jobs to native Zuul v3 jobs https://review.opendev.org/725163 | |
| 06:55:55 | gibi | alex_xu: ack, thanks | |
| 08:33:14 | gibi | stephenfin: hi! Do you have an idea why the reno job fails on this backport to stable/ussuri? https://review.opendev.org/#/c/725146/1 It is green for me locally but locally the releasenotes tox target does not try to generate stable/ussuri notes | |
| 08:56:25 | bauzas | gibi: me looks | |
| 08:59:31 | bauzas | gibi: I don't see any problems with the reno file itself, just testing it locally | |
| 08:59:59 | bauzas | Warning, treated as error: <reno.sphinxext stable/ussuri>:115:Duplicate target name, cannot be used as a unique reference: "upgrade notes". | |
| 09:00:07 | bauzas | that's why we get -1 ^ | |
| 09:03:11 | bauzas | ahah I think we probably have something missing https://docs.openstack.org/reno/latest/user/usage.html#updating-stable-branch-release-notes | |
| 09:03:53 | bauzas | "Occasionally it is necessary to update release notes for past releases due to URLs changing or errors not being noticed until after they have been released. In cases like these, it is important to note that any updates to these release notes should be proposed directly to the stable branch where they were introduced." | |
| 09:11:20 | bauzas | ok, so, relnotes are correctly written against master, but cherry-picking the change to my local stable/ussuri branch and retesting | |
| 09:11:43 | bauzas | I think I found the problem | |
| 09:12:31 | bauzas | gibi: we don't have *yet* ussuri release notes so reno writes the patch note against master twice | |
| 09:12:34 | bauzas | hence the duplicate | |