Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-08-08
22:11:03 mriedem like, why do we even need it in the first place in other func tests is my point
22:11:31 mriedem thou shalt be smoten with review nit-pickery
22:11:34 melwitt we have definitely needed it before. I'm not sure what's going on here
22:11:46 melwitt re: ServicesCurrent
22:12:11 dansmith if you have no services you get back the current I think
22:12:18 dansmith we used to return zero,
22:12:23 melwitt the api service doesn't count as a service?
22:12:39 melwitt os_apicompute
22:13:01 mriedem idk, might have something to do with wsgi-intercept usage in OSAPIFixture now
22:13:03 dansmith you're not starting an api service for real here
22:13:05 dansmith right
22:13:14 melwitt O.o
22:13:19 dansmith but anyway if this was A UNIT TEST
22:13:28 mriedem we used to
22:13:29 mriedem self.osapi = service.WSGIService("osapi_compute")
22:13:29 mriedem self.osapi.start()
22:13:29 mriedem self.addCleanup(self.osapi.stop)
22:13:32 melwitt :***(
22:13:37 mriedem which is probably why AllServicesCurrent was added before
22:15:12 mriedem +W
22:15:32 mriedem moving onto soul crusher #2
22:27:51 mriedem bazinga https://review.openstack.org/#/c/582413/8
22:28:04 melwitt argh
22:29:05 melwitt ok that's not so bad
22:29:28 mriedem do you want/need to hold the functional test patch below it out to add checks for core and ram usage?
22:29:57 mriedem or just slap those into this one
22:30:41 melwitt I'm fine with adding them to the second one
22:50:07 openstackgerrit Matt Riedemann proposed openstack/nova master: Docs: Add guide to migrate instance with snapshot https://review.openstack.org/584442
23:00:18 mriedem heh https://review.openstack.org/#/c/382853/
23:00:25 mriedem i knew i'd seen this recently
23:00:26 mriedem https://github.com/starlingx-staging/stx-nova/commit/71acfeae0d1c59fdc77704527d763bd85a276f9a#diff-77f9348ab09642ba46409b6828af4af0R3245
23:00:32 mriedem oct of 2016
23:04:48 openstackgerrit melanie witt proposed openstack/nova stable/ocata: [stable only] Handle quota usage during create/delete races https://review.openstack.org/582413
23:11:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/382853
23:12:57 openstackgerrit Matt Riedemann proposed openstack/nova master: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/382853
23:15:24 mriedem wow so that evacuate fix was approved in march of 2017,
23:15:27 mriedem but apparently needed a rebase
23:15:34 mriedem and then no one ever mentioned needing re-approval
23:17:00 melwitt yikes
23:18:01 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Fix instance evacuation with PCI devices https://review.openstack.org/590059
23:29:22 melwitt I'm going to propose stable releases tomorrow. I meant to do it last week but didn't get to it
23:33:38 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/590062
23:34:41 openstackgerrit Merged openstack/nova master: get provider IDs once when building summaries https://review.openstack.org/589945
23:57:15 openstackgerrit Merged openstack/nova master: libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/589513
23:57:22 openstackgerrit Merged openstack/nova master: Add tempest-slow job to run the tempest slow tests https://review.openstack.org/567697
23:57:32 openstackgerrit Merged openstack/nova master: Fix bad links for admin-guide https://review.openstack.org/580259
23:57:41 openstackgerrit Merged openstack/nova master: [placement] Avoid rp.get_by_uuid in allocation_candidates https://review.openstack.org/589941
23:57:49 openstackgerrit Merged openstack/nova master: api-ref: fix min_version for parent_provider_uuid in responses https://review.openstack.org/579577
23:57:57 openstackgerrit Merged openstack/nova master: doc: mark the max microversion for rocky https://review.openstack.org/589598
#openstack-nova - 2018-08-09
00:24:58 openstackgerrit Merged openstack/nova master: Add functional test for forced live migration rollback allocs https://review.openstack.org/586636
01:36:59 openstackgerrit Merged openstack/nova stable/ocata: [stable only] Add functional regression test for bug 1783613 https://review.openstack.org/588416
01:36:59 openstack bug 1783613 in OpenStack Compute (nova) ocata "[ocata only] quota usage not decremented during boot/delete race" [Undecided,In progress] https://launchpad.net/bugs/1783613 - Assigned to melanie witt (melwitt)
02:08:17 openstackgerrit Chen proposed openstack/nova stable/queens: Fix bad links for admin-guide https://review.openstack.org/590068
02:13:36 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/590070
02:16:02 openstackgerrit Chen proposed openstack/nova stable/queens: Fix bad links for admin-guide https://review.openstack.org/590068
02:16:37 openstackgerrit Chen proposed openstack/nova stable/pike: Fix bad links for admin-guide https://review.openstack.org/590072
02:39:05 openstackgerrit Chen proposed openstack/nova master: Update ssh configuration doc https://review.openstack.org/589844
02:45:27 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Update the parameter explain when updating a volume attachment https://review.openstack.org/565181
02:56:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] api-ref: add description for 1.29 https://review.openstack.org/589407
03:12:36 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081
03:17:19 openstackgerrit Chen proposed openstack/nova master: Trivial fix on migration doc https://review.openstack.org/589028
04:13:29 openstackgerrit Merged openstack/nova stable/queens: Fix bad links for admin-guide https://review.openstack.org/590068
05:32:31 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081
06:05:26 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081
06:55:14 openstackgerrit Sergii Golovatiuk proposed openstack/nova master: libvirt: Always escape IPv6 addresses when used in migration URI https://review.openstack.org/589548
07:15:50 openstackgerrit liuyamin proposed openstack/python-novaclient master: Replace os-client-config to openstacksdk https://review.openstack.org/590141
07:23:14 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150
08:04:48 gibi melwitt, mriedem: opened a versioned notification bp for stein https://blueprints.launchpad.net/nova/+spec/versioned-notification-transformation-stein
08:19:07 mdbooth lyarwood: Passing: https://review.openstack.org/#/c/587013/ !
08:21:47 mdbooth lyarwood: Also passed without the rebase workaround. I'll merge them and resubmit.
08:27:24 openstackgerrit Matthew Booth proposed openstack/nova master: fixtures: Track volume attachments within CinderFixtureNewAttachFlow https://review.openstack.org/587013
08:36:02 openstackgerrit Jose Castro Leon proposed openstack/nova master: Fix get_device_path from network mounted volume https://review.openstack.org/590188
08:49:46 openstackgerrit Matthew Booth proposed openstack/nova master: Add regression test for bug#1784353 https://review.openstack.org/587014
09:13:44 lyarwood mdbooth: cool, thanks again for working through this :)
09:14:27 mdbooth lyarwood: np.
09:14:46 mdbooth lyarwood: Started accidentally while reviewing and seemed silly to stop :)
09:21:19 lyarwood mdbooth: are you going to rebase https://review.openstack.org/#/c/587071/ ?
09:21:30 lyarwood mdbooth: np if not, I have time to work on it this morning finally
09:21:46 mdbooth lyarwood: I was going to leave that for you, that's the real bit :)
09:21:56 lyarwood mdbooth: tis cool, thanks again
09:22:12 mdbooth lyarwood: Incidentally, did you consider the 'fix it in compute' approach
09:22:26 mdbooth I know that's where the patch started, then you moved to conductor
09:22:55 mdbooth But in fixing the fixture I can across other cleanup in compute which already does exactly what I was talking about
09:23:10 lyarwood mdbooth: yeah the remove_volume_connections call
09:23:33 mdbooth Yeah
09:24:12 lyarwood mdbooth: yeah I'll take a look now, it's a shame to flip back again but meh
09:25:02 mdbooth lyarwood: I'm not saying do it, just asking if it's feasible/worth considering
09:25:12 mdbooth Or if you've already considered and rejected it, in fact
09:28:22 lyarwood mdbooth: yeah understood, I think it is feasible and ultimatley a better approach I just haven't looked into the knock on impact of changing shutdown_instance.
09:28:44 mdbooth lyarwood: ack
09:39:39 goutham1 HI all i a facing this issue in rally when i try to create a deployment it throws this error Env manager got invalid spec:
09:39:40 goutham1 ["There is no Platform plugin with name: 'existing@openstack'"]
09:39:40 goutham1 2:57
09:39:40 goutham1 any idea on how to fix it ??
09:39:55 goutham1 rally deployment create --fromenv --name=existing
09:39:56 goutham1 Env manager got invalid spec:
09:39:56 goutham1 ["There is no Platform plugin with name: 'existing@openstack'"]
09:40:14 goutham1 it shows something of this sort any idea on how to fix this ??
09:41:21 mdbooth goutham1: You'll want to try #openstack for user issues, I think
09:41:38 goutham1 thank mdbooth thanks

Earlier   Later