Friday, 2019-01-04

*** rosmaita has left #openstack-cinder00:10
*** dcdamien has quit IRC00:21
*** dcdamien has joined #openstack-cinder00:24
*** Liang__ has joined #openstack-cinder00:52
*** Liang__ is now known as LiangFang00:53
*** markvoelker has quit IRC00:54
*** markvoelker has joined #openstack-cinder00:54
*** lbragstad has quit IRC00:59
*** markvoelker has quit IRC00:59
*** lbragstad has joined #openstack-cinder01:08
*** lbragstad has quit IRC01:09
openstackgerritwanghao proposed openstack/cinder master: Support Incremental Backup Completion In RBD  https://review.openstack.org/62794101:18
*** sayalilunkad has quit IRC01:23
*** dcdamien has quit IRC01:23
*** bhdn has quit IRC01:24
*** hedvig_01 has quit IRC01:26
*** sayalilunkad has joined #openstack-cinder01:28
*** bhdn has joined #openstack-cinder01:32
*** erlon has quit IRC01:38
*** dave-mccowan has quit IRC01:41
*** dave-mccowan has joined #openstack-cinder01:49
*** markvoelker has joined #openstack-cinder01:55
*** mhen has quit IRC02:05
*** mhen has joined #openstack-cinder02:06
openstackgerritwanghao proposed openstack/cinder master: Using wrong parameter in test_backup_ceph  https://review.openstack.org/62833102:42
*** doubletao has joined #openstack-cinder02:46
*** markvoelker has quit IRC02:46
doubletaoHello, All. I am a new contributor of openstack-cinder. There is a new patch of Huawei FusionStorage Cinder Driver, Could you have a code review for it? Here is the patch: https://review.openstack.org/61995202:53
*** psachin has joined #openstack-cinder03:03
*** psachin has quit IRC03:07
*** psachin has joined #openstack-cinder03:08
*** lifeless has joined #openstack-cinder03:10
*** chhagarw has quit IRC03:14
*** abhishekk has joined #openstack-cinder03:35
*** sapd1 has joined #openstack-cinder03:37
*** sapd1_ has quit IRC04:12
*** sapd1__ has joined #openstack-cinder04:12
*** chhagarw has joined #openstack-cinder04:17
*** dave-mccowan has quit IRC04:25
*** sapd1_x has joined #openstack-cinder04:31
*** sapd1 has quit IRC04:31
*** udesale has joined #openstack-cinder04:52
*** chhagarw has quit IRC04:54
openstackgerritYingxin Cheng proposed openstack/cinder master: [wip] Cinder RSD Driver  https://review.openstack.org/62146505:13
*** whoami-rajat has joined #openstack-cinder05:22
*** jbernard has quit IRC06:39
*** jbernard_ has joined #openstack-cinder06:39
*** rcernin has quit IRC06:47
openstackgerritBrin Zhang proposed openstack/cinder master: api-ref: Add cipher to update an encryption type interface  https://review.openstack.org/62539106:57
*** sapd1_x has quit IRC06:57
*** sapd1_x has joined #openstack-cinder06:58
openstackgerritRajat Dhasmana proposed openstack/cinder master: Add missing backup status in API doc  https://review.openstack.org/62796407:05
*** mszwed has joined #openstack-cinder07:18
*** dpawlik has joined #openstack-cinder07:24
openstackgerritwanghao proposed openstack/cinder master: Support Incremental Backup Completion In RBD  https://review.openstack.org/62794107:24
*** dpawlik has quit IRC07:34
*** brinzhang has joined #openstack-cinder07:34
*** dpawlik has joined #openstack-cinder07:38
openstackgerritRajat Dhasmana proposed openstack/python-cinderclient master: WIP: Add volume reimage command  https://review.openstack.org/60689107:40
*** sapd1__ has quit IRC07:41
*** brinzhang has quit IRC07:57
*** brinzhang has joined #openstack-cinder07:58
*** markvoelker has joined #openstack-cinder08:08
*** markvoelker has quit IRC08:13
*** lpetrut has joined #openstack-cinder08:15
*** udesale has quit IRC08:52
*** LiangFang has quit IRC08:53
*** udesale has joined #openstack-cinder08:54
*** luizbag has joined #openstack-cinder09:15
*** lpetrut has quit IRC09:18
*** doubletao has quit IRC09:30
*** udesale has quit IRC09:39
*** luizbag has quit IRC09:39
*** udesale has joined #openstack-cinder09:40
*** ganso has joined #openstack-cinder09:45
*** erlon has joined #openstack-cinder09:52
*** e0ne has joined #openstack-cinder09:52
*** luizbag has joined #openstack-cinder09:57
*** dcdamien has joined #openstack-cinder10:01
*** markvoelker has joined #openstack-cinder10:09
*** abhishekk has quit IRC10:18
*** sapd1_x has quit IRC11:13
openstackgerritlijunjie proposed openstack/cinder master: Fix the misspelling of "except_1"  https://review.openstack.org/62840111:21
openstackgerritlijunjie proposed openstack/cinder master: Fix the misspelling of "except"  https://review.openstack.org/62840311:27
*** brinzhang has quit IRC11:28
*** noonedeadpunk has joined #openstack-cinder12:06
noonedeadpunkhi everyone.12:06
*** lseki has joined #openstack-cinder12:07
noonedeadpunkFolks, I've got some strange cinder behavior, and I even don't know where to start searches/debug. As a backend I'm using ceph (12.2.6). So, I'm creating volume from glance image (qcow/raw), then creating snapshot from it, and new volume creation from this snapshot hangs for some reason. I'd suppose, that it tries to create flatten volume from snapshot, but it's set to False in settings.12:10
noonedeadpunkAnd this happens only with images, that were uploaded to glance not via url, but from file (and stored on rbd on other pool)12:16
noonedeadpunkMy guess would be, that when disk is created from glance image, that is stored in glance rbd pool (not retrieved by http), it fails somewhere here https://github.com/openstack/cinder/blob/stable/rocky/cinder/volume/drivers/rbd.py#L539 but I'm not sure about that theory12:20
*** abishop has joined #openstack-cinder12:27
*** rosmaita has joined #openstack-cinder12:34
*** lifeless has quit IRC12:42
*** markvoelker has quit IRC13:33
*** mvkr has quit IRC13:57
*** prometheanfire has left #openstack-cinder14:00
*** mmethot has joined #openstack-cinder14:00
*** lbragstad has joined #openstack-cinder14:09
*** mvkr has joined #openstack-cinder14:12
openstackgerritYingxin Cheng proposed openstack/cinder master: [wip] Cinder RSD Driver  https://review.openstack.org/62146514:26
*** mriedem has joined #openstack-cinder14:30
*** zul has joined #openstack-cinder14:51
*** erlon has quit IRC14:57
*** enriquetaso has quit IRC14:58
*** udesale has quit IRC14:59
woojayjungleboyj: Good morning.  I added subunit reports to our CI results for LINSTOR driver (https://review.openstack.org/#/c/624233/).  The latest is at http://us.linbit.com:8080/CI-LINSTOR/33/607/testr_results.html.gz  Please let me know if anything else would be helpful.  Thank you.15:02
smcginniswoojay: Looks like it's missing all the artifacts that should be collected.15:06
smcginniswoojay: Various sservice log files, etc files, local.conf file.15:06
woojaycould you point me to the right tool to collect all the artifacts?15:08
smcginnisI guess it depends on what tools you are using to run your CI.15:09
woojayI am using basic jenkins setup w/ gerrit trigger w/ slave running devstack on ubuntu.15:10
*** carlos_silva has joined #openstack-cinder15:10
*** erlon has joined #openstack-cinder15:13
*** erlon has quit IRC15:15
gansojungleboyj, smcginnis: Hi! do you know if there is any cinder job that runs tempest test and is pure python3? or at least runs cinder using python3?15:17
*** jbernard_ is now known as jbernard15:18
smcginnisganso: Yes, there is a py3 tempest job that runs on all patches.15:18
smcginnistempest-full-py315:18
gansosmcginnis: thanks!15:19
smcginnisnp15:19
*** dpawlik has quit IRC15:25
*** lbragstad has quit IRC15:36
jungleboyjwoojay:  I think for the CI we have running it just does an SCP of the logs directory off the server before it is deleted.15:37
woojayjungleboyj: Thank you.  I am doing that for the moment as well, for lack of better way.  I am sending cinder/tempest conf, jenkins console log, stack.sh log, tempest log and subunit log of the tempest log.15:39
*** lbragstad has joined #openstack-cinder15:40
woojayI see somehow local.conf fell out.  I will add that now.  Please let me know if anything else I can add.15:41
smcginniswoojay: Looks like it is setting up the LVM driver too. I would recommend getting that out of there so it's only the Linbit backend to avoid any questions.15:42
*** mszwed has quit IRC15:43
woojaysmcginnis: Thank you.  I will do that right now.15:43
smcginniswoojay: Cool, thanks.15:44
woojayThank you for your time.  I will try to have the latest result in an hour or so.15:44
smcginniswoojay: To be clear, I don't think any of this is blocking for the driver. As long as it's something that we see is being worked on. ;)15:45
woojaysmcginnis: Thank you.  Does make me feel better.  8-)15:45
smcginnis:)15:45
jungleboyjsmcginnis:  Thanks for clarifying that.15:49
jungleboyjwoojay:  Going to review your driver here shortly.  Will let you know if I have questions.h15:50
woojayThank you so much.  I will be here.15:50
openstackgerritzhulingjie proposed openstack/cinder master: Update hacking version to latest  https://review.openstack.org/62851115:59
*** lseki has quit IRC15:59
*** luizbag has quit IRC16:06
*** dpawlik has joined #openstack-cinder16:11
*** dpawlik has quit IRC16:15
*** _alastor1 has joined #openstack-cinder16:16
*** lseki has joined #openstack-cinder16:17
*** luizbag has joined #openstack-cinder16:20
*** psachin has quit IRC16:32
*** itlinux has joined #openstack-cinder16:33
*** e0ne has quit IRC16:36
woojaysmcginnis: jungleboyj: The latest result w/ linstor driver only is at http://us.linbit.com:8080/CI-LINSTOR/33/611/ .  I had it run silent so no comment on the review page but I will start another one now.  Thank you.16:39
smcginniswoojay: Looks better. Once it includes the service logs it should be good.16:40
woojayk I will try to do that.  Thank you.16:41
openstackgerritweizj proposed openstack/os-brick master: Update hacking version to latest  https://review.openstack.org/62859616:42
woojaysmcginnis: Would journalctl logs for each service groups be ok?  Say, each logs for 'cinder' 'neutron' and so one?16:44
*** dcdamien has quit IRC16:46
smcginniswoojay: One for each service like what is seen here: http://logs.openstack.org/41/627941/4/check/tempest-full/2685ac4/controller/logs/16:47
woojayThank you.  Coming up shortly.16:48
*** zul has quit IRC17:08
openstackgerritzhulingjie proposed openstack/cinder-tempest-plugin master: Update hacking version to latest  https://review.openstack.org/62862917:08
jungleboyjsmcginnis:  I posed my two cents on Hedvig.  I missed the fact that they don't have docs.  Not going to block the driver but will remove it from Stein if they don't get that in.17:24
jungleboyjwoojay:  Did you every deprecate your old driver?  Looks like you are just removing it for the new one.17:26
woojayI have not done that.  I wasn't sure how to do that properly.17:26
*** dcdamien has joined #openstack-cinder17:27
*** whoami-rajat has quit IRC17:27
*** hedvig_01 has joined #openstack-cinder17:34
woojayjungleboyj: smcginnis suggested to have a separate patch to deprecate the old driver.  Then shall I format the linstor patch as a new driver?17:35
jungleboyjYeah, we need deprecation of the old driver for at least a release.  So, I would do a patch that marks the existing driver as deprecated.  Remove the code in this patch that removes the drbd driver.  Just make this a new driver.17:37
jungleboyjAfter Stein has closed then you can submit the patch in Train for removing the old driver.17:37
woojaySounds good.  Thank you.17:38
imacdonnis there a known issue with intel_rsd_ci posting non-standard gerrit comments ?17:41
imacdonnnot sure exactly what it's doing wrong, but its comments show up when "Toggle CI" is off17:42
*** e0ne has joined #openstack-cinder17:43
*** e0ne has quit IRC17:44
*** imacdonn has quit IRC17:59
*** imacdonn has joined #openstack-cinder18:00
openstackgerritBrian Rosmaita proposed openstack/cinder master: cinder-volume: Stop masking IOError different than ENOSPC  https://review.openstack.org/61239318:09
jungleboyjwoojay:  Review done.  Let me know if you have questions.18:18
jungleboyjThink it is pretty close though.18:18
*** dcdamien has quit IRC18:22
woojayjungleboyj: Thank you so much.  Going through smcginnis' right now.18:26
smcginnisimacdonn: Yeah, known issue and the folks in charge of their CI are out until next week I think.18:26
imacdonnsmcginnis: ok, thanks18:26
jungleboyjwoojay:  Good.  Was hoping to get mine in so you could just do one patch.18:26
smcginnisWould have their account disabled if it wasn't still quiet. Will do if they don't get it sorted early next week.18:26
jungleboyjimacdonn:  Yeah, I pinged liaxioy1 about it yesterday.  She passed it along.18:27
imacdonnroger18:27
*** luizbag has quit IRC18:30
*** itlinux_ has joined #openstack-cinder18:33
jungleboyjsmcginnis:  Want me to merge this one?18:35
jungleboyjhttps://review.openstack.org/#/c/562977/18:35
jungleboyjThere are two things that need to be fixed but I think can be handled with follow-up patches.18:35
*** itlinux has quit IRC18:35
smcginnisjungleboyj: Yeah, probably OK. Maybe mention that we need to see the follow up or we might consider reverting it?18:36
jungleboyjYep.  That was my plan.18:36
smcginnis++18:36
*** noonedeadpunk has quit IRC18:38
*** _alastor1 is now known as _alastor_18:40
jungleboyjsmcginnis:  I have mid-cycle travel approval so I am going to book that.18:40
smcginnisCool. Still waiting on mine. Guessing it will be next week.18:40
smcginnisjungleboyj: Send me your flight info and I can try to get the same ones. Unless you're going early or staying late or course.18:41
jungleboyjI am planning to get there as early Monday as I can and then leaving Saturday so I have an extra day f2f there.  We can share a car for sure.18:42
smcginnisAwesome, thanks18:42
*** whoami-rajat has joined #openstack-cinder18:57
*** dcdamien has joined #openstack-cinder18:59
openstackgerritMerged openstack/cinder master: Fix CI failed: test_get_volume_filter_options_using_config  https://review.openstack.org/62716919:03
*** dcdamien has quit IRC19:14
*** itlinux_ has quit IRC19:40
*** tpsilva has joined #openstack-cinder19:58
*** lseki has quit IRC19:58
*** tpsilva has quit IRC20:10
*** tpsilva has joined #openstack-cinder20:10
*** lifeless has joined #openstack-cinder20:12
*** tpsilva has quit IRC20:19
*** tpsilva has joined #openstack-cinder20:19
*** abishop_ has joined #openstack-cinder20:21
*** tpsilva has quit IRC20:24
*** abishop has quit IRC20:24
*** asbishop has joined #openstack-cinder20:37
*** abishop_ has quit IRC20:41
*** itlinux has joined #openstack-cinder20:43
imacdonncores, pls approve https://review.openstack.org/61239320:48
imacdonntnx, smcginnis20:53
*** asbishop has quit IRC21:11
*** carlos_silva has quit IRC21:24
*** ganso has quit IRC21:30
openstackgerritDhinesh Balasubramaniam proposed openstack/cinder master: Hedvig Cinder driver implementation  https://review.openstack.org/43334121:33
hedvig_01@jungleboyj :  Thank you for the approval. As per the comments, we have checked in the documentation file in the above patchset. https://review.openstack.org/43334121:41
jungleboyjhedvig_01:  Ah.  Thanks for adding the documentation.21:43
woojayjungleboyj: smcginnis: Thank you for the review gentlemen.  I fixed most of the upfront issues and currently running tests.21:51
jungleboyjwoojay:  Cool.  Looking forward to seeing the new patch.21:51
woojaysmcginnis: If the issue w/ the test structure isn't blocking, I will submit the patch shortly.  I may need a bit more help on that anyway.21:51
jungleboyjwoojay:  I don't think that is a blocking issue.21:52
woojayGreat.  I can certainly develp the tests in the coming days.21:52
jungleboyjYeah, no problem on that one.21:53
woojayk, definitely took out all the reference to drbd, and removed all the sleep calls.  New patch your way shortly.21:53
woojayAfter that I will add more service logs to the CI.21:54
*** itlinux_ has joined #openstack-cinder21:56
*** itlinux has quit IRC21:59
openstackgerritWoojay Poynter proposed openstack/cinder master: Add new LINSTOR volume driver  https://review.openstack.org/62423322:05
*** itlinux_ has quit IRC22:24
*** whoami-rajat has quit IRC22:37
openstackgerritWoojay Poynter proposed openstack/cinder master: Add new LINSTOR volume driver  https://review.openstack.org/62423322:47
openstackgerritDhinesh Balasubramaniam proposed openstack/cinder master: Hedvig Cinder driver implementation  https://review.openstack.org/43334122:53
hedvig_01@smcginnis:  Thank you for the review. Worked on  your comments and uploaded a new patch-set mentioned above https://review.openstack.org/433341.22:57
openstackgerritWoojay Poynter proposed openstack/cinder master: Add new LINSTOR volume driver  https://review.openstack.org/62423323:00
woojayjungleboyj: a quick question for you please.23:15
*** rosmaita has left #openstack-cinder23:31
*** openstack has joined #openstack-cinder23:47
*** ChanServ sets mode: +o openstack23:47
*** erlon has joined #openstack-cinder23:54

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