Friday, 2016-01-22

« Thursday, 2016-01-21 Index Saturday, 2016-01-23 »
adam_gphil_h, heya, found just a couple more minor issues wth the doc patch otherwise looks good00:40
*** x58 has quit IRC00:50
*** x58 has joined #openstack-astara01:00
openstackgerritMerged openstack/astara-horizon: Remove E123 ignore for pep8 check  https://review.openstack.org/27009201:09
openstackgerritMerged openstack/astara-neutron: Fix ignore pep8 errors and a typo  https://review.openstack.org/27008601:26
openstackgerritMerged openstack/astara-appliance: Remove E123 ignore for pep8 check  https://review.openstack.org/27008901:32
openstackgerritOpenStack Proposal Bot proposed openstack/astara-appliance: Updated from global requirements  https://review.openstack.org/26842604:02
*** justinlund has quit IRC05:15
*** justinlund has joined #openstack-astara05:16
*** elo has joined #openstack-astara06:57
openstackgerritAdam Gandelman proposed openstack/astara: Adds a functional test for router interfaces  https://review.openstack.org/26451507:17
openstackgerritAdam Gandelman proposed openstack/astara: Add astara-ctl + API functional tests  https://review.openstack.org/24387407:17
openstackgerritAdam Gandelman proposed openstack/astara: Enrich functional test suite  https://review.openstack.org/21995207:17
openstackgerritAdam Gandelman proposed openstack/astara: fail func tests early (do not merge)  https://review.openstack.org/26666607:17
openstackgerritAdam Gandelman proposed openstack/astara: Handle exception when worker unlocks already unlocked resource  https://review.openstack.org/27115807:17
openstackgerritMerged openstack/astara-appliance: Add reno for release notes management  https://review.openstack.org/26494308:26
openstackgerritMerged openstack/astara-appliance: Ensure interface cache up to date  https://review.openstack.org/26493508:26
*** openstackgerrit has quit IRC11:47
*** openstackgerrit has joined #openstack-astara11:47
*** phil_h has quit IRC12:22
*** phil_h has joined #openstack-astara12:23
*** openstackgerrit has quit IRC12:33
*** openstackgerrit has joined #openstack-astara12:33
*** pcaruana has joined #openstack-astara14:38
openstackgerritPhil Hopkins proposed openstack/astara: Install documentation  https://review.openstack.org/26975515:00
adam_gmarkmcclain, any reason this would be a bad idea? https://review.openstack.org/#/c/271158/18:08
markmcclainhmmmm18:09
markmcclainyeah.. if it's unlocked seems that a release failure is ok18:12
openstackgerritAdam Gandelman proposed openstack/astara-appliance: Remove lambda usage, fix pep8 E731 violation  https://review.openstack.org/27145518:18
adam_ghttps://git.openstack.org/cgit/openstack/astara/tree/astara/worker.py#n55318:19
adam_gwe do the same thing there, just not sure if the context is any different than when we are releasing based on state machine's resource id18:19
openstackgerritAdam Gandelman proposed openstack/astara-appliance: Updated from global requirements  https://review.openstack.org/26842618:21
openstackgerritMerged openstack/astara: Fix a bug for default provider and add an unittest  https://review.openstack.org/26808518:44
ryanpetrelloadam_g you around?18:51
ryanpetrellowhenever you are, for https://review.openstack.org/#/c/255613/2/astara_router/drivers/metadata.py https://screens.objects.dreamhost.com/01-22-2016-13-51-06-f93fd18d-61af-4fb6-a4e0-e0f5db8fccae.png?Signature=c4h4iGLogYenCA1jpAJ%2F6KS1qro%3D&Expires=1453575075&AWSAccessKeyId=8Y1FmVQdlqLchxaCahBd18:51
ryanpetrellomaybe I'm misunderstanding this a bit18:51
openstackgerritMerged openstack/astara: Fix pep8 errors in source code  https://review.openstack.org/27005318:53
openstackgerritMerged openstack/astara: Remove E123 ignore for pep8 check  https://review.openstack.org/27005418:54
adam_gryanpetrello, hi here now19:32
*** ChanServ sets mode: +o adam_g19:35
*** adam_g has left #openstack-astara19:36
*** adam_g has joined #openstack-astara19:36
adam_gryanpetrello, duh yea i think you're right19:38
ryanpetrellojust wanted to make sure I wasn't misunderstanding that19:38
adam_gill fix now and add some tests for that file19:40
ryanpetrellokk19:43
openstackgerritMerged openstack/astara: Filter the 'leader' member from cluster membership list  https://review.openstack.org/26109819:57
openstackgerritMerged openstack/astara: Remove unused logging import  https://review.openstack.org/26941019:59
openstackgerritMerged openstack/astara: Drop orchestrator.ini.sample from git  https://review.openstack.org/26986420:00
openstackgerritMerged openstack/astara: Correct state transition from stop when instance is gone  https://review.openstack.org/26995720:02
openstackgerritMerged openstack/astara: Handle exception when worker unlocks already unlocked resource  https://review.openstack.org/27115820:02
openstackgerritMerged openstack/astara-neutron: remove dead floating IP code  https://review.openstack.org/26828620:02
openstackgerritMerged openstack/astara-horizon: Resolve astara admin. API via keystone  https://review.openstack.org/24603920:03
openstackgerritMerged openstack/astara: Replace range() to six.moves.range() for both python2 and python3  https://review.openstack.org/26948320:03
openstackgerritMerged openstack/astara: Enrich functional test suite  https://review.openstack.org/21995220:06
openstackgerritAdam Gandelman proposed openstack/astara-appliance: Accept new orchestrator config bucket  https://review.openstack.org/25561320:13
adam_gryanpetrello, ^ thanks for catching that20:13
*** pcaruana has quit IRC20:14
ryanpetrellok, +120:20
ryanpetrelloI'm gonna let markmcclain and rods weigh in on that one, too :)20:20
ryanpetrellosince I just get done +2a on a bunch of other stuff20:20
adam_gryanpetrello, thanks duder20:26
adam_ggah rebases20:27
ryanpetrellohttps://i.imgur.com/JPx7R.jpg20:27
openstackgerritAdam Gandelman proposed openstack/astara: Push orchestrator config into the appliance  https://review.openstack.org/25561220:35
openstackgerritAdam Gandelman proposed openstack/astara: Cleanup SM management during rebalance events.  https://review.openstack.org/25921620:35
openstackgerritAdam Gandelman proposed openstack/astara: Adds a new rebalance takeover state  https://review.openstack.org/26074820:35
adam_gryanpetrello, https://review.openstack.org/#/c/255612/ lost your +2 with that20:35
* ryanpetrello looking20:38
ryanpetrellothat just a rebase?20:39
adam_gyya20:45
adam_gi need to go get my eyes harassed by the eye doctor. back in a while20:46
*** pcaruana has joined #openstack-astara21:48
openstackgerritMerged openstack/astara: Fix removal functions in oslo_utils for astara  https://review.openstack.org/27002822:42
openstackgerritMerged openstack/astara-appliance: Updated from global requirements  https://review.openstack.org/26842622:55
« Thursday, 2016-01-21 Index Saturday, 2016-01-23 »

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