| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-14 | |||
| 00:49:38 | sean-k-mooney | i proably should be less tired when reviewing this but this seams reversed to me | |
| 00:52:44 | melwitt | yeah, I think it will get all the aggregates, add any without the trait to the "no" bin, and then pass the "no" bin to placement and say "these are the forbidden aggregates, don't return hosts in them" | |
| 00:53:02 | sean-k-mooney | i think the logic is inverted. | |
| 00:53:13 | melwitt | which does seem backward but I assume it's the only way to do it | |
| 00:53:26 | sean-k-mooney | its treating the image and flaovr as the attoritive source nto the metadata | |
| 00:53:40 | sean-k-mooney | this is broken in the same way the existing filter is | |
| 00:53:58 | sean-k-mooney | and it the different betten the out of tree one and the in tree one | |
| 00:55:01 | sean-k-mooney | for this prefilter to work every trait that is in the image+flavor would have to be on the host_aggreate metadata | |
| 00:55:02 | melwitt | it does specifically say (in the proposed doc change) that image/flavor need not be set and will still not land on hosts in aggregates with =required. I just don't know how that's done | |
| 00:55:34 | sean-k-mooney | its done by invertin the relationship | |
| 00:56:10 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/ocata: Fix misuse of nova.objects.base.obj_equal_prims https://review.opendev.org/676295 | |
| 00:56:14 | sean-k-mooney | so what you need to do is get the metadata for all host_aggreates. then for each aggreate you check all the keys are present itn the flvore/image requst | |
| 00:56:32 | sean-k-mooney | if a key is set on the aggreate but not in the flaor/imgae then you add that to the forbindin list | |
| 00:56:34 | melwitt | oh ok | |
| 00:56:38 | melwitt | yeah | |
| 00:56:56 | sean-k-mooney | again exactly what the out of tree filter does :) | |
| 00:57:09 | sean-k-mooney | ok ill comment on the review | |
| 00:58:03 | sean-k-mooney | as implement this would work exactly the same as the aggregate_instance_extra_specs filter | |
| 00:58:24 | sean-k-mooney | well it sort of | |
| 00:58:41 | sean-k-mooney | it use traits instead of any proerty and also looks at the image | |
| 00:58:41 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/ocata: Fix misuse of nova.objects.base.obj_equal_prims https://review.opendev.org/676295 | |
| 00:58:47 | sean-k-mooney | but conceptuly its the same | |
| 01:01:14 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/ocata: Fix misuse of nova.objects.base.obj_equal_prims https://review.opendev.org/676295 | |
| 01:04:05 | sean-k-mooney | melwitt: actully it might be correct https://review.opendev.org/#/c/671074/6/nova/objects/aggregate.py@476 | |
| 01:04:58 | sean-k-mooney | in need to figure out what sql that generates | |
| 01:10:15 | melwitt | sean-k-mooney: note the ~ (not) squiggle | |
| 01:11:47 | sean-k-mooney | ya looking at the doc sting i think its correct after all | |
| 01:12:16 | melwitt | phew good | |
| 01:12:49 | sean-k-mooney | so it selecting the aggrreate if the aggreate metadta key is not in the set passed in | |
| 01:16:34 | openstackgerrit | ZHOU YAO proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/621646 | |
| 01:32:32 | sean-k-mooney | ok im goign to log off for the night o/ | |
| 01:43:44 | sean-k-mooney | oh infra change how logs are renderd in the gate. | |
| 01:44:12 | sean-k-mooney | i dont think i like it just becasue its different the it has been for 6+ years | |
| 01:47:51 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix collapse of 'host_status' description https://review.opendev.org/676301 | |
| 02:23:53 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access https://review.opendev.org/631242 | |
| 02:23:53 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Refactor some methods for reuse by Cyborg-related code. https://review.opendev.org/673734 | |
| 02:23:54 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243 | |
| 02:23:54 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | |
| 02:23:55 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Get resolved Cyborg ARQs and add PCI BDFs to VM's domain XML. https://review.opendev.org/631245 | |
| 02:23:55 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: WIP: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735 | |
| 02:44:00 | openstackgerrit | Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476 | |
| 03:06:03 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Make evacuation respects anti-affinity rule https://review.opendev.org/649963 | |
| 03:06:52 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously https://review.opendev.org/651969 | |
| 03:08:59 | boxiang | hi gibi , I have finished the regression functional test for this https://review.opendev.org/649963 | |
| 04:05:31 | openstackgerrit | Merged openstack/nova master: Execute TargetDBSetupTask https://review.opendev.org/633853 | |
| 05:49:21 | openstackgerrit | ZHOU YAO proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/621646 | |
| 06:22:31 | openstackgerrit | Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476 | |
| 07:27:11 | openstackgerrit | Merged openstack/nova master: api-ref: Fix collapse of 'host_status' description https://review.opendev.org/676301 | |
| 09:55:39 | openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API https://review.opendev.org/612626 | |
| 10:05:57 | ut2k3 | Hi guys, I have unfortunately two Volumes stock in "detaching" - Is there any chance I can force detach these from the instances? `nova volume-detach...` as well reseting once their state with `cinder reset-state...` did not help | |
| 10:56:11 | openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient master: Microversion 2.75 - Multiple API cleanup changes https://review.opendev.org/676275 | |
| 10:58:32 | stephenfin | efried: I'm kind of stuck on the reshape for VCPU -> PCPU. I think what we're doing at the moment is wrong, and the tests are hiding that fact https://review.opendev.org/#/c/674895/7/nova/virt/libvirt/driver.py@6891 | |
| 10:59:39 | openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient master: Microversion 2.75 - Multiple API cleanup changes https://review.opendev.org/676275 | |
| 11:00:33 | stephenfin | efried: What I think we need to do is a multi-step process, (a) figure out if any instance on the host is using pinned instances by looking at the (host) NUMATopology.pinned_cpus attribute, if so (b) figure out if there are any PCPU allocations against the host, if not (c) figure out how many VCPUs to migrate to PCPUs and do that | |
| 11:01:52 | stephenfin | efried: But I can't do (b) easily since the pattern we have for reshaping is to only provide allocations to 'update_provider_tree' if there's a 'ReshapeNeeded' exception raised, so I'm stuck | |
| 11:04:37 | openstackgerrit | YAMAMOTO Takashi proposed openstack/nova master: WIP: midonet doesn't have plug-time vif events https://review.opendev.org/676388 | |
| 11:23:17 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Testing tls with ipv6 also https://review.opendev.org/676391 | |
| 11:48:20 | openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient master: Microversion 2.75 - Multiple API cleanup changes https://review.opendev.org/676275 | |
| 11:58:34 | stephenfin | alex_xu, gibi: More than happy to let you handle the update :) | |
| 11:58:58 | stephenfin | I shall sit at the back of the room and judge mercilessly :P | |
| 11:59:56 | gibi | stephenfin: I will do the placement project update with tetsuro so you can have the nova project update :) | |
| 12:01:10 | alex_xu | the world depends on three of us~ | |
| 12:04:28 | gibi | will be fun :) | |
| 12:04:41 | alex_xu | hah | |
| 12:07:33 | sean-k-mooney | stephenfin: your not allowed to anounce that we are killing nova and truning it into a sig :P | |
| 12:09:03 | alex_xu | haha | |
| 12:09:17 | alex_xu | I'm thinking how a sig works | |
| 12:10:02 | sean-k-mooney | alex_xu: they are allowed to have repos but have no ptl to heard the cats in a common direction | |
| 12:11:16 | alex_xu | that is fun | |
| 12:12:45 | sean-k-mooney | hehe yep which is why stephenfin's first act as docs ptl was to eliminate his postion so he will be the final docs ptl | |
| 12:14:28 | alex_xu | haha :) | |
| 12:42:40 | alemgeta | hello please i'm doing my msc thesis on openstack nova, what exactly failure detection algorithm used in openstack ,and place of the code, its my appreciation | |
| 12:43:49 | alemgeta | hello please i'm doing my msc thesis on openstack nova, what exactly failure detection algorithm used in openstack ,and place of the code, its my appreciation | |
| 12:55:39 | tssurya | mriedem, dansmith: are you both around ? | |
| 12:55:55 | dansmith | I am but I'm about to jump on a call in 4 minutes | |
| 12:56:20 | tssurya | what did we finally decide on ? tweaking the reno and adding the fact that config on ironic side should be diasbled ? or pushing the task state update to driver level | |
| 12:57:03 | dansmith | not sure we decided anything specific, but I'd prefer to move the task state setting out of the api | |
| 12:57:23 | tssurya | ok let me try to do that then | |
| 12:57:36 | dansmith | mriedem isn't around right now | |
| 12:58:09 | tssurya | ok I'll push the rest of his suggestions and we can come back to that when he comes online | |
| 12:58:45 | alemgeta | hello someone i have little question about openstack nova | |
| 12:59:09 | alemgeta | please someone | |
| 12:59:22 | alemgeta | about failer detection | |
| 12:59:40 | alemgeta | doing msc thesis | |
| 13:02:07 | luyao | dansmith: when will you back?😀 | |
| 13:02:11 | sean-k-mooney | nova does not perfrom failure detection of any of the workloads that are deploy with it if that is what you were wondering. it obviously validates that the action you asked it to perfom succeeded but it does not monitor the apllication lifetime | |
| 13:03:42 | sean-k-mooney | dansmith: speak his name an he shall appear | |
| 13:04:33 | sean-k-mooney | mriedem: tssurya wanted to know if you were around ~10 minuts ago | |
| 13:04:37 | dansmith | in an hour | |
| 13:05:37 | luyao | dansmith: got it! See you in an hour. :) | |
| 13:06:57 | mriedem | well here i am | |
| 13:08:55 | sean-k-mooney | mriedem: ill adress your vPMU feed back later today. thanks for reviewing it before you dropped off yesterday. | |
| 13:10:34 | mriedem | yw | |
| 13:23:08 | mriedem | sean-k-mooney: more lxc failures https://logs.opendev.org/24/676024/5/experimental/nova-lxc/9c06394/controller/logs/screen-n-cpu.txt.gz#_Aug_13_23_16_20_191786 | |
| 13:26:20 | mriedem | https://github.com/lxc/lxc/issues/1057 sounds similar | |
| 13:32:26 | sean-k-mooney | fun. debian does not symlink /sbin to /usr/sbin so perhaps its in /usr/sbin/init | |
| 13:43:21 | efried | stephenfin: That's an interesting one (your reshape chicken/egg) | |
| 13:50:39 | sean-k-mooney | stephenfin: by the way regardign the reshap you code needs to be able to handel a reshap on non upgrade cases as well. | |
| 13:50:55 | sean-k-mooney | e.g. i need to handel the reshap when then new config options are defiend | |
| 13:51:33 | sean-k-mooney | so you could deploy with train and have no config options set. then set them after and restart the compute agent whic would trigger teh reshape | |