Wednesday, 2018-07-11

*** gildub has quit IRC00:12
*** r-mibu has joined #openstack-sdks00:13
*** gildub has joined #openstack-sdks00:15
*** gildub_ has joined #openstack-sdks00:18
*** gildub has quit IRC00:20
*** gildub_ has quit IRC00:24
*** gildub_ has joined #openstack-sdks00:25
openstackgerritDean Troyer proposed openstack/osc-lib master: Replace autodoc with sphinxcontrib-apidoc  https://review.openstack.org/58153800:43
*** edmondsw has joined #openstack-sdks01:05
*** edmondsw has quit IRC01:09
openstackgerritNguyen Hai proposed openstack-infra/requestsexceptions master: fix tox python3 overrides  https://review.openstack.org/57292601:33
*** mhen has quit IRC01:54
openstackgerritLingxian Kong proposed openstack/openstacksdk master: Support to wait for load balancer to be ACTIVE  https://review.openstack.org/58119801:55
*** mhen has joined #openstack-sdks01:58
*** mriedem has left #openstack-sdks02:00
*** mriedem has joined #openstack-sdks02:01
*** mriedem has quit IRC02:28
*** edmondsw has joined #openstack-sdks02:53
*** edmondsw has quit IRC02:57
openstackgerritLingxian Kong proposed openstack/openstacksdk master: Support to wait for load balancer to be ACTIVE  https://review.openstack.org/58119803:19
*** gildub_ has quit IRC03:39
openstackgerritVu Cong Tuan proposed openstack/cliff master: Switch to stestr  https://review.openstack.org/58156404:28
openstackgerritDao Cong Tien proposed openstack/os-client-config master: Invalid link of doc reference  https://review.openstack.org/58156604:40
*** e0ne has joined #openstack-sdks05:16
*** e0ne has quit IRC05:19
openstackgerritDean Troyer proposed openstack/osc-lib master: Replace autodoc with sphinxcontrib-apidoc  https://review.openstack.org/58153805:20
openstackgerritDean Troyer proposed openstack/osc-lib master: Stop processing requests verify args  https://review.openstack.org/58079605:26
openstackgerrittianhui proposed openstack/python-openstackclient master: Compute: Add tag support for server add network  https://review.openstack.org/58157305:53
*** ianw is now known as ianw_pto06:00
openstackgerritDao Cong Tien proposed openstack/openstacksdk master: Invalid link of doc reference  https://review.openstack.org/58158006:23
openstackgerritVu Cong Tuan proposed openstack/cliff master: Switch to stestr  https://review.openstack.org/58156406:31
*** kimamisa has joined #openstack-sdks06:46
openstackgerritFeilong Wang proposed openstack/openstacksdk master: Add Magnum /certificates support  https://review.openstack.org/58159807:01
*** gtema has joined #openstack-sdks07:06
openstackgerritTuan Do Anh proposed openstack/python-openstackclient master: Fix lower-constraints.txt  https://review.openstack.org/58160307:07
*** e0ne has joined #openstack-sdks07:11
*** e0ne has quit IRC07:16
*** peereb has joined #openstack-sdks07:17
*** e0ne has joined #openstack-sdks07:19
*** gkadam has joined #openstack-sdks07:20
openstackgerritChen proposed openstack/python-openstackclient master: Re-implement remove fixed ip function  https://review.openstack.org/58141807:20
*** kimamisa has quit IRC07:22
openstackgerritChen proposed openstack/python-openstackclient master: Re-implement remove fixed ip function  https://review.openstack.org/58141807:30
*** jpich has joined #openstack-sdks07:56
*** kimamisa has joined #openstack-sdks07:58
*** ttsiouts has joined #openstack-sdks08:01
openstackgerritVu Cong Tuan proposed openstack/cliff master: Switch to stestr  https://review.openstack.org/58156408:06
*** markvoelker has quit IRC08:07
*** Luzi has joined #openstack-sdks08:39
*** Luzi has quit IRC08:46
openstackgerritMerged openstack/osc-lib master: Stop processing requests verify args  https://review.openstack.org/58079608:49
*** cdent has joined #openstack-sdks09:44
*** ttsiouts has quit IRC10:04
*** ttsiouts has joined #openstack-sdks10:06
*** markvoelker has joined #openstack-sdks10:08
*** ttsiouts has quit IRC10:10
*** markvoelker has quit IRC10:42
*** ttsiouts has joined #openstack-sdks11:04
*** r-mibu has quit IRC11:21
*** kimamisa_ has joined #openstack-sdks11:35
*** kimamisa has quit IRC11:35
*** EvilienM is now known as EmilienM11:36
*** markvoelker has joined #openstack-sdks11:40
*** abishop has joined #openstack-sdks11:49
*** e0ne has quit IRC12:03
*** markvoelker has quit IRC12:12
*** e0ne has joined #openstack-sdks12:22
*** markvoelker has joined #openstack-sdks12:22
*** edmondsw has joined #openstack-sdks12:27
*** mriedem has joined #openstack-sdks12:33
*** ttsiouts has quit IRC12:36
*** e0ne has quit IRC12:41
openstackgerritMonty Taylor proposed openstack-infra/shade master: Make OpenStackCloud a subclass of Connection  https://review.openstack.org/57764712:49
*** ttsiouts has joined #openstack-sdks12:51
*** e0ne has joined #openstack-sdks12:51
*** e0ne has quit IRC12:52
openstackgerritDoug Hellmann proposed openstack/os-api-ref master: import zuul job settings from project-config  https://review.openstack.org/58172412:52
*** e0ne has joined #openstack-sdks12:55
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Remove local openstack.config overrides  https://review.openstack.org/58090112:56
*** openstack has joined #openstack-sdks13:05
*** ChanServ sets mode: +o openstack13:05
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Add task manager parameter to Connection  https://review.openstack.org/57761613:10
mordredgtema: if you feel like thinking about *really* fun things ... in https://review.openstack.org/#/c/574285/ dtantsur|afk mentions that TaskManager is very similar to the futurist library - and that maybe we should just add the ratelimiting support to futurist and switch shade/sdk/nodepool to using it13:20
mordredgtema: I haven't dug in on that yet- as I should probably finish some of my hundred other tasks people are waiting on me for13:21
gtemamordred: you'd bet.13:22
gtema https://review.openstack.org/#/c/574285/ is nice13:23
mordredgtema: :) I realized that shade/sdk support some logic for nodepool that's impossible to completely test in shade/sdk because the multi-threaded/rate-limiting taskmanager is not in tree13:25
mordredgtema: so every time I try to talk to someone about improving/finishing the caching tier in shade/sdk it goes badly, since the whole problem space isn't in front of people13:26
mordredwhoops13:26
dhellmannmordred , dtroyer : I've had someone ask me whether they should use shade or openstacksdk for a python app and in the course of trying to explain that things are merging I realized I don't actually know the state of that. What tool should someone use today?13:27
mordreddhellmann: sdk13:30
mordreddhellmann: they can use shade if they feel like it - it will continue to work indefinitely13:30
mordreddhellmann: but the shade code is in sdk, and https://review.openstack.org/577647 (above, should land today) makdes the shade main class a subclass of the sdk class13:31
dhellmannmordred : ok, cool, I couldn't even remember which direction things were merging :-)13:31
mordreddhellmann: the "Connection" class in sdk contains all of the shade methods13:31
mordreddhellmann: :)13:31
dhellmannETOOMANYPROJECTS13:31
mordreddhellmann: I'm *very* much looking forward to finishing off collapsing the codebasees13:31
dhellmannI'm sure13:32
*** ttsiouts has quit IRC13:41
*** e0ne has quit IRC13:45
*** ttsiouts has joined #openstack-sdks13:46
*** crunchengine has joined #openstack-sdks13:47
*** lifeless has joined #openstack-sdks13:47
openstackgerritMerged openstack/python-openstackclient master: Fix lower-constraints.txt  https://review.openstack.org/58160313:56
*** mriedem has quit IRC14:01
*** mriedem has joined #openstack-sdks14:02
*** e0ne has joined #openstack-sdks14:07
openstackgerritYuki Nishiwaki proposed openstack/osc-lib master: Check auth_url as well as identity version api  https://review.openstack.org/52641314:13
openstackgerritMerged openstack/openstacksdk master: Fix Magnum cluster update  https://review.openstack.org/58115714:17
openstackgerritMonty Taylor proposed openstack-infra/shade master: Make OpenStackCloud a subclass of Connection  https://review.openstack.org/57764714:22
openstackgerritMonty Taylor proposed openstack-infra/shade master: Use the openstacksdk task_manager constructor argument  https://review.openstack.org/58047514:22
openstackgerritMerged openstack/openstacksdk master: Add Magnum /certificates support  https://review.openstack.org/58159814:29
*** markvoelker has quit IRC14:44
*** markvoelker has joined #openstack-sdks14:44
*** ttsiouts has quit IRC14:48
*** markvoelker has quit IRC14:49
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Add task manager parameter to Connection  https://review.openstack.org/57761614:54
*** ttsiouts has joined #openstack-sdks14:55
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Add task manager parameter to Connection  https://review.openstack.org/57761614:57
*** markvoelker has joined #openstack-sdks15:14
*** peereb has quit IRC15:46
openstackgerritMerged openstack/python-openstackclient master: Replace pbr autodoc with sphinxcontrib-apidoc  https://review.openstack.org/58141516:02
openstackgerritMerged openstack/python-openstackclient master: Skip calls to glance and nova when got no servers  https://review.openstack.org/56834416:02
openstackgerritMerged openstack/openstacksdk master: Support to wait for load balancer to be ACTIVE  https://review.openstack.org/58119816:08
*** ttsiouts has quit IRC16:09
*** jpich has quit IRC16:13
*** e0ne has quit IRC16:39
*** kimamisa_ has quit IRC16:44
*** markvoelker has quit IRC16:52
*** bobh has joined #openstack-sdks16:55
*** Yuki has joined #openstack-sdks17:03
*** Yuki is now known as Guest7971817:03
Shrewsmordred: re: 577647 and list_flavors() comment, could we undecorate the shade method and depend on sdk's caching?17:16
Shrewsseems weird that shade has to know that sdk decorates that method and we need to manipulate the call17:17
*** Guest79718 has quit IRC17:17
Shrewsand fragile, to boot17:18
Shrewsalso posted a comment with that question17:24
*** markvoelker has joined #openstack-sdks17:24
*** abishop_ has joined #openstack-sdks17:25
*** abishop has quit IRC17:28
*** abishop_ has quit IRC17:39
*** abishop_ has joined #openstack-sdks17:40
*** kimamisa has joined #openstack-sdks17:45
*** bobh has quit IRC17:59
*** mriedem1 has joined #openstack-sdks18:02
*** asbishop has joined #openstack-sdks18:05
*** mriedem has quit IRC18:05
*** edmondsw has quit IRC18:07
*** edmondsw has joined #openstack-sdks18:07
*** abishop_ has quit IRC18:08
*** mriedem1 is now known as mriedem18:12
*** gtema has quit IRC18:13
*** e0ne has joined #openstack-sdks18:16
*** bobh has joined #openstack-sdks18:18
*** GreenBlood has quit IRC18:18
*** kuzko has quit IRC18:18
mordredShrews: yes - I think we can - but I think we need to unwind more first - like - I think we should probably unwind the caching completely so that there are no cache decorators in shade at all18:24
Shrewsmordred: i guess i don't see why we can't do it as we convert the methods (list_flavors here)18:33
Shrewswait... if we just delete shade's list_flavors, wouldn't that work?18:36
*** e0ne has quit IRC18:49
*** cdent has quit IRC18:52
*** kuzko has joined #openstack-sdks18:59
*** GreenBlood has joined #openstack-sdks18:59
*** harlowja has joined #openstack-sdks19:00
*** e0ne has joined #openstack-sdks19:01
*** e0ne has quit IRC19:06
*** e0ne has joined #openstack-sdks19:17
*** e0ne has quit IRC19:24
*** e0ne has joined #openstack-sdks19:28
*** e0ne has quit IRC19:31
*** e0ne has joined #openstack-sdks19:41
*** edmondsw has quit IRC19:42
*** edmondsw has joined #openstack-sdks19:43
*** gkadam has quit IRC19:54
*** e0ne has quit IRC20:31
*** e0ne has joined #openstack-sdks20:35
*** e0ne has quit IRC20:42
*** asbishop has quit IRC21:15
openstackgerritDaniel Speichert proposed openstack/osc-lib master: Fixes a bug with issubclass() being called on an instance  https://review.openstack.org/58188621:17
*** crunchengine has quit IRC21:43
*** kimamisa has quit IRC22:07
*** bobh has quit IRC22:32
*** harlowja has quit IRC23:05
*** edmondsw has quit IRC23:12
*** edmondsw has joined #openstack-sdks23:13
*** edmondsw has quit IRC23:17
mordredShrews: yes- except - the issue is that the list_flavors in shade needs to have different default values for the kwarg get_extra23:41
mordredShrews: what we probably want to do is make a patch that deletes all the methods except for the ones where we want to change arguments23:41
mordredbut I keep avoiding that because it hurts my head23:42
mordredShrews: once we do that though, I think we can also rip out the shade-side caching and the code will be *much* nicer23:42

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