*** tosky has quit IRC | 00:14 | |
*** martinkennelly has quit IRC | 00:21 | |
*** martinkennelly has joined #openstack-cinder | 00:21 | |
*** whoami-rajat___ has joined #openstack-cinder | 01:06 | |
*** carloss has quit IRC | 01:34 | |
*** Liang__ has joined #openstack-cinder | 01:37 | |
*** Liang__ has quit IRC | 01:39 | |
*** sapd1 has joined #openstack-cinder | 01:41 | |
*** rcernin_ has joined #openstack-cinder | 01:42 | |
*** xinliang has joined #openstack-cinder | 01:42 | |
*** rcernin has quit IRC | 01:43 | |
*** martinkennelly has quit IRC | 01:58 | |
*** spatel has joined #openstack-cinder | 02:10 | |
*** spatel has quit IRC | 02:11 | |
*** spatel has joined #openstack-cinder | 02:12 | |
*** spatel has quit IRC | 02:14 | |
*** jhdedrick has quit IRC | 02:23 | |
*** xinliang has quit IRC | 02:39 | |
*** kaisers has quit IRC | 02:45 | |
*** ajitha has joined #openstack-cinder | 03:01 | |
*** amar7ibm has joined #openstack-cinder | 03:17 | |
*** hamalq has quit IRC | 03:21 | |
*** rcernin_ has quit IRC | 03:24 | |
*** rcernin_ has joined #openstack-cinder | 03:33 | |
*** rcernin_ has quit IRC | 03:47 | |
*** sapd1 has quit IRC | 03:57 | |
*** udesale has joined #openstack-cinder | 04:04 | |
*** Liang__ has joined #openstack-cinder | 04:13 | |
*** rcernin_ has joined #openstack-cinder | 04:19 | |
*** rcernin_ has quit IRC | 04:19 | |
*** rcernin has joined #openstack-cinder | 04:19 | |
*** manoj_kumar_kata has joined #openstack-cinder | 04:29 | |
*** vishalmanchanda has joined #openstack-cinder | 04:31 | |
*** rcernin has quit IRC | 05:13 | |
*** rcernin has joined #openstack-cinder | 05:14 | |
*** sharathkacham has joined #openstack-cinder | 05:31 | |
sharathkacham | Please 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 IRC | 05:59 | |
*** rcernin has quit IRC | 06:37 | |
*** rcernin has joined #openstack-cinder | 06:50 | |
*** Liang__ has quit IRC | 07:06 | |
*** Liang__ has joined #openstack-cinder | 07:07 | |
*** thonic has joined #openstack-cinder | 07:09 | |
*** lpetrut has joined #openstack-cinder | 07:11 | |
*** rcernin has quit IRC | 07:12 | |
*** melwitt has quit IRC | 07:20 | |
*** melwitt has joined #openstack-cinder | 07:21 | |
*** jmccrory has quit IRC | 07:22 | |
*** jmccrory has joined #openstack-cinder | 07:22 | |
*** zenkuro has quit IRC | 07:27 | |
*** rcernin has joined #openstack-cinder | 07:33 | |
*** rcernin has quit IRC | 07:43 | |
*** pcaruana has quit IRC | 07:52 | |
*** pcaruana has joined #openstack-cinder | 08:04 | |
*** tosky has joined #openstack-cinder | 08:13 | |
*** hoonetorg has quit IRC | 08:21 | |
*** hoonetorg has joined #openstack-cinder | 08:26 | |
*** rpittau|afk is now known as rpittau | 08:39 | |
*** ociuhandu has joined #openstack-cinder | 09:06 | |
*** ociuhandu has quit IRC | 09:16 | |
*** ociuhandu has joined #openstack-cinder | 09:22 | |
*** martinkennelly has joined #openstack-cinder | 09:25 | |
*** ociuhandu has quit IRC | 09:25 | |
*** ociuhandu has joined #openstack-cinder | 09:26 | |
*** suryasingh has joined #openstack-cinder | 09:40 | |
noonedeadpunk | Hi 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-cinder | 09:44 | |
*** udesale has quit IRC | 09:47 | |
*** thonic has quit IRC | 09:54 | |
*** ociuhandu has quit IRC | 09:57 | |
*** ociuhandu has joined #openstack-cinder | 09:58 | |
*** ociuhandu has quit IRC | 10:01 | |
*** ociuhandu has joined #openstack-cinder | 10:02 | |
*** kaisers has joined #openstack-cinder | 10:02 | |
*** Liang__ has quit IRC | 10:10 | |
*** CeeMac has joined #openstack-cinder | 10:22 | |
*** noonedeadpunk has quit IRC | 10:32 | |
*** noonedeadpunk has joined #openstack-cinder | 10:32 | |
*** e0ne has joined #openstack-cinder | 10:46 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Add explanations on safe delete https://review.opendev.org/761535 | 10:55 |
geguileo | does anyone know how to reject a blueprint in launchpad? | 11:01 |
*** noonedeadpunk has quit IRC | 11:21 | |
*** noonedeadpunk has joined #openstack-cinder | 11:25 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: WIP: Support format info in fs type drivers https://review.opendev.org/761152 | 11:58 |
*** ociuhandu has quit IRC | 12:00 | |
*** ociuhandu has joined #openstack-cinder | 12:01 | |
*** manoj_kumar_kata has quit IRC | 12:04 | |
*** manoj_kumar_kata has joined #openstack-cinder | 12:05 | |
*** ociuhandu has quit IRC | 12:06 | |
*** ociuhandu has joined #openstack-cinder | 13:14 | |
*** sapd1 has joined #openstack-cinder | 13:18 | |
*** suryasingh has quit IRC | 13:33 | |
hemna | mep | 13:42 |
openstackgerrit | Tobias Urdin proposed openstack/cinder master: Add backup_swift_container_storage_policy config opt https://review.opendev.org/761113 | 13:56 |
jungleboyj | boop | 14:13 |
*** carloss has joined #openstack-cinder | 14:27 | |
*** vishalmanchanda has quit IRC | 14:30 | |
*** manoj_kumar_kata has quit IRC | 14:34 | |
*** manoj_kumar_kata has joined #openstack-cinder | 14:34 | |
*** zenkuro has joined #openstack-cinder | 14:38 | |
*** sharathkacham has joined #openstack-cinder | 14:50 | |
hemna | ceph-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-cinder | 14:57 | |
*** amar7ibm has quit IRC | 14:57 | |
*** amar7ibm58 has quit IRC | 14:58 | |
*** andrebeltrami has joined #openstack-cinder | 14:58 | |
*** amar7ibm has joined #openstack-cinder | 14:58 | |
smcginnis | amar7ibm: 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 IRC | 15:04 | |
*** amar7ibm has joined #openstack-cinder | 15:05 | |
*** thgcorrea has joined #openstack-cinder | 15:08 | |
*** manoj_kumar_kata has quit IRC | 15:15 | |
*** manoj_kumar_kata has joined #openstack-cinder | 15:15 | |
*** ociuhandu has quit IRC | 15:21 | |
*** venkatakrishnath has joined #openstack-cinder | 15:24 | |
venkatakrishnath | Patch 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-cinder | 15:28 | |
gksk | To the Core Reviewers please review this feature https://review.opendev.org/#/c/751837/ | 15:29 |
*** manoj_kumar_kata has quit IRC | 15:38 | |
*** manoj_kumar_kata has joined #openstack-cinder | 15:38 | |
*** ociuhandu has joined #openstack-cinder | 15:45 | |
*** gksk has quit IRC | 15:48 | |
*** amar7ibm has quit IRC | 15:59 | |
*** jhdedrick has joined #openstack-cinder | 16:03 | |
*** zenkuro has quit IRC | 16:03 | |
*** zenkuro has joined #openstack-cinder | 16:04 | |
rosmaita | geguileo: 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 disappear | 16:08 |
geguileo | rosmaita: I don't have the reject option XDXD | 16:09 |
rosmaita | geguileo: what's the BP? | 16:09 |
geguileo | rosmaita: https://blueprints.launchpad.net/cinder/+spec/specify-data-secure-deletion | 16:09 |
rosmaita | geguileo: for "Series goal" click on "Proposed" | 16:10 |
geguileo | rosmaita: oooooooh, nice!! | 16:11 |
rosmaita | i learned about that last week | 16:11 |
geguileo | rosmaita: but then again, direction it still says "Needs approval" and that's weird | 16:11 |
*** sharathkacham has quit IRC | 16:11 | |
rosmaita | yeah, the fields are all independent as far as i can tell | 16:12 |
tosky | but are launchpad blueprints really needed when there are specs? | 16:12 |
geguileo | rosmaita: then there should be something in the Direction | 16:12 |
fungi | yeah, honestly i know basically nothing about the lp blueprints feature. i've never been heavily involved in any project which relied on them | 16:12 |
geguileo | tosky: we don't want people to look at the BP and think that this is something that's going to be happening | 16:12 |
rosmaita | tosky: yes, the BPs are automatically hooked up to Gerrit and get updates | 16:12 |
rosmaita | and what geguileo said | 16:12 |
tosky | geguileo: sure, but the point is: why create them in the first place if there is a spec? | 16:13 |
tosky | if they don't exist, people can't look at them | 16:13 |
fungi | gerrit 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 think | 16:13 |
rosmaita | we use them for driver features | 16:13 |
geguileo | tosky: because that's how all the patches are grouped | 16:13 |
fungi | but that's the extent of our gerrit "integration" for blueprints | 16:13 |
rosmaita | fungi: it is pretty handy, though | 16:14 |
geguileo | tosky: you have a BP and you can have your spec patch and implementation patches all linked through it | 16:14 |
tosky | geguileo: right, because launchpad is used | 16:14 |
fungi | having not used blueprints, i can't judge the handiness of the integration myself but will take your word for it ;) | 16:14 |
clarkb | fungi: 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 db | 16:14 |
tosky | now that I think about it, in sahara that grouping thing was done through storyboard | 16:14 |
smcginnis | I 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 |
smcginnis | That may have changed now. | 16:14 |
fungi | it shouldn't break with gerrit upgrade, i don't think the bp integration uses account lookups, only the bug integration for lp does that | 16:14 |
smcginnis | So really up to however rosmaita wants to manage things. | 16:15 |
rosmaita | yeah, i think they are helpful since we are still using launchpad | 16:15 |
clarkb | ah its welcome message that is the other one that uses it | 16:15 |
fungi | right | 16:15 |
*** udesale_ has quit IRC | 16:27 | |
*** jhdedrick has quit IRC | 16:36 | |
*** sapd1 has quit IRC | 16:51 | |
*** venkatakrishnath has quit IRC | 17:00 | |
*** rpittau is now known as rpittau|afk | 17:21 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Port status check https://review.opendev.org/761643 | 17:28 |
*** hamalq has joined #openstack-cinder | 17:38 | |
*** manoj_kumar_kata has quit IRC | 17:54 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Add explanations on safe delete https://review.opendev.org/761535 | 17:54 |
lseki | whoami-rajat__, rosmaita: thgcorrea just tested this patch https://review.opendev.org/#/c/729657/ and looks safe to merge | 18:03 |
lseki | sfernand just removed his -W | 18:03 |
lseki | so it's OK to merge it to stable/stein | 18:04 |
*** ociuhandu has quit IRC | 18:04 | |
*** ociuhandu has joined #openstack-cinder | 18:12 | |
*** ociuhandu has quit IRC | 18:16 | |
rosmaita | lseki: ack | 18:17 |
hemna | has anyone messed with rally lately? | 18:51 |
hemna | seems to be utterly broken trying to run anything meaningful now | 18:51 |
jungleboyj | People have messed with rallies, but not with rally. | 18:51 |
openstackgerrit | Merged openstack/cinder stable/victoria: [IBM DS8K]: Fixed rest API issue to get bundle https://review.opendev.org/759817 | 18:52 |
jungleboyj | I am not surprised by that. I haven't heard anyone talk about it in a while. | 18:52 |
*** andrebeltrami has quit IRC | 18:57 | |
openstackgerrit | Lance Bragstad proposed openstack/cinder master: Add common RBAC personas to cinder/policies/base.py https://review.opendev.org/759942 | 19:15 |
*** lpetrut has quit IRC | 19:16 | |
*** lbragstad has quit IRC | 19:18 | |
openstackgerrit | Ade Lee proposed openstack/cinder master: Replace md5 with oslo version https://review.opendev.org/753848 | 19:26 |
hemna | rallies? | 20:08 |
*** e0ne has quit IRC | 20:21 | |
*** rchurch has quit IRC | 20:22 | |
*** e0ne has joined #openstack-cinder | 20:37 | |
*** ociuhandu has joined #openstack-cinder | 20:57 | |
*** ociuhandu has quit IRC | 20:58 | |
*** ociuhandu_ has joined #openstack-cinder | 20:58 | |
*** e0ne has quit IRC | 21:02 | |
*** ade_lee has joined #openstack-cinder | 21:16 | |
ade_lee | whoami-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 synology | 21:17 |
ade_lee | whoami-rajat__, ah gotcha -- I'm not sure what to do about that :/ | 21:19 |
ade_lee | whoami-rajat__, rechecks? | 21:20 |
*** ociuhandu_ has quit IRC | 21:23 | |
ade_lee | whoami-rajat__, which ci jobs have synology and veritas? | 21:24 |
tosky | most likely a 3rd-party CI | 21:25 |
*** ociuhandu has joined #openstack-cinder | 21:25 | |
*** thgcorrea has quit IRC | 21:26 | |
*** rcernin has joined #openstack-cinder | 21: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 is | 21:27 |
whoami-rajat__ | http://paste.openstack.org/show/799758/ | 21:28 |
whoami-rajat__ | ok got it | 21:29 |
ade_lee | whoami-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_lee | same for synology .. | 21:34 |
*** ociuhandu has quit IRC | 21: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 IRC | 21:53 | |
*** rcernin has joined #openstack-cinder | 21:54 | |
rosmaita | whoami-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 it | 21:56 |
ade_lee | works for me | 21:57 |
ade_lee | brb | 21:57 |
*** e0ne has joined #openstack-cinder | 22:03 | |
*** hamalq has quit IRC | 22:11 | |
whoami-rajat__ | rosmaita: same thoughts, thanks for the input | 22:16 |
rosmaita | np | 22:17 |
*** e0ne has quit IRC | 22:17 | |
*** ajitha has quit IRC | 22:30 | |
openstackgerrit | Merged openstack/os-brick master: Replace md5 with oslo version https://review.opendev.org/756151 | 22:53 |
*** rcernin has quit IRC | 22:57 | |
openstackgerrit | Merged openstack/cinder stable/ussuri: Do not fail when depth is greater than rbd_max_clone_depth https://review.opendev.org/761180 | 22:59 |
*** rcernin has joined #openstack-cinder | 23:00 | |
*** spatel has joined #openstack-cinder | 23:20 | |
*** spatel has quit IRC | 23:25 | |
*** ociuhandu has joined #openstack-cinder | 23:36 | |
*** ociuhandu has quit IRC | 23:40 | |
*** tosky has quit IRC | 23:55 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!