Tuesday, 2021-01-26

« Monday, 2021-01-25 Index Wednesday, 2021-01-27 »
*** rcernin has joined #openstack-cinder00:03
*** brinzhang has quit IRC00:05
*** brinzhang has joined #openstack-cinder00:17
*** rcernin has quit IRC00:24
*** rcernin has joined #openstack-cinder00:27
*** benfelin has quit IRC00:59
*** pcaruana has quit IRC01:12
*** pcaruana has joined #openstack-cinder01:12
*** sfernand has quit IRC01:16
*** rcernin has quit IRC01:18
openstackgerritMerged openstack/cinder stable/victoria: Fixed an issue with creating a backup from snapshot with NFS volume driver.  https://review.opendev.org/c/openstack/cinder/+/76515001:31
*** baojg has joined #openstack-cinder01:39
*** rcernin has joined #openstack-cinder01:55
*** rcernin has quit IRC02:02
*** tinwood has quit IRC02:08
*** tinwood has joined #openstack-cinder02:11
*** rcernin has joined #openstack-cinder02:15
*** zzzeek has quit IRC02:19
*** rcernin has quit IRC02:20
*** zzzeek has joined #openstack-cinder02:22
*** rcernin has joined #openstack-cinder02:23
*** ociuhandu has joined #openstack-cinder02:24
*** spatel has joined #openstack-cinder02:27
*** ociuhandu has quit IRC02:28
*** spatel has quit IRC02:29
*** spatel has joined #openstack-cinder02:30
*** spatel has quit IRC02:31
*** spatel has joined #openstack-cinder02:33
*** rcernin has quit IRC02:38
*** zzzeek has quit IRC02:49
*** spatel has quit IRC02:49
*** zzzeek has joined #openstack-cinder02:51
*** spatel has joined #openstack-cinder02:55
*** johanssone has quit IRC03:00
*** johanssone has joined #openstack-cinder03:01
*** rcernin has joined #openstack-cinder03:34
*** rcernin has quit IRC03:39
*** senrique_ has quit IRC03:49
*** rcernin has joined #openstack-cinder05:18
*** brinzhang has quit IRC05:20
*** brinzhang has joined #openstack-cinder05:21
*** rcernin has quit IRC05:26
*** rcernin has joined #openstack-cinder05:40
*** rcernin has quit IRC05:45
*** spatel has quit IRC05:47
*** m75abrams has joined #openstack-cinder06:04
*** rcernin has joined #openstack-cinder06:09
*** rcernin has quit IRC06:49
*** spotz has quit IRC06:54
*** Reepicheep has quit IRC06:57
*** Reepicheep has joined #openstack-cinder06:57
openstackgerritMasahiro Okuri proposed openstack/cinder master: created s3 cinder backup driver  https://review.opendev.org/c/openstack/cinder/+/74656107:06
*** xinliang has joined #openstack-cinder07:11
*** rcernin has joined #openstack-cinder07:38
*** BakaKuna has joined #openstack-cinder07:50
*** zoharm has joined #openstack-cinder07:55
*** rpittau|afk_ is now known as rpittau07:57
*** m75abrams has quit IRC08:09
*** m75abrams has joined #openstack-cinder08:10
*** rcernin has quit IRC08:17
zoharmHi all, please review KIOXIA driver: https://review.opendev.org/c/openstack/cinder/+/76857408:29
zoharmZuul was successful before but was only failing recently in cinder-tempest-plugin-lvm-lio-barbican and then cinder-grenade-mn-sub-volbak which I believe is unrelated to this patchset.08:29
zoharmWe also had a CI up posting tempest results for a week, you can see all the patchsets it ran on here: http://104.254.65.37/08:29
zoharmCI is not always in the most stable state (interminttent issues during stack.sh) but I am actively working on improving and robustifying it. Aside from that, the priority is always responding to review comments!08:29
openstackgerritZohar Mamedov proposed openstack/os-brick master: NVMeOF Connector support MDRAID replication  https://review.opendev.org/c/openstack/os-brick/+/76857508:31
openstackgerritZohar Mamedov proposed openstack/cinder master: Add KIOXIA KumoScale NVMeOF driver  https://review.opendev.org/c/openstack/cinder/+/76857408:31
*** m75abrams has quit IRC08:40
*** m75abrams has joined #openstack-cinder08:41
*** rcernin has joined #openstack-cinder08:42
*** benfelin has joined #openstack-cinder08:45
*** tosky has joined #openstack-cinder08:45
*** rcernin has quit IRC08:46
*** ociuhandu has joined #openstack-cinder08:56
*** ociuhandu has quit IRC08:58
*** ociuhandu has joined #openstack-cinder08:59
*** ociuhandu has quit IRC09:19
*** ociuhandu has joined #openstack-cinder09:25
*** rcernin has joined #openstack-cinder09:27
*** ociuhandu has quit IRC09:30
openstackgerritMasahiro Okuri proposed openstack/cinder master: created s3 cinder backup driver  https://review.opendev.org/c/openstack/cinder/+/74656109:32
*** rcernin has quit IRC09:45
*** e0ne has joined #openstack-cinder09:49
*** ociuhandu has joined #openstack-cinder09:54
*** rcernin has joined #openstack-cinder09:58
*** e0ne has quit IRC10:11
*** e0ne has joined #openstack-cinder10:11
*** brinzhang has quit IRC10:14
*** rcernin has quit IRC10:27
*** hack-char has quit IRC10:29
*** udesale has joined #openstack-cinder10:29
*** hack-char has joined #openstack-cinder10:30
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver - Allow for volume uuid in manage volume  https://review.opendev.org/c/openstack/cinder/+/77238810:41
*** e0ne has quit IRC10:50
*** xinliang has quit IRC10:58
*** rcernin has joined #openstack-cinder11:04
*** brinzhang has joined #openstack-cinder11:04
*** rpittau is now known as rpittau|bbl11:09
*** masterpe has quit IRC11:19
*** masterpe has joined #openstack-cinder11:29
*** baojg has quit IRC11:32
*** baojg has joined #openstack-cinder11:34
*** baojg has quit IRC11:34
*** baojg has joined #openstack-cinder11:35
*** rcernin has quit IRC11:35
*** baojg has quit IRC11:35
*** baojg has joined #openstack-cinder11:35
*** baojg has quit IRC11:36
*** baojg has joined #openstack-cinder11:36
*** baojg has quit IRC11:37
*** baojg has joined #openstack-cinder11:37
*** baojg has quit IRC11:37
*** baojg has joined #openstack-cinder11:38
*** baojg has quit IRC11:38
*** baojg has joined #openstack-cinder11:39
*** baojg has quit IRC11:39
*** baojg has joined #openstack-cinder11:39
*** baojg has quit IRC11:40
*** baojg has joined #openstack-cinder11:40
*** baojg has quit IRC11:41
*** baojg has joined #openstack-cinder11:41
*** baojg has quit IRC11:41
*** baojg has joined #openstack-cinder11:42
*** baojg has quit IRC11:42
*** baojg has joined #openstack-cinder11:43
*** baojg has quit IRC11:43
*** baojg has joined #openstack-cinder11:43
*** baojg has quit IRC11:44
*** udesale_ has joined #openstack-cinder11:46
*** udesale has quit IRC11:49
*** m75abrams has quit IRC11:56
*** rosmaita has left #openstack-cinder11:58
*** baojg has joined #openstack-cinder12:01
*** baojg has quit IRC12:02
*** baojg has joined #openstack-cinder12:04
*** baojg has quit IRC12:07
*** m75abrams has joined #openstack-cinder12:09
*** ociuhandu has quit IRC12:19
*** ociuhandu has joined #openstack-cinder12:19
*** ociuhandu has quit IRC12:24
*** senrique_ has joined #openstack-cinder12:45
*** rosmaita has joined #openstack-cinder12:54
*** ociuhandu has joined #openstack-cinder12:56
*** rpittau|bbl is now known as rpittau12:59
zoharmHi Cinder folks! please review KIOXIA driver: https://review.opendev.org/c/openstack/cinder/+/76857413:10
zoharmZuul was +1 before but was having failures (that seem unrelated to this patchset) such as cinder-tempest-plugin-lvm-lio-barbican and cinder-grenade-mn-sub-volbak and now other lvm failures here: https://zuul.opendev.org/t/openstack/build/da189dbb614e499197787253a511878213:10
zoharmWe also had a CI up posting tempest volume results, you can see all the patchsets it ran on here: http://104.254.65.37/13:10
zoharmCI is not always stable (interminttent issues during stack.sh) but I am actively working on improving and robustifying it. Aside from that, the priority is always addressing review comments!13:10
openstackgerritBrian Rosmaita proposed openstack/cinderlib stable/ussuri: Update requirements and lower-constraints  https://review.opendev.org/c/openstack/cinderlib/+/77107913:20
rosmaitazoharm: thanks for the updates13:21
*** m75abrams has quit IRC13:21
openstackgerritHelen Walsh proposed openstack/cinder master: Cinder - Creating clone of encrypted volume fails  https://review.opendev.org/c/openstack/cinder/+/77251713:27
*** martinkennelly has joined #openstack-cinder13:33
openstackgerritBrian Rosmaita proposed openstack/cinderlib master: Update requirements  https://review.opendev.org/c/openstack/cinderlib/+/77252013:40
rosmaitaoops, that was not supposed to be to master13:41
*** m75abrams has joined #openstack-cinder13:44
openstackgerritBrian Rosmaita proposed openstack/cinderlib stable/victoria: Update requirements  https://review.opendev.org/c/openstack/cinderlib/+/77252213:51
*** spatel has joined #openstack-cinder14:02
*** m75abrams has quit IRC14:11
*** stand has quit IRC14:24
*** brinzhang has quit IRC14:25
*** brinzhang has joined #openstack-cinder14:25
*** stand has joined #openstack-cinder14:26
*** zenkuro has quit IRC14:26
*** zenkuro has joined #openstack-cinder14:27
*** m75abrams has joined #openstack-cinder14:30
*** stand has quit IRC14:43
*** m75abrams has quit IRC14:46
*** m75abrams has joined #openstack-cinder14:47
*** brinzhang has quit IRC14:48
*** brinzhang has joined #openstack-cinder14:51
*** brinzhang has quit IRC14:51
*** brinzhang has joined #openstack-cinder14:52
jungleboyjsmcginnis: Thank you for getting the review analysis running again.15:10
smcginnisjungleboyj: Yeah, that was weird. The new gerrit changes from using HTTPDigestAuth to HTTPBasicAuth. Took a bit to figure that one out.15:11
jungleboyjAh, interesting.  Thanks for figuring it out.15:12
caiqilongHi, core-reviewers. I'm going to propose a change to improve some things about "TOYOU ACS5000 Storage" driver. Can I do this between Milestone2 and Milestone3? And are there any differences when the proposed change is about improve something instead of add a new driver?15:14
openstackgerritBrian Rosmaita proposed openstack/cinderlib master: Update contributor docs  https://review.opendev.org/c/openstack/cinderlib/+/77255815:15
smcginniscaiqilong: Yes, improvements can be submitted at any time. As we get closer to the release candidate (RC) towards the end of the cycle, then we start to limit things to just critical bugfixes.15:18
rosmaitacaiqilong: you can propose at any time; if what you are proposing is a bugfix, file a bug and mention it in the commit message; if you are following up on issues that came up during review of the driver, you can say that in the commit message15:18
caiqilongThanks for the both warm-hearted reply. My confusion has been away.15:23
*** gary_perkins has quit IRC15:28
*** strigazi has quit IRC15:45
*** strigazi has joined #openstack-cinder15:47
*** sapd1_x has joined #openstack-cinder15:54
*** ociuhandu has quit IRC15:56
*** ociuhandu has joined #openstack-cinder15:57
*** ociuhandu has quit IRC16:01
*** ociuhandu has joined #openstack-cinder16:04
*** m75abrams has quit IRC16:06
*** abishop_ has joined #openstack-cinder16:14
*** abishop has quit IRC16:17
*** abishop_ is now known as abishop16:23
*** GirishChilukuri has joined #openstack-cinder16:25
*** e0ne has joined #openstack-cinder16:27
*** ociuhandu_ has joined #openstack-cinder16:31
*** udesale_ has quit IRC16:33
*** ociuhandu has quit IRC16:35
*** ociuhandu_ has quit IRC16:36
*** gary_perkins has joined #openstack-cinder16:36
*** zenkuro has quit IRC16:43
hemnarosmaita hey man whats the status on the ceph-iscsi deadline ?16:45
hemnalooks like all the patches are passing, but not getting reviews16:46
*** ociuhandu has joined #openstack-cinder16:49
*** ociuhandu has quit IRC16:53
*** zenkuro has joined #openstack-cinder16:58
*** rpittau is now known as rpittau|afk17:27
rosmaitahemna: merge by feb 5. TOYOU has merged and s3 backup is close, so that will leave only ceph-iscsi and kioxia ... hopefully people will be able to concentrate17:40
*** sapd1_x has quit IRC17:41
*** zoharm has quit IRC17:49
*** zenkuro has quit IRC17:52
*** tzumainn has joined #openstack-cinder17:59
*** jv has quit IRC18:15
*** GirishChilukuri has quit IRC18:16
*** senrique_ is now known as enriquetaso18:47
lbragstadrosmaita so - when the project ID in the path mismatches the project id in the context, i get a 40018:50
rosmaitadoes the error message say something about "malformed URL" ?18:50
openstackgerritEric Harney proposed openstack/cinder master: Move brick calls from cinder.utils to volume_utils  https://review.opendev.org/c/openstack/cinder/+/77129018:52
openstackgerritEric Harney proposed openstack/cinder master: Move trace methods from utils to volume_utils  https://review.opendev.org/c/openstack/cinder/+/77258218:52
lbragstadrosmaita checking18:53
lbragstadrosmaita yes18:58
lbragstad{'badRequest': {'code': 400, 'message': 'Malformed request url'}}18:58
*** e0ne has quit IRC18:59
*** ianw_pto is now known as ianw18:59
rosmaitalbragstad: yeah, that's kind of cryptic for this situation18:59
enriquetasoLooks like https://bugs.launchpad.net/cinder doesn't  'date' filter18:59
lbragstadso - how does a project admin (in today's world) create a volume on behalf of another project?18:59
lbragstadbecuase it looks like that is unsupported19:00
openstackgerritLance Bragstad proposed openstack/cinder master: Apply consistent personas to existing policy tests  https://review.opendev.org/c/openstack/cinder/+/77243919:03
openstackgerritLance Bragstad proposed openstack/cinder master: WIP: experiement with policy testing approaches for secure rbac  https://review.opendev.org/c/openstack/cinder/+/77258319:03
rosmaitalbragstad: i thought if you had admin credentials, you could do it directly with the target project in the URL19:05
rosmaitaotherwise, we have a volume transfer API19:05
lbragstadthere appears to be some code in cinder that matches the project ID from the path against the project ID in the context (which is good, i think)19:05
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver - RDF checks  https://review.opendev.org/c/openstack/cinder/+/76810519:06
lbragstadbut - it assumes that all callers will invoke the API with project scoped tokens and that might need to change in the future if cinder wants to support system admins creating volumes in projects for other users19:06
rosmaitai will put that on my list to investigate19:09
*** sfernand has joined #openstack-cinder19:12
lbragstadrosmaita ok - should i go back to dusting off? https://review.opendev.org/c/openstack/cinder/+/77187519:20
lbragstadsounds like working on the ddt approach will be more useful19:21
rosmaitai think that makes sense19:21
lbragstadok19:22
lbragstadi'll see if i can get that passing19:22
lbragstadi think everything in cinder/tests/unit/polices can go into the legacy_rbac tests19:23
rosmaitalbragstad: when you have some time, please take a look at the introductory info in https://review.opendev.org/c/openstack/cinder/+/763306/8/doc/source/configuration/block-storage/policy-personas.rst to make sure it conforms with current thinking on the personas19:24
lbragstadrosmaita can i track that against secure-rbac?19:24
rosmaitasure, do you need me to change the topic?19:25
lbragstaddone19:26
rosmaitagreat19:26
lbragstadnope - i just wanted to make sure19:26
*** e0ne has joined #openstack-cinder19:30
*** e0ne has quit IRC19:32
*** hamalq has joined #openstack-cinder19:36
tzumainnhi! I'm looking to use multipath outside of the context of nova (hopefully for ironic); is there documentation or examples regarding how to do that?20:08
*** abishop_ has joined #openstack-cinder20:13
*** simondodsley has quit IRC20:14
*** abishop_ has quit IRC20:14
*** abishop_ has joined #openstack-cinder20:14
*** abishop has quit IRC20:15
*** jopdorp_ has quit IRC20:15
*** simondodsley has joined #openstack-cinder20:16
*** abishop has joined #openstack-cinder20:25
*** abishop_ has quit IRC20:25
*** abishop_ has joined #openstack-cinder20:34
*** abishop has quit IRC20:34
openstackgerritBrian Rosmaita proposed openstack/cinder stable/ussuri: Correct group:reset_group_snapshot_status policy  https://review.opendev.org/c/openstack/cinder/+/77254520:40
*** brinzhang_ has joined #openstack-cinder20:44
*** brinzhang has quit IRC20:47
*** hamalq has quit IRC21:21
*** hamalq has joined #openstack-cinder21:21
*** ajitha has quit IRC21:23
openstackgerritLance Bragstad proposed openstack/cinder master: Introduce basic plumbing for protection tests with ddt  https://review.opendev.org/c/openstack/cinder/+/77187521:25
lbragstadrosmaita i think most of the utilities in _FunctionalTestBase are written to assume NoAuthMiddleware - plugging them in with a mocked keystone_authtoken middleware seems to break a lot of things22:04
lbragstadi'm wondering if i should be rethinking this https://review.opendev.org/c/openstack/cinder/+/771875/6/cinder/tests/unit/policies/test_rbac.py#3122:05
lbragstadif I take ^ that approach, most of the _poll_$resource_while utilities are going to break because they're using the self.api.api_request() method, which will always attempt to authenticate against the auth_url22:07
lbragstadfor example - i'm writing some _create_resource methods so that we have volumes to test updates, deletes, and list with http://paste.openstack.org/show/802029/22:08
rosmaitayeah, i don't have a good idea for how to decouple that stuff22:10
lbragstadok22:15
*** rcernin has joined #openstack-cinder22:16
*** abishop_ has quit IRC22:16
lbragstadi was thinking about trying to expose the auth strategy a little higher in the test code so we could set it somewhere...22:16
*** abishop has joined #openstack-cinder22:17
lbragstadbut i'm backing out my changes again and i'm just going to try and retrofit NoAuthMiddleware22:17
*** rcernin has quit IRC22:24
*** rcernin has joined #openstack-cinder22:24
*** abishop has quit IRC22:25
*** abishop has joined #openstack-cinder22:27
*** abishop_ has joined #openstack-cinder22:41
openstackgerritLance Bragstad proposed openstack/cinder master: Update NoAuthMiddleware to be less opinioned about auth  https://review.opendev.org/c/openstack/cinder/+/77260822:41
lbragstadrosmaita ^22:41
rosmaitalbragstad: ty, will take a look later22:41
lbragstadthat's another approach we could take - but i'll require a refactor of the existing functional tests in cinder22:41
lbragstadsounds good - thanks for reviewing22:42
lbragstadit'll require*22:42
rosmaitacinder meeting is 1400 utc tomorrow if you want to ask for opinions22:42
lbragstadyeah - i'll be there22:42
*** abishop has quit IRC22:42
rosmaitawill be in video + irc: http://lists.openstack.org/pipermail/openstack-discuss/2021-January/020024.html22:43
lbragstadexcellent22:43
lbragstadthank you rosmaita22:43
*** spatel has quit IRC22:50
*** akrpan-pure has joined #openstack-cinder23:11
akrpan-pureI'm working on setting up third party CI in Devstack, and am running into a strange issue where the LVM driver is still trying to load despite there being no LVM stanza and another driver is enabled instead23:12
akrpan-pureI see "DEBUG cinder.volume.drivers.lvm [None req-d8e60a6b-e096-49a5-a420-058b13051519 None None] Attempting to initialize LVM driver with the following target_driver: cinder.volume.targets.tgt.TgtAdm"23:12
*** jgriffith has quit IRC23:13
akrpan-pureDefault volume type is set properly as well23:13
akrpan-pureAny immediate ideas as to what I'm missing?23:13
*** tosky has quit IRC23:18
*** jgriffith has joined #openstack-cinder23:41
*** ircuser-1 has joined #openstack-cinder23:46
« Monday, 2021-01-25 Index Wednesday, 2021-01-27 »

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