Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-05
21:37:43 mriedem i've got a -2 toward the bottom
21:37:52 dansmith mriedem: you should bug me to look at three of those tomorrow
21:38:01 mriedem ok
23:17:47 openstackgerrit Merged openstack/nova master: PCI: do not force remove allocated devices https://review.openstack.org/626381
23:27:07 openstackgerrit sean mooney proposed openstack/nova master: Add free for claimed, allocated devices https://review.openstack.org/616120
23:27:07 openstackgerrit sean mooney proposed openstack/nova master: Libvirt: do not set mac when unplugging macvtap vf https://review.openstack.org/624842
23:27:08 openstackgerrit sean mooney proposed openstack/nova master: Add get_instance_pci_request_from_vif https://review.openstack.org/619929
23:27:08 openstackgerrit sean mooney proposed openstack/nova master: Allow per-port modification of vnic_type and profile https://review.openstack.org/607365
23:27:09 openstackgerrit sean mooney proposed openstack/nova master: libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589
23:27:09 openstackgerrit sean mooney proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115
23:29:37 openstackgerrit sean mooney proposed openstack/nova stable/rocky: PCI: do not force remove allocated devices https://review.openstack.org/635071
23:29:55 openstackgerrit sean mooney proposed openstack/nova stable/queens: PCI: do not force remove allocated devices https://review.openstack.org/635072
23:38:31 openstackgerrit sean mooney proposed openstack/nova stable/pike: PCI: do not force remove allocated devices https://review.openstack.org/635074
23:38:56 openstackgerrit sean mooney proposed openstack/nova stable/ocata: PCI: do not force remove allocated devices https://review.openstack.org/635075
#openstack-nova - 2019-02-06
00:28:55 artom What takes precedence again? Flavor extra specs or image props?
00:30:09 sean-k-mooney that depends we generally now raise an exception if there is a conflict but original flavor took prescedence over image properties
00:31:15 sean-k-mooney im not sure we we have converted all conficts into exections currently but stephenfin did that for several of them
00:31:42 artom How are you still working o_O!
00:31:48 artom I'm happy you're here though :D
00:32:05 artom I'm asking because mem_page_size is in both
00:32:55 sean-k-mooney hehe i often take a few hours away around 7 my time and come back online for an hour or two around 10 ish
00:33:03 sean-k-mooney ya
00:33:32 sean-k-mooney so for mempage size if the flavor has set a spefici page size and the image does not agree its an error
00:34:21 sean-k-mooney if the flavoer was hw:mem_page_size=large or hw:mem_page_size=any and the image had hw_mem_page_size=1G that should be allow but i dont know if we support it or not
00:34:31 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add prep_snapshot_based_resize_at_dest compute method https://review.openstack.org/633293
00:34:32 openstackgerrit Matt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_source compute method https://review.openstack.org/634832
00:34:32 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtDestTask https://review.openstack.org/627890
00:34:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Move finish_resize.(start|end) notifications to helper method https://review.openstack.org/635079
00:34:33 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtSourceTask https://review.openstack.org/627891
00:34:34 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add finish_snapshot_based_resize_at_dest compute method https://review.openstack.org/635080
00:35:09 artom sean-k-mooney, so based on unit tests it looks like mem_page_size in image is outright forbidden...
00:35:19 mriedem artom: i've always though image overrides flavor since the user can sometimes upload their own images
00:35:35 mriedem stephenfin might think otherwise as well since i think he's written something up about this brfeore
00:35:37 mriedem *before
00:36:10 sean-k-mooney well the user should not be able to overide the admin but if ther is no conflict then its fine
00:36:13 artom mriedem, hrmm, doesn't look like that's universally true
00:36:19 artom sean-k-mooney, mriedem, https://github.com/openstack/nova/blob/f7252b586b4b9f5a098fedfa27715b8f7e662af6/nova/tests/unit/virt/test_hardware.py#L2253-L2264
00:36:32 artom To me that reads as we bail if there's a page size in image props
00:36:56 sean-k-mooney artom: https://github.com/openstack/glance/blob/master/etc/metadefs/compute-guest-memory-backing.json
00:37:40 sean-k-mooney artom no in test_get_requested_mempages_pagesize_from_image_forbidden
00:38:18 sean-k-mooney the flavor is hw:mem_page_size=small so not hugepages but the image asks for 2m hugepages so that is not allowed because teh admin said no
00:39:00 sean-k-mooney test_get_requested_mempages_pagesize_from_image_forbidden2 fails befause for the image to be allow to ask for hugepages teh flaovr must have hw:mem_page_size=large or any
00:39:24 artom Ah, so https://github.com/openstack/nova/blob/f7252b586b4b9f5a098fedfa27715b8f7e662af6/nova/virt/hardware.py#L1120 exists
00:39:36 sean-k-mooney artom: basically you can only aske for a specific page size via the image if the admin said it was ok in the flavor
00:39:41 sean-k-mooney at least that was the intent
00:40:54 sean-k-mooney artom https://github.com/openstack/nova/blob/f7252b586b4b9f5a098fedfa27715b8f7e662af6/nova/virt/hardware.py#L1169-L1170
00:42:18 sean-k-mooney artom: the idea wa that if the admin wanted to supprot hugepage guest they would only ever set hw:mem_page_size=large. and if the tenant cared about the size of the hugepage they would then say so in the image
00:42:59 artom sean-k-mooney, aha, that makes sense
00:44:09 artom Concretely, in our live migration claim case, it looks like we'll have to override both with the actual pagesize on the source host
00:44:21 artom Somehow
00:46:26 sean-k-mooney artom: the instance_numa_topology object has the pagesize
00:46:27 sean-k-mooney https://github.com/openstack/nova/blob/f7252b586b4b9f5a098fedfa27715b8f7e662af6/nova/objects/instance_numa_topology.py#L51
00:46:42 sean-k-mooney so when you are claiming the memory for the instance jsut pull it out of that
00:46:47 artom sean-k-mooney, yeah, that wasn't what I was bugging on
00:47:23 artom So actually, because https://github.com/openstack/nova/blob/f7252b586b4b9f5a098fedfa27715b8f7e662af6/nova/virt/hardware.py#L1168-L1170, if we set a numerical pagesize in the flavor, it'll override anything in the image
00:47:27 artom OK, I'm good
00:47:48 sean-k-mooney artom: yes but we should not mess with the flavor
00:47:54 sean-k-mooney or the image
00:48:04 sean-k-mooney just keep it as a seperate value
00:48:12 artom sean-k-mooney, not on the original, but if we deepcopy it we can do whatever we want :)
00:48:32 artom I know it's ugly, but I'd rather not dive into the claims code to make it understand a separate pagesize
00:48:57 sean-k-mooney its a nova object wich is a ovo so you all have to reset the fields to make shour it does not get persited back into the db in some cases
00:50:48 sean-k-mooney artom: well throw up a poc and then we can see what it looks like
00:50:57 artom sean-k-mooney, yep
01:39:01 sean-k-mooney night all o/
02:21:02 openstackgerrit Merged openstack/nova master: Move retry from _update to _update_to_placement https://review.openstack.org/634252
03:26:08 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] New objects to transmit NUMA config from dest to source https://review.openstack.org/634827
03:26:08 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Create claims when live migrating https://review.openstack.org/634606
03:26:09 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Send dest NUMA info inside LiveMigrateData https://review.openstack.org/634828
07:10:21 openstackgerrit Takashi NATSUME proposed openstack/nova master: Replace glance command with openstack command https://review.openstack.org/635102
07:13:56 openstackgerrit Takashi NATSUME proposed openstack/nova master: Replace glance command with openstack command https://review.openstack.org/635102
07:55:32 openstackgerrit Jan Gutter proposed openstack/nova master: Ignore some PendingDeprecationWarnings for os-vif https://review.openstack.org/634928
08:20:15 openstackgerrit Jan Gutter proposed openstack/os-vif master: Convert hardcoded regexes to raw strings for py36 https://review.openstack.org/635111
10:10:34 openstackgerrit Surya Seetharaman proposed openstack/nova master: Modify InstanceMappingList.get_not_deleted_by_cell_and_project() https://review.openstack.org/635120
10:10:34 openstackgerrit Surya Seetharaman proposed openstack/nova master: API microversion 2.68: Handles Down Cells https://review.openstack.org/591657
10:10:35 openstackgerrit Surya Seetharaman proposed openstack/nova master: Plumbing for ignoring list_records_by_skipping_down_cells https://review.openstack.org/635121
10:53:54 openstackgerrit Andrey Volkov proposed openstack/nova master: Check hosts have no instances for AZ rename https://review.openstack.org/509206
11:18:06 openstackgerrit Stephen Finucane proposed openstack/python-novaclient master: Microversion 2.68: Remove 'forced' live migrations, evacuations https://review.openstack.org/635131
11:22:03 openstackgerrit Stephen Finucane proposed openstack/nova master: API: Remove evacuate/live-migrate 'force' parameter https://review.openstack.org/634600
12:41:53 openstackgerrit Surya Seetharaman proposed openstack/nova master: Plumbing for ignoring list_records_by_skipping_down_cells https://review.openstack.org/635121
12:41:54 openstackgerrit Surya Seetharaman proposed openstack/nova master: Plumbing for allowing the all-tenants filter https://review.openstack.org/635145
12:41:54 openstackgerrit Surya Seetharaman proposed openstack/nova master: API microversion 2.68: Handles Down Cells https://review.openstack.org/591657
12:41:55 openstackgerrit Surya Seetharaman proposed openstack/nova master: API microversion 2.68: Handles Down Cells Documentation https://review.openstack.org/635147
12:41:55 openstackgerrit Surya Seetharaman proposed openstack/nova master: Plumbing required in ViewBuilder to construct partial results https://review.openstack.org/635146
13:10:30 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Create claims when live migrating https://review.openstack.org/634606
13:10:31 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Send dest NUMA info inside LiveMigrateData https://review.openstack.org/634828
13:10:31 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] New objects to transmit NUMA config from dest to source https://review.openstack.org/634827
13:45:07 bauzas good afternoon Nova, now begins my upstream day
13:46:12 sean-k-mooney o/ hello upstream bauzas :)
13:46:26 bauzas sylvainb != bauzas :)
13:46:41 bauzas so 'upstream bauzas' is just a pleonasm
13:47:40 sean-k-mooney bauzas: the real question is who is the evil one and who is the good one sylvainb or bauzas
13:48:02 bauzas it's schizophrenic, I know
13:48:16 bauzas mwuaaahahahah
13:49:50 mriedem tssurya: i could be wrong but it seems the commit message and comment in https://review.openstack.org/#/c/635120/ are backward
13:50:53 sean-k-mooney bauzas: the real question is who is the evil one and who is the good one sylvainb or bauzas
13:51:26 bauzas I can't answer this question without the presence of my lawyer
14:26:56 tssurya mriedem: um, no I meant for the ones where the data migration is run, it would be False/True while for the new ones its None/NULL at least in the database..
14:27:05 tssurya I'll update it on the review.
14:29:02 mriedem i'm not sure what "new ones" means

Earlier   Later