Thursday, 2019-09-12

*** markvoelker has quit IRC00:29
*** markvoelker has joined #openstack-cinder00:29
hemna_anyone here mess w/ helm?00:32
*** laurent\ has joined #openstack-cinder00:34
*** hemna has quit IRC00:36
*** henriqueof has joined #openstack-cinder00:45
*** henriqueof1 has quit IRC00:46
*** zhengMa has joined #openstack-cinder00:50
*** zhengMa has left #openstack-cinder00:50
*** henriqueof1 has joined #openstack-cinder00:56
*** henriqueof has quit IRC00:57
*** markvoelker has quit IRC01:03
*** markvoelker has joined #openstack-cinder01:04
*** henriqueof1 has quit IRC01:05
*** henriqueof has joined #openstack-cinder01:05
*** hemna has joined #openstack-cinder01:09
*** pcaruana has quit IRC01:09
*** jgriffith has quit IRC01:10
*** lseki has quit IRC01:10
*** brinzhang_ has joined #openstack-cinder01:13
*** brinzhang has quit IRC01:17
openstackgerritzhufl proposed openstack/cinder master: [api-ref]control_location should be mandatory  https://review.opendev.org/67786201:38
*** dklyle has quit IRC01:42
*** david-lyle has joined #openstack-cinder01:42
*** hemna has quit IRC01:43
*** david-lyle has quit IRC01:46
*** dklyle has joined #openstack-cinder01:47
*** whfnst has joined #openstack-cinder01:51
openstackgerritweibin proposed openstack/cinder master: Add support for using ceph RBD ereasure code  https://review.opendev.org/68118701:58
*** hemna has joined #openstack-cinder02:43
*** carloss has joined #openstack-cinder02:56
*** hemna has quit IRC03:16
*** Garyx_ has joined #openstack-cinder03:33
*** Garyx has quit IRC03:33
*** hemna has joined #openstack-cinder03:48
openstackgerritQian Mingyue proposed openstack/cinder master: Move the constants to constants.py  https://review.opendev.org/68163803:50
*** KeithMnemonic has quit IRC04:04
openstackgerritMerged openstack/cinder stable/queens: RBD: remove redundant exception log to reduce noise  https://review.opendev.org/68083804:17
*** spatel has joined #openstack-cinder04:17
*** hemna has quit IRC04:23
*** sapd1_x has joined #openstack-cinder04:23
*** udesale has joined #openstack-cinder04:31
*** pcaruana has joined #openstack-cinder04:32
*** Luzi has joined #openstack-cinder04:46
*** markvoelker has quit IRC04:48
*** dave-mccowan has quit IRC04:57
*** pcaruana has quit IRC04:58
*** spatel has quit IRC04:58
*** pots has joined #openstack-cinder05:10
*** sapd1_x has quit IRC05:13
*** hemna has joined #openstack-cinder05:15
*** vishalmanchanda has joined #openstack-cinder05:20
*** brinzhang has joined #openstack-cinder05:48
*** brinzhang_ has quit IRC05:51
openstackgerrityenai proposed openstack/cinder master: Fix signature_verified not matched  https://review.opendev.org/65566605:56
*** tkajinam has quit IRC06:00
*** carloss has quit IRC06:05
*** markvoelker has joined #openstack-cinder06:06
*** tkajinam has joined #openstack-cinder06:06
*** pcaruana has joined #openstack-cinder06:08
*** markvoelker has quit IRC06:11
*** hemna has quit IRC06:18
*** pcaruana has quit IRC06:41
*** sahid has joined #openstack-cinder06:46
*** tkajinam_ has joined #openstack-cinder06:53
*** tkajinam has quit IRC06:56
*** spsurya has joined #openstack-cinder07:07
*** pcaruana has joined #openstack-cinder07:11
*** trident has quit IRC07:15
*** Zettastor has joined #openstack-cinder07:17
*** awalende has joined #openstack-cinder07:17
Zettastorif I plan to submit cinder driver to master branch, must I use the master branch keystone to setup devstack environment?07:18
Zettastorcan I use stein branch keystone for submitting cinder driver codes to master?07:19
*** trident has joined #openstack-cinder07:24
*** trident has quit IRC07:28
openstackgerritSneha Rai proposed openstack/cinder master: HPE 3PAR: Updated supported operations for multiattach feature in the driver document.  https://review.opendev.org/68165507:37
*** trident has joined #openstack-cinder07:37
Zettastoropenstackgerrit: could you share you devstack local.conf to me for submitting codes to master branch?07:39
*** e0ne has joined #openstack-cinder07:57
*** e0ne has quit IRC08:08
*** tkajinam_ has quit IRC08:09
*** hemna has joined #openstack-cinder08:14
*** whoami-rajat has joined #openstack-cinder08:15
*** markvoelker has joined #openstack-cinder08:16
*** threestrands has quit IRC08:16
*** ag-47 has joined #openstack-cinder08:20
*** markvoelker has quit IRC08:20
ag-47Hello everybody, is there any chance to backport the commit that resolve the increment backup with ceph  (https://opendev.org/openstack/cinder/commit/5018727f8e18992e64c55b84f0d5b39ae7c70b32) to  version 12.0.8 (queens)08:22
whoami-rajatag-47: IMO feature backport isn't allowed08:26
ag-47is there anyway to overcome the probleme without upgrading ?08:32
whoami-rajatag-47:  you can make the changes manually to your local code. I think this is the only alternative.08:33
ag-47whoami-rajat: Thank you08:35
*** sapd1 has quit IRC08:44
*** hemna has quit IRC08:47
*** raghavendrat has joined #openstack-cinder08:55
raghavendratis whoami-rajat: online?08:58
whoami-rajatraghavendrat: Hi08:59
*** ociuhandu has joined #openstack-cinder09:00
raghavendratthis is regarding adding Peer Persistence support in HPE 3PAR cinder driver.09:02
raghavendratdriver.hpe_3par=complete09:02
whoami-rajatsorry i'm not getting much time for reviews, will take look at the peer persistence patch09:03
*** e0ne has joined #openstack-cinder09:03
*** e0ne has quit IRC09:03
*** udesale has quit IRC09:04
*** ociuhandu has quit IRC09:04
raghavendratok. i think i was not able to explain my query well.09:06
*** e0ne has joined #openstack-cinder09:07
*** udesale has joined #openstack-cinder09:07
whoami-rajatraghavendrat: oh, i didn't get the query09:11
*** FlorianFa has quit IRC09:13
*** e0ne has quit IRC09:15
*** hemna has joined #openstack-cinder09:15
*** lpetrut has joined #openstack-cinder09:16
*** rcernin has quit IRC09:16
openstackgerritRaghavendra Tilay proposed openstack/cinder master: 3PAR: Add Peer Persistence support  https://review.opendev.org/67794509:19
*** ociuhandu has joined #openstack-cinder09:19
*** e0ne has joined #openstack-cinder09:19
*** ociuhandu has quit IRC09:21
*** e0ne has quit IRC09:21
*** ociuhandu has joined #openstack-cinder09:21
*** lpetrut has quit IRC09:23
*** lpetrut has joined #openstack-cinder09:24
*** e0ne has joined #openstack-cinder09:39
*** Zettastor has quit IRC09:42
*** udesale has quit IRC09:49
*** udesale has joined #openstack-cinder09:50
*** hemna has quit IRC09:50
openstackgerritMerged openstack/cinder stable/stein: 3PAR: Add config for NSP single path attach  https://review.opendev.org/67780009:57
*** peschk_l has joined #openstack-cinder10:01
*** lpetrut has quit IRC10:17
*** hemna has joined #openstack-cinder10:19
*** lpetrut has joined #openstack-cinder10:19
*** markvoelker has joined #openstack-cinder10:26
*** markvoelker has quit IRC10:32
*** carloss has joined #openstack-cinder10:38
*** hemna has quit IRC10:52
*** sapd1_x has joined #openstack-cinder10:56
*** hemna has joined #openstack-cinder11:25
raghavendrathi hemna: whenever you get time, may i request you to have a look at https://review.opendev.org/#/c/677945/11:38
*** n-saito has quit IRC11:41
*** pots has quit IRC11:41
*** pots has joined #openstack-cinder11:42
*** whoami-rajat has quit IRC11:54
*** hemna has quit IRC12:00
*** markvoelker has joined #openstack-cinder12:01
raghavendrathi e0ne: whenever you get time, can you please have a look at  https://review.opendev.org/#/c/677945/12:34
*** eharney has joined #openstack-cinder12:36
raghavendrathi eharney:12:40
e0neraghavendrat: I'll take a look on it once it passes CI12:41
raghavendratthanks e0ne:12:42
*** mriedem has joined #openstack-cinder12:43
*** e0ne has quit IRC12:44
*** e0ne has joined #openstack-cinder12:44
*** ociuhandu has quit IRC12:50
raghavendrateharney: whenever you get time, may I request you to have a look at https://review.opendev.org/#/c/677945/12:52
*** lseki has joined #openstack-cinder12:52
*** mriedem has quit IRC12:54
openstackgerritEric Harney proposed openstack/cinder stable/stein: Fix up test_snapshot notify tests  https://review.opendev.org/68173712:56
*** ociuhandu has joined #openstack-cinder12:58
*** mriedem has joined #openstack-cinder13:00
*** _erlon_ has quit IRC13:01
*** _erlon_ has joined #openstack-cinder13:02
*** jmlowe has quit IRC13:03
*** raghavendrat has left #openstack-cinder13:03
*** spatel has joined #openstack-cinder13:13
*** spatel has quit IRC13:15
*** jmlowe has joined #openstack-cinder13:24
*** enriquetaso has joined #openstack-cinder13:25
*** jmlowe has quit IRC13:25
*** Luzi has quit IRC13:28
abishophi cores, please consider https://review.opendev.org/681515 (a backport)13:29
abishopwe have a customer who hit this13:29
*** pcaruana has quit IRC13:30
*** jmlowe has joined #openstack-cinder13:36
*** markvoelker has quit IRC13:40
*** ociuhandu has quit IRC13:41
*** markvoelker has joined #openstack-cinder13:42
*** KeithMnemonic has joined #openstack-cinder13:53
*** dave-mccowan has joined #openstack-cinder13:54
*** tkajinam has joined #openstack-cinder13:55
*** hemna has joined #openstack-cinder13:55
rosmaitasmcginnis: can you take a look at the cinderclient release patch? https://review.opendev.org/#/c/681604/13:58
smcginnisBombs away13:59
rosmaita:D13:59
*** ociuhandu has joined #openstack-cinder14:00
*** whoami-rajat has joined #openstack-cinder14:01
*** pcaruana has joined #openstack-cinder14:08
openstackgerritOpenStack Release Bot proposed openstack/python-cinderclient stable/train: Update .gitreview for stable/train  https://review.opendev.org/68176014:22
openstackgerritOpenStack Release Bot proposed openstack/python-cinderclient stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68176114:22
openstackgerritOpenStack Release Bot proposed openstack/python-cinderclient master: Update master for stable/train  https://review.opendev.org/68176214:22
*** mriedem has quit IRC14:23
*** henriqueof1 has joined #openstack-cinder14:27
*** hemna has quit IRC14:29
*** henriqueof has quit IRC14:30
*** mriedem has joined #openstack-cinder14:32
jungleboyjrosmaita:  Ugh, I meant to put my vote on that.14:41
jungleboyjSorry I didn't.  Last night got away from me.14:41
*** dave-mccowan has quit IRC14:41
rosmaitajungleboyj: np, i figured you were busy14:41
jungleboyj:-(  Yeah, that has been the theme lately.14:42
openstackgerritRajat Dhasmana proposed openstack/cinder master: Untyped to Default Volume Type  https://review.opendev.org/63918014:44
*** abishop has quit IRC14:45
eharneyi think tempest may no longer pass on stable/ocata due to https://review.opendev.org/#/c/603339/14:46
rosmaitajungleboyj: smcginnis: is it OK for me to ninja-approve the bot-generated patches for gitreview/upper-constraints on stable/train ?14:47
eharney(seen on https://review.opendev.org/#/c/681477/ )14:47
smcginnisrosmaita: Yeah, that's been my policy.14:48
rosmaitaok, cool14:48
jungleboyjrosmaita:  Yeah, that is fine.14:48
smcginniseharney: Well, that's annoying to enforce schema validation on a branch that didn't validate schema. :/14:48
eharneyand it looks like tempest changes like that don't gate on cinder ocata, so... hrm.  do we need to just pin tempest on that branch?14:48
rosmaitawhy can't they mind their own business?14:48
smcginniseharney: I'm not sure if we are able to pin tempest.14:49
rosmaitayeah, it's "unversioned" because it is always backward compatible14:49
whoami-rajatrosmaita: is it possible to update the extra '=' here https://review.opendev.org/#/c/681762/1/releasenotes/source/train.rst14:49
rosmaita:P14:49
eharneyyeah..14:49
smcginnisI would say we propose a revert of that patch. Tempest shouldn't be doing that.14:49
smcginnisAt least until all supported cinder branches have our own schema validation.14:50
smcginnisBut if we are doing our own schema validation, I don't see why tempest needs to know or care.14:50
rosmaitawhoami-rajat: is it causing a failure?14:50
jungleboyjsmcginnis: ++14:51
whoami-rajatrosmaita: i don't think so14:51
rosmaitai think you are right that it is incorrect RST14:52
jungleboyjrosmaita:  It is.14:52
rosmaitajungleboyj: smcginnis: should we fix ourselves, or ask release team to put up a revision?14:53
rosmaitawhoami-rajat: ^^14:53
jungleboyjrosmaita:  I would think you can push up a revision.14:53
smcginnisrosmaita: For now, I think we need to just fix ourselves. I will have to track down the automation script that generates that so future ones are generated correctly.14:53
rosmaitaok, whoami-rajat: do you want to edit that patch?14:54
rosmaita(then i can approve it)14:54
whoami-rajatrosmaita: sure14:54
rosmaitawhoami-rajat: please also fix brick-cinderclient-ext patch: https://review.opendev.org/#/c/68131214:55
whoami-rajatsmcginnis: jungleboyj rosmaita the automation script has posted same mistake for every module https://review.opendev.org/#/c/681675/1/releasenotes/source/train.rst14:55
jungleboyj:-(14:56
openstackgerritRajat Dhasmana proposed openstack/python-brick-cinderclient-ext master: Update master for stable/train  https://review.opendev.org/68131214:56
smcginnishttps://opendev.org/openstack/project-config/src/branch/master/roles/copy-release-tools-scripts/files/release-tools/add_release_note_page.sh#L5214:56
jungleboyj (ノ ゜Д゜)ノ ︵ ┻━┻14:56
openstackgerritRajat Dhasmana proposed openstack/python-cinderclient master: Update master for stable/train  https://review.opendev.org/68176214:56
whoami-rajatrosmaita: done ^^14:57
rosmaitawhoami-rajat: also https://review.opendev.org/#/c/680327/114:57
*** hemna has joined #openstack-cinder14:57
rosmaita(for os-brick)14:57
*** pcaruana has quit IRC14:57
openstackgerritRajat Dhasmana proposed openstack/os-brick master: Update master for stable/train  https://review.opendev.org/68032714:57
smcginnisIt's been that way for a very long time now. I don't believe it actually causes an error, but I forget if we enforce doc8 on that directory now or not.14:57
*** jmlowe has quit IRC15:01
smcginnisFWIW, all existing cycle files have the title header like that and running doc8 against them does not produce an error.15:02
smcginnisSo probably fine just leaving it as is.15:02
rosmaitawhoami-rajat: i think your change is bad RST also! i think for a level one heading, the top/bottom markers must line up exactly with the text, or you are allowed to have the markers one longer on *both* sides of the text15:05
*** jmlowe has joined #openstack-cinder15:05
*** tkajinam has quit IRC15:06
smcginnisIt formats OK, but I believe rosmaita is technically correct - https://762ea4c4c4fb203eceec-1a0f674a3837e35101e72ddbc77fd8ba.ssl.cf2.rackcdn.com/681762/2/check/build-openstack-releasenotes/de5c6af/docs/train.html15:07
rosmaitabut you are not allowed to mix (who the heck wrote the rules for RST? i thought it was supposed to be intuitive & easy -- or am i confusing it with markdown)15:07
eharneyfiled https://bugs.launchpad.net/tempest/+bug/1843762 for tempest15:08
openstackLaunchpad bug 1843762 in tempest "tempest volume quota validation breaks cinder stable/ocata runs" [Undecided,New]15:08
whoami-rajatrosmaita: oh, the text starts with 1 extra space, should i trim it or add additional marker to the end?15:08
eharneyrosmaita: you are confusing it w/ markdown, rst is surprisingly complicated :)15:08
smcginniseharney: Want to also propose a revert on that patch?15:08
smcginnisPowerful but complicated.15:09
rosmaitawhoami-rajat: i think do the space on each side style, i think that's what openstack usually does project-wide15:09
openstackgerritRajat Dhasmana proposed openstack/os-brick master: Update master for stable/train  https://review.opendev.org/68032715:10
jungleboyjWho doesn't love RST?15:10
eharneysmcginnis: done15:10
openstackgerritRajat Dhasmana proposed openstack/os-brick master: Update master for stable/train  https://review.opendev.org/68032715:11
whoami-rajatrosmaita: is that what's intended ? https://review.opendev.org/#/c/680327/4/releasenotes/source/train.rst15:11
*** thgcorrea has joined #openstack-cinder15:11
rosmaitalooking15:11
rosmaitawhoami-rajat: yes, that is good15:12
whoami-rajatgreat, let me do similar changes to other repos15:12
openstackgerritRajat Dhasmana proposed openstack/python-brick-cinderclient-ext master: Update master for stable/train  https://review.opendev.org/68131215:13
openstackgerritRajat Dhasmana proposed openstack/python-cinderclient master: Update master for stable/train  https://review.opendev.org/68176215:14
whoami-rajatdone, *phew*15:14
smcginniswhoami-rajat: Please don't generate a bunch of these updates. Especially right now.15:14
smcginnisA lot of folks are trying to get milestone critical patches through, so we don't need to waste gate resources for something that works as is.15:15
smcginnisI agree it's something that would be good to fix up so future generated patches are "more correct", but I don't think we need to update every existing patch right now.15:15
*** spsurya has quit IRC15:16
whoami-rajatsmcginnis: understood, that was the last one.15:16
smcginnisThanks!15:18
rosmaitawhoami-rajat: thanks!15:19
*** lpetrut has quit IRC15:25
*** hemna has quit IRC15:31
KeithMnemonicjungleboyj Thanks for the review on https://review.opendev.org/#/c/677801/2 anyone else want to help me finally put this 3par $#%$ to rest ;-)15:32
openstackgerritMerged openstack/cinder stable/stein: Add context to cloning snapshots in remotefs driver  https://review.opendev.org/68151515:35
openstackgerritMerged openstack/cinder master: Denote max microversion for Train  https://review.opendev.org/68152815:35
*** boxiang has quit IRC15:37
*** boxiang has joined #openstack-cinder15:37
*** sapd1_x has quit IRC15:41
openstackgerritRajat Dhasmana proposed openstack/cinder master: Untyped to Default Volume Type  https://review.opendev.org/63918015:49
openstackgerritPawel Kaminski proposed openstack/os-brick master: connectors/nvme: Wait until nvme device shows up in kernel  https://review.opendev.org/68178815:53
*** dave-mccowan has joined #openstack-cinder16:03
*** e0ne has quit IRC16:08
openstackgerritPawel Kaminski proposed openstack/os-brick master: connectors/nvme: Wait until nvme device shows up in kernel  https://review.opendev.org/67203116:11
openstackgerritMerged openstack/cinder master: Rename volume/utils.py to volume/volume_utils.py  https://review.opendev.org/67749216:12
openstackgerritMerged openstack/cinder master: Continue renaming volume_utils (core)  https://review.opendev.org/68047416:12
*** ociuhandu has quit IRC16:13
*** ag-47 has quit IRC16:13
*** whfnst has quit IRC16:13
*** jmlowe has quit IRC16:15
openstackgerritMerged openstack/cinder master: Continue renaming of volume_utils (drivers)  https://review.opendev.org/68047516:20
*** enriquetaso has quit IRC16:29
*** sahid has quit IRC16:30
*** udesale has quit IRC16:32
*** dave-mccowan has quit IRC16:34
openstackgerritMerged openstack/cinder master: PowerMax Driver - SnapVX NoCopy Mode  https://review.opendev.org/67123516:37
openstackgerritMerged openstack/cinder master: PowerMax driver - check cylinder count of source and target volumes  https://review.opendev.org/67136016:38
*** dave-mccowan has joined #openstack-cinder16:39
openstackgerritMerged openstack/cinder stable/stein: Move DotHill release note to correct location  https://review.opendev.org/67037617:01
*** ociuhandu has joined #openstack-cinder17:11
*** ociuhandu has quit IRC17:16
*** awalende has quit IRC17:22
*** awalende has joined #openstack-cinder17:23
hemna_sup peeps17:26
hemna_feature freeze tomorrow ?17:26
*** awalende has quit IRC17:27
*** hemna has joined #openstack-cinder17:27
*** e0ne has joined #openstack-cinder17:29
rosmaitahemna_: technically, it happens when M-3 is released, but since we don't do milestone releases anymore, i think tomorrow is correct for FF17:29
*** itlinux has joined #openstack-cinder17:35
*** markvoelker has quit IRC17:45
*** markvoelker has joined #openstack-cinder17:46
openstackgerritEric Harney proposed openstack/cinder master: Untyped to Default Volume Type  https://review.opendev.org/63918017:50
eharneyresolved a merge conflict on that one to keep it moving ^17:50
whoami-rajatwas just doing that, Thanks eharney17:51
eharneywhoami-rajat: i had one open question on there still17:52
whoami-rajateharney: i addressed that in next patchset, was replying now17:52
whoami-rajateharney: i'm still unsure about if we set `default_volume_type = None` in cinder.conf, it takes "None" as a string and not a python keyword, should this be handled explicitly too?17:55
*** boxiang has quit IRC17:55
*** boxiang has joined #openstack-cinder17:56
eharneywhoami-rajat: i was more trying to figure out if there was still a way that somebody would trick it into creating volumes w/ no type17:56
*** pcaruana has joined #openstack-cinder17:58
whoami-rajateharney: the previous code logged exception and continued if default volume type is not found, i've raised an exception in that case (if we provide invalid volume type [type doesn't exist] in cinder.conf)17:58
eharneywhoami-rajat: sounds good17:58
*** hemna has quit IRC18:00
*** oomichi_ has joined #openstack-cinder18:01
oomichi_smcginnis: Hi, can we talk about https://review.opendev.org/#/c/681776/ ?18:06
oomichi_Is that right 'groups' property has been added to Quota API's response body after Ocata?18:08
openstackgerritMerged openstack/python-brick-cinderclient-ext stable/train: Update .gitreview for stable/train  https://review.opendev.org/68130918:11
openstackgerritMerged openstack/python-brick-cinderclient-ext stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68131018:11
openstackgerritMerged openstack/python-brick-cinderclient-ext master: Update master for stable/train  https://review.opendev.org/68131218:11
openstackgerritMerged openstack/python-cinderclient stable/train: Update .gitreview for stable/train  https://review.opendev.org/68176018:17
openstackgerritMerged openstack/python-cinderclient stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68176118:18
openstackgerritMerged openstack/python-cinderclient master: Update master for stable/train  https://review.opendev.org/68176218:18
eharneyi think it was added here, in Pike: https://review.opendev.org/#/c/439461/18:21
smcginnisoomichi_: I don't understand that. We enforce response schema in the service now. Why does tempest need to try to do so too?18:25
*** hemna has joined #openstack-cinder18:28
oomichi_eharney: Thanks :-)18:34
oomichi_smcginnis because it is important to keep API backward compatibility for user POV.18:35
oomichi_If some property is removed at the time, user APPs can be broken18:35
oomichi_At this time, we added JSONSchema based on official API document which describes the latest API.18:36
oomichi_So we missed the point, sorry about that18:36
*** mriedem is now known as mriedem_afk18:39
*** jmlowe has joined #openstack-cinder18:40
*** thgcorrea has quit IRC18:41
jungleboyjsmcginnis: rosmaita I am officially approved for travel to Shanghai.  :-)18:42
rosmaitagood thing, we were counting on you to do a lot of stuff!18:42
jungleboyj:-)  Yeah and I have like one flight that will work to get me there in a way that will work.  So, hoping there are still tickets.18:43
*** savihou has joined #openstack-cinder18:52
jungleboyjrosmaita:  Do you know where you are staying?18:55
*** enriquetaso has joined #openstack-cinder18:59
savihouHowdy everyone, may I ask for help reviewing https://review.opendev.org/#/c/670715/ , Thanks!19:01
jungleboyjsavihou:  Sure.  Will look.19:02
savihoujungleboyj: thank you!19:02
*** hemna has quit IRC19:03
*** enriquetaso has quit IRC19:06
*** enriquetaso has joined #openstack-cinder19:07
*** itlinux is now known as itlinux-away19:22
*** awalende has joined #openstack-cinder19:23
*** itlinux-away is now known as itlinux19:24
*** itlinux is now known as itlinux-away19:24
*** itlinux-away is now known as itlinux19:24
*** itlinux is now known as itlinux-away19:24
*** itlinux-away is now known as itlinux19:26
*** itlinux is now known as itlinux-away19:26
*** itlinux-away is now known as itlinux19:27
*** itlinux is now known as itlinux-away19:27
*** awalende has quit IRC19:28
*** itlinux-away is now known as itlinux19:31
*** itlinux is now known as itlinux-away19:31
*** itlinux-away is now known as itlinux19:32
*** itlinux is now known as itlinux-away19:32
*** itlinux-away is now known as itlinux19:34
*** itlinux is now known as itlinux-away19:34
*** itlinux-away is now known as itlinux19:43
*** itlinux is now known as itlinux-away19:43
*** savihou has quit IRC19:49
*** gnufied has quit IRC19:50
*** itlinux-away is now known as itlinux19:51
*** itlinux is now known as itlinux-away19:51
*** itlinux-away is now known as itlinux19:55
*** itlinux is now known as itlinux-away19:55
*** itlinux-away is now known as itlinux19:55
*** itlinux is now known as itlinux-away19:56
henriqueof1Where can I report a driver's bug?19:57
hemna_guwak19:57
henriqueof1I am pretty sure the Dell iSCSI driver has a problem when migratin volumes.19:57
smcginnishenriqueof1: There are several Dell iSCSI drivers. Which one?19:58
*** itlinux-away is now known as itlinux19:59
*** itlinux is now known as itlinux-away19:59
henriqueof1smcginnis: cinder.volume.drivers.dell_emc.sc.storagecenter_iscsi.SCISCSIDriver19:59
smcginnishenriqueof1: You can file a bug here: https://bugs.launchpad.net/cinder/+bugs19:59
henriqueof1Sure.19:59
smcginnishenriqueof1: Thanks! Any details you can add there will help.20:00
*** itlinux-away is now known as itlinux20:02
*** itlinux is now known as itlinux-away20:02
*** itlinux-away is now known as itlinux20:02
*** itlinux is now known as itlinux-away20:03
*** e0ne has quit IRC20:03
smcginnisitlinux: Not sure if you're aware, but your nick changes are flooding several channels for those that don't have those kinds of message suppressed.20:03
*** itlinux-away is now known as itlinux20:04
*** itlinux is now known as itlinux-away20:04
*** itlinux-away is now known as itlinux20:06
*** enriquetaso has quit IRC20:06
*** itlinux is now known as itlinux-away20:07
henriqueof1smcginnis: https://bugs.launchpad.net/cinder/+bug/184382820:09
openstackLaunchpad bug 1843828 in Cinder "Error migrating volumes inside same storage center" [Undecided,New]20:09
henriqueof1i hope it have enough details.20:09
*** markvoelker has quit IRC20:11
*** markvoelker has joined #openstack-cinder20:11
*** itlinux-away is now known as itlinux20:13
*** itlinux is now known as itlinux-away20:13
smcginnishenriqueof1: Thanks!20:14
smcginnishenriqueof1: Hmm, two cinder services managing the same storage backend isn't really supported. Any given storage backend should really only be managed by one service.20:15
smcginnisNot saying that's the issue here, but just to be aware of that and that it could cause odd side effects.20:15
*** itlinux-away is now known as itlinux20:18
*** itlinux is now known as itlinux-away20:18
henriqueof1I understand completely my concern is that since the backend allow two volumes with same name, that is used as identifier by cinder, maybe we should use another value, one that is really unique.20:19
*** itlinux-away is now known as itlinux20:20
*** itlinux is now known as itlinux-away20:20
*** oomichi_ has quit IRC20:20
*** itlinux-away is now known as itlinux20:21
*** itlinux is now known as itlinux-away20:21
*** itlinux-away is now known as itlinux20:21
*** itlinux is now known as itlinux-away20:21
smcginnishenriqueof1: What release are you using? If I remember right, it used to operate based on volume name but then switched to storing off the actual volume ID to avoid naming conflict issues.20:23
*** itlinux-away is now known as itlinux20:23
*** itlinux is now known as itlinux-away20:23
*** itlinux-away is now known as itlinux20:23
smcginnishenriqueof1: Might also be better to look at unmanaging volumes from one service, then remanaging the same volume from the other service rather than going through a migration.20:23
*** itlinux is now known as itlinux-away20:23
*** itlinux-away is now known as itlinux20:27
*** itlinux is now known as itlinux-away20:27
henriqueof1smcginnis: I am using Stein from kolla-ansible, the sugestion about unmanaging and managing again sounds great but it can't be done with attached volumes.20:27
*** lpetrut has joined #openstack-cinder20:30
smcginnisThis is going to be a mess anyway with moving between multiple cinder-volume services. Maybe it would be easier to just update the cinder DB. As much as I hate even suggesting that anyone touchs the DB.20:30
*** itlinux-away is now known as itlinux20:31
*** itlinux is now known as itlinux-away20:31
*** itlinux-away is now known as itlinux20:32
*** itlinux is now known as itlinux-away20:32
*** pcaruana has quit IRC20:32
*** itlinux-away is now known as itlinux20:33
*** itlinux is now known as itlinux-away20:33
*** itlinux-away is now known as itlinux20:33
*** itlinux is now known as itlinux-away20:33
*** itlinux-away is now known as itlinux20:36
*** itlinux is now known as itlinux-away20:36
*** e0ne has joined #openstack-cinder20:37
*** markvoelker has quit IRC20:37
*** trident has quit IRC20:39
*** markvoelker has joined #openstack-cinder20:39
*** mriedem_afk is now known as mriedem20:39
*** itlinux-away is now known as itlinux20:45
*** itlinux is now known as itlinux-away20:45
*** lpetrut has quit IRC20:46
*** trident has joined #openstack-cinder20:51
*** itlinux-away is now known as itlinux20:54
*** itlinux is now known as itlinux-away20:54
*** itlinux-away is now known as itlinux20:54
*** itlinux is now known as itlinux-away20:55
*** itlinux-away is now known as itlinux20:55
*** itlinux is now known as itlinux-away20:55
*** hemna has joined #openstack-cinder20:58
*** itlinux-away is now known as itlinux21:01
*** itlinux is now known as itlinux-away21:01
*** itlinux-away is now known as itlinux21:03
*** itlinux is now known as itlinux-away21:03
*** itlinux-away is now known as itlinux21:03
*** itlinux is now known as itlinux-away21:03
*** itlinux-away is now known as itlinux21:03
*** zhubx has joined #openstack-cinder21:25
*** boxiang has quit IRC21:27
*** henriqueof1 has quit IRC21:31
*** e0ne has quit IRC21:32
*** hemna has quit IRC21:32
*** markvoelker has quit IRC21:32
*** henriqueof has joined #openstack-cinder21:40
*** whoami-rajat has quit IRC21:40
*** markvoelker has joined #openstack-cinder21:50
openstackgerritMerged openstack/os-brick stable/train: Update .gitreview for stable/train  https://review.opendev.org/68032522:06
openstackgerritMerged openstack/os-brick stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68032622:06
openstackgerritMerged openstack/os-brick master: Update master for stable/train  https://review.opendev.org/68032722:06
*** hemna has joined #openstack-cinder22:15
*** eharney has quit IRC22:19
*** rcernin has joined #openstack-cinder22:19
*** markvoelker has quit IRC22:32
*** whoami-rajat has joined #openstack-cinder22:37
*** markvoelker has joined #openstack-cinder22:46
*** markvoelker has quit IRC22:51
*** hemna has quit IRC22:56
*** tkajinam has joined #openstack-cinder23:03
*** mriedem is now known as mriedem_afk23:08
*** awalende has joined #openstack-cinder23:24
*** awalende has quit IRC23:28

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