| *** dviroel is now known as dviroel|out | 00:28 | |
| *** abdi_ is now known as abdi | 01:37 | |
| *** geguileo is now known as Guest1080 | 02:14 | |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Add manage related support https://review.opendev.org/c/openstack/cinder/+/851170 | 02:51 |
|---|---|---|
| *** akekane is now known as abhishekk | 05:36 | |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Add manage related support https://review.opendev.org/c/openstack/cinder/+/851170 | 09:37 |
| *** dviroel|out is now known as dviroel | 11:12 | |
| *** Guest1080 is now known as geguileo | 11:29 | |
| *** akekane is now known as abhishekk | 14:05 | |
| opendevreview | Konrad Gube proposed openstack/cinder-specs master: Add extend volume completion action https://review.opendev.org/c/openstack/cinder-specs/+/866718 | 14:11 |
| kgube | whoami-rajat: regarding the attached resize for NFS, I have decided to propose an alternative spec, proposing a new volume action in Cinder for async feedback from Nova | 14:15 |
| kgube | whoami-rajat: because Nova reviewers were not convinced of the synchronous API | 14:16 |
| tobias-urdin | whoami-rajat: can u check https://review.opendev.org/c/openstack/cinder/+/864701 when you have some time available, thanks! | 14:24 |
| opendevreview | Konrad Gube proposed openstack/cinder-specs master: Add extend volume completion action https://review.opendev.org/c/openstack/cinder-specs/+/866718 | 14:33 |
| *** dviroel is now known as dviroel|lunch | 15:31 | |
| *** dviroel|lunch is now known as dviroel | 16:34 | |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: nit: correct comment in backup manager https://review.opendev.org/c/openstack/cinder/+/866788 | 17:46 |
| whoami-rajat | kgube, that's a bummer but luckily we already have the async approach in alternatives | 17:50 |
| whoami-rajat | tobias-urdin, hey, can you answer my query on the patch, just want to be sure we are backporting the right fix and not missing anything | 17:51 |
| kgube | whoami-rajat, I already submitted it as a new spec: https://review.opendev.org/c/openstack/cinder-specs/+/866718 | 17:57 |
| kgube | though it probably would have been better to rewrite the existing one | 17:57 |
| whoami-rajat | kgube, i prefer to keep history of all the discussion but not sure how much relevant it is in this case | 17:59 |
| whoami-rajat | also we've reference to original specs so it should be good | 17:59 |
| whoami-rajat | rosmaita, what do you think ^ | 18:00 |
| rosmaita | looking | 18:05 |
| rosmaita | whoami-rajat: kgube: i think in this case, it's fine to have a different spec ... you have a reference to the old one in case anyone's interested, but since you are going in a different direction, i think it makes sense to keep the discussion focused on your new approach | 18:13 |
| whoami-rajat | rosmaita, thanks for taking a look | 18:13 |
| rosmaita | np | 18:13 |
| whoami-rajat | kgube, so the new spec looks good, I will take a look | 18:13 |
| kgube | whoami-rajat, rosmaita thanks! | 18:14 |
| whoami-rajat | kgube, just to be on the safe side, nova team does agree with the new approach right? so we don't end up reviewing again something that they don't approve of | 18:15 |
| kgube | I still need to submit the new nova-spec | 18:16 |
| kgube | or rather, sean mooney has suggested that I rewrite the existing one if we are not adding the new API | 18:18 |
| kgube | whoami-rajat, I will try to get this done till tomorrow | 18:19 |
| kgube | whoami-rajat, they have not given me a lot of feedback on the direction | 18:20 |
| whoami-rajat | kgube, ack | 18:51 |
| *** dviroel is now known as dviroel|biab | 19:48 | |
| abdi | iekhgbneltrf | 20:11 |
| *** marlinc is now known as Guest1159 | 20:50 | |
| *** marlinc is now known as Guest1168 | 22:14 | |
| *** marlinc is now known as Guest1173 | 23:32 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!