| opendevreview | Takashi Kajinami proposed openstack/cinder master: Fix release note for bug 1957073 https://review.opendev.org/c/openstack/cinder/+/859203 | 00:49 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Fix release note for bug 1957073 https://review.opendev.org/c/openstack/cinder/+/859203 | 00:55 |
| tkajinam | whoami-rajat, hi. regarding your vote on https://review.opendev.org/c/openstack/cinder/+/859203 | 04:46 |
| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/859186 | 04:46 |
| tkajinam | I noticed the patch was merged AFTER zed branch was cut off. actually the original note is present in the current release, not in the zed release. | 04:46 |
| tkajinam | I was confused because of it was merged with the merge commit. original commit was created before zed cut off but merged after that... | 04:46 |
| tkajinam | https://docs.openstack.org/releasenotes/cinder/unreleased.html | 04:47 |
| tkajinam | so in this case we should fix it in master first. please let me know in case I misunderstood something. sorry for past confusion caused by my misunderstanding | 04:47 |
| whoami-rajat | tkajinam, ah, sorry, i got confused | 04:48 |
| whoami-rajat | tkajinam, then it makes sense to have it in master | 04:48 |
| whoami-rajat | i will revote | 04:48 |
| tkajinam | I did confused you guys :-( | 04:48 |
| tkajinam | confuse * | 04:48 |
| tkajinam | thanks ! | 04:48 |
| tkajinam | we better merge it in master and backport it to zed and release rc2 | 04:48 |
| whoami-rajat | tkajinam, yeah, I've approved it | 04:51 |
| whoami-rajat | let me know when a backport is ready | 04:52 |
| opendevreview | Takashi Kajinami proposed openstack/cinder stable/zed: Fix release note for bug 1957073 https://review.opendev.org/c/openstack/cinder/+/859208 | 04:53 |
| tkajinam | ^^^ whoami-rajat just pushed it :-) | 04:53 |
| whoami-rajat | thanks, will keep an eye on it | 04:53 |
| opendevreview | Merged openstack/cinder master: Fix release note for bug 1957073 https://review.opendev.org/c/openstack/cinder/+/859203 | 05:10 |
| opendevreview | Merged openstack/cinder stable/zed: Fix release note for bug 1957073 https://review.opendev.org/c/openstack/cinder/+/859208 | 10:05 |
| sfv880 | Hello whoami-rajat, I have addressed the comments on the https://review.opendev.org/c/openstack/cinder/+/850320 - could you please review? Thank you very much! | 10:15 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder-tempest-plugin master: Update metadata in setup.cfg https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/854989 | 10:16 |
| opendevreview | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Set snapshot_support to true for cephfsnfstype https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/858265 | 13:13 |
| opendevreview | Merged openstack/devstack-plugin-ceph master: Add cephfs_nfs_cluster_id option https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/857742 | 14:43 |
| *** dviroel is now known as dviroel|lunch | 15:10 | |
| *** dviroel|lunch is now known as dviroel | 16:20 | |
| opendevreview | Eric Harney proposed openstack/cinder master: Move ComparableMixin to api_utils https://review.opendev.org/c/openstack/cinder/+/859316 | 16:53 |
| opendevreview | Eric Harney proposed openstack/cinder master: Move scheduler-specific code to the scheduler https://review.opendev.org/c/openstack/cinder/+/859317 | 16:53 |
| opendevreview | Eric Harney proposed openstack/cinder master: Move check_exclusive_options to volume_utils https://review.opendev.org/c/openstack/cinder/+/859318 | 16:53 |
| opendevreview | Eric Harney proposed openstack/cinder master: Move check_ssh_injection to ssh_utils https://review.opendev.org/c/openstack/cinder/+/859319 | 16:53 |
| *** dviroel is now known as dviroel|dr_appt | 17:20 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!