Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-12-04
15:58:38 openstackgerrit Alexandre arents proposed openstack/nova master: WIP libvirt: Make cross cell resize spawn from snapshot image https://review.opendev.org/c/openstack/nova/+/765561
16:00:14 aarents gibi: lyarwood I gave a try to fix https://bugs.launchpad.net/nova/+bug/1906428 ^^
16:00:14 openstack Launchpad bug 1906428 in OpenStack Compute (nova) "test_cold_migrate_unshelved_instance failing with cat: can't open '/mnt/timestamp': No such file or directory" [Critical,In progress]
16:00:42 aarents need more work on unit test but idea is here
16:09:48 artom aarents, sorry, I'm just imagining a cat pulling a wire and causing the test to fail
16:12:01 aarents :)
17:05:22 sean-k-mooney aarents: i feel like we fixed this before
17:25:14 dcapone2004 sean-k-mooney: I filed that blueprint, I don't know how I did, https://blueprints.launchpad.net/nova/+spec/add-support-for-hyperv-in-windows-instances-on-kvm
17:43:20 dcapone2004 I was also thinking of making an attempt to work on a patch.... can you point me in the correct direction as to where image properties are defined to add image properties?
18:04:24 melwitt dcapone2004: have fun :) https://github.com/openstack/nova/blob/master/nova/objects/image_meta.py#L151 also disclaimer: I don't know what any of this is about but that's where the image properties are
18:05:21 melwitt on the glance side they're documented here https://docs.openstack.org/glance/latest/admin/useful-image-properties.html
18:08:44 dcapone2004 thx
22:38:56 openstackgerrit Merged openstack/nova master: nova-multi-cell: Skip test_cold_migrate_unshelved_instance https://review.opendev.org/c/openstack/nova/+/765141
23:19:21 gmann finally merged and gate is back ^^
#openstack-nova - 2020-12-05
08:49:48 openstackgerrit Adit Sarfaty proposed openstack/nova master: Retry on vmware create_vm when it fails https://review.opendev.org/c/openstack/nova/+/764586
08:51:29 openstackgerrit Adit Sarfaty proposed openstack/nova master: Retry on vmware create_vm when it fails https://review.opendev.org/c/openstack/nova/+/764586
11:36:04 openstackgerrit Merged openstack/nova master: virt: Remove 'reset_network' API https://review.opendev.org/c/openstack/nova/+/749315
11:38:08 openstackgerrit Merged openstack/nova master: virt: Remove 'get_all_bw_counters' API https://review.opendev.org/c/openstack/nova/+/749312
13:45:44 openstackgerrit Takashi Natsume proposed openstack/nova master: Fix a deprecation warning of 'collections' https://review.opendev.org/c/openstack/nova/+/765646
14:11:20 openstackgerrit Merged openstack/nova master: pci: Add a '_filter_pools' helper function https://review.opendev.org/c/openstack/nova/+/764375
14:28:21 openstackgerrit Merged openstack/nova master: pci: Improve docs for '_filter_*' functions https://review.opendev.org/c/openstack/nova/+/764376
15:44:56 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/c/openstack/nova/+/728094
15:45:50 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/c/openstack/nova/+/728109
15:47:06 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/c/openstack/nova/+/728117
16:33:45 openstackgerrit Merged openstack/nova master: Remove unused bindir config option https://review.opendev.org/c/openstack/nova/+/760331
17:01:20 openstackgerrit Merged openstack/nova master: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/764246
17:01:55 openstackgerrit Merged openstack/nova master: Update supported transports for iscsi connector https://review.opendev.org/c/openstack/nova/+/524443
#openstack-nova - 2020-12-06
04:22:53 openstackgerrit Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List Servers APIs https://review.opendev.org/c/openstack/nova/+/764292
05:03:00 openstackgerrit Brin Zhang proposed openstack/nova master: Replace all_tenants with all_projects in List Server APIs https://review.opendev.org/c/openstack/nova/+/765311
05:41:01 openstackgerrit Merged openstack/nova master: Improve documentation of (unpin|pin)_cpus_with_siblings() https://review.opendev.org/c/openstack/nova/+/740327
05:51:43 openstackgerrit Merged openstack/nova master: pci: Add logging for filtering https://review.opendev.org/c/openstack/nova/+/764377
05:54:20 openstackgerrit Merged openstack/nova stable/victoria: zuul: Replace nova-live-migration with zuulv3 jobs https://review.opendev.org/c/openstack/nova/+/764432
06:33:48 openstackgerrit Brin Zhang proposed openstack/nova master: Replace all_tenants with all_projects in List Server APIs https://review.opendev.org/c/openstack/nova/+/765311
09:44:54 openstackgerrit Adit Sarfaty proposed openstack/nova master: Retry on vmware create_vm when it fails https://review.opendev.org/c/openstack/nova/+/764586
10:43:40 openstackgerrit Merged openstack/nova master: libvirt: check for AMD SEV only on x86-64 https://review.opendev.org/c/openstack/nova/+/714425
12:19:29 openstackgerrit Takashi Natsume proposed openstack/nova master: Add a description in the PTL guide https://review.opendev.org/c/openstack/nova/+/765682
12:29:58 openstackgerrit Merged openstack/nova stable/victoria: Use cell targeted context to query BDMs for metadata https://review.opendev.org/c/openstack/nova/+/761424
#openstack-nova - 2020-12-07
03:40:31 chengsheng Originally, the code I submitted passed zuul. Today I rechecked and found that the unit test reported a lot of failures. I checked the log and found that the unit test code was different from the patch I submitted. I am a newbie and want to know what is going on.
07:05:39 openstackgerrit MIKI Nobuhiro proposed openstack/nova master: libvirt: add IP address to libvirt guest metadata https://review.opendev.org/c/openstack/nova/+/750552
07:47:48 openstackgerrit Brin Zhang proposed openstack/nova master: Replace all_tenants with all_projects in List Server APIs https://review.opendev.org/c/openstack/nova/+/765311
07:53:27 openstackgerrit Brin Zhang proposed openstack/nova master: Replace tenants* with projects* of policies https://review.opendev.org/c/openstack/nova/+/765315
08:42:09 openstackgerrit Balazs Gibizer proposed openstack/nova master: Make remove allocation symmetric with add allocation https://review.opendev.org/c/openstack/nova/+/757110
08:43:37 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out port resource deallocation https://review.opendev.org/c/openstack/nova/+/757111
08:45:10 openstackgerrit Balazs Gibizer proposed openstack/nova master: Refactor update_pci_request_spec_with_allocated_interface_name https://review.opendev.org/c/openstack/nova/+/757614
08:46:50 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support interface attach with qos ports https://review.opendev.org/c/openstack/nova/+/756530
09:30:32 openstackgerrit Wenping Song proposed openstack/nova-specs master: libvirt supports composing cyborg owned vGPU accelerator into domain XML https://review.opendev.org/c/openstack/nova-specs/+/750116
09:35:32 kashyap chengsheng: Do you mean the unit test failure is unrelated to your patch? If so, sometimes the CI hits issues that are not your fault
09:35:32 lyarwood \o morning all
09:36:47 kashyap chengsheng: If you can post the failure here, others might recognize it. Alternatively, there might be intermittent failures; so you try to type "recheck" (without quotes) in a comment in your change, and see if that helps.
09:38:10 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/765769
09:38:47 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/765770
09:39:15 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/765771
09:40:07 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/765772
09:40:32 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/765773
09:41:07 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/765774
09:43:22 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: Add regression test for bug #1899649 https://review.opendev.org/c/openstack/nova/+/764611
09:43:22 openstack bug 1899649 in OpenStack Compute (nova) "Volume marked as available after a failure to build" [Undecided,In progress] https://launchpad.net/bugs/1899649 - Assigned to Lee Yarwood (lyarwood)
09:43:24 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: compute: Don't detach volumes when RescheduledException raised without retry https://review.opendev.org/c/openstack/nova/+/764612
09:50:19 openstackgerrit chengsheng proposed openstack/nova master: CPU selection with hypervisor consideration https://review.opendev.org/c/openstack/nova/+/762330
09:54:27 gibi aarents: I have a question in https://review.opendev.org/c/openstack/nova/+/765561
09:55:06 gibi lyarwood: welcome back o/
10:00:06 openstackgerrit Aleksey Ulyanov proposed openstack/nova master: Fix graceful shutdown timeout https://review.opendev.org/c/openstack/nova/+/764927
10:22:08 aarents Hi gibi lyarwood
10:22:13 aarents gibi: I just replied
10:22:17 gibi looking
10:26:13 gibi aarents: so resize depends on image_base_image_ref being present in metadata and some instances doesn't have it as the previous fix only fixed it for new instances
10:26:17 gibi Am I correct?
10:26:42 aarents gibi: correct
10:27:44 gibi then I think only the comment need a bit of rewording to state this
10:27:52 gibi as from the bug link it was not totally clear
10:28:02 gibi but your review answer cleared it
10:29:54 gibi aarents: replied in the review, thanks for the clarification
10:33:23 aarents gibi: oky
10:33:39 aarents lyarwood: thnsk I will address the nit
12:03:43 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: Update modernize-os-hypervisors-api spec, pt. 2 https://review.opendev.org/c/openstack/nova-specs/+/765796
12:03:44 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: Update modernize-os-hypervisors-api spec, pt. 3 https://review.opendev.org/c/openstack/nova-specs/+/765797
12:11:02 gibi fyi I see this happening quite a lot recently on the gate https://bugs.launchpad.net/tempest/+bug/1907084
12:11:02 openstack Launchpad bug 1907084 in tempest "test_attach_scsi_disk_with_config_drive intermittently fails at detaching volume" [Undecided,New]
12:11:40 gibi I think it is just a race condition in the test. So I will try to propose a re-query of the volume attachemnts in tempest
12:14:27 sean-k-mooney gibi: well that or its realted to the race lyarwood was working around with the retry interval
12:15:27 gibi sean-k-mooney: do you have a link for that?
12:16:56 sean-k-mooney https://review.opendev.org/c/openstack/nova/+/755799 was the workaround
12:17:10 sean-k-mooney the fix is to use libvirt events
12:18:17 gibi sean-k-mooney: ahh, could be related to the fact that now the race happens more than before as I guess we delayed the detach on the libvirt side
12:18:30 gibi making the bdm.destroy even more delayed
12:18:39 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Merge 'test_hypervisor_status' into 'test_hypervisors' https://review.opendev.org/c/openstack/nova/+/764039
12:18:39 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Drop statistics-style fields from os-hypervisors https://review.opendev.org/c/openstack/nova/+/764040
12:18:39 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: Change default policy for '/os-hypervisors' https://review.opendev.org/c/openstack/nova/+/765798
12:18:57 sean-k-mooney well its load related, so 10 seconds was just a guess based on what we were seeing
12:19:10 sean-k-mooney but a slighly slower node could still nit it
12:19:37 sean-k-mooney it may be something else but it sounds related
12:21:35 lyarwood gmann: ^ btw, smells like an issue with https://review.opendev.org/c/openstack/tempest/+/764407
12:22:02 lyarwood I don't think this is an issue with https://review.opendev.org/c/openstack/nova/+/755799 fwiw sean-k-mooney, at least at first glance
12:22:36 lyarwood gmann: ignore that ping, the assertion is in the test itself, not cleanup.
12:23:24 sean-k-mooney that test chagne merged 9 days ago, gibi are you still seeing it after that?
12:23:51 gibi sean-k-mooney: yepp it is actively happening
12:24:00 gibi logstash link is in the bugreport
12:24:29 gibi but I now 70% confident that the cinder volume becomes available _befor_ nova deletes the BDM
12:24:35 gibi so the tempest test is simply racing on this
12:24:43 lyarwood that shouldn't be possible

Earlier   Later