*** zhanglong has joined #openstack-cinder | 00:25 | |
*** zhanglong has quit IRC | 00:32 | |
*** zhanglong has joined #openstack-cinder | 00:42 | |
*** brinzhang has joined #openstack-cinder | 00:51 | |
*** benfelin has quit IRC | 00:59 | |
*** ociuhandu has joined #openstack-cinder | 01:00 | |
*** hack-char has joined #openstack-cinder | 01:01 | |
*** ociuhandu has quit IRC | 01:05 | |
*** whoami-rajat__ has joined #openstack-cinder | 02:21 | |
*** ociuhandu has joined #openstack-cinder | 02:25 | |
*** ociuhandu has quit IRC | 02:29 | |
*** brinzhang_ has joined #openstack-cinder | 02:34 | |
*** brinzhang has quit IRC | 02:37 | |
*** dave-mccowan has quit IRC | 02:44 | |
*** sapd1 has joined #openstack-cinder | 02:50 | |
*** brinzhang0 has joined #openstack-cinder | 03:04 | |
*** brinzhang_ has quit IRC | 03:07 | |
*** psachin has joined #openstack-cinder | 03:37 | |
*** m75abrams has joined #openstack-cinder | 04:11 | |
*** zhanglong has quit IRC | 04:27 | |
*** e0ne has joined #openstack-cinder | 04:32 | |
*** e0ne has quit IRC | 04:37 | |
*** e0ne has joined #openstack-cinder | 04:52 | |
*** e0ne has quit IRC | 04:56 | |
*** brinzhang_ has joined #openstack-cinder | 05:04 | |
*** brinzhang0 has quit IRC | 05:06 | |
*** sapd1 has quit IRC | 05:11 | |
*** e0ne has joined #openstack-cinder | 05:11 | |
*** e0ne has quit IRC | 05:16 | |
*** abdysn has joined #openstack-cinder | 05:26 | |
*** zhanglong has joined #openstack-cinder | 05:28 | |
*** e0ne has joined #openstack-cinder | 05:28 | |
*** e0ne has quit IRC | 05:33 | |
*** udesale has joined #openstack-cinder | 05:43 | |
*** brinzhang0 has joined #openstack-cinder | 05:54 | |
*** brinzhang_ has quit IRC | 05:57 | |
*** udesale has quit IRC | 06:10 | |
*** sapd1 has joined #openstack-cinder | 06:10 | |
openstackgerrit | Ajitha Robert proposed openstack/cinder master: Add multiattach in Nimble driver https://review.opendev.org/740125 | 06:11 |
---|---|---|
*** sapd1 has joined #openstack-cinder | 06:13 | |
*** zhanglong has quit IRC | 06:37 | |
*** zhanglong has joined #openstack-cinder | 06:40 | |
*** vishalmanchanda has joined #openstack-cinder | 06:49 | |
*** e0ne has joined #openstack-cinder | 07:14 | |
*** zhanglong has quit IRC | 07:28 | |
*** zhanglong has joined #openstack-cinder | 07:30 | |
*** benfelin has joined #openstack-cinder | 07:39 | |
*** tosky has joined #openstack-cinder | 07:55 | |
*** zhanglong has quit IRC | 08:00 | |
*** brinzhang_ has joined #openstack-cinder | 08:04 | |
*** brinzhang0 has quit IRC | 08:07 | |
*** zhanglong has joined #openstack-cinder | 08:13 | |
*** dsariel has joined #openstack-cinder | 08:14 | |
*** dsariel is now known as dsariel|PTO | 08:14 | |
*** martinkennelly has joined #openstack-cinder | 08:18 | |
*** pcaruana has joined #openstack-cinder | 08:22 | |
*** Lucas_Gray has joined #openstack-cinder | 08:30 | |
*** zhanglong has quit IRC | 08:52 | |
*** zhanglong has joined #openstack-cinder | 08:54 | |
*** arne_wiebalck has quit IRC | 09:11 | |
*** simondodsley has quit IRC | 09:11 | |
*** rajinir has quit IRC | 09:12 | |
*** arne_wiebalck has joined #openstack-cinder | 09:14 | |
*** simondodsley has joined #openstack-cinder | 09:17 | |
*** rajinir has joined #openstack-cinder | 09:19 | |
*** ociuhandu has joined #openstack-cinder | 09:44 | |
*** n-saito has quit IRC | 09:51 | |
*** tkajinam has quit IRC | 09:55 | |
*** Luzi has joined #openstack-cinder | 09:59 | |
*** jopdorp has joined #openstack-cinder | 10:00 | |
*** brinzhang_ has quit IRC | 10:04 | |
*** brinzhang_ has joined #openstack-cinder | 10:04 | |
jopdorp | hi | 10:05 |
jopdorp | I've implemented a cinder backup s3 driver | 10:05 |
jopdorp | working on the unit tests for it now | 10:05 |
jopdorp | anyone interested | 10:05 |
*** brinzhang_ has quit IRC | 10:06 | |
jopdorp | ? | 10:06 |
*** brinzhang_ has joined #openstack-cinder | 10:06 | |
e0ne | jopdorp: hi. it will be awesome! | 10:10 |
e0ne | jopdorp: it's something I was going to to for a while | 10:10 |
jopdorp | I can just submit it with git-review right? | 10:11 |
e0ne | jopdorp: yes | 10:11 |
jopdorp | I'm writing it on the cinder master branch | 10:11 |
jopdorp | okay | 10:11 |
jopdorp | e0ne, are you a core review? | 10:11 |
e0ne | jopdorp: did you test it with RadosGW? | 10:11 |
jopdorp | reviewer | 10:11 |
jopdorp | I haven't tested it yet | 10:11 |
jopdorp | wrote it on friday | 10:11 |
e0ne | jopdorp: at lease, I was last week ;) | 10:11 |
jopdorp | haha good | 10:11 |
jopdorp | I based the driver on the swift driver | 10:12 |
e0ne | jopdorp: I don't have s3 account now, but I can't test it with Ceph/RadosGW | 10:12 |
jopdorp | right | 10:12 |
jopdorp | I do have an s3 account | 10:12 |
jopdorp | but the truth is I want to use it with backblaze their s3 compatible api | 10:12 |
e0ne | it's absolutely reasonable use0case | 10:13 |
jopdorp | I want to make a unit test based on "test_backup_swift.py" | 10:13 |
*** pcaruana has quit IRC | 10:15 | |
*** pcaruana has joined #openstack-cinder | 10:19 | |
*** jopdorp has quit IRC | 10:27 | |
*** ociuhandu has quit IRC | 10:37 | |
*** priteau has joined #openstack-cinder | 10:40 | |
*** jopdorp has joined #openstack-cinder | 10:41 | |
*** jopdorp has quit IRC | 10:49 | |
*** jopdorp has joined #openstack-cinder | 10:50 | |
*** sapd1 has quit IRC | 10:52 | |
*** udesale has joined #openstack-cinder | 11:20 | |
*** breizhkoala has joined #openstack-cinder | 11:26 | |
*** tkajinam has joined #openstack-cinder | 11:27 | |
*** brinzhang0 has joined #openstack-cinder | 11:34 | |
*** brinzhang_ has quit IRC | 11:37 | |
*** dave-mccowan has joined #openstack-cinder | 11:39 | |
openstackgerrit | Simon O'Donovan proposed openstack/cinder master: PowerMax Driver - Remove mandatory failover BID https://review.opendev.org/746486 | 11:45 |
*** brinzhang_ has joined #openstack-cinder | 12:14 | |
*** udesale_ has joined #openstack-cinder | 12:14 | |
*** udesale has quit IRC | 12:17 | |
*** brinzhang0 has quit IRC | 12:17 | |
*** andrebeltrami has joined #openstack-cinder | 12:19 | |
*** rosmaita has joined #openstack-cinder | 12:26 | |
*** zhanglong has quit IRC | 12:29 | |
*** rishabhhpe has joined #openstack-cinder | 12:30 | |
*** dave-mccowan has quit IRC | 12:32 | |
*** ociuhandu has joined #openstack-cinder | 12:37 | |
*** dave-mccowan has joined #openstack-cinder | 12:39 | |
*** ociuhandu has quit IRC | 12:42 | |
*** rishabhhpe has quit IRC | 12:52 | |
*** rishabhhpe has joined #openstack-cinder | 12:53 | |
openstackgerrit | Girish Chilukuri proposed openstack/cinder master: [Storwize]:check_vdisk_fc_mappings method is not deleting the clone fc maps. https://review.opendev.org/744664 | 13:01 |
rishabhhpe | Hi All, Please help me with the error pasted here -: http://paste.openstack.org/show/796886/ | 13:05 |
rishabhhpe | most of my TC are failing with above error. | 13:05 |
tosky | I don't know the reason, but if tests consistently fail with that error, there may be an issue with cinder server | 13:08 |
tosky | the logs are as usual the starting point for troubleshooting | 13:08 |
tosky | that log just says that the server returned an error | 13:09 |
*** udesale_ has quit IRC | 13:09 | |
rishabhhpe | tosky: can u please check this log | 13:17 |
rishabhhpe | got this error in cinder service | 13:17 |
rishabhhpe | http://paste.openstack.org/show/796887/ | 13:23 |
tosky | and since when it started failing? | 13:26 |
tosky | are there any other errors or warnings? | 13:26 |
*** Luzi has quit IRC | 13:26 | |
rishabhhpe | From yesterday | 13:28 |
tosky | uhm, there haven't been relevant changes in cinder, os-brick or devstack yesterday | 13:29 |
rishabhhpe | tosky: anything before yesterday ? | 13:30 |
rishabhhpe | after 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 exist | 13:32 |
*** sfernand has joined #openstack-cinder | 13:32 | |
*** brinzhang0 has joined #openstack-cinder | 13:33 | |
tosky | I don't see anything relevant to default volume types since July 1st | 13:34 |
tosky | I would recheck the previous commands in devstack and compare with the devstack logs from devstack-based jobs on the main gates | 13:35 |
*** brinzhang0 has quit IRC | 13:35 | |
tosky | and see if something is different | 13:35 |
*** brinzhang0 has joined #openstack-cinder | 13:35 | |
*** brinzhang_ has quit IRC | 13:36 | |
*** ociuhandu has joined #openstack-cinder | 13:37 | |
rosmaita | rishabhhpe: that table is added by a patch that hasn't merged yet: https://review.opendev.org/#/c/737707/ | 13:47 |
openstackgerrit | Girish Chilukuri proposed openstack/cinder master: [Storwize]:check_vdisk_fc_mappings method is not deleting the clone fc maps. https://review.opendev.org/744664 | 13:51 |
smcginnis | rosmaita: Quick one? https://review.opendev.org/746319 | 13:53 |
smcginnis | rosmaita: Quick two? :) https://review.opendev.org/#/c/745276/ | 13:53 |
rishabhhpe | rosmaita : so when can we expect ? otherwise our CI will fail | 13:54 |
rosmaita | ok ... got a non-quick one for you when you have some time: https://review.opendev.org/#/c/741498/ | 13:54 |
smcginnis | rosmaita: Will do! | 13:55 |
rosmaita | rishabhhpe: 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 |
smcginnis | rishabhhpe: 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 |
rosmaita | smcginnis: thanks for https://review.opendev.org/746319 !!! | 13:57 |
smcginnis | ;) | 13:58 |
*** jv_ has joined #openstack-cinder | 14:03 | |
rosmaita | smcginnis: doesn't https://review.opendev.org/746319 make https://review.opendev.org/#/c/745276/ unnecessary? | 14:06 |
rosmaita | or i may just need more coffee | 14:06 |
*** tkajinam has quit IRC | 14:06 | |
*** rishabhhpe has quit IRC | 14:12 | |
*** rishabhhpe has joined #openstack-cinder | 14:13 | |
*** abdysn has quit IRC | 14:14 | |
openstackgerrit | Ghanshyam Mann proposed openstack/python-cinderclient master: [goal] Migrate python-cinderclient jobs to focal https://review.opendev.org/743082 | 14:14 |
smcginnis | rosmaita: Oh, yep. I just do the same things over and over again and don't remember I've done them before. :) | 14:16 |
smcginnis | I will abandon the single note typo patch. | 14:16 |
rosmaita | sounds good, thanks | 14:16 |
openstackgerrit | Girish Chilukuri proposed openstack/cinder master: [Storwize]:clone fcmap is not deleting in cleanup https://review.opendev.org/744664 | 14:23 |
*** brinzhang has joined #openstack-cinder | 14:25 | |
*** ircuser-1 has joined #openstack-cinder | 14:26 | |
*** brinzhang0 has quit IRC | 14:27 | |
*** brinzhang_ has joined #openstack-cinder | 14:34 | |
*** brinzhang has quit IRC | 14:37 | |
*** psachin has quit IRC | 14:45 | |
*** m75abrams has quit IRC | 14:50 | |
*** ajitha has joined #openstack-cinder | 14:56 | |
openstackgerrit | Luigi Toscano proposed openstack/cinder stable/ussuri: Brocade: Python 3 support https://review.opendev.org/746534 | 15:01 |
openstackgerrit | Luigi Toscano proposed openstack/cinder stable/ussuri: Brocade: Fix AttributeError when raising exception https://review.opendev.org/746535 | 15:01 |
openstackgerrit | Luigi Toscano proposed openstack/cinder stable/ussuri: Brocade: Fix lookup UnboundLocalError https://review.opendev.org/746536 | 15:01 |
*** brinzhang0 has joined #openstack-cinder | 15:04 | |
openstackgerrit | Luigi Toscano proposed openstack/os-brick stable/ussuri: Improve WWN detection https://review.opendev.org/746539 | 15:05 |
*** brinzhang_ has quit IRC | 15:06 | |
*** sapd1 has joined #openstack-cinder | 15:08 | |
openstackgerrit | Simon O'Donovan proposed openstack/cinder master: PowerMax Driver - Prevent unmanage with snapvx https://review.opendev.org/746542 | 15:09 |
*** ajitha has quit IRC | 15:10 | |
openstackgerrit | Merged openstack/cinder master: Normalize release note bug links https://review.opendev.org/746319 | 15:30 |
*** martinkennelly has quit IRC | 15:41 | |
*** rishabhhpe has quit IRC | 15:43 | |
*** rishabhhpe has joined #openstack-cinder | 15:43 | |
*** dsariel|PTO has quit IRC | 15:44 | |
openstackgerrit | Brian Rosmaita proposed openstack/os-brick master: Add release note for scaleio connector upgrade https://review.opendev.org/746551 | 15:46 |
rosmaita | smcginnis: jungleboyj: ^^ lmk if you think this is necessary ^^ | 15:46 |
smcginnis | rosmaita: 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 |
rosmaita | yes | 15:47 |
rosmaita | and i think we had a M-2 release of brick from master | 15:48 |
jungleboyj | Looking. | 15:48 |
*** rishabhhpe has quit IRC | 15:49 | |
jungleboyj | I think having release notes is good personally. Better to have more than needed. | 15:50 |
smcginnis | ++ | 15:50 |
rosmaita | ok, thanks | 15:50 |
rosmaita | so looks like another festival o' backports & releases for os-brick this week | 15:51 |
jungleboyj | rosmaita: Ok, let me know and I will help review. | 15:51 |
rosmaita | thanks | 15:51 |
*** priteau has quit IRC | 15:57 | |
*** martinkennelly has joined #openstack-cinder | 16:17 | |
openstackgerrit | Merged openstack/os-brick master: ScaleIO: Connection info backward compatibility https://review.opendev.org/746109 | 16:20 |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Modify default/delete volume type logic https://review.opendev.org/741498 | 16:26 |
whoami-rajat__ | rosmaita: smcginnis ^^ updated | 16:27 |
openstackgerrit | Jegor van Opdorp proposed openstack/cinder master: created s3 cinder backup driver https://review.opendev.org/746561 | 16:29 |
jopdorp | that's the s3 cinder backup driver | 16:30 |
jopdorp | I only tested with unit tests right now | 16:31 |
jopdorp | will test with backblaze s3 compatible storage and amazon s3 backends on wednesday | 16:31 |
jopdorp | e0ne this is my first addition to the cinder project, please let me know if I'm not using any conventions | 16:32 |
rosmaita | whoami-rajat__: i'm not convinced that removing 'required' is a good idea, i will have to think about it | 16:34 |
tosky | jopdorp: is there a way to have CI for that? Maybe using ceph and/or swift/s3? | 16:35 |
smcginnis | rosmaita: Why would we need required if we provide a default value? | 16:35 |
tosky | jopdorp: (and welcome and thanks for the work!) | 16:35 |
jopdorp | yes, we could connect to the swift s3 compatible api with a ci | 16:36 |
rosmaita | smcginnis: i don't know ... just superstitious, i guess | 16:36 |
jopdorp | tosky now i think the ci would run the unit tests | 16:36 |
smcginnis | jopdorp: 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.html | 16:37 |
smcginnis | And editing the file to just have an item under the "features" section describing the new functionality. | 16:37 |
jopdorp | smcginnis, will do! I need to go now though and have a free day tomorrow (my son's birthday) so I' | 16:37 |
jopdorp | i'll do it on wednesday | 16:37 |
smcginnis | jopdorp: Enjoy! | 16:37 |
jopdorp | tosky: where would the config for integration tests through the ci be? | 16:38 |
tosky | jopdorp: the CI runs the unit tests, but it would be nice to have higher level tests too | 16:38 |
*** brinzhang_ has joined #openstack-cinder | 16:38 | |
tosky | jopdorp: the baseline is the execution of the existing cinder-backup tempest tests in this specific scenario | 16:38 |
jopdorp | okay | 16:39 |
jopdorp | tosky: yes, I could have a look at adding tests for this driver to tempest. | 16:39 |
jopdorp | tosky: would that not depend on the configuration of the environment with which the ci runs the tests? | 16:40 |
tosky | jopdorp: 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 there | 16:40 |
jopdorp | I'm not too familiar with the ci | 16:40 |
tosky | the CI runs several jobs for each change | 16:40 |
tosky | some jobs only run unit tests | 16:40 |
tosky | other jobs run higher level integration tests | 16:40 |
jopdorp | where are the ci config files? | 16:40 |
jopdorp | rally-jobs/cinder.yaml? | 16:41 |
*** brinzhang0 has quit IRC | 16:42 | |
jopdorp | doesn't look like there are any backup tests there yet | 16:42 |
tosky | jopdorp: this is a good starting point: https://docs.openstack.org/project-team-guide/testing.html | 16:43 |
tosky | look for .zuul.yaml or zuul* directories | 16:43 |
smcginnis | It 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 |
smcginnis | I haven't looked what is all in there already, but I know there are at least a handful. | 16:44 |
openstackgerrit | Merged openstack/os-brick master: Leverage the iSCSI mpath to get the WWN https://review.opendev.org/732424 | 16:45 |
smcginnis | Huh, or one. https://opendev.org/openstack/tempest/src/branch/master/tempest/scenario/test_volume_backup_restore.py | 16:46 |
smcginnis | And https://opendev.org/openstack/tempest/src/branch/master/tempest/api/volume/test_volumes_backup.py | 16:46 |
jopdorp | I'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 driver | 16:50 |
*** jopdorp has quit IRC | 16:52 | |
*** Lucas_Gray has quit IRC | 16:52 | |
*** sapd1 has quit IRC | 16:56 | |
openstackgerrit | Rajini Karthik proposed openstack/os-brick stable/ussuri: ScaleIO: Connection info backward compatibility https://review.opendev.org/746572 | 17:20 |
*** josephillips has joined #openstack-cinder | 17:23 | |
openstackgerrit | Merged openstack/os-brick master: Add release note for scaleio connector upgrade https://review.opendev.org/746551 | 17:25 |
*** brinzhang0 has joined #openstack-cinder | 17:37 | |
*** vishalmanchanda has quit IRC | 17: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 IRC | 17:40 | |
openstackgerrit | Rajini Karthik proposed openstack/os-brick stable/ussuri: Add release note for scaleio connector upgrade https://review.opendev.org/746576 | 17:55 |
*** jopdorp has joined #openstack-cinder | 18:26 | |
jopdorp | Is this unregistered or openstack-cinder? | 18:27 |
josephillips | jopdorp openstack-cinder | 18:28 |
josephillips | lol | 18:28 |
jopdorp | Haha okay, I just downloaded an app fory mobile phone, it wasn't so clear | 18:28 |
jopdorp | Is there a cinder meeting on Tuesday? | 18:29 |
*** hoonetorg has joined #openstack-cinder | 18:29 | |
openstackgerrit | katari manoj kumar proposed openstack/cinder master: [storwize]: optimize create_flashcopy_to_consistgrp https://review.opendev.org/745719 | 18:29 |
rosmaita | jopdorp: wednesday at 1400 utc | 18:29 |
jopdorp | rosmaita: is that on any video conferencing platform? | 18:30 |
rosmaita | jopdorp: on IRC in #openstack-meeting-alt | 18:30 |
smcginnis | https://wiki.openstack.org/wiki/CinderMeetings | 18:30 |
*** breizhkoala has quit IRC | 18:36 | |
*** e0ne has quit IRC | 18:48 | |
*** e0ne has joined #openstack-cinder | 19:03 | |
*** e0ne has quit IRC | 19:03 | |
*** smcginnis has quit IRC | 19:19 | |
*** smcginnis has joined #openstack-cinder | 19:21 | |
*** ChanServ sets mode: +o smcginnis | 19:21 | |
*** e0ne has joined #openstack-cinder | 19:29 | |
*** martinkennelly has quit IRC | 19:31 | |
*** martinkennelly has joined #openstack-cinder | 19:31 | |
openstackgerrit | Merged openstack/cinder stable/stein: RBD: Cleanup temporary file during exception https://review.opendev.org/746041 | 19:40 |
openstackgerrit | Merged openstack/os-brick stable/ussuri: ScaleIO: Connection info backward compatibility https://review.opendev.org/746572 | 21:12 |
openstackgerrit | Merged openstack/os-brick stable/ussuri: Add release note for scaleio connector upgrade https://review.opendev.org/746576 | 21:12 |
*** e0ne has quit IRC | 21: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 great | 21:30 |
*** e0ne has joined #openstack-cinder | 21:31 | |
*** e0ne has quit IRC | 21:37 | |
*** ociuhandu has quit IRC | 21:39 | |
*** brinzhang_ has joined #openstack-cinder | 21:44 | |
*** brinzhang0 has quit IRC | 21:47 | |
*** benfelin has quit IRC | 21:58 | |
*** rcernin has joined #openstack-cinder | 22:33 | |
*** Lucas_Gray has joined #openstack-cinder | 22:41 | |
*** tosky has quit IRC | 22:57 | |
*** martinkennelly has quit IRC | 23:00 | |
*** breizhkoala has joined #openstack-cinder | 23:03 | |
*** Lucas_Gray has quit IRC | 23:04 | |
*** tkajinam has joined #openstack-cinder | 23:07 | |
*** brinzhang0 has joined #openstack-cinder | 23:09 | |
*** brinzhang_ has quit IRC | 23:12 | |
*** ociuhandu has joined #openstack-cinder | 23:39 | |
*** ociuhandu has quit IRC | 23:45 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!