Wednesday, 2017-11-29

*** schmots has quit IRC00:05
*** caowei has joined #openstack-manila01:01
openstackgerritzhongjun proposed openstack/manila master: [policy in code] Add support for service and quota resource [5/10]  https://review.openstack.org/52290901:13
openstackgerritzhongjun proposed openstack/manila master: [policy in code] Add support for group resource [6/10]  https://review.openstack.org/52331801:33
zhongjuntbarron  bswartz gouthamr: This is ready for review for policy in code feature :) https://review.openstack.org/#/q/project:openstack/manila++topic:policy-and-docs-in-code+(status:open+OR+status:merged)  thanks in advance01:35
openstackgerritMerged openstack/puppet-manila master: Prepare for Queens Milestone 2  https://review.openstack.org/52358201:59
openstackgerritzhongshengping proposed openstack/puppet-manila master: Deprecate os_region_name option  https://review.openstack.org/52329102:15
*** junboli has joined #openstack-manila02:20
*** raissa has quit IRC02:58
*** threestrands has joined #openstack-manila03:05
*** jiaopengju_ has joined #openstack-manila03:21
*** jiaopengju_ has left #openstack-manila03:23
*** jiaopengju has joined #openstack-manila03:24
gouthamrzhongjun: late ack, have that in my backlog - will review03:30
*** caowei has quit IRC03:40
*** caowei has joined #openstack-manila03:41
zhongjungouthamr: thanks :)03:43
*** threestrands has quit IRC04:29
*** threestrands has joined #openstack-manila04:41
*** threestrands has quit IRC04:41
*** threestrands has joined #openstack-manila04:41
*** gouthamr has quit IRC04:43
*** eharney has quit IRC05:00
*** caowei has quit IRC05:04
*** sticker has quit IRC05:08
*** caowei has joined #openstack-manila05:25
*** threestrands has quit IRC05:32
*** gouthamr has joined #openstack-manila05:46
*** gouthamr has quit IRC06:04
openstackgerritzhongjun proposed openstack/manila master: [policy in code] Add support for replica and network resource [7/10]  https://review.openstack.org/52369606:17
openstackgerritChris Yang proposed openstack/manila master: QNAP: Add support for QES 1.1.4  https://review.openstack.org/52301706:18
openstackgerritzhongjun proposed openstack/manila master: [policy in code] Add support for replica and network resource [7/10]  https://review.openstack.org/52369606:18
*** lpetrut has joined #openstack-manila06:38
*** junboli has quit IRC07:15
*** dsariel has joined #openstack-manila07:26
*** lpetrut has quit IRC07:37
*** AlexeyAbashkin has joined #openstack-manila07:47
*** lpetrut has joined #openstack-manila07:53
*** Manor has joined #openstack-manila07:53
*** lpetrut has quit IRC07:54
*** Manor has quit IRC07:56
*** Manor has joined #openstack-manila07:56
openstackgerritzhongjun proposed openstack/manila master: [policy in code] Add support for replica and network resource [7/10]  https://review.openstack.org/52369607:59
*** Manor_ has joined #openstack-manila08:02
*** pcaruana has joined #openstack-manila08:04
*** Manor has quit IRC08:05
*** Manor_ has quit IRC08:55
*** Manor has joined #openstack-manila08:56
openstackgerritOpenStack Proposal Bot proposed openstack/manila master: Updated from global requirements  https://review.openstack.org/52374209:00
openstackgerritzhongjun proposed openstack/manila master: [policy in code] Add support for share and type extra resource [8/10]  https://review.openstack.org/52377009:11
openstackgerritOpenStack Proposal Bot proposed openstack/python-manilaclient master: Updated from global requirements  https://review.openstack.org/52379409:18
*** Manor_ has joined #openstack-manila09:29
*** Manor has quit IRC09:31
openstackgerritzhongjun proposed openstack/manila master: [policy in code] Add support for AZ, scheduler and message resource [9/10]  https://review.openstack.org/52381909:35
openstackgerritzhongjun proposed openstack/manila master: [policy in code] Add support for AZ, scheduler and message resource [9/10]  https://review.openstack.org/52381909:37
openstackgerritzhongjun proposed openstack/manila master: [policy in code] Add support for AZ, scheduler and message resource [9/10]  https://review.openstack.org/52381909:38
*** caowei has quit IRC09:38
*** gcb has joined #openstack-manila09:39
*** a-pugachev has joined #openstack-manila09:46
*** caowei has joined #openstack-manila09:50
*** ganso has joined #openstack-manila09:51
*** Manor_ has quit IRC10:05
*** a-pugachev has quit IRC10:11
openstackgerritChris Yang proposed openstack/manila master: QNAP: Add support for QES 1.1.4  https://review.openstack.org/52301710:13
*** a-pugachev has joined #openstack-manila10:21
*** caowei has quit IRC10:27
*** AlexeyAbashkin has quit IRC10:28
amito-infinidatAnyway to run local cover tests in my manila environment? zhongjun maybe you can help? :)10:57
*** a-pugachev has quit IRC11:03
*** gcb has quit IRC11:05
*** AlexeyAbashkin has joined #openstack-manila11:19
*** a-pugachev has joined #openstack-manila11:21
*** AlexeyAbashkin has quit IRC11:23
*** erlon has joined #openstack-manila11:23
*** AlexeyAbashkin has joined #openstack-manila11:27
*** raissa has joined #openstack-manila12:12
*** eharney has joined #openstack-manila12:20
*** threestrands has joined #openstack-manila12:42
*** threestrands has quit IRC12:42
*** threestrands has joined #openstack-manila12:42
openstackgerritAmit Oren proposed openstack/manila master: INFINIDAT add Manila driver  https://review.openstack.org/51543212:46
*** threestrands has quit IRC13:01
*** schmots has joined #openstack-manila13:06
*** eharney has quit IRC13:07
*** openstackstatus has quit IRC13:43
*** openstack has joined #openstack-manila13:46
*** ChanServ sets mode: +o openstack13:46
*** eharney has joined #openstack-manila13:54
tbarronamito-infinidat: run 'tox -ecover' then look in the cover subdirectory, you can run a browser on that subdirectory13:56
amito-infinidattbarron: thanks, it fails when trying to install the lxml dependency.13:56
*** dustins has joined #openstack-manila13:58
tbarronamito-infinidat: i'm running it now to see if I hit that issue14:14
*** dsariel has quit IRC14:20
tbarronamito-infinidat: it worked for me.  My advice would be to clone a new manila workspace, run 'tox -e cover' and if that works cherry-pick your changes on top of it and run it again.14:20
tbarronamito-infinidat: you could also just 'rm -rf .tox' and run the command again in your current workspace14:21
tbarronamito-infinidat: that will be a bit faster if it works; the first suggestion should however ensure that all requirements are up to date, etc. too14:21
amito-infinidattbarron: how much coverage is needed for the tox-cover job to succeed?14:27
tbarronamito-infinidat: it's a non-voting job and it's criteria for success is crude.14:28
tbarronamito-infinidat: it checks out a workspace at the previous commit & collects coverage results14:29
tbarronamito-infinidat: then with your candidate commit14:29
tbarronamito-infinidat: if there are four lines or more less being covered with the new commit it fails14:29
tbarronamito-infinidat: we don't really treat negatives on that report as decisive14:29
tbarronamito-infinidat: but in your review gouthamr looked at the actual coverage reports for your new functional code ...14:30
tbarronamito-infinidat: does that make sense?14:31
amito-infinidattbarron: what do you mean? wasn't what gouthamr pointed to extracted from the job results?14:32
* tbarron looks at the review again14:32
amito-infinidat:)14:33
tbarronamito-infinidat: his remarks on patch set 14 w.r.t. lines 227, 480, 497 have links to the coverage job reports w.r.t. manila_share_drivers_infinidat.py14:34
tbarronamito-infinidat: you can see there in red lines not covered by unit tests and those he cites correspond to changes you are making in your patch14:35
tbarronamito-infinidat: so you can modify your unit tests to cover these branches in the code path (basically testing exception paths so far as I can see from a quick look) and check this output file after you run the coverage test again14:36
amito-infinidattbarron: OK, so this is indeed from the tox-cover job14:37
amito-infinidattbarron: thanks :)14:37
tbarronamito-infinidat: yw, and good luck.  Looks like you're making good progress.14:37
*** openstack has quit IRC14:46
*** openstack has joined #openstack-manila14:51
*** ChanServ sets mode: +o openstack14:51
*** a-pugachev has quit IRC14:53
*** a-pugachev_ is now known as a-pugachev14:53
openstackgerritAmit Oren proposed openstack/manila master: INFINIDAT add Manila driver  https://review.openstack.org/51543215:11
*** gouthamr has joined #openstack-manila15:22
*** dsariel has joined #openstack-manila15:24
*** dsariel has quit IRC15:42
*** dsariel has joined #openstack-manila15:42
*** a-pugachev has quit IRC16:01
*** a-pugachev has joined #openstack-manila16:03
openstackgerritMerged openstack/python-manilaclient master: Increase code coverage  https://review.openstack.org/51496416:08
*** AlexeyAbashkin has quit IRC16:23
*** a-pugachev has quit IRC16:24
*** dsariel has quit IRC16:26
*** dsariel has joined #openstack-manila16:28
*** a-pugachev has joined #openstack-manila16:32
openstackgerritkedy zhao proposed openstack/manila master: fix allow the use of blank in user group name to access the share  https://review.openstack.org/52244016:41
openstackgerritkedy zhao proposed openstack/python-manilaclient master: fix allow the use of blank in user group name to access the share  https://review.openstack.org/52244416:45
*** eharney has quit IRC16:50
*** eharney has joined #openstack-manila16:54
amito-infinidatzhongjun gouthamr: the new patch set is ready :) https://review.openstack.org/51543217:04
*** dsariel has quit IRC17:09
openstackgerritMerged openstack/manila master: Updated from global requirements  https://review.openstack.org/52374217:15
*** vivsoni_ has joined #openstack-manila17:31
*** a-pugachev has quit IRC17:47
*** vivsoni_ has quit IRC18:32
*** ociuhandu has joined #openstack-manila19:32
*** ociuhandu has quit IRC19:36
openstackgerritMerged openstack/python-manilaclient master: Updated from global requirements  https://review.openstack.org/52379420:01
*** AlexeyAbashkin has joined #openstack-manila20:14
*** a-pugachev has joined #openstack-manila20:15
*** AlexeyAbashkin has quit IRC20:18
*** threestrands has joined #openstack-manila20:47
*** threestrands has quit IRC20:47
*** threestrands has joined #openstack-manila20:47
*** erlon has quit IRC20:53
*** pcaruana has quit IRC21:00
*** sticker has joined #openstack-manila21:22
*** ganso has quit IRC21:27
*** eharney has quit IRC22:05
*** dustins has quit IRC22:08
*** jmlowe has quit IRC22:16
*** jmlowe has joined #openstack-manila22:30
*** schmots has quit IRC22:51
*** schmots has joined #openstack-manila23:07
*** hoonetorg has joined #openstack-manila23:10
*** jmlowe has quit IRC23:13
*** AlexeyAbashkin has joined #openstack-manila23:14
*** AlexeyAbashkin has quit IRC23:19
*** jmlowe has joined #openstack-manila23:21
*** absubram has joined #openstack-manila23:55

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