Wednesday, 2016-12-14

*** ctayal has quit IRC00:00
*** bpokorny has quit IRC00:02
*** bpokorny has joined #openstack-horizon00:02
*** ctayal has joined #openstack-horizon00:04
*** mtanino has quit IRC00:10
*** itisha has quit IRC00:12
*** katyafervent_awa has quit IRC00:14
*** katyafervent_awa has joined #openstack-horizon00:14
*** ctayal has quit IRC00:16
*** Pavo has quit IRC00:16
*** tsufiev has quit IRC00:16
*** ducttape_ has joined #openstack-horizon00:17
*** tsufiev has joined #openstack-horizon00:17
*** ChanServ sets mode: +v tsufiev00:17
openstackgerritYing Zuo proposed openstack/horizon: Add charts to show volume quotas on Python launch instance modal  https://review.openstack.org/39040900:18
*** Pavo has joined #openstack-horizon00:21
*** rcernin has quit IRC00:22
*** ctayal has joined #openstack-horizon00:24
*** kzaitsev_mb has joined #openstack-horizon00:29
*** harlowja has joined #openstack-horizon00:30
*** alexvictorchan has quit IRC00:44
*** ducttape_ has quit IRC00:53
*** sacharya has joined #openstack-horizon00:58
openstackgerritCindy Lu proposed openstack/horizon: Add Angular keystone user details use registry  https://review.openstack.org/33639400:59
*** sacharya has quit IRC01:03
*** kzaitsev_mb has quit IRC01:05
*** Zer0Byte__ has quit IRC01:06
*** ctayal has quit IRC01:06
*** shu-mutou-AWAY is now known as shu-mutou01:19
openstackgerritRichard Hagarty proposed openstack/horizon: WIP - Add cinder group types to Admin volume panel  https://review.openstack.org/41049001:28
*** piet has joined #openstack-horizon01:30
*** bpokorny_ has joined #openstack-horizon01:32
*** bpokorny_ has quit IRC01:32
*** yamamoto has quit IRC01:33
*** bpokorny has quit IRC01:35
*** zhugaoxiao has quit IRC01:51
*** zhugaoxiao has joined #openstack-horizon01:52
*** yamamoto has joined #openstack-horizon01:52
*** ducttape_ has joined #openstack-horizon01:53
*** ducttape_ has quit IRC01:55
*** ducttape_ has joined #openstack-horizon01:55
*** ctayal has joined #openstack-horizon01:59
*** ruoyu has joined #openstack-horizon02:02
*** ctayal_ has joined #openstack-horizon02:05
*** ctayal has quit IRC02:05
*** Jeffrey4l has quit IRC02:05
*** tqtran has quit IRC02:07
*** Pavo has quit IRC02:09
*** ducttape_ has quit IRC02:09
*** dave-mccowan has quit IRC02:10
*** ruoyu has quit IRC02:10
*** Pavo has joined #openstack-horizon02:11
*** dave-mccowan has joined #openstack-horizon02:11
*** ruoyu has joined #openstack-horizon02:11
*** Pavo has quit IRC02:12
*** Pavo has joined #openstack-horizon02:13
*** Jeffrey4l has joined #openstack-horizon02:15
*** pghmt1 has joined #openstack-horizon02:18
*** ruoyu has quit IRC02:18
*** pghmt has quit IRC02:18
*** ducttape_ has joined #openstack-horizon02:21
*** ruoyu has joined #openstack-horizon02:29
*** ruoyu has quit IRC02:31
*** ruoyu has joined #openstack-horizon02:32
*** clu_ has quit IRC02:33
*** lblanchard has quit IRC02:35
*** ruoyu has quit IRC02:41
*** ruoyu has joined #openstack-horizon02:41
openstackgerritRichard Jones proposed openstack/horizon: Update for ui bootstrap changes  https://review.openstack.org/40124502:44
*** mrunge has quit IRC02:49
*** bpokorny has joined #openstack-horizon02:51
*** mrunge has joined #openstack-horizon02:54
*** ChanServ sets mode: +v mrunge02:54
*** bpokorny has quit IRC02:55
*** ducttape_ has quit IRC02:56
*** sacharya has joined #openstack-horizon02:59
*** ruoyu has quit IRC03:01
*** ruoyu has joined #openstack-horizon03:02
*** pghmt1 has quit IRC03:02
*** sacharya has quit IRC03:04
*** pghmt has joined #openstack-horizon03:08
*** ruoyu has quit IRC03:09
openstackgerritchenyingnan proposed openstack/horizon: Replace six.iteritems(iter) with iter.items()  https://review.openstack.org/40961503:15
*** bpokorny has joined #openstack-horizon03:24
*** sacharya has joined #openstack-horizon03:25
*** jtriley has joined #openstack-horizon03:25
*** pghmt has quit IRC03:25
*** pghmt has joined #openstack-horizon03:25
*** bpokorny has quit IRC03:29
openstackgerritRichard Jones proposed openstack/horizon: Update for ui bootstrap changes  https://review.openstack.org/40124503:34
*** piet has quit IRC03:35
*** dave-mccowan has quit IRC03:39
*** pghmt has quit IRC03:41
*** bpokorny has joined #openstack-horizon03:55
*** bpokorny has quit IRC03:56
*** ducttape_ has joined #openstack-horizon03:57
*** ducttape_ has quit IRC04:01
*** tqtran has joined #openstack-horizon04:03
*** tqtran has quit IRC04:08
*** shu-mutou has quit IRC04:09
*** jtriley has quit IRC04:09
*** Pavo has quit IRC04:13
*** udesale has joined #openstack-horizon04:14
*** shu-mutou has joined #openstack-horizon04:15
*** Pavo has joined #openstack-horizon04:17
*** ducttape_ has joined #openstack-horizon04:33
openstackgerritanusha proposed openstack/horizon:  Networks panel can't be actually disabled  https://review.openstack.org/40858304:38
*** ducttape_ has quit IRC04:44
*** chlong has joined #openstack-horizon04:44
openstackgerritanusha proposed openstack/horizon:  Networks panel can't be actually disabled  https://review.openstack.org/40858304:45
*** jrcloud has joined #openstack-horizon04:50
*** zackf has quit IRC04:53
*** tqtran has joined #openstack-horizon05:06
*** tqtran has quit IRC05:10
*** jrcloud has quit IRC05:12
*** masco has joined #openstack-horizon05:12
*** ratailor has joined #openstack-horizon05:13
*** wolverineav has joined #openstack-horizon05:18
*** ctayal_ has quit IRC05:22
*** wolverineav has quit IRC05:22
*** ducttape_ has joined #openstack-horizon05:44
*** ducttape_ has quit IRC05:49
*** adriant has quit IRC05:53
openstackgerritanusha proposed openstack/horizon:  Networks panel can't be actually disabled  https://review.openstack.org/40858306:07
*** amotoki has joined #openstack-horizon06:12
*** Pavo has quit IRC06:13
*** ctayal has joined #openstack-horizon06:13
*** amotoki has quit IRC06:14
*** amotoki has joined #openstack-horizon06:17
*** Pavo has joined #openstack-horizon06:17
*** hurgleburgler has quit IRC06:27
*** hurgleburgler has joined #openstack-horizon06:28
*** ChanServ sets mode: +v hurgleburgler06:28
*** hurgleburgler has quit IRC06:28
*** zackf has joined #openstack-horizon06:31
*** zackf has quit IRC06:35
*** zackf has joined #openstack-horizon06:51
*** tobberydberg has joined #openstack-horizon07:04
*** ducttape_ has joined #openstack-horizon07:04
*** tobberyd_ has joined #openstack-horizon07:06
*** tobberyd_ has joined #openstack-horizon07:06
openstackgerritYakup Adaklı proposed openstack/horizon: Text and checkbox should align in "Create Project" form  https://review.openstack.org/40980407:07
*** tobberydberg has quit IRC07:09
openstackgerritanusha proposed openstack/horizon:  Networks panel can't be actually disabled  https://review.openstack.org/40858307:09
*** zackf has quit IRC07:12
*** bapalm has quit IRC07:18
*** yamamoto has quit IRC07:18
*** ducttape_ has quit IRC07:24
*** jprovazn has joined #openstack-horizon07:24
*** ctayal has quit IRC07:25
*** sacharya has quit IRC07:26
openstackgerritOpenStack Proposal Bot proposed openstack/horizon: Imported Translations from Zanata  https://review.openstack.org/41056807:32
*** bapalm has joined #openstack-horizon07:38
*** Foxlik has joined #openstack-horizon07:38
*** Foxlik1 has joined #openstack-horizon07:40
*** ctayal has joined #openstack-horizon07:41
*** Foxlik has quit IRC07:42
*** rcernin has joined #openstack-horizon07:56
*** yamamoto has joined #openstack-horizon07:56
*** rcernin has quit IRC07:56
*** rcernin has joined #openstack-horizon07:57
*** pcaruana has joined #openstack-horizon07:59
*** sacharya has joined #openstack-horizon08:00
*** sacharya has quit IRC08:05
*** tqtran has joined #openstack-horizon08:08
openstackgerritYakup Adaklı proposed openstack/horizon: Text and checkbox should align in "Create Project" form  https://review.openstack.org/40980408:10
*** tqtran has quit IRC08:12
*** Pavo has quit IRC08:12
*** Pavo has joined #openstack-horizon08:18
*** ducttape_ has joined #openstack-horizon08:25
*** jpich has joined #openstack-horizon08:27
*** ducttape_ has quit IRC08:30
openstackgerritMerged openstack/horizon: Imported Translations from Zanata  https://review.openstack.org/41056808:32
*** breitz has quit IRC08:33
*** breitz has joined #openstack-horizon08:33
*** Zer0Byte__ has joined #openstack-horizon08:39
*** ctayal has quit IRC08:45
*** openstackgerrit has quit IRC08:48
robcresswellrhagarty: Let's discuss microversions. There are at least 3 people now working in isolation on the implementation.08:50
*** shu-mutou is now known as shu-mutou-AWAY08:59
*** ianychoi has joined #openstack-horizon09:05
*** openstackgerrit has joined #openstack-horizon09:10
openstackgerritKenji Ishii proposed openstack/horizon: Add Port-Create in Project Dashboard  https://review.openstack.org/32020309:10
*** tobberyd_ has quit IRC09:10
openstackgerritanusha proposed openstack/horizon:  Networks panel can't be actually disabled  https://review.openstack.org/40858309:12
openstackgerritanusha proposed openstack/horizon:  Networks panel can't be actually disabled  https://review.openstack.org/40858309:14
openstackgerritRadomir Dopieralski proposed openstack/horizon: Simple tenant usage pagination  https://review.openstack.org/41033709:15
*** rmoe has quit IRC09:16
*** tobberyd_ has joined #openstack-horizon09:21
*** tobberyd_ has quit IRC09:26
*** rmoe has joined #openstack-horizon09:30
*** itisha has joined #openstack-horizon09:39
*** Zer0Byte__ has quit IRC09:41
*** tobberydberg has joined #openstack-horizon09:52
*** rmoe has quit IRC09:52
*** rmoe has joined #openstack-horizon09:55
*** ducttape_ has joined #openstack-horizon09:56
*** ducttape_ has quit IRC10:00
*** tobberydberg has quit IRC10:03
*** tobberydberg has joined #openstack-horizon10:04
*** tobberydberg has quit IRC10:08
*** tqtran has joined #openstack-horizon10:09
*** zackf has joined #openstack-horizon10:10
*** brad[] has quit IRC10:11
*** Pavo has quit IRC10:13
*** tqtran has quit IRC10:13
*** tobberydberg has joined #openstack-horizon10:14
*** zackf has quit IRC10:14
*** Pavo has joined #openstack-horizon10:17
*** tobberydberg has quit IRC10:19
*** rmoe has quit IRC10:24
*** mvk has quit IRC10:24
*** tobberydberg has joined #openstack-horizon10:36
*** tobberydberg has quit IRC10:47
*** wolverineav has joined #openstack-horizon10:50
*** tosky has joined #openstack-horizon10:55
*** tobberydberg has joined #openstack-horizon10:56
*** mvk has joined #openstack-horizon10:56
openstackgerritRadomir Dopieralski proposed openstack/horizon: Properly compare versions in APIVersionManager  https://review.openstack.org/41068810:59
*** tobberydberg has quit IRC11:01
*** udesale has quit IRC11:05
*** yamamoto has quit IRC11:12
openstackgerritYakup Adaklı proposed openstack/horizon: Flavor detail page added.  https://review.openstack.org/41069311:13
*** rmoe has joined #openstack-horizon11:23
*** brad[] has joined #openstack-horizon11:24
*** yamamoto has joined #openstack-horizon11:26
*** ducttape_ has joined #openstack-horizon11:26
*** ratailor has quit IRC11:28
*** tobberydberg has joined #openstack-horizon11:29
*** ducttape_ has quit IRC11:31
*** tobberyd_ has joined #openstack-horizon11:38
*** tobbery__ has joined #openstack-horizon11:40
*** tobberydberg has quit IRC11:41
*** tobber___ has joined #openstack-horizon11:41
*** tobberyd_ has quit IRC11:43
*** tobbery__ has quit IRC11:44
*** tobberydberg has joined #openstack-horizon11:44
*** tobber___ has quit IRC11:45
*** tobberydberg has quit IRC11:49
*** tobberydberg has joined #openstack-horizon11:50
*** sacharya has joined #openstack-horizon12:02
*** dave-mccowan has joined #openstack-horizon12:04
*** sacharya has quit IRC12:07
*** tqtran has joined #openstack-horizon12:11
*** ^Gal^ has joined #openstack-horizon12:12
*** Pavo has quit IRC12:12
*** tqtran has quit IRC12:15
openstackgerritRodrigo Barbieri proposed openstack/manila-ui: Implement Share Migration Ocata improvements  https://review.openstack.org/40630712:16
*** zackf has joined #openstack-horizon12:16
*** Pavo has joined #openstack-horizon12:17
*** zackf has quit IRC12:20
*** aortega has quit IRC12:40
*** jamielennox is now known as jamielennox|away12:43
rdopierarobcresswell: ok, lets12:47
*** jamielennox|away is now known as jamielennox12:50
robcresswellrdopiera: ?12:51
robcresswellah yes, now there are 4 patches to API handling12:52
robcresswell:D12:52
rdopiera\o/12:52
rdopierawhat's with this  `except Exception: pass` recently?12:54
rdopieraI've been seeing that in our patches more and more12:54
robcresswellrdopiera: Why handle exceptions when you could bury them and pretend they didnt happen12:56
robcresswellrdopiera: But, do you have an example?12:56
*** ducttape_ has joined #openstack-horizon12:57
*** masco has quit IRC12:57
rdopierarobcresswell: that microversion patch, for instance13:00
rdopierahttps://review.openstack.org/#/c/410490/1/openstack_dashboard/api/base.py line 13913:00
robcresswellidk13:00
robcresswellI would at least log something and return None there13:01
robcresswellNot sure why it is just being swallowed. That patch is WIP and needs a lot of work though.13:01
*** ducttape_ has quit IRC13:01
rdopierayeah, but I've seen that a lot recently in various patches13:03
rdopieramaybe it's just selection bias13:03
robcresswellThe only thing I can think is that people are just sketching out their ideas in code and forgetting to update it later on13:04
rdopieraprobably13:05
rdopieragood thing we are reviewing it meticulously :)13:05
robcresswell:D13:05
*** ducttape_ has joined #openstack-horizon13:09
*** mvk has quit IRC13:15
robcresswellrdopiera: It might even be worth just removing all 'pass' and replacing it with a debug(e) statement.13:20
robcresswellAs in just logging the error... not sure why burying it is ever useful really.13:20
rdopierarobcresswell: it doesn't help much when people don't look at the test logs anyways13:20
rdopierarobcresswell: well, burying it is useful when you *expect* the error most of the time13:21
robcresswellI suppose13:21
rdopierarobcresswell: but then you should catch the exact error you expect13:21
robcresswellright13:21
*** brad[] has quit IRC13:28
robcresswellhow has this not merged :(13:29
robcresswellhttps://review.openstack.org/#/c/155939/13:29
rdopieraI guess because I didn't give workflow+1, to give you a chance to look at the nits :)13:29
robcresswellhaha13:30
rdopierabut if you think it's fine, do workflow+1 yourself13:30
robcresswellrdopiera: I was hoping ana would update it13:30
rdopieraah, no, sorry, that's nto your patch13:30
rdopieraignore what I just said13:30
*** chlong has quit IRC13:32
*** aortega has joined #openstack-horizon13:34
*** jdwald has joined #openstack-horizon13:35
*** ducttape_ has quit IRC13:45
*** brad[] has joined #openstack-horizon13:52
openstackgerritYakup Adaklı proposed openstack/horizon: AngularJS based panels page title updated  https://review.openstack.org/40975313:54
*** sacharya has joined #openstack-horizon14:03
*** sacharya has quit IRC14:08
openstackgerritRadomir Dopieralski proposed openstack/horizon: Properly compare versions in APIVersionManager  https://review.openstack.org/41068814:12
rdopierastupid json with its stupid serialization, grumble14:12
*** Pavo has quit IRC14:12
*** tqtran has joined #openstack-horizon14:12
*** Pavo has joined #openstack-horizon14:13
*** Pavo has quit IRC14:14
*** Pavo has joined #openstack-horizon14:14
*** ducttape_ has joined #openstack-horizon14:15
*** tqtran has quit IRC14:17
*** lblanchard has joined #openstack-horizon14:21
robcresswellrdopiera: :D14:29
robcresswellrdopiera: I'm still wrestling with how best to do microversions14:29
rdopieraI don't have an opinion14:32
robcresswellMy stance is still totally hostile, which is making it difficult to write logical code.14:33
robcresswelloh well.14:33
*** ducttape_ has quit IRC14:34
*** mvk has joined #openstack-horizon14:36
*** breitz has quit IRC14:37
*** breitz_ has joined #openstack-horizon14:37
*** jtriley has joined #openstack-horizon14:45
*** udesale has joined #openstack-horizon14:47
*** ruoyu has joined #openstack-horizon14:50
openstackgerritMerged openstack/horizon: Add generic session/cookie value getters/setters  https://review.openstack.org/15593914:50
*** mtanino has joined #openstack-horizon14:52
*** ^Gal^ has quit IRC15:01
*** mvk has quit IRC15:02
*** hurgleburgler has joined #openstack-horizon15:03
*** ChanServ sets mode: +v hurgleburgler15:03
*** lblanchard has quit IRC15:03
*** ruoyu has quit IRC15:05
*** ruoyu has joined #openstack-horizon15:05
openstackgerritRadomir Dopieralski proposed openstack/horizon: Move all Django configuration into .conf file  https://review.openstack.org/40473515:05
*** lblanchard has joined #openstack-horizon15:08
openstackgerritRadomir Dopieralski proposed openstack/horizon: Move all Django configuration into .conf file  https://review.openstack.org/40473515:09
*** mvk has joined #openstack-horizon15:16
*** ducttape_ has joined #openstack-horizon15:16
*** ruoyu has quit IRC15:16
*** chlong has joined #openstack-horizon15:21
*** ruoyu has joined #openstack-horizon15:21
*** tosky has quit IRC15:22
*** fedruantine_ has quit IRC15:30
rdopierarobcresswell: the way I see it, whichever way we go, it's going to be painful15:36
robcresswellrdopiera: I think I have a solution that is not too bad locally. I'm just poking it with sticks15:37
rdopierarobcresswell: so lets to the simplest thing that could possibly work, and let people handle special cases themselves, and when we have a better understanding and more actual use cases, we can replace it with something horrible15:37
robcresswellrdopiera: Currently the horrible part is having to test and hard code a version for every microversion feature :(15:37
ediardoo/15:38
rdopierarobcresswell: aren't most features supported with a >= ?15:38
rdopierarobcresswell: the ones for which you need to downgrade should be rare?15:38
robcresswellrdopiera: *most* yes, but not guaranteed15:38
rdopierasure, then let people code the exceptions15:39
robcresswellrdopiera: Things can also be supported for a range only15:39
robcresswellrdopiera: That will get hacky, quickly, and break without us knowing15:39
rdopierado we have exceptions of such features already?15:39
rdopieraor is this just theoretical?15:39
rdopieras/exceptions/examples15:39
robcresswellThere are already features in both cinder and nova that work only for a range15:39
rdopierasorry15:39
robcresswellgrep the codebase and scream15:40
rdopieraI think it's ok to have a hacky code for a hacky thing15:40
rdopieraif that makes majority of the code simpler15:40
robcresswellI don't want Horizon to have to maintain hacky code on top of hacky nova code15:40
robcresswellNo, it makes it more complex15:40
robcresswellThis way is simple code, but a little more maintenance of feature list15:41
rdopieraI think that coming up with a general solution that handles every possible case would be more complex?15:41
*** Marcellin__ has joined #openstack-horizon15:41
robcresswellNo, because you just fix the version for each feature15:41
robcresswellSo it can never break, really.15:41
rdopieraah, but then you create a new client for every call?15:41
rdopieraand a new connection15:42
rdopierarobcresswell: have you seen how they do they microversions in nova api?15:42
rdopierarobcresswell: they have a neat decorator for it15:42
robcresswellrdopiera: Yes, potentially. The design right now has a fixed list per feature, so if the microversion changes on a request it will use a new client15:43
robcresswellWell, decorators are just syntax sugar. I could write a neat decorator too but it doesnt change how shitty the implementation is :)15:44
rdopieraas long as nobody looks... ;)15:44
rdopierayay!15:44
robcresswellrdopiera: Look at examples like this: http://docs.openstack.org/developer/nova/api_microversion_dev.html#removing-an-api-method15:44
rdopieraI finally squashed the tempest problem with that config patch!15:44
rdopierarobcresswell: did you have a chance to take a look at it?15:45
robcresswellIts in the documentation to only use something between 2.1 and 2.4 for example, horrible.15:45
robcresswellnice one15:45
robcresswellno, not yet, sorry :(15:45
rdopierano worries, as long as I can guilt-pressure it into reviewing it someday :)15:45
rdopieras/it/you15:45
rdopierawell, not necessarily a proper review15:46
rdopierajust comments on the approach, etc.15:46
*** dave-mccowan has quit IRC15:46
*** udesale has quit IRC15:47
robcresswellhaha, yes its on my list, but then all of a sudden microversions and everyone decided to solve it without talking to each other15:48
rdopierathat's the best way15:48
robcresswellso now im doing magic implementation no.4 to fix the myriad implementations <insert relevant xkcd>15:48
rdopieraright after a staring contest, that is15:48
rdopieraby the way, that tempest-horizon failed legitimately15:50
rdopieraand no other test caught that problem15:50
robcresswell:D15:50
robcresswellgo temepst15:50
rdopieraso I think it is useful after all15:50
robcresswelltempest*15:50
rdopierathough it was difficult to figure out why it was failing15:50
robcresswellrdopiera: What was the cause of the failure?15:54
SamYapleHello. "Create New Volume" seems to be set to yes by default in Newton. How can I set that to "no" by default?15:55
rdopierarobcresswell: I was handling the default values for stuff like LOGIN_URL, etc. in the defaults for the conf file, but the python config, which gets loaded afterwards, changed WEBROOT15:55
robcresswellSamYaple: IIRC there's no way to change it without editing source15:56
rdopierarobcresswell: I had to put back the code that sets the values for LOGIN_URL, etc. based on the WEBROOT back in the settings.py15:56
*** tobberyd_ has joined #openstack-horizon15:56
SamYaplewow. really? that seems really bad. Thats really messed up some workflows causing lots of cinder volume confusion15:56
SamYapleany reason that was changed to be a default?15:57
*** _ducttape_ has joined #openstack-horizon15:57
robcresswellSamYaple: https://github.com/openstack/horizon/blob/e6647d4a0a176ddea82f44faae3cbce239ef69a6/openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/source/source.controller.js#L40615:57
*** GMAzrael has joined #openstack-horizon15:58
GMAzraelAnyone familiar with idle timeouts and NoVNC?15:58
robcresswellSamYaple: Feedback from others that making it default made more sense15:58
SamYaplerobcresswell: thanks for the link. perhaps this could be made configurable? I am not familiar enough with horizon to know if that is easy to do15:59
SamYaplerobcresswell: and thanks for the link15:59
SamYapleoh i said that :)15:59
*** tobberydberg has quit IRC15:59
*** ducttape_ has quit IRC16:00
*** tobberyd_ has quit IRC16:00
robcresswellSamYaple: Its fairly trivial, but not high on my list16:01
*** jlopezgu_ has joined #openstack-horizon16:02
robcresswellBest bet for a quick fix is just to modify the source. Or click No on that selection.16:02
robcresswellpatches welcome too, of course :)16:03
rdopieraSamYaple: if you submit a patch, the reviewers will help you get it into a state in which it can be merged16:03
*** sacharya has joined #openstack-horizon16:04
*** sacharya has quit IRC16:08
robcresswellrdopiera: Mind if I adopt your version patch code?16:09
rdopierarobcresswell: go ahead16:10
robcresswell\o/16:10
rdopieraI think all the clients have a similar thing in them16:10
robcresswellThey do16:10
rdopieralike novaclient.api_versions.APIVersion()16:10
robcresswellAnnoyingly its not standard, so will be difficult to rely upon I think16:10
robcresswellYeah, thats what I am currently using16:11
rdopieramaybe it's something to put in oslo?16:11
robcresswellit might be covered by versionedobject16:11
robcresswellidk16:11
*** Pavo has quit IRC16:12
robcresswellactually, the clients themselves will probably expect those objects, rather than a string16:13
robcresswellhmmm16:13
*** Pavo has joined #openstack-horizon16:13
rdopierathey don't accept strings too?16:13
*** rcernin has quit IRC16:14
robcresswellback into the source I go...16:14
openstackgerritAnkur proposed openstack/horizon: Add Reset-State Action for Admins  https://review.openstack.org/31563016:16
*** dave-mccowan has joined #openstack-horizon16:17
*** zackf has joined #openstack-horizon16:18
robcresswellrdopiera: https://github.com/openstack/python-novaclient/blob/master/novaclient/api_versions.py#L32616:20
robcresswellSo, yeah, if thats just a string it will explode16:20
rdopieraso we need a separate wrapper for every client?16:21
robcresswell:(16:21
rdopieraor we could add a get_string method to our Version object16:22
rdopieraand every other method that is necessary16:22
rdopieraas long as the implementations are not inconsistent...16:22
robcresswelland hope that Cinder also use get_string16:22
*** weezS has joined #openstack-horizon16:22
rdopieraif they use semthing else, that;s not a problem16:22
*** zackf has quit IRC16:22
rdopieraa problem would be if it used the same method, but expected it to return something else...16:22
robcresswellNo, its just extra hack to have a mnethod for each service16:23
robcresswellI wonder if it might be safer to just use each services APIVersion wrapper16:23
*** Foxlik1 has quit IRC16:24
*** zackf has joined #openstack-horizon16:26
rdopieraor wrap them to accept strings16:26
rdopierathis should really be in oslo16:27
rdopierapossible together with the whole microversion handling16:27
robcresswellyes, yes it should16:27
*** zackf has quit IRC16:30
*** zackf has joined #openstack-horizon16:30
*** mvk has quit IRC16:32
*** _ducttape_ has quit IRC16:32
*** ducttape_ has joined #openstack-horizon16:33
*** chlong has quit IRC16:45
*** rcernin has joined #openstack-horizon16:46
openstackgerritRichard Hagarty proposed openstack/horizon: Add microversion support for consistency groups  https://review.openstack.org/41086016:51
*** alexvictorchan has joined #openstack-horizon16:52
rdopierarhagarty: are you here?16:53
rhagartyrdopiera: yes, what's up?16:54
*** ctayal has joined #openstack-horizon16:54
*** yamamoto has quit IRC16:55
robcresswellrhagarty: We've got like 3/4 different version patches now16:56
robcresswellPlease don't work in isolation :(16:57
*** yamamoto has joined #openstack-horizon16:57
rhagartyrobcresswell: agreed... wasn't aware16:58
*** chlong has joined #openstack-horizon16:58
*** TravT has joined #openstack-horizon16:58
*** ChanServ sets mode: +v TravT16:58
rhagartyrobcresswell: just got in, so looking at comments now...16:59
*** piet has joined #openstack-horizon16:59
robcresswellThere's a few things, like it seems like you're using ranges again, which we've decided not to do17:00
robcresswellI'm just finishing up my local patch too, so we can see what works best17:00
rhagartyok - I will wait for that...17:01
*** yamamoto has quit IRC17:02
*** Zer0Byte__ has joined #openstack-horizon17:03
rdopierarobcresswell: by the way, I wonder if we could put the feature-version mapping in the client itself17:03
rdopierarobcresswell: and have the project's devs maintain it17:03
*** piet has quit IRC17:05
*** sacharya has joined #openstack-horizon17:05
*** GMAzrael has quit IRC17:11
*** edtubill has joined #openstack-horizon17:14
*** jpich has quit IRC17:18
*** tqtran has joined #openstack-horizon17:21
rhagartyrobcresswell: so how to proceed? I think cinder provides a good POC env - we have a feature added in mv 3.11 and another going away post 3.1117:29
*** weezS has quit IRC17:42
*** piet has joined #openstack-horizon17:45
*** ctayal has quit IRC17:51
*** tosky has joined #openstack-horizon17:54
*** yamamoto has joined #openstack-horizon17:56
*** Jeffrey4l has quit IRC17:58
*** mvandijk has left #openstack-horizon17:59
*** yamamoto has quit IRC18:01
robcresswellrhagarty: I'll get my patch up once I finish documenting and testing18:03
robcresswellrhagarty: Should be next 24 hrs18:03
robcresswellI think it is fairly complete now in adding nova/cinder support in a fairly clean way.18:04
*** jlopezgu_ has quit IRC18:04
robcresswellrhagarty: Cinder client v2 is gone right? We should just use v3?18:05
rhagartystill supported18:06
robcresswellack18:07
rhagartyyeah, it would make life easier (and cleaner)18:10
*** Pavo has quit IRC18:12
*** Pavo has joined #openstack-horizon18:17
*** david-lyle has quit IRC18:21
*** yamamoto has joined #openstack-horizon18:26
*** yamamoto has quit IRC18:27
*** yamamoto has joined #openstack-horizon18:27
*** yamamoto has quit IRC18:27
*** david-lyle has joined #openstack-horizon18:30
*** ChanServ sets mode: +o david-lyle18:30
*** ctayal has joined #openstack-horizon18:31
*** piet has quit IRC18:31
*** ruoyu has quit IRC18:38
*** ducttape_ has quit IRC18:39
*** jprovazn has quit IRC18:48
*** ducttape_ has joined #openstack-horizon18:50
*** wolverineav has quit IRC18:55
*** tosky has quit IRC19:00
*** weezS has joined #openstack-horizon19:01
*** wolverineav has joined #openstack-horizon19:02
*** wolverin_ has joined #openstack-horizon19:06
*** wolverineav has quit IRC19:06
*** stupidnic has quit IRC19:16
*** stupidnic has joined #openstack-horizon19:18
*** ctayal_ has joined #openstack-horizon19:25
*** ctayal has quit IRC19:27
*** yamamoto has joined #openstack-horizon19:28
*** ctayal has joined #openstack-horizon19:29
*** wolverin_ has quit IRC19:29
*** ctayal_ has quit IRC19:32
*** yamamoto has quit IRC19:36
openstackgerritMerged openstack/horizon: Replace six.iteritems(iter) with iter.items()  https://review.openstack.org/40961519:47
*** gugl has joined #openstack-horizon19:50
r1chardj0n3smorning o/19:52
*** TravT has quit IRC19:52
r1chardj0n3sHorizon weekly meeting in 8 minutes in #openstack-meeting-319:53
*** TravT has joined #openstack-horizon19:56
*** ChanServ sets mode: +v TravT19:56
r1chardj0n3sthis thing has now started ^^ :-)20:01
*** ruoyu has joined #openstack-horizon20:07
*** Pavo has quit IRC20:12
*** Pavo has joined #openstack-horizon20:17
*** ruoyu has quit IRC20:19
*** ruoyu has joined #openstack-horizon20:19
*** ruoyu has quit IRC20:23
*** ctayal has quit IRC20:26
*** ruoyu has joined #openstack-horizon20:30
*** TravT has quit IRC20:31
*** pcaruana has quit IRC20:33
*** TravT has joined #openstack-horizon20:37
*** ChanServ sets mode: +v TravT20:37
*** tobberydberg has joined #openstack-horizon20:39
*** clu_ has joined #openstack-horizon20:40
*** ChanServ sets mode: +v clu_20:40
r1chardj0n3sgood meeting, thanks all!20:45
rdopieraon my last project, our mascot was a house fly20:45
r1chardj0n3s... because?20:46
rdopieraso doge is an improvemtn20:46
rdopierar1chardj0n3s: because it was called Ralph20:46
r1chardj0n3s... ok :-)20:46
*** mvk has joined #openstack-horizon20:46
rdopierar1chardj0n3s: after a character from Lord of Flies20:46
r1chardj0n3sahh!20:47
rdopieraI wanted to use a dead pig's head20:48
rdopierabut was overruled20:49
ducttape_we have a manager we call "lord of the flies" around here20:49
r1chardj0n3syou guys have the funnest workplaces20:49
*** _ducttape_ has joined #openstack-horizon20:58
robcresswellhttp://www.abc.net.au/news/image/1086698-3x2-940x627.jpg21:00
robcresswellr1chardj0n3s: ^^ still makes me laugh21:00
*** ducttape_ has quit IRC21:02
r1chardj0n3shaha, yeah, it's a shame it's fake ;-)21:02
r1chardj0n3sthough "catastrophic" is waaay better than the actual "code red"21:02
r1chardj0n3ssince the previous danger level is also red21:02
*** tobberydberg has quit IRC21:06
*** weezS has quit IRC21:08
openstackgerritMerged openstack/horizon: Text and checkbox should align in "Create Project" form  https://review.openstack.org/40980421:08
*** _ducttape_ has quit IRC21:09
robcresswellHmm, that looks like a CSS hack...21:10
*** ducttape_ has joined #openstack-horizon21:10
*** ying_zuo_ has joined #openstack-horizon21:13
*** ying_zuo_ has quit IRC21:15
*** ianychoi has quit IRC21:28
*** erlon has quit IRC21:30
*** adriant has joined #openstack-horizon21:39
*** itisha has quit IRC21:42
*** betherly_ has quit IRC21:51
*** weezS has joined #openstack-horizon22:01
*** jtriley has quit IRC22:07
*** aortega has quit IRC22:09
*** lblanchard has quit IRC22:10
*** Pavo has quit IRC22:12
*** Pavo has joined #openstack-horizon22:17
*** edtubill has quit IRC22:26
*** ctayal has joined #openstack-horizon22:34
*** jdwald has quit IRC22:34
*** alexvictorchan has quit IRC22:35
*** ruoyu has quit IRC22:39
*** ruoyu has joined #openstack-horizon22:39
*** aortega has joined #openstack-horizon22:49
openstackgerritVictoria Martinez de la Cruz proposed openstack/manila-ui: Only show available share protocols on share creation modal  https://review.openstack.org/37280522:50
openstackgerritCindy Lu proposed openstack/horizon: NG Flavor Panel add filter facets  https://review.openstack.org/36695722:50
*** ducttape_ has quit IRC22:51
*** ruoyu has quit IRC22:52
*** ruoyu has joined #openstack-horizon22:54
*** dave-mccowan has quit IRC22:56
*** ruoyu has quit IRC22:57
*** ruoyu has joined #openstack-horizon22:57
*** ruoyu has quit IRC22:59
*** ruoyu has joined #openstack-horizon22:59
*** alexvictorchan has joined #openstack-horizon23:00
*** ruoyu has quit IRC23:04
*** rcernin has quit IRC23:05
*** mtanino has quit IRC23:15
*** mtanino has joined #openstack-horizon23:15
*** dave-mccowan has joined #openstack-horizon23:15
*** aortega has quit IRC23:16
*** ruoyu has joined #openstack-horizon23:18
*** ruoyu_ has joined #openstack-horizon23:20
*** ruoyu has quit IRC23:20
*** ruoyu has joined #openstack-horizon23:22
*** ruoyu__ has joined #openstack-horizon23:23
*** ducttape_ has joined #openstack-horizon23:24
*** ianychoi has joined #openstack-horizon23:25
*** ruoyu_ has quit IRC23:25
*** ruoyu has quit IRC23:26
*** dave-mccowan has quit IRC23:27
*** ruoyu__ has quit IRC23:28
*** ruoyu has joined #openstack-horizon23:31
*** ruoyu_ has joined #openstack-horizon23:32
*** ruoyu__ has joined #openstack-horizon23:34
*** ruoyu has quit IRC23:35
*** ruoyu_ has quit IRC23:36
*** chlong has quit IRC23:37
*** ruoyu__ has quit IRC23:38
*** ppiela has quit IRC23:51

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