Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-08-08
15:45:12 mriedem edleafe: both are on the path
15:45:16 mriedem server_id and port_id
15:45:49 mriedem 404 cool for all things on the path then?
15:45:56 melwitt mriedem: that's a blast from the past. will have to reload a lot of context when I go back to review
15:46:42 edleafe mriedem: is this the GET or DELETE call?
15:46:46 mriedem edleafe: anyway, it's essentially dead code anyway
15:46:48 mriedem DELETE
15:47:07 mriedem https://docs.openstack.org/api-ref/compute/#detach-interface
15:47:44 edleafe The call doesn't delete the server. The resource it is concerned with is the port. If that resource doesn't exist, then 404 is correct
15:47:46 aspiers efried: I mean, why is the *total* inventory so damn low, not the usage
15:47:56 aspiers I guess that's in n-cpu
15:48:04 efried yes
15:48:07 mriedem edleafe: ack
15:48:11 edleafe Of course, if you subscribe to the idempotent DELETE school of thought, 204 would be correct
15:48:20 efried aspiers: I assume libvirt driver?
15:48:21 mriedem like i said, it's dead code at this point unless we do a proxy neutronv2.api.show_port call on that port_id
15:48:29 openstackgerrit Shilpa Devharakar proposed openstack/nova-specs master: Update spec: filtering of alloc candidates by forbidden aggregates https://review.opendev.org/675384
15:49:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: allow getting resource request of every bound ports of an instance https://review.opendev.org/655110
15:49:14 aspiers efried: virsh shows no pools
15:49:31 efried I don't know what that means
15:49:37 aspiers virsh pool-list
15:49:38 efried look at _get_local_gb_info
15:49:45 aspiers ok thanks
15:49:51 efried what's your [libvirt]images_type?
15:50:26 mriedem artom: you know, could just move this validation to the api https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6373
15:50:34 sean-k-mooney aspiers: i dont think openstack uses libvirt storage pools
15:50:42 aspiers sean-k-mooney: yeah that was a guess
15:50:43 mriedem compute is already checking the info cache before doing anything with the port at the neutron level
15:51:29 sean-k-mooney aspiers: mybe we might for lvm image backend? but im pretty sure tis not used for qcow/raw/flat iamge type
15:51:59 aspiers libvirt.images_type = default
15:52:03 efried aspiers: assuming your libvirt.images_type is unset or anything other than lvm/rbd, it'll be looking at the free space on your instances_path
15:52:12 aspiers that makes more sense
15:54:24 efried aspiers: ...which looks like it'll be by default whatever file system your nova installation is
15:54:57 efried if your nova is in /foo/bar/nova, your instances_path will by default be /foo/bar/instances, I think.
15:55:13 efried so
15:55:23 efried df -h /foo/bar ought to tell you how much space it thinks you have.
15:55:41 openstackgerrit Balazs Gibizer proposed openstack/nova master: Pass network API to the conducor's MigrationTask https://review.opendev.org/655111
15:55:54 aspiers yeah it's basic sysadmin / lvm stuff now
15:56:03 aspiers I'll take it from here. thanks for pointers :)
15:56:07 efried good luck
15:57:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix rebuild of baremetal instance when vm_state is ERROR https://review.opendev.org/523559
15:57:19 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: run test_rebuild_server_in_error_state in ironic CI job https://review.opendev.org/675391
15:58:51 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: run test_rebuild_server_in_error_state in ironic CI job https://review.opendev.org/675391
16:15:14 mriedem gibi: efried: seeing another notification sample race here https://logs.opendev.org/36/614036/46/check/nova-tox-functional/b505ffb/testr_results.html.gz
16:16:04 mriedem idk if the db stuff in there is just noise
16:17:19 mriedem btw, unrelated to that, i noticed we have a novnc test that fails a lot in multinode jobs http://status.openstack.org/elastic-recheck/#1669468
16:19:48 openstackgerrit Merged openstack/nova stable/rocky: Avoid logging traceback when detach device not found https://review.opendev.org/674068
16:19:56 openstackgerrit Merged openstack/nova stable/rocky: Add functional regression test for bug 1837955 https://review.opendev.org/673536
16:19:56 openstack bug 1837955 in OpenStack Compute (nova) rocky "MaxRetriesExceeded sometime fails with messaging exception" [Medium,In progress] https://launchpad.net/bugs/1837955 - Assigned to Matt Riedemann (mriedem)
16:22:11 openstackgerrit Merged openstack/nova stable/rocky: Cleanup when hitting MaxRetriesExceeded from no host_available https://review.opendev.org/673553
16:25:43 artom mriedem, ack
16:26:01 artom mriedem, although. Since we should be raising a 400 in the port not found case, technically it's a new microversion
16:26:27 artom So, being the lazy schmuck that I am, do I want to bother?
16:26:44 mriedem artom: you missed edleafe saying it should be a 404
16:27:09 artom Ah, so I did.
16:29:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Dump versioned notifications when len assertions fail https://review.opendev.org/675404
16:29:47 mriedem ^ can help with debugging racy versioned notification functional test failures ^
16:33:42 mriedem this is very weird https://logs.opendev.org/32/634832/43/check/nova-tox-functional-py36/d4f3be5/testr_results.html.gz
16:33:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Dump versioned notifications when len assertions fail https://review.opendev.org/675404
16:33:55 mriedem the test is disabling a service and asserting the notifications,
16:34:17 mriedem but in the err output there is a failing resize happening b/c of not using the neuron fixture
16:34:25 mriedem but the test isn't doing anything with resize
16:39:58 mriedem http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22ERROR%20%5Bnova.network.neutronv2.api%5D%20The%20%5Bneutron%5D%20section%20of%20your%20nova%20configuration%20file%20must%20be%20configured%20for%20authentication%20with%20the%20networking%20service%20endpoint.%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d
16:40:10 mriedem looks like it started around 8/5
16:40:12 mriedem so i wonder if it's this https://github.com/openstack/nova/commit/8db712fe040b15f2b8bc5538338658d3aac246e3
16:47:41 openstackgerrit Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
16:47:42 openstackgerrit Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420
16:47:42 openstackgerrit Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334
16:47:43 openstackgerrit Adam Spiers proposed openstack/nova master: Add and to config.py https://review.opendev.org/636318
16:47:43 openstackgerrit Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565
16:47:44 openstackgerrit Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616
16:48:15 mriedem https://bugs.launchpad.net/nova/+bug/1839515
16:48:15 openstack Launchpad bug 1839515 in OpenStack Compute (nova) "Weird functional test failures hitting neutron API in unrelated resize flows since 8/5" [Undecided,New]
16:48:28 mriedem gibi: ^ your recent functional compute service restart test might have a similar issue
16:48:36 mriedem https://review.opendev.org/#/c/667913/ i mean
16:52:12 openstackgerrit Adam Spiers proposed openstack/nova master: Allow assertXmlEqual() to pass options to matchers.XMLMatches https://review.opendev.org/674628
16:52:13 openstackgerrit Adam Spiers proposed openstack/nova master: Fix libvirt driver tests to use LibvirtConfigCapsGuest instances https://review.opendev.org/674629
16:52:13 openstackgerrit Adam Spiers proposed openstack/nova master: Track libvirt host/domain capabilities for multiple machine types https://review.opendev.org/673151
16:52:14 openstackgerrit Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
16:52:14 openstackgerrit Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420
16:52:14 openstackgerrit Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334
16:52:15 openstackgerrit Adam Spiers proposed openstack/nova master: Add and to config.py https://review.opendev.org/636318
16:52:16 openstackgerrit Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565
16:52:16 openstackgerrit Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616
17:02:08 openstackgerrit Merged openstack/nova stable/rocky: Revert "[libvirt] Filter hypervisor_type by virt_type" https://review.opendev.org/672747
17:02:20 openstackgerrit Merged openstack/nova stable/rocky: Drop source node allocations if finish_resize fails https://review.opendev.org/669393
17:02:29 openstackgerrit Merged openstack/nova stable/rocky: Add functional recreate test for bug 1764556 https://review.opendev.org/673816
17:02:29 openstack bug 1764556 in OpenStack Compute (nova) rocky ""nova list" fails with exception.ServiceNotFound if service is deleted and has no UUID" [Medium,In progress] https://launchpad.net/bugs/1764556 - Assigned to Matt Riedemann (mriedem)
17:02:35 openstackgerrit Merged openstack/nova stable/rocky: Add functional regression test for bug 1778305 https://review.opendev.org/673818
17:02:35 openstack bug 1778305 in OpenStack Compute (nova) rocky "Nova may erronously look up service version of a deleted service, when hostname have been reused" [Medium,In progress] https://launchpad.net/bugs/1778305 - Assigned to Matt Riedemann (mriedem)
17:03:08 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Don't generate service UUID for deleted services https://review.opendev.org/673821
17:12:30 sean-k-mooney efried: mriedem is tehre a target milestone for the U release yet?
17:12:58 efried sean-k-mooney: what do you mean?
17:13:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Skip test_migrate_disk_and_power_off_crash_finish_revert_migration https://review.opendev.org/675417
17:13:12 mriedem gibi: efried: ^
17:13:24 mriedem skip that functional test for now since it's leaking into other tests on the same worker and failing
17:13:42 mriedem sean-k-mooney: https://releases.openstack.org/
17:13:51 mriedem sean-k-mooney: you mean release date?
17:13:57 mriedem that's a question for ttx probably
17:13:59 mriedem or smcginnis

Earlier   Later