Thursday, 2016-01-14

openstackgerritAdam Gandelman proposed openstack/astara: Adds a functional test for router interfaces  https://review.openstack.org/26451500:07
openstackgerritAdam Gandelman proposed openstack/astara: Add astara-ctl + API functional tests  https://review.openstack.org/24387400:07
openstackgerritAdam Gandelman proposed openstack/astara: Enrich functional test suite  https://review.openstack.org/21995200:07
openstackgerritAdam Gandelman proposed openstack/astara: fail func tests early (do not merge)  https://review.openstack.org/26666600:07
openstackgerritMerged openstack/astara: gate fixer: Fix devstack auth issue and new PEP8 violation  https://review.openstack.org/26716500:34
*** ryanpetrello has quit IRC00:49
*** ryanpetrello has joined #openstack-astara00:50
*** ryanpetrello has quit IRC00:55
*** cleverdevil has quit IRC00:56
*** ryanpetrello has joined #openstack-astara00:57
*** ryanpetrello has quit IRC01:02
*** ryanpetrello has joined #openstack-astara01:06
*** ryanpetrello has quit IRC01:11
*** ryanpetrello has joined #openstack-astara01:13
openstackgerritAdam Gandelman proposed openstack/astara: Log error on functional gate job failure  https://review.openstack.org/25608601:19
*** elo has quit IRC01:21
openstackgerritAdam Gandelman proposed openstack/astara: Rename self.driver to self.resource where applicable  https://review.openstack.org/26659001:22
openstackgerritAdam Gandelman proposed openstack/astara: WIP Add clustering capabilities to instance manager  https://review.openstack.org/26726301:22
*** xiayu has joined #openstack-astara01:35
xiayuwhy we always got a test_appliance_is_alive ... FAIL?01:37
adam_gxiayu, there's a bug, which i believe is due to the job landing on slow slaves, where the first config update to the appliance hangs in the applinace VM01:48
adam_gyou'll notice it when you look at the astara log for a faile djob and the last line is: http://logs.openstack.org/39/263839/1/check/gate-functional-dsvm-astara/8d3abc9/logs/screen-astara.txt.gz#_2016-01-13_20_53_41_19501:49
adam_gim hoping patch #256086 can echo an error msg when we hit it so we can have ER catch it and comment on the patch01:50
adam_galthough, we also did have a bug in os-client-config break our gate job today01:50
xiayuok, i got it01:54
*** stanchan has quit IRC01:54
*** elo has joined #openstack-astara02:11
openstackgerritYang Li proposed openstack/astara: Fix loadbalancer typo in unittest  https://review.openstack.org/26727202:13
openstackgerritxiayu proposed openstack/astara: Automatically generate etc/orchestrator.ini file  https://review.openstack.org/26557602:23
openstackgerritMerged openstack/astara: devstack: Use trueorfalse function  https://review.openstack.org/26562502:28
openstackgerritMerged openstack/astara: Use the router driver to remove the vrrp ports  https://review.openstack.org/26601202:28
openstackgerritMerged openstack/astara: Remove 'FIXME' if it's already be done  https://review.openstack.org/26664302:34
*** elo has quit IRC02:42
openstackgerritYang Hongyang proposed openstack/astara: Refactor ensure_cache for loadbalancer driver  https://review.openstack.org/26674102:45
openstackgerritYang Hongyang proposed openstack/astara: Refactor ensure_cache for router driver  https://review.openstack.org/26674202:45
*** xiayu1 has joined #openstack-astara02:51
*** xiayu has quit IRC02:52
*** yanghy has joined #openstack-astara02:57
*** leonstack has quit IRC03:39
*** leonstack has joined #openstack-astara03:40
*** xiayu1 has quit IRC03:45
*** xiayu has joined #openstack-astara03:47
*** ryanpetrello has quit IRC04:33
*** ryanpetrello has joined #openstack-astara04:38
*** ryanpetrello has quit IRC04:43
*** stanchan has joined #openstack-astara04:44
*** stanchan has quit IRC04:52
*** ryanpetrello has joined #openstack-astara05:00
*** ryanpetrello has quit IRC05:06
openstackgerritAdam Gandelman proposed openstack/astara: ensure_cache() during router pre_plug()  https://review.openstack.org/26730905:39
xiayuwe use firewall/rules interface to get astara-appliance's alive info,  why?06:41
xiayudef is_alive(host, port):06:41
xiayu    path = ASTARA_BASE_PATH + 'firewall/rules'06:41
xiayu    try:06:41
xiayu        s = _get_proxyless_session()06:41
xiayu        r = s.get(_mgt_url(host, port, path), timeout=cfg.CONF.alive_timeout)06:41
xiayu        if r.status_code == 200:06:42
xiayu            return True06:42
xiayu    except Exception as e:06:42
xiayu        LOG.debug('is_alive for %s failed: %s', host, str(e))06:42
xiayu    return False06:42
*** elo has joined #openstack-astara07:09
openstackgerritAdam Gandelman proposed openstack/astara: Add astara-ctl + API functional tests  https://review.openstack.org/24387408:04
openstackgerritAdam Gandelman proposed openstack/astara: ensure_cache() during router pre_plug()  https://review.openstack.org/26730908:04
openstackgerritAdam Gandelman proposed openstack/astara: fail func tests early (do not merge)  https://review.openstack.org/26666608:04
*** ronis has joined #openstack-astara08:37
leonstackhi xiayu. I think it make sense...It just wants to make sure it can get request data from astara. After that astara will sent data to astara-appliance, then astara-appliance config the firewall rules in instance..:)08:40
xiayuthis API just get firewall rules already config in instance,  if we want to check the instance's live, why not use another api.08:46
*** reedip is now known as outofmemory08:57
leonstackxiayu, I saw it was used in InstanceManager.update_state function, and in this function it just want to make sure it can be arrived by astara and update its status, no matter it's configured or not. And the final purpose of this function is to communicate with astara-application, so I think it's fine here....09:32
openstackgerritYang Hongyang proposed openstack/astara: Enable pez unit tests  https://review.openstack.org/26743509:45
openstackgerritYang Hongyang proposed openstack/astara: Enable pez unit tests  https://review.openstack.org/26743509:55
openstackgerritRosario Di Somma proposed openstack/astara: Delete detached vrrp ports when a newtork is removed  https://review.openstack.org/26708809:56
openstackgerritMerged openstack/astara: Modify 'router' to 'loadbalancer' for loadbalancer driver  https://review.openstack.org/26619111:47
openstackgerritYang Li proposed openstack/astara: Modify xrange() to six.moves.range() for both python2 and python3  https://review.openstack.org/26752212:43
*** justinlund has quit IRC14:16
*** xiayu has quit IRC14:32
*** yanghy has quit IRC14:51
*** justinlund has joined #openstack-astara15:13
*** ryanpetrello has joined #openstack-astara15:13
*** ronis has quit IRC16:06
*** ronis has joined #openstack-astara16:17
*** ronis has quit IRC16:23
*** xiayu has joined #openstack-astara16:40
*** justinlund has quit IRC16:41
*** prithiv has joined #openstack-astara16:47
*** prithiv has quit IRC16:48
*** prithiv has joined #openstack-astara16:49
*** cleverdevil has joined #openstack-astara16:51
*** prithiv has quit IRC16:54
*** prithiv has joined #openstack-astara17:06
*** cleverdevil has quit IRC17:35
*** cleverdevil has joined #openstack-astara17:37
*** ronis has joined #openstack-astara17:46
*** justinlund has joined #openstack-astara17:46
*** cleverdevil- has joined #openstack-astara17:51
*** cleverdevil has quit IRC17:54
*** cleverdevil has joined #openstack-astara17:56
*** cleverdevil_ has joined #openstack-astara17:57
*** cleverdevil- has quit IRC17:58
*** prithiv has quit IRC18:01
*** cleverdevil has quit IRC18:01
*** prithiv has joined #openstack-astara18:01
*** prithiv has quit IRC18:22
openstackgerritmark mcclain proposed openstack/astara: Adds reno for release notes management  https://review.openstack.org/26394218:41
openstackgerritMerged openstack/astara: Enable pez unit tests  https://review.openstack.org/26743518:53
openstackgerritMerged openstack/astara: Fix loadbalancer typo in unittest  https://review.openstack.org/26727218:56
openstackgerritMerged openstack/astara: Astara oslo.rootwrap  https://review.openstack.org/26475918:57
openstackgerritMerged openstack/astara: Modify xrange() to six.moves.range() for both python2 and python3  https://review.openstack.org/26752218:57
openstackgerritAdam Gandelman proposed openstack/astara: Add astara-ctl + API functional tests  https://review.openstack.org/24387419:03
openstackgerritAdam Gandelman proposed openstack/astara: ensure_cache() during router pre_plug()  https://review.openstack.org/26730919:03
openstackgerritAdam Gandelman proposed openstack/astara: Enrich functional test suite  https://review.openstack.org/21995219:03
openstackgerritAdam Gandelman proposed openstack/astara: Allow API listening address to be specified in config  https://review.openstack.org/24600519:03
openstackgerritAdam Gandelman proposed openstack/astara: fail func tests early (do not merge)  https://review.openstack.org/26666619:03
openstackgerritAdam Gandelman proposed openstack/astara: Drop astara.openstack and astara.service  https://review.openstack.org/26713019:06
adam_gmarkmcclain, fixed a trivial pep8 on that last one if you wanna re +W it19:06
openstackgerritMerged openstack/astara: Ignore AttributeError for instance provider  https://review.openstack.org/26504519:12
*** ronis has quit IRC19:24
openstackgerritMerged openstack/astara: Remove response body from cli output  https://review.openstack.org/26511319:33
openstackgerritAdam Gandelman proposed openstack/astara: Push orchestrator config into the appliance  https://review.openstack.org/25561219:44
openstackgerritAdam Gandelman proposed openstack/astara: Cleanup SM management during rebalance events.  https://review.openstack.org/25921619:44
openstackgerritAdam Gandelman proposed openstack/astara: Adds a new rebalance takeover state  https://review.openstack.org/26074819:44
openstackgerritAdam Gandelman proposed openstack/astara: Push orchestrator config into the appliance  https://review.openstack.org/25561219:55
openstackgerritAdam Gandelman proposed openstack/astara: Cleanup SM management during rebalance events.  https://review.openstack.org/25921619:55
openstackgerritAdam Gandelman proposed openstack/astara: Adds a new rebalance takeover state  https://review.openstack.org/26074819:55
openstackgerritAdam Gandelman proposed openstack/astara: Cleanup deleted resource from the tenant resource cache  https://review.openstack.org/26434020:02
openstackgerritAdam Gandelman proposed openstack/astara: Cleanup deleted resource from the tenant resource cache  https://review.openstack.org/26434020:03
openstackgerritMerged openstack/astara: Use assertEqual instead of assertEquals  https://review.openstack.org/26680420:03
*** cleverdevil_ has quit IRC20:05
markmcclainelo, adam_g: have you all had a chance to test: https://review.openstack.org/#/c/266586/ ?20:12
adam_gmarkmcclain, planning on doing it in a few mins here20:12
adam_gwaiting on a restack20:12
markmcclaincool20:13
elonot yet.20:29
elowill test it out now20:29
adam_gmarkmcclain, FYI trying to reproduce the issue, im hitting the issue addressed by https://review.openstack.org/#/c/264935/ pretty trivial fix20:35
markmcclainI've reviewed it.. ryanpetrello or rods: ^^^^20:37
rodslooking20:38
*** nolan has quit IRC20:43
*** nolan has joined #openstack-astara20:43
adam_g markmcclain working good for me20:45
adam_gmarkmcclain, gonna put a patch that adds reno to  astara-neutron and rebase it on that with a relnote20:46
markmcclainadam_g: perfect20:48
*** cleverdevil has joined #openstack-astara20:48
adam_gusing reno across multiple repos for a single project will actually kinda  PITA20:49
adam_gwonder if neutron or any of the other projects have a good way to collect all sub-projects relnotes in a central place20:49
markmcclainnot that I know of20:53
openstackgerritAdam Gandelman proposed openstack/astara-neutron: allow DHCP from router interfaces  https://review.openstack.org/26658621:04
openstackgerritAdam Gandelman proposed openstack/astara-neutron: Add reno for release notes management  https://review.openstack.org/26781221:04
adam_gmarkmcclain, rods ^  theres another patch adding reno to astara-appliance  that could use a quick approval at  https://review.openstack.org/#/c/264943/21:07
*** ryanpetrello has quit IRC21:18
*** ryanpetrello has joined #openstack-astara21:18
*** cleverdevil has quit IRC21:18
*** cleverdevil has joined #openstack-astara21:20
*** nolan has quit IRC21:35
*** nolan has joined #openstack-astara21:35
*** justinlund has quit IRC21:39
openstackgerritMerged openstack/astara: Python3: replace dumps with dump_as_bytes  https://review.openstack.org/26633521:44
openstackgerritMerged openstack/astara: Log rendered cloud-init to debug log  https://review.openstack.org/26404921:44
adam_ghmm new pep8 err in astara-appliance22:18
openstackgerritMerged openstack/astara: Allow running individual test using tox  https://review.openstack.org/26675822:31
openstackgerritAdam Gandelman proposed openstack/astara-appliance: Remove lambda usage, fix pep8 E731 violation  https://review.openstack.org/26784922:42
adam_grods, markmcclain ^22:42
*** justinlund has joined #openstack-astara23:06
*** stanchan has joined #openstack-astara23:25
openstackgerritAdam Gandelman proposed openstack/astara: Add astara-ctl + API functional tests  https://review.openstack.org/24387423:33
openstackgerritAdam Gandelman proposed openstack/astara: fail func tests early (do not merge)  https://review.openstack.org/26666623:33
openstackgerritAdam Gandelman proposed openstack/astara: gate fixer: Fix devstack auth issue and new PEP8 violation  https://review.openstack.org/26786223:39

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