Friday, 2017-06-02

*** markstur has quit IRC00:00
*** markstur has joined #openstack-manila00:29
tbarronsomething is breaking the releasenotes jobs but I haven't been able to see what it is.00:31
tbarronI think we've always treated warnings as errors, but it's breaking on a warning regarding releasenotes/notes/bug_1564623_change-e286060a27b02f64.yaml00:34
tbarron"The fixes section of releasenotes/notes/bug_1564623_change-e286060a27b02f64.yaml does not parse as a list of strings. Is the YAML input escaped properly?"00:34
tbarronI don't see anything wrong with the *current* contents of that file, but it has an interesting history:00:35
tbarronhttps://paste.fedoraproject.org/paste/agV55knvr~~AhRhhjBqmcl5M1UNdIGYhyRLivL9gydE=00:37
tbarronit's as if the old bad commit is causing a warning that is making the releasenotes job fail ....00:38
tbarronbut all the changes around that reno file were a long time ago.00:38
tbarronFresh eyes might see what's going on ...  I don't have a reno I'm trying to commit right now, just noticed the problem.00:38
*** markstur_ has joined #openstack-manila01:05
*** markstur has quit IRC01:09
*** tuanluong has joined #openstack-manila01:31
*** kaisers_ has joined #openstack-manila01:51
*** kaisers has quit IRC01:55
zhongjuntbarron: It also have the releasenotes warning after I remove the bug_1564623_change-e286060a27b02f64.yaml file(https://review.openstack.org/469407). It sounds like the gate run all the historical versions.  The error came out in May 3101:56
zhongjuntbarron: Maybe It could have some changed about openstack reno or some other related project for 3 days ago01:59
zhongjunhey, anyone would like to take a look at this patch? The API change has been merged.  But we still need to do it in manilaclient.   https://review.openstack.org/#/c/468287/  thanks in advance :)02:10
*** markstur_ has quit IRC02:48
openstackgerritzhongjun proposed openstack/manila master: Use uwsgi for devstack  https://review.openstack.org/46938902:48
openstackgerritzhongjun proposed openstack/manila master: Use uwsgi for devstack  https://review.openstack.org/46938902:56
*** markstur has joined #openstack-manila03:07
*** winston-d_ has joined #openstack-manila03:12
*** markstur_ has joined #openstack-manila03:28
*** markstur has quit IRC03:31
openstackgerritzhongjun proposed openstack/manila master: TEST  https://review.openstack.org/46940703:46
*** arnewiebalck_ has joined #openstack-manila04:12
*** markstur_ has quit IRC04:55
*** markstur_ has joined #openstack-manila05:10
*** gouthamr has quit IRC05:15
*** arnewiebalck_ has quit IRC05:20
*** markstur_ has quit IRC05:41
*** markstur has joined #openstack-manila06:02
*** markstur has quit IRC06:06
*** kaisers_ has quit IRC06:13
*** kaisers has joined #openstack-manila06:13
*** jprovazn has joined #openstack-manila06:56
*** markstur has joined #openstack-manila07:01
*** dsariel has joined #openstack-manila07:03
*** dmellado_ has joined #openstack-manila07:05
*** dmellado_ has quit IRC07:05
*** markstur has quit IRC07:06
zhongjuntbarron: Hi ,  I found the reason about why we need the releasenotes gate error07:31
zhongjuns/need/meet07:31
zhongjuntbarron: We miss a '-' in bug_1564623_change-e286060a27b02f64.yaml  in mitaka, please see:  https://github.com/openstack/manila/blob/stable/mitaka/releasenotes/notes/bug_1564623_change-e286060a27b02f64.yaml07:33
zhongjuntbarron: We have a problem in master because the releasenotes gate run all the version.07:34
zhongjuntbarron: We only get error in mitaka,  not in other versions.07:36
*** markstur has joined #openstack-manila07:45
zhongjuntbarron, ganso, bswartz, gouthamr: I just change it in this patch, please take a look: https://review.openstack.org/#/c/470182/07:47
*** lpetrut has joined #openstack-manila07:48
*** markstur has quit IRC07:50
*** a-pugachev has joined #openstack-manila07:52
*** arnewiebalck_ has joined #openstack-manila07:56
*** csaba has quit IRC07:59
*** csaba has joined #openstack-manila08:00
*** dsariel has quit IRC08:03
*** markstur has joined #openstack-manila08:05
openstackgerritOpenStack Proposal Bot proposed openstack/manila-ui master: Imported Translations from Zanata  https://review.openstack.org/47019008:07
*** markstur has quit IRC08:09
zhongjuntbarron: It just change the logic in reno:  link: https://github.com/openstack/reno/commit/e2d60c07931f9095d5e4d9a45535f5ce1a7e0dd308:12
zhongjuns/change/changed08:12
*** markstur has joined #openstack-manila08:54
*** arnewiebalck_ has quit IRC08:58
*** markstur has quit IRC08:59
zhongjuntbarron: We get the error:  http://logs.openstack.org/82/470182/1/check/gate-manila-releasenotes/a74776b/console.html#_2017-06-02_07_49_34_618393, because of the reno project changed about earliest_version( https://github.com/openstack/reno/commit/e2d60c07931f9095d5e4d9a45535f5ce1a7e0dd3).09:05
openstackgerritzhongjun proposed openstack/manila master: Remove pbr warnerrors in favor of sphinx check  https://review.openstack.org/46940709:12
*** markstur has joined #openstack-manila09:19
*** a-pugachev has quit IRC09:21
*** markstur has quit IRC09:24
openstackgerritValeriy Ponomaryov proposed openstack/manila master: Add quotas per share type  https://review.openstack.org/45215809:41
*** a-pugachev has joined #openstack-manila09:42
tbarronzhongjun: interesting09:44
tbarronzhongjun: I'm not sure the relevant variable is the version of reno though09:44
tbarronzhongjun: both reno and sphinx versions seem to be getting set by global-requirements, so our settings in test-requirements don't really have effect09:45
tbarronzhongjun: as an experiment, I changed tox.ini to no longer use global requirements and played with the reno and sphinx versions09:46
tbarronif I back off reno to 2.3.0 I still get that error09:46
tbarronif I back off reno to 2.2.0 I get a different error (warning as error) but it still fails09:47
tbarronif I leave reno at 2.3.1 and back off sphinx to 1.5.6 then the releasenotes build OK09:48
tbarronmaybe sphinx 'treat warnings as error' argument is stricter now??09:49
zhongjuntbarron: maybe, No one knows what they've done :)09:51
zhongjuntbarron: It is better to fix the releasenote bug in mitaka version09:55
tbarronzhongjun: agree,09:55
tbarronzhongjun: if we can, isn't mitaka EOL?09:55
tbarronzhongjun: even if it is maybe bswartz can work some magic09:56
zhongjuntbarron: It looks like it is not EOL, I put a pathch in mitaka(https://review.openstack.org/470182)09:57
tbarronzhongjun: cool09:57
zhongjuntbarron: sure, bswartz can work some magic09:58
zhongjuntbarron: haha09:58
*** jprovazn has quit IRC10:00
openstackgerritMerged openstack/manila-ui master: Imported Translations from Zanata  https://review.openstack.org/47019010:11
*** tuanluong has quit IRC10:35
*** ganso has joined #openstack-manila10:41
openstackgerritValeriy Ponomaryov proposed openstack/manila-ui master: Add support of share groups  https://review.openstack.org/46889910:43
*** markstur has joined #openstack-manila10:56
*** markstur has quit IRC11:00
openstackgerritRamana Raja proposed openstack/manila master: WIP ganesha: dynamically update access of share  https://review.openstack.org/42459811:13
openstackgerritRamana Raja proposed openstack/manila master: WIP cephfs/driver: add nfs protocol support  https://review.openstack.org/42460511:16
*** jprovazn has joined #openstack-manila11:30
*** gcb has quit IRC11:34
*** gcb has joined #openstack-manila11:38
*** gcb has quit IRC11:43
*** gcb has joined #openstack-manila11:52
*** catintheroof has joined #openstack-manila12:08
openstackgerritValeriy Ponomaryov proposed openstack/manila-ui master: Add support of share groups  https://review.openstack.org/46889913:00
*** gcb has quit IRC13:04
*** kaisers has quit IRC13:13
bswartzvponomaryov: https://review.openstack.org/#/c/446044/11/manila_tempest_tests/tests/api/base.py13:25
tbarronbswartz: the theory is that you need to merge https://review.openstack.org/#/c/470182/ so that releasenotes job will work in master13:29
tbarronbswartz: reno cares about past history and old releases13:30
tbarronbswartz: and it has gotten stricter13:30
bswartzand stable/mitaka is still open?13:30
bswartzlong live EOL branches I guess13:30
tbarronbswartz: I think mitaka is EOL in theory but appears to be still open13:30
bswartzschrodinger's branch13:31
tbarronbswartz: I think this will take a ninja merge b/c of missing dependencies that cause the lvm devstack build to fail13:31
bswartzwe won't know until we open the box13:31
tbarronbswartz: but that has nothing to do with the reno yaml syntax fix13:31
tbarronbswartz: meoooow13:32
*** markstur has joined #openstack-manila13:42
*** cknight has joined #openstack-manila13:46
*** markstur has quit IRC13:46
bswartztbarron: so how can we fix the LVM job and get this patch to merge?13:49
bswartzis there another patch that solve the mitaka gate problem?13:50
tbarronbswartz: not that I'm aware of.  Can you just force it to merge and EOL mitaka?13:50
bswartzthere's no such thing as a "force to merge"13:50
tbarronmitaka is EOL b/c no one is keeping dependencies for it correct anymore.13:50
bswartzit has to pass the gate13:50
bswartzall we can do is disable the gate job for mitaka13:51
bswartzwhich is an infra change13:51
*** MVenesio has joined #openstack-manila13:51
*** gouthamr has joined #openstack-manila13:51
bswartzis there any hope of solving the dependency problem and getting the job to pass?13:51
bswartzif not, then we should submit a change to infra to turn off that stable job13:51
tbarronbswartz: I haven't looked at it; can do so at 11 but have to go to a meeting now.13:52
tbarronbswartz: if we're going to EOL it officially then we would remove that job anyways, but let me see if there's a fix that isn't too onerous.13:53
bswartztbarron: http://logs.openstack.org/82/470182/2/check/gate-manila-tempest-minimal-dsvm-lvm-ubuntu-trusty/d8c8161/logs/devstack-early.txt.gz13:54
*** rraja has joined #openstack-manila13:59
*** eharney has joined #openstack-manila14:05
*** markstur has joined #openstack-manila14:06
*** dsariel has joined #openstack-manila14:08
*** markstur has quit IRC14:11
*** jmlowe has quit IRC14:29
*** markstur has joined #openstack-manila14:39
*** catintheroof has quit IRC14:45
openstackgerritValeriy Ponomaryov proposed openstack/manila master: [Share groups] Add scheduler filter ConsistentSnapshotFilter  https://review.openstack.org/44604414:49
*** jmlowe has joined #openstack-manila14:51
*** jprovazn has quit IRC15:00
tbarronbswartz: back15:18
tbarronbswartz: that's weird b/c RABBIT_PASSWORD was set: http://logs.openstack.org/82/470182/2/check/gate-manila-tempest-minimal-dsvm-lvm-ubuntu-trusty/d8c8161/logs/local.conf.txt.gz15:18
openstackgerritValeriy Ponomaryov proposed openstack/manila-ui master: Add support of share groups  https://review.openstack.org/46889915:19
bswartztbarron yeah...15:21
*** vponomaryov has quit IRC15:23
*** markstur has quit IRC15:24
*** dims has quit IRC15:28
*** markstur has joined #openstack-manila15:29
*** lpetrut has quit IRC16:03
*** arnewiebalck_ has joined #openstack-manila16:04
*** gaurangt_ has quit IRC16:08
*** gaurangt has joined #openstack-manila16:11
*** dsariel has quit IRC16:14
*** pcaruana has quit IRC16:16
tbarronbswartz: put this one in the pipeline in case ... https://review.openstack.org/#/c/470358/16:18
*** a-pugachev has quit IRC16:22
*** rraja has quit IRC16:38
*** arnewiebalck_ has quit IRC16:41
bswartztbarron: precisely what I had in mind16:43
bswartztbarron: got a minute?16:43
tbarronsure16:48
*** eharney has quit IRC16:48
*** eharney has joined #openstack-manila17:03
*** dims has joined #openstack-manila17:10
openstackgerritVictoria Martinez de la Cruz proposed openstack/manila master: Add share notifications  https://review.openstack.org/46658617:15
*** xyang1 has joined #openstack-manila17:17
*** dims has quit IRC17:24
*** dims has joined #openstack-manila17:26
*** harlowja has quit IRC17:28
vkmcgouthamr, hey there! I'm aware you have been involved on the replication work and I was a bit dubious on how the notifications I'm adding would behave on that case17:48
gouthamrvkmc: hey! what kind of notifications did you have in mind?'17:48
vkmcgouthamr, create share notifications precisely17:49
gouthamrvkmc: the provisioning notifications would be similar17:49
gouthamrvkmc: to all other resources: shares, networks, types, groups.17:50
vkmcgouthamr, I was thinking... when a new replica is created, shall we notify about those new shares?17:50
vkmcyeah17:50
gouthamrvkmc: yep.. 2 mins.. brb17:52
tbarrongouthamr: vkmc one thing I wanted to sanity check was what will happen with replicas and "share notifications"17:52
tbarronthe "share notifications" are really triggered on share-instance creates and deletes ...17:52
vkmctbarron, yeah, looking at the code it seems as it shouldn't be a problem... like, create_replica doesn't rely on create_share_instance on manager.py17:53
tbarronshare-instances are hidden from regular cloud users but visible to admins17:53
vkmcso the "share.create" notification won't be triggered when we create replicas17:53
vkmcyeah, I stick to that premise17:54
tbarronvkmc: k, probably fine then, but getting goutham to nod his head when he gets back would be good too17:54
vkmcof course17:54
gouthamrvkmc: yep.. https://github.com/openstack/manila/blob/master/manila/scheduler/manager.py#L266 was intentionally different17:56
vkmcyeah17:56
vkmcgouthamr++17:57
gouthamri like this :) maybe we can start sending our health monitoring notifications next17:57
vkmcyeah :D17:58
openstackgerritMerged openstack/manila master: Remove pbr warnerrors in favor of sphinx check  https://review.openstack.org/46940718:06
*** dims has quit IRC18:16
*** dims has joined #openstack-manila18:17
*** markstur has quit IRC18:30
*** markstur has joined #openstack-manila18:30
*** markstur has quit IRC18:35
*** catintheroof has joined #openstack-manila18:38
*** catintheroof has quit IRC19:02
*** a-pugachev has joined #openstack-manila19:03
*** eharney has quit IRC19:07
*** dsariel has joined #openstack-manila19:12
*** eharney has joined #openstack-manila19:20
*** a-pugachev has quit IRC19:24
openstackgerritOpenStack Proposal Bot proposed openstack/manila master: Updated from global requirements  https://review.openstack.org/46957019:25
*** hongbin has joined #openstack-manila19:43
*** dsariel has quit IRC19:44
*** eharney has quit IRC19:56
*** harlowja has joined #openstack-manila20:04
*** eharney has joined #openstack-manila20:11
*** markstur has joined #openstack-manila20:31
*** gouthamr has quit IRC20:41
*** cknight has quit IRC20:49
*** winston-d_ has quit IRC20:55
*** jmlowe has quit IRC20:57
*** gouthamr has joined #openstack-manila20:59
*** kaisers1 has quit IRC21:07
*** eharney has quit IRC21:15
*** cknight has joined #openstack-manila21:31
*** MVenesio has quit IRC21:39
openstackgerritOpenStack Proposal Bot proposed openstack/manila master: Updated from global requirements  https://review.openstack.org/46957021:57
openstackgerritOpenStack Proposal Bot proposed openstack/manila-ui master: Updated from global requirements  https://review.openstack.org/47046121:57
*** kaisers has joined #openstack-manila22:04
openstackgerritOpenStack Proposal Bot proposed openstack/python-manilaclient master: Updated from global requirements  https://review.openstack.org/47014122:06
*** ganso has quit IRC22:20
*** andreaf has quit IRC22:35
*** andreaf has joined #openstack-manila22:40
*** gouthamr has quit IRC22:40
*** gouthamr has joined #openstack-manila22:42
*** cFouts has quit IRC22:42
*** gnarld_ has joined #openstack-manila22:45
*** cknight has quit IRC22:49
*** ianychoi has quit IRC22:59
*** ianychoi has joined #openstack-manila23:01
*** xyang1 has quit IRC23:14

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