Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-05
22:19:23 dansmith I think we call delete in the compute cell and then it's replicated backwards in that case right?
22:19:28 dansmith the opposite of most calls
22:19:30 melwitt dansmith: from the code, it looks like it doesn't re-run the compute/api part in compute cells, is what I was observing. the use of the conditional made me think it was
22:19:41 dansmith melwitt: but the comment says it does right?
22:20:33 melwitt dansmith: it does. maybe it used to? unless I'm blind as I look in cells_api.py
22:21:57 dansmith well, we force a local delete in the api,
22:22:05 melwitt looks like the delete_everywhere will do a replay
22:22:08 dansmith then call direct to compute via rpc in there
22:22:09 melwitt but not the normal delete
22:22:17 dansmith ah
22:22:19 melwitt gawd
22:23:04 melwitt lol!
22:23:18 dtp :) so summary:
22:23:38 dtp no cell types in v2. ignore that if in there. just delete the tokens.
22:23:46 dtp with the conditional melwitt outlined above
22:23:54 dtp why even have the conditional if cells v1 is gone in ocata?
22:24:04 dansmith I would prefer it if we can avoid the conditional, but if we can't then yes
22:24:14 dansmith dtp: unfortunately cellsv1 will be with us for longer than that
22:24:27 dansmith dtp: until cellsv2 is a replacement and we have a migration path
22:24:43 openstackgerrit Ed Leafe proposed openstack/nova: Remove the Allocation.create() method. https://review.openstack.org/416751
22:25:19 melwitt dtp: we can technically avoid the conditional because the delete_tokens rpc api is a cast i.e. broadcast that doesn't wait for a response. so it could just get sent anyway even if nothing will be there to listen to it
22:25:36 dansmith melwitt: what won't be there to listen for it?
22:25:41 dansmith melwitt: the instance having moved?
22:25:51 melwitt dansmith: the consoleauth service in compute cells
22:25:52 dansmith oh waiut
22:25:56 dansmith right, right
22:26:05 dansmith okay then you should do the conditional
22:26:12 dansmith IMHO of course
22:27:53 melwitt yeah, I hadn't thought of the possibility of just letting it run in compute cells and it not hurting anything
22:28:13 dansmith dtp: so, all good?
22:29:03 melwitt dansmith: wait, did you say do the conditional or did you mean don't do it?
22:29:03 dtp i think so. thanks y'all
22:29:32 dansmith melwitt: I mean _do_ the conditional so we're not just firing off a message into a queue that we know is unlistened to
22:29:40 melwitt yeah, okay
22:29:46 dansmith melwitt: I was originally thinking the reason for the conditional was something else
22:29:52 melwitt I was thinking you said the opposite
22:30:13 melwitt dtp: so yes on the conditional
22:30:46 dtp roger that
22:31:16 dansmith melwitt: remember when we told dtp this would be all easysauce? good times.
22:31:33 melwitt dansmith: yeah, I know. I keep wanting to apologize to him
22:31:36 melwitt sorry dtp
22:31:41 dtp hehe
22:31:57 dansmith it's like telling the driver to get into the other lane because it's faster and then getting stuck behind an accident or something
22:32:12 dansmith should'a kept quiet
22:32:18 melwitt haha
22:36:01 dtp you can make it up to be by having a look at https://review.openstack.org/#/c/336293/ :D
22:40:08 openstackgerrit OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978
22:40:39 melwitt dtp: hah, okay. hm, new config option
22:42:07 dtp yes :) it is a very, very exciting config option
22:43:23 mriedem yes i've already passively aggressively said this is a blueprint
22:44:16 dtp oh, sorry mriedem. should i create a blueprint?
22:44:39 mriedem well, see,
22:44:46 mriedem the thing i'm really going to say is this would be for pike
22:44:50 mriedem w/o wanting to hurt your feelings
22:45:14 mriedem that may be fine for you guys as i know godaddy would already be running with this patch and you're just upstreaming it
22:45:34 dtp yes
22:47:37 dtp so a pike blueprint?
22:48:17 mriedem yeah
22:48:27 dims mriedem : sdague : dansmith : i got a heat integration test going (devstack+python3.5), since it exercises nova, we can spot what we need to fix in nova too - http://logs.openstack.org/86/412486/17/check/gate-heat-dsvm-functional-convg-mysql-lbaasv2-ubuntu-xenial/960bacc/logs/screen-n-cpu.txt.gz
22:48:28 mriedem https://blueprints.launchpad.net/nova/pike
22:48:43 dtp thanks
22:48:56 mriedem dims: you know we don't have to officially care about py3 until pike right? :P
22:49:32 dims mriedem : haha, true, giving you a way to figure out, so it can proceed at a leisurely pace :)
22:49:47 openstackgerrit Derek Higgins proposed openstack/nova: libvirt: fix nova can't delete the instance with nvram https://review.openstack.org/357190
22:49:47 openstackgerrit Derek Higgins proposed openstack/nova: Remove mox in libvirt destory tests https://review.openstack.org/417227
22:49:56 mriedem i'm going to lean on you for all of this you know
22:49:58 mriedem you and gcb
22:51:58 dims mriedem : ack.
22:52:22 dims gcb : look there and start fixing :)
22:52:42 openstackgerrit OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978
22:53:38 mriedem dansmith: melwitt: before i head out, https://review.openstack.org/#/c/417206/ should fix the current CI failure mess
22:53:57 mriedem ooo looky there top check queue failure http://status.openstack.org//elastic-recheck/index.html
22:57:59 openstackgerrit Samantha Blanco proposed openstack/nova: Add missing CLI commands in support-matrix.ini https://review.openstack.org/417232
23:08:01 openstackgerrit Anusha Unnam proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395
23:19:45 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix a parameter in os-availability-zone.inc https://review.openstack.org/417233
#openstack-nova - 2017-01-06
00:25:30 openstackgerrit Takashi NATSUME proposed openstack/nova-specs: Amend cold-migration-with-target-ocata spec https://review.openstack.org/397193
00:25:46 openstackgerrit Takashi NATSUME proposed openstack/nova-specs: Abort Cold Migration https://review.openstack.org/334732
00:25:49 openstackgerrit Takashi NATSUME proposed openstack/nova-specs: List/show all server migration types https://review.openstack.org/407237
00:25:55 openstackgerrit Takashi NATSUME proposed openstack/nova-specs: Fix formatting errors and warnings https://review.openstack.org/407364
00:26:48 stewie925 hello has anyone been able to successfully install a multi-node (controller, compute, network) Openstack on their machine?
00:42:36 mriedem gfdi http://logs.openstack.org/06/417206/2/gate/gate-nova-python35-db/61fcf07/console.html#_2017-01-05_23_10_44_738623
00:52:39 openstackgerrit Matt Riedemann proposed openstack/nova: Fix ksa mocking in test_cinderclient_unsupported_v1 https://review.openstack.org/417206
01:05:28 mriedem easy +W https://review.openstack.org/#/c/415776/2
01:17:13 openstackgerrit Dan Peschman proposed openstack/nova: No API cell up-call to delete consoleauth tokens https://review.openstack.org/415922
02:06:49 openstackgerrit int32bit proposed openstack/python-novaclient: Allow multiple tag add/delete from cli https://review.openstack.org/414635
02:12:07 openstackgerrit huangtianhua proposed openstack/python-novaclient: Make _console() public https://review.openstack.org/414364
02:35:56 openstackgerrit Zhenyu Zheng proposed openstack/python-novaclient: Fix "Message object do not support addition". https://review.openstack.org/414529
02:44:55 openstackgerrit Dan Peschman proposed openstack/nova: No API cell up-call to delete consoleauth tokens https://review.openstack.org/415922
02:59:52 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571
03:06:25 openstackgerrit Matt Riedemann proposed openstack/python-novaclient: Add support for showing aggregate UUIDs (v2.41) https://review.openstack.org/417265
03:11:31 openstackgerrit Matt Riedemann proposed openstack/python-novaclient: Add support for showing aggregate UUIDs (v2.41) https://review.openstack.org/417265
03:38:27 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955
04:17:01 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964
04:17:32 openstackgerrit Zhenyu Zheng proposed openstack/python-novaclient: Fix "Message object do not support addition". https://review.openstack.org/414529
04:17:48 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
04:17:59 openstackgerrit Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926
04:18:39 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient: Microversion 2.41 - Enable cold migration with target host https://review.openstack.org/406707
05:24:43 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Add set/get_traits method to ResourceProvder object https://review.openstack.org/376201
05:24:44 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Add Traits API to placement service https://review.openstack.org/376200
05:24:44 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Caches and synchronizes the traits from os_traits https://review.openstack.org/416007

Earlier   Later