| *** dviroel|brb is now known as dviroel | 00:03 | |
| *** dviroel is now known as dviroel_ | 00:19 | |
| *** dviroel_ is now known as dviroel | 00:19 | |
| *** dviroel is now known as dviroel|out | 01:47 | |
| opendevreview | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 07:03 |
|---|---|---|
| *** hemna4 is now known as hemna | 07:38 | |
| *** akekane_ is now known as abhishekk | 08:51 | |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: This patch is bugfixing and refactoring for the following patch: https://review.opendev.org/c/openstack/cinder/+/786873 https://review.opendev.org/c/openstack/cinder/+/827259 | 10:23 |
| *** dviroel|out is now known as dviroel | 11:27 | |
| yuval | tempest random volume type nameing | 12:33 |
| yuval | I upload a patch | 12:33 |
| yuval | https://review.opendev.org/c/openstack/tempest/+/827272/1/tempest/api/volume/admin/test_volume_retype.py | 12:33 |
| yuval | but then I noticed there is also a problem in cinder-tempet-plugin | 12:34 |
| opendevreview | yuval proposed openstack/cinder-tempest-plugin master: test_create_from_image - add random prefix https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/827278 | 12:37 |
| yuval | there is some duplication in the two patches - so if its a problem please point me to where I should put the duplicated code | 12:38 |
| rosmaita | yuval: ty, will take a look later | 13:28 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder stable/victoria: Volume transfers: Remove duplicate policy check https://review.opendev.org/c/openstack/cinder/+/827242 | 14:37 |
| *** bhagyashris_ is now known as bhagyashris | 14:42 | |
| rosmaita | whoami-rajat: i put together this dashboard because i keep missing patches proposed by the release team, you may find it useful: http://tiny.cc/cinder-release-patches | 14:43 |
| whoami-rajat | cool, thanks! | 14:44 |
| eharney | hemna: https://review.opendev.org/c/openstack/cinder/+/800703 could use another look when you get a chance | 14:53 |
| Adri2000 | hi :) anyone familiar with the netapp/nfs driver codebase could have a look at this patch https://review.opendev.org/c/openstack/cinder/+/820578 ? it's been in review for the last 2 months, the change itself looks pretty simple and as a user I confirmed it fixes the problem (resizing attached volume now works). thanks! | 15:14 |
| *** dviroel is now known as dviroel|lunch | 15:40 | |
| opendevreview | yuval proposed openstack/os-brick master: Lightbits LightOS driver https://review.opendev.org/c/openstack/os-brick/+/821603 | 16:01 |
| yuval | <geguileo> following your comments ^ | 16:02 |
| *** abishop_ is now known as abishop | 16:11 | |
| *** dviroel|lunch is now known as dviroel | 16:35 | |
| geguileo | rosmaita: there are a couple of changes that I wanted to see in yuval connector info, but we are too close to the deadline | 16:41 |
| geguileo | rosmaita: can we merge what we have now and make the changes as a copule of simple follow up patches? | 16:41 |
| geguileo | rosmaita: it will be ok as long as we don't do an actual release of os-brick or cinder before those 2 merge | 16:41 |
| rosmaita | i am ok with that | 16:41 |
| rosmaita | brick release is scheduled for next friday, i think | 16:42 |
| geguileo | rosmaita: so we can merge these 2 patches that use the new hostnqn entry in the connector properties, and then we add a couple of patches that change it to use the existing nqn instead | 16:42 |
| rosmaita | "Changes to be included in the Yoga release of the os-brick library must be merged by Thursday 10 February 2022 (20:00 UTC)" | 16:43 |
| geguileo | awesome | 16:44 |
| rosmaita | geguileo: i am ok with that, let's be clear on the gerrit review what followups we are requiring | 16:44 |
| geguileo | yuval: I +2 the os-brick patch and added the comment with a summary of what we discussed about the hostnqn and nqn | 16:48 |
| geguileo | yuval: feel free to ping me for review when you have the follow up cinder and os-brick patches | 16:48 |
| yuval | fixe for nqn till 10 Feb | 16:49 |
| yuval | got it | 16:49 |
| smcginnis | "brick release is scheduled for next friday, i think" Hey jungleboyj, do you want to tell rosmaita what day release deadlines happen on? XD | 16:51 |
| rosmaita | smcginnis: we are actually early by a week from official openstack | 16:52 |
| rosmaita | i'm trying an experiment so we can have 2 weeks between os-brick release and python-cinderclient release | 16:52 |
| smcginnis | Hah, OK. Just gave jungleboyj a hard time once over release deadlines. ;) | 16:52 |
| rosmaita | smcginnis: i know, *thursday* is the day1 | 16:53 |
| geguileo | rofl | 16:53 |
| smcginnis | It was more fun giving Jay a hard time about it! | 16:53 |
| rosmaita | well, Jay is just more fun overall, i think | 16:54 |
| smcginnis | :D | 16:54 |
| rosmaita | if someone has a few minutes, this is a pretty quick review that will help with CI for nvme, which as you know, is so popular nowadays: https://review.opendev.org/c/openstack/cinder/+/791929 | 17:05 |
| jungleboyj | smcginnis: So, you go and disappear for a long time and then show up to harass me !?! ;-) | 17:07 |
| smcginnis | Of course! :P | 17:08 |
| jungleboyj | I don't even remember what that whole fiasco was. | 17:08 |
| smcginnis | Probably just met harassing you on something. | 17:09 |
| rosmaita | smcginnis: ty! | 17:10 |
| smcginnis | 👍 | 17:10 |
| jungleboyj | Indeed. I probably tried to move a release day to Monday or something. | 17:11 |
| jungleboyj | I put my two cents but didn't vote since I saw smcginnis started the merge. | 17:12 |
| rosmaita | cool, thanks for looking at it | 17:14 |
| jungleboyj | NP. | 17:15 |
| *** dviroel is now known as dviroel|biab | 17:55 | |
| *** dviroel|biab is now known as dviroel | 18:08 | |
| opendevreview | Merged openstack/cinder master: LVM: Added NVMe TCP support for the nvmet target https://review.opendev.org/c/openstack/cinder/+/791929 | 20:24 |
| *** dviroel is now known as dviroel|afk | 20:35 | |
| opendevreview | Merged openstack/cinder master: [SVf] Add cleanrate in volume-type extra specs https://review.opendev.org/c/openstack/cinder/+/811877 | 20:41 |
| jungleboyj | msg smcginnis Are you going to the Berlin Summit? | 20:41 |
| jungleboyj | Well, that was a fail. | 20:41 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: bugfix and refactoring for add maintenance parameters https://review.opendev.org/c/openstack/cinder/+/827259 | 23:31 |
| *** dviroel|afk is now known as dviroel | 23:44 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!