Wednesday, 2016-01-13

*** markmcclain has quit IRC00:53
*** markmcclain has joined #openstack-astara00:55
*** cleverdevil has quit IRC01:09
*** markmcclain has quit IRC02:39
*** markmcclain has joined #openstack-astara02:40
openstackgerritYang Li proposed openstack/astara: Remove 'FIXME' if it's alreasy be done  https://review.openstack.org/26664302:47
*** markmcclain has quit IRC03:12
*** markmcclain has joined #openstack-astara03:15
*** markmcclain has quit IRC03:37
*** markmcclain has joined #openstack-astara03:38
openstackgerritAdam Gandelman proposed openstack/astara: fail func tests early (do not merge)  https://review.openstack.org/26666603:52
*** markmcclain has quit IRC03:52
*** markmcclain has joined #openstack-astara04:24
*** markmcclain has quit IRC04:29
*** markmcclain has joined #openstack-astara05:12
*** markmcclain has quit IRC05:21
*** markmcclain has joined #openstack-astara05:50
*** markmcclain has quit IRC05:56
openstackgerritYang Hongyang proposed openstack/astara: Refactor ensure_cache for instance manager  https://review.openstack.org/26674007:27
openstackgerritYang Hongyang proposed openstack/astara: Refactor ensure_cache for loadbalancer driver  https://review.openstack.org/26674107:27
openstackgerritYang Hongyang proposed openstack/astara: Refactor ensure_cache for router driver  https://review.openstack.org/26674207:27
openstackgerritYang Hongyang proposed openstack/astara: Allow running individual test using tox  https://review.openstack.org/26675808:00
openstackgerritYang Hongyang proposed openstack/astara: Refactor ensure_cache for loadbalancer driver  https://review.openstack.org/26674108:15
openstackgerritYang Hongyang proposed openstack/astara: Refactor ensure_cache for instance manager  https://review.openstack.org/26674008:15
openstackgerritYang Hongyang proposed openstack/astara: Refactor ensure_cache for router driver  https://review.openstack.org/26674208:15
*** ronis has joined #openstack-astara08:48
openstackgerritYang Hongyang proposed openstack/astara: Use assertEqual instead of assertEquals  https://review.openstack.org/26680409:26
*** owlbot has quit IRC09:30
*** owlbot has joined #openstack-astara09:31
openstackgerritYang Li proposed openstack/astara: Remove 'FIXME' if it's already be done  https://review.openstack.org/26664310:20
openstackgerritxiayu proposed openstack/astara: Astara oslo.rootwrap  https://review.openstack.org/26475910:47
*** leonstack has quit IRC11:06
*** leonstack has joined #openstack-astara11:07
openstackgerritxiayu proposed openstack/astara: Automatically generate etc/orchestrator.ini file  https://review.openstack.org/26557612:28
*** davidlenwell has quit IRC14:57
*** davidlenwell has joined #openstack-astara14:59
*** ronis has quit IRC16:24
rodsadam_g looks like https://review.openstack.org/#/c/266551/1 is failing because devstack is not able to create the public network http://logs.openstack.org/51/266551/1/check/gate-functional-dsvm-astara/ef85e22/logs/devstacklog.txt.gz16:25
*** cleverdevil has joined #openstack-astara17:09
*** Guest32 has joined #openstack-astara17:12
Guest32adam_g: I know the cause of the test failures17:25
*** Guest32 is now known as markmcclain117:25
*** ronis has joined #openstack-astara17:30
*** cleverdevil has quit IRC17:36
*** leonstack1 has joined #openstack-astara18:05
*** leonstack has quit IRC18:07
adam_gmarkmcclain1, whats the cause?18:18
adam_gi noticed that failing late last night18:18
markmcclain1mordred and I are discussing -dev18:18
adam_gi think this is what i was hitting18:19
adam_ghttps://review.openstack.org/#/c/263839/18:19
adam_g(rechecking to find out)18:19
*** leonstack has joined #openstack-astara18:39
*** leonstack1 has quit IRC18:40
openstackgerritRosario Di Somma proposed openstack/astara: Delete detached vrrp ports when a newtork is removed  https://review.openstack.org/26708818:43
rodsadam_g markmcclain1 ^^ that should give you an idea about what's needed to fix the issue18:46
markmcclain1rods: awesome18:55
*** cleverdevil has joined #openstack-astara19:11
openstackgerritAdam Gandelman proposed openstack/astara: Drop unused call to non-existent function  https://review.openstack.org/26712719:31
openstackgerritAdam Gandelman proposed openstack/astara: Drop astara.openstack and astara.service  https://review.openstack.org/26713019:42
*** davidlenwell has quit IRC19:57
*** davidlenwell has joined #openstack-astara19:58
*** stanchan has joined #openstack-astara20:05
markmcclain1adam_g: it's like watching paint dry waiting for the functional job to run on your patch20:11
adam_gmarkmcclain1, http://i.dailymail.co.uk/i/pix/2013/03/27/article-2300124-18F5C407000005DC-552_634x490.jpg20:12
markmcclain1haha20:12
adam_gmarkmcclain1, so about HA + managing instances20:12
markmcclain1yeah20:12
adam_gmarkmcclain1, my initial plan is to handle everything/most of it in the instance manager20:13
adam_gso that a single instance manager is capable of managing a single or multiple instances20:13
adam_greplacing self.instance_manager with self.instances, which is a resource container similar to tenant.py's state machine ResourceContainer20:13
adam_gself.instances would have the logic for doing things like pushing config/verifying interfaces/etc across multiples20:13
adam_gwe need to account for cases where a subset of the managed instances are down/are out of date/etc20:14
adam_gso im thinking a states.DEGRADED be added20:14
markmcclain1yeah20:14
markmcclain1that would make sense20:14
adam_gas you mentioned yesterday, update_state and lots of other things in instance manager are really ready for a refactor so this might be a substantial stack of patches20:15
markmcclain1yeah20:16
markmcclain1also concerned how the state machine would change20:16
markmcclain1because checkboot, replug, and rebuild would be handled20:16
markmcclain1in the HA case for rebuild20:17
adam_gdlchmm20:17
markmcclain1seems like you want to attempt to rebuild one at a time20:18
adam_gmarkmcclain1,  you being the user, or you being the orchestrator?20:19
markmcclain1and orchestrator20:19
adam_gid think the operator would want to rebuild all the nodes in a group in one command (ie, to upgrade image)20:19
markmcclain1but that has some interesting side-effects in state machine20:19
adam_gthe orchestrator could stagger it across the nodes for sake of uptime20:20
adam_gmarkmcclain1, im thinking for things like replug, any exit from replug() that currently ends up in RESTART could also exit in DEGRADED20:23
markmcclain1yeah.. assuming alive could exit into degraded20:23
adam_gthe DEGRADED state would be a new one where we cycle through the managed instances ensuring  essentially all the things we do now for singles (the instance is there, plugged, configured, etc)20:23
markmcclain1am concerned that if one of the pairs get stuck down20:24
markmcclain1that we're preventing the healthy node from getting current20:24
adam_gmarkmcclain1, (in my head) that should be non-blocking for the healthy node--it'd just be stuck in a degraded state with only 1/2 routers functional20:25
markmcclain1ok.. so essentially we're breaking down into machines for each instance20:25
markmcclain1and the master machine is really for the logical one?20:26
adam_ghmm maybe20:26
markmcclain1might actually make things simpler20:28
*** markmcclain has joined #openstack-astara20:38
markmcclainfzylogic: much later ping20:47
markmcclainfzylogic: s/ping/pong/20:47
fzylogicwe were having some problems yesterday that seem to suggest the ml2 driver isn't using the floatingip_subnet config option20:49
fzylogicfloating IPs were being assigned from an ipv6 subnet20:50
markmcclainfzylogic: urg20:50
markmcclainkilo right?20:50
fzylogicyeah20:51
fzylogicfrom floatingip-show:20:51
fzylogic| ef032214-3393-4dc3-8d73-5983ca32dab2 | 10.10.10.2       | 2607:f298:5:101d:f816:3eff:fe0c:ecd9 | ad08aae3-ea08-48a9-b5ac-a340bb2c773b |20:51
openstackgerritAdam Gandelman proposed openstack/astara: gate fixer: Fix devstack auth issue and new PEP8 violation  https://review.openstack.org/26716520:55
*** ronis has quit IRC21:05
markmcclainfzylogic: I think I know what's happening21:13
fzylogicyeah?21:13
markmcclainwhile the proper class is being mixed in21:13
markmcclainit's not being mixed in the correct place, so REST call is dispatched to the original version not the updated version21:14
fzylogicaha21:14
markmcclainneutron's stable/liberty has some code which kind of does the right thing, but not fully21:28
markmcclaindigging deeper will have to test a bit and see if the code needs a bit of modernization21:29
fzylogicok21:29
*** stanchan has quit IRC21:29
markmcclainthis file did not have coverage so tests never exercised it21:29
*** stanchan has joined #openstack-astara21:30
markmcclainalso is it still a requirement that floating-ips come out of certain subnets?21:30
markmcclainor can it be simplified to ensure they're just v421:30
fzylogicthat's a tough one...21:32
fzylogicI think the v4 issue is the larger one we have to deal with21:32
fzylogicthe impetus behind wanting to specify specific subnets was so we could retire older ones if we ever needed to reclaim them21:32
*** justinlund has joined #openstack-astara21:32
fzylogicbut even with that ability, we never managed to actually retire the old masseffect network after all this time21:33
fzylogicso I think I'd be OK losing that support for now21:33
markmcclainok21:33
fzylogicit'd mean one less thing for us to maintain21:33
markmcclainseems like updating the allocation pool to be empty could serve as a workaround21:34
markmcclainif you need that functionality again21:34
fzylogicthat has its own set of complications, but yes21:36
rodsadam_g markmcclain if you find a sec later to leave a comment on https://review.openstack.org/#/c/267088/1 would be great22:26
rodsthx22:26
rodssee you all tomorrow22:26
markmcclainrods: will do22:27
*** markmcclain1 has quit IRC22:41

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!