*** doug-fish has quit IRC | 00:00 | |
*** ducttape_ has quit IRC | 00:03 | |
*** apuimedo has quit IRC | 00:14 | |
*** yamamoto has quit IRC | 00:21 | |
*** diogogmt has quit IRC | 00:21 | |
*** yamamoto has joined #openstack-lbaas | 00:22 | |
*** apuimedo has joined #openstack-lbaas | 00:26 | |
rm_work | sbalukoff: had to run earlier, injured myself doing a "quick" household task ... but i also am planning to basically spend the whole summit on architectural discussions for "moving forward", as not many of the talks actually interest me | 00:34 |
---|---|---|
rm_work | i'm primarily there for the planning and such for our stuff | 00:34 |
rm_work | so if we can find rooms | 00:34 |
rm_work | i'm down for a 4 day design summit... | 00:34 |
*** SumitNaiksatam has joined #openstack-lbaas | 00:40 | |
*** SumitNaiksatam has quit IRC | 00:40 | |
*** bharathm has quit IRC | 00:44 | |
*** bharathm has joined #openstack-lbaas | 00:47 | |
*** Alex_Stef has quit IRC | 00:51 | |
*** minwang2 has quit IRC | 00:52 | |
*** minwang2 has joined #openstack-lbaas | 00:53 | |
*** fnaval has quit IRC | 00:56 | |
*** minwang2 has quit IRC | 00:58 | |
*** bharathm has quit IRC | 01:00 | |
*** bharathm has joined #openstack-lbaas | 01:03 | |
*** Alex_Stef has joined #openstack-lbaas | 01:06 | |
*** Alex_Stef has quit IRC | 01:15 | |
*** apuimedo has quit IRC | 01:16 | |
*** mestery has quit IRC | 01:18 | |
*** Brian_shang has quit IRC | 01:20 | |
*** Brian_shang has joined #openstack-lbaas | 01:20 | |
*** bharathm has quit IRC | 01:20 | |
*** bana_k has quit IRC | 01:22 | |
*** Alex_Stef has joined #openstack-lbaas | 01:30 | |
*** Alex_Stef has quit IRC | 01:35 | |
*** ducttape_ has joined #openstack-lbaas | 01:43 | |
*** bana_k has joined #openstack-lbaas | 01:44 | |
*** Brian_shang has quit IRC | 01:50 | |
*** Brian_shang has joined #openstack-lbaas | 01:50 | |
*** Alex_Stef has joined #openstack-lbaas | 01:52 | |
*** diogogmt has joined #openstack-lbaas | 01:57 | |
sbalukoff1 | rm_work: Awesome! If not finding a room, finding a quiet corner works for me, too. | 02:01 |
*** Alex_Stef has quit IRC | 02:02 | |
*** Alex_Stef has joined #openstack-lbaas | 02:17 | |
*** ducttape_ has quit IRC | 02:26 | |
*** ducttape_ has joined #openstack-lbaas | 02:27 | |
*** yamamoto has quit IRC | 02:31 | |
*** Alex_Stef has quit IRC | 02:32 | |
*** Alex_Stef has joined #openstack-lbaas | 02:47 | |
*** vivek-ebay has quit IRC | 02:50 | |
*** KunalGandhi has quit IRC | 02:50 | |
*** Brian_shang has quit IRC | 02:51 | |
*** Brian_shang has joined #openstack-lbaas | 02:51 | |
*** Alex_Stef has quit IRC | 03:01 | |
*** yamamoto has joined #openstack-lbaas | 03:15 | |
*** Alex_Stef has joined #openstack-lbaas | 03:16 | |
*** ducttape_ has quit IRC | 03:26 | |
*** diogogmt has quit IRC | 03:27 | |
*** Alex_Stef has quit IRC | 03:28 | |
openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Add ability for octavia to allocate vip https://review.openstack.org/214058 | 03:33 |
*** Alex_Stef has joined #openstack-lbaas | 03:43 | |
*** bharathm has joined #openstack-lbaas | 04:21 | |
*** bharathm has quit IRC | 04:25 | |
*** minwang2 has joined #openstack-lbaas | 04:35 | |
*** amotoki has quit IRC | 04:57 | |
*** amotoki_ has joined #openstack-lbaas | 04:57 | |
*** minwang2 has quit IRC | 05:22 | |
*** minwang2 has joined #openstack-lbaas | 05:30 | |
*** michchap has joined #openstack-lbaas | 05:44 | |
michchap | Hi channel. I have recently upgraded from Juno to Kilo and have active lbaas v1 resources. If I swap the driver over to v2, will those resources continue to be managed? If not, is there a migration procedure? | 05:51 |
michchap | s/driver/plugin | 05:52 |
*** bana_k has quit IRC | 06:07 | |
*** macjack has joined #openstack-lbaas | 06:12 | |
*** apuimedo has joined #openstack-lbaas | 06:18 | |
*** bana_k has joined #openstack-lbaas | 06:18 | |
*** bana_k has quit IRC | 06:25 | |
*** bana_k has joined #openstack-lbaas | 06:26 | |
*** numans has joined #openstack-lbaas | 06:30 | |
*** kiran-r has joined #openstack-lbaas | 06:30 | |
*** amotoki has joined #openstack-lbaas | 06:32 | |
*** amotoki has quit IRC | 06:32 | |
*** apuimedo has quit IRC | 06:36 | |
*** minwang2 has quit IRC | 06:38 | |
*** armax has quit IRC | 06:44 | |
*** bana_k has quit IRC | 07:01 | |
openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Add ability for octavia to allocate vip https://review.openstack.org/214058 | 07:04 |
*** kobis has joined #openstack-lbaas | 07:09 | |
*** St3F_A13x has joined #openstack-lbaas | 07:14 | |
*** Alex_Stef has quit IRC | 07:16 | |
*** evgenyf has joined #openstack-lbaas | 07:22 | |
*** mhickey has joined #openstack-lbaas | 07:27 | |
*** St3F_A13x has quit IRC | 07:30 | |
*** KunalGandhi has joined #openstack-lbaas | 07:40 | |
*** KunalGan_ has joined #openstack-lbaas | 07:45 | |
*** KunalGandhi has quit IRC | 07:46 | |
*** St3F_A13x has joined #openstack-lbaas | 07:47 | |
*** amotoki_ has quit IRC | 07:55 | |
*** amotoki has joined #openstack-lbaas | 08:06 | |
*** St3F_A13x has quit IRC | 08:06 | |
*** jschwarz has joined #openstack-lbaas | 08:07 | |
*** KunalGan_ has quit IRC | 08:11 | |
*** St3F_A13x has joined #openstack-lbaas | 08:23 | |
*** kobis has quit IRC | 08:33 | |
*** St3F_A13x has quit IRC | 08:34 | |
*** kobis has joined #openstack-lbaas | 08:42 | |
*** amotoki has quit IRC | 08:43 | |
*** St3F_A13x has joined #openstack-lbaas | 08:49 | |
*** amotoki has joined #openstack-lbaas | 08:53 | |
*** kiranr has joined #openstack-lbaas | 08:57 | |
*** kiran-r has quit IRC | 08:57 | |
*** amotoki has quit IRC | 09:03 | |
*** evgenyf has quit IRC | 09:48 | |
*** evgenyf has joined #openstack-lbaas | 10:34 | |
*** numans has quit IRC | 10:53 | |
*** kiranr has quit IRC | 10:59 | |
*** numans has joined #openstack-lbaas | 11:06 | |
*** ducttape_ has joined #openstack-lbaas | 11:38 | |
*** Brian_shang has quit IRC | 11:47 | |
*** ducttape_ has quit IRC | 11:54 | |
*** numans has quit IRC | 12:01 | |
*** yamamoto has quit IRC | 12:08 | |
*** numans has joined #openstack-lbaas | 12:14 | |
*** bradjones has joined #openstack-lbaas | 12:16 | |
*** bradjones has quit IRC | 12:16 | |
*** bradjones has joined #openstack-lbaas | 12:16 | |
*** doug-fis_ has quit IRC | 12:18 | |
*** doug-fish has joined #openstack-lbaas | 12:18 | |
*** amotoki has joined #openstack-lbaas | 12:26 | |
*** ducttape_ has joined #openstack-lbaas | 12:36 | |
*** ducttape_ has quit IRC | 12:39 | |
*** mestery has joined #openstack-lbaas | 12:45 | |
*** Brian_shang has joined #openstack-lbaas | 12:48 | |
*** Brian_shang has quit IRC | 12:52 | |
*** yamamoto has joined #openstack-lbaas | 12:54 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-lbaas: Merge tag '7.0.0' https://review.openstack.org/235311 | 12:55 |
*** amotoki has quit IRC | 13:00 | |
*** mestery has quit IRC | 13:00 | |
*** mestery has joined #openstack-lbaas | 13:01 | |
*** Brian_shang has joined #openstack-lbaas | 13:03 | |
*** Brian_shang has quit IRC | 13:04 | |
*** Brian_shang has joined #openstack-lbaas | 13:05 | |
*** Brian_shang has quit IRC | 13:07 | |
*** amotoki has joined #openstack-lbaas | 13:12 | |
*** Brian_shang has joined #openstack-lbaas | 13:12 | |
*** St3F_A13x has quit IRC | 13:13 | |
*** ducttape_ has joined #openstack-lbaas | 13:13 | |
*** Brian_shang has quit IRC | 13:16 | |
*** Brian_shang has joined #openstack-lbaas | 13:19 | |
*** jschwarz has quit IRC | 13:25 | |
*** jschwarz has joined #openstack-lbaas | 13:25 | |
*** St3F_A13x has joined #openstack-lbaas | 13:27 | |
*** ducttape_ has quit IRC | 13:28 | |
*** St3F_A13x has quit IRC | 13:31 | |
*** yamamoto has quit IRC | 13:34 | |
*** macjack has quit IRC | 13:42 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-lbaas: Removed new=True argument from create_connection https://review.openstack.org/235353 | 13:46 |
*** Brian_shang has quit IRC | 13:47 | |
*** Brian_shang has joined #openstack-lbaas | 13:50 | |
*** yamamoto has joined #openstack-lbaas | 13:51 | |
*** Brian_shang has quit IRC | 13:57 | |
*** Brian_shang has joined #openstack-lbaas | 13:57 | |
*** ducttape_ has joined #openstack-lbaas | 14:07 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-lbaas: Removed new=True argument from create_connection https://review.openstack.org/235353 | 14:18 |
*** Brian_shang has quit IRC | 14:20 | |
*** TrevorV|Home has joined #openstack-lbaas | 14:25 | |
*** armax has joined #openstack-lbaas | 14:54 | |
*** Brian_shang has joined #openstack-lbaas | 14:56 | |
*** Brian_shang has quit IRC | 15:01 | |
*** amotoki has quit IRC | 15:03 | |
*** Brian_shang has joined #openstack-lbaas | 15:05 | |
*** kobis has quit IRC | 15:08 | |
*** Brian_shang has quit IRC | 15:10 | |
*** numans has quit IRC | 15:11 | |
*** bharathm has joined #openstack-lbaas | 15:20 | |
*** diogogmt has joined #openstack-lbaas | 15:22 | |
*** openstackgerrit has quit IRC | 15:31 | |
*** openstackgerrit has joined #openstack-lbaas | 15:32 | |
*** evgenyf has quit IRC | 15:44 | |
*** diogogmt has quit IRC | 15:48 | |
*** diogogmt has joined #openstack-lbaas | 15:49 | |
*** vivek-ebay has joined #openstack-lbaas | 15:51 | |
*** bana_k has joined #openstack-lbaas | 15:55 | |
*** TrevorV|Home has quit IRC | 15:56 | |
*** TrevorV has joined #openstack-lbaas | 15:58 | |
*** crc32 has joined #openstack-lbaas | 16:05 | |
*** vivek-eb_ has joined #openstack-lbaas | 16:14 | |
*** vivek-ebay has quit IRC | 16:14 | |
*** KunalGandhi has joined #openstack-lbaas | 16:23 | |
*** jschwarz has quit IRC | 16:30 | |
*** Alex_Stef has joined #openstack-lbaas | 16:33 | |
*** Alex_Stef has quit IRC | 16:38 | |
*** vivek-eb_ has quit IRC | 16:51 | |
*** blogan_ has joined #openstack-lbaas | 16:53 | |
*** fnaval has joined #openstack-lbaas | 16:55 | |
*** Alex_Stef has joined #openstack-lbaas | 16:56 | |
*** blogan__ has joined #openstack-lbaas | 16:57 | |
*** blogan_ has quit IRC | 17:00 | |
*** Alex_Stef has quit IRC | 17:01 | |
*** amotoki has joined #openstack-lbaas | 17:06 | |
*** vivek-ebay has joined #openstack-lbaas | 17:13 | |
*** bitblt has joined #openstack-lbaas | 17:13 | |
*** numans has joined #openstack-lbaas | 17:16 | |
*** yamamoto has quit IRC | 17:31 | |
*** bana_k has quit IRC | 17:31 | |
*** yamamoto has joined #openstack-lbaas | 17:32 | |
*** yamamoto has quit IRC | 17:33 | |
*** bharathm has quit IRC | 17:36 | |
*** ducttape_ has quit IRC | 17:36 | |
johnsom | sbalukoff sbalukoff1 Here is how the spec renders for me: https://drive.google.com/file/d/0Bw-iunw0Bbd2NjZJaFNnd042YlE/view?usp=sharing | 17:40 |
*** fnaval has quit IRC | 17:52 | |
*** ducttape_ has joined #openstack-lbaas | 17:59 | |
*** numans has quit IRC | 17:59 | |
*** bana_k has joined #openstack-lbaas | 18:01 | |
*** bharathm has joined #openstack-lbaas | 18:01 | |
*** SumitNaiksatam has joined #openstack-lbaas | 18:02 | |
*** amotoki has quit IRC | 18:03 | |
openstackgerrit | Carlos Garza proposed openstack/octavia: Implementing EventStreamer https://review.openstack.org/218735 | 18:07 |
*** fnaval has joined #openstack-lbaas | 18:11 | |
*** alejandrito has joined #openstack-lbaas | 18:14 | |
*** bitblt has quit IRC | 18:16 | |
*** numans has joined #openstack-lbaas | 18:17 | |
markvan | Hi, devstack question. Has there been any thought for add devstack-gate DEVSTACK_GATE_NEUTRON_LBAASV2? This would allow other infra jobs to easily start making use of LBaas v2. Or is there a plan to push for LBaas v2 as default in devstack at some point? | 18:28 |
*** ajmiller_ is now known as ajmiller | 18:29 | |
ajmiller | markvan The plan is for V2 to be the default at some point in the (hopefully not-too-distant) future, and for V1 to be deprecated. In fact the beginning of the M cycle seems like a good time to make that switch. | 18:30 |
*** yamamoto has joined #openstack-lbaas | 18:33 | |
markvan | ajmiller: yup, I would agree. So, if/when v2 becomes default, there will still be a way to enable v1 for existing testing written against that (like in Heat)? | 18:33 |
johnsom | FYI, LBaaSv1 is now deprecated: https://wiki.openstack.org/wiki/ReleaseNotes/Liberty | 18:34 |
ajmiller | markvan, yes, there is a flag in devstack that will allow V1 to be specified, even if the default changes. | 18:36 |
markvan | ajmiller: What flag is that? | 18:39 |
markvan | johnsom: so v1 will be removed at some point? N release? | 18:40 |
ajmiller | Lets make sure we are on the same page, I'm assuming your are talking about the devstack flags, right? If you enable q-lbaas, right now that is treated as LBaaS V1. | 18:40 |
johnsom | Yes, the plan is to remove it. I think we need two cycles in deprecated, so likely O release. | 18:40 |
*** woodster_ has joined #openstack-lbaas | 18:43 | |
markvan | johnsom: thx, that sounds right. | 18:43 |
markvan | ajmiller: yup, q-lbaas gives just v1 right now. Need to enable lbaas devstack plugin to get v2. | 18:44 |
ajmiller | markvan Right. There are q-lbaas, q-lbaasv1, and q-lbaasv2 flags in the plugin, and right now q-lbaas is treated as q-lbaasv1. | 18:45 |
johnsom | markvan Right now the test nodes don't have nested virtualization, so using lbaasv2 with the Octavia driver can be a bit slow. | 18:45 |
johnsom | Either the test nodes need to enable that in KVM or we get containers going before we really can switch everything over. | 18:46 |
johnsom | Otherwise qemu takes too long to boot vms for us | 18:46 |
markvan | ajmiller: yeah those flags are only in the lbaas devstack plugin, not in the devstack tree. I was referring to enabling it in the default devstack tree. | 18:47 |
ajmiller | markvan, Ahh, the ultimate plan is to remove lbaas completely from the devstack tree. | 18:48 |
markvan | ajmiller: the issue I have is that the devstack-gate does not know about this, so it takes extra steps to enable it. And that mean every gate job that wants to use it will be doing these extra steps which is ugly. | 18:48 |
markvan | johnsom: for the Heat integration tests, they are using the default devstack gate and spinning up a heat stack with LBaas | 18:49 |
markvan | ajmiller: ok, I can see removing it from the devstack tree, but gate jobs still need it, so that's why I was asking about support in the devstack-gate tree | 18:50 |
*** mhickey has quit IRC | 18:51 | |
ajmiller | Right. We ran into this when we wrote the devstack plugin last spring. Pulled the V1 setup from the devstack tree and broke everything, so we had to put some of that back. | 18:52 |
*** yamamoto has quit IRC | 18:53 | |
markvan | ajmiller: yup still many folks touching it. So, what do you think about a devstack-gate patch for something like DEVSTACK_GATE_NEUTRON_LBAAS to allow gate jobs to easily enable this devstack plugin? | 18:54 |
ajmiller | markvan It seems like that would be a step in the right direction! | 18:55 |
markvan | ajmiller: k, then I'll take a pass at something like that. thx for the chat time. | 18:56 |
*** vivek-eb_ has joined #openstack-lbaas | 18:59 | |
*** vivek-ebay has quit IRC | 19:03 | |
*** armax has quit IRC | 19:05 | |
*** SumitNaiksatam has left #openstack-lbaas | 19:05 | |
*** crc32 has quit IRC | 19:09 | |
*** fnaval has quit IRC | 19:16 | |
bana_k | Hi, I was suggested to divide this https://review.openstack.org/#/c/228598/8 commit into multiple commits, | 19:23 |
bana_k | I need to add a new folder, how does that work? | 19:24 |
bana_k | what I mean is lets say I add lbaas folder and loadbalancer.py in one commit, n I need to add pool.py in the same folder in another commit. | 19:25 |
bana_k | so just add the same folder in second commit n add first commit as depends on? | 19:25 |
*** numans has quit IRC | 19:26 | |
*** fnaval has joined #openstack-lbaas | 19:28 | |
blogan | bana_k: looks like they just want you to add other modules, not another directory | 19:30 |
blogan | bana_k: so lbaas/loadbalancer.py lbaas/listener.py lbaas/pool.py etc | 19:30 |
bana_k | blogan : Please look at the last but one comment | 19:31 |
bana_k | blogan: n comment by Sergey Kraynev just above that | 19:32 |
bana_k | I thought they want me to divide the patch into multiple ones | 19:33 |
blogan | bana_k: yeah, thats still what im understanidng that he wants | 19:33 |
blogan | bana_k: oh yeah he does | 19:33 |
bana_k | blogan: so I need multiple commits right ? | 19:33 |
blogan | bana_k: there's a number of ways to do it | 19:33 |
blogan | bana_k: yeah | 19:34 |
blogan | bana_k: would the listener.py depend on the loadbalancer.py? | 19:34 |
bana_k | blogan : yes | 19:34 |
bana_k | blogan: n pool depends on listener | 19:35 |
bana_k | blogan : chain continues | 19:35 |
blogan | okay so then just make new commits on top of the loadbalancer one | 19:36 |
bana_k | oh ok. so I ll clone the loadbalancer change n working on it? | 19:37 |
bana_k | blogan : thats what you mean ? | 19:37 |
blogan | bana_k: i mean just pull out everything except the load balancer code out of loadbalancer.py, git commit --amend that first commit | 19:38 |
blogan | bana_k: then add only the listener code you pulled out, and just do a git commit, which adds a new commit on top of the load balancer one | 19:38 |
blogan | repeat for hte rest | 19:38 |
blogan | and then when you're done, do a git review and it will create reviews for all of the new commits, and add a PS to the load baalncer one | 19:39 |
*** numans has joined #openstack-lbaas | 19:39 | |
bana_k | oh ok thanks blogan, have not done it before, will try it out. | 19:39 |
blogan | bana_k: np, you're entering the world of dependency chains, where up is down, left is right, and you will go insane | 19:41 |
blogan | bana_k: basically be careful | 19:41 |
*** armax has joined #openstack-lbaas | 19:41 | |
bana_k | blogan: he he he sure thanks, but I still dont understand why we need separate commits for each resources, they are kind of tightly coupled. | 19:42 |
blogan | bana_k: easier to review one at a time | 19:42 |
blogan | bana_k: versus 800 lines | 19:43 |
blogan | in one | 19:43 |
bana_k | blogan : hmm I see. | 19:43 |
blogan | oh and also, its just better organized id say, i dont like a huge file of code if it can be separated out into individual modules either | 19:43 |
blogan | but thats opinion, but my opinion is that i'm right :) | 19:44 |
bana_k | blogan: individual module I def agree, | 19:44 |
bana_k | blogan: separate commits are the one I am not sure of :) | 19:44 |
blogan | bana_k: yeah in that case, easier to review | 19:45 |
blogan | bana_k: more psychological than anything, big wall of text is just demoralizing | 19:45 |
bana_k | blogan: he he he agreed :). | 19:46 |
bana_k | thanks a lot for all the input. | 19:46 |
*** mestery_ has joined #openstack-lbaas | 19:53 | |
*** armax has quit IRC | 20:08 | |
*** armax has joined #openstack-lbaas | 20:11 | |
*** crc32 has joined #openstack-lbaas | 20:13 | |
*** mestery_ has quit IRC | 20:19 | |
*** crc32 has quit IRC | 20:19 | |
openstackgerrit | min wang proposed openstack/octavia: Adds scenario test https://review.openstack.org/182554 | 20:22 |
*** armax has quit IRC | 20:26 | |
*** armax has joined #openstack-lbaas | 20:31 | |
*** mestery_ has joined #openstack-lbaas | 20:32 | |
*** mestery_ has quit IRC | 20:32 | |
dougwig | johnsom: ping ? | 20:52 |
johnsom | pong | 20:53 |
johnsom | What is up dougwig? | 20:53 |
dougwig | about the review to have the neutron-lbaas/octavia minimal job use the octavia noop driver, ihar's point about reducing the number of moving parts that all of the rest of openstack incurs seems reasonable. (also ping blogan rm_work ) | 20:53 |
dougwig | you have a -1 on there, so i don't want to merge without folks here having a discussion, though. | 20:54 |
johnsom | Yeah. My only concern is we won't catch those issues like nova breakage since we aren't running actual code | 20:54 |
dougwig | can a change in neutron break that? | 20:55 |
johnsom | Let me look at the comments again, two minutes | 20:55 |
*** crc32 has joined #openstack-lbaas | 20:55 | |
johnsom | Oh, ok, so my reason for -1 was if we make this change, we should undo the other changes we made to get the actual tests to run. I.e. not leave the image download in, etc. | 20:56 |
johnsom | I also worry that neutron changes will break the hot plugging | 20:58 |
dougwig | would that be covered by other neutron tests? if that's coverage that is otherwise missing, i'll remove my +2. | 20:58 |
johnsom | blogan might be best to answer that. | 20:59 |
johnsom | I will remove my -1 if we undo the other changes in this patch. | 20:59 |
johnsom | heck, I will update the patch to do that. | 20:59 |
*** fnaval has quit IRC | 21:03 | |
blogan | johnsom, dougwig: give me a few, in a meeting | 21:05 |
rm_work | dougwig: which CR is this? | 21:07 |
johnsom | https://review.openstack.org/#/c/234633/1 | 21:07 |
dougwig | rm_work: https://review.openstack.org/#/c/234633/ | 21:07 |
rm_work | kk thx | 21:09 |
blogan | so the scenario tests do test the actual real drivers | 21:10 |
blogan | not to the full extent that we would all like, but we won't get that anyway | 21:10 |
blogan | i think its reasonable to have the minimal job use noop, we might even be able to get all the noop tests into one job, so we dont have as many jobs | 21:12 |
blogan | actually, pretty sure we'd be able to | 21:13 |
johnsom | Yeah. Ok, so I would like to see us undo those settings we put in to be able to boot vms. | 21:14 |
blogan | so then why dont we put all the current split tests we have into one job, where its just an api test for lbs, listeners, pools, members and hms | 21:16 |
blogan | keep the scenario tests | 21:16 |
rm_work | man all this testing-musical-chairs | 21:16 |
rm_work | is a little nuts | 21:16 |
blogan | and then add a job that test a minimal lb,listener,pool,hm, and member with nova boots | 21:16 |
blogan | yeah i hate it too but at this point, necessary evils, we're damned if we do damned if we dont | 21:17 |
rm_work | infra is prolly like "what are these guys doing" | 21:17 |
blogan | oh well | 21:17 |
rm_work | especially since so many of our changes have come with us begging them to fast-track | 21:17 |
blogan | these guys are dealing with requirements people heave on us | 21:17 |
*** fnaval has joined #openstack-lbaas | 21:18 | |
johnsom | I would lean towards keeping the slots | 21:18 |
*** fnaval has quit IRC | 21:22 | |
blogan | and use the real drivers? | 21:23 |
rm_work | i thought the system we use now is a decent compromise that actually gets all the testing done | 21:24 |
johnsom | I'm willing to give in on the real drivers, but given the test run window keeps getting smaller, it seems like it would be valuable to have multiple jobs defined and available. Even if we are just doing API. | 21:24 |
rm_work | and not a huge fan of testing just with noop | 21:24 |
blogan | if we use the noop drivers, there's no reason to keep the slots, if we use the real drivers and keep the slots then we can only enable a minimal set of tests, in which case the scenario tests already do that | 21:24 |
rm_work | buuuuut | 21:24 |
rm_work | like everyone is saying, as long as the real testing happens SOMEWHERE (even in 3rd-party CI) I am ok | 21:24 |
johnsom | We still need to get something going for the haproxy driver. When containers come along, there might be value in the multiple slots | 21:25 |
blogan | rm_work: no one is a fan of it, its just the best bad option we've had | 21:25 |
blogan | johnsom: haproxy driver? | 21:26 |
johnsom | Also, what about a slot for "single call" | 21:26 |
blogan | you mean magnum? | 21:26 |
blogan | or something | 21:26 |
rm_work | lol | 21:26 |
blogan | johnsom: i'd assume we'd have a happy path scenario test using that | 21:26 |
rm_work | still need to do single-call | 21:26 |
rm_work | is that on our roadmap anywhere | 21:26 |
blogan | yes | 21:26 |
rm_work | still irks me that wasn't done already | 21:26 |
rm_work | and i kinda want to just sit down and do it... but need to figure out the correct layer for it | 21:27 |
blogan | ive been talking with one of the intel guys that sits next to us, he's giving it a stab | 21:27 |
rm_work | cool | 21:27 |
blogan | if not i can do it whenever we the higher priority things done | 21:27 |
*** fnaval has joined #openstack-lbaas | 21:29 | |
blogan | johnsom, dougwig, rm_work: would yall agree that the neutron cogate minimal job should run using hte noop drivers, then we just need to figure out our own jobs? | 21:29 |
dougwig | blogan: seems reasonable to me, and i'd say we also need to run minimal in our repo for parity's sake. | 21:29 |
blogan | should = would be a better option | 21:29 |
blogan | dougwig: yeah agreed | 21:30 |
*** TrevorV has quit IRC | 21:30 | |
johnsom | I am fine moving minimal to noop. I just want to take out the funk we put in go get VMs to boot when we make this change. I don't want that stuff laying around if it's not needed | 21:31 |
johnsom | It makes my hack-y spidey senses light up. (yes, I know I put them in) | 21:32 |
rm_work | yeah i could care less what the neutron-co-gate tests | 21:32 |
rm_work | as long as it doesn't break the api calls directly | 21:32 |
rm_work | it's already its own slot | 21:32 |
rm_work | -minimal | 21:32 |
rm_work | yeah | 21:32 |
rm_work | no problems there | 21:32 |
blogan | well if you take out the image thing then that'll add anothe 5 minutes to all slots? or is it longer? | 21:33 |
*** vivek-eb_ has quit IRC | 21:33 | |
rm_work | err | 21:33 |
rm_work | yeah | 21:33 |
rm_work | all slots except the minimal stuff | 21:33 |
blogan | its not bad if its 10 or less | 21:33 |
rm_work | it's ... about 10 i think | 21:33 |
rm_work | right johnsom | 21:33 |
rm_work | ? | 21:33 |
johnsom | Yeah, it can be 10 | 21:34 |
blogan | still puts us under an hour for each slot | 21:34 |
blogan | using noops | 21:34 |
*** fnaval has quit IRC | 21:34 | |
blogan | i'd still advocate condensing all slots into one, and then adding additonal slots for high priority tests using real drivers | 21:35 |
blogan | but thats a lot of back and forth with infra so i wouldnt put that as a strong requirement | 21:36 |
johnsom | Right, minimizing what they do and hanging onto them a bit doesn't hurt much | 21:37 |
johnsom | So dougwig, blogan, rm_work what do you think about me adding to this patchset, undoing the changes for the vm boot and removing the image download? | 21:42 |
johnsom | I just worry about leaving an old image laying around that isn't really being used. | 21:43 |
dougwig | johnsom: i'd be ok with it. | 21:43 |
johnsom | Maybe we can hack it to load the cirros image as the amp image to save build time. | 21:43 |
blogan | that'd be nice, but that sounds like a lot of hacking | 21:44 |
johnsom | Undoing the changes is quick and clean. The cirros image named amphora image, not sure | 21:45 |
blogan | i mean about getting cirros to work for us | 21:45 |
johnsom | I didn't say it would work | 21:46 |
johnsom | We don't really need it for noop | 21:46 |
blogan | oh i see what you mean | 21:46 |
*** fnaval has joined #openstack-lbaas | 21:46 | |
johnsom | building a working cirros image is painful. I looked at that once | 21:46 |
blogan | lol yeah thats what i would suspect | 21:46 |
blogan | you looked once and just regained your eyesight | 21:46 |
rm_work | johnsom: err, why would we even need *an* image for noop? | 21:48 |
rm_work | can't we just avoid building the amps entirely? isn't that the point of noop? | 21:49 |
rm_work | maybe I am misunderstanding noop | 21:49 |
johnsom | Because otherwise devstack builds one | 21:49 |
rm_work | noop to me means "no interaction with neutron or nova" | 21:49 |
rm_work | so | 21:49 |
rm_work | we don't need to hack cirros to work | 21:49 |
blogan | rm_work: yeah its not octavia needing it | 21:49 |
rm_work | we just need to not build/add an image at all | 21:49 |
blogan | he's not saying to hack cirros to work | 21:49 |
rm_work | yeah | 21:50 |
rm_work | but like | 21:50 |
rm_work | to load up | 21:50 |
johnsom | So add a flag to the devstack scripts that the gate test inserts that bypasses the image build? | 21:50 |
rm_work | but there isn't even a point in going that far is there? | 21:50 |
rm_work | johnsom: that would work | 21:50 |
johnsom | Seems like we touch less by creating a ghost image. I don't know | 21:50 |
rm_work | hmm | 21:54 |
rm_work | I just feel like it's a bit dirty to do that kind of hack to clone the cirros image and add it again | 21:54 |
rm_work | just to avoid an if/else block or so in the gate script | 21:54 |
johnsom | Ok, so let's do two patches. This one, cleaned up, remove the download, set a flag. Second one to modify octavia to care about the flag | 21:56 |
rm_work | the octavia one would need to be first, right? | 21:57 |
johnsom | No, it was just build an image, so it doesn't really matter the order | 21:57 |
*** vivek-ebay has joined #openstack-lbaas | 21:57 | |
rm_work | hmm | 21:57 |
rm_work | k | 21:57 |
rm_work | maybe i need to see what you're talking about | 21:58 |
openstackgerrit | Merged openstack/neutron-lbaas: Updated from global requirements https://review.openstack.org/235441 | 21:59 |
*** mestery has quit IRC | 22:08 | |
openstackgerrit | Merged openstack/octavia: Add .eggs/ to .gitignore https://review.openstack.org/231128 | 22:10 |
openstackgerrit | Merged openstack/octavia: Minor restructuring of documentation tree https://review.openstack.org/230799 | 22:14 |
openstackgerrit | Michael Johnson proposed openstack/neutron-lbaas: Use noop octavia driver for minimal job too https://review.openstack.org/234633 | 22:28 |
johnsom | rm_work blogan ^^ updated version that removes the amp boot stuff and sets a variable. | 22:29 |
johnsom | I'm pretty sure that variable set will work. | 22:29 |
rm_work | k | 22:29 |
rm_work | johnsom: i don't think pools was supposed to go away? | 22:32 |
rm_work | or did i miss something | 22:32 |
johnsom | I didn't make that change | 22:32 |
rm_work | err | 22:32 |
rm_work | wat | 22:32 |
johnsom | Ihar made that change, I didn't | 22:33 |
rm_work | base -> patch2, lost pools | 22:33 |
rm_work | weird... uh ok | 22:33 |
rm_work | is it... right? i don't see why we'd drop pools | 22:33 |
johnsom | Yeah, I only did 1->2 | 22:33 |
rm_work | ok | 22:33 |
rm_work | i'm a little confused then still, because we still want pools right? | 22:33 |
rm_work | that didn't go away while i was stuck on internal stuff, did it? | 22:34 |
johnsom | I guess I'm not seeing where he removed pool | 22:34 |
johnsom | Wait, now I am confused | 22:34 |
johnsom | He added minimal to the apiv2 testenv | 22:35 |
johnsom | right? | 22:35 |
johnsom | rm_work I commented on the backing file comment. | 22:37 |
johnsom | It was added so there was enough disk allocated in devstack to allow an amp to boot | 22:37 |
johnsom | I added this stuff here: https://review.openstack.org/#/c/223265/16/neutron_lbaas/tests/contrib/gate_hook.sh | 22:38 |
rm_work | ah i think i understand | 22:38 |
rm_work | I was thinking we were leaving the others | 22:38 |
rm_work | and JUST switching the minimal to use noop | 22:38 |
rm_work | but i see you said "too" in the commit message | 22:39 |
rm_work | so NONE of these would use octavia anymore? | 22:39 |
johnsom | I think that is what is being proposed here, yes. All noop | 22:39 |
rm_work | ah | 22:40 |
rm_work | that's a lot harder decision then | 22:40 |
rm_work | I was willing to say without any hesitation that the minimal jobs should absolutely be noop | 22:40 |
johnsom | I haven't traced back the setting Ihar made yet to fully understand that change | 22:40 |
rm_work | but | 22:40 |
johnsom | They already switched all of the others to noop | 22:40 |
rm_work | you do at least see where he removed "pool" right? | 22:40 |
rm_work | I think removing pool is a mistake (like, a typo) | 22:41 |
rm_work | unintentional | 22:41 |
johnsom | The other patchset was here: https://review.openstack.org/#/c/226370/ | 22:41 |
rm_work | he probably meant to ADD minimal to that list | 22:42 |
rm_work | oh these were split out differently again than when I set it up originally | 22:43 |
rm_work | I wasn't using that key slot | 22:43 |
johnsom | rm_work No, I don't see pool removed.... | 22:43 |
johnsom | Download the file | 22:43 |
rm_work | line 13 | 22:43 |
rm_work | hmm | 22:43 |
johnsom | if [ "$2" = "healthmonitor" ] || [ "$2" = "listener" ] || [ "$2" = "loadbalancer" ] || [ "$2" = "member" ] || [ "$2" = "minimal" ] || [ "$2" = "pool" ]; then | 22:43 |
rm_work | AH yeah i just copy/pasted it out and i see it | 22:44 |
rm_work | it's freaking gerrit | 22:44 |
rm_work | won't let me scroll that far to the right | 22:44 |
johnsom | Oh, strange | 22:44 |
rm_work | it cuts it right after "minimal" | 22:44 |
rm_work | k | 22:44 |
johnsom | Mine line wraps | 22:44 |
rm_work | k | 22:45 |
rm_work | yeah mine does not | 22:45 |
rm_work | i think it's new vs. old theme | 22:45 |
rm_work | new theme doesn't linewrap | 22:45 |
rm_work | but it helps a shitton with comment tracking | 22:45 |
rm_work | and looks a bit cleaner to me in general | 22:45 |
rm_work | never notice the linewrap thing because i'm always looking at PEP-8 Python ;P | 22:46 |
rm_work | blah I keep trying to dig into min's review for cert rotation | 22:47 |
rm_work | and keep getting sidetracked | 22:47 |
johnsom | Yeah, it's a long one. I have been through the code, but still need to look at the tests | 22:47 |
rm_work | I think I've been through it about 5 times now | 22:49 |
rm_work | but it is always so long and i don't know what changed so i have to start over... | 22:49 |
*** minwang2 has joined #openstack-lbaas | 22:50 | |
rm_work | err was there a meeting today for lab related stuff? | 22:53 |
rm_work | johnsom / xgerman | 22:53 |
johnsom | Yesterday, you called in | 22:53 |
rm_work | I was in the middle of a bunch of stuff yesterday and was barely able to hear or pay attention to the meeting | 22:53 |
rm_work | i thought i heard something about scheduling a followup | 22:54 |
rm_work | for a different time | 22:54 |
rm_work | but maybe not? | 22:54 |
johnsom | Next week. | 22:54 |
johnsom | 10/21 3:30 pacific | 22:55 |
johnsom | You should have got an invite on your gmail | 22:55 |
rm_work | kk | 22:56 |
*** bharathm has quit IRC | 22:56 | |
*** bharathm has joined #openstack-lbaas | 22:56 | |
rm_work | yeah cool, i see it | 22:57 |
xgerman | yep, and I have you down for the trouble shooting section | 22:58 |
rm_work | lol k awesome | 22:59 |
xgerman | https://docs.google.com/presentation/d/132wYD2XsH0uhkotzrgs2ZpH-FMi1OkDEpnxyW1_jnek/edit#slide=id.gbee66daf0_7_41 | 22:59 |
xgerman | I made a very purple design ;-) | 22:59 |
rm_work | heh | 22:59 |
rm_work | so am I down for WRITING the troubleshooting section, or just presenting it? :P | 22:59 |
xgerman | both | 22:59 |
rm_work | i was thinking when you said troubleshooting that i'd be in charging of running around the lab like a madman literally helping people troubleshoot | 23:00 |
rm_work | LOL ok we'll see how this goes... | 23:00 |
xgerman | yeah, I think it will be collaborative since we all have seen different problems… | 23:01 |
rm_work | xgerman: can you add my rackspace email as an editor | 23:01 |
rm_work | i don't sign in to my personal gmail on my work machine | 23:01 |
xgerman | ok, I guessed an e-mail — let me know if it wporked | 23:02 |
rm_work | heh | 23:02 |
rm_work | it did :P | 23:02 |
xgerman | you guys are almost as predictable as us | 23:02 |
rm_work | it's a pretty tried and true format | 23:02 |
xgerman | yep | 23:02 |
*** crc32 has quit IRC | 23:04 | |
*** chlong has quit IRC | 23:16 | |
*** rm_you|wtf has joined #openstack-lbaas | 23:25 | |
*** Kiall has quit IRC | 23:30 | |
*** _laco has quit IRC | 23:30 | |
*** Kiall has joined #openstack-lbaas | 23:31 | |
*** rm_you| has quit IRC | 23:32 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/octavia: Updated from global requirements https://review.openstack.org/233833 | 23:32 |
*** markvan has quit IRC | 23:34 | |
*** markvan has joined #openstack-lbaas | 23:35 | |
*** _laco_ has joined #openstack-lbaas | 23:44 | |
*** bharathm has quit IRC | 23:54 | |
*** bradjones has quit IRC | 23:54 | |
*** ajmiller has quit IRC | 23:54 | |
*** madhu_ak has quit IRC | 23:54 | |
*** blogan__ has quit IRC | 23:54 | |
*** davidlenwell has quit IRC | 23:54 | |
*** sbalukoff1 has quit IRC | 23:54 | |
*** xgerman has quit IRC | 23:54 | |
*** zigo has quit IRC | 23:54 | |
*** michchap has quit IRC | 23:54 | |
*** clev-away has quit IRC | 23:54 | |
*** vivek-ebay has quit IRC | 23:54 | |
*** telmich has quit IRC | 23:54 | |
*** codekobe has quit IRC | 23:54 | |
*** rm_you|wtf has quit IRC | 23:54 | |
*** armax has quit IRC | 23:54 | |
*** diogogmt has quit IRC | 23:54 | |
*** doug-fish has quit IRC | 23:54 | |
*** mugsie has quit IRC | 23:54 | |
*** shakamunyi has quit IRC | 23:54 | |
*** Kiall has quit IRC | 23:54 | |
*** sbalukoff has quit IRC | 23:54 | |
*** rm_work has quit IRC | 23:54 | |
*** adam_g has quit IRC | 23:54 | |
*** kbyrne has quit IRC | 23:54 | |
*** markvan has quit IRC | 23:54 | |
*** bana_k has quit IRC | 23:54 | |
*** ducttape_ has quit IRC | 23:54 | |
*** thomasem_ has quit IRC | 23:54 | |
*** jorgem has quit IRC | 23:54 | |
*** logan2 has quit IRC | 23:55 | |
*** yuanying has quit IRC | 23:55 | |
*** dougwig has quit IRC | 23:55 | |
*** alejandrito has quit IRC | 23:55 | |
*** openstackgerrit has quit IRC | 23:55 | |
*** ctracey has quit IRC | 23:55 | |
*** amit213 has quit IRC | 23:55 | |
*** kfox1111 has quit IRC | 23:55 | |
*** _laco_ has quit IRC | 23:55 | |
*** numans has quit IRC | 23:55 | |
*** blogan has quit IRC | 23:55 | |
*** HenryG has quit IRC | 23:55 | |
*** eezhova has quit IRC | 23:55 | |
*** pckizer has quit IRC | 23:55 | |
*** woodster_ has quit IRC | 23:55 | |
*** sballe has quit IRC | 23:55 | |
*** clduser has quit IRC | 23:55 | |
*** sc68cal has quit IRC | 23:55 | |
*** pc-pothole has quit IRC | 23:55 | |
*** KunalGandhi has quit IRC | 23:55 | |
*** minwang2 has quit IRC | 23:55 | |
*** bedis has quit IRC | 23:55 | |
*** harlowja has quit IRC | 23:55 | |
*** redrobot has quit IRC | 23:55 | |
*** Dave has quit IRC | 23:55 | |
*** intr1nsic has quit IRC | 23:55 | |
*** crc32|znc has quit IRC | 23:55 | |
*** raginbajin has quit IRC | 23:55 | |
*** johnsom has quit IRC | 23:55 | |
*** divya has quit IRC | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!