*** markstur has quit IRC | 00:00 | |
*** markstur has joined #openstack-manila | 00:29 | |
tbarron | something is breaking the releasenotes jobs but I haven't been able to see what it is. | 00:31 |
---|---|---|
tbarron | I think we've always treated warnings as errors, but it's breaking on a warning regarding releasenotes/notes/bug_1564623_change-e286060a27b02f64.yaml | 00: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 |
tbarron | I don't see anything wrong with the *current* contents of that file, but it has an interesting history: | 00:35 |
tbarron | https://paste.fedoraproject.org/paste/agV55knvr~~AhRhhjBqmcl5M1UNdIGYhyRLivL9gydE= | 00:37 |
tbarron | it's as if the old bad commit is causing a warning that is making the releasenotes job fail .... | 00:38 |
tbarron | but all the changes around that reno file were a long time ago. | 00:38 |
tbarron | Fresh 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-manila | 01:05 | |
*** markstur has quit IRC | 01:09 | |
*** tuanluong has joined #openstack-manila | 01:31 | |
*** kaisers_ has joined #openstack-manila | 01:51 | |
*** kaisers has quit IRC | 01:55 | |
zhongjun | tbarron: 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 31 | 01:56 |
zhongjun | tbarron: Maybe It could have some changed about openstack reno or some other related project for 3 days ago | 01:59 |
zhongjun | hey, 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 IRC | 02:48 | |
openstackgerrit | zhongjun proposed openstack/manila master: Use uwsgi for devstack https://review.openstack.org/469389 | 02:48 |
openstackgerrit | zhongjun proposed openstack/manila master: Use uwsgi for devstack https://review.openstack.org/469389 | 02:56 |
*** markstur has joined #openstack-manila | 03:07 | |
*** winston-d_ has joined #openstack-manila | 03:12 | |
*** markstur_ has joined #openstack-manila | 03:28 | |
*** markstur has quit IRC | 03:31 | |
openstackgerrit | zhongjun proposed openstack/manila master: TEST https://review.openstack.org/469407 | 03:46 |
*** arnewiebalck_ has joined #openstack-manila | 04:12 | |
*** markstur_ has quit IRC | 04:55 | |
*** markstur_ has joined #openstack-manila | 05:10 | |
*** gouthamr has quit IRC | 05:15 | |
*** arnewiebalck_ has quit IRC | 05:20 | |
*** markstur_ has quit IRC | 05:41 | |
*** markstur has joined #openstack-manila | 06:02 | |
*** markstur has quit IRC | 06:06 | |
*** kaisers_ has quit IRC | 06:13 | |
*** kaisers has joined #openstack-manila | 06:13 | |
*** jprovazn has joined #openstack-manila | 06:56 | |
*** markstur has joined #openstack-manila | 07:01 | |
*** dsariel has joined #openstack-manila | 07:03 | |
*** dmellado_ has joined #openstack-manila | 07:05 | |
*** dmellado_ has quit IRC | 07:05 | |
*** markstur has quit IRC | 07:06 | |
zhongjun | tbarron: Hi , I found the reason about why we need the releasenotes gate error | 07:31 |
zhongjun | s/need/meet | 07:31 |
zhongjun | tbarron: 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.yaml | 07:33 |
zhongjun | tbarron: We have a problem in master because the releasenotes gate run all the version. | 07:34 |
zhongjun | tbarron: We only get error in mitaka, not in other versions. | 07:36 |
*** markstur has joined #openstack-manila | 07:45 | |
zhongjun | tbarron, 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-manila | 07:48 | |
*** markstur has quit IRC | 07:50 | |
*** a-pugachev has joined #openstack-manila | 07:52 | |
*** arnewiebalck_ has joined #openstack-manila | 07:56 | |
*** csaba has quit IRC | 07:59 | |
*** csaba has joined #openstack-manila | 08:00 | |
*** dsariel has quit IRC | 08:03 | |
*** markstur has joined #openstack-manila | 08:05 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/manila-ui master: Imported Translations from Zanata https://review.openstack.org/470190 | 08:07 |
*** markstur has quit IRC | 08:09 | |
zhongjun | tbarron: It just change the logic in reno: link: https://github.com/openstack/reno/commit/e2d60c07931f9095d5e4d9a45535f5ce1a7e0dd3 | 08:12 |
zhongjun | s/change/changed | 08:12 |
*** markstur has joined #openstack-manila | 08:54 | |
*** arnewiebalck_ has quit IRC | 08:58 | |
*** markstur has quit IRC | 08:59 | |
zhongjun | tbarron: 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 |
openstackgerrit | zhongjun proposed openstack/manila master: Remove pbr warnerrors in favor of sphinx check https://review.openstack.org/469407 | 09:12 |
*** markstur has joined #openstack-manila | 09:19 | |
*** a-pugachev has quit IRC | 09:21 | |
*** markstur has quit IRC | 09:24 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila master: Add quotas per share type https://review.openstack.org/452158 | 09:41 |
*** a-pugachev has joined #openstack-manila | 09:42 | |
tbarron | zhongjun: interesting | 09:44 |
tbarron | zhongjun: I'm not sure the relevant variable is the version of reno though | 09:44 |
tbarron | zhongjun: both reno and sphinx versions seem to be getting set by global-requirements, so our settings in test-requirements don't really have effect | 09:45 |
tbarron | zhongjun: as an experiment, I changed tox.ini to no longer use global requirements and played with the reno and sphinx versions | 09:46 |
tbarron | if I back off reno to 2.3.0 I still get that error | 09:46 |
tbarron | if I back off reno to 2.2.0 I get a different error (warning as error) but it still fails | 09:47 |
tbarron | if I leave reno at 2.3.1 and back off sphinx to 1.5.6 then the releasenotes build OK | 09:48 |
tbarron | maybe sphinx 'treat warnings as error' argument is stricter now?? | 09:49 |
zhongjun | tbarron: maybe, No one knows what they've done :) | 09:51 |
zhongjun | tbarron: It is better to fix the releasenote bug in mitaka version | 09:55 |
tbarron | zhongjun: agree, | 09:55 |
tbarron | zhongjun: if we can, isn't mitaka EOL? | 09:55 |
tbarron | zhongjun: even if it is maybe bswartz can work some magic | 09:56 |
zhongjun | tbarron: It looks like it is not EOL, I put a pathch in mitaka(https://review.openstack.org/470182) | 09:57 |
tbarron | zhongjun: cool | 09:57 |
zhongjun | tbarron: sure, bswartz can work some magic | 09:58 |
zhongjun | tbarron: haha | 09:58 |
*** jprovazn has quit IRC | 10:00 | |
openstackgerrit | Merged openstack/manila-ui master: Imported Translations from Zanata https://review.openstack.org/470190 | 10:11 |
*** tuanluong has quit IRC | 10:35 | |
*** ganso has joined #openstack-manila | 10:41 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila-ui master: Add support of share groups https://review.openstack.org/468899 | 10:43 |
*** markstur has joined #openstack-manila | 10:56 | |
*** markstur has quit IRC | 11:00 | |
openstackgerrit | Ramana Raja proposed openstack/manila master: WIP ganesha: dynamically update access of share https://review.openstack.org/424598 | 11:13 |
openstackgerrit | Ramana Raja proposed openstack/manila master: WIP cephfs/driver: add nfs protocol support https://review.openstack.org/424605 | 11:16 |
*** jprovazn has joined #openstack-manila | 11:30 | |
*** gcb has quit IRC | 11:34 | |
*** gcb has joined #openstack-manila | 11:38 | |
*** gcb has quit IRC | 11:43 | |
*** gcb has joined #openstack-manila | 11:52 | |
*** catintheroof has joined #openstack-manila | 12:08 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila-ui master: Add support of share groups https://review.openstack.org/468899 | 13:00 |
*** gcb has quit IRC | 13:04 | |
*** kaisers has quit IRC | 13:13 | |
bswartz | vponomaryov: https://review.openstack.org/#/c/446044/11/manila_tempest_tests/tests/api/base.py | 13:25 |
tbarron | bswartz: the theory is that you need to merge https://review.openstack.org/#/c/470182/ so that releasenotes job will work in master | 13:29 |
tbarron | bswartz: reno cares about past history and old releases | 13:30 |
tbarron | bswartz: and it has gotten stricter | 13:30 |
bswartz | and stable/mitaka is still open? | 13:30 |
bswartz | long live EOL branches I guess | 13:30 |
tbarron | bswartz: I think mitaka is EOL in theory but appears to be still open | 13:30 |
bswartz | schrodinger's branch | 13:31 |
tbarron | bswartz: I think this will take a ninja merge b/c of missing dependencies that cause the lvm devstack build to fail | 13:31 |
bswartz | we won't know until we open the box | 13:31 |
tbarron | bswartz: but that has nothing to do with the reno yaml syntax fix | 13:31 |
tbarron | bswartz: meoooow | 13:32 |
*** markstur has joined #openstack-manila | 13:42 | |
*** cknight has joined #openstack-manila | 13:46 | |
*** markstur has quit IRC | 13:46 | |
bswartz | tbarron: so how can we fix the LVM job and get this patch to merge? | 13:49 |
bswartz | is there another patch that solve the mitaka gate problem? | 13:50 |
tbarron | bswartz: not that I'm aware of. Can you just force it to merge and EOL mitaka? | 13:50 |
bswartz | there's no such thing as a "force to merge" | 13:50 |
tbarron | mitaka is EOL b/c no one is keeping dependencies for it correct anymore. | 13:50 |
bswartz | it has to pass the gate | 13:50 |
bswartz | all we can do is disable the gate job for mitaka | 13:51 |
bswartz | which is an infra change | 13:51 |
*** MVenesio has joined #openstack-manila | 13:51 | |
*** gouthamr has joined #openstack-manila | 13:51 | |
bswartz | is there any hope of solving the dependency problem and getting the job to pass? | 13:51 |
bswartz | if not, then we should submit a change to infra to turn off that stable job | 13:51 |
tbarron | bswartz: I haven't looked at it; can do so at 11 but have to go to a meeting now. | 13:52 |
tbarron | bswartz: 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 |
bswartz | tbarron: http://logs.openstack.org/82/470182/2/check/gate-manila-tempest-minimal-dsvm-lvm-ubuntu-trusty/d8c8161/logs/devstack-early.txt.gz | 13:54 |
*** rraja has joined #openstack-manila | 13:59 | |
*** eharney has joined #openstack-manila | 14:05 | |
*** markstur has joined #openstack-manila | 14:06 | |
*** dsariel has joined #openstack-manila | 14:08 | |
*** markstur has quit IRC | 14:11 | |
*** jmlowe has quit IRC | 14:29 | |
*** markstur has joined #openstack-manila | 14:39 | |
*** catintheroof has quit IRC | 14:45 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila master: [Share groups] Add scheduler filter ConsistentSnapshotFilter https://review.openstack.org/446044 | 14:49 |
*** jmlowe has joined #openstack-manila | 14:51 | |
*** jprovazn has quit IRC | 15:00 | |
tbarron | bswartz: back | 15:18 |
tbarron | bswartz: 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.gz | 15:18 |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila-ui master: Add support of share groups https://review.openstack.org/468899 | 15:19 |
bswartz | tbarron yeah... | 15:21 |
*** vponomaryov has quit IRC | 15:23 | |
*** markstur has quit IRC | 15:24 | |
*** dims has quit IRC | 15:28 | |
*** markstur has joined #openstack-manila | 15:29 | |
*** lpetrut has quit IRC | 16:03 | |
*** arnewiebalck_ has joined #openstack-manila | 16:04 | |
*** gaurangt_ has quit IRC | 16:08 | |
*** gaurangt has joined #openstack-manila | 16:11 | |
*** dsariel has quit IRC | 16:14 | |
*** pcaruana has quit IRC | 16:16 | |
tbarron | bswartz: put this one in the pipeline in case ... https://review.openstack.org/#/c/470358/ | 16:18 |
*** a-pugachev has quit IRC | 16:22 | |
*** rraja has quit IRC | 16:38 | |
*** arnewiebalck_ has quit IRC | 16:41 | |
bswartz | tbarron: precisely what I had in mind | 16:43 |
bswartz | tbarron: got a minute? | 16:43 |
tbarron | sure | 16:48 |
*** eharney has quit IRC | 16:48 | |
*** eharney has joined #openstack-manila | 17:03 | |
*** dims has joined #openstack-manila | 17:10 | |
openstackgerrit | Victoria Martinez de la Cruz proposed openstack/manila master: Add share notifications https://review.openstack.org/466586 | 17:15 |
*** xyang1 has joined #openstack-manila | 17:17 | |
*** dims has quit IRC | 17:24 | |
*** dims has joined #openstack-manila | 17:26 | |
*** harlowja has quit IRC | 17:28 | |
vkmc | gouthamr, 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 case | 17:48 |
gouthamr | vkmc: hey! what kind of notifications did you have in mind?' | 17:48 |
vkmc | gouthamr, create share notifications precisely | 17:49 |
gouthamr | vkmc: the provisioning notifications would be similar | 17:49 |
gouthamr | vkmc: to all other resources: shares, networks, types, groups. | 17:50 |
vkmc | gouthamr, I was thinking... when a new replica is created, shall we notify about those new shares? | 17:50 |
vkmc | yeah | 17:50 |
gouthamr | vkmc: yep.. 2 mins.. brb | 17:52 |
tbarron | gouthamr: vkmc one thing I wanted to sanity check was what will happen with replicas and "share notifications" | 17:52 |
tbarron | the "share notifications" are really triggered on share-instance creates and deletes ... | 17:52 |
vkmc | tbarron, 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.py | 17:53 |
tbarron | share-instances are hidden from regular cloud users but visible to admins | 17:53 |
vkmc | so the "share.create" notification won't be triggered when we create replicas | 17:53 |
vkmc | yeah, I stick to that premise | 17:54 |
tbarron | vkmc: k, probably fine then, but getting goutham to nod his head when he gets back would be good too | 17:54 |
vkmc | of course | 17:54 |
gouthamr | vkmc: yep.. https://github.com/openstack/manila/blob/master/manila/scheduler/manager.py#L266 was intentionally different | 17:56 |
vkmc | yeah | 17:56 |
vkmc | gouthamr++ | 17:57 |
gouthamr | i like this :) maybe we can start sending our health monitoring notifications next | 17:57 |
vkmc | yeah :D | 17:58 |
openstackgerrit | Merged openstack/manila master: Remove pbr warnerrors in favor of sphinx check https://review.openstack.org/469407 | 18:06 |
*** dims has quit IRC | 18:16 | |
*** dims has joined #openstack-manila | 18:17 | |
*** markstur has quit IRC | 18:30 | |
*** markstur has joined #openstack-manila | 18:30 | |
*** markstur has quit IRC | 18:35 | |
*** catintheroof has joined #openstack-manila | 18:38 | |
*** catintheroof has quit IRC | 19:02 | |
*** a-pugachev has joined #openstack-manila | 19:03 | |
*** eharney has quit IRC | 19:07 | |
*** dsariel has joined #openstack-manila | 19:12 | |
*** eharney has joined #openstack-manila | 19:20 | |
*** a-pugachev has quit IRC | 19:24 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/manila master: Updated from global requirements https://review.openstack.org/469570 | 19:25 |
*** hongbin has joined #openstack-manila | 19:43 | |
*** dsariel has quit IRC | 19:44 | |
*** eharney has quit IRC | 19:56 | |
*** harlowja has joined #openstack-manila | 20:04 | |
*** eharney has joined #openstack-manila | 20:11 | |
*** markstur has joined #openstack-manila | 20:31 | |
*** gouthamr has quit IRC | 20:41 | |
*** cknight has quit IRC | 20:49 | |
*** winston-d_ has quit IRC | 20:55 | |
*** jmlowe has quit IRC | 20:57 | |
*** gouthamr has joined #openstack-manila | 20:59 | |
*** kaisers1 has quit IRC | 21:07 | |
*** eharney has quit IRC | 21:15 | |
*** cknight has joined #openstack-manila | 21:31 | |
*** MVenesio has quit IRC | 21:39 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/manila master: Updated from global requirements https://review.openstack.org/469570 | 21:57 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/manila-ui master: Updated from global requirements https://review.openstack.org/470461 | 21:57 |
*** kaisers has joined #openstack-manila | 22:04 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-manilaclient master: Updated from global requirements https://review.openstack.org/470141 | 22:06 |
*** ganso has quit IRC | 22:20 | |
*** andreaf has quit IRC | 22:35 | |
*** andreaf has joined #openstack-manila | 22:40 | |
*** gouthamr has quit IRC | 22:40 | |
*** gouthamr has joined #openstack-manila | 22:42 | |
*** cFouts has quit IRC | 22:42 | |
*** gnarld_ has joined #openstack-manila | 22:45 | |
*** cknight has quit IRC | 22:49 | |
*** ianychoi has quit IRC | 22:59 | |
*** ianychoi has joined #openstack-manila | 23:01 | |
*** xyang1 has quit IRC | 23:14 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!