openstackgerrit | Adam Gandelman proposed openstack/astara: Add astara-ctl + API functional tests https://review.openstack.org/243874 | 00:48 |
---|---|---|
openstackgerrit | Adam Gandelman proposed openstack/astara: fail func tests early (do not merge) https://review.openstack.org/266666 | 00:48 |
*** Liuqing has joined #openstack-astara | 00:53 | |
*** xiayu has quit IRC | 01:21 | |
*** shashank_hegde has quit IRC | 01:22 | |
*** xiayu has joined #openstack-astara | 01:25 | |
*** xiayu has quit IRC | 01:27 | |
*** xiayu has joined #openstack-astara | 01:28 | |
*** xiayu has quit IRC | 01:35 | |
openstackgerrit | Adam Gandelman proposed openstack/astara: WIP Add clustering capabilities to instance manager https://review.openstack.org/267263 | 01:35 |
openstackgerrit | Adam Gandelman proposed openstack/astara: Rename self.driver to self.resource where applicable https://review.openstack.org/266590 | 01:35 |
*** Liuqing has quit IRC | 01:47 | |
*** shashank_hegde has joined #openstack-astara | 01:49 | |
*** shashank_hegde has quit IRC | 02:07 | |
*** Liuqing has joined #openstack-astara | 02:11 | |
*** shashank_hegde has joined #openstack-astara | 02:12 | |
*** shashank_hegde has quit IRC | 02:33 | |
*** shashank_hegde has joined #openstack-astara | 03:09 | |
*** yanghy has joined #openstack-astara | 03:24 | |
*** shashank_hegde has quit IRC | 03:25 | |
openstackgerrit | caoyue proposed openstack/astara: Remove unused logging import https://review.openstack.org/269410 | 03:27 |
openstackgerrit | Merged openstack/astara: Adds reno for release notes management https://review.openstack.org/263942 | 03:41 |
openstackgerrit | Yang Li proposed openstack/astara: Fix a bug for default provider and add an unittest https://review.openstack.org/268085 | 03:55 |
*** shashank_hegde has joined #openstack-astara | 04:30 | |
openstackgerrit | mark mcclain proposed openstack/astara-neutron: remove dead floating IP code https://review.openstack.org/268286 | 04:30 |
*** shashank_hegde has quit IRC | 04:52 | |
openstackgerrit | Merged openstack/astara-neutron: Updated from global requirements https://review.openstack.org/269315 | 04:56 |
openstackgerrit | Yang Li proposed openstack/astara: Add an option to get max sleep time from the config file https://review.openstack.org/266213 | 05:03 |
openstackgerrit | Merged openstack/astara: trival: remove redundant code https://review.openstack.org/268849 | 05:07 |
openstackgerrit | Merged openstack/astara: Remove unused class method https://review.openstack.org/268858 | 05:07 |
openstackgerrit | Merged openstack/astara: Py3: use dict.items() instead of dict.iteritems() https://review.openstack.org/268409 | 05:07 |
openstackgerrit | caoyue proposed openstack/astara: Remove unused logging import https://review.openstack.org/269410 | 05:14 |
*** shashank_hegde has joined #openstack-astara | 05:23 | |
*** shashank_hegde has quit IRC | 05:42 | |
*** shashank_hegde has joined #openstack-astara | 05:42 | |
*** elo has quit IRC | 05:51 | |
openstackgerrit | Yang Li proposed openstack/astara: Add an option to get max sleep time from the config file https://review.openstack.org/266213 | 06:01 |
*** elo has joined #openstack-astara | 06:02 | |
*** elo has quit IRC | 06:27 | |
*** elo has joined #openstack-astara | 06:28 | |
*** elo has quit IRC | 06:37 | |
*** elo has joined #openstack-astara | 06:38 | |
*** shashank_hegde has quit IRC | 06:44 | |
*** shashank_hegde has joined #openstack-astara | 07:00 | |
openstackgerrit | Adam Gandelman proposed openstack/astara: Rename self.driver to self.resource where applicable https://review.openstack.org/266590 | 07:08 |
*** shashank_hegde has quit IRC | 07:21 | |
*** elo has quit IRC | 07:23 | |
openstackgerrit | Yang Li proposed openstack/astara: Replace range() to six.moves.range() for both python2 and python3 https://review.openstack.org/269483 | 07:57 |
*** yanghy_ has joined #openstack-astara | 08:27 | |
*** yanghy has quit IRC | 08:30 | |
openstackgerrit | Yang Li proposed openstack/astara: Replace range() to six.moves.range() for both python2 and python3 https://review.openstack.org/269483 | 09:25 |
openstackgerrit | Yang Li proposed openstack/astara: Replace range() to six.moves.range() for both python2 and python3 https://review.openstack.org/269483 | 09:35 |
*** phil_hop has joined #openstack-astara | 11:54 | |
*** phil_hopk has quit IRC | 11:57 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/astara: Updated from global requirements https://review.openstack.org/268425 | 12:01 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/astara-horizon: Updated from global requirements https://review.openstack.org/268427 | 12:01 |
*** Liuqing has quit IRC | 12:06 | |
*** owlbot has quit IRC | 12:32 | |
*** owlbot has joined #openstack-astara | 12:37 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/astara: Updated from global requirements https://review.openstack.org/268425 | 13:45 |
openstackgerrit | Yang Hongyang proposed openstack/astara: Introduce states.internal to map service state back https://review.openstack.org/269640 | 13:51 |
*** prithiv has joined #openstack-astara | 13:54 | |
*** yanghy_ has quit IRC | 15:07 | |
rods | adam_g you around? | 15:50 |
ryanpetrello | hrm | 16:08 |
ryanpetrello | gerrit doesn't like me this AM | 16:08 |
ryanpetrello | won't let me log in | 16:08 |
ryanpetrello | just shows a "Server Error" | 16:08 |
*** Liuqing has joined #openstack-astara | 16:16 | |
*** prithiv has quit IRC | 16:28 | |
*** prithiv has joined #openstack-astara | 16:29 | |
openstackgerrit | Phil Hopkins proposed openstack/astara: Install documentation https://review.openstack.org/269755 | 16:40 |
phil_h | Just pushed the first pass install documentation | 16:42 |
phil_h | I need a second pass to clean some errors. | 16:42 |
*** Liuqing has quit IRC | 16:46 | |
openstackgerrit | Phil Hopkins proposed openstack/astara: Install documentation https://review.openstack.org/269755 | 16:48 |
*** Liuqing has joined #openstack-astara | 16:50 | |
*** Liuqing has quit IRC | 16:51 | |
*** shashank_hegde has joined #openstack-astara | 17:29 | |
*** shashank_hegde has quit IRC | 17:41 | |
*** prithiv has quit IRC | 18:08 | |
adam_g | rods, hi | 18:31 |
rods | adam_g I saw in the chat log that you had an idea to fix the stray port issue, something related to change the device_owner | 18:33 |
rods | I'm available to test that solution if you think it might help | 18:33 |
adam_g | rods, well i just found myself digging through neutron code when i kept hitting network-in-use errors | 18:34 |
adam_g | sorry--on a call | 18:37 |
rods | adam_g np | 18:39 |
adam_g | rods, so i was digging around and found a couple places that seemed to be the appropriate places to override to allow network deletion to continue /w astara ports attached | 18:39 |
adam_g | im not certain we have a good way to override it,tho. and mark brought up the point that having those ports in-use (to some degree) by nova might make it complicated | 18:40 |
rods | yeah, makes sense | 18:41 |
*** shashank_hegde has joined #openstack-astara | 18:41 | |
adam_g | rods, so, i didnt really have any clear ideas just found that neutron has code to deal with the issue we're seeing, with regard to standard DHCP ports | 18:42 |
rods | so maybe a combination of what neutron does with those dhcp ports and a clean up thread? | 18:44 |
rods | adam_g ^^ | 18:44 |
adam_g | rods, yeah, thats what i was thinking... because, if all we have is a cleanup thread, theres still a lag between when a router / router interface gets deleted and when a tenant can actually delete its network | 18:45 |
adam_g | letting the deletion happen and having port cleanup happen async later would fix that | 18:45 |
adam_g | ... and i think neutron handles it similarly | 18:45 |
adam_g | for standard dhcp/routers/etc | 18:46 |
rods | yeah, maybe we can have the rug change the device_owner of those ports once they have been detached and then have a clean up thread that does the cleaning async | 18:47 |
adam_g | rods, i'd have to really dig in to find what the implications are of doing that. i think that might be racey with nova, who also uses device_owner IIRC | 18:55 |
openstackgerrit | Adam Gandelman proposed openstack/astara: Filter the 'leader' member from cluster membership list https://review.openstack.org/261098 | 18:57 |
adam_g | https://etherpad.openstack.org/p/astara-m2-patches | 19:01 |
*** prithiv has joined #openstack-astara | 19:02 | |
adam_g | rods, markmcclain ryanpetrello all ^ theres a list of stuff we need to get in before cutting M2. as i said in email, most have been up for a while and any help getting them reviewed and approve today/tomorrow would be great | 19:02 |
stupidnic | adam_g: I am just reviewing that install doc... Something that stood out (and I am sure it is minor), but the option for service_neutron_metadata_proxy isn't an officially recognized config option in nova.conf per http://docs.openstack.org/liberty/config-reference/content/list-of-compute-config-options.html | 19:10 |
stupidnic | The option is service_metadata_proxy and should be under the [neutron] section... this was the issue I ran into over the weekend | 19:10 |
phil_h | I will fix that, I will need to upload a new patch set anyway | 19:24 |
phil_h | thanks | 19:24 |
adam_g | stupidnic, thanks. in the future dont hesitate to leave comments on the patch itself | 19:24 |
stupidnic | will do... I didn't know if I was allowed since I am not really a developer | 19:25 |
adam_g | stupidnic, sure, you just need to get setup with an account on launchpad/gerrit | 19:26 |
phil_h | Stupidnic, As soon as the jenkins run finishes I will push the change. | 19:32 |
openstackgerrit | Phil Hopkins proposed openstack/astara: Install documentation https://review.openstack.org/269755 | 19:37 |
phil_h | stupidnic: I just pushed the changes with your suggestion. | 19:38 |
adam_g | phil_h, hey phil i commented, but they're attached to patchset #2 | 19:39 |
adam_g | https://review.openstack.org/#/c/269755/2/docs/source/install.rst | 19:39 |
openstackgerrit | Adam Gandelman proposed openstack/astara: Add astara-ctl + API functional tests https://review.openstack.org/243874 | 20:00 |
openstackgerrit | Adam Gandelman proposed openstack/astara: Enrich functional test suite https://review.openstack.org/219952 | 20:00 |
openstackgerrit | Adam Gandelman proposed openstack/astara: fail func tests early (do not merge) https://review.openstack.org/266666 | 20:00 |
openstackgerrit | Phil Hopkins proposed openstack/astara: Install documentation https://review.openstack.org/269755 | 20:07 |
*** prithiv has quit IRC | 20:12 | |
*** phil_h has quit IRC | 20:12 | |
*** phil_hopk has joined #openstack-astara | 20:21 | |
*** phil_hop has quit IRC | 20:24 | |
*** phil_hopk has quit IRC | 20:50 | |
*** phil_hop has joined #openstack-astara | 20:50 | |
*** phil_hop has quit IRC | 21:15 | |
*** phil_hopk has joined #openstack-astara | 21:15 | |
*** phil_hop has joined #openstack-astara | 21:15 | |
*** phil_hop has quit IRC | 21:17 | |
*** phil_hopk has joined #openstack-astara | 21:17 | |
*** phil_hopk has quit IRC | 21:18 | |
*** phil_hop has joined #openstack-astara | 21:18 | |
*** phil_hop has quit IRC | 21:19 | |
*** phil_hopk has joined #openstack-astara | 21:19 | |
openstackgerrit | Adam Gandelman proposed openstack/astara: Drop orchestrator.ini.sample from git https://review.openstack.org/269864 | 21:21 |
openstackgerrit | Merged openstack/astara: Updated from global requirements https://review.openstack.org/268425 | 21:23 |
openstackgerrit | Phil Hopkins proposed openstack/astara: Install documentation https://review.openstack.org/269755 | 21:31 |
openstackgerrit | Merged openstack/astara: Remove unused openstack common conf https://review.openstack.org/267920 | 21:34 |
*** phil_h has joined #openstack-astara | 21:37 | |
phil_h | adam_g: thanks for the comment on my patch. My editor like to insert the whitespace and that is hard to see after it does it. | 21:38 |
adam_g | phil_h, np, which editor ? | 21:38 |
adam_g | ive got a thing in my .vimrc to highlight whitespace for me so i catch it | 21:38 |
phil_h | Well shamefully I am using kate under KDE | 21:39 |
phil_h | I need to get vim set up to stop the problems I have been encountering. | 21:39 |
adam_g | ahh | 21:39 |
phil_h | anyway I think the patchset I just uploaded will do the trick | 21:40 |
*** phil_hopk has quit IRC | 21:48 | |
*** phil_hop has joined #openstack-astara | 21:48 | |
*** phil_hop has quit IRC | 21:49 | |
*** phil_hopk has joined #openstack-astara | 21:49 | |
*** phil_hopk has quit IRC | 21:49 | |
openstackgerrit | Adam Gandelman proposed openstack/astara: Adds a functional test for router interfaces https://review.openstack.org/264515 | 21:55 |
openstackgerrit | Adam Gandelman proposed openstack/astara: Add astara-ctl + API functional tests https://review.openstack.org/243874 | 21:55 |
openstackgerrit | Adam Gandelman proposed openstack/astara: Enrich functional test suite https://review.openstack.org/219952 | 21:55 |
openstackgerrit | Adam Gandelman proposed openstack/astara: Drop orchestrator.ini.sample from git https://review.openstack.org/269864 | 21:55 |
openstackgerrit | Adam Gandelman proposed openstack/astara: fail func tests early (do not merge) https://review.openstack.org/266666 | 21:55 |
*** davidlenwell has quit IRC | 22:37 | |
*** davidlenwell has joined #openstack-astara | 22:38 | |
openstackgerrit | Merged openstack/astara-horizon: Updated from global requirements https://review.openstack.org/268427 | 22:41 |
openstackgerrit | Adam Gandelman proposed openstack/astara: Adds a functional test for router interfaces https://review.openstack.org/264515 | 23:19 |
openstackgerrit | Adam Gandelman proposed openstack/astara: Add astara-ctl + API functional tests https://review.openstack.org/243874 | 23:19 |
openstackgerrit | Adam Gandelman proposed openstack/astara: Enrich functional test suite https://review.openstack.org/219952 | 23:19 |
openstackgerrit | Adam Gandelman proposed openstack/astara: fail func tests early (do not merge) https://review.openstack.org/266666 | 23:19 |
rods | adam_g what about having the healt inspector logging on a different file? | 23:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!