rm_work | .. having issues posting to gerrit | 00:03 |
---|---|---|
rm_work | seems to LOAD fine | 00:03 |
*** catintheroof has quit IRC | 00:03 | |
rm_work | there we go | 00:05 |
rm_work | got few 500s but it works now | 00:05 |
johnsom | Maybe it's running low on disk space again... | 00:06 |
*** ducttape_ has quit IRC | 00:08 | |
*** ducttape_ has joined #openstack-lbaas | 00:24 | |
rm_work | wtf | 00:40 |
rm_work | https://gist.github.com/rm-you/9105ff3a83106caa37097aaf07afc2a5 | 00:42 |
rm_work | different behavior for ONE of the exceptions... | 00:42 |
rm_work | I tested all 4 and only InvalidSortKey does it that way | 00:42 |
rm_work | everything else is the proper way | 00:42 |
rm_work | or, what I assume is the proper way | 00:42 |
johnsom | Umm, looking at your posted code, that is super strange | 00:43 |
rm_work | I lied, it's 50/50 | 00:43 |
johnsom | The first one looks right to me | 00:44 |
rm_work | updated the gist | 00:44 |
rm_work | refresh | 00:44 |
rm_work | I would assume the first two are right | 00:44 |
rm_work | there's no difference in the actual exception code | 00:45 |
rm_work | *where* they're called is ... maybe different? tracking it down | 00:45 |
johnsom | Yeah, the string is right. The exception code looks right | 00:46 |
*** armax has quit IRC | 00:50 | |
*** armax has joined #openstack-lbaas | 00:56 | |
*** fnaval has quit IRC | 00:57 | |
rm_work | yeah don't worry about it | 01:00 |
rm_work | hopefully you aren't :P | 01:01 |
rm_work | it's lower priority | 01:01 |
johnsom | I am testing away | 01:01 |
johnsom | Peeked in between | 01:01 |
rm_work | my guess is that it depends on WHERE I throw the exception | 01:01 |
rm_work | like, what layer | 01:02 |
rm_work | it's only marker and sort-key that are different | 01:02 |
rm_work | and those two are the ones that can be thrown in the apply() | 01:02 |
rm_work | though ... i'd EXPECT the sort-key one to be thrown above that :/ | 01:02 |
johnsom | Yeah, maybe something in between is messing with it, like a reraise or something | 01:02 |
johnsom | https://www.irccloud.com/pastebin/aJ5JNRYQ/ | 01:03 |
johnsom | So existing code isn't broken | 01:04 |
rm_work | yep it's the ones inside the apply() that don't work the same | 01:04 |
rm_work | so i'll have to trace that up | 01:04 |
*** csomerville has quit IRC | 01:04 | |
rm_work | nice | 01:04 |
rm_work | so we're good? | 01:04 |
rm_work | oh | 01:04 |
rm_work | that's just the ubuntu half of the matric | 01:04 |
rm_work | *matrix | 01:04 |
johnsom | Nope, I need to run through the same with the centos image | 01:04 |
*** ducttape_ has quit IRC | 01:06 | |
rm_work | in controllers/load_balancer.py get_all() which is the wsme_exposed method | 01:06 |
rm_work | there's no catches | 01:06 |
rm_work | all the way down to where the error is raised | 01:06 |
rm_work | so i dunno | 01:06 |
rm_work | maybe it's that there SHOULD be? | 01:06 |
rm_work | :/ | 01:06 |
rm_work | but that's definitely it... if it gets raised during the hook processing, it gives what I assume is the "correct" type of 400 response | 01:08 |
rm_work | if it gets raised during the repository processing, it gives the other one | 01:08 |
rm_work | so, either way it DOES return status 400... it's just the body of the response that's different | 01:10 |
rm_work | and I don't think that's my problem? maybe | 01:10 |
rm_work | anywho, fixed one last bug and I guess I'll sit on it for now | 01:10 |
openstackgerrit | Merged openstack/octavia: Fix active/standby under python3 https://review.openstack.org/424851 | 01:13 |
openstackgerrit | Adam Harwell proposed openstack/octavia: Implement sorting and pagination for octavia https://review.openstack.org/382147 | 01:13 |
rm_work | ah should have rebased :P | 01:14 |
rm_work | ok i'm out for a bit, might be on later johnsom so feel free to leave me notes | 01:14 |
rm_work | and by might, i mean... i'll be on later | 01:14 |
johnsom | Ok, enjoy | 01:14 |
rm_work | made some pressure cooker ribs :P #winning | 01:15 |
johnsom | NICE, garlic shrimp with pasta for me tonight | 01:15 |
*** kevo has quit IRC | 01:32 | |
openstackgerrit | Ha Dung proposed openstack/octavia: Fixes some current typo errors on octavia project. https://review.openstack.org/425485 | 01:43 |
*** catintheroof has joined #openstack-lbaas | 01:44 | |
*** gongysh has joined #openstack-lbaas | 01:49 | |
*** catintheroof has quit IRC | 01:49 | |
johnsom | Ugh, why can't the distros agree on anything | 02:03 |
johnsom | centos ifup doesn't have an -a option | 02:03 |
*** armax has quit IRC | 02:12 | |
*** ducttape_ has joined #openstack-lbaas | 02:19 | |
*** ducttape_ has quit IRC | 02:24 | |
*** fnaval has joined #openstack-lbaas | 02:37 | |
*** csomerville has joined #openstack-lbaas | 02:39 | |
johnsom | https://www.irccloud.com/pastebin/K4Oblals/ | 02:49 |
openstackgerrit | Michael Johnson proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors https://review.openstack.org/331841 | 02:50 |
*** csomerville has quit IRC | 02:52 | |
*** reedip_ has joined #openstack-lbaas | 03:00 | |
*** kevo has joined #openstack-lbaas | 03:01 | |
*** ducttape_ has joined #openstack-lbaas | 03:40 | |
*** fnaval has quit IRC | 03:56 | |
*** fnaval has joined #openstack-lbaas | 03:57 | |
*** ducttape_ has quit IRC | 04:00 | |
*** fnaval has quit IRC | 04:01 | |
*** armax has joined #openstack-lbaas | 04:10 | |
*** ducttape_ has joined #openstack-lbaas | 04:22 | |
rm_work | is that a GNU thing or... | 04:24 |
rm_work | sometimes I'm not sure which is actually more "standard" and which isn't | 04:24 |
rm_work | like, I thought OSX was non-standard on some utils but turns out it's actually that ubuntu had a bunch of extra gnu stuff that wasn't standard | 04:25 |
*** armax has quit IRC | 04:34 | |
*** ducttape_ has quit IRC | 04:34 | |
*** reedip_ has quit IRC | 05:05 | |
*** sputnik13 has quit IRC | 05:35 | |
openstackgerrit | Michael Johnson proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors https://review.openstack.org/331841 | 05:38 |
*** ducttape_ has joined #openstack-lbaas | 05:50 | |
*** eezhova has joined #openstack-lbaas | 05:52 | |
*** ducttape_ has quit IRC | 05:55 | |
*** gcheresh_ has joined #openstack-lbaas | 05:58 | |
*** sputnik13 has joined #openstack-lbaas | 06:00 | |
johnsom | https://www.irccloud.com/pastebin/IJJEcKJh/ | 06:47 |
johnsom | So, down to one issue in my testing. We will have to be creative to deal with the lack of ifup -a. | 06:50 |
*** tesseract has joined #openstack-lbaas | 07:11 | |
*** diltram has quit IRC | 07:14 | |
*** eezhova has quit IRC | 07:14 | |
*** ducttape_ has joined #openstack-lbaas | 07:21 | |
*** AlexeyAbashkin has joined #openstack-lbaas | 07:22 | |
*** ducttape_ has quit IRC | 07:26 | |
*** Alex_Staf has quit IRC | 07:28 | |
*** yamamoto has quit IRC | 07:41 | |
*** eezhova has joined #openstack-lbaas | 07:41 | |
*** yamamoto has joined #openstack-lbaas | 08:16 | |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: Active-Active Topology - Cluster DB Tasks https://review.openstack.org/425578 | 08:31 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: Active-Active Topology - Cluster DB Tasks. https://review.openstack.org/409764 | 08:46 |
*** ducttape_ has joined #openstack-lbaas | 08:48 | |
*** kevo has quit IRC | 08:50 | |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp https://review.openstack.org/425175 | 08:56 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: Active-Active Topology - register/uregister amphorae tasks. https://review.openstack.org/409765 | 08:56 |
*** Alex_Staf has joined #openstack-lbaas | 08:57 | |
*** gongysh has quit IRC | 09:10 | |
*** AlexeyAbashkin has quit IRC | 09:10 | |
*** amotoki has quit IRC | 09:27 | |
*** amotoki has joined #openstack-lbaas | 09:28 | |
*** amotoki has quit IRC | 09:32 | |
*** amotoki has joined #openstack-lbaas | 09:39 | |
rm_work | johnsom: why do we even need to run that? do netns not honor auto network configs? | 09:52 |
rm_work | IE, when we move the interface into the netns we can't just set "auto <interfacename>" at the same time? | 09:52 |
rm_work | oh, what... that's *for" bringing up auth interfaces | 09:55 |
rm_work | why doesn't that "just happen" | 09:55 |
rm_work | I have some thoughts but I need to spin that patch | 10:04 |
rm_work | i'll start it running tonight and look at it tomorrow | 10:04 |
*** amotoki has quit IRC | 10:18 | |
*** Brenda has quit IRC | 10:21 | |
*** Brenda has joined #openstack-lbaas | 10:22 | |
nmagnezi | rm_work, i think i've found a solution for what johnsom mentioned. 'ifup *' works on centos, testing if Ubuntu will not spoil the party | 10:23 |
rm_work | hah | 10:24 |
nmagnezi | :) | 10:24 |
rm_work | is it very specific? | 10:24 |
nmagnezi | rm_work, ifup * ? | 10:25 |
rm_work | ugh | 10:27 |
rm_work | macbook just kernel paniced | 10:27 |
nmagnezi | rm_work, yay for Sierra | 10:27 |
rm_work | I'm still on El Capitan | 10:28 |
rm_work | something corporate installed is my guess... | 10:28 |
nmagnezi | rm_work, yay for not upgrading :D | 10:28 |
nmagnezi | so you cannot upgrade until IT allows it? | 10:28 |
rm_work | lol not sure | 10:30 |
rm_work | prolly could? I think the VPN would still work | 10:30 |
rm_work | and that's all that matters | 10:30 |
rm_work | but it's annoying to redo everything <_< | 10:30 |
rm_work | costs me at least a day or two just to get everything set up again the way i want | 10:31 |
rm_work | aaanywho out for a bit | 10:32 |
rm_work | hopefully your fix works | 10:32 |
rm_work | it can't pick based on if -a is valid? | 10:32 |
rm_work | like, try -a and if it's bad, do * | 10:32 |
nmagnezi | # ip netns exec amphora-haproxy ifdown * | 10:36 |
nmagnezi | Unknown interface * | 10:36 |
nmagnezi | rm_work, ^^ that makes me really sad.. | 10:36 |
nmagnezi | same for ifup | 10:36 |
nmagnezi | johnsom, ^^ I will replay with findings in the patch | 10:38 |
*** openstackgerrit has quit IRC | 11:17 | |
*** catintheroof has joined #openstack-lbaas | 12:27 | |
*** catintheroof has quit IRC | 12:38 | |
*** catintheroof has joined #openstack-lbaas | 12:38 | |
*** sferna1_ has joined #openstack-lbaas | 12:41 | |
*** sanfern has quit IRC | 12:45 | |
*** Alex_Staf has quit IRC | 12:49 | |
*** catinthe_ has joined #openstack-lbaas | 12:57 | |
*** catintheroof has quit IRC | 12:59 | |
*** mugsie_ has quit IRC | 13:00 | |
*** mugsie_ has joined #openstack-lbaas | 13:02 | |
*** bedis has quit IRC | 13:05 | |
*** bedis has joined #openstack-lbaas | 13:05 | |
*** amotoki has joined #openstack-lbaas | 13:24 | |
*** openstackgerrit has joined #openstack-lbaas | 13:43 | |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp https://review.openstack.org/425175 | 13:43 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: Active-Active Topology - register/uregister amphorae tasks https://review.openstack.org/409765 | 13:43 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: Active-Active Topology - Cluster DB Tasks https://review.openstack.org/409764 | 13:43 |
*** Alex_Staf has joined #openstack-lbaas | 13:45 | |
*** ducttape_ has quit IRC | 13:53 | |
*** ducttape_ has joined #openstack-lbaas | 14:13 | |
*** matt-borland has joined #openstack-lbaas | 14:21 | |
*** _ducttape_ has joined #openstack-lbaas | 14:31 | |
*** ducttape_ has quit IRC | 14:32 | |
openstackgerrit | Sindhu Devale proposed openstack/octavia: Introduce Octavia v2 API for listeners https://review.openstack.org/405060 | 14:35 |
*** gcheresh_ has quit IRC | 14:45 | |
*** coreycb has joined #openstack-lbaas | 14:46 | |
coreycb | hello, anyone familiar with neutron-lbaas-dashboard? i'm curious why it only has stable/mitaka and master branches, and no stable/newton branch. | 14:46 |
dasm | johnsom: ^ | 14:47 |
xgerman | coreycb there was no developement in newton so no release | 14:47 |
*** beardedeagle has joined #openstack-lbaas | 14:47 | |
*** _ducttape_ has quit IRC | 14:47 | |
coreycb | xgerman, ok but how do you provide stable support for newton? | 14:48 |
xgerman | we will definitely cut a release if there are bug fixes which have been backported | 14:48 |
xgerman | but the dashboard got developed by people who since left the dashboard project so support is slim | 14:49 |
xgerman | we anticipate an O release though to catch up with horizon changes | 14:49 |
coreycb | xgerman, ok so it sounds like it is still supported, if only in maintenance mode though. | 14:50 |
xgerman | yeah, horizon itself is short handed… | 14:51 |
xgerman | I actually installed stable-mitaka on master LBaaS on devstack and it worked… so YMMV | 14:51 |
coreycb | xgerman, appreciate the info, thanks | 14:52 |
dasm | xgerman: but, afaik ankur-gupta-f1 tried to do the same, and had some issues. although i don't know the details | 14:52 |
xgerman | yeah, it’s really YMMV | 14:52 |
dasm | True :) | 14:53 |
*** sferna1_ has quit IRC | 14:55 | |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - Initial Cluster Manager https://review.openstack.org/405238 | 14:56 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp https://review.openstack.org/425175 | 14:56 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - Distributor related tasks https://review.openstack.org/406951 | 14:56 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - Distributor image creation https://review.openstack.org/403594 | 14:56 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology : create distributor network flow https://review.openstack.org/409763 | 14:56 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: Active-Active Topology - register/uregister amphorae tasks https://review.openstack.org/409765 | 14:56 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE - distributor certificate tasks https://review.openstack.org/406952 | 14:56 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: Active-Active Topology - Cluster DB Tasks https://review.openstack.org/409764 | 14:56 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - distributor creation flow https://review.openstack.org/406953 | 14:56 |
openstackgerrit | Abed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - create shared distributor https://review.openstack.org/406954 | 14:56 |
*** jamespage has joined #openstack-lbaas | 14:56 | |
coreycb | jamespage, hey was just chatting with xgerman and dasm about neutron-lbaas-dashboard | 14:56 |
* jamespage waves at xgerman and dasm | 14:57 | |
dasm | o/ | 14:57 |
jamespage | so.... | 14:57 |
jamespage | sounds like we don't really have a lbaas v2 horizon past mitaka atm? | 14:58 |
jamespage | is that accurate? | 14:58 |
coreycb | jamespage, grepping the code it looks like there is lbaas v2 support | 14:59 |
jamespage | coreycb, directly in horizon ? | 14:59 |
xgerman | jamespage just we didn’t release it doesn’t mean it doesn’t work | 15:00 |
coreycb | jamespage, i'm just looking at the lbaas-dashboard code | 15:00 |
jamespage | xgerman, sorry I did not mean to imply its not functional | 15:01 |
coreycb | xgerman, you did mention ymmv | 15:01 |
xgerman | yep | 15:01 |
coreycb | xgerman, thanks again for the info | 15:07 |
xgerman | y.w. | 15:07 |
jamespage | xgerman, is it likely that the 1.0.0 release will just work with newton/ocata? | 15:07 |
xgerman | yep | 15:08 |
jamespage | awesome | 15:08 |
jamespage | xgerman, thanks for the info :-) | 15:08 |
xgerman | our API is the same we just need to make sure Horizon doesn’t make too many breaking changes | 15:08 |
* jamespage crosses fingers | 15:12 | |
johnsom | jamespage There were not enough true changes to cut a newton release, just use the mitaka release on newton. I will be releasing for ocata however | 15:24 |
johnsom | coreycb: ^^^ | 15:24 |
coreycb | johnsom, ok thanks | 15:25 |
nmagnezi | morning johnsom | 15:25 |
johnsom | Morning | 15:26 |
nmagnezi | johnsom, I think I've found a solution to the "ifup -a" issue | 15:26 |
nmagnezi | replied in the patch | 15:26 |
johnsom | Pretty good test results. I did fix a few things to get there, but looking good | 15:26 |
nmagnezi | johnsom, we can iron those out today | 15:27 |
johnsom | That would be good. I will take a look when I get to the office | 15:27 |
nmagnezi | it is indeed too early :) | 15:27 |
nmagnezi | I plan to submit a PS, let me know if that will conflict with stuff that you do | 15:28 |
johnsom | No, I pushed my changes up last night, so go for it | 15:30 |
nmagnezi | ack | 15:31 |
*** fnaval has joined #openstack-lbaas | 15:34 | |
*** armax has joined #openstack-lbaas | 15:37 | |
*** catintheroof has joined #openstack-lbaas | 15:39 | |
*** catinthe_ has quit IRC | 15:42 | |
*** openstackgerrit has quit IRC | 15:48 | |
*** sanfern has joined #openstack-lbaas | 16:01 | |
*** trevormc has joined #openstack-lbaas | 16:02 | |
*** csomerville has joined #openstack-lbaas | 16:05 | |
*** eezhova has quit IRC | 16:11 | |
*** Alex_Staf has quit IRC | 16:24 | |
johnsom | nmagnezi so ifup * works on centos? | 16:25 |
johnsom | I will test that and come up with a plan. I think we can just put a conditional in the template rather than create another separate template | 16:26 |
*** cody-somerville has joined #openstack-lbaas | 16:28 | |
*** cody-somerville has quit IRC | 16:28 | |
*** cody-somerville has joined #openstack-lbaas | 16:28 | |
*** csomerville has quit IRC | 16:30 | |
*** openstackgerrit has joined #openstack-lbaas | 16:47 | |
openstackgerrit | Nakul Dahiwade proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Policy https://review.openstack.org/406328 | 16:47 |
openstackgerrit | Shashank Kumar Shankar proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Rules https://review.openstack.org/406336 | 16:52 |
*** tesseract has quit IRC | 16:55 | |
openstackgerrit | Nakul Dahiwade proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Policy https://review.openstack.org/406328 | 16:59 |
openstackgerrit | Ankur proposed openstack/octavia: Add v2 load balancer type and controllers https://review.openstack.org/423543 | 17:02 |
johnsom | nmagnezi Process: 880 ExecStartPre=/sbin/ip netns exec amphora-haproxy ifup * (code=exited, status=1/FAILURE) | 17:02 |
johnsom | [root@amphora-64e1bb17-0272-47ef-a774-bdf74fce2579 centos]# ifup * | 17:03 |
johnsom | /sbin/ifup: configuration for * not found. | 17:03 |
johnsom | Usage: ifup <configuration> | 17:03 |
*** ducttape_ has joined #openstack-lbaas | 17:12 | |
*** eezhova has joined #openstack-lbaas | 17:21 | |
*** _ducttape_ has joined #openstack-lbaas | 17:26 | |
*** ducttape_ has quit IRC | 17:30 | |
johnsom | I have a plan | 17:35 |
nmagnezi | johnsom, worked for me when I tested it | 17:37 |
nmagnezi | johnsom, but didn't work on Ubuntu | 17:37 |
johnsom | It didn't work for me on centos 7. That is the paste from above. | 17:37 |
johnsom | But I have something that does work, coding it up now | 17:37 |
*** kevo has joined #openstack-lbaas | 17:37 | |
johnsom | nmagnezi are you going to post a patch? | 17:38 |
johnsom | I am going to conditionally run this: | 17:38 |
johnsom | ExecStartPre=-/bin/awk '{ system("/sbin/ip netns exec amphora-haproxy ifup " $2)}' /var/lib/octavia/plugged_interfaces | 17:38 |
*** _ducttape_ has quit IRC | 17:55 | |
*** ducttape_ has joined #openstack-lbaas | 17:55 | |
*** _ducttape_ has joined #openstack-lbaas | 17:56 | |
*** _ducttape_ has quit IRC | 17:58 | |
*** ducttape_ has quit IRC | 18:00 | |
*** SumitNaiksatam has joined #openstack-lbaas | 18:00 | |
*** eezhova has quit IRC | 18:03 | |
xgerman | johnsom you recall how our encrypted ramfs worked | 18:04 |
xgerman | ? | 18:04 |
johnsom | The original version I wrote, yes, kind of | 18:04 |
xgerman | I installed the Lunke image and amp comes up but no ramdisk in /var/lib/octavia/certs | 18:04 |
xgerman | or do I need to switch to the namespace to see it? | 18:05 |
johnsom | https://review.openstack.org/#/c/375905/ | 18:07 |
johnsom | I don't think it was tied to the namespace, I think it was global. | 18:08 |
johnsom | Yeah, looking back at my upstart script for it, it was global, not namespace. | 18:09 |
xgerman | yeah, that’s what I thought | 18:09 |
johnsom | Ah, but I do remember there is a flag you need to use to see it | 18:09 |
xgerman | so I would expect to see it somewhere mounted | 18:09 |
johnsom | Try df -a | 18:10 |
xgerman | https://www.irccloud.com/pastebin/7y9Fq7It/ | 18:10 |
xgerman | nope | 18:10 |
openstackgerrit | Shashank Kumar Shankar proposed openstack/octavia: Introduce Octavia v2 API for Pools with LBaaS align https://review.openstack.org/420195 | 18:11 |
johnsom | xgerman root | 18:11 |
xgerman | same | 18:15 |
xgerman | https://www.irccloud.com/pastebin/0jbgYQYg/ | 18:15 |
xgerman | well, at least this LUKE change doesn’t break anything ;-) | 18:16 |
xgerman | it probably is suppoed to come up with systemd but this eludes me | 18:18 |
johnsom | Yeah, it's there, I wonder if this is an startup order issue with xenial | 18:21 |
johnsom | service cert-ramfs-ecrypt status | 18:21 |
johnsom | ● cert-ramfs-ecrypt.service | 18:21 |
johnsom | Loaded: loaded (/usr/lib/systemd/system/cert-ramfs-ecrypt.service; disabled; | 18:21 |
johnsom | Active: inactive (dead) | 18:21 |
xgerman | mmh, so we can merge Luke but that would be a Noop since the fs doesn’t work | 18:25 |
xgerman | arrgh | 18:25 |
johnsom | Let's not merge that. | 18:25 |
xgerman | agreed | 18:25 |
johnsom | I really want to look at it and understand why I decided to not use it the first time | 18:25 |
xgerman | but do we mark it -1? | 18:25 |
*** SumitNaiksatam has quit IRC | 18:25 | |
johnsom | Yeah, it looks like the existing code has some xenial fallout | 18:26 |
johnsom | xgerman so his patch doesn't work either? It's a very different systemd script | 18:27 |
xgerman | well, you saw my df-a | 18:28 |
xgerman | but it has the systemv servcie | 18:28 |
johnsom | journalctl -xel --no-pager | less | 18:29 |
johnsom | That should show you the error | 18:29 |
johnsom | Ok, yeah, it needs updating for the order anyway | 18:29 |
*** rcernin has joined #openstack-lbaas | 18:32 | |
*** amotoki has quit IRC | 18:33 | |
xgerman | can’t see anyhting but I am also not sure what I am looking for | 18:37 |
johnsom | Ok, let's hold it for deeper inspection | 18:37 |
xgerman | +1 | 18:37 |
xgerman | yeah, I though it would be as easy as maing an amp, and running mount | 18:38 |
johnsom | Yeah, I would expect it to be visible via either df -a or mount | 18:39 |
openstackgerrit | Sindhu Devale proposed openstack/octavia: Octavia v2 API for listeners https://review.openstack.org/424744 | 18:49 |
*** harlowja has quit IRC | 19:02 | |
johnsom | nmagnezi I am ready to commit a patch. Is that ok? | 19:11 |
*** eezhova has joined #openstack-lbaas | 19:19 | |
openstackgerrit | Michael Johnson proposed openstack/octavia: Fix the amphora image support for RH Linux flavors https://review.openstack.org/331841 | 19:29 |
johnsom | nmagnezi ^^^ | 19:29 |
nmagnezi | johnsom, yes sir | 19:30 |
johnsom | FYI, I pushed up a patch with fixes for syslog and the reboot issue | 19:31 |
*** amotoki has joined #openstack-lbaas | 19:34 | |
nmagnezi | johnsom, "Co-Authored-By:" you read my mind :) | 19:44 |
rm_work | oh what, we're supposed to handle &page_reverse=False | 19:47 |
rm_work | and I don't think this does the atom tokens for navigation yet... >_> | 19:48 |
rm_work | yeah this spec is totally not what anyone does yet i think | 19:48 |
rm_work | johnsom: i'm cleaning up the priority review list a little | 19:57 |
johnsom | reverse? Haven't heard of that one | 20:01 |
johnsom | rm_work cool. I just asked the folks working on API stuff to update as well | 20:01 |
blogan | sshank: ill come by in like 30 mins | 20:01 |
johnsom | I am going to run to lunch, then take a last pass on patches and start putting in WIP release patches for O-3 | 20:02 |
rm_work | k, we still trying to merge this centos patch first? | 20:05 |
*** harlowja has joined #openstack-lbaas | 20:09 | |
nmagnezi | johnsom, may i send a ps? | 20:18 |
johnsom | Yes, just make sure you rebase on the one I just pushed | 20:19 |
nmagnezi | johnsom, i did. I just want to make sure I fixed the failing functional tests | 20:19 |
openstackgerrit | Nir Magnezi proposed openstack/octavia: Fix the amphora image support for RH Linux flavors https://review.openstack.org/331841 | 20:20 |
johnsom | rm_work: yeah, I think we can get the RH patch in. It is now passing my tests, it is just down to unit tests | 20:20 |
rm_work | johnsom: how long until we can merge "features" again? | 20:21 |
sshank | blogan, Cool. | 20:25 |
johnsom | Depends how testing goes and if the patch would make backports hard. Worst case 13th. Best case next week | 20:25 |
rm_work | k | 20:25 |
nmagnezi | johnsom, I'm writing a release notes for this one | 20:25 |
openstackgerrit | Shashank Kumar Shankar proposed openstack/octavia: Introduce Octavia v2 API for Pools with LBaaS align https://review.openstack.org/420195 | 20:26 |
johnsom | Grabbing lunch, back in a while | 20:26 |
nmagnezi | johnsom, will also remove this which was added my mistake https://review.openstack.org/#/c/331841/71/octavia/compute/drivers/nova_driver.py | 20:26 |
nmagnezi | johnsom, enjoy | 20:26 |
nmagnezi | :) | 20:26 |
*** gcheresh_ has joined #openstack-lbaas | 20:34 | |
*** gcheresh_ has quit IRC | 20:43 | |
openstackgerrit | Sindhu Devale proposed openstack/octavia: Octavia v2 API for listeners https://review.openstack.org/424744 | 20:53 |
*** trevormc has quit IRC | 21:05 | |
openstackgerrit | Nir Magnezi proposed openstack/octavia: Fix the amphora image support for RH Linux flavors https://review.openstack.org/331841 | 21:28 |
nmagnezi | johnsom, ^^ | 21:29 |
*** yamamoto has quit IRC | 21:30 | |
blogan | sindhu, ndahiwade, sshank: i don't see why there are 2 reviews for each resource | 21:36 |
*** gcheresh_ has joined #openstack-lbaas | 21:36 | |
blogan | there's one for introducing, that just keeps the old v1 references, and then one that just changes those references to v2 | 21:36 |
blogan | why not just do all that in the same patch? | 21:37 |
sshank | blogan, We made Pools separate since members come under it. This way 2 can work on them independently. | 21:39 |
ndahiwade | Same would be for L7 policy and rules | 21:40 |
sshank | blogan, Old pools is parent of old members and new pools. | 21:40 |
blogan | sshank: no that i udnerstand | 21:40 |
blogan | i mean like tehse 2 reviews: https://review.openstack.org/#/c/405922 | 21:41 |
blogan | https://review.openstack.org/#/c/420195/ | 21:41 |
sshank | blogan, The first one is used by https://review.openstack.org/#/c/405932/ | 21:41 |
blogan | so why can't you just combine the 2 reviews i linked and have the members on depend on that one? | 21:42 |
blogan | i'm not understanding why there are 2 pools reviews | 21:42 |
blogan | and listeners, and members of course, but i'm using pools as an example | 21:43 |
johnsom | Ok, I am going to pull down the latest RH patch, make sure I didn't do something dumb with a quick test of those changes, then move on to starting the release patches | 21:46 |
*** catintheroof has quit IRC | 21:47 | |
nmagnezi | johnsom, anything I can do to help you? | 21:47 |
*** gcheresh_ has quit IRC | 21:47 | |
johnsom | Argh, too much of my C/C++/C# code coming out... | 21:48 |
johnsom | nmagnezi So other than my goofs, the tests that are there are passing? | 21:49 |
nmagnezi | johnsom, so far so good. including functional tests. | 21:49 |
nmagnezi | johnsom, i'm watching Zuul. | 21:50 |
johnsom | Ok. So are you good to stop for the day? I can take it from here. I want to add to the release notes a bit, then I will work to open some bugs about missing tests. If we pass the gate tests and my quick test, I think we can merge | 21:51 |
johnsom | Then we have some time to finish the tests | 21:53 |
johnsom | Sound like a plan? | 21:53 |
johnsom | We passed sonar, so that is a good sign. | 21:54 |
openstackgerrit | Shashank Kumar Shankar proposed openstack/octavia: Introduce Octavia v2 API for pools https://review.openstack.org/405922 | 21:57 |
nmagnezi | johnsom, yes. also if you file bugs feel free to assign those to me. | 22:07 |
johnsom | +1 | 22:07 |
nmagnezi | johnsom, will we have time to push those to Ocata? | 22:07 |
nmagnezi | johnsom, btw this failed (non-voting), but seem to be unrelated http://logs.openstack.org/41/331841/72/check/gate-octavia-v1-dsvm-py3x-scenario-ubuntu-xenial-nv/a74912d/logs/testr_results.html.gz | 22:08 |
*** beardedeagle has quit IRC | 22:08 | |
johnsom | Yes, we have time on the bug fixes, but should try to get them in as soon as possible. | 22:09 |
johnsom | Yes, ignore the py3x tests | 22:09 |
*** yamamoto has joined #openstack-lbaas | 22:11 | |
*** eezhova has quit IRC | 22:11 | |
nmagnezi | johnsom, I'm looking at https://releases.openstack.org/ocata/schedule.html but i'm not so sure, when is the final date for those tests? just want to know how much time I have left | 22:12 |
johnsom | nmagenzi The absolute last week is Feb 13th, but we really want them in sooner than that. | 22:13 |
johnsom | Dang, we have a bug again. | 22:16 |
johnsom | haproxy config didn't verify | 22:16 |
nmagnezi | johnsom, I'll prioritize accordingly. | 22:16 |
nmagnezi | johnsom, Ubuntu or Centos? | 22:17 |
johnsom | centos | 22:17 |
nmagnezi | johnsom, any specific type of configuration? | 22:18 |
johnsom | Not sure yet, I have to dig into it | 22:18 |
johnsom | Jan 26 22:14:33 host-192-168-0-9 amphora-agent: 2017-01-26 22:14:33.233 1668 ERR | 22:18 |
johnsom | OR octavia.amphorae.backends.agent.api_server.listener [-] Failed to verify haproxy file: Command '['haproxy', '-c', '-L', 'hB918ZoSmqBl4oQFSdWWUCEouy0', '-f', '/var/lib/octavia/c01377e1-7f73-42f8-a621-485b7438de9d/haproxy.cfg.new']' returned non-zero exit status 1 | 22:18 |
nmagnezi | johnsom, i will look into this as well but i need to respin my devstack so it will take me few min | 22:19 |
johnsom | nmagnezi My mistake, the user group change didn't take | 22:23 |
* nmagnezi note to self: auto-detection to user_group | 22:23 | |
nmagnezi | johnsom, i'll do ^^ right after the tests, which are more important. | 22:24 |
johnsom | Yeah, do the tests. | 22:24 |
johnsom | We need to discuss options on the user_group. I will add that to the release notes. | 22:24 |
johnsom | user_group is a little complicated since we render the config on the controller. So, we just need to decide how best to handle that | 22:25 |
*** matt-borland has quit IRC | 22:26 | |
nmagnezi | johnsom, we have +1 from jenkins | 22:39 |
johnsom | Ok cool. I still have a reboot bug. It didn't detect it was RH for some reason | 22:39 |
johnsom | looking at that now | 22:39 |
nmagnezi | johnsom, is it related to the ifup action for all NICs? or is it something else? | 22:40 |
johnsom | Yes, it's just the conditional not working right | 22:41 |
rm_work | johnsom: i'm out tomorrow... but I can hop on and do a review if you need me to | 22:41 |
rm_work | but i won't be able to do much complex testing | 22:41 |
johnsom | Ok, how late-ish are you on today? | 22:41 |
rm_work | ummm | 22:53 |
rm_work | basically, not after now | 22:53 |
rm_work | but | 22:53 |
rm_work | email me or SMS me | 22:53 |
rm_work | I'll be around much later | 22:53 |
rm_work | like 9pm PST prolly | 22:53 |
johnsom | DArn | 22:53 |
johnsom | https://review.openstack.org/424695 | 22:53 |
johnsom | https://review.openstack.org/423649 | 22:53 |
rm_work | LGTM | 22:53 |
johnsom | I was just finallizing the lbaas list | 22:54 |
rm_work | why did our functional fail on the first one | 22:54 |
johnsom | functional is broken on nlbaas | 22:54 |
rm_work | augh | 22:54 |
johnsom | It does *nothing* | 22:54 |
rm_work | yeah that perms but | 22:54 |
rm_work | *bug | 22:54 |
johnsom | Has been for a long time. I think Doug said it was started, but never finished | 22:54 |
rm_work | +A'd | 22:55 |
rm_work | both look sane | 22:55 |
rm_work | and no major testing issues it seems | 22:55 |
rm_work | brb in 15m and then out for a while | 22:55 |
openstackgerrit | Merged openstack/neutron-lbaas-dashboard: Updated from global requirements https://review.openstack.org/425394 | 23:00 |
nmagnezi | johnsom, it is still stacking so i have yet to debug this. any luck so far? | 23:06 |
johnsom | I had to jump to a release task, will be back to it soon | 23:06 |
nmagnezi | ok | 23:07 |
openstackgerrit | Merged openstack/neutron-lbaas-dashboard: Update for ui bootstrap changes https://review.openstack.org/418218 | 23:10 |
openstackgerrit | Michael Johnson proposed openstack/octavia: Fix the amphora image support for RH Linux flavors https://review.openstack.org/331841 | 23:15 |
johnsom | That will fix the issue | 23:15 |
* nmagnezi looks | 23:18 | |
* johnsom thinks he is doing too many things at once... | 23:19 | |
nmagnezi | johnsom, I think you reverted my last changes | 23:19 |
johnsom | Crumb, I just saw that. Yeah, fixing... | 23:20 |
nmagnezi | johnsom, sorry to be useless about this reboot debug. it takes my machine forever to stack for some reason | 23:21 |
*** ipsecguy has quit IRC | 23:24 | |
nmagnezi | johnsom, anything else I can help with? | 23:28 |
nmagnezi | johnsom, if not, I'm calling it a night. | 23:29 |
johnsom | No, I think I am good. Just updating the release notes now. Have a good night! | 23:29 |
openstackgerrit | Merged openstack/neutron-lbaas: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/406768 | 23:33 |
openstackgerrit | Michael Johnson proposed openstack/octavia: Fix the amphora image support for RH Linux flavors https://review.openstack.org/331841 | 23:34 |
*** ipsecguy has joined #openstack-lbaas | 23:35 | |
openstackgerrit | Merged openstack/neutron-lbaas: Use neutron-lib portbindings api-def https://review.openstack.org/422822 | 23:51 |
openstackgerrit | Merged openstack/neutron-lbaas: Remove psutil dependency https://review.openstack.org/424695 | 23:52 |
openstackgerrit | Merged openstack/neutron-lbaas: Updated from global requirements https://review.openstack.org/423649 | 23:56 |
openstackgerrit | Merged openstack/neutron-lbaas: The lb namespace can not be deleted https://review.openstack.org/343963 | 23:56 |
*** rcernin has quit IRC | 23:56 | |
*** rcernin has joined #openstack-lbaas | 23:57 | |
*** rcernin has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!