Tuesday, 2017-03-07

openstackgerritMerged openstack/cinder master: Disallow modification of in use Volume Types  https://review.openstack.org/44068000:00
*** liverpooler has quit IRC00:01
*** alonma has joined #openstack-cinder00:02
openstackgerritMerged openstack/cinder master: cinder-backup - coordination not initialized  https://review.openstack.org/44119000:02
openstackgerritMerged openstack/cinder master: Add per-backend availability zones  https://review.openstack.org/43343700:02
openstackgerritMerged openstack/cinder master: Extracted HTTP response codes to constants  https://review.openstack.org/42674900:02
*** asselin has quit IRC00:04
*** alonma has quit IRC00:06
*** mriedem has quit IRC00:06
*** yangyapeng has quit IRC00:10
*** delatte has quit IRC00:13
*** _cjones_ has quit IRC00:21
*** _cjones_ has joined #openstack-cinder00:21
*** gcb has quit IRC00:26
*** _cjones_ has quit IRC00:26
*** ckonstan` has quit IRC00:28
*** lixiaoy1 has joined #openstack-cinder00:29
*** chhavi has joined #openstack-cinder00:40
*** zsli has joined #openstack-cinder00:42
*** zsli has quit IRC00:42
*** Shunli has joined #openstack-cinder00:42
*** chhavi has quit IRC00:44
tommylikehuping jgriffith, hope you still around :)00:46
ApoorvaMy Cinder CI is failing with the following error http://openstack-ci.tintri.com/tintri/refs-changes-28-352228-10/stack.sh.log.out has anyone seen this? Do I need to upgrade any dependencies?00:49
*** mriedem has joined #openstack-cinder00:49
smcginnisApoorva: Hmm, something looks stale in your image.00:50
smcginnisApoorva: I haven't seen that. I know pbr was bumped, but I haven't seen that failure anywhere else.00:51
Apoorvasmcginnis last week I upgraded Ubuntu to 16.04 since 14.04 is no longer supported!00:51
smcginnisApoorva: Yep, trusty is not supported anymore. I think you can pass a --force flag to devstack, but otherwise it will not stack.00:52
Apoorvasmcginnis I has successful runs on 16.04 but things started failng after that00:52
smcginnisApoorva: Do you pre-stage instances? Maybe something is left over that is no longer compatible.00:53
*** salv-orlando has joined #openstack-cinder00:53
Apoorvasmcginnis is there any way I can force clean and fresh install?00:54
*** ducttape_ has joined #openstack-cinder00:54
smcginnisApoorva: Well, I would just use a fresh 16.04 image, but for one that has already run devstack you can try running the clean.sh script in the root of the devstack repo.00:55
smcginnisApoorva: After that you could also try "sudo apt autoremove" to try to clean out other packages.00:56
Apoorvasmcginnis thanks. I will try that00:56
*** ducttape_ has quit IRC00:56
*** ducttape_ has joined #openstack-cinder00:57
smcginnisApoorva: Good luck!00:57
*** salv-orlando has quit IRC00:58
*** sdague has quit IRC01:01
*** zsli_ has joined #openstack-cinder01:01
*** namnh has joined #openstack-cinder01:03
*** jiaopengju has joined #openstack-cinder01:03
*** diablo_rojo has quit IRC01:04
*** Shunli has quit IRC01:04
*** zhurong has joined #openstack-cinder01:06
*** ducttape_ has quit IRC01:08
*** kongwei has joined #openstack-cinder01:08
*** ntpttr_ has quit IRC01:11
*** yangyapeng has joined #openstack-cinder01:13
*** yangyapeng has quit IRC01:14
*** yangyapeng has joined #openstack-cinder01:14
tommylikehuping smcginnis01:14
tommylikehusmcginnis: I noticed you and jgriffith's chat on filtering resource on a more generic way, what about the other filter that doesn't related to filtering by database column.01:17
tommylikehuthis one: https://review.openstack.org/#/c/362747/.01:17
jgriffithtommylikehu hey01:18
*** kongwei has quit IRC01:18
tommylikehuhey jgriffith01:18
jgriffithtommylikehu so the things the fk tables is another issue that I haven't completely decided on yet01:18
jgriffithtommylikehu you'll notice the get object and the volume object table itself has the ability to populate/use these filters01:19
jgriffithtommylikehu but like I said in the review, hopefully we'll NOT be needing things like attachment info much longer01:19
jgriffithtommylikehu the instance-uuid is easy since we already have that info01:19
jgriffithtommylikehu things like host shouldn't leak out to the users anyway IMHO01:20
tommylikehujgriffith: I agree with you on the attachment spec:)01:20
jgriffithWe've sort of blurred the lines between user, admin and cloud-admin01:20
jgriffithtommylikehu so you can filter on types already FYI01:21
tommylikehujgriffith: but  I am collecting more reviews for this spec: https://review.openstack.org/#/c/362747/ this filter in this spec will be used at the cinder-scheduler01:21
tommylikehuwhat do you think of this :)01:21
jgriffithlemme look01:22
*** mayela has quit IRC01:22
jgriffithtommylikehu so I'll have to look at it more closely, but in general I think adding the ability isn't a bad idea at all.01:23
jgriffithtommylikehu I would however prefer we just add a --filter or --search-options to the cinderclient command and avoid adding the one-offs in the api constantly01:23
jgriffithtommylikehu does that make sense?01:23
tommylikehuthanks, so what are you proposing is more related to expose database column and refactor client's argument like list --filter abc=xyz?01:24
jgriffithtommylikehu correct, in fact I'm hopefully going to push that up tonight01:24
jgriffithfor volumes01:24
tommylikehujgriffith: that sounds absolutely great !01:24
jgriffiththen once we all settle on that we can look at the other objects01:25
jgriffithcool, glad you like the idea01:25
jgriffithI just figured it would be nice to keep the command switches minimized and give more flexibility in filters going forward01:25
tommylikehujgriffith:yes,  one more question, about the filters01:25
jgriffithand if we can make it consistent across all objects that will be a huge help01:25
jgriffithtommylikehu sure?01:26
tommylikehuwhat if the filter value is a json not a sure?01:26
jgriffithyou mean json vs k=v format?01:26
*** Apoorva has quit IRC01:26
tommylikehuyes, maybe filter a.b = 12301:26
jgriffithfrom the API servers view it's json anyway; but from the client you want to specify json?01:26
jgriffithoh... maybe you haven't seen this: https://gist.github.com/j-griffith/f455e51d3e597af96bfe7022974b5bd901:27
jgriffithor do you mean something different?01:28
*** lucasxu has joined #openstack-cinder01:28
jgriffithof course we would make that so you can pass in multiple filters01:28
tommylikehujgriffith: I mean --filter image_metadata.key1 = 123 something like that01:29
jgriffithtommylikehu Oh!  Yeah, that's sort of what I had in mind for the next step to do relational filters01:32
jgriffithbut honestly I'm not sure how I feel about those yet01:32
*** ducttape_ has joined #openstack-cinder01:32
jgriffithI'd like to get a change with the base object columns first, then work on the relationships01:33
tommylikehujgriffith: thanks, maybe my suggestion only looks right at first sight,01:34
jgriffithtommylikehu but your suggestion of "<table>.<key> = value" is IMO probably the best way to add those01:34
jgriffithtommylikehu it gets a little tricky/ugly in setting up the config settings though... so we need to think about that a bit01:35
tommylikehuconfig of what are validate columns?01:35
jgriffithtommylikehu https://github.com/openstack/cinder/blob/master/cinder/api/common.py#L4201:36
jgriffithtommylikehu so I stumbled across that today while building something else :)01:36
jgriffithtommylikehu so that would allow the cloud-admin to select exactly what he/she wants to expose to users01:37
tommylikehujgriffith: it would add a lot of burden as you are going to support most of the resources and their columns are different01:38
jgriffithtommylikehu yeah, I've been thinking about that....01:38
jgriffithtommylikehu I'm considering pointing to a json file01:38
jgriffithand generating a default json file to be used.01:38
tommylikehuhow about add a new configure option which can specify the resource name01:38
jgriffithKeep in mind though.. this is "regular user" visible, so in theory the list shouldn't be that large01:39
jgriffithtommylikehu not sure.. can you elaborate?01:39
jgriffithtommylikehu you mean like just turn xyz resources on or off?01:39
tommylikehuvolumes.name/volumes.status something like that01:40
jgriffithhmm.... still not sure I'm following; but here's what I was thinking:01:40
jgriffith{volumes: status, volumes: name.....}01:41
jgriffith{snapshots: status, snapshots: name....}01:41
tommylikehu--query_resource_filter [volumes.name, snapshot.name]01:41
jgriffithbut maybe something like {volume_filters: [name, type...], snapshot_filters: [status, source_volume_id...]}01:42
tommylikehujgriffith: yeah01:43
jgriffithtommylikehu ok, for sure.. and I'd like to provide an API call to query that... is that the other part you were thinking about?01:43
tommylikehuprovide a API call to collect the config options?01:44
jgriffithtommylikehu well just the filters in particular:  `cinder filters-list`. dumps them all and `cinder filters-list --resource <resource-type>` for a specific resource01:45
jgriffithtommylikehu the main thing for me is that right now the API is kinda silly, because we have specific arguments and micro-versions to just explicitly pass in the filter arguments that just go straight to the DB anyway01:46
jgriffithtommylikehu that's mostly fixed in the server side, just needs worked up in the client side to start01:46
tommylikehujgriffith: great~01:47
jgriffithI'll ping you as soon as I get the first round of things posted01:47
*** ducttape_ has quit IRC01:47
jgriffithI'd greatly appreciate your feedback01:47
tommylikehujgriffith: no problem01:48
*** wanghao has joined #openstack-cinder01:50
openstackgerritTommyLike proposed openstack/cinder master: Add filter, sorter for group snapshot  https://review.openstack.org/44165101:52
*** salv-orlando has joined #openstack-cinder01:55
*** salv-orlando has quit IRC01:59
*** edmondsw has joined #openstack-cinder02:00
*** edmondsw has quit IRC02:00
*** edmondsw has joined #openstack-cinder02:00
*** Apoorva has joined #openstack-cinder02:00
*** zsli__ has joined #openstack-cinder02:01
*** zsli_ has quit IRC02:03
*** mtanino_ has quit IRC02:09
*** erlon has quit IRC02:15
*** markvoelker has quit IRC02:16
*** ducttape_ has joined #openstack-cinder02:16
*** ducttape_ has quit IRC02:18
openstackgerritTommyLike proposed openstack/cinder master: Fix generic group's unit testcase  https://review.openstack.org/44223602:23
*** akapil has joined #openstack-cinder02:40
*** akapil has quit IRC02:44
*** alonma has joined #openstack-cinder02:45
*** alonma has quit IRC02:49
*** dhinesh has quit IRC02:50
*** Apoorva has quit IRC02:50
*** edmondsw has quit IRC02:55
*** edmondsw has joined #openstack-cinder02:56
openstackgerritKen'ichi Ohmichi proposed openstack/cinder master: Switch to decorators.idempotent_id  https://review.openstack.org/44223902:58
*** wanghao has quit IRC02:58
*** wanghao has joined #openstack-cinder02:58
*** ducttape_ has joined #openstack-cinder03:00
*** edmondsw has quit IRC03:01
openstackgerritwangxiyuan proposed openstack/cinder master: [WIP]Don't change volume's status when create backups from snapshots  https://review.openstack.org/44224103:01
*** bkopilov has quit IRC03:11
*** tlian has quit IRC03:14
*** ducttape_ has quit IRC03:14
*** lucasxu has quit IRC03:15
*** sudipto has joined #openstack-cinder03:15
*** sudipto_ has joined #openstack-cinder03:15
openstackgerritTommyLike proposed openstack/cinder master: Fix generic group's unit testcase  https://review.openstack.org/44223603:16
*** lucasxu has joined #openstack-cinder03:24
*** lucasxu has quit IRC03:24
*** wanghao_ has joined #openstack-cinder03:25
*** bswartz has quit IRC03:25
*** wanghao has quit IRC03:29
*** Apoorva has joined #openstack-cinder03:32
*** tlian has joined #openstack-cinder03:33
openstackgerritTommyLike proposed openstack/cinder-specs master: Cinder volume revert to snapshot  https://review.openstack.org/31654003:34
*** links has joined #openstack-cinder03:35
openstackgerritxing-yang proposed openstack/cinder master: Tiramisu: replication group support  https://review.openstack.org/35222803:40
tommylikehuhey anyone would like to review this patch that fix the testcases  https://review.openstack.org/442236  ? It's simple and esay to review :)03:41
openstackgerritxing-yang proposed openstack/cinder master: Tiramisu: replication group support  https://review.openstack.org/35222803:52
*** gouthamr has quit IRC03:55
*** salv-orlando has joined #openstack-cinder03:55
*** mriedem has quit IRC03:56
*** alonma has joined #openstack-cinder03:57
*** salv-orlando has quit IRC04:00
*** Apoorva_ has joined #openstack-cinder04:01
*** Apoorva has quit IRC04:01
*** alonma has quit IRC04:01
*** chenhb has quit IRC04:10
*** chenhb has joined #openstack-cinder04:10
openstackgerritxing-yang proposed openstack/python-cinderclient master: Tiramisu: replication group support  https://review.openstack.org/35222904:12
*** ducttape_ has joined #openstack-cinder04:15
*** markvoelker has joined #openstack-cinder04:16
*** chhavi has joined #openstack-cinder04:17
*** ducttape_ has quit IRC04:20
*** markvoelker has quit IRC04:21
*** psachin has joined #openstack-cinder04:25
*** dhinesh has joined #openstack-cinder04:30
*** sudipto has quit IRC04:32
*** sudipto_ has quit IRC04:32
*** sudipto has joined #openstack-cinder04:33
*** sudipto_ has joined #openstack-cinder04:33
*** pdeore has joined #openstack-cinder04:33
*** bkopilov has joined #openstack-cinder04:34
*** Apoorva_ has quit IRC04:35
*** bswartz has joined #openstack-cinder04:36
*** nicolasbock has quit IRC04:38
*** akapil has joined #openstack-cinder04:38
*** akapil has quit IRC04:42
openstackgerritSoffie Huang proposed openstack/cinder master: Extend Cinder driver to utilize multiple FSS storage pools  https://review.openstack.org/42246204:43
*** sudipto_ has quit IRC04:43
*** sudipto has quit IRC04:43
*** mdnadeem has joined #openstack-cinder04:55
*** Apoorva has joined #openstack-cinder04:56
*** Poornima has joined #openstack-cinder05:10
*** ducttape_ has joined #openstack-cinder05:16
*** ducttape_ has quit IRC05:21
openstackgerritMerged openstack/cinder master: Fix generic group's unit testcase  https://review.openstack.org/44223605:21
*** wanghao_ has quit IRC05:36
*** wanghao has joined #openstack-cinder05:36
*** rcernin has joined #openstack-cinder05:39
*** salv-orlando has joined #openstack-cinder05:39
*** sudipto has joined #openstack-cinder05:39
*** sudipto_ has joined #openstack-cinder05:39
*** salv-orlando has quit IRC05:43
*** Jack_I has joined #openstack-cinder05:44
*** markvoelker has joined #openstack-cinder05:46
*** tlian has quit IRC05:47
*** markvoelker has quit IRC05:50
*** wanghao has quit IRC05:54
*** wanghao has joined #openstack-cinder05:55
*** alonmarx has joined #openstack-cinder05:55
*** Jack_Iv has joined #openstack-cinder05:55
*** Jack_Iv has quit IRC05:57
*** zsli__ has quit IRC05:58
*** zsli__ has joined #openstack-cinder05:59
*** pgadiya has joined #openstack-cinder05:59
*** Jack_Iv has joined #openstack-cinder06:00
*** yangyape_ has joined #openstack-cinder06:01
*** wanghao has quit IRC06:02
*** wanghao has joined #openstack-cinder06:02
*** yangyapeng has quit IRC06:03
*** wanghao_ has joined #openstack-cinder06:07
*** wanghao has quit IRC06:11
*** yangyapeng has joined #openstack-cinder06:12
*** Apoorva has quit IRC06:12
*** rcernin has quit IRC06:13
*** Jack_Iv has quit IRC06:15
*** yangyape_ has quit IRC06:15
*** yangyape_ has joined #openstack-cinder06:16
*** salv-orlando has joined #openstack-cinder06:16
*** ducttape_ has joined #openstack-cinder06:17
*** adisky_ has joined #openstack-cinder06:19
*** yangyapeng has quit IRC06:20
*** yangyape_ has quit IRC06:21
*** Jack_Iv has joined #openstack-cinder06:22
*** ducttape_ has quit IRC06:22
*** yangyapeng has joined #openstack-cinder06:23
openstackgerritjiamin proposed openstack/cinder master: DS8K: specify pool and lss in extra-specs  https://review.openstack.org/43585706:25
*** gcb has joined #openstack-cinder06:28
*** ChubYann has quit IRC06:33
openstackgerritRick Chen proposed openstack/cinder master: HTTPConnect SSL certification verification failed  https://review.openstack.org/44228806:33
*** e0ne has joined #openstack-cinder06:36
*** markvoelker has joined #openstack-cinder06:36
*** Jack_Iv has quit IRC06:37
*** savihou has joined #openstack-cinder06:39
*** nkrinner_afk is now known as nkrinner06:40
*** e0ne has quit IRC06:40
*** namnh has quit IRC06:48
*** markvoelker has quit IRC06:49
*** lamt has joined #openstack-cinder06:51
*** zsli__ has quit IRC06:56
*** zsli__ has joined #openstack-cinder06:57
*** e0ne has joined #openstack-cinder06:57
*** lpetrut has joined #openstack-cinder06:58
*** lamt has quit IRC07:06
*** zsli__ has quit IRC07:08
*** e0ne has quit IRC07:15
*** lkuchlan has joined #openstack-cinder07:18
*** ducttape_ has joined #openstack-cinder07:18
*** salv-orl_ has joined #openstack-cinder07:23
*** ducttape_ has quit IRC07:23
*** salv-orlando has quit IRC07:26
*** namnh has joined #openstack-cinder07:35
*** salv-orl_ has quit IRC07:35
*** lpetrut has quit IRC07:43
*** gcb has quit IRC07:45
*** tesseract has joined #openstack-cinder07:47
*** alonmarx has quit IRC07:49
*** pcaruana has joined #openstack-cinder07:50
*** dhinesh has quit IRC07:50
*** isaacb has joined #openstack-cinder07:52
*** salv-orlando has joined #openstack-cinder07:53
*** rcernin has joined #openstack-cinder07:55
openstackgerritSoffie Huang proposed openstack/cinder master: Extend Cinder driver to utilize multiple FSS storage pools  https://review.openstack.org/42246207:57
*** dalgaaf has quit IRC07:57
*** dalgaaf has joined #openstack-cinder07:57
*** dosaboy has quit IRC07:58
*** gcb has joined #openstack-cinder07:58
*** dosaboy has joined #openstack-cinder07:59
*** akapil has joined #openstack-cinder08:00
openstackgerritwangxiyuan proposed openstack/cinder master: Don't change volume's status when create backups from snapshots  https://review.openstack.org/44224108:02
*** akapil has quit IRC08:05
*** akapil has joined #openstack-cinder08:05
*** alonma has joined #openstack-cinder08:10
*** lpetrut has joined #openstack-cinder08:12
*** yangyape_ has joined #openstack-cinder08:19
*** yangyapeng has quit IRC08:22
*** s-shiono has quit IRC08:40
*** sam_nowitzki has joined #openstack-cinder08:49
openstackgerritjiaohaolin proposed openstack/cinder master: Inspur Cinder FC/iSCSI driver  https://review.openstack.org/43948808:51
*** lixiaoy1 has quit IRC08:52
openstackgerritchenhb proposed openstack/cinder master: Fix the missing args bug while lvm creates volume failed  https://review.openstack.org/44232208:59
*** zzzeek has quit IRC09:00
*** pgadiya has quit IRC09:01
*** zzzeek has joined #openstack-cinder09:01
*** chhavi has quit IRC09:05
*** pgadiya has joined #openstack-cinder09:06
*** akapil has quit IRC09:14
*** lkuchlan has quit IRC09:14
*** lpetrut has quit IRC09:15
*** akapil has joined #openstack-cinder09:15
*** wanghao_ has quit IRC09:17
*** lpetrut has joined #openstack-cinder09:17
*** ducttape_ has joined #openstack-cinder09:19
*** mvk has quit IRC09:21
*** faizy has joined #openstack-cinder09:22
*** wanghao has joined #openstack-cinder09:23
*** belmoreira has joined #openstack-cinder09:24
*** lkuchlan has joined #openstack-cinder09:24
*** ducttape_ has quit IRC09:25
*** karthikp_ has quit IRC09:25
*** bkopilov has quit IRC09:26
*** wanghao has quit IRC09:27
*** bkopilov has joined #openstack-cinder09:29
*** karthikp_ has joined #openstack-cinder09:31
*** chhavi has joined #openstack-cinder09:31
*** Poornima has quit IRC09:35
*** redondo-mk has joined #openstack-cinder09:41
*** Jack_Iv has joined #openstack-cinder09:45
openstackgerritMerged openstack/cinder master: Fix action on services raises TypeError  https://review.openstack.org/43940309:45
*** lkuchlan has quit IRC09:46
*** aarefiev_afk is now known as aarefiev09:47
*** Jack_Iv has quit IRC09:47
*** mvk has joined #openstack-cinder09:52
*** lkuchlan has joined #openstack-cinder09:55
*** bkopilov has quit IRC09:56
*** e0ne has joined #openstack-cinder10:03
*** links has quit IRC10:05
*** Poornima has joined #openstack-cinder10:06
*** bkopilov has joined #openstack-cinder10:09
*** markvoelker has joined #openstack-cinder10:10
*** ociuhandu has joined #openstack-cinder10:10
*** namnh_ has joined #openstack-cinder10:10
*** e0ne has quit IRC10:12
*** namnh has quit IRC10:13
*** yangyape_ has quit IRC10:14
*** namnh_ has quit IRC10:16
openstackgerritjiamin proposed openstack/cinder master: DS8K: specify pool and lss in extra-specs  https://review.openstack.org/43585710:17
*** links has joined #openstack-cinder10:19
*** ducttape_ has joined #openstack-cinder10:21
*** alonma has quit IRC10:21
*** alonma has joined #openstack-cinder10:22
*** salv-orl_ has joined #openstack-cinder10:23
*** e0ne has joined #openstack-cinder10:24
*** ducttape_ has quit IRC10:26
*** gcb has quit IRC10:26
*** alonma has quit IRC10:26
*** salv-orlando has quit IRC10:27
*** salv-orl_ has quit IRC10:27
*** mvk has quit IRC10:28
*** Poornima has quit IRC10:30
*** Poornima has joined #openstack-cinder10:30
*** Poornima has quit IRC10:31
*** Poornima has joined #openstack-cinder10:32
*** alonma has joined #openstack-cinder10:33
*** openstackgerrit has quit IRC10:33
*** alonma has quit IRC10:37
*** links has quit IRC10:39
*** sam_nowitzki has quit IRC10:39
*** dalgaaf has quit IRC10:40
*** mvk has joined #openstack-cinder10:41
*** Guest6667 is now known as Guest666610:43
*** dalgaaf has joined #openstack-cinder10:44
*** alonma has joined #openstack-cinder10:44
*** akapil has quit IRC10:45
*** alonma has quit IRC10:48
*** isaacb has quit IRC10:50
*** pdeore has quit IRC10:55
*** alonma has joined #openstack-cinder10:55
*** pdeore has joined #openstack-cinder10:55
*** links has joined #openstack-cinder10:55
*** akapil has joined #openstack-cinder10:55
*** alonma has quit IRC10:58
*** alonma has joined #openstack-cinder10:58
*** akapil has quit IRC11:00
*** akapil has joined #openstack-cinder11:05
*** Poornima has quit IRC11:05
*** akapil has quit IRC11:06
*** akapil has joined #openstack-cinder11:06
*** chhavi has quit IRC11:13
*** jiaopengju has quit IRC11:15
*** edmondsw has joined #openstack-cinder11:19
*** chhavi has joined #openstack-cinder11:19
*** alonma has quit IRC11:19
*** alonma has joined #openstack-cinder11:20
*** nicolasbock has joined #openstack-cinder11:21
*** edmondsw has quit IRC11:23
*** alonma has quit IRC11:24
*** sdague has joined #openstack-cinder11:27
*** alonma has joined #openstack-cinder11:31
*** alonma has quit IRC11:31
*** alonma has joined #openstack-cinder11:32
*** Poornima has joined #openstack-cinder11:33
*** isaacb has joined #openstack-cinder11:34
*** sudipto has quit IRC11:36
*** sudipto_ has quit IRC11:36
*** yangyapeng has joined #openstack-cinder11:39
*** gkadam has joined #openstack-cinder11:40
*** yangyapeng has quit IRC11:40
*** yangyapeng has joined #openstack-cinder11:40
gkadamhi , is cinder v1 api deprecated? what is the status? I want to run the tempest test for a cloud env having ceph backend. so i am confused as to ,  if api_v1 = true and api_v2 = true needed in [volume-feature-enabled] section11:41
*** wanghao has joined #openstack-cinder11:42
gkadamin tempest.conf11:42
e0negkadam: hi. both v1 and v2 APIs are deprecated now11:44
smcginnisgkadam: Hi. Yes, the v1 API has been deprecated for some time now and is now defaulted to be off.11:44
linksgkadam, https://developer.openstack.org/api-ref/block-storage/11:50
*** markvoelker has quit IRC11:50
gkadamlinks, smcginnis, e0ne  thanks for the prompt reply. so if i understand correct, these should be set to false or not mentioned at all in tempest.conf right?11:52
gkadamthis is the error i am getting for both the apis :11:53
gkadamCaptured traceback:11:53
gkadam    Traceback (most recent call last):11:53
gkadam      File "/tempest/tempest/api/volume/admin/test_volume_types.py", line 66, in test_volume_crud_with_volume_type_and_extra_specs11:53
gkadam        volume['id'], 'available')11:53
gkadam      File "/tempest/tempest/common/waiters.py", line 180, in wait_for_volume_status11:53
gkadam        raise exceptions.VolumeBuildErrorException(volume_id=volume_id)11:53
gkadam    tempest.exceptions.VolumeBuildErrorException: Volume f58e1c60-6096-41fc-9449-30cac0d31eb1 failed to build and is in ERROR status11:53
smcginnisgkadam: v1 should definitely be set to false.11:54
smcginnisgkadam: For now, I believe v2 defaults to true.11:54
gkadamsmcginnis, thanks. i will try that!11:54
smcginnisgkadam: You'll have to look in your logs as far as figuring out why you are not able to create that volume.11:55
gkadamsmcginnis, ack11:56
*** openstackgerrit has joined #openstack-cinder11:58
openstackgerritwangxiyuan proposed openstack/cinder master: Support sort snapshots with name  https://review.openstack.org/43148811:58
openstackgerritAndrey Volkov proposed openstack/cinder master: api-ref cleanup: remove quotas user related calls  https://review.openstack.org/40600111:59
openstackgerritAndrey Volkov proposed openstack/cinder master: api-ref cleanup: required flag for tenants  https://review.openstack.org/40605611:59
*** wanghao has quit IRC12:07
*** salv-orlando has joined #openstack-cinder12:09
*** faizy has quit IRC12:12
*** bkopilov has quit IRC12:13
*** isaacb has quit IRC12:20
*** dgonzalez has quit IRC12:22
*** markvoelker has joined #openstack-cinder12:23
*** catintheroof has joined #openstack-cinder12:24
*** wanghao has joined #openstack-cinder12:27
*** salv-orlando has quit IRC12:29
*** pdeore has quit IRC12:33
openstackgerritVipin Balachandran proposed openstack/cinder master: VMware: Refactor vmdk unit tests  https://review.openstack.org/44242312:35
*** wanghao has quit IRC12:37
*** wanghao has joined #openstack-cinder12:40
*** erlon has joined #openstack-cinder12:43
*** zhurong has quit IRC12:45
openstackgerritMerged openstack/cinder master: NetApp cDOT: Add check if copyoffload tool is available  https://review.openstack.org/42761312:50
*** isaacb has joined #openstack-cinder12:50
*** nikeshm has joined #openstack-cinder12:55
*** dave-mccowan has joined #openstack-cinder12:55
*** Poornima has quit IRC12:56
*** edmondsw has joined #openstack-cinder13:00
*** chlong has joined #openstack-cinder13:00
*** porrua has joined #openstack-cinder13:01
*** faizy has joined #openstack-cinder13:01
*** dustins has joined #openstack-cinder13:02
*** porrua_ has joined #openstack-cinder13:02
*** esker has joined #openstack-cinder13:03
*** wanghao has quit IRC13:05
*** porrua has quit IRC13:06
*** edmondsw has quit IRC13:06
*** lkuchlan has quit IRC13:06
*** diegows has quit IRC13:07
*** esker has quit IRC13:08
*** edmondsw has joined #openstack-cinder13:08
*** sudipto has joined #openstack-cinder13:09
*** sudipto_ has joined #openstack-cinder13:09
*** eharney has quit IRC13:12
*** edmondsw has quit IRC13:12
*** tushar_ has joined #openstack-cinder13:13
*** isaacb has quit IRC13:14
*** edmondsw has joined #openstack-cinder13:14
*** edmondsw has quit IRC13:14
tushar_We are facing some issue while running cinder testcases in CI with iSCSI connectivity....with FC its running properly13:15
*** edmondsw has joined #openstack-cinder13:15
tushar_Is there any changed required in the configuration ...We are running on ubuntu xenial13:15
smcginnistushar_: What kind of errors are you getting?13:17
tushar_smcginnis : Cinder tescases are failing.13:18
*** isaacb has joined #openstack-cinder13:19
tushar_smcginnis : the error message which I saw is - FailedISCSITargetPortalLogin: Could not login to any iSCSI portal...Earlier CI   was  running on Trusty13:20
smcginnistushar_: Well, you'll have to look through your logs to see what it was trying to do and figure out why it can't login.13:20
*** faizy has quit IRC13:20
smcginnistushar_: Sounds like maybe the wrong IP is being returned or something. Should be easy enough to try manually to verify.13:21
*** liverpooler has joined #openstack-cinder13:23
tushar_smcginnis  : Ok. I was looking in configuration part because it starts failing after moving from Tusty to Xenial13:23
*** ducttape_ has joined #openstack-cinder13:23
*** bkopilov has joined #openstack-cinder13:24
tushar_smcginnis : and the strange thing is Cinder ci with FC connectivity is running with no issues13:25
*** Poornima has joined #openstack-cinder13:25
*** alonma has quit IRC13:25
smcginnistushar_: Well, since the error is that it can't log in to the iSCSI portal, that kind of makes sense.13:25
*** alonma has joined #openstack-cinder13:26
*** salv-orlando has joined #openstack-cinder13:28
*** ducttape_ has quit IRC13:28
*** Jack_Iv has joined #openstack-cinder13:28
tushar_smcginnis : ok.13:29
*** bkopilov has quit IRC13:29
*** wN has quit IRC13:30
*** alonma has quit IRC13:30
*** alonma has joined #openstack-cinder13:32
*** zhurong has joined #openstack-cinder13:35
*** salv-orl_ has joined #openstack-cinder13:36
*** pgadiya has quit IRC13:36
*** tommylikehu_ has joined #openstack-cinder13:37
*** salv-orlando has quit IRC13:39
*** bkopilov has joined #openstack-cinder13:41
*** gouthamr has joined #openstack-cinder13:43
*** salv-orl_ has quit IRC13:46
*** ducttape_ has joined #openstack-cinder13:49
*** lpetrut has quit IRC13:49
*** links has quit IRC13:51
*** sudipto_ has quit IRC13:54
*** sudipto has quit IRC13:54
*** crushil has joined #openstack-cinder13:55
openstackgerritMerged openstack/cinder master: Switch to decorators.idempotent_id  https://review.openstack.org/44223913:56
*** gouthamr has quit IRC13:59
*** gouthamr has joined #openstack-cinder14:02
openstackgerritkavitha h r proposed openstack/os-brick master: The python 3.5 is added  https://review.openstack.org/44250714:03
*** mdnadeem has quit IRC14:06
*** lpetrut has joined #openstack-cinder14:09
*** ducttape_ has quit IRC14:12
*** Poornima has quit IRC14:12
*** lamt has joined #openstack-cinder14:14
*** xyang1 has joined #openstack-cinder14:15
*** zhurong has quit IRC14:16
*** pdeore has joined #openstack-cinder14:17
*** nikeshm has quit IRC14:17
*** pdeore has quit IRC14:18
*** pdeore has joined #openstack-cinder14:18
*** ociuhandu has quit IRC14:18
*** laurent\ has quit IRC14:20
*** mriedem has joined #openstack-cinder14:24
openstackgerritMate Lakat proposed openstack/cinder master: Create indexes for foreign keys  https://review.openstack.org/43731314:26
*** ociuhandu has joined #openstack-cinder14:29
*** lamt has quit IRC14:32
*** Jack_Iv has quit IRC14:37
*** Jack_Iv has joined #openstack-cinder14:37
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - volumes part of CG not managed correctly  https://review.openstack.org/44119914:38
*** Jack_Iv has quit IRC14:41
*** liliars has joined #openstack-cinder14:42
*** Jack_Iv has joined #openstack-cinder14:46
*** cknight has joined #openstack-cinder14:47
*** mtanino has joined #openstack-cinder14:49
*** Yogi1 has joined #openstack-cinder14:49
*** mlakat_ has joined #openstack-cinder14:52
*** arne_r has joined #openstack-cinder14:52
mlakat_Hi cinder experts, I'd like to ask about a change where I introduce some indexes to the database.14:52
mlakat_What is the policy - do we add indexes to models.py, or just leave them in migrations?14:53
mlakat_This is the change in question: https://review.openstack.org/#/c/437313/14:53
mlakat_It was -1 'd because I added the indexes in models.py (which I believe is the correct approach) - but for the latest patchset I removed it, and tbh I don't quite see how can that work later.14:54
mlakat_So I would need some expert opinion on it.14:54
*** ducttape_ has joined #openstack-cinder14:55
*** sandanar has joined #openstack-cinder14:59
*** ducttape_ has quit IRC14:59
xinlierlon: @smcginnis: good news that NetApp also fixed CI issue and voted on patch 336092. I also verified that no code conflict today morning.15:02
*** nicolasbock has quit IRC15:03
*** lamt has joined #openstack-cinder15:03
*** Julien-zte has joined #openstack-cinder15:04
*** pcaruana has quit IRC15:09
*** lucasxu has joined #openstack-cinder15:11
*** arne_r has quit IRC15:14
*** Jack_Iv has quit IRC15:22
*** ducttape_ has joined #openstack-cinder15:22
*** pcaruana has joined #openstack-cinder15:23
*** Jack_Iv has joined #openstack-cinder15:23
*** dgonzalez has joined #openstack-cinder15:24
*** tomh__ has quit IRC15:24
*** salv-orlando has joined #openstack-cinder15:24
*** isaacb has quit IRC15:26
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Change vendor_name from EMC to Dell EMC  https://review.openstack.org/44256015:27
*** Jack_Iv has quit IRC15:27
*** sandanar has quit IRC15:33
*** tommylikehu_ has quit IRC15:33
jungleboyjmlakat_:  Good question.  My initial reaction is to agree that we don't touch models.py .15:34
jungleboyjsmcginnis: Thoughts ^^15:34
mlakat_jungleboyj, but then how do we merge the migrations? is it done by hand?15:34
jungleboyjmlakat_:  What do you mean?15:35
mlakat_jungleboyj, what I meant (maybe I'm missing something) is that from time to time we "squash" migrations to one single init script, as above15:36
jungleboyjmlakat_:  Correct.15:36
jungleboyjWe do that at the end of each release.15:37
mlakat_So I was thinking how that is created - I though that the script is created by reflecting the schema?15:37
mlakat_And if it is, then we need to add the indexes to the schema as well (by schema i mean models.py )15:37
mlakat_jungleboyj, how do we create the file above!15:37
jungleboyjIt is just created by hand.  smcginnis takes any new migrations that were created during the release and puts them in the new init file.15:38
jungleboyjSo, at the end of Pike, we would take your change and add it to the latest cinder_init migration.15:38
mlakat_Ah, right. In that case it could work. I thought that it is somehow created from models.py15:39
*** alonma has quit IRC15:39
jungleboyjmlakat_:  No, it is not.  :-)15:39
jungleboyjmlakat_:  It isn't that fancy.15:39
*** ducttape_ has quit IRC15:39
*** alonma has joined #openstack-cinder15:39
mlakat_I thought that we are aiming for metadata.create_all(engine) == latest migrated database15:40
*** ducttape_ has joined #openstack-cinder15:40
mlakat_but thinking about that, it would be kind of difficult to achieve given the diverse database implementations.15:41
jungleboyjSo, in the future, if we have migrations that add Foreign Keys we should make sure that they also add an index?15:41
mlakat_jungleboyj, for postgresql, yes.15:41
*** ducttape_ has quit IRC15:42
mlakat_mysql automatically adds indexes to those, but postgres doesn't15:42
*** ducttape_ has joined #openstack-cinder15:42
mlakat_jungleboyj, another thing we might want to add is indexes on project_id.15:42
mlakat_But that's another step.15:42
mlakat_jungleboyj, I also added this models.py question to tomorrow's meeting, do you think we still need to ask someone else about this, or are we confident that it's the way (I think I am convinced :-)15:43
*** psachin has quit IRC15:43
*** alonma has quit IRC15:44
jungleboyjYeah, so looking at the code again here is what I think will happen ...15:44
jungleboyjThis checks to see if there is an index, if there isn't one, it creates it.15:45
jungleboyjAt the end of Pike that would get added to our init file so this will always be done in the future.15:45
*** eharney has joined #openstack-cinder15:45
*** alonma has joined #openstack-cinder15:45
mlakat_right - makes sense.15:45
*** alonma has quit IRC15:49
*** crose has joined #openstack-cinder15:50
*** lamt has quit IRC15:50
openstackgerritJustin A Wilson proposed openstack/python-cinderclient master: Fix discover_version  https://review.openstack.org/38562915:51
*** alonma has joined #openstack-cinder15:52
*** nkrinner is now known as nkrinner_afk15:53
jungleboyjhemna: Morning.15:53
jungleboyjmlakat_: Sorry, had a phone call and got distracted.15:53
jungleboyjSo, as core reviewers we need to then be aware that any migrations that add indexes in the future should do something similar.15:54
hemnajungleboyj, url?15:54
*** diablo_rojo has joined #openstack-cinder15:54
jungleboyjhemna: https://review.openstack.org/#/c/437313/15:55
jungleboyjmlakat_: The one thing, though, is we may then need to put the ensure_indexed function somewhere that others can share it.15:56
*** alonma has quit IRC15:56
*** chhavi has quit IRC15:56
hemnashouldn't this be done at the end of cinder manage db sync ?15:57
*** chhavi has joined #openstack-cinder15:57
hemnain addition to the db migration15:58
hemnathen we'll never have to worry about it again15:58
jungleboyjhemna:  Good thought.15:58
jungleboyjmlakat_:  Is there a way to just automatically find all indexes?15:58
*** lamt has joined #openstack-cinder16:00
*** alonmarx has joined #openstack-cinder16:00
mlakat_jungleboyj, I have some scripts, yes16:01
mlakat_So what I did: created mysql database -> listed all the indexes16:01
mlakat_created postgresql database -> listed all the indexes16:01
mlakat_and compared the two.16:01
mlakat_jungleboyj, don't you want to automatically find foreign keys instead?16:02
mlakat_I think I did that as well initially: discover foreign keys, and add indexes where they exist.16:03
mlakat_Assuming that no multi-column index covers that.16:03
hemnathat would be best to do that at the end of cinder-manage db sync16:03
hemnathen we wouldn't ever have to worry about it16:03
*** lpetrut has quit IRC16:03
mlakat_hemna, the other thing we need to find out is if sqlalchemy does it concurrently or not - or is there a policy that db migrations can cause downtime?16:04
jungleboyjmlakat_: I am sorry, yes, I meant foreign keys.16:04
jungleboyjYeah, if there is a way to do it automatically at migration time, that is the best approach.16:05
mlakat_by concurrently I meant CREATE INDEX CONCURRENTLY (https://www.postgresql.org/docs/9.1/static/sql-createindex.html)16:05
*** alonma has joined #openstack-cinder16:06
mlakat_jungleboyj, that would hurt the principle of migrations, being well-defined versions of database schema ?16:06
mlakat_what I mean is that if after each db sync, we ran an add indexes - that kind of smells, or is it just me?16:07
jungleboyjmlakat_:  Does it though?  We are just making sure that all foreign keys have indexes.  That doesn't really break the schema.16:07
*** ducttape_ has quit IRC16:07
*** dave-mcc_ has joined #openstack-cinder16:08
mlakat_ok, is there a hook that is executed on each dbsync? do we already have some sort of mechanism for this somewhere?16:09
*** ducttape_ has joined #openstack-cinder16:09
mlakat_jungleboyj, I need to run now, leaving the item on tomorrow's agenda, that will not hurt.16:10
*** pdeore has quit IRC16:10
*** alonma has quit IRC16:10
*** chenhb has quit IRC16:10
jungleboyjmlakat_:  Ok.  Yeah, it will be good to discuss there.16:10
*** dave-mccowan has quit IRC16:10
*** chenhb has joined #openstack-cinder16:10
*** mlakat_ is now known as mlakat16:11
*** dave-mccowan has joined #openstack-cinder16:11
*** e0ne has quit IRC16:12
*** dave-mcc_ has quit IRC16:12
*** akapil_ has joined #openstack-cinder16:17
*** akapil_ has quit IRC16:17
*** redondo-mk has left #openstack-cinder16:20
*** akapil has quit IRC16:20
*** redondo-mk has joined #openstack-cinder16:20
*** belmoreira has quit IRC16:22
*** crushil has quit IRC16:22
*** Julien-zte has quit IRC16:25
*** Julien-zte has joined #openstack-cinder16:25
*** Julien-zte has quit IRC16:25
*** Julien-zte has joined #openstack-cinder16:26
*** pcaruana has quit IRC16:26
*** Julien-zte has quit IRC16:26
*** Julien-zte has joined #openstack-cinder16:27
*** Julien-zte has quit IRC16:28
*** Julien-zte has joined #openstack-cinder16:28
*** Julien-zte has quit IRC16:29
*** esker has joined #openstack-cinder16:29
*** esker has quit IRC16:29
*** Julien-zte has joined #openstack-cinder16:29
*** Julien-zte has quit IRC16:29
*** rcernin has quit IRC16:30
*** chhavi_ has joined #openstack-cinder16:33
*** chhavi has quit IRC16:35
*** eharney_ has joined #openstack-cinder16:36
*** markvoelker has quit IRC16:37
*** lucasxu has quit IRC16:38
*** tesseract has quit IRC16:38
*** eharney has quit IRC16:38
*** tushar_ has quit IRC16:38
redondo-mkHi. I had issue with EMC Unity because of older version that is not supported (4.0.1) but now after the upgrade to 4.1.1 things are still not working as expected. Creating a volume from image is not stable and creating a volume from a volume does not work (error in dd command execution).16:38
*** eharney_ is now known as eharney16:39
redondo-mkI'm referring to this: https://bugs.launchpad.net/cinder/+bug/166258416:39
openstackLaunchpad bug 1662584 in Cinder "EMC Unity Volume Cloning " [High,Fix released] - Assigned to Ryan Liang (ryan-liang)16:39
*** JoseMello has joined #openstack-cinder16:39
xinlixyang1: thank you very much for fixng the EMC CI broken issue and EMC CI has voted on patch 336092. Could you please help to review the patch: https://review.openstack.org/#/c/336092/16:39
*** lucasxu has joined #openstack-cinder16:40
xyang1xinli: sure16:40
redondo-mkI would guess that the issue is not in the driver or cinder but I'm not sure...16:40
xinlixyang1: thanks, Most of 3rd party CIs broken issues have been resolved and the CIs have voted on this patch. I will continue follow up HPMSA, HPE, Nexenta, Pure(before the CI broken occurred, patch 336092 passed all the listed CIs tests)16:41
xyang1xinli: ok, thanks16:41
*** alonma has joined #openstack-cinder16:42
*** alonma has quit IRC16:46
*** crushil has joined #openstack-cinder16:48
*** nicolasbock has joined #openstack-cinder16:52
*** alonma has joined #openstack-cinder16:53
*** strigazi is now known as strigazi_AFK16:53
*** isaacb has joined #openstack-cinder16:54
*** alonma has quit IRC16:58
*** aarefiev is now known as aarefiev_afk17:00
*** chhavi_ has quit IRC17:02
openstackgerritMerged openstack/os-brick master: Add Python 3.5 classifier and venv  https://review.openstack.org/42303217:04
*** alonma has joined #openstack-cinder17:04
*** alonma has quit IRC17:09
*** savihou has quit IRC17:09
*** markvoelker has joined #openstack-cinder17:12
*** ntpttr has joined #openstack-cinder17:15
geguileojungleboyj: patrickeast hemna Could you have another look at the multipath friendly names patch?  https://review.openstack.org/43310217:16
*** lucasxu has quit IRC17:16
*** ntpttr_ has joined #openstack-cinder17:16
bluexdo we still support config file based default quota limits, or can I remove code that support this (deprecated since 2013)17:17
jungleboyjgeguileo:  Sure looking.17:20
geguileojungleboyj: thanks!17:20
*** isaacb has quit IRC17:25
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements  https://review.openstack.org/43205517:25
*** Jack_V has joined #openstack-cinder17:25
*** Apoorva has joined #openstack-cinder17:26
*** Jack_V has quit IRC17:28
*** Jack_I has quit IRC17:29
*** Jack_I has joined #openstack-cinder17:29
*** nikeshm has joined #openstack-cinder17:30
*** lucasxu has joined #openstack-cinder17:31
*** prashkre has joined #openstack-cinder17:38
*** ntpttr_ has quit IRC17:39
*** erhudy has joined #openstack-cinder17:42
nikeshmhi in my CI env, quota is not updating sometimes17:46
nikeshmdid any one face the issue17:46
nikeshmis there any good solution rather than logging in nova database and do some reset17:47
*** Jack_Iv has joined #openstack-cinder17:47
nikeshmpatrickeast:  did you face the issue17:48
*** sudipto has joined #openstack-cinder17:48
*** sudipto_ has joined #openstack-cinder17:48
nikeshmpatrickeast:  BTW, i found and solved the problem due to which our CI was not running cherry picking the cinder patch code automatically17:49
nikeshmsome dns issue17:49
*** Jack_Iv has quit IRC17:50
*** Jack_Iv has joined #openstack-cinder17:51
*** david-lyle_ has joined #openstack-cinder17:52
nikeshmsmcginnis patrickeast: after correcting dns issue inside nodepool VM, Kaminario CI is running tests on top of the patch code which is under investigation, now it is working as expected :)17:54
*** david-lyle has quit IRC17:54
*** david-lyle_ is now known as david-lyle17:55
*** Jack_Iv has quit IRC17:57
*** ducttape_ has quit IRC17:59
*** sudipto has quit IRC18:00
*** sudipto_ has quit IRC18:00
jgriffithHey hemna what's the link to that fancy whiz bang tool you have to check 3'rd party CI stats?18:01
*** links has joined #openstack-cinder18:01
hemnajgriffith, https://github.com/openstack/third-party-ci-tools/blob/master/monitoring/lastcomment-scoreboard/cireporter.py18:01
jgriffithhemna thanks!18:01
smcginnisjgriffith: Regularly updated output, if that's all you're looking for: http://cinderstats-dellstorage.rhcloud.com/cireport.txt18:02
hemnathere is also this:18:02
*** dhinesh has joined #openstack-cinder18:02
hemnathat takes the input of cinder's generate_driver_list18:02
hemnaand runs all the CI's listed in there.18:02
hemnaor the cireporter.py can run against 1 or N Ci's18:03
smcginnishemna: Hmm, I'm running ci_reporter. Is cinder_ci basically the same, but dynamically updated?18:04
*** david-lyle has quit IRC18:04
*** ducttape_ has joined #openstack-cinder18:04
*** david-lyle has joined #openstack-cinder18:05
*** _cjones_ has joined #openstack-cinder18:06
jgriffithI was running the reporter just to isolate mine for now18:13
*** chhavi_ has joined #openstack-cinder18:13
jgriffith96% average out of the last 50 W00T18:13
jgriffith82% out of the last 200, not terrible18:13
jgriffithand I'm even posting the logs again now18:14
jgriffithso I can go back to ignoring it again until the first week of the month :)18:14
smcginnisjgriffith: I'm guessing that's better than average,18:14
jgriffith@smcginnis yeah, I haven't looked... just have a recurring thing on my calendar to "inspect CI" the first Monday of each month" :)18:15
*** _cjones_ has quit IRC18:15
*** _cjones_ has joined #openstack-cinder18:20
*** david-lyle has quit IRC18:23
*** david-lyle has joined #openstack-cinder18:25
jungleboyjjgriffith: Nice!  Can you teach other CI owners?  ;-)18:27
*** david-lyle has quit IRC18:29
jgriffithjungleboyj sure....   https://support.google.com/calendar/answer/37115?co=GENIE.Platform%3DDesktop&hl=en18:30
jgriffithnot hard :)18:30
hemnasmcginnis, cinder_ci consumes the JSON output of cinder's generate_driver_list.py > cinder_drivers.json18:31
hemnaand uses that data to determine which CI's to report on with cireporter.py18:31
hemnaso run generate_driver_list.py first, output that to a file.   use that file to feed into cinder_ci.py18:32
hemnaand you'll get whatever cinder drivers there are and their associated CIs (from CI_WIKI_NAME)18:33
*** david-lyle has joined #openstack-cinder18:36
*** mvk has quit IRC18:36
jungleboyjjgriffith: Smart Alec.18:37
jgriffithjungleboyj :)18:38
nikeshmdid any one face https://ask.openstack.org/en/question/494/how-to-reset-incorrect-quota-count/18:39
*** salv-orlando has quit IRC18:39
*** eharney has quit IRC18:41
*** salv-orlando has joined #openstack-cinder18:44
*** eharney has joined #openstack-cinder18:45
*** xyang has joined #openstack-cinder18:46
nikeshmgood n818:50
*** nikeshm has quit IRC18:50
jungleboyjsmcginnis:  We can add config options with a backport, we just can remove or change them.  Correct?18:52
smcginnisjungleboyj: Good point. I believe so, but let me take another look at the guidelines.18:52
jungleboyjOk.  I am pretty sure that is the case.18:52
smcginnisjungleboyj: Yeah, I think this is the only issue to watch out for, in that respect: "Incompatible config file changes"18:53
smcginnisjungleboyj: https://docs.openstack.org/project-team-guide/stable-branches.html#review-guidelines18:54
jungleboyjsmcginnis: That was how I remembered it.  Thanks for confirming.  :-)18:54
smcginnisjungleboyj: Good to clarify that. ;)18:55
*** ducttape_ has quit IRC18:56
jungleboyjOk those backports are taken care of.18:57
*** adisky_ has quit IRC18:59
*** asselin has joined #openstack-cinder18:59
*** sdague has quit IRC19:02
*** david-lyle has quit IRC19:04
*** david-lyle has joined #openstack-cinder19:06
*** mvk has joined #openstack-cinder19:07
*** crushil has quit IRC19:08
*** alonma has joined #openstack-cinder19:11
*** ducttape_ has joined #openstack-cinder19:13
*** _ducttape_ has joined #openstack-cinder19:14
*** alonma has quit IRC19:15
*** ducttape_ has quit IRC19:18
*** crushil has joined #openstack-cinder19:18
*** _ducttape_ has quit IRC19:18
*** markstur has joined #openstack-cinder19:19
*** xyang has quit IRC19:19
*** salv-orl_ has joined #openstack-cinder19:22
erlonxinli: hey, have reviewed the patch. Thanks for the work on it. Its almost good, just need to address the comments I added.19:24
*** salv-orlando has quit IRC19:25
*** Jack_Iv has joined #openstack-cinder19:26
erlonjgriffith: can you also check this (https://review.openstack.org/#/c/336092/) when you have a chance? You suggested this approach at first so, good to have your opinion.19:27
prashkreHi. Could you please someone check the bug created https://bugs.launchpad.net/cinder/+bug/1670801.19:33
openstackLaunchpad bug 1670801 in Cinder "Filters are not applied on Consistency group snapshots GET API" [Undecided,New]19:33
prashkreis it valid or due to any constraints the filters are not applied?19:34
xyang1prashkre: they are not implemented.  There is a patch to add it to group snapshots though19:35
xyang1prashkre: https://review.openstack.org/#/c/441651/19:37
openstackgerritchristopher.uhler proposed openstack/os-brick master: os-brick connector for Veritas HyperScale  https://review.openstack.org/44275419:39
*** ChubYann has joined #openstack-cinder19:43
*** xyang has joined #openstack-cinder19:44
prashkrexyang1: Thanks for pointing me to https://review.openstack.org/#/c/441651/.19:45
*** xyang has quit IRC19:45
*** redondo-mk has quit IRC19:45
prashkrexyang1: so this is available for v3 when patch merges. I guess we need to support the v2 API as well?19:47
*** xyang has joined #openstack-cinder19:47
xyang1prashkre: group snapshots are only introduced in v3.  Also we are talking about deprecating CG API some day, so I don't think we need to add this there19:49
prashkrexyang1: Thanks for info. when can we expect this patch will merge?19:52
*** crose has quit IRC19:52
xyang1prashkre: this was just submitted.  Will go thru the normal code review process.  Can't tell you exactly when it will be merged, but should be available in pike19:53
prashkrexyang1: sure. Thank you!.19:55
xyang1prashkre: you are welcome19:55
*** e0ne has joined #openstack-cinder19:57
*** lucasxu has quit IRC19:57
*** chhavi_ has quit IRC20:00
*** gouthamr has quit IRC20:01
*** salv-orl_ has quit IRC20:03
Swansonxyang1, How do I tell if a group is of a cg type?20:04
Swansonxyang1, or that has a consistent snapshot property?20:04
*** lucasxu has joined #openstack-cinder20:04
xyangSwanson: are you using Groups command or CG command?  Is this a migrated group from CG?20:04
Swansonxyang, I'm coding create_group()20:05
xyangSwanson: your group type should have consistent_group_snapshot_enabled set to "<is> True"20:06
Swansonxyang, you're way more useful than smcginnis. Thanks!20:07
xyangSwanson: :)20:07
smcginnisI'm good with that.20:07
xyangSwanson: Take a look of this section: https://github.com/openstack/cinder/blob/master/doc/source/devref/groups.rst#group-type-and-group-specs--volume-types-and-extra-specs20:07
*** ducttape_ has joined #openstack-cinder20:07
*** ducttape_ has quit IRC20:08
*** ducttape_ has joined #openstack-cinder20:08
*** links has quit IRC20:09
*** wN has joined #openstack-cinder20:12
*** wN has quit IRC20:12
*** wN has joined #openstack-cinder20:12
*** simondodsley has joined #openstack-cinder20:13
*** dustins has quit IRC20:15
jgriffitherlon still around?20:15
jgriffitherlon I don't really remember the conversation on https://review.openstack.org/#/c/336092/2920:15
jgriffitherlon I'm not sure I follow exactly what you have going on there, or if it's the best option20:16
jgriffitherlon I do sort of recall something more along the lines of checking the volume-size when the existing create call returns, and if it's not correct issue the extend call20:16
erlonjgriffith: that was about creating a commom method to drivers to fix the create_from_* problem20:16
jgriffitherlon right... but I think what I had in mind was a bit different20:17
jgriffithand didn't change the method or the drivers20:17
erlonjgriffith: initially I implemented in flow_manager, than you suggested to add a common function in the base class20:17
jgriffitherlon sorry... I honestly don't remember the details of what we talked about there, and I guess the problem is that the drivers are broke so my statement about checking and extending doesn't actually work either20:18
erlonjgriffith: yeah the thing is, several drivers solved the bug in their drivers code, while another bunch no, so, we either would fix driver by driver or implement a common way, and 'revert' the fixes drivers had added. Thats why we neeed to change so many drivers20:19
jgriffitherlon so I don't follow exactly how your patch works I'm afraid20:19
jgriffitherlon well, what sucks is that we 'don't know' if a driver did the right thing or not20:19
jgriffithUnless we create a tempest test that attaches the volume and checks the size from the Instance to make sure it matches what Cinder reports20:20
Swansonxyang, create_group_from_src now required?20:20
xyangSwanson: yes20:20
jgriffitherlon anyway... back to trying to fix it in the flow etc...20:20
erlonjgriffith: so, to recap, the problem happens when a driver tries to create a volume from a snapshot or other volume that have a a smaller size than the volume being created20:20
erlonjgriffith: this is anoother problem20:20
jgriffithI'm not sure how you're code does this exactly other than you inspected the driver yourself and tried to interpret how it works20:20
erlonjgriffith: checking if the backend is really extending20:21
jgriffitherlon oh, I'm familiar with the problem, I'm having difficulty getting my head wrapped around the proposed fix though :)20:21
erlonjgriffith: yes, tough work, but that was the approach20:21
jgriffitherlon right, but now we have kind of an ugly thing...20:21
jgriffitherlon we have this function variable thing you set up based on drivers that did the right thing, and others use the old method still20:22
*** crose has joined #openstack-cinder20:22
erlonjgriffith: we implemented a common behavior in the base class. The driver creates a volume and if needed it extend right after, by calling driver.extend()20:22
jgriffitherlon what I mean is, it *might* fix the actual problem, but in turn it kind of creates a new one20:22
jgriffitherlon but only *some* of the drivers use that method near as I can tell.... others still use the old one20:23
openstackgerritEric Harney proposed openstack/cinder master: Tests: Use six.moves.urllib urlencode  https://review.openstack.org/44276920:23
jgriffithand the "calling extend if needed" is the part that I'm saying other than looking at the driver code I'm not sure now how you determine that?20:23
erlonjgriffith: looking at the driver code and checking if the CI is passing after the fix20:23
jgriffitherlon I dislike having unique paths based on the driver being called is my point I guess20:24
jgriffitherlon so if you wanted to do something like what you have here I'd like to offer an alternative suggestion if you don't mind...20:24
erlonjgriffith: thats why I firstly checked all CI results, and now xinli did the same20:24
openstackgerritKen'ichi Ohmichi proposed openstack/cinder master: Switch to use stable data_utils  https://review.openstack.org/44277020:25
jgriffitherlon implement a new driver method "verify_volume_size"20:25
erlonjgriffith: what you mean?20:25
erlonjgriffith: with the unique paths20:25
jgriffitherlon https://review.openstack.org/#/c/336092/29/cinder/volume/driver.py20:26
jgriffithLine #574 and 58120:26
*** _cjones_ has quit IRC20:27
jgriffithoh wait... no, that's not it20:27
jgriffithjust a sec20:27
erlonjgriffith: hmm, got it20:27
jgriffitherlon so like this is confusing to me. https://review.openstack.org/#/c/336092/29/cinder/volume/drivers/lvm.py20:28
jgriffitherlon so that base driver does exactly what it's supposed to... but now we break it out into 2 seperate driver calls using a clone_func variable20:29
*** gouthamr has joined #openstack-cinder20:29
erlonjgriffith: honestly I dind't liked the final result as well, having to touch all drivers that already had the problem solved. That would be more acceptable if was done before everyone had its own solution, but, I think xinli fall in this patch I did because I believe he was trying to fix the easy way and someone pointed to a common solution again20:30
jgriffithI also don't quite understand, the SolidFire driver for example is another one that does things correctly; but it's not changed here20:30
jgriffitherlon yeah, sadly I can't tell you how many times I have what I think is going to be a great and elegant solution, but once I get done with it I'm not as happy :(20:30
jgriffitherlon Yeah, fixing in the driver without actually modifying the driver might not be something we can figure out now that I think about it more20:31
openstackgerritMerged openstack/os-brick master: Fix multipath flush when using friendly names  https://review.openstack.org/43310220:31
jgriffithand if we have to go to that level then the driver should just be fixed :)20:31
jgriffithin other words, as much time and effort as you put into this inspecting and working each driver, maybe it would be just as easy to fix the 100 drivers by adding the check and extend call inside of them?20:32
jgriffithThen making sure we have a tempest test that exposes the failure20:32
jgriffithI dunno20:32
erlonjgriffith: you raised a good point in the LVM driver, it seems it should only have the extend removed, and not add the function to overhide the default behaviour20:32
jgriffiththe problem with so many drivers is there's no easy way to fix this or advance the ball20:32
jgriffithwe've kinda screwed ourselves :)20:33
erlonjgriffith: man I kill you!!20:34
*** Jack_Iv has quit IRC20:34
*** Jack_Iv has joined #openstack-cinder20:34
erlonjgriffith: haha20:34
*** Jack_Iv has quit IRC20:35
*** JoseMello has quit IRC20:35
erlonjgriffith: well, in the end, drivers that didnt fixed that already are the one that are not actively maintaining20:35
smcginnisThe problem is, some drivers can more optimally get the new, correct, size during the clone.20:35
*** Jack_Iv has joined #openstack-cinder20:35
smcginnisAnd others have to do the fallback of just extending after clonging.20:35
smcginnisI think when we discussed this in the past, really the only way to do the right thing is to change the driver interface.20:36
smcginnisAnd add the smarts into that interaction so the driver can say whether it was able to get the new size or not.20:36
erlonsmcginnis: I know that, and the patch handles those cases too20:36
openstackgerritMerged openstack/os-brick master: Retry multipath flush when map is in use  https://review.openstack.org/43310320:37
erlonsmcginnis: the problem here is, what is better, to remove snipped on drivers that already individually fixed the problem and then use the commom solution, or just add the extend function to the others that havent fixed that already?20:39
erlonsmcginnis: if this was solution was put when we first reported the problem, and no drivers had that fixed, the commom method would me more appealing, now, is not that much anymore20:40
smcginniserlon: Well, my point isn't about drivers that have "fixed the problem". It's about ones that can more optimally create the larger cloned volume and would therefore be more inefficient by resorting to a generic approach.20:40
*** ducttape_ has quit IRC20:40
*** Jack_Iv has quit IRC20:40
*** Jack_Iv has joined #openstack-cinder20:40
*** rhe00 has quit IRC20:41
erlonsmcginnis: hmm, so, those ones are the ones that created the volumes using its optimized way with the correct size. For those we skip the generic way: https://review.openstack.org/#/c/336092/29/cinder/volume/drivers/violin/v7000_fcp.py20:42
erlonsmcginnis: jgriffith : xinli : got to go, we pick that later20:43
smcginniserlon: But as it is now, like jgriffith pointed out, that requires knowledge of the driver and whether they can do that or not.20:43
*** Jack_Iv_ has joined #openstack-cinder20:43
*** Jack_Iv has quit IRC20:47
xinli@smcginnis: so far only two vendors have really fixed the issues 1) IBM Storawize 2) VMAX, all the other dirvers have not fixed the issue.20:49
smcginnisxinli: The "issue" being, being able to create a clone that is larger than the original?20:54
xinli@smcginnis: yes, regarding the 33609220:55
*** prashkre has quit IRC20:55
*** dustins has joined #openstack-cinder20:55
smcginnisxinli: That is not correct. Quite a few drivers actually handle that correctly.20:56
xinli@smcginnis: in this case, what is the next step? just leaving the drivers which handles correctly ?20:57
smcginnisxinli: Well, I'm not sure really.20:58
smcginnisxinli: I think to really fix this in an optimal way would require changing the driver interface.20:58
jgriffithxinli erlon @smcginnis I think we have a couple options:20:58
jgriffithto @smcginnis we could change the interface20:58
xinli@smcginnis: at least run all the driver code aginst CI and make all the CIs issue resolved. : )20:59
jgriffithjust go on a rampage and fix the drivers manually20:59
*** xyang has quit IRC20:59
jgriffithimplement a verification check in tempest, expose the losers and ridicule them mercilessly20:59
jgriffith@erlon I'm fine with what you were doing if we just do as @smcginnis pointed out and make it a *new* interface and remove the old one, but the problem I have with that is the point about some doing it when they actually issue the clone call etc.  So it gets clunky no matter what.  IMHO I think the driver should just be responsible to do what it's supposed to.  and in the future we should be better when21:01
jgriffith reviewing to make sure this sort of thing doesn't slip through the cracks (or again.. a verification test for it)21:01
*** Jack_I has quit IRC21:02
jgriffitherlon I kinda feel crappy because you obviously put a ton of work into that patch21:02
smcginnisjgriffith: I actually kind of like the idea of adding a scenario test that does what you described.21:02
*** sdague has joined #openstack-cinder21:03
smcginnisjgriffith: Create a volume, write to it, detach, clone it to a new volume of a larger size, attach, verify new size.21:03
smcginnisSomething like that.21:03
jgriffith@smcginnis @erlon there's one other option that we haven't mentioned that fixes ALL of this21:03
smcginnisjgriffith: Just use LVM?21:03
jgriffithhaah!  Well that's not bad.. but no :)21:03
jgriffithjust disable clone and create from snapshot of different size in cinder/volume/api.py21:04
jgriffithdon't even let it be a thing anymore21:04
jgriffithuser can still extend the volume on their own if they like21:04
jgriffiththis chaining things together to make the user *happier* sometimes isn't really worth it IMO21:04
jgriffithThat would *fix* the problem, and not break anything at the same time21:05
jgriffithwell.. not break anything on our side :)21:05
smcginnisWell, kind of hard to disable cloning now. But I think some drivers did have the same issue with create from snapshot. At least at one point.21:06
jgriffith@smcginnis no no.. not saying dsiable cloning21:06
smcginnisJust internally make it a create from snapshot?21:06
*** lpetrut has joined #openstack-cinder21:07
jgriffithI mean enforce that the size MUST be the same21:07
jgriffithso like "if src_sz != dest_sz:  PUKE"21:07
jgriffithI don't remember all the conversations and arguments around not just doing that.  I may have even been one to argue in favor of allowing an auto-extend (but I don't think so)21:08
jgriffithusers have the API's to do all of these things, we don't necessarily gain much by doing it all for them IMO21:09
*** xyang has joined #openstack-cinder21:09
smcginnisjgriffith: I don't think that would help us now.21:09
smcginnisjgriffith: It would be an API change.21:09
smcginnisjgriffith: So we could add a mv that does it that way.21:09
jgriffith@smcginnis yeah21:09
jgriffithbut this shows how useless mv's are21:09
smcginnisjgriffith: But I think we would still need to support the old way.21:09
jgriffiththen we just have to support "MORE" things21:09
smcginnisjgriffith: Yeah, just cumulative, at least so far.21:10
jgriffithwe're really screwed21:10
jgriffithhub clone openstack/cinder21:10
*** alonma has joined #openstack-cinder21:12
xinlijgriffith: @smcginnis: at this time, we need to hold and I will wait for your final decision.21:13
*** eharney has quit IRC21:13
smcginnisxinli: Sorry for not having a clear answer for you.21:14
xinli@smcginnis: it's OK, I also need to learn the lesson, the good thing is that this patch need to pass all the 3rd CIs, and I have worked with all of them to make most of the CI working now : )21:15
smcginnisxinli: Yes, thank you for all you did trying to get CIs to pass. That got a lot of maintainers to pay more attention to their broken CIs.21:15
smcginnisxinli: That's very appreciated.21:16
*** alonma has quit IRC21:17
jgriffithI'm not even sure if this was an intentional *feature* :(21:17
smcginnisjgriffith: I don't think it was.21:17
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements  https://review.openstack.org/43205521:17
jgriffithWell, I'll let the API working group figure out how to deal with stuff like this as per their view of the world21:18
jgriffithseems like the only option now is to fix the broken drivers21:19
jgriffithor fork cinder :)21:19
*** e0ne has quit IRC21:20
smcginnisReally, I think there were maybe just a handful of drivers that did not accomodate for the larger destination size now.21:20
smcginnisSo a tempest test that would make their CIs fail would probably get their attention pretty quick and get it fixed.21:21
smcginnisOr get them removed. Either way, problem solved.21:21
smcginnisWe've been pretty good on new driver reviews of checking for it.21:21
smcginnisSo I think the newer drivers should all be good.21:21
smcginnisTempest tests would make sure they all are correct.21:22
*** alonma has joined #openstack-cinder21:22
jgriffith@smcginnis agreed21:25
smcginnisLooks like there is an API test that gets the volume and verifies the size, but that's just what Cinder has in the DB.21:26
smcginnisWe would need a full scenario test that actually attaches and looks at the volume.21:26
*** alonma has quit IRC21:26
jgriffith@smcginnis right21:26
jgriffith@smcginnis I guess I was under the impression that erlon had already identified all of the culprits based on the Bugs in the commit:  https://review.openstack.org/#/c/336092/2921:27
jgriffithor *you* did :)21:27
smcginnisjgriffith: I think most(all?) of those are just ones I filed when we first realized this was how it worked and that we needed to fix some drivers.21:28
jgriffithso 1821:28
smcginnisjgriffith: I went through and filed a bug for every driver that I didn't see explicitly checking the requested size.21:28
smcginnisjgriffith: I do think some of those have already been fixes and/or actually do handle it correctly.21:28
jgriffith@smcginnis that's a pretty admirable effort!21:29
*** eharney has joined #openstack-cinder21:29
smcginnisjgriffith: It was all based on my quick survey of the driver code.21:29
jgriffithoh.. well that's good21:29
jgriffithso let's split it up21:29
smcginnisjgriffith: Sadly, when I filed those I assumed they would all get jumped on and resolved by the end of that cycle.21:29
jgriffithget volunteers tomorrow, split it up based on alphabet21:29
smcginnisbut that was kilo or mitaka I think.21:29
jgriffithcheck every driver21:30
jgriffithor take driver-list/num-volunteers21:30
jgriffithcreate a comittee.. or TigerTeam or whatever you want to call it21:30
jgriffithjust fix this crap and move on21:30
jgriffithor not21:31
jgriffithjust an idea21:31
smcginnisOooh, tiger teams. Exciting! :D21:32
*** Yogi1 has quit IRC21:36
*** gouthamr has quit IRC21:37
*** salv-orlando has joined #openstack-cinder21:38
*** ducttape_ has joined #openstack-cinder21:40
*** frontrunner has joined #openstack-cinder21:41
*** shaner has quit IRC21:41
*** shaner has joined #openstack-cinder21:43
*** redondo-mk has joined #openstack-cinder21:44
*** ducttape_ has quit IRC21:46
*** asselin has quit IRC21:47
*** crose has quit IRC21:48
*** cknight has quit IRC21:51
*** redondo-mk has quit IRC21:52
*** dave-mcc_ has joined #openstack-cinder21:53
*** xyang has quit IRC21:54
*** dave-mccowan has quit IRC21:54
*** xyang has joined #openstack-cinder21:56
*** gouthamr has joined #openstack-cinder21:59
*** crushil has quit IRC22:00
*** xyang has quit IRC22:02
*** lucasxu has quit IRC22:02
*** lucasxu has joined #openstack-cinder22:03
*** dave-mcc_ has quit IRC22:03
*** porrua_ has quit IRC22:06
xinli@smcginnis: I also fixed all your asw ell as erlon's comments and try to commit it, is it OK.22:11
smcginnisxinli: Sure, if you want to submit those changes, if we are able to move forward in some way with that it doesn't hurt.22:13
*** cknight has joined #openstack-cinder22:13
xinli@smcginnis: based on your talk with jgriffith, do we need to split the bugs in several groups?22:14
*** ducttape_ has joined #openstack-cinder22:16
smcginnisxinli: If we end up just having each driver make sure they handle the size change correctly, then there would be a patch per bug.22:17
smcginnisxinli: If you're interested, I think the best thing would be to have a tempest scenario test that verifies this.22:18
smcginnisxinli: I think we need to do something like this scenario test: https://github.com/openstack/tempest/blob/master/tempest/scenario/test_volume_migrate_attached.py22:18
smcginnisxinli: But that creates and instance and attaches a volume and writes something like a timestamp on it.22:18
smcginnisxinli: Then detaches the volume, create a snapshot, create a new larger volume from that snapshot, and attach that new volume to the instance.22:19
*** cknight1 has joined #openstack-cinder22:19
smcginnisAnd within the instance assert that the timestamp is still there and that the volume shows up as a larger size.22:19
smcginnisIf we had a tempest test like that, that would highlight which drivers really are not supporting this right.22:20
smcginnisAnd they would be forced to fix it since their CI would fail on every patch once this tempest test is merged to tempest.22:20
smcginnisUnless of course they skip the test, but we can check on that.22:20
*** alonmarx has quit IRC22:21
smcginnisJust an idea for now if you are highly motivated. :) Otherwise we can wait and see if there are any better ideas.22:21
*** cknight has quit IRC22:21
xinli@smcginnis: it seems that the erlon's generic solution won't be used in this case?22:22
smcginnisxinli: Well, maybe, maybe not. I think there is value in having a tempest test like that either way.22:22
*** laurent\ has joined #openstack-cinder22:23
openstackgerritchristopher.uhler proposed openstack/os-brick master: os-brick connector for Veritas HyperScale  https://review.openstack.org/44275422:24
*** lpetrut has quit IRC22:26
*** cknight1 has quit IRC22:28
*** Jack_Iv_ has quit IRC22:29
xinli@smcginnis: if core team wants to split the big patch into diff. smaller one for each drivers, can I also pick some , since I am familiar with these code?22:30
*** Jack_Iv has joined #openstack-cinder22:30
smcginnisxinli: Yeah, if you would want to help with that effort, that would be great. You do have a good knowledge of that area now after all you've gone through. :)22:32
*** Jack_Iv has quit IRC22:32
*** lucasxu has quit IRC22:36
*** lucasxu has joined #openstack-cinder22:40
*** catintheroof has quit IRC22:41
*** catintheroof has joined #openstack-cinder22:42
*** Jack_Iv has joined #openstack-cinder22:44
*** Jack_Iv has quit IRC22:45
*** catintheroof has quit IRC22:47
openstackgerritXinli Guan proposed openstack/cinder master: Fix 'driver does not honor clone/snapshot' bugs  https://review.openstack.org/33609222:48
*** dustins has quit IRC22:52
*** sdague has quit IRC22:57
*** ducttape_ has quit IRC23:04
*** erlon has quit IRC23:05
*** ducttape_ has joined #openstack-cinder23:07
openstackgerritEric Harney proposed openstack/cinder master: Tests: Use six.moves.urllib urlencode  https://review.openstack.org/44276923:07
openstackgerritEric Harney proposed openstack/cinder master: Bump pylint to 1.4.3  https://review.openstack.org/44285723:11
*** lucasxu has quit IRC23:12
*** mlakat has quit IRC23:15
*** edmondsw has quit IRC23:16
*** eharney has quit IRC23:16
*** ociuhandu has quit IRC23:17
*** pck_ has quit IRC23:18
*** grumpycatt has quit IRC23:18
*** leseb_ has quit IRC23:18
*** grumpycatt has joined #openstack-cinder23:18
*** pck has joined #openstack-cinder23:19
*** leseb_ has joined #openstack-cinder23:20
*** lamt has quit IRC23:23
*** grumpycatt has quit IRC23:25
*** grumpycatt has joined #openstack-cinder23:25
*** xyang1 has quit IRC23:27
*** s-shiono has joined #openstack-cinder23:31
*** catintheroof has joined #openstack-cinder23:32
*** xinli has quit IRC23:33
*** mriedem has quit IRC23:36
*** ociuhandu has joined #openstack-cinder23:40
*** alonmarx has joined #openstack-cinder23:43
*** edmondsw has joined #openstack-cinder23:52
*** ducttape_ has quit IRC23:55
*** ducttape_ has joined #openstack-cinder23:56
*** edmondsw has quit IRC23:56

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