Thursday, 2020-07-02

*** n-saito has joined #openstack-cinder00:31
*** zhanglong has joined #openstack-cinder00:39
*** jmlowe has quit IRC01:16
*** Liang__ has joined #openstack-cinder01:18
*** spatel has joined #openstack-cinder01:57
*** Lucas_Gray has quit IRC02:23
openstackgerritKazumasa Nomura proposed openstack/cinder master: Add Hitachi Block Storage Driver  https://review.opendev.org/71283202:57
*** brinzhang_ has quit IRC03:24
*** psachin has joined #openstack-cinder03:24
*** brinzhang_ has joined #openstack-cinder03:25
*** markvoelker has joined #openstack-cinder03:28
openstackgerritMerged openstack/cinder master: Disable siblings for the cindelib functional tests  https://review.opendev.org/73827303:57
*** markvoelker has quit IRC04:05
*** zhanglong has quit IRC04:08
*** markvoelker has joined #openstack-cinder04:33
*** markvoelker has quit IRC04:38
*** brinzhang0 has joined #openstack-cinder04:46
*** brinzhang_ has quit IRC04:48
*** ykarel|away is now known as ykarel04:49
*** vishalmanchanda has joined #openstack-cinder04:49
*** udesale has joined #openstack-cinder04:55
*** brinzhang_ has joined #openstack-cinder04:57
*** brinzhang0 has quit IRC05:00
*** abdysn has joined #openstack-cinder05:06
*** bhagyashris|afk is now known as bhagyashris05:11
*** m75abrams has joined #openstack-cinder05:11
*** priteau has joined #openstack-cinder05:15
*** markvoelker has joined #openstack-cinder05:26
*** markvoelker has quit IRC05:30
*** brinzhang0 has joined #openstack-cinder05:40
*** zhanglong has joined #openstack-cinder05:41
*** brinzhang_ has quit IRC05:43
*** brinzhang_ has joined #openstack-cinder05:48
*** brinzhang0 has quit IRC05:51
*** brinzhang0 has joined #openstack-cinder05:59
*** brinzhang_ has quit IRC06:02
*** brinzhang_ has joined #openstack-cinder06:09
*** brinzhang0 has quit IRC06:12
*** knomura has joined #openstack-cinder06:16
*** vishakha has joined #openstack-cinder06:17
*** spatel has quit IRC06:30
*** yaawang has quit IRC06:38
*** yaawang has joined #openstack-cinder06:44
*** brinzhang0 has joined #openstack-cinder06:52
*** brinzhang_ has quit IRC06:55
*** brinzhang_ has joined #openstack-cinder07:01
*** brinzhang0 has quit IRC07:04
*** yaawang has quit IRC07:17
*** e0ne has joined #openstack-cinder07:19
*** knomura has quit IRC07:22
*** knomura has joined #openstack-cinder07:23
*** markvoelker has joined #openstack-cinder07:27
*** bhagyashris is now known as bhagyashris|lunc07:28
*** markvoelker has quit IRC07:31
*** yaawang has joined #openstack-cinder07:31
*** Lucas_Gray has joined #openstack-cinder07:40
*** rcernin has quit IRC07:45
*** tosky has joined #openstack-cinder07:46
*** Lucas_Gray has quit IRC07:50
openstackgerritRaghavendra Tilay proposed openstack/cinder stable/train: HPE 3PAR: Support duplicated FQDN in network  https://review.opendev.org/72427107:58
openstackgerritLuigi Toscano proposed openstack/cinder stable/ussuri: Disable siblings for the cindelib functional tests  https://review.opendev.org/73897808:01
*** Lucas_Gray has joined #openstack-cinder08:07
*** markvoelker has joined #openstack-cinder08:08
*** markvoelker has quit IRC08:12
openstackgerritRajat Dhasmana proposed openstack/cinder master: Don't show host_name to non-admins  https://review.opendev.org/73252508:18
*** ociuhandu has quit IRC08:22
openstackgerritRajat Dhasmana proposed openstack/cinder master: Fix: listing volumes with filters  https://review.opendev.org/73551308:33
*** bhagyashris|lunc is now known as bhagyashris08:50
*** ociuhandu has joined #openstack-cinder08:50
*** tkajinam has quit IRC09:09
geguileowhoami-rajat: you cannot make project_id primary key because we do soft deletes09:12
*** raghavendrat has joined #openstack-cinder09:16
raghavendrathi geguileo: are you online ?09:17
*** pcaruana has quit IRC09:17
geguileoraghavendrat: yup09:18
*** zenkuro2 has joined #openstack-cinder09:18
raghavendratthis is regarding https://review.opendev.org/#/c/724271 ... from cinder/volume/driver.py removed definition of _get_oslo_driver_opts09:19
zenkuro2hello, can anybody please recommend an article on how to write unit tests for openstack cinder? beside https://wiki.openstack.org/wiki/How_To_Contribute#If_you.27re_a_developer09:19
raghavendratadded new function "get_fqdn_options" to fetch fqdn_options09:20
raghavendratwhenever you have some time, may i request you to please check. Thanks.09:20
geguileoraghavendrat: no need to be so formal when asking for a follow up review ;-)09:34
geguileoraghavendrat: I have downvoted it09:34
geguileoraghavendrat: not because it's wrong, but because it could create issues for other backports09:34
geguileoraghavendrat: the requested change should be small09:34
geguileosorry about it :-(09:34
raghavendratthanks geguileo: for quick review. Its small change; i will do it. Please don't say sorry. You are just doing your job :-)09:39
geguileoraghavendrat: ping me when you push the patch in case I miss the announcement here09:40
raghavendratsure geguileo:09:40
*** udesale_ has joined #openstack-cinder09:48
*** udesale has quit IRC09:50
*** raghavendrat has quit IRC09:56
*** priteau has quit IRC09:58
*** m75abrams has quit IRC09:58
*** markvoelker has joined #openstack-cinder10:03
*** markvoelker has quit IRC10:08
*** knomura has quit IRC10:17
openstackgerritRaghavendra Tilay proposed openstack/cinder stable/train: HPE 3PAR: Support duplicated FQDN in network  https://review.opendev.org/72427110:19
* geguileo looks at ^10:19
*** raghavendrat has joined #openstack-cinder10:20
raghavendratgeguileo: pushed changes to https://review.opendev.org/#/c/724271/10:23
geguileoraghavendrat: I've just realized that we have introduced a bug into the Kaminario bug10:25
* geguileo facepalms10:25
raghavendratohh10:25
geguileoif they had defined the unique_fqdn_network in the driver section before10:26
geguileoit would be ignored now  :-(10:26
*** brinzhang has joined #openstack-cinder10:28
openstackgerritAleksandr Fedoseev proposed openstack/cinder master: Lock quota reservation per-project to prevent races  https://review.opendev.org/69509610:29
*** n-saito has quit IRC10:29
*** brinzhang_ has quit IRC10:29
raghavendratgeguileo: can you please elaborate.10:37
geguileoraghavendrat: the Kaminario driver already had that configuration option10:37
geguileoand we could define it in the driver section of the cinder.conf file10:37
geguileoor in the backend_defaults section10:38
geguileoif we have a deployment where we defined the unique_fqdn_network as false in the driver specific section10:38
geguileoand then upgraded, that option would get ignored10:38
geguileobecause with the new code it only looks in the backend_defaults section10:39
raghavendrathhmm.10:40
*** Liang__ has quit IRC10:43
*** knomura has joined #openstack-cinder10:48
*** knomura has quit IRC10:52
*** zhanglong has quit IRC10:53
*** zhanglong has joined #openstack-cinder10:57
*** rajinir has quit IRC11:00
*** zhanglong has quit IRC11:14
raghavendratgeguileo: may i propose some logic (with my understanding).11:44
raghavendratgeguileo: may i propose some logic (with my understanding).11:44
raghavendratgeguileo: may i propose some logic (with my understanding).11:44
raghavendratextremely sorry... some issue with my keypad11:44
*** jmlowe has joined #openstack-cinder11:45
geguileoraghavendrat: I have a fix ready, I'm just having trouble with the tests11:47
raghavendratok. nice to hear that fix is ready11:48
raghavendratgeguileo: let me know, if there is something i can help11:52
*** markvoelker has joined #openstack-cinder11:58
*** markvoelker has quit IRC12:00
*** raghavendrat has quit IRC12:16
*** raghavendrat has joined #openstack-cinder12:17
openstackgerritGorka Eguileor proposed openstack/cinder master: Kaminario: Fix unique_fqdn_network option  https://review.opendev.org/73903312:41
geguileoraghavendrat: ^ that's the patch, it took me longer than expected12:41
raghavendratok geguileo:12:44
geguileoI haven't actually tested it, but it should be correct (I think)12:45
raghavendrattaking look. After problem was discovered, patch was submitted in just two hours. Thats really quick :-)12:48
openstackgerritGorka Eguileor proposed openstack/cinder master: Kaminario: Fix unique_fqdn_network option  https://review.opendev.org/73903312:49
*** Abdallahyas has joined #openstack-cinder12:50
*** abdysn has quit IRC12:50
*** Lucas_Gray has quit IRC12:57
*** Abdallahyas has quit IRC13:01
*** abdysn has joined #openstack-cinder13:01
raghavendratgeguileo: at a first glance, patch looks fine. Let me perform some manual test (from HPE 3PAR side). Will inform once its done.13:04
geguileoraghavendrat: thank you!!13:04
*** Lucas_Gray has joined #openstack-cinder13:07
*** raghavendrat has quit IRC13:09
*** zhanglong has joined #openstack-cinder13:11
*** zhanglong has quit IRC13:18
*** zhanglong has joined #openstack-cinder13:19
*** zhanglong has quit IRC13:24
*** hoonetorg has quit IRC13:26
openstackgerritBrian Rosmaita proposed openstack/cinder stable/ussuri: Default volume_type set too early  https://review.opendev.org/73879113:28
*** hoonetorg has joined #openstack-cinder13:31
*** dave-mccowan has quit IRC13:40
*** Xuchu has joined #openstack-cinder13:48
*** vishalmanchanda has quit IRC13:48
*** psachin has quit IRC13:54
*** abdysn has quit IRC13:59
*** bhagyashris is now known as bhagyashris|afk14:15
*** zenkuro2 has quit IRC14:30
*** priteau has joined #openstack-cinder14:51
*** vesper has joined #openstack-cinder15:13
*** vesper11 has quit IRC15:14
*** redrobot has quit IRC15:14
*** irclogbot_0 has quit IRC15:14
*** mnasiadka has quit IRC15:14
*** mnasiadka_ has joined #openstack-cinder15:14
*** dosaboy has quit IRC15:14
*** _pewp_ has quit IRC15:14
*** lifeless has quit IRC15:14
*** tonyb has quit IRC15:15
*** _pewp_ has joined #openstack-cinder15:15
*** elod has quit IRC15:15
*** elod has joined #openstack-cinder15:16
*** tonyb has joined #openstack-cinder15:16
*** ykarel is now known as ykarel|away15:17
*** irclogbot_1 has joined #openstack-cinder15:18
*** dosaboy has joined #openstack-cinder15:19
*** lifeless has joined #openstack-cinder15:20
*** knomura has joined #openstack-cinder15:30
*** knomura has quit IRC15:35
*** e0ne has quit IRC15:44
openstackgerritLucio Seki proposed openstack/cinder master: Generic NFS: skip qemu-img resize if volume is attached  https://review.opendev.org/73907915:48
*** hamalq has joined #openstack-cinder15:57
*** hamalq_ has joined #openstack-cinder15:58
*** hamalq has quit IRC16:02
*** Xuchu has quit IRC16:06
*** udesale_ has quit IRC16:19
*** ociuhandu_ has joined #openstack-cinder16:33
*** ociuhandu has quit IRC16:36
*** ociuhandu_ has quit IRC16:38
*** Lucas_Gray has quit IRC16:40
openstackgerritLuigi Toscano proposed openstack/cinder master: DNM test the coverage link  https://review.opendev.org/73910016:45
openstackgerritLuigi Toscano proposed openstack/cinder master: DNM test the coverage link  https://review.opendev.org/73910016:57
openstackgerritLuigi Toscano proposed openstack/cinder master: DNM test the coverage link  https://review.opendev.org/73910017:02
*** ircuser-1 has joined #openstack-cinder17:29
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: Fix volume retype with AZ  https://review.opendev.org/73640417:46
gansorosmaita: hi! could you please take a look? https://review.opendev.org/#/c/738943 waiting for the last +2 =)17:54
*** KeithMnemonic has joined #openstack-cinder17:58
*** zenkuro has quit IRC18:00
KeithMnemonicrosmaita, this has passed now, https://review.opendev.org/#/c/736415/ could someone please review so this CVE for FlexOS can get resolved . Thanks in advance18:01
*** knomura has joined #openstack-cinder18:12
*** knomura has quit IRC18:17
*** ociuhandu has joined #openstack-cinder18:34
*** ociuhandu has quit IRC18:39
*** brinzhang_ has joined #openstack-cinder18:56
*** brinzhang has quit IRC19:00
KeithMnemonicthanks rosmaita! Have a Happy (and safe ) 4th19:30
rosmaitaKeithMnemonic: thanks, you too19:30
*** vishakha has quit IRC19:33
*** dave-mccowan has joined #openstack-cinder19:42
gansorosmaita: thanks!19:46
*** dave-mccowan has quit IRC20:25
*** ociuhandu has joined #openstack-cinder20:35
openstackgerritMerged openstack/cinder stable/ussuri: Fix cross-project incremental backups  https://review.opendev.org/73894320:39
*** ociuhandu has quit IRC20:40
*** zhanglong has joined #openstack-cinder20:40
*** priteau has quit IRC20:44
*** ociuhandu has joined #openstack-cinder20:45
*** ociuhandu has quit IRC20:49
*** priteau has joined #openstack-cinder20:52
*** priteau has quit IRC20:52
*** Lucas_Gray has joined #openstack-cinder20:57
*** ociuhandu has joined #openstack-cinder21:14
*** gnufied has quit IRC21:16
*** ociuhandu has quit IRC21:20
*** zhanglong has quit IRC21:47
*** josephillips has joined #openstack-cinder21:52
openstackgerritMerged openstack/os-brick stable/rocky: Fix "Remove VxFlex OS credentials" regression  https://review.opendev.org/73641521:55
*** zhanglong has joined #openstack-cinder21:58
*** zhanglong has quit IRC22:28
*** zhanglong has joined #openstack-cinder22:29
*** zhanglong has quit IRC22:39
openstackgerritRodrigo Barbieri proposed openstack/cinder stable/train: Fix cross-project incremental backups  https://review.opendev.org/73915422:41
*** rcernin has joined #openstack-cinder22:42
*** tosky has quit IRC22:54
*** tkajinam has joined #openstack-cinder22:55
*** markvoelker has joined #openstack-cinder22:58
*** Wryhder has joined #openstack-cinder23:02
*** markvoelker has quit IRC23:02
*** Lucas_Gray has quit IRC23:03
*** Wryhder is now known as Lucas_Gray23:03
*** hamalq_ has quit IRC23:52
*** penick has joined #openstack-cinder23:58

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