Thursday, 2020-11-05

*** tosky has quit IRC00:14
*** martinkennelly has quit IRC00:21
*** martinkennelly has joined #openstack-cinder00:21
*** whoami-rajat___ has joined #openstack-cinder01:06
*** carloss has quit IRC01:34
*** Liang__ has joined #openstack-cinder01:37
*** Liang__ has quit IRC01:39
*** sapd1 has joined #openstack-cinder01:41
*** rcernin_ has joined #openstack-cinder01:42
*** xinliang has joined #openstack-cinder01:42
*** rcernin has quit IRC01:43
*** martinkennelly has quit IRC01:58
*** spatel has joined #openstack-cinder02:10
*** spatel has quit IRC02:11
*** spatel has joined #openstack-cinder02:12
*** spatel has quit IRC02:14
*** jhdedrick has quit IRC02:23
*** xinliang has quit IRC02:39
*** kaisers has quit IRC02:45
*** ajitha has joined #openstack-cinder03:01
*** amar7ibm has joined #openstack-cinder03:17
*** hamalq has quit IRC03:21
*** rcernin_ has quit IRC03:24
*** rcernin_ has joined #openstack-cinder03:33
*** rcernin_ has quit IRC03:47
*** sapd1 has quit IRC03:57
*** udesale has joined #openstack-cinder04:04
*** Liang__ has joined #openstack-cinder04:13
*** rcernin_ has joined #openstack-cinder04:19
*** rcernin_ has quit IRC04:19
*** rcernin has joined #openstack-cinder04:19
*** manoj_kumar_kata has joined #openstack-cinder04:29
*** vishalmanchanda has joined #openstack-cinder04:31
*** rcernin has quit IRC05:13
*** rcernin has joined #openstack-cinder05:14
*** sharathkacham has joined #openstack-cinder05:31
sharathkachamPlease review code changes done in https://review.opendev.org/#/c/755241/05:31
*** whoami-rajat___ is now known as whoami-rajat__05:32
*** sharathkacham has quit IRC05:59
*** rcernin has quit IRC06:37
*** rcernin has joined #openstack-cinder06:50
*** Liang__ has quit IRC07:06
*** Liang__ has joined #openstack-cinder07:07
*** thonic has joined #openstack-cinder07:09
*** lpetrut has joined #openstack-cinder07:11
*** rcernin has quit IRC07:12
*** melwitt has quit IRC07:20
*** melwitt has joined #openstack-cinder07:21
*** jmccrory has quit IRC07:22
*** jmccrory has joined #openstack-cinder07:22
*** zenkuro has quit IRC07:27
*** rcernin has joined #openstack-cinder07:33
*** rcernin has quit IRC07:43
*** pcaruana has quit IRC07:52
*** pcaruana has joined #openstack-cinder08:04
*** tosky has joined #openstack-cinder08:13
*** hoonetorg has quit IRC08:21
*** hoonetorg has joined #openstack-cinder08:26
*** rpittau|afk is now known as rpittau08:39
*** ociuhandu has joined #openstack-cinder09:06
*** ociuhandu has quit IRC09:16
*** ociuhandu has joined #openstack-cinder09:22
*** martinkennelly has joined #openstack-cinder09:25
*** ociuhandu has quit IRC09:25
*** ociuhandu has joined #openstack-cinder09:26
*** suryasingh has joined #openstack-cinder09:40
noonedeadpunkHi everyone! I'm wondering how safe in terms of security and possible privilege escalation is defining auth_type (and all service creds) in nova section?09:44
*** udesale_ has joined #openstack-cinder09:44
*** udesale has quit IRC09:47
*** thonic has quit IRC09:54
*** ociuhandu has quit IRC09:57
*** ociuhandu has joined #openstack-cinder09:58
*** ociuhandu has quit IRC10:01
*** ociuhandu has joined #openstack-cinder10:02
*** kaisers has joined #openstack-cinder10:02
*** Liang__ has quit IRC10:10
*** CeeMac has joined #openstack-cinder10:22
*** noonedeadpunk has quit IRC10:32
*** noonedeadpunk has joined #openstack-cinder10:32
*** e0ne has joined #openstack-cinder10:46
openstackgerritGorka Eguileor proposed openstack/cinder master: Add explanations on safe delete  https://review.opendev.org/76153510:55
geguileodoes anyone know how to reject a blueprint in launchpad?11:01
*** noonedeadpunk has quit IRC11:21
*** noonedeadpunk has joined #openstack-cinder11:25
openstackgerritRajat Dhasmana proposed openstack/cinder master: WIP: Support format info in fs type drivers  https://review.opendev.org/76115211:58
*** ociuhandu has quit IRC12:00
*** ociuhandu has joined #openstack-cinder12:01
*** manoj_kumar_kata has quit IRC12:04
*** manoj_kumar_kata has joined #openstack-cinder12:05
*** ociuhandu has quit IRC12:06
*** ociuhandu has joined #openstack-cinder13:14
*** sapd1 has joined #openstack-cinder13:18
*** suryasingh has quit IRC13:33
hemnamep13:42
openstackgerritTobias Urdin proposed openstack/cinder master: Add backup_swift_container_storage_policy config opt  https://review.opendev.org/76111313:56
jungleboyjboop14:13
*** carloss has joined #openstack-cinder14:27
*** vishalmanchanda has quit IRC14:30
*** manoj_kumar_kata has quit IRC14:34
*** manoj_kumar_kata has joined #openstack-cinder14:34
*** zenkuro has joined #openstack-cinder14:38
*** sharathkacham has joined #openstack-cinder14:50
hemnaceph-iscsi driver looks good for reviews now.  passing CI.  would be nice to get that in as well as the chain of patches associated with it.14:56
*** amar7ibm58 has joined #openstack-cinder14:57
*** amar7ibm has quit IRC14:57
*** amar7ibm58 has quit IRC14:58
*** andrebeltrami has joined #openstack-cinder14:58
*** amar7ibm has joined #openstack-cinder14:58
smcginnisamar7ibm: I think you need to write up a spec for the work you are trying to do. That will help you get feedback from the team on how it should fit in with Cinder.15:04
*** amar7ibm has quit IRC15:04
*** amar7ibm has joined #openstack-cinder15:05
*** thgcorrea has joined #openstack-cinder15:08
*** manoj_kumar_kata has quit IRC15:15
*** manoj_kumar_kata has joined #openstack-cinder15:15
*** ociuhandu has quit IRC15:21
*** venkatakrishnath has joined #openstack-cinder15:24
venkatakrishnathPatch https://review.opendev.org/#/c/757082 got reviewed along with a successful IBM storage CI run and waiting for +2.15:28
*** gksk has joined #openstack-cinder15:28
gkskTo the Core Reviewers please review this feature https://review.opendev.org/#/c/751837/15:29
*** manoj_kumar_kata has quit IRC15:38
*** manoj_kumar_kata has joined #openstack-cinder15:38
*** ociuhandu has joined #openstack-cinder15:45
*** gksk has quit IRC15:48
*** amar7ibm has quit IRC15:59
*** jhdedrick has joined #openstack-cinder16:03
*** zenkuro has quit IRC16:03
*** zenkuro has joined #openstack-cinder16:04
rosmaitageguileo: we may need to ask fungi about that, i think as long as the BP hasn't been marked accepted, you can reject it, but once it's accepted that option seems to disappear16:08
geguileorosmaita: I don't have the reject option XDXD16:09
rosmaitageguileo: what's the BP?16:09
geguileorosmaita: https://blueprints.launchpad.net/cinder/+spec/specify-data-secure-deletion16:09
rosmaitageguileo: for "Series goal" click on "Proposed"16:10
geguileorosmaita: oooooooh, nice!!16:11
rosmaitai learned about that last week16:11
geguileorosmaita: but then again, direction it still says "Needs approval" and that's weird16:11
*** sharathkacham has quit IRC16:11
rosmaitayeah, the fields are all independent as far as i can tell16:12
toskybut are launchpad blueprints really needed when there are specs?16:12
geguileorosmaita: then there should be something in the Direction16:12
fungiyeah, honestly i know basically nothing about the lp blueprints feature. i've never been heavily involved in any project which relied on them16:12
geguileotosky: we don't want people to look at the BP and think that this is something that's going to be happening16:12
rosmaitatosky: yes, the BPs are automatically hooked up to Gerrit and get updates16:12
rosmaitaand what geguileo said16:12
toskygeguileo: sure, but the point is: why create them in the first place if there is a spec?16:13
toskyif they don't exist, people can't look at them16:13
fungigerrit runs a hook script when changes are uploaded and if a change mentions the name of a launchpad blueprint it tries to find that bp in lp and add a reference i think16:13
rosmaitawe use them for driver features16:13
geguileotosky: because that's how all the patches are grouped16:13
fungibut that's the extent of our gerrit "integration" for blueprints16:13
rosmaitafungi: it is pretty handy, though16:14
geguileotosky: you have a BP and you can have your spec patch and implementation patches all linked through it16:14
toskygeguileo: right, because launchpad is used16:14
fungihaving not used blueprints, i can't judge the handiness of the integration myself but will take your word for it ;)16:14
clarkbfungi: and that is something that will break with the gerrit upgrade unless someone finds time to update that integration to handle the removal of the db16:14
toskynow that I think about it, in sahara that grouping thing was done through storyboard16:14
smcginnisI had thought of getting rid of them for Cinder, but at the time at least, it turned out some people were relying on looking at what blueprints were targeted to which milestones to do some planning.16:14
smcginnisThat may have changed now.16:14
fungiit shouldn't break with gerrit upgrade, i don't think the bp integration uses account lookups, only the bug integration for lp does that16:14
smcginnisSo really up to however rosmaita wants to manage things.16:15
rosmaitayeah, i think they are helpful since we are still using launchpad16:15
clarkbah its welcome message that is the other one that uses it16:15
fungiright16:15
*** udesale_ has quit IRC16:27
*** jhdedrick has quit IRC16:36
*** sapd1 has quit IRC16:51
*** venkatakrishnath has quit IRC17:00
*** rpittau is now known as rpittau|afk17:21
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver - Port status check  https://review.opendev.org/76164317:28
*** hamalq has joined #openstack-cinder17:38
*** manoj_kumar_kata has quit IRC17:54
openstackgerritGorka Eguileor proposed openstack/cinder master: Add explanations on safe delete  https://review.opendev.org/76153517:54
lsekiwhoami-rajat__, rosmaita: thgcorrea just tested this patch https://review.opendev.org/#/c/729657/ and looks safe to merge18:03
lsekisfernand just removed his -W18:03
lsekiso it's OK to merge it to stable/stein18:04
*** ociuhandu has quit IRC18:04
*** ociuhandu has joined #openstack-cinder18:12
*** ociuhandu has quit IRC18:16
rosmaitalseki: ack18:17
hemnahas anyone messed with rally lately?18:51
hemnaseems to be utterly broken trying to run anything meaningful now18:51
jungleboyjPeople have messed with rallies, but not with rally.18:51
openstackgerritMerged openstack/cinder stable/victoria: [IBM DS8K]: Fixed rest API issue to get bundle  https://review.opendev.org/75981718:52
jungleboyjI am not surprised by that.  I haven't heard anyone talk about it in a while.18:52
*** andrebeltrami has quit IRC18:57
openstackgerritLance Bragstad proposed openstack/cinder master: Add common RBAC personas to cinder/policies/base.py  https://review.opendev.org/75994219:15
*** lpetrut has quit IRC19:16
*** lbragstad has quit IRC19:18
openstackgerritAde Lee proposed openstack/cinder master: Replace md5 with oslo version  https://review.opendev.org/75384819:26
hemnarallies?20:08
*** e0ne has quit IRC20:21
*** rchurch has quit IRC20:22
*** e0ne has joined #openstack-cinder20:37
*** ociuhandu has joined #openstack-cinder20:57
*** ociuhandu has quit IRC20:58
*** ociuhandu_ has joined #openstack-cinder20:58
*** e0ne has quit IRC21:02
*** ade_lee has joined #openstack-cinder21:16
ade_leewhoami-rajat__, sorry -- I got into wrong channel to begin with :)21:16
whoami-rajat__ade_lee: np, my concern is the CIs of the driver we changed isn't reporting i.e. veritas and synology21:17
ade_leewhoami-rajat__, ah gotcha -- I'm not sure what to do about that :/21:19
ade_leewhoami-rajat__, rechecks?21:20
*** ociuhandu_ has quit IRC21:23
ade_leewhoami-rajat__, which ci jobs have synology and veritas?21:24
toskymost likely a 3rd-party CI21:25
*** ociuhandu has joined #openstack-cinder21:25
*** thgcorrea has quit IRC21:26
*** rcernin has joined #openstack-cinder21:27
whoami-rajat__ade_lee: so veritas is not supported i.e. it's CI won't run and there's a debate if we should modify unsupported drivers or not (i won't go into it), for synology i see success rate is very less and not sure what's it's recheck command is21:27
whoami-rajat__http://paste.openstack.org/show/799758/21:28
whoami-rajat__ok got it21:29
ade_leewhoami-rajat__, if we feel uncomfortable about modifying the veritas driver , I can remove those changes and skip those tests when running on a fips enabled system.21:34
ade_leesame for synology ..21:34
*** ociuhandu has quit IRC21:35
whoami-rajat__ade_lee: I've triggered synology CI so that might post results in few hours, about veritas changes, rosmaita  what do you think? should we keep the changes or avoid changes in the unsupported driver?21:51
*** rcernin has quit IRC21:53
*** rcernin has joined #openstack-cinder21:54
rosmaitawhoami-rajat__: officially, the project team is supporting "unsupported" drivers on a best-effort basis, this seems like a simple change, i think we should do it ... that way, fewer problems if the vendor decides to re-support it21:56
ade_leeworks for me21:57
ade_leebrb21:57
*** e0ne has joined #openstack-cinder22:03
*** hamalq has quit IRC22:11
whoami-rajat__rosmaita: same thoughts, thanks for the input22:16
rosmaitanp22:17
*** e0ne has quit IRC22:17
*** ajitha has quit IRC22:30
openstackgerritMerged openstack/os-brick master: Replace md5 with oslo version  https://review.opendev.org/75615122:53
*** rcernin has quit IRC22:57
openstackgerritMerged openstack/cinder stable/ussuri: Do not fail when depth is greater than rbd_max_clone_depth  https://review.opendev.org/76118022:59
*** rcernin has joined #openstack-cinder23:00
*** spatel has joined #openstack-cinder23:20
*** spatel has quit IRC23:25
*** ociuhandu has joined #openstack-cinder23:36
*** ociuhandu has quit IRC23:40
*** tosky has quit IRC23:55

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