| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-05-11 | |||
| 15:50:30 | dansmith | similar to the cyborg case of starting the programming at bind time from the conductor in parallel to the rest of the instance bringup | |
| 15:50:36 | gibi | it would have a side effect to cache image to a compute that otherwise will not be selected | |
| 15:51:18 | gibi | hm or not | |
| 15:51:31 | dansmith | in the tight case you mean right? We'd start caching an image on a host that the scheduler was going to exclude anyway, which is true | |
| 15:52:11 | gibi | yeah for the thight case when the image would fit into the cache but the instance root disk would not any more | |
| 15:52:16 | dansmith | yep, for sure | |
| 15:52:24 | gibi | but that really tight | |
| 15:53:54 | gibi | what if for thight cases we allow disabling the cache entirely? it is thight so no space for cache | |
| 15:54:30 | dansmith | doing that would require a substantial resdesign of the whole image backend for libvirt I think | |
| 15:55:03 | gibi | is it because we assume that there is a backing file for the root fs image which happen to be the cached image? | |
| 15:56:41 | dansmith | I'm actually not sure what happens if you configure qcow2 and "flatten_images" actually, I'd have to look | |
| 15:56:50 | dansmith | that might have the same effect, I'm not sure | |
| 15:59:21 | gibi | do you mean force_raw_images conf option? | |
| 16:00:26 | gibi | or use_cow_images = False? or a linear combination of the two :) | |
| 16:00:44 | markguz_ | Hi nova folks. i have an instance that failed that i migrated (not live) to another compute host, but upon restarting it errors with | |
| 16:00:45 | markguz_ | Unsupported VIF type binding_failed convert '_nova_to_osvif_vif_binding_failed' | |
| 16:00:58 | dansmith | gibi: yeah there's some combination that results in full flattening, but I'm not sure what they are | |
| 16:01:10 | dansmith | gibi: not sure if tat actually results in the cache image going away, or getting copied or what | |
| 16:01:24 | dansmith | actually, as to be the latter I think since we have to expand the size of it | |
| 16:01:26 | markguz_ | i've tried the various solutions found via google but non work | |
| 16:01:46 | dansmith | so you probably end up with 2x the space initially at least, and then you'd need to immediately purge the original or something | |
| 16:02:22 | gibi | dansmith: ack | |
| 16:05:47 | gibi | dansmith: hm even when the flat backend copies the image to raw it does update the cache to keep the base image https://github.com/openstack/nova/blob/d6450879c7f7dd19366b6f002301fbbf87918026/nova/virt/libvirt/imagebackend.py#L585 | |
| 16:08:05 | gibi | dansmith: anyhow thanks for your thoughts I have to drop today soon so I will add some summary of this discussion to the bug. | |
| 16:10:06 | dansmith | gibi: yeah, that's what I was thinking above when I said "but then you'd need to immediately purge" | |
| 16:10:16 | dansmith | gibi: the cache is trying to be a cache | |
| 16:12:45 | gibi | yeah, I see now | |
| 16:23:26 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.reraise https://review.opendev.org/726898 | |
| 16:30:14 | gibi | lyarwood, dansmith: updated the bug 1878024 with what we talked about above. | |
| 16:30:14 | openstack | bug 1878024 in OpenStack Compute (nova) "disk usage of the nova image cache is not counted as used disk space" [Undecided,New] https://launchpad.net/bugs/1878024 | |
| 16:30:24 | dansmith | cool | |
| 16:30:57 | gibi | and now I go and bake some bread for dinner | |
| 16:31:04 | gibi | see you tomorrow | |
| 18:15:35 | openstackgerrit | Merged openstack/nova master: Support for --force flag for nova-manage placement heal_allocations command https://review.opendev.org/715395 | |
| 18:15:48 | openstackgerrit | Merged openstack/nova stable/queens: Include only required fields in ironic node cache https://review.opendev.org/724862 | |
| 18:16:01 | openstackgerrit | Merged openstack/nova stable/queens: Lowercase ironic driver hash ring and ignore case in cache https://review.opendev.org/723054 | |
| 18:16:10 | openstackgerrit | Merged openstack/nova stable/rocky: Add config option for neutron client retries https://review.opendev.org/722819 | |
| 18:16:27 | openstackgerrit | Merged openstack/nova master: Suppress remaining policy warnings in unit tests https://review.opendev.org/726272 | |
| 18:22:04 | markguz_ | anyone know how to get out of Unsupported VIF type binding_failed convert '_nova_to_osvif_vif_binding_failed' hell? | |
| 19:35:26 | openstackgerrit | Harshavardhan Metla proposed openstack/nova master: [Nova] Add reference to Placement installation guide https://review.opendev.org/726936 | |
| 19:42:27 | markguz_ | for anyone that's interested setting neutron.ml2_port_bindings.vif_type to "ovs" in the database fixed this for me | |
| 19:43:35 | sean-k-mooney | markguz_: if you get that error its because neutron failed to bind the port | |
| 19:43:47 | markguz_ | i know | |
| 19:43:50 | sean-k-mooney | which normally means there was an error on cthe compute node | |
| 19:44:10 | markguz_ | but once it happens it seems next to impossible to fix it via the normal methods. | |
| 19:44:46 | sean-k-mooney | markguz_: you fix it by setting the host filed to "" or "none" then back to the original hostname | |
| 19:45:09 | markguz_ | the instance will not boot due the "binding_failed" being written into the vif_type field | |
| 19:45:33 | markguz_ | sean-k-mooney: or by updating that field i just mentioned in the neutron db | |
| 19:46:02 | sean-k-mooney | yes but you can do "openstack --os-cloud=admin port set --host none baf2b165-797b-4305-bc6b-5b63250b890d" follow by " openstack --os-cloud=admin port set --host workstation baf2b165-797b-4305-bc6b-5b63250b890d" | |
| 19:46:09 | sean-k-mooney | to do it from the api without db hacking | |
| 19:46:36 | sean-k-mooney | it will actully cause port binding to happen properly recalualting the correct values | |
| 19:46:41 | markguz_ | sean-k-mooney: ok. thanks for that | |
| 19:47:01 | sean-k-mooney | so i did that yesterday becauses i was swaping form the iptables firewall dirver to the ovs one | |
| 19:48:18 | markguz_ | this happened to me when i did a non-live migrate of a shutdown instance to a new host | |
| 19:48:42 | sean-k-mooney | hum it should not happen in that case | |
| 19:49:10 | sean-k-mooney | something obviosly went wrong there should be an error in the neutron server log | |
| 19:49:46 | markguz_ | yeah. haven't had time to deep dive. Was focussed on getting the instance back online | |
| 19:50:11 | sean-k-mooney | ist the instance still in resize_verify or did this happen after that point | |
| 19:50:33 | sean-k-mooney | if you had not confirmed the migrate/resize then you could have reverted | |
| 19:51:02 | sean-k-mooney | if you had then ya db edit or unest and reset the host to rebind the port then hard reboot | |
| #openstack-nova - 2020-05-12 | |||
| 00:43:17 | openstackgerrit | xuyuanhao proposed openstack/nova master: Optimization the soft-delete logical https://review.opendev.org/724260 | |
| 01:25:32 | openstackgerrit | xuyuanhao proposed openstack/nova master: Optimization the soft-delete logical https://review.opendev.org/724260 | |
| 02:03:13 | openstackgerrit | Wenping Song proposed openstack/nova master: error may occur when filter scheduler with accelerator https://review.opendev.org/722651 | |
| 03:16:56 | openstackgerrit | xuyuanhao proposed openstack/nova master: Optimization the soft-delete logical https://review.opendev.org/724260 | |
| 04:29:41 | openstackgerrit | Wenping Song proposed openstack/nova master: error may occur when filter scheduler with accelerator https://review.opendev.org/722651 | |
| 05:44:08 | openstackgerrit | Harshavardhan Metla proposed openstack/nova master: [Nova] Add reference to Placement installation guide https://review.opendev.org/726936 | |
| 06:38:17 | gibi | good morning Nova | |
| 06:47:33 | ikla | how do I passthrough multiple pci devices in nova? | |
| 06:53:36 | gibi | ikla: https://docs.openstack.org/nova/pike/admin/pci-passthrough.html#configure-a-flavor-controller | |
| 07:00:54 | sean-k-mooney | ikla: you just create a flavor with multiple alia's referenced | |
| 07:09:56 | ikla | I have rocky | |
| 07:10:37 | jkulik | hi, is there something like sub-flavors somewhere on nova's roadmap? Having a flavor that maps to different specific flavors per hardware generation would be great. | |
| 07:16:01 | ikla | do I need to define the devices in the nova controller? | |
| 07:18:55 | gibi | ikla: the alias needs to be defined both on the controller and on the computes, the passthrough_whitelist needs to be defined on each compute | |
| 07:19:59 | gibi | jkulik: I don't know about such feature on the roadmap. I think you can implement that top of nova | |
| 07:20:27 | ikla | do I have multiple lines with passthrough_whitelist ? or can I do it all on one line? | |
| 07:21:51 | bauzas | good morning nov | |
| 07:21:57 | bauzas | err, nova even | |
| 07:22:13 | gibi | ikla: you can have a full json dict, see the config doc https://docs.openstack.org/nova/rocky/configuration/config.html#pci.passthrough_whitelist | |
| 07:22:26 | gibi | ikla: or even a list of such dicts | |
| 07:22:36 | gibi | bauzas: o/ | |
| 07:25:23 | ikla | same for alias? | |
| 07:26:12 | gibi | ikla: I think alias is a single dict, so you have to have multiple line of alias = {...} in your config to have multiple aliases | |
| 07:28:24 | ikla | thanks | |
| 07:28:33 | ikla | passthrough cannot have multiple lines? | |
| 07:29:09 | jkulik | gibi, by on top of nova, do you mean on the client? Even if we would control the client, which we don't necessarily, we would have to decide on the precise flavor before going into scheduling afaics. So if a user doesn't care about the hardware version, we would have to make a decision anyways instead of taking "what's free" automatically. | |
| 07:32:29 | ikla | nvrmind :) | |
| 07:34:48 | gibi | ikla: I think passthrough_whitelist also can be used multiple times | |
| 07:36:06 | ikla | does passthrough_whitelist work on rocky or does it need to be prefixed with pci_ ? | |
| 07:37:39 | gibi | ikla: in the [pci] section you can use passthrough_whitelist config | |
| 07:50:49 | openstackgerrit | Harshavardhan Metla proposed openstack/nova master: [Nova] Add reference to Placement installation guide https://review.opendev.org/726936 | |
| 07:59:03 | openstackgerrit | James Page proposed openstack/nova stable/queens: hardware: fix memory check usage for small/large pages https://review.opendev.org/726867 | |
| 07:59:03 | openstackgerrit | James Page proposed openstack/nova stable/queens: Fix overcommit for NUMA-based instances https://review.opendev.org/726868 | |
| 08:19:27 | openstackgerrit | sean mooney proposed openstack/nova-specs master: move implemented specs in ussuri https://review.opendev.org/721278 | |
| 08:19:55 | sean-k-mooney | gibi: sorry ment to do that last week ^ if you want any changes let me know and ill respin the patch | |
| 08:29:20 | huaqiang | stephenfin: There are ~20 patches in bp/use-pcpu-and-vcpu-in-one instance, for roles of py-checks, realtime policy, and fixes for hardware.py | |
| 08:29:37 | huaqiang | these are not depending on each other so much | |
| 08:29:51 | huaqiang | how about let me re-arrange the order | |
| 08:30:40 | huaqiang | and resolving some dependencies with slight change to make it possible to be reviewed parallelly | |
| 08:32:18 | openstackgerrit | Nalini Varshney proposed openstack/nova master: Add migration to make key field type VARBINARY in aggregate_metadata table, https://review.opendev.org/725522 | |