openstackgerrit | Steve McLellan proposed openstack/searchlight: Add neutron subnets and routers https://review.openstack.org/293771 | 00:59 |
---|---|---|
*** bpokorny has quit IRC | 01:04 | |
*** lakshmiS has joined #openstack-searchlight | 01:52 | |
*** lei-zh has joined #openstack-searchlight | 02:07 | |
*** tsufiev has quit IRC | 02:19 | |
*** tsufiev has joined #openstack-searchlight | 02:25 | |
openstackgerrit | Lakshmi N Sampath proposed openstack/searchlight: Swift plugin https://review.openstack.org/271622 | 02:43 |
TravT | lakshmiS: can you take a look at the subnets and routers patch? | 02:43 |
lakshmiS | ok | 02:44 |
*** GB21 has joined #openstack-searchlight | 03:09 | |
*** GB21 has quit IRC | 03:15 | |
*** yingjun has joined #openstack-searchlight | 03:55 | |
*** yingjun has quit IRC | 03:57 | |
openstackgerrit | Merged openstack/searchlight: Swift plugin https://review.openstack.org/271622 | 05:06 |
*** lakshmiS has quit IRC | 05:13 | |
*** lakshmiS has joined #openstack-searchlight | 05:18 | |
*** GB21 has joined #openstack-searchlight | 05:22 | |
*** GB21 has quit IRC | 05:31 | |
*** GB21 has joined #openstack-searchlight | 05:33 | |
*** GB21 has quit IRC | 05:51 | |
*** GB21 has joined #openstack-searchlight | 05:51 | |
*** GB21 has quit IRC | 05:57 | |
*** GB21 has joined #openstack-searchlight | 05:59 | |
*** lakshmiS has quit IRC | 06:01 | |
*** itisha has quit IRC | 06:19 | |
*** GB21 has quit IRC | 06:19 | |
*** GB21 has joined #openstack-searchlight | 06:21 | |
*** GB21 has quit IRC | 06:49 | |
*** GB21 has joined #openstack-searchlight | 06:49 | |
*** GB21 has quit IRC | 07:13 | |
*** GB21 has joined #openstack-searchlight | 07:13 | |
*** GB21 has quit IRC | 07:24 | |
*** GB21 has joined #openstack-searchlight | 08:00 | |
*** GB21 has quit IRC | 08:15 | |
*** GB21 has joined #openstack-searchlight | 08:15 | |
*** pcaruana has joined #openstack-searchlight | 08:24 | |
*** GB21 has quit IRC | 08:28 | |
*** GB21 has joined #openstack-searchlight | 08:30 | |
*** GB21 has quit IRC | 09:04 | |
*** GB21 has joined #openstack-searchlight | 09:28 | |
*** preetika has joined #openstack-searchlight | 09:36 | |
*** GB21 has quit IRC | 09:44 | |
*** lei-zh has quit IRC | 09:57 | |
*** GB21 has joined #openstack-searchlight | 10:11 | |
*** GB21 has quit IRC | 10:52 | |
*** GB21 has joined #openstack-searchlight | 10:52 | |
*** sjmc7 has quit IRC | 10:56 | |
*** sjmc7 has joined #openstack-searchlight | 10:57 | |
*** GB21 has quit IRC | 10:58 | |
*** GB21 has joined #openstack-searchlight | 10:58 | |
*** GB21 has quit IRC | 11:07 | |
*** GB21 has joined #openstack-searchlight | 11:08 | |
*** preetika has quit IRC | 11:37 | |
*** GB21 has quit IRC | 11:47 | |
*** openstackgerrit has quit IRC | 11:48 | |
*** GB21 has joined #openstack-searchlight | 11:48 | |
*** openstackgerrit has joined #openstack-searchlight | 11:48 | |
*** GB21 has quit IRC | 11:53 | |
*** GB21 has joined #openstack-searchlight | 11:55 | |
*** GB21 has quit IRC | 12:01 | |
*** GB21 has joined #openstack-searchlight | 12:03 | |
*** GB21 has quit IRC | 12:24 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 13:47 | |
*** lakshmiS_ has joined #openstack-searchlight | 14:25 | |
*** GB21 has joined #openstack-searchlight | 14:25 | |
*** pcaruana has quit IRC | 14:28 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 14:40 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 14:41 | |
*** lei-zh has joined #openstack-searchlight | 14:53 | |
TravT | Courtesy Searchlight meeting reminder in one minute in #openstack-meeting-4: lakshmiS, nikhil_k, rosmaita, TravT, david-lyle, sjmc7, abhijeetm, itisha, GB21, briancline, lei-zh, yingjun, RickA-HP | 14:59 |
*** yingjun has joined #openstack-searchlight | 15:00 | |
*** GB21 has quit IRC | 15:15 | |
*** bpokorny has joined #openstack-searchlight | 15:50 | |
*** lakshmiS_ has left #openstack-searchlight | 15:54 | |
*** yingjun has quit IRC | 15:56 | |
*** lei-zh has left #openstack-searchlight | 15:56 | |
*** yingjun has joined #openstack-searchlight | 15:56 | |
*** khushbu has joined #openstack-searchlight | 15:56 | |
*** yingjun has quit IRC | 16:01 | |
*** khushbu has quit IRC | 16:02 | |
*** khushbu has joined #openstack-searchlight | 16:14 | |
*** pcaruana has joined #openstack-searchlight | 16:22 | |
*** lakshmiS has joined #openstack-searchlight | 16:24 | |
khushbu | hi nikhil | 16:28 |
khushbu | I am writing a heat stack plugin | 16:28 |
khushbu | for a bug | 16:28 |
openstackgerrit | Travis Tripp proposed openstack/searchlight: Simple Script for Generating Resources https://review.openstack.org/230697 | 16:29 |
*** itisha has joined #openstack-searchlight | 16:32 | |
*** khushbu has quit IRC | 16:33 | |
*** preetika has joined #openstack-searchlight | 16:36 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 16:45 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 16:45 | |
openstackgerrit | Rick Aulino proposed openstack/searchlight: Fix duplicate entries displayed w/"index sync". https://review.openstack.org/294168 | 16:55 |
openstackgerrit | Rick Aulino proposed openstack/searchlight: Fix duplicate entries displayed w/"index sync". https://review.openstack.org/294168 | 17:01 |
david-lyle | TravT: are you sure you don't want independent for release model on searchlight-ui? | 17:21 |
david-lyle | most other plugins follow that | 17:21 |
david-lyle | aside from trove and sahara | 17:23 |
david-lyle | but those are more mature | 17:24 |
david-lyle | just a thought | 17:24 |
TravT | david-lyle: that might be a good idea. | 17:38 |
TravT | i want to be able to state X release is compatible with X release of openstack | 17:39 |
sjmc7 | lakshmiS: about neutron interface add… i don’t see the router change when i add a port | 17:39 |
TravT | and i think it may quickly get out of sync. | 17:40 |
lakshmiS | ok thats good then it jsut the port create | 17:40 |
sjmc7 | i’m not sure what ‘routes’ is | 17:40 |
sjmc7 | after a week of looking at neutron i’m still partially baffled :) | 17:40 |
TravT | i guess the first time it depends on a horizon change in the next release, we'd branch stable | 17:40 |
sjmc7 | the subnet doesn’t seem to change either | 17:40 |
lakshmiS | so what field in router has the information about its interfaces? | 17:41 |
sjmc7 | none | 17:41 |
sjmc7 | there’s a separate API command for it | 17:41 |
sjmc7 | it seems to be port-based | 17:41 |
sjmc7 | the DHCP ports are also a bit of a problem; not sure where they come from | 17:42 |
sjmc7 | i guess from subnet creation | 17:42 |
sjmc7 | will check that too. but neutron seems to get everything off the port info | 17:42 |
sjmc7 | ‘device_owner’ gets set differently for these magic ports | 17:42 |
lakshmiS | yeah the router interface could be either subnet or port | 17:45 |
sjmc7 | right, but the subnet is just a shortcut i think | 17:45 |
sjmc7 | it sets up a port for you | 17:45 |
sjmc7 | versus using an existing one | 17:46 |
sjmc7 | we’ll have to test it a bit more next week to see if there are any crucial edge cases | 17:46 |
sjmc7 | i’m about 95% sure i’ve got it right | 17:46 |
lakshmiS | would be good if we can get a neutron core or sme to review it too | 17:46 |
sjmc7 | rick might be friendly with some of our neutron folks in FtC | 17:48 |
sjmc7 | i’ll add some documentation too noting any odd bits | 17:49 |
sjmc7 | good catch on the ha flag too | 17:57 |
sjmc7 | weird that that one’s not shown | 17:57 |
*** sigmavirus24 is now known as sigmavirus24_awa | 18:00 | |
lakshmiS | i am sure the admin fields might be hidden in some neutron code like swift did. that might be a better souce to find out than staring at cmd line output :) | 18:08 |
sjmc7 | i diffed the API output :) | 18:09 |
sjmc7 | yeah, i’m sure it is somewhere | 18:09 |
TravT | well too devstack failures in a row... | 18:10 |
TravT | two | 18:10 |
TravT | geez | 18:10 |
sjmc7 | :( | 18:10 |
TravT | 2016-03-17 18:02:13.963 | ovs-ofctl: br-tun is not a bridge or a socket | 18:10 |
TravT | 2016-03-17 18:02:13.983 | ovs-ofctl: br-tun is not a bridge or a socket | 18:10 |
lakshmiS | you are being lucky | 18:10 |
sjmc7 | you should fix your brtun | 18:11 |
TravT | get your own brtun | 18:11 |
sjmc7 | i’ve got plenty! | 18:11 |
TravT | well, i'm gonna go back to old stack and just do an offline update with latest from horizon and searchlight. | 18:12 |
TravT | i have an update to the local.conf example i think i'll push shortly | 18:12 |
*** nehap has joined #openstack-searchlight | 18:12 | |
nehap | Hi sjmc7 | 18:15 |
sjmc7 | hello | 18:16 |
nehap | sjmc7: Can you please review this https://review.openstack.org/#/c/288853/ and let me know if I am going in the right direction | 18:17 |
*** pcaruana has quit IRC | 18:18 | |
sjmc7 | yeah, it’s the right direction. we won’t merge this for the mitaka release, but i’ll try to review it once we’re done with the remaining mitaka work | 18:18 |
nehap | sjmc7: Thanks. For my outreachy application, should I work on some other bug as well | 18:20 |
nehap | sjmc7: Is there any other easy bug I should work on | 18:22 |
TravT | nehap, let me take a look | 18:22 |
sjmc7 | just looking at the bug list | 18:22 |
sjmc7 | one sec | 18:22 |
sjmc7 | https://bugs.launchpad.net/searchlight/+bug/1558606 shouldn’t be too hard | 18:23 |
openstack | Launchpad bug 1558606 in OpenStack Search (Searchlight) "Remove duplicate project/tenant facets from cinder plugins" [Low,New] | 18:23 |
sjmc7 | https://bugs.launchpad.net/searchlight/+bug/1550494 | 18:23 |
openstack | Launchpad bug 1550494 in OpenStack Search (Searchlight) "Generated version numbers are returned as strings" [Low,New] - Assigned to Ruchi Parmar (ruchiparmar143) | 18:23 |
sjmc7 | you may also discover your own bugs :) | 18:25 |
nehap | sjmc7: I had looked at the bug https://bugs.launchpad.net/searchlight/+bug/1550494 before | 18:26 |
openstack | Launchpad bug 1550494 in OpenStack Search (Searchlight) "Generated version numbers are returned as strings" [Low,New] - Assigned to Ruchi Parmar (ruchiparmar143) | 18:26 |
nehap | sjmc7: I just needed one clarification: truncate_timestamp should be returned as string. Am I right | 18:27 |
*** sigmavirus24_awa is now known as sigmavirus24 | 18:28 | |
sjmc7 | umm.. maybe? truncate_timestamp is a string. the function should return a number rather than a string | 18:28 |
*** pcaruana has joined #openstack-searchlight | 18:30 | |
nehap | sjmc7: ok, thanks a lot | 18:33 |
sjmc7 | sure. sorry we’re so busy. things should be a bit less crazy next week | 18:34 |
sjmc7 | release weeks are always a scramble | 18:35 |
openstackgerrit | Steve McLellan proposed openstack/searchlight: Add neutron subnets and routers https://review.openstack.org/293771 | 18:40 |
sjmc7 | lakshmiS: put a new patch up. i need to eat something, and i want to take a look at dhcp port events. i’ll put a -1 workflow on for now | 18:41 |
lakshmiS | ok sounds good | 18:42 |
lakshmiS | will check it out later | 18:43 |
sjmc7 | everything should be working though | 18:46 |
sjmc7 | just tested interface create/delete | 18:46 |
*** nehap has quit IRC | 18:48 | |
*** nehap has joined #openstack-searchlight | 18:52 | |
*** nehap has quit IRC | 18:56 | |
*** yingjun has joined #openstack-searchlight | 18:56 | |
*** nehap has joined #openstack-searchlight | 18:58 | |
*** yingjun has quit IRC | 19:01 | |
openstackgerrit | Rick Aulino proposed openstack/searchlight: Zero Downtime Re-indexing Error Handling. https://review.openstack.org/293079 | 19:03 |
*** pcaruana has quit IRC | 20:17 | |
openstackgerrit | Steve McLellan proposed openstack/searchlight: Add neutron subnets and routers https://review.openstack.org/293771 | 20:23 |
sjmc7 | ^ is good for reviews. i’m gonna figure out DHCP ports as a separate patch; it’s kind of a bug against networks & ports | 20:28 |
*** pcaruana has joined #openstack-searchlight | 20:30 | |
*** matt-borland has joined #openstack-searchlight | 20:32 | |
matt-borland | o/ | 20:32 |
sjmc7 | TravT, matt-borland : got a neutron question | 20:32 |
matt-borland | k | 20:32 |
TravT | hey | 20:33 |
sjmc7 | there are some neutron ports created to implement magical things like router interfaces and DHCP | 20:33 |
sjmc7 | i don’t *think* horizon shows these | 20:33 |
sjmc7 | and i’m wondering whether instead of cobbling hacks together to implement them, we should ignore them | 20:33 |
TravT | hmm | 20:33 |
sjmc7 | oo, wait, i’m wrong, it does show them | 20:34 |
sjmc7 | grrr | 20:34 |
TravT | i think you see them in the network details page | 20:34 |
matt-borland | I seem to recall a few magical ports showing up in Horizo... | 20:34 |
matt-borland | yeah. | 20:34 |
sjmc7 | that’s unfortunate. they seem like useless information | 20:34 |
sjmc7 | oh well. i guess i’ll actually have to do some work for once | 20:34 |
matt-borland | from my perspective, we're here to write what is useful | 20:34 |
sjmc7 | sorry to interrupt you, i should’ve checked more carefully | 20:34 |
TravT | yeah, what's wrong with you sjmc7... get back in your dungeon | 20:35 |
sjmc7 | yes master | 20:35 |
matt-borland | now the entirety of Young Frankenstein is playing in my head | 20:35 |
sjmc7 | yeah, maybe the UI would choose to hide them but i think we have to provide the option | 20:36 |
sjmc7 | haha, now THAT is torture | 20:36 |
matt-borland | WHAT? | 20:36 |
sjmc7 | TravT: the router patch includes magic router ports but i’m gonna do DHCP separately because it’s hard | 20:36 |
* matt-borland loves that movie | 20:36 | |
*** matt-borland has left #openstack-searchlight | 20:36 | |
TravT | that was interesting cameo from matt | 20:37 |
TravT | sjmc7: that's fine. | 20:37 |
sjmc7 | i summoned him | 20:39 |
sjmc7 | beetlejuice! | 20:39 |
openstackgerrit | Neha proposed openstack/searchlight: Remove duplicate project facets from cinder plugins Closes-Bug: #1558606 https://review.openstack.org/294285 | 21:05 |
openstack | bug 1558606 in OpenStack Search (Searchlight) "Remove duplicate project/tenant facets from cinder plugins" [Low,New] https://launchpad.net/bugs/1558606 - Assigned to Neha (neha-m-pande) | 21:05 |
TravT | @lakshmiS @sjmc7 | 21:17 |
sjmc7 | lakshmiS: good catch on the neutron policy stuff | 21:17 |
TravT | just read that as well. | 21:17 |
TravT | i think that adds additional sharing | 21:17 |
sjmc7 | i’m not sure about trying to implement that at this stage | 21:18 |
sjmc7 | i think i have missed some public/private rules, which i’ll look at | 21:18 |
sjmc7 | i think though for newton we need to standardize how we’re doing additional rbac | 21:19 |
TravT | the extra sharing can be put in as a bug. | 21:19 |
lakshmiS | looks like a pattern on shared objects across services | 21:19 |
TravT | security is always high and we have until mar 28. | 21:19 |
lakshmiS | atleast neutron is less fine grained than swift | 21:19 |
sjmc7 | yeah, they’re all basically the same tenant:user/tenant:* | 21:20 |
sjmc7 | yeah | 21:20 |
sjmc7 | let’s make that a target for next release. i think public/shared is just based on the network state? | 21:20 |
TravT | i think base feature that is more restrictive can go in. | 21:20 |
sjmc7 | TravT: can you file a bug for ports to alter the rbac to include ports on public networks? | 21:21 |
sjmc7 | if indeed that is correct | 21:21 |
TravT | yep | 21:21 |
sjmc7 | which i’m not 100% sure about | 21:21 |
sjmc7 | i’ll check for subnets too | 21:21 |
TravT | hmm, really | 21:21 |
sjmc7 | it sounds sensible, i think? | 21:21 |
lakshmiS | i think ports are not shared | 21:21 |
sjmc7 | hmmm | 21:22 |
sjmc7 | but subnets are? | 21:22 |
lakshmiS | yes | 21:22 |
sjmc7 | how magical | 21:22 |
lakshmiS | sharing ports would be a secuirty breach | 21:22 |
TravT | lakshmiS you spent a bit of time in your past on openstack networking and are digging in more now | 21:23 |
sjmc7 | hahaha | 21:23 |
lakshmiS | who knows how it comes useful | 21:23 |
sjmc7 | :D | 21:23 |
lakshmiS | but neutron is a beast | 21:23 |
sjmc7 | yeah, i’m learning all kidns of stuff | 21:23 |
TravT | so i think i'd like to defer to your judgment on how to handle this | 21:24 |
sjmc7 | can i make a private subnet on a shared network? | 21:24 |
lakshmiS | haven't tried that | 21:24 |
TravT | my perspective is that if we support base concepts that is okay so long as the rbac policies can't make them more open | 21:24 |
lakshmiS | agree its just more restrictive | 21:25 |
sjmc7 | ok, you’re right - shared subnets are visible, ports are not | 21:25 |
sjmc7 | how wondrous! | 21:25 |
sjmc7 | ok, that’s an easy fix, good catch. yeah, it looks like those policies can only add access, not take it away | 21:26 |
lakshmiS | right | 21:26 |
lakshmiS | separate bug would be ok for it | 21:26 |
sjmc7 | give me 10 minutes to fix the subnet sharing | 21:28 |
lakshmiS | have a hard stop. will check it out in evening | 21:28 |
TravT | if everybody actually could use the same policy concepts as keystone, it seems all this sharing could just be implemented as a common database schema and library | 21:29 |
sjmc7 | yeah, i’ve got to go in about 45 minutes | 21:29 |
TravT | yeah, that's fine. | 21:29 |
sjmc7 | yeah.. like the quota mailing list discussion | 21:29 |
sjmc7 | it’s mad everyone does everything themeslves | 21:29 |
TravT | i'm not feeling particularly well today... | 21:30 |
TravT | probably going to drop off soon as well | 21:30 |
lakshmiS | we should introduce common rbac client since we are doing it for everyone anyway :) | 21:30 |
*** nehap has quit IRC | 21:30 | |
TravT | there already is one... | 21:30 |
lakshmiS | ? | 21:30 |
TravT | policy enforcer | 21:30 |
TravT | but it has to be done on post result filtering | 21:30 |
TravT | which just don't work for performance | 21:31 |
TravT | the keystone policy language is pretty powerful. | 21:31 |
lakshmiS | oh yeah, that needs more logic which is a little difficult with some projects like glance but should be converged | 21:31 |
TravT | anyway | 21:31 |
TravT | great work guys | 21:32 |
TravT | lakshmi, definitely please give another round of reviews later. | 21:32 |
lakshmiS | sure | 21:32 |
TravT | once sjmc7 posts, i'll also take a look | 21:32 |
TravT | i have some ui bugs i'm trying to figure out as well. | 21:33 |
*** lakshmiS has quit IRC | 21:34 | |
sjmc7 | ok. written the test (12 minutes), will upload after i write the code (2 minutes) :) | 21:47 |
sjmc7 | gotta love TDD! | 21:47 |
openstackgerrit | Steve McLellan proposed openstack/searchlight: Add neutron subnets and routers https://review.openstack.org/293771 | 22:00 |
sjmc7 | TravT: subnet patch ^ | 22:01 |
TravT | okay cool. will pull now | 22:01 |
sjmc7 | sounds like ports doesn’t need altering? | 22:01 |
sjmc7 | i.e. a port on a shared network isn’t shared | 22:01 |
TravT | that's what lakshmi said | 22:01 |
TravT | but i'd rather confirms | 22:01 |
TravT | he | 22:01 |
TravT | wow | 22:01 |
sjmc7 | ok. i need to go shortly, but will just visually compare horizon to SL subnet results | 22:02 |
TravT | ok | 22:02 |
sjmc7 | if it knows what’s good for it, it’ll look right | 22:03 |
sjmc7 | otherwise… why i oughtta... | 22:03 |
sjmc7 | i get the right number of networks, so let’s call that a win | 22:04 |
sjmc7 | is it weird that after over 15 years in this industry i’m still surprised when stuff i wrote works? :) | 22:04 |
TravT | no | 22:05 |
sjmc7 | good | 22:06 |
sjmc7 | subnets look right | 22:06 |
sjmc7 | yay | 22:06 |
TravT | https://s-media-cache-ak0.pinimg.com/736x/7a/2a/2d/7a2a2d80f6155fa309fdebc739908599.jpg | 22:07 |
sjmc7 | ok, so the only remaining thing is ports for DHCP which i think will have to wait til tomorrow | 22:07 |
sjmc7 | in terms of tagging RC1, i think we could consider doing it once subnets/routers is merged | 22:08 |
sjmc7 | unless you would rather clear the last couple of criticals | 22:08 |
TravT | just looking at the list | 22:09 |
TravT | i'm not seeing anything particular worth holding the tag for after this | 22:10 |
TravT | although we have to go through stable branch fun after that | 22:10 |
TravT | master -> cherry pick | 22:10 |
sjmc7 | the duplicate entries thing we can probably get merged first thing tomorrow | 22:11 |
sjmc7 | dhcp ports i think we can do as a cherrypick | 22:11 |
TravT | duplicate entries is down to nitpicks | 22:11 |
sjmc7 | yeah, it works fine | 22:11 |
sjmc7 | now we’ve got to find whatever other bugs we created this week :) | 22:11 |
sjmc7 | ok, i should go. i’ll keep an eye out for feedback on that patch and try to get any changes done tonight | 22:12 |
TravT | ok | 22:12 |
TravT | put an ice pack on your brain | 22:13 |
TravT | it probably needs it | 22:13 |
TravT | its been going on overdrive | 22:13 |
sjmc7 | i feel like 25% of it’s dedicated to being a python compiler | 22:14 |
*** bpokorny_ has joined #openstack-searchlight | 22:31 | |
*** bpokorny has quit IRC | 22:34 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 22:40 | |
*** TravT has quit IRC | 23:11 | |
*** TravT has joined #openstack-searchlight | 23:12 | |
*** bpokorny_ has quit IRC | 23:31 | |
*** bpokorny has joined #openstack-searchlight | 23:32 | |
*** bpokorny has quit IRC | 23:32 | |
*** bpokorny has joined #openstack-searchlight | 23:33 | |
openstackgerrit | Travis Tripp proposed openstack/searchlight: Update Docs for Mitaka Release https://review.openstack.org/294333 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!