*** markvoelker has quit IRC | 00:29 | |
*** markvoelker has joined #openstack-cinder | 00:29 | |
hemna_ | anyone here mess w/ helm? | 00:32 |
---|---|---|
*** laurent\ has joined #openstack-cinder | 00:34 | |
*** hemna has quit IRC | 00:36 | |
*** henriqueof has joined #openstack-cinder | 00:45 | |
*** henriqueof1 has quit IRC | 00:46 | |
*** zhengMa has joined #openstack-cinder | 00:50 | |
*** zhengMa has left #openstack-cinder | 00:50 | |
*** henriqueof1 has joined #openstack-cinder | 00:56 | |
*** henriqueof has quit IRC | 00:57 | |
*** markvoelker has quit IRC | 01:03 | |
*** markvoelker has joined #openstack-cinder | 01:04 | |
*** henriqueof1 has quit IRC | 01:05 | |
*** henriqueof has joined #openstack-cinder | 01:05 | |
*** hemna has joined #openstack-cinder | 01:09 | |
*** pcaruana has quit IRC | 01:09 | |
*** jgriffith has quit IRC | 01:10 | |
*** lseki has quit IRC | 01:10 | |
*** brinzhang_ has joined #openstack-cinder | 01:13 | |
*** brinzhang has quit IRC | 01:17 | |
openstackgerrit | zhufl proposed openstack/cinder master: [api-ref]control_location should be mandatory https://review.opendev.org/677862 | 01:38 |
*** dklyle has quit IRC | 01:42 | |
*** david-lyle has joined #openstack-cinder | 01:42 | |
*** hemna has quit IRC | 01:43 | |
*** david-lyle has quit IRC | 01:46 | |
*** dklyle has joined #openstack-cinder | 01:47 | |
*** whfnst has joined #openstack-cinder | 01:51 | |
openstackgerrit | weibin proposed openstack/cinder master: Add support for using ceph RBD ereasure code https://review.opendev.org/681187 | 01:58 |
*** hemna has joined #openstack-cinder | 02:43 | |
*** carloss has joined #openstack-cinder | 02:56 | |
*** hemna has quit IRC | 03:16 | |
*** Garyx_ has joined #openstack-cinder | 03:33 | |
*** Garyx has quit IRC | 03:33 | |
*** hemna has joined #openstack-cinder | 03:48 | |
openstackgerrit | Qian Mingyue proposed openstack/cinder master: Move the constants to constants.py https://review.opendev.org/681638 | 03:50 |
*** KeithMnemonic has quit IRC | 04:04 | |
openstackgerrit | Merged openstack/cinder stable/queens: RBD: remove redundant exception log to reduce noise https://review.opendev.org/680838 | 04:17 |
*** spatel has joined #openstack-cinder | 04:17 | |
*** hemna has quit IRC | 04:23 | |
*** sapd1_x has joined #openstack-cinder | 04:23 | |
*** udesale has joined #openstack-cinder | 04:31 | |
*** pcaruana has joined #openstack-cinder | 04:32 | |
*** Luzi has joined #openstack-cinder | 04:46 | |
*** markvoelker has quit IRC | 04:48 | |
*** dave-mccowan has quit IRC | 04:57 | |
*** pcaruana has quit IRC | 04:58 | |
*** spatel has quit IRC | 04:58 | |
*** pots has joined #openstack-cinder | 05:10 | |
*** sapd1_x has quit IRC | 05:13 | |
*** hemna has joined #openstack-cinder | 05:15 | |
*** vishalmanchanda has joined #openstack-cinder | 05:20 | |
*** brinzhang has joined #openstack-cinder | 05:48 | |
*** brinzhang_ has quit IRC | 05:51 | |
openstackgerrit | yenai proposed openstack/cinder master: Fix signature_verified not matched https://review.opendev.org/655666 | 05:56 |
*** tkajinam has quit IRC | 06:00 | |
*** carloss has quit IRC | 06:05 | |
*** markvoelker has joined #openstack-cinder | 06:06 | |
*** tkajinam has joined #openstack-cinder | 06:06 | |
*** pcaruana has joined #openstack-cinder | 06:08 | |
*** markvoelker has quit IRC | 06:11 | |
*** hemna has quit IRC | 06:18 | |
*** pcaruana has quit IRC | 06:41 | |
*** sahid has joined #openstack-cinder | 06:46 | |
*** tkajinam_ has joined #openstack-cinder | 06:53 | |
*** tkajinam has quit IRC | 06:56 | |
*** spsurya has joined #openstack-cinder | 07:07 | |
*** pcaruana has joined #openstack-cinder | 07:11 | |
*** trident has quit IRC | 07:15 | |
*** Zettastor has joined #openstack-cinder | 07:17 | |
*** awalende has joined #openstack-cinder | 07:17 | |
Zettastor | if I plan to submit cinder driver to master branch, must I use the master branch keystone to setup devstack environment? | 07:18 |
Zettastor | can I use stein branch keystone for submitting cinder driver codes to master? | 07:19 |
*** trident has joined #openstack-cinder | 07:24 | |
*** trident has quit IRC | 07:28 | |
openstackgerrit | Sneha Rai proposed openstack/cinder master: HPE 3PAR: Updated supported operations for multiattach feature in the driver document. https://review.opendev.org/681655 | 07:37 |
*** trident has joined #openstack-cinder | 07:37 | |
Zettastor | openstackgerrit: could you share you devstack local.conf to me for submitting codes to master branch? | 07:39 |
*** e0ne has joined #openstack-cinder | 07:57 | |
*** e0ne has quit IRC | 08:08 | |
*** tkajinam_ has quit IRC | 08:09 | |
*** hemna has joined #openstack-cinder | 08:14 | |
*** whoami-rajat has joined #openstack-cinder | 08:15 | |
*** markvoelker has joined #openstack-cinder | 08:16 | |
*** threestrands has quit IRC | 08:16 | |
*** ag-47 has joined #openstack-cinder | 08:20 | |
*** markvoelker has quit IRC | 08:20 | |
ag-47 | Hello 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-rajat | ag-47: IMO feature backport isn't allowed | 08:26 |
ag-47 | is there anyway to overcome the probleme without upgrading ? | 08:32 |
whoami-rajat | ag-47: you can make the changes manually to your local code. I think this is the only alternative. | 08:33 |
ag-47 | whoami-rajat: Thank you | 08:35 |
*** sapd1 has quit IRC | 08:44 | |
*** hemna has quit IRC | 08:47 | |
*** raghavendrat has joined #openstack-cinder | 08:55 | |
raghavendrat | is whoami-rajat: online? | 08:58 |
whoami-rajat | raghavendrat: Hi | 08:59 |
*** ociuhandu has joined #openstack-cinder | 09:00 | |
raghavendrat | this is regarding adding Peer Persistence support in HPE 3PAR cinder driver. | 09:02 |
raghavendrat | driver.hpe_3par=complete | 09:02 |
whoami-rajat | sorry i'm not getting much time for reviews, will take look at the peer persistence patch | 09:03 |
*** e0ne has joined #openstack-cinder | 09:03 | |
*** e0ne has quit IRC | 09:03 | |
*** udesale has quit IRC | 09:04 | |
*** ociuhandu has quit IRC | 09:04 | |
raghavendrat | ok. i think i was not able to explain my query well. | 09:06 |
*** e0ne has joined #openstack-cinder | 09:07 | |
*** udesale has joined #openstack-cinder | 09:07 | |
whoami-rajat | raghavendrat: oh, i didn't get the query | 09:11 |
*** FlorianFa has quit IRC | 09:13 | |
*** e0ne has quit IRC | 09:15 | |
*** hemna has joined #openstack-cinder | 09:15 | |
*** lpetrut has joined #openstack-cinder | 09:16 | |
*** rcernin has quit IRC | 09:16 | |
openstackgerrit | Raghavendra Tilay proposed openstack/cinder master: 3PAR: Add Peer Persistence support https://review.opendev.org/677945 | 09:19 |
*** ociuhandu has joined #openstack-cinder | 09:19 | |
*** e0ne has joined #openstack-cinder | 09:19 | |
*** ociuhandu has quit IRC | 09:21 | |
*** e0ne has quit IRC | 09:21 | |
*** ociuhandu has joined #openstack-cinder | 09:21 | |
*** lpetrut has quit IRC | 09:23 | |
*** lpetrut has joined #openstack-cinder | 09:24 | |
*** e0ne has joined #openstack-cinder | 09:39 | |
*** Zettastor has quit IRC | 09:42 | |
*** udesale has quit IRC | 09:49 | |
*** udesale has joined #openstack-cinder | 09:50 | |
*** hemna has quit IRC | 09:50 | |
openstackgerrit | Merged openstack/cinder stable/stein: 3PAR: Add config for NSP single path attach https://review.opendev.org/677800 | 09:57 |
*** peschk_l has joined #openstack-cinder | 10:01 | |
*** lpetrut has quit IRC | 10:17 | |
*** hemna has joined #openstack-cinder | 10:19 | |
*** lpetrut has joined #openstack-cinder | 10:19 | |
*** markvoelker has joined #openstack-cinder | 10:26 | |
*** markvoelker has quit IRC | 10:32 | |
*** carloss has joined #openstack-cinder | 10:38 | |
*** hemna has quit IRC | 10:52 | |
*** sapd1_x has joined #openstack-cinder | 10:56 | |
*** hemna has joined #openstack-cinder | 11:25 | |
raghavendrat | hi 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 IRC | 11:41 | |
*** pots has quit IRC | 11:41 | |
*** pots has joined #openstack-cinder | 11:42 | |
*** whoami-rajat has quit IRC | 11:54 | |
*** hemna has quit IRC | 12:00 | |
*** markvoelker has joined #openstack-cinder | 12:01 | |
raghavendrat | hi e0ne: whenever you get time, can you please have a look at https://review.opendev.org/#/c/677945/ | 12:34 |
*** eharney has joined #openstack-cinder | 12:36 | |
raghavendrat | hi eharney: | 12:40 |
e0ne | raghavendrat: I'll take a look on it once it passes CI | 12:41 |
raghavendrat | thanks e0ne: | 12:42 |
*** mriedem has joined #openstack-cinder | 12:43 | |
*** e0ne has quit IRC | 12:44 | |
*** e0ne has joined #openstack-cinder | 12:44 | |
*** ociuhandu has quit IRC | 12:50 | |
raghavendrat | eharney: 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-cinder | 12:52 | |
*** mriedem has quit IRC | 12:54 | |
openstackgerrit | Eric Harney proposed openstack/cinder stable/stein: Fix up test_snapshot notify tests https://review.opendev.org/681737 | 12:56 |
*** ociuhandu has joined #openstack-cinder | 12:58 | |
*** mriedem has joined #openstack-cinder | 13:00 | |
*** _erlon_ has quit IRC | 13:01 | |
*** _erlon_ has joined #openstack-cinder | 13:02 | |
*** jmlowe has quit IRC | 13:03 | |
*** raghavendrat has left #openstack-cinder | 13:03 | |
*** spatel has joined #openstack-cinder | 13:13 | |
*** spatel has quit IRC | 13:15 | |
*** jmlowe has joined #openstack-cinder | 13:24 | |
*** enriquetaso has joined #openstack-cinder | 13:25 | |
*** jmlowe has quit IRC | 13:25 | |
*** Luzi has quit IRC | 13:28 | |
abishop | hi cores, please consider https://review.opendev.org/681515 (a backport) | 13:29 |
abishop | we have a customer who hit this | 13:29 |
*** pcaruana has quit IRC | 13:30 | |
*** jmlowe has joined #openstack-cinder | 13:36 | |
*** markvoelker has quit IRC | 13:40 | |
*** ociuhandu has quit IRC | 13:41 | |
*** markvoelker has joined #openstack-cinder | 13:42 | |
*** KeithMnemonic has joined #openstack-cinder | 13:53 | |
*** dave-mccowan has joined #openstack-cinder | 13:54 | |
*** tkajinam has joined #openstack-cinder | 13:55 | |
*** hemna has joined #openstack-cinder | 13:55 | |
rosmaita | smcginnis: can you take a look at the cinderclient release patch? https://review.opendev.org/#/c/681604/ | 13:58 |
smcginnis | Bombs away | 13:59 |
rosmaita | :D | 13:59 |
*** ociuhandu has joined #openstack-cinder | 14:00 | |
*** whoami-rajat has joined #openstack-cinder | 14:01 | |
*** pcaruana has joined #openstack-cinder | 14:08 | |
openstackgerrit | OpenStack Release Bot proposed openstack/python-cinderclient stable/train: Update .gitreview for stable/train https://review.opendev.org/681760 | 14:22 |
openstackgerrit | OpenStack Release Bot proposed openstack/python-cinderclient stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train https://review.opendev.org/681761 | 14:22 |
openstackgerrit | OpenStack Release Bot proposed openstack/python-cinderclient master: Update master for stable/train https://review.opendev.org/681762 | 14:22 |
*** mriedem has quit IRC | 14:23 | |
*** henriqueof1 has joined #openstack-cinder | 14:27 | |
*** hemna has quit IRC | 14:29 | |
*** henriqueof has quit IRC | 14:30 | |
*** mriedem has joined #openstack-cinder | 14:32 | |
jungleboyj | rosmaita: Ugh, I meant to put my vote on that. | 14:41 |
jungleboyj | Sorry I didn't. Last night got away from me. | 14:41 |
*** dave-mccowan has quit IRC | 14:41 | |
rosmaita | jungleboyj: np, i figured you were busy | 14:41 |
jungleboyj | :-( Yeah, that has been the theme lately. | 14:42 |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Untyped to Default Volume Type https://review.opendev.org/639180 | 14:44 |
*** abishop has quit IRC | 14:45 | |
eharney | i think tempest may no longer pass on stable/ocata due to https://review.opendev.org/#/c/603339/ | 14:46 |
rosmaita | jungleboyj: 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 |
smcginnis | rosmaita: Yeah, that's been my policy. | 14:48 |
rosmaita | ok, cool | 14:48 |
jungleboyj | rosmaita: Yeah, that is fine. | 14:48 |
smcginnis | eharney: Well, that's annoying to enforce schema validation on a branch that didn't validate schema. :/ | 14:48 |
eharney | and 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 |
rosmaita | why can't they mind their own business? | 14:48 |
smcginnis | eharney: I'm not sure if we are able to pin tempest. | 14:49 |
rosmaita | yeah, it's "unversioned" because it is always backward compatible | 14:49 |
whoami-rajat | rosmaita: is it possible to update the extra '=' here https://review.opendev.org/#/c/681762/1/releasenotes/source/train.rst | 14:49 |
rosmaita | :P | 14:49 |
eharney | yeah.. | 14:49 |
smcginnis | I would say we propose a revert of that patch. Tempest shouldn't be doing that. | 14:49 |
smcginnis | At least until all supported cinder branches have our own schema validation. | 14:50 |
smcginnis | But if we are doing our own schema validation, I don't see why tempest needs to know or care. | 14:50 |
rosmaita | whoami-rajat: is it causing a failure? | 14:50 |
jungleboyj | smcginnis: ++ | 14:51 |
whoami-rajat | rosmaita: i don't think so | 14:51 |
rosmaita | i think you are right that it is incorrect RST | 14:52 |
jungleboyj | rosmaita: It is. | 14:52 |
rosmaita | jungleboyj: smcginnis: should we fix ourselves, or ask release team to put up a revision? | 14:53 |
rosmaita | whoami-rajat: ^^ | 14:53 |
jungleboyj | rosmaita: I would think you can push up a revision. | 14:53 |
smcginnis | rosmaita: 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 |
rosmaita | ok, whoami-rajat: do you want to edit that patch? | 14:54 |
rosmaita | (then i can approve it) | 14:54 |
whoami-rajat | rosmaita: sure | 14:54 |
rosmaita | whoami-rajat: please also fix brick-cinderclient-ext patch: https://review.opendev.org/#/c/681312 | 14:55 |
whoami-rajat | smcginnis: jungleboyj rosmaita the automation script has posted same mistake for every module https://review.opendev.org/#/c/681675/1/releasenotes/source/train.rst | 14:55 |
jungleboyj | :-( | 14:56 |
openstackgerrit | Rajat Dhasmana proposed openstack/python-brick-cinderclient-ext master: Update master for stable/train https://review.opendev.org/681312 | 14:56 |
smcginnis | https://opendev.org/openstack/project-config/src/branch/master/roles/copy-release-tools-scripts/files/release-tools/add_release_note_page.sh#L52 | 14:56 |
jungleboyj | (ノ ゜Д゜)ノ ︵ ┻━┻ | 14:56 |
openstackgerrit | Rajat Dhasmana proposed openstack/python-cinderclient master: Update master for stable/train https://review.opendev.org/681762 | 14:56 |
whoami-rajat | rosmaita: done ^^ | 14:57 |
rosmaita | whoami-rajat: also https://review.opendev.org/#/c/680327/1 | 14:57 |
*** hemna has joined #openstack-cinder | 14:57 | |
rosmaita | (for os-brick) | 14:57 |
*** pcaruana has quit IRC | 14:57 | |
openstackgerrit | Rajat Dhasmana proposed openstack/os-brick master: Update master for stable/train https://review.opendev.org/680327 | 14:57 |
smcginnis | It'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 IRC | 15:01 | |
smcginnis | FWIW, all existing cycle files have the title header like that and running doc8 against them does not produce an error. | 15:02 |
smcginnis | So probably fine just leaving it as is. | 15:02 |
rosmaita | whoami-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 text | 15:05 |
*** jmlowe has joined #openstack-cinder | 15:05 | |
*** tkajinam has quit IRC | 15:06 | |
smcginnis | It 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.html | 15:07 |
rosmaita | but 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 |
eharney | filed https://bugs.launchpad.net/tempest/+bug/1843762 for tempest | 15:08 |
openstack | Launchpad bug 1843762 in tempest "tempest volume quota validation breaks cinder stable/ocata runs" [Undecided,New] | 15:08 |
whoami-rajat | rosmaita: oh, the text starts with 1 extra space, should i trim it or add additional marker to the end? | 15:08 |
eharney | rosmaita: you are confusing it w/ markdown, rst is surprisingly complicated :) | 15:08 |
smcginnis | eharney: Want to also propose a revert on that patch? | 15:08 |
smcginnis | Powerful but complicated. | 15:09 |
rosmaita | whoami-rajat: i think do the space on each side style, i think that's what openstack usually does project-wide | 15:09 |
openstackgerrit | Rajat Dhasmana proposed openstack/os-brick master: Update master for stable/train https://review.opendev.org/680327 | 15:10 |
jungleboyj | Who doesn't love RST? | 15:10 |
eharney | smcginnis: done | 15:10 |
openstackgerrit | Rajat Dhasmana proposed openstack/os-brick master: Update master for stable/train https://review.opendev.org/680327 | 15:11 |
whoami-rajat | rosmaita: is that what's intended ? https://review.opendev.org/#/c/680327/4/releasenotes/source/train.rst | 15:11 |
*** thgcorrea has joined #openstack-cinder | 15:11 | |
rosmaita | looking | 15:11 |
rosmaita | whoami-rajat: yes, that is good | 15:12 |
whoami-rajat | great, let me do similar changes to other repos | 15:12 |
openstackgerrit | Rajat Dhasmana proposed openstack/python-brick-cinderclient-ext master: Update master for stable/train https://review.opendev.org/681312 | 15:13 |
openstackgerrit | Rajat Dhasmana proposed openstack/python-cinderclient master: Update master for stable/train https://review.opendev.org/681762 | 15:14 |
whoami-rajat | done, *phew* | 15:14 |
smcginnis | whoami-rajat: Please don't generate a bunch of these updates. Especially right now. | 15:14 |
smcginnis | A 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 |
smcginnis | I 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 IRC | 15:16 | |
whoami-rajat | smcginnis: understood, that was the last one. | 15:16 |
smcginnis | Thanks! | 15:18 |
rosmaita | whoami-rajat: thanks! | 15:19 |
*** lpetrut has quit IRC | 15:25 | |
*** hemna has quit IRC | 15:31 | |
KeithMnemonic | jungleboyj 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 |
openstackgerrit | Merged openstack/cinder stable/stein: Add context to cloning snapshots in remotefs driver https://review.opendev.org/681515 | 15:35 |
openstackgerrit | Merged openstack/cinder master: Denote max microversion for Train https://review.opendev.org/681528 | 15:35 |
*** boxiang has quit IRC | 15:37 | |
*** boxiang has joined #openstack-cinder | 15:37 | |
*** sapd1_x has quit IRC | 15:41 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Untyped to Default Volume Type https://review.opendev.org/639180 | 15:49 |
openstackgerrit | Pawel Kaminski proposed openstack/os-brick master: connectors/nvme: Wait until nvme device shows up in kernel https://review.opendev.org/681788 | 15:53 |
*** dave-mccowan has joined #openstack-cinder | 16:03 | |
*** e0ne has quit IRC | 16:08 | |
openstackgerrit | Pawel Kaminski proposed openstack/os-brick master: connectors/nvme: Wait until nvme device shows up in kernel https://review.opendev.org/672031 | 16:11 |
openstackgerrit | Merged openstack/cinder master: Rename volume/utils.py to volume/volume_utils.py https://review.opendev.org/677492 | 16:12 |
openstackgerrit | Merged openstack/cinder master: Continue renaming volume_utils (core) https://review.opendev.org/680474 | 16:12 |
*** ociuhandu has quit IRC | 16:13 | |
*** ag-47 has quit IRC | 16:13 | |
*** whfnst has quit IRC | 16:13 | |
*** jmlowe has quit IRC | 16:15 | |
openstackgerrit | Merged openstack/cinder master: Continue renaming of volume_utils (drivers) https://review.opendev.org/680475 | 16:20 |
*** enriquetaso has quit IRC | 16:29 | |
*** sahid has quit IRC | 16:30 | |
*** udesale has quit IRC | 16:32 | |
*** dave-mccowan has quit IRC | 16:34 | |
openstackgerrit | Merged openstack/cinder master: PowerMax Driver - SnapVX NoCopy Mode https://review.opendev.org/671235 | 16:37 |
openstackgerrit | Merged openstack/cinder master: PowerMax driver - check cylinder count of source and target volumes https://review.opendev.org/671360 | 16:38 |
*** dave-mccowan has joined #openstack-cinder | 16:39 | |
openstackgerrit | Merged openstack/cinder stable/stein: Move DotHill release note to correct location https://review.opendev.org/670376 | 17:01 |
*** ociuhandu has joined #openstack-cinder | 17:11 | |
*** ociuhandu has quit IRC | 17:16 | |
*** awalende has quit IRC | 17:22 | |
*** awalende has joined #openstack-cinder | 17:23 | |
hemna_ | sup peeps | 17:26 |
hemna_ | feature freeze tomorrow ? | 17:26 |
*** awalende has quit IRC | 17:27 | |
*** hemna has joined #openstack-cinder | 17:27 | |
*** e0ne has joined #openstack-cinder | 17:29 | |
rosmaita | hemna_: technically, it happens when M-3 is released, but since we don't do milestone releases anymore, i think tomorrow is correct for FF | 17:29 |
*** itlinux has joined #openstack-cinder | 17:35 | |
*** markvoelker has quit IRC | 17:45 | |
*** markvoelker has joined #openstack-cinder | 17:46 | |
openstackgerrit | Eric Harney proposed openstack/cinder master: Untyped to Default Volume Type https://review.opendev.org/639180 | 17:50 |
eharney | resolved a merge conflict on that one to keep it moving ^ | 17:50 |
whoami-rajat | was just doing that, Thanks eharney | 17:51 |
eharney | whoami-rajat: i had one open question on there still | 17:52 |
whoami-rajat | eharney: i addressed that in next patchset, was replying now | 17:52 |
whoami-rajat | eharney: 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 IRC | 17:55 | |
*** boxiang has joined #openstack-cinder | 17:56 | |
eharney | whoami-rajat: i was more trying to figure out if there was still a way that somebody would trick it into creating volumes w/ no type | 17:56 |
*** pcaruana has joined #openstack-cinder | 17:58 | |
whoami-rajat | eharney: 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 |
eharney | whoami-rajat: sounds good | 17:58 |
*** hemna has quit IRC | 18:00 | |
*** oomichi_ has joined #openstack-cinder | 18: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 |
openstackgerrit | Merged openstack/python-brick-cinderclient-ext stable/train: Update .gitreview for stable/train https://review.opendev.org/681309 | 18:11 |
openstackgerrit | Merged openstack/python-brick-cinderclient-ext stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train https://review.opendev.org/681310 | 18:11 |
openstackgerrit | Merged openstack/python-brick-cinderclient-ext master: Update master for stable/train https://review.opendev.org/681312 | 18:11 |
openstackgerrit | Merged openstack/python-cinderclient stable/train: Update .gitreview for stable/train https://review.opendev.org/681760 | 18:17 |
openstackgerrit | Merged openstack/python-cinderclient stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train https://review.opendev.org/681761 | 18:18 |
openstackgerrit | Merged openstack/python-cinderclient master: Update master for stable/train https://review.opendev.org/681762 | 18:18 |
eharney | i think it was added here, in Pike: https://review.opendev.org/#/c/439461/ | 18:21 |
smcginnis | oomichi_: 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-cinder | 18: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 broken | 18: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 that | 18:36 |
*** mriedem is now known as mriedem_afk | 18:39 | |
*** jmlowe has joined #openstack-cinder | 18:40 | |
*** thgcorrea has quit IRC | 18:41 | |
jungleboyj | smcginnis: rosmaita I am officially approved for travel to Shanghai. :-) | 18:42 |
rosmaita | good 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-cinder | 18:52 | |
jungleboyj | rosmaita: Do you know where you are staying? | 18:55 |
*** enriquetaso has joined #openstack-cinder | 18:59 | |
savihou | Howdy everyone, may I ask for help reviewing https://review.opendev.org/#/c/670715/ , Thanks! | 19:01 |
jungleboyj | savihou: Sure. Will look. | 19:02 |
savihou | jungleboyj: thank you! | 19:02 |
*** hemna has quit IRC | 19:03 | |
*** enriquetaso has quit IRC | 19:06 | |
*** enriquetaso has joined #openstack-cinder | 19:07 | |
*** itlinux is now known as itlinux-away | 19:22 | |
*** awalende has joined #openstack-cinder | 19:23 | |
*** itlinux-away is now known as itlinux | 19:24 | |
*** itlinux is now known as itlinux-away | 19:24 | |
*** itlinux-away is now known as itlinux | 19:24 | |
*** itlinux is now known as itlinux-away | 19:24 | |
*** itlinux-away is now known as itlinux | 19:26 | |
*** itlinux is now known as itlinux-away | 19:26 | |
*** itlinux-away is now known as itlinux | 19:27 | |
*** itlinux is now known as itlinux-away | 19:27 | |
*** awalende has quit IRC | 19:28 | |
*** itlinux-away is now known as itlinux | 19:31 | |
*** itlinux is now known as itlinux-away | 19:31 | |
*** itlinux-away is now known as itlinux | 19:32 | |
*** itlinux is now known as itlinux-away | 19:32 | |
*** itlinux-away is now known as itlinux | 19:34 | |
*** itlinux is now known as itlinux-away | 19:34 | |
*** itlinux-away is now known as itlinux | 19:43 | |
*** itlinux is now known as itlinux-away | 19:43 | |
*** savihou has quit IRC | 19:49 | |
*** gnufied has quit IRC | 19:50 | |
*** itlinux-away is now known as itlinux | 19:51 | |
*** itlinux is now known as itlinux-away | 19:51 | |
*** itlinux-away is now known as itlinux | 19:55 | |
*** itlinux is now known as itlinux-away | 19:55 | |
*** itlinux-away is now known as itlinux | 19:55 | |
*** itlinux is now known as itlinux-away | 19:56 | |
henriqueof1 | Where can I report a driver's bug? | 19:57 |
hemna_ | guwak | 19:57 |
henriqueof1 | I am pretty sure the Dell iSCSI driver has a problem when migratin volumes. | 19:57 |
smcginnis | henriqueof1: There are several Dell iSCSI drivers. Which one? | 19:58 |
*** itlinux-away is now known as itlinux | 19:59 | |
*** itlinux is now known as itlinux-away | 19:59 | |
henriqueof1 | smcginnis: cinder.volume.drivers.dell_emc.sc.storagecenter_iscsi.SCISCSIDriver | 19:59 |
smcginnis | henriqueof1: You can file a bug here: https://bugs.launchpad.net/cinder/+bugs | 19:59 |
henriqueof1 | Sure. | 19:59 |
smcginnis | henriqueof1: Thanks! Any details you can add there will help. | 20:00 |
*** itlinux-away is now known as itlinux | 20:02 | |
*** itlinux is now known as itlinux-away | 20:02 | |
*** itlinux-away is now known as itlinux | 20:02 | |
*** itlinux is now known as itlinux-away | 20:03 | |
*** e0ne has quit IRC | 20:03 | |
smcginnis | itlinux: 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 itlinux | 20:04 | |
*** itlinux is now known as itlinux-away | 20:04 | |
*** itlinux-away is now known as itlinux | 20:06 | |
*** enriquetaso has quit IRC | 20:06 | |
*** itlinux is now known as itlinux-away | 20:07 | |
henriqueof1 | smcginnis: https://bugs.launchpad.net/cinder/+bug/1843828 | 20:09 |
openstack | Launchpad bug 1843828 in Cinder "Error migrating volumes inside same storage center" [Undecided,New] | 20:09 |
henriqueof1 | i hope it have enough details. | 20:09 |
*** markvoelker has quit IRC | 20:11 | |
*** markvoelker has joined #openstack-cinder | 20:11 | |
*** itlinux-away is now known as itlinux | 20:13 | |
*** itlinux is now known as itlinux-away | 20:13 | |
smcginnis | henriqueof1: Thanks! | 20:14 |
smcginnis | henriqueof1: 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 |
smcginnis | Not 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 itlinux | 20:18 | |
*** itlinux is now known as itlinux-away | 20:18 | |
henriqueof1 | I 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 itlinux | 20:20 | |
*** itlinux is now known as itlinux-away | 20:20 | |
*** oomichi_ has quit IRC | 20:20 | |
*** itlinux-away is now known as itlinux | 20:21 | |
*** itlinux is now known as itlinux-away | 20:21 | |
*** itlinux-away is now known as itlinux | 20:21 | |
*** itlinux is now known as itlinux-away | 20:21 | |
smcginnis | henriqueof1: 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 itlinux | 20:23 | |
*** itlinux is now known as itlinux-away | 20:23 | |
*** itlinux-away is now known as itlinux | 20:23 | |
smcginnis | henriqueof1: 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-away | 20:23 | |
*** itlinux-away is now known as itlinux | 20:27 | |
*** itlinux is now known as itlinux-away | 20:27 | |
henriqueof1 | smcginnis: 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-cinder | 20:30 | |
smcginnis | This 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 itlinux | 20:31 | |
*** itlinux is now known as itlinux-away | 20:31 | |
*** itlinux-away is now known as itlinux | 20:32 | |
*** itlinux is now known as itlinux-away | 20:32 | |
*** pcaruana has quit IRC | 20:32 | |
*** itlinux-away is now known as itlinux | 20:33 | |
*** itlinux is now known as itlinux-away | 20:33 | |
*** itlinux-away is now known as itlinux | 20:33 | |
*** itlinux is now known as itlinux-away | 20:33 | |
*** itlinux-away is now known as itlinux | 20:36 | |
*** itlinux is now known as itlinux-away | 20:36 | |
*** e0ne has joined #openstack-cinder | 20:37 | |
*** markvoelker has quit IRC | 20:37 | |
*** trident has quit IRC | 20:39 | |
*** markvoelker has joined #openstack-cinder | 20:39 | |
*** mriedem_afk is now known as mriedem | 20:39 | |
*** itlinux-away is now known as itlinux | 20:45 | |
*** itlinux is now known as itlinux-away | 20:45 | |
*** lpetrut has quit IRC | 20:46 | |
*** trident has joined #openstack-cinder | 20:51 | |
*** itlinux-away is now known as itlinux | 20:54 | |
*** itlinux is now known as itlinux-away | 20:54 | |
*** itlinux-away is now known as itlinux | 20:54 | |
*** itlinux is now known as itlinux-away | 20:55 | |
*** itlinux-away is now known as itlinux | 20:55 | |
*** itlinux is now known as itlinux-away | 20:55 | |
*** hemna has joined #openstack-cinder | 20:58 | |
*** itlinux-away is now known as itlinux | 21:01 | |
*** itlinux is now known as itlinux-away | 21:01 | |
*** itlinux-away is now known as itlinux | 21:03 | |
*** itlinux is now known as itlinux-away | 21:03 | |
*** itlinux-away is now known as itlinux | 21:03 | |
*** itlinux is now known as itlinux-away | 21:03 | |
*** itlinux-away is now known as itlinux | 21:03 | |
*** zhubx has joined #openstack-cinder | 21:25 | |
*** boxiang has quit IRC | 21:27 | |
*** henriqueof1 has quit IRC | 21:31 | |
*** e0ne has quit IRC | 21:32 | |
*** hemna has quit IRC | 21:32 | |
*** markvoelker has quit IRC | 21:32 | |
*** henriqueof has joined #openstack-cinder | 21:40 | |
*** whoami-rajat has quit IRC | 21:40 | |
*** markvoelker has joined #openstack-cinder | 21:50 | |
openstackgerrit | Merged openstack/os-brick stable/train: Update .gitreview for stable/train https://review.opendev.org/680325 | 22:06 |
openstackgerrit | Merged openstack/os-brick stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train https://review.opendev.org/680326 | 22:06 |
openstackgerrit | Merged openstack/os-brick master: Update master for stable/train https://review.opendev.org/680327 | 22:06 |
*** hemna has joined #openstack-cinder | 22:15 | |
*** eharney has quit IRC | 22:19 | |
*** rcernin has joined #openstack-cinder | 22:19 | |
*** markvoelker has quit IRC | 22:32 | |
*** whoami-rajat has joined #openstack-cinder | 22:37 | |
*** markvoelker has joined #openstack-cinder | 22:46 | |
*** markvoelker has quit IRC | 22:51 | |
*** hemna has quit IRC | 22:56 | |
*** tkajinam has joined #openstack-cinder | 23:03 | |
*** mriedem is now known as mriedem_afk | 23:08 | |
*** awalende has joined #openstack-cinder | 23:24 | |
*** awalende has quit IRC | 23:28 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!