Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-09-10
10:05:17 openstackgerrit Balazs Gibizer proposed openstack/nova master: Cleanup unused code https://review.opendev.org/750947
10:08:38 admin0 hi all .. is there a way to remove a entry from the host list ..like that host is removed and will not be added back
10:08:41 openstackgerrit wu.shiming proposed openstack/os-vif master: Remove Babel requirement https://review.opendev.org/750948
10:08:53 admin0 host as in hypervisor
10:09:34 admin0 google search lists direct database commands
10:39:51 lyarwood admin0: https://docs.openstack.org/python-openstackclient/latest/cli/command-objects/compute-service.html
10:47:05 lyarwood stephenfin: is the ordering wrong between https://review.opendev.org/#/c/749303/ and https://review.opendev.org/#/c/749304/1
10:47:22 lyarwood stephenfin: actually it doesn't matter
10:47:30 stephenfin I don't think so
10:47:40 stephenfin tooling is no use without the driver. Drive doesn't depend on tooling
10:47:47 stephenfin The driver
10:49:26 lyarwood stephenfin: oh sorry I thought the driver was using that
10:49:32 lyarwood stephenfin: so it's just stand alone tooling?
10:49:46 stephenfin afaict yes
10:49:49 lyarwood ah my bad
10:54:27 lyarwood stephenfin: remind me again, what were we waiting on for https://review.opendev.org/#/c/749308/ to start passing?
10:59:46 stephenfin lyarwood:
10:59:46 stephenfin <admin0> google search lists direct database commands
10:59:49 stephenfin whoops
10:59:57 stephenfin lyarwood: https://review.opendev.org/#/c/749527/
11:01:07 stephenfin Well that's needed for the next change. It looks like https://review.opendev.org/#/c/749308/ removes an exception that's still required until the following patch.
11:01:25 stephenfin I'll reshuffle that series later today to put the patches that break tempest to the end, if possible
11:02:22 lyarwood stephenfin: ack yeah sorry I didn't look into the failure, I just recalled you saying something about tests being borked after a certain point and assumed I had found it ^_^
11:04:15 lyarwood stephenfin: https://review.opendev.org/#/q/topic:sriov-functional-tests was the other series you wanted reviews on right?
11:04:39 admin0 @stephenfin, thanks .. it worked
11:04:59 admin0 @lyarwood, that worked :)
11:05:02 admin0 thanks guys
11:05:34 lyarwood np
11:17:17 sean-k-mooney wait xenapi is the reason os-agents is a thing. that always confused me since all the agents are listed under os-services
11:18:18 sean-k-mooney at least when using libvirt
11:29:27 lyarwood stephenfin: https://review.opendev.org/#/c/746946/ - small nit here, shouldn't you hit the resize API in these tests even if the eventual codepaths are the same at the moment
11:43:56 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: compute: refactor volume bdm rollback error handling https://review.opendev.org/749225
11:43:59 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Add regression tests for bug #1889108 https://review.opendev.org/749226
11:43:59 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: compute: Don't delete the original attachment during pre LM rollback https://review.opendev.org/749227
11:44:00 openstack bug 1889108 in OpenStack Compute (nova) rocky "failures during driver.pre_live_migration remove source attachments during rollback" [Undecided,In progress] https://launchpad.net/bugs/1889108 - Assigned to Lee Yarwood (lyarwood)
11:54:03 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: libvirt: Provide VIR_MIGRATE_PARAM_PERSIST_XML during live migration https://review.opendev.org/747976
12:05:51 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: libvirt: Provide VIR_MIGRATE_PARAM_PERSIST_XML during live migration https://review.opendev.org/747976
12:19:12 openstackgerrit Merged openstack/nova master: virt: Highlight soon-to-be-unused driver APIs https://review.opendev.org/749300
12:19:20 openstackgerrit Merged openstack/nova master: hacking: Stop special casing 'plugins.xenserver' https://review.opendev.org/749301
12:19:20 stephenfin lyarwood: ah, the name of the test is wrong. It should be test_migrate_server
12:19:31 openstackgerrit Merged openstack/nova master: docs: Remove references to XenAPI driver https://review.opendev.org/749302
12:19:40 openstackgerrit Merged openstack/nova master: tools: Remove xenserver tooling https://review.opendev.org/749303
12:19:48 openstackgerrit Merged openstack/nova master: doc: Fix note directives https://review.opendev.org/750094
12:19:58 openstackgerrit Merged openstack/nova master: functional: Provide default 'host_info' for '_get_connection' https://review.opendev.org/746942
12:23:41 sean-k-mooney stephenfin: by the way sphinx support building docs in parralel but we dont use it currenlty because some of the sphix extenons we have dont declare supprot for that
12:23:58 sean-k-mooney stephenfin: i think they are ones we/you wrote
12:24:33 stephenfin yup, smcginnis added support for parallel reading/writing to a couple of the oslo.* Sphinx extensions but I don't think we've done any of the in-tree ones
12:24:35 sean-k-mooney parallel builds are much faster think its worth enabling it and fix/declareing support in the extensions
12:25:04 sean-k-mooney stephenfin: ya some of the tox envs can just run as is
12:25:24 sean-k-mooney it jsut depens on the extenions used in the tox env
12:25:47 sean-k-mooney i had a patch locally to just add -j auto to all them but it failed so i abandoned it
12:26:19 sean-k-mooney but i noticed gibi mentionsed slow pdf doc in a recheck so i was wondering if we wantted to give it a try
12:27:26 gibi I'm hoping that timeout was a one time thing
12:27:36 sean-k-mooney ya i know
12:27:52 sean-k-mooney but i was building doc for gmann change a few days ago
12:28:02 sean-k-mooney and i noticed how slow the release note where
12:28:13 sean-k-mooney so i tried adding -j auto to it and it was way faster
12:28:53 sean-k-mooney then i was sad when addign it to all the docs env failed because some had extensions that did not declare support
12:30:24 smcginnis Some of the local tests I did really sped things up running parallel.
12:31:08 smcginnis Probably not as much in gate since there are less available cores, but still faster.
12:31:35 sean-k-mooney i use a dev vm that emulates the gate with 8 threads
12:31:41 sean-k-mooney and it was noticable
12:32:37 sean-k-mooney i say emulates because its got 2 numa node and 2 cpu socket with 2 core and 2 threads per core resulting in 8 thread total
12:32:53 sean-k-mooney so its not exactly the same but still it was faster
12:36:49 stephenfin weird, looks like the mirrors are not getting updated at the moment
12:37:11 stephenfin those last few merged patches aren't appearing
12:52:40 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add 'start_compute' helper https://review.opendev.org/746943
12:52:41 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Use tempdir for CONF.instances_path https://review.opendev.org/746944
12:52:41 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Clean up PCI tests https://review.opendev.org/746945
12:52:41 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add cold migration tests for PCI servers https://review.opendev.org/746946
12:52:42 openstackgerrit Stephen Finucane proposed openstack/nova master: conductor: Don't use setattr https://review.opendev.org/746947
12:52:42 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add test for SR-IOV neutron ports https://review.opendev.org/746948
12:52:43 openstackgerrit Stephen Finucane proposed openstack/nova master: fixtures: Handle binding of first port https://review.opendev.org/746949
12:52:43 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add live migration tests for PCI, SR-IOV servers https://review.opendev.org/746950
12:52:44 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Expand SR-IOV live migration tests with NUMA https://review.opendev.org/749360
13:08:53 openstackgerrit Stephen Finucane proposed openstack/nova master: xenapi: Remove driver and tests https://review.opendev.org/749304
13:08:54 openstackgerrit Stephen Finucane proposed openstack/nova master: virt: Remove 'is_xenapi' helper https://review.opendev.org/749305
13:08:54 openstackgerrit Stephen Finucane proposed openstack/nova master: xenapi: Remove Xen-only code paths https://review.opendev.org/749306
13:08:55 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Remove 'XenapiLiveMigrateData' https://review.opendev.org/749307
13:08:55 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Final cleanup https://review.opendev.org/749308
13:08:56 openstackgerrit Stephen Finucane proposed openstack/nova master: virt: Remove 'get_console_pool_info' API https://review.opendev.org/749311
13:08:56 openstackgerrit Stephen Finucane proposed openstack/nova master: virt: Remove 'post_interrupted_snapshot_cleanup' API https://review.opendev.org/749313
13:08:57 openstackgerrit Stephen Finucane proposed openstack/nova master: virt: Remove 'set_bootable' API https://review.opendev.org/749314
13:08:57 openstackgerrit Stephen Finucane proposed openstack/nova master: virt: Remove various aggregate APIs https://review.opendev.org/749317
13:08:58 openstackgerrit Stephen Finucane proposed openstack/nova master: virt: Remove 'get_per_instance_usage' API https://review.opendev.org/749318
13:08:58 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Remove 'os-agents' API https://review.opendev.org/749309
13:08:59 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Remove Agent, AgentList https://review.opendev.org/749310
13:08:59 openstackgerrit Stephen Finucane proposed openstack/nova master: virt: Remove 'get_all_bw_counters' API https://review.opendev.org/749312
13:09:00 openstackgerrit Stephen Finucane proposed openstack/nova master: virt: Remove 'reset_network' API https://review.opendev.org/749315
13:09:00 openstackgerrit Stephen Finucane proposed openstack/nova master: virt: Remove 'change_instance_metadata' API https://review.opendev.org/749316
13:14:39 lyarwood stephenfin: this is not helping my +ratio ^_^
13:14:44 lyarwood stephenfin: what happened there?
13:15:20 stephenfin lyarwood: https://review.opendev.org/#/c/749304/2/nova/exception.py@1293 :(
13:15:39 stephenfin it should be trivially reapprovable O:)
13:16:09 stephenfin and for the SR-IOV series, see last comment from me on https://review.opendev.org/#/c/746943/
13:16:48 lyarwood ACK I'll let CI run on both and reapprove once we are sure it's passing
13:17:17 stephenfin cool
13:17:50 stephenfin addressed your comment on https://review.opendev.org/#/c/746946/ too while I was respinning
13:19:35 lyarwood yup thanks
13:23:38 stephenfin gibi, lyarwood: One of you want to hit this vTPM doc change real quick? https://review.opendev.org/#/c/750675/

Earlier   Later