Tuesday, 2018-02-13

« Monday, 2018-02-12 Index Wednesday, 2018-02-14 »
*** openstackgerrit has quit IRC01:03
*** slaweq has joined #openstack-lbaas01:03
*** slaweq has quit IRC01:08
*** Swami has quit IRC01:37
*** AlexeyAbashkin has joined #openstack-lbaas02:21
*** AlexeyAbashkin has quit IRC02:25
*** links has joined #openstack-lbaas02:35
*** fnaval has joined #openstack-lbaas02:41
*** openstackgerrit has joined #openstack-lbaas02:55
openstackgerritMichael Johnson proposed openstack/octavia-tempest-plugin master: Create scenario tests for load balancers  https://review.openstack.org/54303402:55
openstackgerritMichael Johnson proposed openstack/octavia-tempest-plugin master: Create scenario tests for load balancers  https://review.openstack.org/54303402:56
*** harlowja has quit IRC03:04
*** sapd has quit IRC03:42
*** links has quit IRC04:05
*** links has joined #openstack-lbaas04:24
*** sanfern has joined #openstack-lbaas04:28
*** yamamoto has joined #openstack-lbaas04:44
*** dougwig has quit IRC04:46
*** threestrands has quit IRC04:50
*** slaweq has joined #openstack-lbaas05:05
*** slaweq has quit IRC05:10
*** gcheresh has joined #openstack-lbaas05:20
*** gcheresh has quit IRC05:25
*** gcheresh has joined #openstack-lbaas05:31
*** harlowja has joined #openstack-lbaas05:52
*** slaweq has joined #openstack-lbaas06:01
*** slaweq has quit IRC06:02
*** harlowja has quit IRC06:33
*** yamamoto has quit IRC06:41
*** yamamoto has joined #openstack-lbaas06:43
*** yamamoto has quit IRC06:50
*** yamamoto has joined #openstack-lbaas06:55
openstackgerritAllen proposed openstack/octavia-dashboard master: Add filterFacets for loadBalancerResourceType  https://review.openstack.org/54211407:14
*** b_bezak has joined #openstack-lbaas07:21
*** numans_ has joined #openstack-lbaas07:24
*** rcernin has quit IRC07:25
*** b_bezak has quit IRC07:26
*** numans has quit IRC07:28
*** kobis has joined #openstack-lbaas07:29
*** numans_ has quit IRC07:35
*** numans has joined #openstack-lbaas07:40
*** pcaruana has joined #openstack-lbaas07:41
*** AlexeyAbashkin has joined #openstack-lbaas07:48
*** slaweq_ has joined #openstack-lbaas08:03
*** b_bezak has joined #openstack-lbaas08:03
*** slaweq_ has quit IRC08:08
*** numans_ has joined #openstack-lbaas08:10
*** numans has quit IRC08:13
*** numans_ has quit IRC08:15
openstackgerritAllen proposed openstack/octavia-dashboard master: Add filterFacets for loadBalancerResourceType  https://review.openstack.org/54211408:15
*** tesseract has joined #openstack-lbaas08:26
*** yamamoto has quit IRC08:35
openstackgerritAllen proposed openstack/octavia-dashboard master: Add filterFacets for listenerResourceType  https://review.openstack.org/54384108:35
*** Alex_Staf has joined #openstack-lbaas08:37
*** yamamoto has joined #openstack-lbaas08:39
openstackgerritAllen proposed openstack/octavia-dashboard master: Add filterFacets for loadBalancerResourceType  https://review.openstack.org/54211408:39
openstackgerritAllen proposed openstack/octavia-dashboard master: Add filterFacets for loadBalancerResourceType  https://review.openstack.org/54211408:54
*** slaweq_ has joined #openstack-lbaas08:55
*** slaweq_ has quit IRC08:59
*** kong has quit IRC09:07
openstackgerritAllen proposed openstack/octavia-dashboard master: Add filterFacets for loadBalancerResourceType  https://review.openstack.org/54211409:07
*** slaweq_ has joined #openstack-lbaas09:36
*** slaweq_ has quit IRC09:41
openstackgerritAllen proposed openstack/octavia-dashboard master: Add filterFacets for loadBalancerResourceType  https://review.openstack.org/54211409:41
*** b_bezak has quit IRC09:50
*** numans has joined #openstack-lbaas09:50
*** b_bezak has joined #openstack-lbaas09:50
*** salmankhan has joined #openstack-lbaas09:53
*** b_bezak_ has joined #openstack-lbaas09:54
*** b_bezak has quit IRC09:55
*** yamamoto has quit IRC09:58
*** yamamoto has joined #openstack-lbaas10:15
*** salmankhan1 has joined #openstack-lbaas10:16
*** salmankhan has quit IRC10:17
*** salmankhan1 is now known as salmankhan10:17
*** yamamoto has quit IRC10:22
openstackgerritAllen proposed openstack/octavia-dashboard master: Fix bug when click into pool detail page  https://review.openstack.org/54388410:24
openstackgerritAllen proposed openstack/octavia-dashboard master: Add filterFacets for loadBalancerResourceType  https://review.openstack.org/54211410:29
openstackgerritAllen proposed openstack/octavia-dashboard master: Fix bug when click into pool detail page  https://review.openstack.org/54388410:32
*** links has quit IRC10:36
*** links has joined #openstack-lbaas10:49
*** sanfern has quit IRC10:56
*** yamamoto has joined #openstack-lbaas11:13
*** b_bezak_ has quit IRC11:48
*** b_bezak has joined #openstack-lbaas11:48
*** salmankhan has quit IRC11:50
*** salmankhan has joined #openstack-lbaas11:52
*** b_bezak has quit IRC11:53
*** b_bezak has joined #openstack-lbaas12:41
*** ianychoi_ has joined #openstack-lbaas12:42
*** ianychoi has quit IRC12:46
*** slaweq_ has joined #openstack-lbaas13:01
*** sanfern has joined #openstack-lbaas13:03
*** slaweq_ has quit IRC13:05
openstackgerritAllen proposed openstack/octavia-dashboard master: Add the missing type for Health Monitor  https://review.openstack.org/54321113:06
openstackgerritAllen proposed openstack/octavia-dashboard master: Fix bug when click into pool detail page  https://review.openstack.org/54388413:08
*** pcaruana has quit IRC13:16
*** atoth has joined #openstack-lbaas13:22
*** oanson has quit IRC13:33
*** oanson has joined #openstack-lbaas13:35
*** yamamoto has quit IRC13:41
*** slaweq_ has joined #openstack-lbaas13:45
*** ianychoi_ has quit IRC13:48
*** ianychoi_ has joined #openstack-lbaas13:49
*** slaweq_ has quit IRC13:50
*** fnaval has quit IRC14:16
*** yamamoto has joined #openstack-lbaas14:27
*** fnaval has joined #openstack-lbaas14:28
*** fnaval has quit IRC14:33
*** openstackgerrit has quit IRC14:33
*** Alex_Staf has quit IRC14:39
*** slaweq_ has joined #openstack-lbaas14:47
*** links has quit IRC14:47
*** fnaval has joined #openstack-lbaas14:47
*** bbzhao has quit IRC14:50
*** bbzhao has joined #openstack-lbaas14:50
*** slaweq_ has quit IRC14:52
*** ying_zuo has quit IRC14:58
*** ying_zuo has joined #openstack-lbaas14:58
*** sanfern has quit IRC15:35
*** slaweq_ has joined #openstack-lbaas15:38
*** gcheresh has quit IRC15:41
*** slaweq_ has quit IRC15:42
*** b_bezak has quit IRC16:09
*** slaweq_ has joined #openstack-lbaas16:24
*** kobis has quit IRC16:25
*** kobis has joined #openstack-lbaas16:26
*** kobis has quit IRC16:26
*** slaweq_ has quit IRC16:29
*** b_bezak has joined #openstack-lbaas16:34
*** pcaruana has joined #openstack-lbaas16:47
*** dougwig has joined #openstack-lbaas16:54
*** bcafarel has quit IRC17:07
johnsomHmm, we have two options:17:15
johnsom1. If we get an API call for delete on an object in "DELETED" state we can return 204 all is good.17:15
johnsom2. If we get an API call for delete on an object in "DELETED" state we can return 404 not found.17:15
*** bbzhao has quit IRC17:15
*** bbzhao has joined #openstack-lbaas17:16
johnsomI guess for consistency we should do 404 not found.  That is how list LBs returns.17:16
johnsomHmm, yeah, we didn't do this well.  List returns no LBs, but an explicit get (show details) on the LB ID does return the object in "DELETED"17:18
johnsomrm_work nmagnezi xgerman_ Thoughts?17:19
*** yamamoto has quit IRC17:21
xgerman_we decided to return 204 long agao since deleting a deleted object leads to the same state deleted17:21
johnsomSo you lean towards returning 204 on objects that are already in "DELETED"17:22
*** yamamoto has joined #openstack-lbaas17:24
*** yamamoto has quit IRC17:24
xgerman_yes17:25
johnsomI lean that way too17:26
johnsomCurrently we return: "Invalid state DELETED of loadbalancer resource ab6bf03a-721c-42f6-886c-314d67c6aff4"17:28
johnsomI've opened a bug and started a patch, but didn't want to get too far down the path if I was forgetting something.17:28
xgerman_yeah, that seems wrong regardless how we look at it17:28
johnsomRight, I think it either should be 404 or 20417:29
xgerman_+117:30
xgerman_can we run LBaaS V2 dashaboard and Octavia i the same install?17:31
johnsomYes, but you will get duplicate "Load Balancer" panels in the list17:31
xgerman_thanks17:33
*** AlexeyAbashkin has quit IRC17:38
*** yamamoto has joined #openstack-lbaas17:40
*** yamamoto has quit IRC17:45
*** bcafarel has joined #openstack-lbaas17:55
*** pcaruana has quit IRC17:55
*** JudeC has joined #openstack-lbaas17:59
*** pcaruana has joined #openstack-lbaas18:06
*** yamamoto has joined #openstack-lbaas18:06
*** yamamoto has quit IRC18:06
*** slaweq has joined #openstack-lbaas18:25
*** slaweq has quit IRC18:26
*** slaweq has joined #openstack-lbaas18:26
rm_workjohnsom: i thought that was what we WANTED to have happen18:33
rm_work(show DELETED objects when you query directly, but not on LIST, unless deleted=True or something?)18:34
rm_workI agree that they are mismatched right now if delete returns 40418:34
johnsomRight, my issue is mostly about the delete a DELETED18:34
rm_workdoes it?18:34
rm_workcurrently18:34
johnsomI am going to leave the show, it's the delete I'm fixing18:34
rm_workso it returns 204?18:35
rm_worki think I agree18:35
johnsomRight18:35
rm_workfirst instinct is 404 on delete deleted18:35
rm_workbut18:35
rm_worksecond instinct is i like showing deleted things and we need to be consistent18:35
rm_workand it's not like this stuff is secret18:35
rm_workLIST *does* have a way to show deleted, right?18:35
rm_workor do we need to add that18:35
rm_workIIRC we'd intended to allow showing18:35
johnsomNo, list doesn't show them at all18:36
rm_worknot optional? hrm18:36
rm_workI might also vote to add deleted=True to the GET for the list call18:37
rm_workswear we added it18:37
rm_workbut maybe I'm thinking of a different project18:37
johnsomNot at the moment. I would support adding an "ALL" param though18:37
rm_worknot deleted=True but all=True?18:37
rm_workwhat are the other cases18:37
johnsomI would make it consistent across the endpoints18:38
rm_workerr18:38
rm_workare there other endpoints where `all` makes sense?18:39
*** slaweq has quit IRC18:39
rm_workI guess it's forward-looking, but that's assuming we don't want to allow anything fine-grained in the future too18:39
*** slaweq has joined #openstack-lbaas18:40
*** tesseract has quit IRC18:40
johnsomYeah, I would think an "all" param on lists makes sense, in "openstack server list" there is --all18:42
johnsomIDK, I'm just going to fix the delete now as it's bugging me in the tempest work with the cleanup methods. It's clear "409 Invalid status" is not the right response to deleting something already deleted.18:43
rm_workerrrr wut18:43
johnsomI'm just working on the functional tests now, should have it up for review in a bit18:44
rm_workit's not a 404 OR a 204??!18:44
rm_workalso, are you sure that's not because of PENDING_DELETE?18:44
johnsomNo, right now it's 40918:44
johnsomYes18:44
johnsomJust a sec, I will paste it18:44
johnsom{"debuginfo": null, "faultcode": "Client", "faultstring": "Invalid state DELETED of loadbalancer resource ab6bf03a-721c-42f6-886c-314d67c6aff4"}stack@devstackpy/json" -H "X-Auth-Token: $current_token" -H "Accept: application/json" $test_API_ENDPOINT/v2.0/lbaas/loadbalancers/ab6bf03a-721c-42f6-886c-314d67c6aff418:45
*** pcaruana has quit IRC18:45
*** slaweq has quit IRC18:45
johnsomRight now it falls through to the locking logic which rejects it as immutable18:46
johnsomI'm switching it to 20418:46
rm_workT_T18:46
rm_workyeah k18:46
rm_worksomehow I didn't notice that18:46
rm_workor else I had it ignore that stuff or something18:46
johnsomYeah, me either until I wrote the delete test that had a cleanup for the LB.18:46
johnsomAnyway, I will post the patch in a bit18:47
johnsomI'm just adding the test coverage for it18:47
rm_workk18:48
rm_workyay tests lol18:48
johnsomMan, our functional tests are efficient, it lights up all of the cores in my CPU when I light it off.  472 API tests...19:02
*** harlowja has joined #openstack-lbaas19:03
*** yamamoto has joined #openstack-lbaas19:06
*** salmankhan has quit IRC19:06
rm_workyes T_T19:07
rm_workmy laptop takes off like a jet19:07
rm_workand kinda hovers on my desk from all the fan activity19:07
*** gcheresh has joined #openstack-lbaas19:08
*** yamamoto has quit IRC19:15
*** openstackgerrit has joined #openstack-lbaas19:27
openstackgerritMichael Johnson proposed openstack/octavia master: Updates API for deleting "DELETED" objects  https://review.openstack.org/54404619:27
johnsomQuestion is should we do an RC2 for that?19:27
*** slaweq has joined #openstack-lbaas19:45
*** slaweq has quit IRC19:49
*** b_bezak has quit IRC19:49
*** gcheresh has quit IRC19:58
*** gcheresh has joined #openstack-lbaas20:07
*** kobis has joined #openstack-lbaas20:08
openstackgerritErik Olof Gunnar Andersson proposed openstack/neutron-lbaas master: Log failed cert delete, and raise TLSContainerInvalid instead  https://review.openstack.org/54406220:10
eanderssonBecause we all love more patches to the now deprecated neutron-lbaas :p20:14
*** gcheresh has quit IRC20:21
johnsomHa, well, as long as it's a bug fix....20:30
*** salmankhan has joined #openstack-lbaas20:47
*** salmankhan has quit IRC20:51
*** gcheresh has joined #openstack-lbaas20:53
eanderssonNot sure why in the world neutron-lbaas should be responsible for deleting "invalid" secrets lol but that is a different topic21:08
eanderssonIt looks like it's already fixed in octavia21:08
johnsomeandersson Yet another reason to switch!  Grin21:10
eanderssonhaha21:10
eanderssonindeed21:10
eanderssonsoonTM21:10
johnsomNice21:10
*** slaweq has joined #openstack-lbaas21:11
*** gcheresh has quit IRC21:31
*** openstackstatus has quit IRC21:43
*** openstackstatus has joined #openstack-lbaas21:45
*** ChanServ sets mode: +v openstackstatus21:45
*** rcernin has joined #openstack-lbaas22:05
*** Alex_Staf has joined #openstack-lbaas22:08
*** kobis has quit IRC22:08
*** kong has joined #openstack-lbaas22:55
*** fnaval has quit IRC23:10
*** Alex_Staf has quit IRC23:19
*** fnaval has joined #openstack-lbaas23:27
openstackgerritMerged openstack/octavia-dashboard master: Add the missing protocol for listener  https://review.openstack.org/54207523:41
openstackgerritMerged openstack/octavia-dashboard master: Remove the unused codes  https://review.openstack.org/54206923:43
*** slaweq has quit IRC23:45
*** fnaval has quit IRC23:48
*** fnaval has joined #openstack-lbaas23:48
openstackgerritMichael Johnson proposed openstack/neutron-lbaas master: Gate API test for the lbaasv2-proxy plugin  https://review.openstack.org/53935023:56
« Monday, 2018-02-12 Index Wednesday, 2018-02-14 »

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