| opendevreview | Brian Rosmaita proposed openstack/cinder master: Add release note prelude for the Xena release https://review.opendev.org/c/openstack/cinder/+/809305 | 00:22 |
|---|---|---|
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Clean up policy matrix for Xena release https://review.opendev.org/c/openstack/cinder/+/809306 | 01:47 |
| opendevreview | Chris M proposed openstack/cinder master: Dell PowerVault: Add missing "cinder manageable-list" functionality https://review.opendev.org/c/openstack/cinder/+/808837 | 03:54 |
| opendevreview | Chris M proposed openstack/cinder master: Dell PowerVault: Fix "cinder manageable-list" https://review.opendev.org/c/openstack/cinder/+/808837 | 03:58 |
| opendevreview | Merged openstack/cinder master: Update policies related to user visible extra specs https://review.opendev.org/c/openstack/cinder/+/809277 | 04:18 |
| *** bhagyashris|off is now known as bhagyashris | 05:34 | |
| opendevreview | Merged openstack/cinder master: Clean up policy matrix for Xena release https://review.opendev.org/c/openstack/cinder/+/809306 | 05:44 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add release note about xena policy changes https://review.opendev.org/c/openstack/cinder/+/809303 | 06:14 |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Re-use existing initiator group/host. https://review.opendev.org/c/openstack/cinder/+/787764 | 09:57 |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Improve error handling around deletes https://review.opendev.org/c/openstack/cinder/+/796286 | 10:23 |
| raghavendrat | hi geguileo: are you around ? | 10:41 |
| geguileo | raghavendrat: yup | 10:41 |
| raghavendrat | whenever you get time, it would be great if you can have a look at: | 10:43 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/806725 | 10:43 |
| geguileo | raghavendrat: I see that you have: Note: These parameters apply to Alletra 6k Storage as well. | 10:45 |
| geguileo | raghavendrat: did you intend to create a note box, or it's just a way of adding that information? | 10:45 |
| geguileo | I'm asking because the note formatting is: .. note:: | 10:46 |
| raghavendrat | Just a way of adding information | 10:46 |
| geguileo | ok | 10:49 |
| geguileo | thanks | 10:49 |
| raghavendrat | Thank you geguileo: for quick review & approval. | 10:52 |
| geguileo | np | 10:52 |
| *** dviroel|out is now known as dviroel | 11:20 | |
| opendevreview | Merged openstack/cinder master: Nimble: Add Alletra 6k related information https://review.opendev.org/c/openstack/cinder/+/806725 | 12:26 |
| opendevreview | Merged openstack/cinder master: Add release note about xena policy changes https://review.opendev.org/c/openstack/cinder/+/809303 | 12:27 |
| rosmaita | release note prelude for xena could use some reviews: https://review.opendev.org/c/openstack/cinder/+/809305 | 12:41 |
| rosmaita | eharney: geguileo: abishop: features you worked on are mentioned there, please check for correctness ^^ | 12:41 |
| *** abhishekk is now known as akekane|home | 13:24 | |
| *** akekane|home is now known as abhishekk | 13:25 | |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Correct "Add release note about xena policy changes" https://review.opendev.org/c/openstack/cinder/+/809417 | 13:57 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: RBD driver https://review.opendev.org/c/openstack/cinder/+/774288 | 14:27 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: Fix unused type: ignore in manager.py https://review.opendev.org/c/openstack/cinder/+/809420 | 14:27 |
| opendevreview | Merged openstack/cinder stable/victoria: Support Images API v2.11 https://review.opendev.org/c/openstack/cinder/+/806038 | 14:27 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Correct "Clean up policy matrix for Xena release" https://review.opendev.org/c/openstack/cinder/+/809422 | 14:37 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: RBD: Call trash operation when plain deletion fails https://review.opendev.org/c/openstack/cinder/+/805887 | 15:27 |
| rosmaita | we are down to 4 small/extra-small patches before we release RC-1: https://etherpad.opendev.org/p/cinder-xena-final-patches-for-RC | 15:39 |
| rosmaita | e0ne eharney geguileo hemna jungleboyj smcginnis whoami-rajat enriquetaso ^^ | 15:39 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: RBD: Call trash operation when plain deletion fails https://review.opendev.org/c/openstack/cinder/+/805887 | 15:42 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: RBD: Call trash operation when plain deletion fails https://review.opendev.org/c/openstack/cinder/+/805887 | 15:43 |
| pots | rosmaita Hi, could you take a quick look at https://review.opendev.org/c/openstack/cinder/+/808837 to see whether it can make it into Xena or needs to be broken up into multiple patches in order to get some of them in? | 16:14 |
| opendevreview | Merged openstack/cinder master: Native multibackend-matrix Zuul v3 job https://review.opendev.org/c/openstack/cinder/+/751579 | 16:24 |
| opendevreview | Merged openstack/cinder master: Correct "Clean up policy matrix for Xena release" https://review.opendev.org/c/openstack/cinder/+/809422 | 16:33 |
| opendevreview | Luigi Toscano proposed openstack/cinder stable/wallaby: Native multibackend-matrix Zuul v3 job https://review.opendev.org/c/openstack/cinder/+/809439 | 16:49 |
| opendevreview | Merged openstack/cinder master: Add release note prelude for the Xena release https://review.opendev.org/c/openstack/cinder/+/809305 | 16:58 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: RBD: Call trash operation when plain deletion fails https://review.opendev.org/c/openstack/cinder/+/805887 | 17:11 |
| opendevreview | Merged openstack/cinder master: mypy: Fix unused type: ignore in manager.py https://review.opendev.org/c/openstack/cinder/+/809420 | 18:03 |
| opendevreview | Luigi Toscano proposed openstack/cinder stable/victoria: Native multibackend-matrix Zuul v3 job https://review.opendev.org/c/openstack/cinder/+/809447 | 18:18 |
| opendevreview | Luigi Toscano proposed openstack/cinder stable/ussuri: Native multibackend-matrix Zuul v3 job https://review.opendev.org/c/openstack/cinder/+/809448 | 18:20 |
| opendevreview | Luigi Toscano proposed openstack/cinder stable/train: Native multibackend-matrix Zuul v3 job https://review.opendev.org/c/openstack/cinder/+/809449 | 18:21 |
| tosky | (ihihihihih) | 18:21 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Correct "Add release note about xena policy changes" https://review.opendev.org/c/openstack/cinder/+/809417 | 18:26 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Turn off inline comments for mypy zuul job https://review.opendev.org/c/openstack/cinder/+/804294 | 18:46 |
| mloza | hello, if I change the multibackend name and volume_backend_name in cinder.conf and restart cinder service, will it update the os-vol-host-attr:host for existing volumes? | 18:55 |
| opendevreview | Merged openstack/cinder master: Correct "Add release note about xena policy changes" https://review.opendev.org/c/openstack/cinder/+/809417 | 19:42 |
| enriquetaso | Hello, I've updated the release note with 'upgrade' information https://review.opendev.org/c/openstack/cinder/+/805887 | 19:55 |
| enriquetaso | please feel free to review it :) It has already one +2 | 19:55 |
| rosmaita | cinder RC-1 release patch is ready if anyone has a minute to verify the hash: https://review.opendev.org/c/openstack/releases/+/808608 | 19:57 |
| *** dviroel is now known as dviroel|out | 21:00 | |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: RBD: Call trash operation when plain deletion fails https://review.opendev.org/c/openstack/cinder/+/805887 | 21:28 |
| enriquetaso | thanks rosmaita and geguileo, I've updated the commit msg and the release note ^ | 21:29 |
| rosmaita | enriquetaso: you are too quick! I was just adding another comment | 21:31 |
| enriquetaso | uh | 21:40 |
| enriquetaso | sorry about that | 21:40 |
| enriquetaso | I just saw your new comments, feel free to take your time to review, i won't update again today :P | 21:42 |
| rosmaita | enriquetaso: sorry i got it on there late, i did review the entire patch, so i am +2 ... you can update today or tomorrow, whatever works for you | 21:49 |
| mloza | hello, if I have two backends e.g. ceph-1 and ceph-2 can they share the same keyring or should be separate? | 22:01 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!