| *** breitz has quit IRC | 00:01 | |
| *** mtanino has quit IRC | 00:12 | |
| *** gouthamr has quit IRC | 00:12 | |
| *** gouthamr has joined #openstack-manila | 00:13 | |
| *** gouthamr has quit IRC | 00:26 | |
| *** akerr has joined #openstack-manila | 00:33 | |
| *** akerr_ has joined #openstack-manila | 00:38 | |
| *** cknight has quit IRC | 00:39 | |
| *** akerr has quit IRC | 00:42 | |
| *** esker has joined #openstack-manila | 00:43 | |
| *** EinstCrazy has joined #openstack-manila | 00:56 | |
| openstackgerrit | Merged openstack/manila: New Manila HDS HNAS Driver https://review.openstack.org/204692 | 01:12 |
|---|---|---|
| openstackgerrit | Merged openstack/manila-ui: Fix set/unset extra specs for share type update form https://review.openstack.org/213106 | 01:16 |
| *** akerr_ has quit IRC | 01:23 | |
| *** jasonsb has quit IRC | 01:25 | |
| *** Guest62749 has joined #openstack-manila | 01:34 | |
| *** Guest62749 has quit IRC | 01:36 | |
| *** akerr has joined #openstack-manila | 01:45 | |
| *** cknight has joined #openstack-manila | 01:45 | |
| *** cknight1 has joined #openstack-manila | 01:47 | |
| *** cknight has quit IRC | 01:49 | |
| *** haomaiwang has joined #openstack-manila | 02:23 | |
| *** jasonsb has joined #openstack-manila | 02:23 | |
| *** haomaiwang has quit IRC | 02:24 | |
| *** haomaiwang has joined #openstack-manila | 02:25 | |
| openstackgerrit | Merged openstack/manila: Updated from global requirements https://review.openstack.org/216967 | 02:42 |
| *** haomaiwang has quit IRC | 02:44 | |
| *** haomaiwang has joined #openstack-manila | 02:58 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-manila | 03:02 | |
| *** bill_az has quit IRC | 03:07 | |
| *** haomaiwang has quit IRC | 03:18 | |
| *** haomaiwang has joined #openstack-manila | 03:27 | |
| *** tbarron has quit IRC | 03:28 | |
| *** tbarron has joined #openstack-manila | 03:32 | |
| *** haomaiwang has quit IRC | 03:36 | |
| *** akerr is now known as akerr_away | 03:42 | |
| *** akerr_away is now known as akerr | 03:48 | |
| *** akerr has quit IRC | 03:49 | |
| *** martyturner has joined #openstack-manila | 04:18 | |
| *** baojg has joined #openstack-manila | 04:19 | |
| *** martyturner has quit IRC | 04:22 | |
| *** haomaiwang has joined #openstack-manila | 04:26 | |
| *** BharatK has joined #openstack-manila | 04:42 | |
| *** haomaiwang has quit IRC | 04:50 | |
| *** deepakcs has joined #openstack-manila | 04:56 | |
| *** cknight1 has quit IRC | 04:59 | |
| *** vbellur has quit IRC | 05:14 | |
| openstackgerrit | Merged openstack/manila: Fix Tempest tests targeting user based access rules https://review.openstack.org/217623 | 05:26 |
| *** haomaiwang has joined #openstack-manila | 05:33 | |
| *** haomaiwang has quit IRC | 05:35 | |
| *** BharatK has quit IRC | 05:51 | |
| *** anubhav has joined #openstack-manila | 05:59 | |
| *** haomaiwang has joined #openstack-manila | 06:02 | |
| *** vbellur has joined #openstack-manila | 06:06 | |
| *** BharatK has joined #openstack-manila | 06:06 | |
| *** haomaiwang has quit IRC | 06:23 | |
| openstackgerrit | Merged openstack/manila: glusterfs_native: Add create share from snapshot https://review.openstack.org/215033 | 06:29 |
| *** baojg has quit IRC | 06:33 | |
| *** nkrinner has joined #openstack-manila | 06:48 | |
| mkoderer | vponomaryov: thx for fixing the rally plugin | 06:48 |
| mkoderer | vponomaryov: do you think we can get rid of all noqa flags? | 06:49 |
| openstackgerrit | Thomas Bechtold proposed openstack/manila: Ignore unavailable volumes when deleting a share https://review.openstack.org/217863 | 06:50 |
| openstackgerrit | Marc Koderer proposed openstack/manila: Use Tempest plugin interface https://review.openstack.org/201955 | 07:01 |
| openstackgerrit | Marc Koderer proposed openstack/manila: Use Tempest plugin interface https://review.openstack.org/201955 | 07:03 |
| openstackgerrit | Marc Koderer proposed openstack/manila: Ignore git backup merge files https://review.openstack.org/218123 | 07:06 |
| *** EinstCrazy has quit IRC | 07:13 | |
| *** haomaiwang has joined #openstack-manila | 07:38 | |
| u_glide | toabctl: ping | 07:39 |
| csaba | u_glide: ping | 07:40 |
| u_glide | csaba: pong | 07:40 |
| toabctl | u_glide: pong | 07:40 |
| u_glide | toabctl: Could you please review this patch https://review.openstack.org/#/c/209487/ ? | 07:41 |
| csaba | u_glide: I tried to update the master branch to get the fix you referred to, but it involved a db upgrade and that broke: http://paste.openstack.org/show/429947/ | 07:42 |
| toabctl | u_glide: have you had a look why some of the non-voting jobs are failing? | 07:43 |
| *** sgotliv has joined #openstack-manila | 07:44 | |
| *** haomaiwang has quit IRC | 07:44 | |
| *** haomaiwa_ has joined #openstack-manila | 07:45 | |
| u_glide | toabctl: scenario and glusterfs jobs not so stable yet | 07:45 |
| u_glide | toabctl: sometimes they pass, sometimes fail | 07:45 |
| toabctl | u_glide: ok. so unrelated to your changeset I guess | 07:46 |
| u_glide | csaba: could you please fill bug on launchpad? | 07:47 |
| *** haomaiw__ has joined #openstack-manila | 07:49 | |
| *** haomaiwa_ has quit IRC | 07:50 | |
| csaba | u_glide: sure | 07:56 |
| *** haomaiw__ has quit IRC | 07:58 | |
| openstackgerrit | Yusuke Hayashi proposed openstack/manila: Replace assertEqual(None, *) with assertIsNone in tests https://review.openstack.org/213443 | 08:01 |
| *** sgotliv has quit IRC | 08:11 | |
| toabctl | u_glide: do we need an API for share_instances? isn't it enough to be able to get the share instances per share? so /share/%shareid/instances and something like /share/%sharedid/instance/%instanceid/ ? | 08:19 |
| toabctl | u_glide: what's the usecase to list all share instances independent from the shares ? | 08:20 |
| u_glide | toabctl: for example admin could retrieve all share instances with error state or get all instances in specific AZ, etc | 08:22 |
| chen12 | vponomaryov, ping | 08:24 |
| toabctl | u_glide: but if a instance from a share is in error state, the share itself is in eror state, too. isn't it ? | 08:25 |
| toabctl | u_glide: and for the AZ, aren't the share_instances of a share all in the same AZ as the share? | 08:26 |
| chen12 | vponomaryov, could you help me review https://review.openstack.org/#/c/218092/, this enables multi-backends test for HDFS CI. Thanks! | 08:26 |
| toabctl | u_glide: I thought share_instances are somewhow just an implementaion detail...but maybe I'm wrong. | 08:27 |
| u_glide | u_glide: with replication share will have instances in different AZ's | 08:27 |
| u_glide | toabctl: ^ | 08:28 |
| toabctl | yeah :) | 08:28 |
| toabctl | u_glide: hm.ok. but I still think you don't need to get share_instances independent from shares... | 08:29 |
| toabctl | u_glide: should I add this question to the review comment and see what others think about it? | 08:29 |
| u_glide | toabctl: sure | 08:30 |
| toabctl | u_glide: I'm unsure. you are the one who's implementing it. I'm not to deep into that :) | 08:31 |
| u_glide | toabctl: from my perspective it's useful for driver developers and functional tests | 08:31 |
| u_glide | because drivers now interact only with share instances | 08:32 |
| u_glide | so admin or developer could filter share instances on specific host | 08:33 |
| *** lpetrut has joined #openstack-manila | 08:34 | |
| u_glide | currently value of this api for admins small, but with replication it will increase | 08:34 |
| *** lpetrut has left #openstack-manila | 08:39 | |
| vponomaryov | chen12: looks ok | 08:47 |
| chen12 | vponomaryov, thanks! | 08:47 |
| *** amit213 has quit IRC | 08:50 | |
| *** amit213 has joined #openstack-manila | 08:51 | |
| *** baojg has joined #openstack-manila | 08:53 | |
| openstackgerrit | Merged openstack/manila: Ignore unavailable volumes when deleting a share https://review.openstack.org/217863 | 08:58 |
| vponomaryov | mkoderer: "noqa" was set to avoid false pep errors related to grouping imports by third-partyness | 09:21 |
| vponomaryov | mkoderer: so, having "plugin"-approach we should not have this problem anymore | 09:22 |
| vponomaryov | mkoderer: if I am not mistaken | 09:22 |
| vponomaryov | mkoderer: also, have you seen that bug we want to reproduce from time to time appears in our voting Tempest jobs? | 09:22 |
| vponomaryov | mkoderer: for example, https://review.openstack.org/#/c/217624/ can not be merged, because it faces this bug - http://logs.openstack.org/24/217624/3/gate/gate-manila-tempest-dsvm-neutron-multibackend/c8bd843/logs/screen-m-shr.txt.gz?level=TRACE | 09:23 |
| *** baojg has quit IRC | 09:26 | |
| *** haomaiwang has joined #openstack-manila | 09:34 | |
| openstackgerrit | Merged openstack/manila-ui: Remove overrides reference from Manila-UI plug-in https://review.openstack.org/215781 | 09:35 |
| openstackgerrit | Yusuke Hayashi proposed openstack/manila: Replace assertEqual(None, *) with assertIsNone in tests https://review.openstack.org/213443 | 09:37 |
| openstackgerrit | Silvan Kaiser proposed openstack/manila: Corrects capabilities returned by Quobyte Manila driver https://review.openstack.org/215556 | 09:38 |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwa_ has joined #openstack-manila | 10:02 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila-ui: Disable button 'create snapshot' if share not snapshottable https://review.openstack.org/213242 | 10:17 |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila-ui: Disable button 'create snapshot' if share not snapshottable https://review.openstack.org/213242 | 10:19 |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila-ui: Disable button 'create snapshot' if share not snapshottable https://review.openstack.org/213242 | 10:20 |
| *** lpetrut1 has joined #openstack-manila | 10:25 | |
| *** haomaiwa_ has quit IRC | 10:27 | |
| *** jasonsb has quit IRC | 10:27 | |
| *** tbarron has quit IRC | 10:27 | |
| *** jwang has quit IRC | 10:27 | |
| *** lpetrut1 is now known as lpetrut | 10:27 | |
| *** tbarron has joined #openstack-manila | 10:29 | |
| *** jwang has joined #openstack-manila | 10:29 | |
| *** lpetrut is now known as 18VAAKDGR | 10:29 | |
| *** haomaiwa_ has joined #openstack-manila | 10:30 | |
| *** jasonsb has joined #openstack-manila | 10:30 | |
| *** haomaiwa_ has quit IRC | 10:30 | |
| *** jasonsb has quit IRC | 10:30 | |
| *** jasonsb has joined #openstack-manila | 10:32 | |
| csaba | u_glide: https://bugs.launchpad.net/manila/+bug/1489827 | 10:36 |
| openstack | Launchpad bug 1489827 in Manila "db upgrade fails in "add_availability_zones_table"" [Undecided,New] | 10:36 |
| u_glide | csaba: thanks! | 10:36 |
| openstackgerrit | Merged openstack/manila: Tempest: wait for access rule to be deleted https://review.openstack.org/217624 | 11:21 |
| csaba | vponomaryov: ping | 11:31 |
| *** porrua has joined #openstack-manila | 11:39 | |
| *** esker has quit IRC | 11:40 | |
| *** rraja has quit IRC | 11:43 | |
| *** porrua has quit IRC | 11:46 | |
| *** tpsilva has joined #openstack-manila | 11:59 | |
| openstackgerrit | Alin Balutoiu proposed openstack/manila: Tempest: wrong assertion on the number of shares created https://review.openstack.org/218229 | 12:01 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Share Migration support in generic driver https://review.openstack.org/179791 | 12:02 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Add Share Migration feature https://review.openstack.org/179790 | 12:02 |
| openstackgerrit | Tiago Pasqualini da Silva proposed openstack/manila: Fix HDS HNAS snapshot creation always succeeding https://review.openstack.org/215271 | 12:02 |
| vponomaryov | csaba: pong | 12:07 |
| *** deepakcs has quit IRC | 12:14 | |
| openstackgerrit | Igor Malinovskiy proposed openstack/manila: Fix data copying issue in DB migration 1f0bd302c1a6 https://review.openstack.org/218239 | 12:18 |
| *** martyturner has joined #openstack-manila | 12:20 | |
| u_glide | csaba: fix for DB migration issue https://review.openstack.org/#/c/218239/ | 12:22 |
| csaba | vponomaryov: I had a weird issue on devstack with generic driver -- that is, the share server did not got the keys injected. it's available on the network, but can't login with ssh. Do you know anything of this issue? | 12:22 |
| vponomaryov | csaba: we suffer from this issue for a while, see https://bugs.launchpad.net/manila/+bug/1480870 | 12:23 |
| openstack | Launchpad bug 1480870 in Manila "tempest scenario job is unstable" [High,In progress] - Assigned to Valeriy Ponomaryov (vponomaryov) | 12:23 |
| csaba | u_glide: thanks. So I should just check this out and re-run db sync? | 12:23 |
| vponomaryov | csaba: so, would be great to know "steps to reproduce" | 12:23 |
| csaba | vponomaryov: does it happen non-deterministically? | 12:25 |
| vponomaryov | csaba: it is unstable | 12:25 |
| csaba | vponomaryov: is there a workaround? | 12:25 |
| vponomaryov | csaba: we do not know the reason | 12:26 |
| vponomaryov | csaba: no workaround | 12:26 |
| csaba | vponomaryov: so if this occurs I should just ditch the devstack setup and try from scratch hoping for more luck? | 12:26 |
| vponomaryov | csaba: why devstack? just recreate share-server | 12:27 |
| vponomaryov | csaba: no need to reinstall env | 12:28 |
| *** akerr has joined #openstack-manila | 12:28 | |
| *** vbellur has quit IRC | 12:29 | |
| csaba | vponomaryov: OK sounds better | 12:31 |
| *** chlong has joined #openstack-manila | 12:34 | |
| ganso_ | u_glide: ping | 12:40 |
| u_glide | ganso_: pong | 12:41 |
| u_glide | csaba: yes, you can cherry-pick this fix | 12:42 |
| ganso_ | u_glide: hi, I am thinking about adding a try-catch block inside loop in line 751 here: https://review.openstack.org/#/c/179790/26/manila/share/api.py | 12:42 |
| *** BharatK has quit IRC | 12:42 | |
| ganso_ | u_glide: if I am trying to deny a certain access rule, I have 2 instance, only one has the access rule and the other does not, it fails and does not remove the access rule from the one that has, because it fails to get the access_mapping for the one that does not have | 12:43 |
| ganso_ | u_glide: this is random because it depends on which instance the access_mapping is obtained first | 12:43 |
| *** gouthamr has joined #openstack-manila | 12:44 | |
| ganso_ | u_glide: I am thinking about saving the exception and throwing it as the loop ends | 12:45 |
| *** haomaiwa_ has joined #openstack-manila | 12:45 | |
| ganso_ | u_glide: I believe that NotFound or InvalidAccessRule errors/exceptions should be thrown, although I debate if it is really an error when I try to delete an access rule and it is already not there... | 12:46 |
| u_glide | ganso_: I think you should catch exception and log it | 12:47 |
| ganso_ | u_glide: There are other ways I can fix this in share migration without changes to the API, but I would like to know your opinion on this | 12:47 |
| ganso_ | u_glide: ok so you agree NotFound/InvalidAccessRule exceptions on access_deny should not be thrown? | 12:48 |
| openstackgerrit | Marc Koderer proposed openstack/manila: Ignore git backup merge files https://review.openstack.org/218123 | 12:48 |
| u_glide | ganso_: I'm agree that NotFound exceptions on access_deny should not be thrown | 12:49 |
| ganso_ | u_glide: ok, I will include it in my patch | 12:50 |
| ganso_ | u_glide: what about the loop in access_allow? if ShareAccessExists error happens for one instance, but not the other, do you think should it also be logged and not thrown? | 12:50 |
| u_glide | ganso_: yes, it should be just logged | 12:51 |
| ganso_ | u_glide: thanks! | 12:51 |
| u_glide | ganso_: because it shouldn't affect anything, but we should notify admin | 12:52 |
| *** timcl has joined #openstack-manila | 12:53 | |
| *** nkrinner has quit IRC | 12:58 | |
| *** dustins has joined #openstack-manila | 13:00 | |
| *** haomaiwa_ has quit IRC | 13:01 | |
| *** haomaiwang has joined #openstack-manila | 13:02 | |
| openstackgerrit | Igor Malinovskiy proposed openstack/manila: Add Share instances Admin API https://review.openstack.org/209487 | 13:02 |
| *** dzamboni has joined #openstack-manila | 13:02 | |
| *** nkrinner has joined #openstack-manila | 13:03 | |
| *** pgbridge has joined #openstack-manila | 13:14 | |
| ganso_ | u_glide, vponomaryov: Could you please take a look at https://review.openstack.org/#/c/215271/ ? It is a small patch needed to close the blueprint | 13:14 |
| *** anubhav has quit IRC | 13:16 | |
| openstackgerrit | Igor Malinovskiy proposed openstack/manila: Use Share Instance ID in 'name' property https://review.openstack.org/218263 | 13:20 |
| *** cknight has joined #openstack-manila | 13:25 | |
| ganso_ | vponomaryov: ping | 13:27 |
| *** breitz has joined #openstack-manila | 13:28 | |
| *** bill_az has joined #openstack-manila | 13:34 | |
| vponomaryov | ganso_: pong | 13:36 |
| ganso_ | vponomaryov: hi. Patch https://review.openstack.org/#/c/215271/ complements the driver because the driver does not track whether snapshot creation succeeds or not. I know the title says "Fix...", should it really be considered a bug? At first I thought it would be a feature because "tracking snapshot creation" is a functionality that should have been | 13:39 |
| ganso_ | included in the driver and was not | 13:39 |
| vponomaryov | ganso_: then it is not "fix", it is implementation | 13:39 |
| vponomaryov | ganso_: so, either wording should be changed or reference to bug added | 13:39 |
| ganso_ | vponomaryov: ok, we will change the commit message and description | 13:39 |
| ganso_ | vponomaryov: thanks | 13:40 |
| openstackgerrit | Tiago Pasqualini da Silva proposed openstack/manila: Implement snapshot tracking in HDS HNAS driver https://review.openstack.org/215271 | 13:46 |
| ganso_ | vponomaryov: ^ | 13:47 |
| *** eharney has joined #openstack-manila | 13:53 | |
| tpsilva | cknight: Could you please take a look at https://review.openstack.org/#/c/215271/? I fixed your comments... | 13:54 |
| *** haomaiwang has quit IRC | 14:01 | |
| *** haomaiwang has joined #openstack-manila | 14:02 | |
| *** kaisers has quit IRC | 14:02 | |
| cknight | tpsilva: Yes. | 14:23 |
| *** vbellur has joined #openstack-manila | 14:27 | |
| *** timcl has quit IRC | 14:33 | |
| *** mtanino has joined #openstack-manila | 14:44 | |
| *** timcl has joined #openstack-manila | 14:46 | |
| *** haomaiwang has quit IRC | 15:01 | |
| openstackgerrit | Chuck Fouts proposed openstack/python-manilaclient: Manila REST API Microversion Support for Client https://review.openstack.org/218332 | 15:02 |
| *** haomaiwang has joined #openstack-manila | 15:02 | |
| *** mtanino has quit IRC | 15:03 | |
| *** lpabon has joined #openstack-manila | 15:03 | |
| *** ayma has joined #openstack-manila | 15:07 | |
| ayma | hi, I was going over the presentation here: https://www.youtube.com/watch?v=1zUSFzbhThM (vBrownBag 2014 OpenStack Paris - Bill Owen, Csaba Henk – Supporting NFS Ganesha in Manila) and am a bit confused about how things should be configured. | 15:09 |
| cFouts | vponomaryov: https://review.openstack.org/218332 Contains the changes for bug #1489450 that you opened. Would really appreciate your review | 15:10 |
| openstack | bug 1489450 in python-manilaclient "manilaclient is unable to provide microversion header" [High,In progress] https://launchpad.net/bugs/1489450 - Assigned to Chuck Fouts (fchuck) | 15:10 |
| vponomaryov | cFouts: actually I already did it | 15:11 |
| vponomaryov | cFouts: =) | 15:11 |
| ayma | i see this: http://docs.openstack.org/developer/manila/devref/ganesha.html which seems to be a lib for manila to use to support ganesha...looking at the http://docs.openstack.org/developer/manila/devref/glusterfs_driver.html I see that there is a spot for glusterfs_nfs_server_type where I can specify ganesha. However I'm a bit confused abut how this service VM that has the nfs-ganesha server and gluster FSAL contained in it | 15:15 |
| ayma | also a bit confused about where install NFS-Ganesha 2.1 or higher...Should it belong in: a) my glusterFS VM since the rest of the config options in the manila glusterfsShareDriver points to tags like glusterfs_target and then I just add the ganesha_config_dir to the gluster section | 15:21 |
| vponomaryov | ayma: csaba and rraja are best people to help you | 15:21 |
| ayma | b) my devstack or c) a service VM that I manually spawn? | 15:21 |
| ayma | vponomaryov: thanks for the reference, i guess i'll need to wait for them to respond...or how best to contact them? | 15:24 |
| vponomaryov | ayma: here - in IRC, but also it would be useful for lots of other people if mail-list is used | 15:27 |
| ayma | oh how to use the mail-list...is there a link on how to use that for manila? | 15:32 |
| vponomaryov | ayma: https://wiki.openstack.org/wiki/Mailing_Lists | 15:33 |
| ayma | vponomaryov:thx | 15:34 |
| *** timcl has quit IRC | 15:36 | |
| *** chlong has quit IRC | 15:39 | |
| cFouts | vponomaryov: regarding being able to set the api version in the client. Clinton was wanting the same thing but there is an issue that currently the client is always written to work with the latest manila api | 15:41 |
| cFouts | vponomaryov: if we allow the client user to specify the api version to send in the header then the Manger classes in the client would also need to support creating requests and handling responses for versions | 15:42 |
| cFouts | vponomaryov: what do you think? | 15:42 |
| vponomaryov | cFouts: it is definitely use case for users - they should be able to use any new client being able to work with old server that does not support cleint's fixed version | 15:44 |
| vponomaryov | cFouts: we should answer question "how will we do it?", not "will we do it in general?". | 15:45 |
| *** lpabon has quit IRC | 15:46 | |
| *** mtanino has joined #openstack-manila | 15:46 | |
| cFouts | vponomaryov: yes, agree | 15:46 |
| vponomaryov | cFouts: so, yes, look slike client needs versioning too. And, for the moment, there are too few changes to worry | 15:46 |
| *** markstur_ has quit IRC | 15:47 | |
| cFouts | vponomaryov: I can go ahead and provide the option to override the version sent in the header. I'll also bring this up as a topic in the next manila meeting | 15:47 |
| vponomaryov | cFouts: also, client should restrict setting/usage of version that is newer that it knows | 15:47 |
| cknight | vponomaryov: Can we merge the client with the hard-coded version now to unblock everyone, and follow up ASAP with a configurable version? | 15:48 |
| vponomaryov | cFouts: because there could be different behaviour | 15:48 |
| cFouts | vponomaryov: manila api will throw an error if the version is newer than known | 15:48 |
| cFouts | vponomaryov: oh, I see what you are saying. need to adhere to max supported version | 15:49 |
| vponomaryov | cFouts: server could be newer than client, and newer server version can differ than client old client expects | 15:50 |
| vponomaryov | cFouts: that is why client should be ceiling version | 15:50 |
| vponomaryov | should have* | 15:51 |
| *** lpabon has joined #openstack-manila | 15:52 | |
| vponomaryov | cFouts: why do you need IRC discussion? | 15:54 |
| vponomaryov | cFouts: I guess all people will be for having "range of supported versions by client" | 15:54 |
| *** markstur has joined #openstack-manila | 15:54 | |
| vponomaryov | cFouts: so, client will know how to work with specific set of versions known to it | 15:55 |
| vponomaryov | cFouts: and possibility to define it either using env var or optional param is common approach | 15:55 |
| *** haomaiwang has quit IRC | 15:58 | |
| *** haomaiwang has joined #openstack-manila | 15:58 | |
| cFouts | vponomaryov: ok, so it is up to the user to know that when they are using Kilo Manila to specify --os-share-api-version 1.0 or to be able to set it as an env variable | 15:59 |
| cFouts | vponomaryov: and the user for Liberty wouldn't have to do anything for the client released with Liberty | 15:59 |
| cFouts | vponomaryov: is that the end user experience? | 16:00 |
| *** haomaiwang has quit IRC | 16:01 | |
| u_glide | cFouts: it's better than current experience because we should force users to use latest versions of API | 16:01 |
| vponomaryov | cFouts: possibility to support "range" of APIs by client allows users to use "one" package with different servers - old and new | 16:01 |
| cFouts | u_glide: agree with that. I am fairly new so I'm just asking questions to understand what is needed. not disagreeing with anything :) | 16:02 |
| *** haomaiwang has joined #openstack-manila | 16:02 | |
| u_glide | vponomaryov: +1 good point | 16:02 |
| cFouts | vponomaryov, u_glide: need to step out for a bit. thanks for the feedback so far | 16:03 |
| vponomaryov | cFouts: also we will be able to use new client with bugfixes when new version was released | 16:03 |
| vponomaryov | cFouts: against the same old server | 16:03 |
| *** jasonsb has quit IRC | 16:04 | |
| *** jasonsb has joined #openstack-manila | 16:05 | |
| u_glide | vponomaryov, cknight, markstur, toabctl: ping https://review.openstack.org/#/c/209487/ still needs review | 16:06 |
| *** jasonsb has quit IRC | 16:10 | |
| ganso_ | vponomaryov: hi, how do I start jenkins again to merge https://review.openstack.org/#/c/215271/ ? | 16:15 |
| vponomaryov | ganso_: comment "recheck" | 16:15 |
| ganso_ | vponomaryov: thanks, I was not sure if it would trigger merging | 16:16 |
| vponomaryov | ganso_: if there is "workflow+1" it does | 16:16 |
| 18VAAKDGR | Hi guys. We are almost ready with the Windows SMB CI. We run jobs for both cases: when the driver handles the share servers and when it doesn't. Here are some logs: http://64.119.130.115/manila/179790/26/ The next runs will include logs for both scenarios. | 16:19 |
| *** 18VAAKDGR is now known as lpetrut1 | 16:20 | |
| vponomaryov | lpetrut1: everything is skipped there | 16:20 |
| lpetrut1 | whops, well, we're still working on it :) | 16:22 |
| *** chlong has joined #openstack-manila | 16:22 | |
| *** timcl has joined #openstack-manila | 16:26 | |
| ganso_ | cknight: ping | 16:30 |
| *** lpabon has quit IRC | 16:30 | |
| *** lpabon has joined #openstack-manila | 16:35 | |
| *** esker has joined #openstack-manila | 16:37 | |
| *** esker has quit IRC | 16:38 | |
| *** esker has joined #openstack-manila | 16:39 | |
| *** chlong has quit IRC | 16:45 | |
| *** gouthamr has quit IRC | 16:52 | |
| *** dzamboni has quit IRC | 16:52 | |
| *** timcl has quit IRC | 16:59 | |
| *** mtanino has quit IRC | 17:00 | |
| *** haomaiwang has quit IRC | 17:01 | |
| *** haomaiwang has joined #openstack-manila | 17:02 | |
| *** lpabon has quit IRC | 17:12 | |
| *** timcl has joined #openstack-manila | 17:15 | |
| *** lpabon has joined #openstack-manila | 17:18 | |
| *** lpetrut1 has quit IRC | 17:19 | |
| cknight | ganso_: pong | 17:47 |
| ganso_ | cknight: Hi, I am trying to implement experimental microversion in Share Migration, more specifically, I am adding "@wsgi.Controller.api_version("1.2", None, True)" in line 118 https://review.openstack.org/#/c/179790/26/manila/api/contrib/admin_actions.py | 17:48 |
| ganso_ | cknight: I am sending Curl with -H "X-OpenStack-Manila-API-Version: 1.1" -H "X-OpenStack-Manila-API-Experimental: True" and it still runs the method | 17:48 |
| ganso_ | cknight: it should block method execution instead | 17:49 |
| cknight | ganso_: There is already a decorator on that method. Did you add the microversion as the outermost decorator? | 17:49 |
| ganso_ | yes, above "@wsgi.action('os-migrate_share')" | 17:50 |
| cknight | ganso_: Oh, wait. That's an extension API, right? | 17:50 |
| ganso_ | cknight: yes | 17:50 |
| cknight | ganso_: Microversions only apply to core APIs. Extensions have always been, and remain, unversioned. | 17:50 |
| ganso_ | cknight: ok so you think it is a good idea to move share migration to core api with experimental=true or leave it as extension? | 17:51 |
| cknight | ganso_: You will want to implement the migration APIs as versioned, experimental core APIs. | 17:51 |
| ganso_ | cknight: ok, thanks | 17:51 |
| cknight | ganso_: That is Ben's direction for migration, consistency groups, replication, etc. in their first iterations. | 17:51 |
| ganso_ | cknight: I will include that in the next patch | 17:51 |
| ganso_ | cknight: should be done today | 17:51 |
| *** jasonsb has joined #openstack-manila | 17:52 | |
| cknight | ganso_: Great, thanks. And I'm working on the /v2 thing, hopefully also up today. | 17:52 |
| ganso_ | cknight: great! :) | 17:53 |
| *** haomaiwang has quit IRC | 18:01 | |
| *** esker has quit IRC | 18:01 | |
| *** 20WABZC7L has joined #openstack-manila | 18:02 | |
| *** timcl has quit IRC | 18:13 | |
| *** lpabon has quit IRC | 18:15 | |
| *** lpabon has joined #openstack-manila | 18:17 | |
| ganso_ | cknight: extend is as extension also... shouldn't a patch to make it as microversioned core API be in the works until next thursday? or can it be done until L-4? | 18:18 |
| *** lpabon has quit IRC | 18:19 | |
| ganso_ | cknight: same for allow/deny | 18:19 |
| cknight | ganso_: All the extensions that existed prior to addition of microversions are effectively part of core already. The plan is to move all of those into core in Mitaka. | 18:19 |
| ganso_ | cknight: got it | 18:19 |
| ganso_ | cknight: thanks | 18:19 |
| *** lpabon has joined #openstack-manila | 18:20 | |
| *** lpabon has quit IRC | 18:22 | |
| *** lpabon has joined #openstack-manila | 18:23 | |
| *** mtanino has joined #openstack-manila | 18:23 | |
| cknight | ganso_: But extensions have a slightly different URL, so each one that moves into core will be a microversion bump. | 18:29 |
| *** xyang1 has joined #openstack-manila | 18:52 | |
| *** timcl has joined #openstack-manila | 18:56 | |
| *** 20WABZC7L has quit IRC | 19:01 | |
| *** haomaiwang has joined #openstack-manila | 19:02 | |
| *** tpsilva has quit IRC | 19:12 | |
| *** pgbridge has quit IRC | 19:14 | |
| *** lpabon has quit IRC | 19:16 | |
| *** jasonsb has quit IRC | 19:21 | |
| *** lpabon has joined #openstack-manila | 19:22 | |
| *** jasonsb has joined #openstack-manila | 19:22 | |
| *** jasonsb has quit IRC | 19:26 | |
| *** devlaps has joined #openstack-manila | 19:37 | |
| ganso_ | cknight: is there a patch for manilaclient to work with microversioned API? | 19:40 |
| *** rushil has joined #openstack-manila | 19:42 | |
| *** eharney has quit IRC | 19:43 | |
| *** bill_az has quit IRC | 19:51 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Share Migration support in generic driver https://review.openstack.org/179791 | 19:53 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Add Share Migration feature https://review.openstack.org/179790 | 19:53 |
| *** rushil has quit IRC | 19:54 | |
| *** rushil has joined #openstack-manila | 19:54 | |
| *** lpetrut has joined #openstack-manila | 19:56 | |
| cknight | ganso_: Yes: https://review.openstack.org/#/c/218332/ | 19:59 |
| *** lpabon has quit IRC | 19:59 | |
| *** haomaiwang has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-manila | 20:02 | |
| ganso_ | cknight: thanks! | 20:03 |
| openstackgerrit | Merged openstack/manila: Implement snapshot tracking in HDS HNAS driver https://review.openstack.org/215271 | 20:04 |
| cknight | ganso_: The version the client is requesting is currently hard-coded, and you'll probably have to bump that manually when testing migration. | 20:04 |
| *** lpabon has joined #openstack-manila | 20:04 | |
| *** pgbridge has joined #openstack-manila | 20:10 | |
| *** rushil has quit IRC | 20:13 | |
| *** akerr has quit IRC | 20:19 | |
| *** bill_az has joined #openstack-manila | 20:20 | |
| *** jasonsb has joined #openstack-manila | 20:27 | |
| *** lpetrut has quit IRC | 20:36 | |
| *** pgbridge has quit IRC | 20:39 | |
| *** dustins_ has joined #openstack-manila | 20:44 | |
| *** lpabon has quit IRC | 20:44 | |
| *** dustins has quit IRC | 20:45 | |
| *** dustins_ is now known as dustins | 20:45 | |
| *** lpabon has joined #openstack-manila | 20:52 | |
| *** esker has joined #openstack-manila | 20:58 | |
| *** esker has quit IRC | 20:58 | |
| *** timcl has quit IRC | 20:58 | |
| *** esker has joined #openstack-manila | 20:59 | |
| *** haomaiwang has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-manila | 21:01 | |
| *** dustins has quit IRC | 21:10 | |
| *** dustins has joined #openstack-manila | 21:10 | |
| *** dustins has quit IRC | 21:11 | |
| *** eharney has joined #openstack-manila | 21:28 | |
| *** jasonsb has quit IRC | 21:29 | |
| *** martyturner has quit IRC | 21:36 | |
| *** lpabon has quit IRC | 21:41 | |
| *** lpabon has joined #openstack-manila | 21:44 | |
| *** martyturner has joined #openstack-manila | 21:46 | |
| *** martyturner has left #openstack-manila | 21:49 | |
| *** lpabon has quit IRC | 21:54 | |
| *** lpabon has joined #openstack-manila | 21:59 | |
| *** xyang1 has quit IRC | 22:00 | |
| *** alyson_ has quit IRC | 22:00 | |
| *** haomaiwang has quit IRC | 22:01 | |
| *** haomaiwang has joined #openstack-manila | 22:02 | |
| *** ganso_ has quit IRC | 22:25 | |
| *** lpabon has quit IRC | 22:26 | |
| *** ayma has quit IRC | 22:40 | |
| openstackgerrit | Merged openstack/manila: Tempest: wrong assertion on the number of shares created https://review.openstack.org/218229 | 22:42 |
| *** eharney has quit IRC | 22:47 | |
| *** lpabon has joined #openstack-manila | 22:55 | |
| *** lpabon has quit IRC | 23:00 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** haomaiwa_ has joined #openstack-manila | 23:02 | |
| *** mtanino has quit IRC | 23:21 | |
| *** david-lyle has quit IRC | 23:24 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Share Migration support in generic driver https://review.openstack.org/179791 | 23:35 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Add Share Migration feature https://review.openstack.org/179790 | 23:35 |
| *** ganso_ has joined #openstack-manila | 23:35 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/manila: Updated from global requirements https://review.openstack.org/218518 | 23:43 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!