| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-01-21 | |||
| 16:19:24 | dansmith | yeah ^ | |
| 16:19:38 | artom | Makes all of our usual debugging tools (read: os-log-merger) completely useless | |
| 16:19:47 | sean-k-mooney | one of our aisa based customer does it | |
| 16:20:18 | sean-k-mooney | lave i think will work with the time stamps but really it depend on how they did it | |
| 16:20:41 | artom | sean-k-mooney, and it's a finicky external events evacuation issue as well | |
| 16:21:08 | artom | Where you need to correlate timestampe and requests across Neutron and Nova :( | |
| 16:23:34 | sean-k-mooney | oh ya i started looking at that but did not open the sos report | |
| 16:24:06 | sean-k-mooney | they should be aware that hw:numa_mempolicy or whatever it is is not a thing | |
| 16:24:25 | sean-k-mooney | oh wait imight be mixing too issues | |
| 16:25:00 | sean-k-mooney | your looking at https://bugzilla.redhat.com/show_bug.cgi?id=1917675 | |
| 16:25:01 | openstack | bugzilla.redhat.com bug 1917675 in openstack-nova "Nova evacuate fails due to timeout waiting for a network-vif-plugged event for instance" [High,New] - Assigned to nova-maint | |
| 16:25:34 | sean-k-mooney | the one with the fake extra spec is a different one i was look at breifly too | |
| 16:25:42 | artom | sean-k-mooney, yeah, that one. We can move downstream to discuss the specifics, tho | |
| 16:26:06 | sean-k-mooney | hehe ya i know just confirm there was not a duplicate | |
| 16:26:39 | sean-k-mooney | i just glanced at it so have not looked at the logs or anything | |
| 16:26:58 | artom | Their format makes them unworkable :( | |
| 16:47:55 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Load and cache volume drivers on-demand https://review.opendev.org/c/openstack/nova/+/741545 | |
| 16:50:45 | openstackgerrit | Stephen Finucane proposed openstack/placement master: tests: Suppress policy deprecation warnings https://review.opendev.org/c/openstack/placement/+/771852 | |
| 16:53:56 | stephenfin | lbragstad: When you've a chance, I've a question left on https://review.opendev.org/c/openstack/placement/+/760235 | |
| 16:56:54 | lyarwood | now I have to fix all of these broken tests | |
| 16:56:57 | lyarwood | gah | |
| 16:57:20 | stephenfin | tests broke? | |
| 16:57:22 | stephenfin | huh? | |
| 16:57:47 | lyarwood | passing in strings instead of dicts etc | |
| 16:58:18 | lyarwood | at least I think that's what this error means | |
| 16:58:20 | stephenfin | Oh, those won't get picked up by mypy unless you run it on them. Don't run mypy on tests :) | |
| 16:58:25 | stephenfin | Hmm, I doubt it | |
| 16:58:30 | stephenfin | Paste? | |
| 16:58:31 | lyarwood | TypeError: Parameters to generic types must be types. Got 0. | |
| 16:58:53 | lyarwood | http://paste.openstack.org/show/801843/ | |
| 17:00:38 | lyarwood | self.volume_drivers = ty.Dict[str, volume.LibvirtBaseVolumeDriver] is what I have for that dict | |
| 17:00:52 | stephenfin | yup, I just pulled down to see what's going on | |
| 17:01:20 | lyarwood | read* | |
| 17:01:30 | stephenfin | lyarwood: oh /o\ | |
| 17:01:36 | stephenfin | self.volume_drivers = ty.Dict[str, volume.LibvirtBaseVolumeDriver] | |
| 17:01:40 | stephenfin | should be | |
| 17:01:44 | stephenfin | self.volume_drivers: ty.Dict[str, volume.LibvirtBaseVolumeDriver] = {} | |
| 17:01:52 | stephenfin | var: type = value | |
| 17:01:58 | lyarwood | aaaaaah | |
| 17:02:22 | stephenfin | sorry, I missed that | |
| 17:05:21 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Load and cache volume drivers on-demand https://review.opendev.org/c/openstack/nova/+/741545 | |
| 17:05:22 | lyarwood | stephenfin: np thanks again ^ | |
| 17:08:53 | openstackgerrit | Merged openstack/nova master: Add regression test for bug #1899835 https://review.opendev.org/c/openstack/nova/+/758239 | |
| 17:08:55 | openstack | bug 1899835 in OpenStack Compute (nova) "n-cpu attempts to disconnect volumes after early pre_live_migration failures on the destination during a live migration" [Medium,In progress] https://launchpad.net/bugs/1899835 - Assigned to Lee Yarwood (lyarwood) | |
| 17:09:50 | lbragstad | stephenfin done | |
| 17:10:08 | openstackgerrit | Lance Bragstad proposed openstack/placement master: Implement secure RBAC for aggregates https://review.opendev.org/c/openstack/placement/+/760235 | |
| 17:10:09 | openstackgerrit | Lance Bragstad proposed openstack/placement master: Implement secure RBAC for allocations https://review.opendev.org/c/openstack/placement/+/760236 | |
| 17:10:10 | openstackgerrit | Lance Bragstad proposed openstack/placement master: Implement secure RBAC for allocation candidates https://review.opendev.org/c/openstack/placement/+/760237 | |
| 17:10:11 | openstackgerrit | Lance Bragstad proposed openstack/placement master: Implement secure RBAC for inventories https://review.opendev.org/c/openstack/placement/+/760238 | |
| 17:10:12 | openstackgerrit | Lance Bragstad proposed openstack/placement master: Implement secure RBAC for resource classes https://review.opendev.org/c/openstack/placement/+/760239 | |
| 17:10:13 | openstackgerrit | Lance Bragstad proposed openstack/placement master: Implement secure RBAC for traits https://review.opendev.org/c/openstack/placement/+/760241 | |
| 17:10:14 | openstackgerrit | Lance Bragstad proposed openstack/placement master: Implement secure RBAC for usage https://review.opendev.org/c/openstack/placement/+/760242 | |
| 18:00:31 | manoj_kumar_kata | hi network experts, we have 4-node openstack cluster and a recent lab poweroff resulted in losing network configuration. we worked on it and now the controller node could create vms but VMs are not getting the ip assigned . | |
| 18:00:41 | manoj_kumar_kata | appreciate your help here | |
| 19:08:04 | openstackgerrit | Merged openstack/nova master: zuul: Skip test_attach_cloned_encrypted_volume in nova-ceph-multistore https://review.opendev.org/c/openstack/nova/+/771777 | |
| 19:34:05 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: libvirt: Do not reference VIR_ERR_DEVICE_MISSING when libvirt is < v4.1.0 https://review.opendev.org/c/openstack/nova/+/747362 | |
| 19:35:10 | lyarwood | melwitt / elod ; https://review.opendev.org/c/openstack/nova/+/747362/ would you mind looking at this stack of stable/queens changes again, should be good to go now. | |
| 19:36:06 | lyarwood | I just tried to rebase that in the new gerrit UI and it has ended up with a merge conflict, wat?! | |
| 19:38:57 | lyarwood | ha, it just didn't report an actual conflict during the rebase attempt, fun | |
| 19:40:12 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: sync_guest_time: use the proper errno https://review.opendev.org/c/openstack/nova/+/742423 | |
| 19:40:13 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: libvirt: Remove reference to transient domain when detaching devices https://review.opendev.org/c/openstack/nova/+/608186 | |
| 19:40:14 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: libvirt: Handle VIR_ERR_DEVICE_MISSING when detaching devices https://review.opendev.org/c/openstack/nova/+/742424 | |
| 19:40:15 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: libvirt: Do not reference VIR_ERR_DEVICE_MISSING when libvirt is < v4.1.0 https://review.opendev.org/c/openstack/nova/+/747362 | |
| 19:40:36 | lyarwood | there rebased and hopefully ready | |
| 20:02:39 | elod | lyarwood: ack, I will review tomorrow :) | |
| 20:12:37 | openstackgerrit | Lee Yarwood proposed openstack/nova master: docs: Add reference docs for internal block device structures https://review.opendev.org/c/openstack/nova/+/771711 | |
| 20:34:05 | melwitt | sean-k-mooney: I read the backscroll from the nova meeting but I don't understand how it answers the question I had on the review, how can an operator prevent a user from leveraging numa aware vswitch? | |
| 20:34:51 | melwitt | "As an operator i wish to utilize numa aware vswitch but still be able to disable it for individual vm interfaces." | |
| 20:40:01 | sean-k-mooney | melwitt: what i ment by that was and still allow the tenant to disable it for indivigual vm interfaces | |
| 20:40:20 | sean-k-mooney | i had a other part of the spec that i planned to do but have not included too | |
| 20:40:37 | sean-k-mooney | which was to add a config option for the default polcy for numa vswitches | |
| 20:41:08 | melwitt | sean-k-mooney: ok, so tenant instead of operator. and that means there's a numa_affinity_policy that means disable? or by disable do you mean opt out | |
| 20:41:19 | sean-k-mooney | line 179 https://etherpad.opendev.org/p/nova-wallaby-ptg | |
| 20:41:55 | sean-k-mooney | melwitt: so if you have numa vsiwtchs configured thne you get strict affinity for every numa vm on the host | |
| 20:42:12 | sean-k-mooney | withthe port policy you coudl set it to prefer | |
| 20:42:25 | sean-k-mooney | which is basically best effort or no affinity enforced | |
| 20:42:53 | melwitt | ok, so there's not a concept of "disabling" it i.e. preventing it | |
| 20:43:07 | sean-k-mooney | the prefer policy is the closet thing | |
| 20:43:16 | sean-k-mooney | there is not way to request anti affinity | |
| 20:43:26 | sean-k-mooney | or disable all numa checks | |
| 20:43:34 | melwitt | ok, thanks | |
| 20:43:43 | sean-k-mooney | you can just say for this 1 interface do X | |
| 21:21:17 | openstackgerrit | Merged openstack/nova-specs master: add port scoped numa affinity spec https://review.opendev.org/c/openstack/nova-specs/+/765901 | |
| #openstack-nova - 2021-01-22 | |||
| 00:02:44 | brinzhang | sean-k-mooney, gibi: got it, thanks, will check | |
| 00:03:45 | gmann | lbragstad: left comment for placement policy - https://review.opendev.org/c/openstack/placement/+/760240/14/placement/tests/functional/fixtures/gabbits.py#763 | |
| 00:04:27 | gmann | lbragstad: if wee can run same or new tests with fixture to check if old token still work or not when enforce_scopy is false | |
| 00:06:23 | gmann | which is what we ship as default. | |
| 00:17:40 | brinzhang | sean-k-mooney: the os-simple-tenant-usage action will replace with os-simple-project-usage action, the tempest check the os-simple-tenant-usage action, if I dont skip the tempest check, it seems cannot pass | |
| 00:17:48 | brinzhang | sean-k-mooney: is it right? | |
| 00:18:07 | brinzhang | the nova change: https://review.opendev.org/c/openstack/nova/+/768852/7/nova/api/openstack/compute/routes.py#689 | |
| 00:20:03 | brinzhang | gmann: I know you are an expert in tempest, can you help to check? | |
| 00:20:30 | brinzhang | gmann: the tempest patch https://review.opendev.org/c/openstack/tempest/+/769120 | |
| 00:22:01 | gmann | brinzhang: sure, you want to move to openstack-qa ? | |
| 00:24:07 | brinzhang | gmann: of course | |
| 00:50:48 | gmann | brinzhang_: replied on gerrit for your query about os-simple-tenant-usage action https://review.opendev.org/c/openstack/nova/+/768852 | |
| 00:51:19 | gmann | brinzhang_: we still nee old API route and controller/sample/test etc as this is still valid for old microversion | |
| 00:53:06 | gmann | we do not have microversion knowledge in routes so we can add new one with old one still there and in API method in controller we can return 404 for old API for >2.89 | |
| 00:53:44 | brinzhang_ | gmann: greate, will check | |
| 00:54:02 | gmann | which is nothing but this @wsgi.Controller.api_version("2.1", "2.89") | |
| 00:55:09 | gmann | and new API method can be decorated with @wsgi.Controller.api_version("2.90") | |