Thursday, 2019-12-19

*** brinzhang has joined #openstack-cinder00:40
*** Kuirong has quit IRC00:41
*** brinzhang_ has joined #openstack-cinder00:42
*** brinzhang_ has quit IRC00:44
*** brinzhang_ has joined #openstack-cinder00:45
*** brinzhang has quit IRC00:47
*** brinzhang has joined #openstack-cinder00:47
*** brinzhang has quit IRC00:49
*** brinzhang has joined #openstack-cinder00:49
*** brinzhang has quit IRC00:49
*** brinzhang has joined #openstack-cinder00:50
*** brinzhang_ has quit IRC00:50
*** brinzhang_ has joined #openstack-cinder00:51
*** brinzhang has quit IRC00:55
*** Liang__ has joined #openstack-cinder01:01
*** brinzhang has joined #openstack-cinder01:05
*** brinzhang has quit IRC01:06
*** brinzhang_ has quit IRC01:09
*** yedongcan has joined #openstack-cinder02:25
*** dave-mccowan has quit IRC02:26
*** irclogbot_1 has quit IRC02:56
*** lvbin01 has joined #openstack-cinder03:07
*** irclogbot_3 has joined #openstack-cinder03:08
*** irclogbot_3 has quit IRC03:09
*** irclogbot_2 has joined #openstack-cinder03:14
*** dave-mccowan has joined #openstack-cinder03:20
*** psachin has joined #openstack-cinder03:41
openstackgerritDavid Vallee Delisle proposed openstack/os-brick master: Read mounts from /proc/mounts instead of running mount  https://review.opendev.org/69986103:48
openstackgerritDavid Vallee Delisle proposed openstack/os-brick master: Read mounts from /proc/mounts instead of running mount  https://review.opendev.org/69986103:51
openstackgerritfutaotao proposed openstack/cinder master: Huawei Cinder Driver Support Dorado V6 Storage.(iSCSI, FC)  https://review.opendev.org/69990903:57
*** whfnst has joined #openstack-cinder04:08
*** ganso has quit IRC04:22
*** dosaboy has quit IRC04:23
*** ociuhandu has joined #openstack-cinder04:32
openstackgerritShatadru Bandyopadhyay proposed openstack/cinder master: doc: clarify usage of use_multipath_for_image_xfer parameter  https://review.opendev.org/69955904:34
*** n-saito has joined #openstack-cinder04:35
*** ociuhandu has quit IRC04:37
openstackgerritDavid Vallee Delisle proposed openstack/os-brick master: Read mounts from /proc/mounts instead of running mount  https://review.opendev.org/69986104:43
*** whoami-rajat has joined #openstack-cinder04:47
*** bhagyashris has joined #openstack-cinder04:59
*** Lucas_Gray has joined #openstack-cinder05:01
*** whoami-rajat has quit IRC05:03
*** whoami-rajat__ has joined #openstack-cinder05:04
*** ganso has joined #openstack-cinder05:04
*** Lucas_Gray has quit IRC05:21
*** n-saito has quit IRC05:27
*** psachin has quit IRC05:36
*** dave-mccowan has quit IRC05:39
*** pcaruana has joined #openstack-cinder05:43
*** psachin has joined #openstack-cinder05:55
*** bhagyashris has quit IRC06:00
*** bhagyashris has joined #openstack-cinder06:06
*** rcernin_ has joined #openstack-cinder06:09
*** rcernin has quit IRC06:12
*** whoami-rajat__ has quit IRC06:28
*** bhagyashris has quit IRC06:31
*** bhagyashris has joined #openstack-cinder06:38
*** lpetrut has joined #openstack-cinder07:06
*** rcernin_ has quit IRC07:09
*** ganso has quit IRC07:10
*** ganso has joined #openstack-cinder07:12
*** lvbin01 has quit IRC07:12
*** lvbin01 has joined #openstack-cinder07:13
*** awalende has joined #openstack-cinder07:15
*** jawad_axd has joined #openstack-cinder07:17
*** awalende has quit IRC07:20
*** whoami-rajat has joined #openstack-cinder07:26
*** psachin has quit IRC07:44
*** irclogbot_2 has quit IRC07:44
*** irclogbot_2 has joined #openstack-cinder07:44
*** psachin has joined #openstack-cinder07:45
*** whfnst has quit IRC07:45
*** whfnst has joined #openstack-cinder07:56
*** yedongcan has quit IRC08:10
*** yedongcan has joined #openstack-cinder08:11
*** tosky has joined #openstack-cinder08:14
*** tkajinam has quit IRC08:15
*** bhagyashris has quit IRC08:24
*** yaawang has quit IRC09:07
*** bhagyashris has joined #openstack-cinder09:16
*** rcernin_ has joined #openstack-cinder09:20
*** Liang__ has quit IRC09:23
*** rcernin_ has quit IRC09:29
*** sahid has joined #openstack-cinder09:31
*** e0ne has joined #openstack-cinder09:45
*** davidsha has joined #openstack-cinder09:47
*** yaawang has joined #openstack-cinder09:51
*** tosky_ has joined #openstack-cinder10:07
*** tosky is now known as Guest9772710:07
*** tosky_ is now known as tosky10:07
*** Guest97727 has quit IRC10:08
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Fix configuration options  https://review.opendev.org/69920310:22
geguileosmcginnis: rosmaita ^ I had another pass at the patch, and now I'm not using any internal oslo.config variables or anything, only public interfaces  O:-)10:24
geguileosmcginnis: thanks for reminding me that using internal stuff just 'cause it's easy is NEVER a good idea  XD10:24
*** awalende has joined #openstack-cinder11:31
*** icey has joined #openstack-cinder11:35
*** bhagyashris_ has joined #openstack-cinder12:21
*** bhagyashris has quit IRC12:23
*** ociuhandu has joined #openstack-cinder12:24
*** martinkennelly has joined #openstack-cinder12:24
*** bhagyashris_ has quit IRC12:27
*** bhagyashris has joined #openstack-cinder12:28
*** Liang__ has joined #openstack-cinder12:47
*** Liang__ has quit IRC12:52
*** yedongcan has left #openstack-cinder12:59
*** bhagyashris has quit IRC12:59
rosmaitageguileo: that fix looks much better!13:00
geguileorosmaita: yeah, smaller and without any weird stuff13:00
geguileoor at least less of the weird stuff  XD13:00
rosmaitageguileo: ++ to no weird stuff13:00
*** ociuhandu has quit IRC13:01
geguileonothing like reviews to keep me honest and non-lazy  XD13:01
geguileoalthough this solution took a lot less efort than the previous 213:01
rosmaita"virtue is its own reward"13:01
smcginnis:)13:10
*** awalende has quit IRC13:11
*** awalende has joined #openstack-cinder13:13
*** ociuhandu has joined #openstack-cinder13:23
openstackgerritSean McGinnis proposed openstack/cinder master: Remove Sheepdog Driver  https://review.opendev.org/69998513:26
openstackgerritGorka Eguileor proposed openstack/cinderlib stable/train: Fix configuration options  https://review.opendev.org/69998613:31
openstackgerritSean McGinnis proposed openstack/os-brick master: Remove Sheepdog connector  https://review.opendev.org/69998713:35
smcginnisrosmaita: We didn't explicitly deprecate the connector for sheepdog, but I think we can consider the Cinder driver deprecation and removal as enough. ^^13:36
rosmaitasmcginnis: i agree13:37
smcginnisNice to get things cleaned up. ;)13:38
openstackgerritSean McGinnis proposed openstack/os-brick master: Remove Sheepdog connector  https://review.opendev.org/69998713:44
toskysmcginnis, rosmaita : sooo, cinder-tempest-plugin; the fix just merged fix an important issue which prevented its usage and I think it's important for the consumers (starting from RDO)13:49
rosmaitatosky: which fix is that?13:49
toskyrosmaita: the one you merged yesterday13:49
rosmaitaok, release patch is up13:49
rosmaitahttps://review.opendev.org/#/c/699774/13:50
toskyyes, I was trying to answer to the objection13:50
rosmaitaoh, sorry, i didn't look at it13:50
rosmaitasmcginnis: what do you suggest? should i propose it in deliverables/train for 0.3.1, or deliverables/ussuri for 0.4.0 ?13:52
*** eharney has joined #openstack-cinder13:55
rosmaitasmcginnis: i thought since the plugin is branchless, it didn't matter13:57
*** KeithMnemonic has joined #openstack-cinder13:58
toskylet's say that the tempest plugins release policy has not been exactly stable over time13:58
openstackgerritGorka Eguileor proposed openstack/cinderlib master: WIP: Figure out why XtremIO cI fails  https://review.opendev.org/69999314:04
openstackgerritShatadru Bandyopadhyay proposed openstack/cinder master: doc: clarify usage of use_multipath_for_image_xfer parameter  https://review.opendev.org/69955914:14
*** tosky_ has joined #openstack-cinder14:23
*** tosky is now known as Guest2789014:23
*** tosky_ is now known as tosky14:23
*** enriquetaso has joined #openstack-cinder14:23
*** Guest27890 has quit IRC14:26
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Docs -  corrections and improvements  https://review.opendev.org/68956414:26
*** KeithMnemonic has quit IRC14:31
*** dosaboy has joined #openstack-cinder14:52
*** dviroel has joined #openstack-cinder14:58
*** andrebeltrami has joined #openstack-cinder15:00
*** do3meli has joined #openstack-cinder15:00
rosmaitasmcginnis: hate to be a PITA, but about https://review.opendev.org/#/c/699774/ -- what do you prefer?  should i propose it in deliverables/train for 0.3.1, or deliverables/ussuri for 0.4.0 ? i don't have a preference, just need to get a release out15:02
smcginnisrosmaita: I guess it's good as is.15:04
smcginnisIt's kind of odd how RDO appears to be using those.15:04
smcginnisThe only reason we even started tagging tempest-plugins was to have just a marker at the end of the cycle to indicate where things were at the end of the cycle.15:05
smcginnisBecause since those are branchless, they should just be picked up and used wherever they are unless you need to make sure you are using the version that was meant for whatever stable branch and tempest at the time of that branch.15:06
smcginnisNow it appears they need regular releases, which no one intended.15:06
rosmaitayeah, i guess we should have a discussion with RDO about how these are being consumed, i guess it applies to all the tempest plugins, not just us15:08
toskysmcginnis: afaik and iirc, the reason why enforced tagging tempest plugins was to make sure that at least one release was compatibile with the corresponding tempest tag which support a certain release15:08
toskyand I've been disagreeing with the specific cycle-automatic since the beginning15:08
toskyI understand the problem, but cycle-automatic shouldn't disallow new releases when needed15:08
toskyit was introduced for, uhm, maintainers who didn't care for the tempest plugins15:09
smcginnisWell, what should happen is whatever is in the repo gets used.15:09
rosmaitado we need to change to the intermediate release model, or is that not quite right either15:09
toskyI think the solution would just allow intermediate releases for cycle-automatic, making it a special case of cycle-with-intermediates15:10
smcginnisI'll put it on the release team meeting agenda.15:10
smcginnistosky: It does. We allow intermediate releases. It's just not what was original intended with those plugins.15:10
toskyand I've always disagreed with that :)15:11
toskyfor sahara-tests, for example, I've always released at least once for cycle, and more if needed (bugfixes)15:11
rosmaitayeah, it does make sense to allow for bugfixes!15:12
toskyit had to follow the changes in the workflow, so it went from independent, to cycle-with-intermediary, to cycle-automatic15:12
smcginnisBut they weren't intended to be deliverables. You clone the tempest plugin repo and run the tests.15:12
smcginnisRDO is the only one I'm aware of that decided it would work with released version of them.15:12
toskyyes and no - even tempest tags specific versions15:12
smcginnisOnly as a checkpoint to show what was from a given cycle.15:12
rosmaitaso for now (because i am a bear of very simple brain), for cinder-tempest-plugin, that release patch is OK?15:13
toskybut it has intermediate releases as well15:13
smcginnisAt least (again), that was the intial reason for it.15:13
smcginnisrosmaita: Yep, it's good.15:13
rosmaitacool, ty15:13
*** jawad_axd has quit IRC15:20
*** jawad_axd has joined #openstack-cinder15:21
*** awalende has quit IRC15:22
*** awalende has joined #openstack-cinder15:23
*** jawad_ax_ has joined #openstack-cinder15:23
*** awalende_ has joined #openstack-cinder15:25
*** jawad_axd has quit IRC15:26
*** awalende has quit IRC15:27
*** jawad_ax_ has quit IRC15:28
*** awalende_ has quit IRC15:29
*** ociuhandu has quit IRC15:41
*** awalende has joined #openstack-cinder15:43
*** hoonetorg has quit IRC15:45
*** do3meli has quit IRC16:01
*** jawad_axd has joined #openstack-cinder16:22
*** jawad_axd has quit IRC16:25
*** jawad_axd has joined #openstack-cinder16:26
*** jawad_axd has quit IRC16:27
*** lpetrut has quit IRC16:28
*** sahid has quit IRC16:39
*** hoonetorg has joined #openstack-cinder16:42
*** icey has quit IRC16:47
openstackgerritSean McGinnis proposed openstack/cinder master: Enable flake8-logging-format extension  https://review.opendev.org/70002616:51
*** dosaboy_ has joined #openstack-cinder16:52
*** dave-mccowan has joined #openstack-cinder16:59
openstackgerritMerged openstack/cinderlib master: Fix configuration options  https://review.opendev.org/69920317:01
*** dave-mccowan has quit IRC17:05
*** awalende has quit IRC17:06
geguileosmcginnis: rosmaita ^ merged in master, the train backport is proposed here https://review.opendev.org/#/c/699986/17:09
smcginnisThanks17:09
*** whoami-rajat has quit IRC17:09
rosmaitageguileo: i have the release patch ready, will submit as soon as the backport merges17:10
geguileorosmaita: smcginnis thanks17:15
openstackgerritJay Bryant proposed openstack/cinder master: Remove ProphetStor Flexvisor Driver  https://review.opendev.org/70003417:19
*** abhishekk has joined #openstack-cinder17:24
*** awalende has joined #openstack-cinder17:45
openstackgerritDavid Vallee Delisle proposed openstack/os-brick master: Read mounts from /proc/mounts instead of running mount  https://review.opendev.org/69986117:46
*** lpetrut has joined #openstack-cinder17:49
*** awalende has quit IRC17:50
*** abhishekk has quit IRC17:54
*** martinkennelly has quit IRC17:57
openstackgerritJay Bryant proposed openstack/cinder master: Remove Nimble Storage Driver  https://review.opendev.org/70004318:00
*** udesale has joined #openstack-cinder18:03
*** e0ne has quit IRC18:03
*** davidsha has quit IRC18:05
*** dosaboy_ has quit IRC18:13
*** dosaboy has quit IRC18:13
*** dosaboy has joined #openstack-cinder18:14
*** lvbin02 has joined #openstack-cinder18:22
*** lvbin01 has quit IRC18:25
*** lvbin02 is now known as lvbin0118:25
*** lpetrut has quit IRC18:26
openstackgerritMerged openstack/cinder master: DS8k Cinder Driver support Python3  https://review.opendev.org/69796018:33
*** udesale has quit IRC18:37
*** lpetrut has joined #openstack-cinder18:37
openstackgerritMerged openstack/cinderlib stable/train: Fix configuration options  https://review.opendev.org/69998618:46
*** enriquetaso has quit IRC18:50
*** enriquetaso has joined #openstack-cinder18:51
openstackgerritSean McGinnis proposed openstack/cinder master: Raise hacking version to 2.0.0  https://review.opendev.org/70005218:54
rosmaitageguileo: https://review.opendev.org/#/c/700053/18:55
geguileorosmaita: thanks!! hash is correct18:57
rosmaitagreat18:57
*** andrebeltrami has quit IRC18:58
*** psachin has quit IRC19:28
*** spotz has quit IRC19:44
*** e0ne has joined #openstack-cinder19:55
*** e0ne has quit IRC20:00
*** hemna_ has quit IRC20:05
*** hemna has joined #openstack-cinder20:05
*** irclogbot_2 has quit IRC20:06
*** irclogbot_2 has joined #openstack-cinder20:09
*** whfnst has quit IRC20:15
*** enriquetaso has quit IRC20:16
*** enriquetaso has joined #openstack-cinder20:31
openstackgerritDavid Vallee Delisle proposed openstack/os-brick master: Read mounts from /proc/mounts instead of running mount  https://review.opendev.org/69986120:34
openstackgerritSean McGinnis proposed openstack/cinder master: Raise hacking version to 2.0.0  https://review.opendev.org/70005220:59
*** awalende has joined #openstack-cinder21:08
*** eharney has quit IRC21:08
*** awalende has quit IRC21:12
*** enriquetaso has quit IRC21:20
*** pcaruana has quit IRC21:23
*** e0ne has joined #openstack-cinder21:42
*** ociuhandu has joined #openstack-cinder21:50
*** e0ne has quit IRC22:07
*** dviroel has quit IRC22:48
openstackgerritMerged openstack/cinder master: doc: clarify usage of use_multipath_for_image_xfer parameter  https://review.opendev.org/69955922:57
*** ociuhandu has quit IRC23:03
*** tkajinam has joined #openstack-cinder23:04
*** Liang__ has joined #openstack-cinder23:08
*** tkajinam has quit IRC23:17
*** tkajinam has joined #openstack-cinder23:17
*** tosky has quit IRC23:22
*** rosmaita has left #openstack-cinder23:30
*** rcernin has joined #openstack-cinder23:39

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!