| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-01-27 | |||
| 17:54:51 | sean-k-mooney | lbragstad: e.g. it will hit keystone to valideate it | |
| 17:55:01 | sean-k-mooney | as part of normal token validation | |
| 17:55:41 | lbragstad | sean-k-mooney ksm cleans out those headers and repopulates them based on the token validation response | |
| 17:55:42 | sean-k-mooney | assuming yes then i think i understand how the tests are workign now | |
| 17:56:22 | sean-k-mooney | lbragstad: ok so the imporant thin is in the project we can trust the roles in the context because ksm has checked for us | |
| 17:56:31 | lbragstad | https://opendev.org/openstack/keystonemiddleware/src/branch/master/keystonemiddleware/auth_token/_request.py#L75 | |
| 17:56:38 | sean-k-mooney | thats what i was expecting but just wanted to confirm | |
| 17:56:49 | lbragstad | https://opendev.org/openstack/keystonemiddleware/src/branch/master/keystonemiddleware/auth_token/_request.py#L224-L227 | |
| 17:57:29 | openstackgerrit | Stephen Finucane proposed openstack/placement master: policy: Add note about keystone's expansion of roles https://review.opendev.org/c/openstack/placement/+/772752 | |
| 17:57:29 | sean-k-mooney | lbragstad: thanks perfect | |
| 17:57:37 | lbragstad | yep | |
| 17:57:40 | stephenfin | sean-k-mooney: lemme know if that's useful ^ | |
| 17:58:04 | stephenfin | We can refine wording if so | |
| 17:58:20 | sean-k-mooney | stephenfin: yep no that make sense to me | |
| 17:58:41 | sean-k-mooney | im not sure why gerrit thinks you change the last 3 lins | |
| 17:59:04 | sean-k-mooney | i should have a , after yep | |
| 17:59:30 | stephenfin | Oh no, I really did add spacing :) | |
| 17:59:37 | stephenfin | a'ight, and with that | |
| 17:59:56 | stephenfin | kashyap: I'll take a look at that tomorrow | |
| 18:00:13 | kashyap | stephenfin: Thank you; it is fixing some real gnarly problem | |
| 18:35:43 | openstackgerrit | Merged openstack/nova stable/victoria: compute: Don't detach volumes when RescheduledException raised without retry https://review.opendev.org/c/openstack/nova/+/764612 | |
| 19:15:01 | gmann | stephenfin: lbragstad replied on this. as per my finding/run oslo.policy does not change rule's self.check_str even it only change rule's self.check which is internal attribute for oslo policy to play with - https://review.opendev.org/c/openstack/placement/+/772508/1//COMMIT_MSG#13 | |
| 19:15:53 | gmann | I ran few scenario and did not find oslo policy changing defined rule check_str. | |
| 19:18:10 | openstackgerrit | Merged openstack/nova stable/victoria: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/c/openstack/nova/+/761809 | |
| 19:37:40 | openstackgerrit | melanie witt proposed openstack/nova stable/ussuri: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/c/openstack/nova/+/761810 | |
| 20:24:28 | openstackgerrit | Vlad Gusev proposed openstack/nova stable/train: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/c/openstack/nova/+/761811 | |
| 20:31:13 | openstackgerrit | Vlad Gusev proposed openstack/nova stable/train: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/c/openstack/nova/+/761811 | |
| 20:34:14 | openstackgerrit | Vlad Gusev proposed openstack/nova stable/stein: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/c/openstack/nova/+/761812 | |
| 20:35:35 | openstackgerrit | Vlad Gusev proposed openstack/nova stable/rocky: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/c/openstack/nova/+/761813 | |
| 20:37:04 | openstackgerrit | Vlad Gusev proposed openstack/nova stable/queens: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/c/openstack/nova/+/761814 | |
| 21:08:17 | openstackgerrit | Merged openstack/nova stable/stein: Update pci stat pools based on PCI device changes https://review.opendev.org/c/openstack/nova/+/761727 | |
| 22:04:56 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: extra specs/image pros: add `socket PCI NUMA affinity https://review.opendev.org/c/openstack/nova/+/772748 | |
| 22:04:56 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Add `socket` PCI NUMA affinity policy request prefilter https://review.opendev.org/c/openstack/nova/+/772749 | |
| 22:04:57 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: pci: implement the SOCKET NUMA affinity policy https://review.opendev.org/c/openstack/nova/+/772779 | |
| 22:15:01 | openstackgerrit | Ghanshyam proposed openstack/nova master: DNM:try l-c with direct deps https://review.opendev.org/c/openstack/nova/+/772780 | |
| 22:48:37 | openstackgerrit | Ghanshyam proposed openstack/placement master: Move policy deprecation to base rules https://review.opendev.org/c/openstack/placement/+/772784 | |
| 23:06:56 | gmann | brinzhang_: few comments left to fix in https://review.opendev.org/c/openstack/nova/+/764292 | |
| #openstack-nova - 2021-01-28 | |||
| 00:25:16 | brinzhang_ | gmann: ack, thanks | |
| 00:25:22 | brinzhang_ | gmann: https://review.opendev.org/c/openstack/nova/+/766726/13/nova/api/openstack/compute/views/servers.py | |
| 00:25:41 | brinzhang_ | I was confusing with this comments, what does your mean? | |
| 00:27:03 | gmann | brinzhang_: i commented in this file that we do not need to change get_instance_security_groups method itself https://review.opendev.org/c/openstack/nova/+/766726/11/nova/network/security_group_api.py#279 | |
| 00:27:39 | gmann | we get the sg from neutron and neutron does return tenant_id and project_id in sg response but both are same thing | |
| 00:28:01 | gmann | they kept both for backward compatibility as they do not have microversion concept. | |
| 00:29:19 | brinzhang_ | gmann:in "def _convert_to_nova_security_group_format(" we update the security_group[], does it need to be consider? | |
| 00:29:58 | gmann | brinzhang_: this one right https://review.opendev.org/c/openstack/nova/+/766726/11/nova/network/security_group_api.py#279 | |
| 00:30:35 | gmann | we can just change L275 from 'nova_group['project_id'] = security_group['tenant_id']' -> nova_group['project_id'] = security_group['project_id'] | |
| 00:31:12 | gmann | and without microversion condition. that way we do not need to change the signature of any interface | |
| 00:31:48 | brinzhang_ | I think I get you idea | |
| 00:31:59 | brinzhang_ | s/you/your/ | |
| 00:32:04 | gmann | cool | |
| 00:33:08 | brinzhang_ | in neutron api, we dont need to be consider the nova microversion, we just need to get the sg keep the same way | |
| 00:33:36 | gmann | yeah and fetch project_id from neutron sg instead of tenant_id | |
| 00:34:24 | gmann | because at some point in future neutron also will remove the tenant_id form their API so we can take care of that in advance | |
| 00:34:37 | brinzhang_ | yeah | |
| 00:34:45 | gmann | brinzhang_: and in first commit, we can remove the 2.89 sample. i commented in previous PS also. sorry about asking for that in initial reiview. this one - https://review.opendev.org/c/openstack/nova/+/764292/14/nova/tests/functional/api_sample_tests/test_servers.py#659 | |
| 00:36:05 | brinzhang_ | gmann: np, will update in next patch | |
| 00:36:12 | gmann | brinzhang_: thanks | |
| 00:36:56 | brinzhang_ | gmann: did you review the os-simple-project-suage patch? the router change, and that hit the test issue? | |
| 00:38:01 | gmann | brinzhang_: I did until 766726 but I will check the simple-project-usage tomorrow | |
| 00:38:04 | brinzhang_ | that I requested 2.90, but it always goto <=2.89 index or show APis | |
| 00:38:51 | gmann | humm strange | |
| 00:38:52 | brinzhang_ | gmann: dont worry^, my env has broken yesterday, firstly, I will restore my env | |
| 00:39:11 | gmann | I will debug that tomorrow. dinner time for me | |
| 00:39:31 | gmann | routes.py looks fine to me in first glance may be API controller causing it | |
| 00:39:52 | brinzhang_ | gmann: thanks, have a good dinner ^^ | |
| 06:15:32 | openstackgerrit | Hemanth N proposed openstack/nova stable/rocky: Update pci stat pools based on PCI device changes https://review.opendev.org/c/openstack/nova/+/761824 | |
| 11:00:11 | lyarwood | stephenfin: looking to undeprecate ['glance']/allowed_direct_url_schemes now that it's used by https://review.opendev.org/q/topic:bp/nova-image-download-via-rbd | |
| 11:01:03 | lyarwood | stephenfin: is that just a case of removing the deprecation notes etc? | |
| 11:01:32 | stephenfin | Yeah, remove the deprecation note from the config opt itself and if it's not already done, add a release note | |
| 11:04:25 | lyarwood | ack thanks | |
| 11:12:31 | zigo | Swift fails with Python 3.9 under Debian Unstable with Python 3.9: http://paste.openstack.org/show/802063/ | |
| 11:12:31 | zigo | I got the same problem with neutron-rpc-server when trying to tell Nova that my VM port is up: http://paste.openstack.org/show/802071/ | |
| 11:12:31 | zigo | This looks like general to OpenStack, and feels like yet-another-problem-with-eventlet... :( | |
| 11:20:55 | stephenfin | anyone want to speed up our docs build? https://review.opendev.org/c/openstack/nova/+/751034 | |
| 11:40:25 | kashyap | stephenfin: Nice | |
| 11:41:12 | kashyap | stephenfin: Thanks for that; several times I wished for speeder doc builds! | |
| 11:44:43 | kashyap | Gave my lowly +1, FWIW | |
| 11:59:21 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Use specific user when probing encrypted rbd disks during extend https://review.opendev.org/c/openstack/nova/+/772869 | |
| 12:01:39 | lyarwood | stephenfin: looking | |
| 12:01:53 | lyarwood | stephenfin: also I've lost the tabs with the reviews you asked me about yesterday, what were they again? | |
| 12:02:44 | stephenfin | https://review.opendev.org/c/openstack/nova/+/75655, https://review.opendev.org/c/openstack/nova/+/752912 and if possible https://review.opendev.org/c/openstack/nova/+/756552 | |
| 12:03:24 | lyarwood | ack | |
| 12:03:26 | lyarwood | thanks | |
| 12:18:09 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Increase SWIFT_LOOPBACK_DISK_SIZE within nova-lvm job https://review.opendev.org/c/openstack/nova/+/772702 | |
| 12:18:09 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Mark nova-lvm as voting https://review.opendev.org/c/openstack/nova/+/772871 | |
| 12:47:21 | openstackgerrit | Lee Yarwood proposed openstack/nova master: glance: Remove [glance]/allowed_direct_url_schemes https://review.opendev.org/c/openstack/nova/+/772874 | |
| 13:30:41 | openstackgerrit | Takashi Natsume proposed openstack/nova-specs master: Create specs directory for Xena https://review.opendev.org/c/openstack/nova-specs/+/772878 | |
| 13:57:43 | openstackgerrit | Ghanshyam proposed openstack/placement master: Move policy deprecation to base rules https://review.opendev.org/c/openstack/placement/+/772784 | |
| 14:00:02 | lyarwood | stephenfin: https://review.opendev.org/c/openstack/nova/+/771981 - would you mind hitting this today if you have some docs review bandwidth | |
| 14:00:23 | lyarwood | https://review.opendev.org/c/openstack/nova/+/772874 might also be something you'd be interested in | |
| 14:02:12 | lyarwood | gibi: https://review.opendev.org/c/openstack/nova/+/772871/ - could you take a look at this change marking the nova-lvm job as voting (and the fix underneath it)? | |
| 14:02:56 | gibi | lyarwood: ack, added to my queue. (that queue getting log in the last couple of days) | |
| 14:03:35 | lyarwood | gibi: ack np, if there's anything I can trade let me know | |
| 14:03:54 | lyarwood | gibi: systemctl restart nova-compute? ;) | |
| 14:05:10 | gibi | yepp that what couple of our customers do weekly but they getting impatient | |
| 14:05:24 | gibi | fun stuf | |
| 14:05:26 | lyarwood | ouch | |
| 14:06:29 | gibi | I had the same experience a decade ago with jvm :D the solution was the same, restart it during maintenance window to free up ram | |
| 14:08:00 | sean-k-mooney | gibi: given python is garbage collected any memory leask is likely form interaction with libvirt right | |
| 14:08:17 | sean-k-mooney | e.g. the proxy objects | |