| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-08-28 | |||
| 16:55:28 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM: tracing the PCI allocation code path https://review.opendev.org/748454 | |
| 16:56:39 | openstackgerrit | sean mooney proposed openstack/os-vif master: deprecate ovs-vsctl driver and make native the default https://review.opendev.org/744816 | |
| 17:29:54 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: testing tempest copy-image test https://review.opendev.org/742547 | |
| 18:35:10 | openstackgerrit | Merged openstack/nova stable/ussuri: Removes the delta file once image is extracted https://review.opendev.org/748034 | |
| 21:30:41 | openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: DO NOT REVIEW: Test OVN devstack module https://review.opendev.org/748226 | |
| #openstack-nova - 2020-08-29 | |||
| 00:31:01 | openstack | bug 1888395 in OpenStack Compute (nova) "shared live migration of a vm with a vif is broken in train" [High,In progress] https://launchpad.net/bugs/1888395 - Assigned to sean mooney (sean-k-mooney) | |
| 00:31:01 | openstackgerrit | sean mooney proposed openstack/nova master: add functional regression test for bug #1888395 https://review.opendev.org/747454 | |
| 00:31:37 | openstackgerrit | sean mooney proposed openstack/nova master: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/742180 | |
| 01:41:03 | openstackgerrit | Brin Zhang proposed openstack/nova master: Cyborg evacuate support https://review.opendev.org/715326 | |
| 01:51:29 | openstackgerrit | Hao Sun proposed openstack/nova stable/stein: Add checks for volume status when rebuilding https://review.opendev.org/748559 | |
| 03:28:53 | openstackgerrit | melanie witt proposed openstack/nova master: Default user_id when not specified in check_num_instances_quota https://review.opendev.org/748550 | |
| 03:28:55 | openstack | bug 1893284 in OpenStack Compute (nova) "Per-user quotas not scoped correctly when creating a server" [Low,In progress] https://launchpad.net/bugs/1893284 - Assigned to melanie witt (melwitt) | |
| 03:28:55 | openstackgerrit | melanie witt proposed openstack/nova master: Add regression test for bug 1893284 https://review.opendev.org/748779 | |
| 06:16:35 | openstackgerrit | Brin Zhang proposed openstack/nova master: Cyborg evacuate support https://review.opendev.org/715326 | |
| 08:25:05 | openstackgerrit | Wenping Song proposed openstack/nova master: Reject resize operation for accelerator https://review.opendev.org/748560 | |
| 08:27:43 | openstackgerrit | Wenping Song proposed openstack/nova master: Reject resize operation for accelerator https://review.opendev.org/748560 | |
| 09:38:43 | openstackgerrit | Brin Zhang proposed openstack/nova master: Cyborg evacuate support https://review.opendev.org/715326 | |
| 10:56:27 | openstackgerrit | Elod Illes proposed openstack/nova stable/train: Add checks for volume status when rebuilding https://review.opendev.org/748558 | |
| #openstack-nova - 2020-08-30 | |||
| 12:45:02 | openstackgerrit | Merged openstack/nova master: Add a lock to prevent race during detach/attach of interface https://review.opendev.org/747957 | |
| 12:55:43 | openstackgerrit | Merged openstack/nova master: QEMU/KVM: accept vmxnet3 NIC https://review.opendev.org/738695 | |
| #openstack-nova - 2020-08-31 | |||
| 01:18:36 | openstackgerrit | Keigo Noha proposed openstack/nova stable/ussuri: Change default num_retries for glance to 3 https://review.opendev.org/748936 | |
| 02:20:01 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Coding style and test cases improvment https://review.opendev.org/748939 | |
| 02:22:22 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Coding style and test cases improvement https://review.opendev.org/748939 | |
| 07:21:43 | swp20 | gibi: hi gibi, morning. | |
| 07:23:34 | swp20 | I have a question: As we donnot delete allocations when evacuate, so this case `nova.tests.functional.wsgi.test_services.TestServicesAPI.test_evacuate_then_delete_compute_service` is not reasonable, right? | |
| 07:35:59 | gibi | swp20: the source host allocation is not deleted during evacuation. It is left for the source host to delete it when it is recovered. If the source host is never recovered but deleted instead then we might need to delete the source side of the allocation of successfull evacuations | |
| 07:40:08 | gibi | swp20: alternatively we can reject the service delete if there is allocations against the RP due to finished evacuations | |
| 07:41:34 | gibi | and ask the admin to clean that up manually before delete the service | |
| 07:41:44 | swp20 | so how can we process the exception when we delete rp failed? we cannot raise the exception. | |
| 07:42:19 | swp20 | please see the patch: https://review.opendev.org/#/c/748339/ | |
| 07:42:48 | bauzas | good morning | |
| 07:43:30 | gibi | swp20: I will check | |
| 07:43:34 | gibi | bauzas: good morning | |
| 07:43:59 | swp20 | gibi: thanks | |
| 07:54:38 | suryasingh | @gibi hi... Sorry to address directly. Do you know if graceful shutdown of nova-compute is supported from nova side (For ex: like SIGTERM send to nova-compute to stopped in the middle of instance boot operation) | |
| 07:59:44 | gibi | suryasingh: hi! good questions. I'm not sure. | |
| 08:01:45 | gibi | suryasingh: looking at the nova.service.Service impl, I don't see specific implementtin | |
| 08:02:22 | gibi | nova.service.Service.stop just stops the RPC service and then calls manager.clean_up | |
| 08:03:06 | gibi | we do cancel ongoing live migration though | |
| 08:03:16 | gibi | via nova.compute.manager.ComputeManager._cleanup_live_migrations_in_pool | |
| 08:03:27 | gibi | but I don't see any more gracefullness | |
| 08:05:51 | suryasingh | I see gibi, Thanks for response though. | |
| 08:09:25 | gibi | suryasingh: if you would need more gracefullness the I suggest to describe you use case in a mail on the mailing list or in a nova specification | |
| 08:11:56 | suryasingh | gibi: thanks for info. I will do once feel require to mail. | |
| 08:12:14 | bauzas | gibi: suryasingh: we just hook the SIGHUP signal | |
| 08:12:22 | bauzas | hook up* | |
| 08:12:38 | bauzas | we don't indeed have any other signal for a graceful restart | |
| 08:13:04 | bauzas | but operators just disable the service before they restart | |
| 08:13:17 | bauzas | so they don't need to wait | |
| 08:23:53 | bauzas | gibi: IIRC, oslo.service also waits gracefully | |
| 08:24:02 | brinzhang | gibi, stephenfin: Fixed stephenfin's comments inline in Cyborg evacuate patch, hope you can review again https://review.opendev.org/#/c/715326/ | |
| 08:24:28 | bauzas | https://docs.openstack.org/oslo.service/ocata/history.html#id17 | |
| 08:24:32 | bauzas | gibi: ^ | |
| 08:24:35 | bauzas | suryasingh: ^ | |
| 08:25:24 | openstack | Launchpad bug 1861401 in OpenStack Compute (nova) "Renaming instance brokes DNS integration" [Low,Opinion] | |
| 08:25:24 | frickler | bauzas: I have two question regarding https://bugs.launchpad.net/neutron/+bug/1861401: a) is "hostname is immutable" documented somewhere? nearest thing I found is https://bugs.launchpad.net/nova/+bug/1068154 | |
| 08:25:25 | openstack | Launchpad bug 1068154 in OpenStack Compute (nova) "Renaming Instance Name doesn't change hostname on a Rebuild" [Wishlist,Opinion] | |
| 08:25:39 | frickler | and b: is it possible to retrieve the original hostname via the API? | |
| 08:26:05 | bauzas | suryasingh: gibi: https://docs.openstack.org/oslo.service/latest/reference/service.html#oslo_service.service.Service.stop | |
| 08:26:58 | bauzas | frickler: looking | |
| 08:27:43 | suryasingh | bauzas: thanks for heads up, sorry i couldn't get much. How disabling benefits in graceful shutdown ? >> so they don't need to wait ? | |
| 08:28:11 | bauzas | suryasingh: by disabling the service, you avoid having instances be going to it | |
| 08:28:33 | suryasingh | bauzas: yes that's correct | |
| 08:28:53 | bauzas | suryasingh: so, then, when restarting the compute service, the instances are not restarted | |
| 08:29:40 | frickler | bauzas: ah, for b) I found OS-EXT-SRV-ATTR:hostname, though it's admin-only by default, which kind of restricts it's usefulness | |
| 08:30:07 | bauzas | frickler: corrrect, but you can change the policy | |
| 08:30:37 | bauzas | frickler: I mean, it's a cloud API, right ? :) | |
| 08:30:53 | bauzas | so that's why it's defaulted to No | |
| 08:31:09 | suryasingh | bauzas: what about on going instance booting process, suppose i disabled the nova-compute even before nova-compute gets net-id, or requested volume or object from other openstack services (neutron, cinder, swift) | |
| 08:31:58 | bauzas | frickler: for the first question, it's because we have persisted objects that use the name for knowing which compute they are related | |
| 08:32:20 | bauzas | so, changing it would mean that you should also have to change the objects too | |
| 08:33:09 | bauzas | frickler: it's possible I think, and maybe some operators have some tools for it | |
| 08:33:19 | frickler | bauzas: I don't question why it is immutable, I'd just like to see that clearly documented, so I can point people to it | |
| 08:34:01 | frickler | bauzas: for the API I do question why the default is admin-only, though, because that data seems to be readily available via metadata | |
| 08:34:20 | bauzas | frickler: ahah good point | |
| 08:34:40 | gibi | bauzas: would that service stop wait for every eventlet thread to finish too? that would mean a sort-of gracefull shutdown then | |
| 08:34:50 | bauzas | frickler: you get the hostname thru the metadata API, really ? | |
| 08:35:09 | gibi | bauzas: I guess suryasingh or me could try in devstack to see what happens with on the fly spawns | |
| 08:35:17 | bauzas | gibi: I'm not an oslo.service expert, but I'd think it does | |
| 08:35:42 | bauzas | gibi: that's what I'd expect at least | |
| 08:35:55 | bauzas | (the service wait) | |
| 08:36:22 | frickler | bauzas: http://169.254.169.254/latest/meta-data/hostname gives me the original hostname, not the changed (display_)name, yes | |
| 08:38:00 | bauzas | eeek | |
| 08:38:01 | frickler | oh, wait, is that an even different hostname? | |
| 08:40:13 | bauzas | frickler: wait | |
| 08:40:28 | bauzas | frickler: hostname will give you the display_name of the instance VM I'd say | |
| 08:40:39 | bauzas | that's what I'd expect | |
| 08:41:18 | bauzas | at least it's what EC2 Metadata format will give you https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/ec2-instance-metadata.html | |
| 08:42:14 | bauzas | https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/instancedata-data-categories.html => | |
| 08:42:15 | bauzas | ||
| 08:42:15 | bauzas | hostname The private IPv4 DNS hostname of the instance. In cases where multiple network interfaces are present, this refers to the eth0 device (the device for which the device number is 0). | |
| 08:44:26 | frickler | bauzas: nope, those data seem immutable in my test, too (rocky). the openstack meta_data.json gives "hostname": "oldname.novalocal", "name": "newname" | |
| 08:45:20 | frickler | which is consistent with how neutron seems to handle dns at least | |
| 08:52:30 | bauzas | frickler: lemme rephrase, this hostname value matches the original instance name, not the compute service hostname | |
| 08:55:24 | openstackgerrit | Mamduh proposed openstack/os-vif master: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/746673 | |
| 09:06:02 | frickler | bauzas: not sure what you mean by "compute service hostname". the metadata matches the original instance name and OS-EXT-SRV-ATTR:hostname, not the new instance name | |
| 09:08:55 | openstackgerrit | Akhil Gudise proposed openstack/nova master: Moved all calls from _ENFORCER.authorize to a separate _authorize method https://review.opendev.org/739460 | |
| 09:08:56 | bauzas | frickler: okay, my bad, I see the confusion | |
| 09:09:07 | gibi | swp20: left comment in https://review.opendev.org/#/c/748339 | |
| 09:09:28 | bauzas | frickler: I thought you were asking whether the compute service hostame was immutable | |