Monday, 2020-08-17

*** zhanglong has joined #openstack-cinder00:25
*** zhanglong has quit IRC00:32
*** zhanglong has joined #openstack-cinder00:42
*** brinzhang has joined #openstack-cinder00:51
*** benfelin has quit IRC00:59
*** ociuhandu has joined #openstack-cinder01:00
*** hack-char has joined #openstack-cinder01:01
*** ociuhandu has quit IRC01:05
*** whoami-rajat__ has joined #openstack-cinder02:21
*** ociuhandu has joined #openstack-cinder02:25
*** ociuhandu has quit IRC02:29
*** brinzhang_ has joined #openstack-cinder02:34
*** brinzhang has quit IRC02:37
*** dave-mccowan has quit IRC02:44
*** sapd1 has joined #openstack-cinder02:50
*** brinzhang0 has joined #openstack-cinder03:04
*** brinzhang_ has quit IRC03:07
*** psachin has joined #openstack-cinder03:37
*** m75abrams has joined #openstack-cinder04:11
*** zhanglong has quit IRC04:27
*** e0ne has joined #openstack-cinder04:32
*** e0ne has quit IRC04:37
*** e0ne has joined #openstack-cinder04:52
*** e0ne has quit IRC04:56
*** brinzhang_ has joined #openstack-cinder05:04
*** brinzhang0 has quit IRC05:06
*** sapd1 has quit IRC05:11
*** e0ne has joined #openstack-cinder05:11
*** e0ne has quit IRC05:16
*** abdysn has joined #openstack-cinder05:26
*** zhanglong has joined #openstack-cinder05:28
*** e0ne has joined #openstack-cinder05:28
*** e0ne has quit IRC05:33
*** udesale has joined #openstack-cinder05:43
*** brinzhang0 has joined #openstack-cinder05:54
*** brinzhang_ has quit IRC05:57
*** udesale has quit IRC06:10
*** sapd1 has joined #openstack-cinder06:10
openstackgerritAjitha Robert proposed openstack/cinder master: Add multiattach in Nimble driver  https://review.opendev.org/74012506:11
*** sapd1 has joined #openstack-cinder06:13
*** zhanglong has quit IRC06:37
*** zhanglong has joined #openstack-cinder06:40
*** vishalmanchanda has joined #openstack-cinder06:49
*** e0ne has joined #openstack-cinder07:14
*** zhanglong has quit IRC07:28
*** zhanglong has joined #openstack-cinder07:30
*** benfelin has joined #openstack-cinder07:39
*** tosky has joined #openstack-cinder07:55
*** zhanglong has quit IRC08:00
*** brinzhang_ has joined #openstack-cinder08:04
*** brinzhang0 has quit IRC08:07
*** zhanglong has joined #openstack-cinder08:13
*** dsariel has joined #openstack-cinder08:14
*** dsariel is now known as dsariel|PTO08:14
*** martinkennelly has joined #openstack-cinder08:18
*** pcaruana has joined #openstack-cinder08:22
*** Lucas_Gray has joined #openstack-cinder08:30
*** zhanglong has quit IRC08:52
*** zhanglong has joined #openstack-cinder08:54
*** arne_wiebalck has quit IRC09:11
*** simondodsley has quit IRC09:11
*** rajinir has quit IRC09:12
*** arne_wiebalck has joined #openstack-cinder09:14
*** simondodsley has joined #openstack-cinder09:17
*** rajinir has joined #openstack-cinder09:19
*** ociuhandu has joined #openstack-cinder09:44
*** n-saito has quit IRC09:51
*** tkajinam has quit IRC09:55
*** Luzi has joined #openstack-cinder09:59
*** jopdorp has joined #openstack-cinder10:00
*** brinzhang_ has quit IRC10:04
*** brinzhang_ has joined #openstack-cinder10:04
jopdorphi10:05
jopdorpI've implemented a cinder backup s3 driver10:05
jopdorpworking on the unit tests for it now10:05
jopdorpanyone interested10:05
*** brinzhang_ has quit IRC10:06
jopdorp?10:06
*** brinzhang_ has joined #openstack-cinder10:06
e0nejopdorp: hi. it will be awesome!10:10
e0nejopdorp: it's something I was going to to for a while10:10
jopdorpI can just submit it with git-review right?10:11
e0nejopdorp: yes10:11
jopdorpI'm writing it on the cinder master branch10:11
jopdorpokay10:11
jopdorpe0ne, are you a core review?10:11
e0nejopdorp: did you test it with RadosGW?10:11
jopdorpreviewer10:11
jopdorpI haven't tested it yet10:11
jopdorpwrote it on friday10:11
e0nejopdorp: at lease, I was last week ;)10:11
jopdorphaha good10:11
jopdorpI based the driver on the swift driver10:12
e0nejopdorp: I don't have s3 account now, but I can't test it with Ceph/RadosGW10:12
jopdorpright10:12
jopdorpI do have an s3 account10:12
jopdorpbut the truth is I want to use it with backblaze their s3 compatible api10:12
e0neit's absolutely reasonable use0case10:13
jopdorpI want to make a unit test based on "test_backup_swift.py"10:13
*** pcaruana has quit IRC10:15
*** pcaruana has joined #openstack-cinder10:19
*** jopdorp has quit IRC10:27
*** ociuhandu has quit IRC10:37
*** priteau has joined #openstack-cinder10:40
*** jopdorp has joined #openstack-cinder10:41
*** jopdorp has quit IRC10:49
*** jopdorp has joined #openstack-cinder10:50
*** sapd1 has quit IRC10:52
*** udesale has joined #openstack-cinder11:20
*** breizhkoala has joined #openstack-cinder11:26
*** tkajinam has joined #openstack-cinder11:27
*** brinzhang0 has joined #openstack-cinder11:34
*** brinzhang_ has quit IRC11:37
*** dave-mccowan has joined #openstack-cinder11:39
openstackgerritSimon O'Donovan proposed openstack/cinder master: PowerMax Driver - Remove mandatory failover BID  https://review.opendev.org/74648611:45
*** brinzhang_ has joined #openstack-cinder12:14
*** udesale_ has joined #openstack-cinder12:14
*** udesale has quit IRC12:17
*** brinzhang0 has quit IRC12:17
*** andrebeltrami has joined #openstack-cinder12:19
*** rosmaita has joined #openstack-cinder12:26
*** zhanglong has quit IRC12:29
*** rishabhhpe has joined #openstack-cinder12:30
*** dave-mccowan has quit IRC12:32
*** ociuhandu has joined #openstack-cinder12:37
*** dave-mccowan has joined #openstack-cinder12:39
*** ociuhandu has quit IRC12:42
*** rishabhhpe has quit IRC12:52
*** rishabhhpe has joined #openstack-cinder12:53
openstackgerritGirish Chilukuri proposed openstack/cinder master: [Storwize]:check_vdisk_fc_mappings method is not  deleting the clone fc maps.  https://review.opendev.org/74466413:01
rishabhhpeHi All, Please help me with the error pasted here -: http://paste.openstack.org/show/796886/13:05
rishabhhpemost of my TC are failing with above error.13:05
toskyI don't know the reason, but if tests consistently fail with that error, there may be an issue with cinder server13:08
toskythe logs are as usual the starting point for troubleshooting13:08
toskythat log just says that the server returned an error13:09
*** udesale_ has quit IRC13:09
rishabhhpetosky: can u please check this log13:17
rishabhhpegot this error in cinder service13:17
rishabhhpehttp://paste.openstack.org/show/796887/13:23
toskyand since when it started failing?13:26
toskyare there any other errors or warnings?13:26
*** Luzi has quit IRC13:26
rishabhhpeFrom yesterday13:28
toskyuhm, there haven't been relevant changes in cinder, os-brick or devstack yesterday13:29
rishabhhpetosky: anything before yesterday ?13:30
rishabhhpeafter running cinder-manage db sync command it is running fine ... but i installed fresh devstack and it should have all the services up to date.. i am not sure why this error was coming cinder.default volume type does not exist13:32
*** sfernand has joined #openstack-cinder13:32
*** brinzhang0 has joined #openstack-cinder13:33
toskyI don't see anything relevant to default volume types since July 1st13:34
toskyI would recheck the previous commands in devstack and compare with the devstack logs from devstack-based jobs on the main gates13:35
*** brinzhang0 has quit IRC13:35
toskyand see if something is different13:35
*** brinzhang0 has joined #openstack-cinder13:35
*** brinzhang_ has quit IRC13:36
*** ociuhandu has joined #openstack-cinder13:37
rosmaitarishabhhpe: that table is added by a patch that hasn't merged yet: https://review.opendev.org/#/c/737707/13:47
openstackgerritGirish Chilukuri proposed openstack/cinder master: [Storwize]:check_vdisk_fc_mappings method is not deleting the clone fc maps.  https://review.opendev.org/74466413:51
smcginnisrosmaita: Quick one? https://review.opendev.org/74631913:53
smcginnisrosmaita: Quick two? :) https://review.opendev.org/#/c/745276/13:53
rishabhhperosmaita : so when can we expect ? otherwise our CI will fail13:54
rosmaitaok ... got a non-quick one for you when you have some time: https://review.opendev.org/#/c/741498/13:54
smcginnisrosmaita: Will do!13:55
rosmaitarishabhhpe: not sure what your question is -- if you are failing on a un-merged patch, then maybe that is correct behavior if there is a problem with the patch?13:55
smcginnisrishabhhpe: If you are failing on that, that actually looks like you may have a CI setup issue. ARe you reusing your environment (venv, database, etc.) between test runs?13:56
rosmaitasmcginnis: thanks for https://review.opendev.org/746319 !!!13:57
smcginnis;)13:58
*** jv_ has joined #openstack-cinder14:03
rosmaitasmcginnis: doesn't https://review.opendev.org/746319 make https://review.opendev.org/#/c/745276/ unnecessary?14:06
rosmaitaor i may just need more coffee14:06
*** tkajinam has quit IRC14:06
*** rishabhhpe has quit IRC14:12
*** rishabhhpe has joined #openstack-cinder14:13
*** abdysn has quit IRC14:14
openstackgerritGhanshyam Mann proposed openstack/python-cinderclient master: [goal] Migrate python-cinderclient jobs to focal  https://review.opendev.org/74308214:14
smcginnisrosmaita: Oh, yep. I just do the same things over and over again and don't remember I've done them before. :)14:16
smcginnisI will abandon the single note typo patch.14:16
rosmaitasounds good, thanks14:16
openstackgerritGirish Chilukuri proposed openstack/cinder master: [Storwize]:clone fcmap is not deleting in cleanup  https://review.opendev.org/74466414:23
*** brinzhang has joined #openstack-cinder14:25
*** ircuser-1 has joined #openstack-cinder14:26
*** brinzhang0 has quit IRC14:27
*** brinzhang_ has joined #openstack-cinder14:34
*** brinzhang has quit IRC14:37
*** psachin has quit IRC14:45
*** m75abrams has quit IRC14:50
*** ajitha has joined #openstack-cinder14:56
openstackgerritLuigi Toscano proposed openstack/cinder stable/ussuri: Brocade: Python 3 support  https://review.opendev.org/74653415:01
openstackgerritLuigi Toscano proposed openstack/cinder stable/ussuri: Brocade: Fix AttributeError when raising exception  https://review.opendev.org/74653515:01
openstackgerritLuigi Toscano proposed openstack/cinder stable/ussuri: Brocade: Fix lookup UnboundLocalError  https://review.opendev.org/74653615:01
*** brinzhang0 has joined #openstack-cinder15:04
openstackgerritLuigi Toscano proposed openstack/os-brick stable/ussuri: Improve WWN detection  https://review.opendev.org/74653915:05
*** brinzhang_ has quit IRC15:06
*** sapd1 has joined #openstack-cinder15:08
openstackgerritSimon O'Donovan proposed openstack/cinder master: PowerMax Driver - Prevent unmanage with snapvx  https://review.opendev.org/74654215:09
*** ajitha has quit IRC15:10
openstackgerritMerged openstack/cinder master: Normalize release note bug links  https://review.opendev.org/74631915:30
*** martinkennelly has quit IRC15:41
*** rishabhhpe has quit IRC15:43
*** rishabhhpe has joined #openstack-cinder15:43
*** dsariel|PTO has quit IRC15:44
openstackgerritBrian Rosmaita proposed openstack/os-brick master: Add release note for scaleio connector upgrade  https://review.opendev.org/74655115:46
rosmaitasmcginnis: jungleboyj: ^^ lmk if you think this is necessary ^^15:46
smcginnisrosmaita: I was going to say we don't need it since we just added that change, but we did actually release stable branches with it already, didn't we?15:47
rosmaitayes15:47
rosmaitaand i think we had a M-2 release of brick from master15:48
jungleboyjLooking.15:48
*** rishabhhpe has quit IRC15:49
jungleboyjI think having release notes is good personally.  Better to have more than needed.15:50
smcginnis++15:50
rosmaitaok, thanks15:50
rosmaitaso looks like another festival o' backports & releases for os-brick this week15:51
jungleboyjrosmaita:  Ok, let me know and I will help review.15:51
rosmaitathanks15:51
*** priteau has quit IRC15:57
*** martinkennelly has joined #openstack-cinder16:17
openstackgerritMerged openstack/os-brick master: ScaleIO: Connection info backward compatibility  https://review.opendev.org/74610916:20
openstackgerritRajat Dhasmana proposed openstack/cinder master: Modify default/delete volume type logic  https://review.opendev.org/74149816:26
whoami-rajat__rosmaita: smcginnis ^^ updated16:27
openstackgerritJegor van Opdorp proposed openstack/cinder master: created s3 cinder backup driver  https://review.opendev.org/74656116:29
jopdorpthat's the s3 cinder backup driver16:30
jopdorpI only tested with unit tests right now16:31
jopdorpwill test with backblaze s3 compatible storage and amazon s3 backends on wednesday16:31
jopdorpe0ne this is my first addition to the cinder project, please let me know if I'm not using any conventions16:32
rosmaitawhoami-rajat__: i'm not convinced that removing 'required' is a good idea, i will have to think about it16:34
toskyjopdorp: is there a way to have CI for that? Maybe using ceph and/or swift/s3?16:35
smcginnisrosmaita: Why would we need required if we provide a default value?16:35
toskyjopdorp: (and welcome and thanks for the work!)16:35
jopdorpyes, we could connect to the swift s3 compatible api with a ci16:36
rosmaitasmcginnis: i don't know ... just superstitious, i guess16:36
jopdorptosky now i think the ci would run the unit tests16:36
smcginnisjopdorp: We should also have a release note on there announcing the new feature. That's a matter of running the command listed here: https://docs.openstack.org/reno/latest/user/usage.html16:37
smcginnisAnd editing the file to just have an item under the "features" section describing the new functionality.16:37
jopdorpsmcginnis, will do! I need to go now though and have a free day tomorrow (my son's birthday) so I'16:37
jopdorpi'll do it on wednesday16:37
smcginnisjopdorp: Enjoy!16:37
jopdorptosky: where would the config for integration tests through the ci be?16:38
toskyjopdorp: the CI runs the unit tests, but it would be nice to have higher level tests too16:38
*** brinzhang_ has joined #openstack-cinder16:38
toskyjopdorp: the baseline is the execution of the existing cinder-backup tempest tests in this specific scenario16:38
jopdorpokay16:39
jopdorptosky: yes, I could have a look at adding tests for this driver to tempest.16:39
jopdorptosky: would that not depend on the configuration of the environment with which the ci runs the tests?16:40
toskyjopdorp: the idea is to add another job, or to change the configuration of an existing job, so that it deploys also an s3 service and executes the backup there16:40
jopdorpI'm not too familiar with the ci16:40
toskythe CI runs several jobs for each change16:40
toskysome jobs only run unit tests16:40
toskyother jobs run higher level integration tests16:40
jopdorpwhere are the ci config files?16:40
jopdorprally-jobs/cinder.yaml?16:41
*** brinzhang0 has quit IRC16:42
jopdorpdoesn't look like there are any backup tests there yet16:42
toskyjopdorp: this is a good starting point: https://docs.openstack.org/project-team-guide/testing.html16:43
toskylook for .zuul.yaml or zuul* directories16:43
smcginnisIt would probably be a matter of running the existing base tempest test with the addition of the configuration to set up this driver as the backup driver to use for the backup scenario tests.16:44
smcginnisI haven't looked what is all in there already, but I know there are at least a handful.16:44
openstackgerritMerged openstack/os-brick master: Leverage the iSCSI mpath to get the WWN  https://review.opendev.org/73242416:45
smcginnisHuh, or one. https://opendev.org/openstack/tempest/src/branch/master/tempest/scenario/test_volume_backup_restore.py16:46
smcginnisAnd https://opendev.org/openstack/tempest/src/branch/master/tempest/api/volume/test_volumes_backup.py16:46
jopdorpI'm still not clear on how to configure the ci to point to an s3 compatible api and make it run opendev.org/openstack/tempest/src/branch/master/tempest/scenario/test_volume_backup_restore.py with the s3 cinder backup driver16:50
*** jopdorp has quit IRC16:52
*** Lucas_Gray has quit IRC16:52
*** sapd1 has quit IRC16:56
openstackgerritRajini Karthik proposed openstack/os-brick stable/ussuri: ScaleIO: Connection info backward compatibility  https://review.opendev.org/74657217:20
*** josephillips has joined #openstack-cinder17:23
openstackgerritMerged openstack/os-brick master: Add release note for scaleio connector upgrade  https://review.opendev.org/74655117:25
*** brinzhang0 has joined #openstack-cinder17:37
*** vishalmanchanda has quit IRC17:37
whoami-rajat__Hi, everyone, would be great if i could get some reviews on these stable backports https://review.opendev.org/#/c/742581/ and https://review.opendev.org/#/c/746041/17:38
*** brinzhang_ has quit IRC17:40
openstackgerritRajini Karthik proposed openstack/os-brick stable/ussuri: Add release note for scaleio connector upgrade  https://review.opendev.org/74657617:55
*** jopdorp has joined #openstack-cinder18:26
jopdorpIs this unregistered or openstack-cinder?18:27
josephillipsjopdorp openstack-cinder18:28
josephillipslol18:28
jopdorpHaha okay, I just downloaded an app fory mobile phone, it wasn't so clear18:28
jopdorpIs there a cinder meeting on Tuesday?18:29
*** hoonetorg has joined #openstack-cinder18:29
openstackgerritkatari manoj kumar proposed openstack/cinder master: [storwize]: optimize create_flashcopy_to_consistgrp  https://review.opendev.org/74571918:29
rosmaitajopdorp: wednesday at 1400 utc18:29
jopdorprosmaita: is that on any video conferencing platform?18:30
rosmaitajopdorp: on IRC in #openstack-meeting-alt18:30
smcginnishttps://wiki.openstack.org/wiki/CinderMeetings18:30
*** breizhkoala has quit IRC18:36
*** e0ne has quit IRC18:48
*** e0ne has joined #openstack-cinder19:03
*** e0ne has quit IRC19:03
*** smcginnis has quit IRC19:19
*** smcginnis has joined #openstack-cinder19:21
*** ChanServ sets mode: +o smcginnis19:21
*** e0ne has joined #openstack-cinder19:29
*** martinkennelly has quit IRC19:31
*** martinkennelly has joined #openstack-cinder19:31
openstackgerritMerged openstack/cinder stable/stein: RBD: Cleanup temporary file during exception  https://review.opendev.org/74604119:40
openstackgerritMerged openstack/os-brick stable/ussuri: ScaleIO: Connection info backward compatibility  https://review.opendev.org/74657221:12
openstackgerritMerged openstack/os-brick stable/ussuri: Add release note for scaleio connector upgrade  https://review.opendev.org/74657621:12
*** e0ne has quit IRC21:14
whoami-rajat__smcginnis: rosmaita Hi, it doesn't make much difference in the bug fix if we keep the required field or not, i don't mind keeping or removing it but a general agreement would be great21:30
*** e0ne has joined #openstack-cinder21:31
*** e0ne has quit IRC21:37
*** ociuhandu has quit IRC21:39
*** brinzhang_ has joined #openstack-cinder21:44
*** brinzhang0 has quit IRC21:47
*** benfelin has quit IRC21:58
*** rcernin has joined #openstack-cinder22:33
*** Lucas_Gray has joined #openstack-cinder22:41
*** tosky has quit IRC22:57
*** martinkennelly has quit IRC23:00
*** breizhkoala has joined #openstack-cinder23:03
*** Lucas_Gray has quit IRC23:04
*** tkajinam has joined #openstack-cinder23:07
*** brinzhang0 has joined #openstack-cinder23:09
*** brinzhang_ has quit IRC23:12
*** ociuhandu has joined #openstack-cinder23:39
*** ociuhandu has quit IRC23:45

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!