| *** dviroel|afk is now known as dviroel | 00:00 | |
| *** dviroel is now known as dviroel|out | 00:24 | |
| *** dviroel|out is now known as dviroel | 00:59 | |
| *** dviroel is now known as dviroel|out | 01:24 | |
| opendevreview | likui proposed openstack/python-cinderclient master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/python-cinderclient/+/841522 | 02:31 |
|---|---|---|
| opendevreview | Ghanshyam proposed openstack/python-brick-cinderclient-ext master: Update python testing as per zed cycle testing runtime https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/841523 | 02:32 |
| opendevreview | Ghanshyam proposed openstack/python-cinderclient master: Update python testing as per zed cycle testing runtime https://review.opendev.org/c/openstack/python-cinderclient/+/841524 | 02:32 |
| opendevreview | Ghanshyam proposed openstack/python-brick-cinderclient-ext master: Update python testing as per zed cycle testing runtime https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/841523 | 03:33 |
| opendevreview | Ghanshyam proposed openstack/python-cinderclient master: Update python testing as per zed cycle testing runtime https://review.opendev.org/c/openstack/python-cinderclient/+/841524 | 04:16 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add test to create multiple volume from resource https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/795753 | 04:33 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add test to create multiple volume from resource https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/795753 | 05:02 |
| opendevreview | Lior Friedman proposed openstack/os-brick master: Add NVMeoF Multipathing support Phase 1 (Native) implementation of NVMeoF Multipathing. https://review.opendev.org/c/openstack/os-brick/+/830800 | 06:24 |
| opendevreview | Lucian Petrut proposed openstack/os-brick master: Avoid volume extension errors caused by multipath-tools version https://review.opendev.org/c/openstack/os-brick/+/762776 | 07:53 |
| opendevreview | Merged openstack/cinder stable/xena: Prevent temporary volume from being deleted accidentally https://review.opendev.org/c/openstack/cinder/+/831186 | 07:58 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [IBM DS8000] Fixed Detach for multi-attach volumes https://review.opendev.org/c/openstack/cinder/+/837699 | 08:50 |
| *** dviroel_ is now known as dviroel | 11:33 | |
| yuval | arunkv here? | 12:02 |
| zenkuro | hi, can somebody review documentation fix patch? https://review.opendev.org/c/openstack/cinder/+/824995 thank you | 12:59 |
| rosmaita | zenkuro: done | 13:17 |
| rosmaita | zenkuro: think about the followup i suggested on that patch, though | 13:17 |
| rosmaita | abishop: https://review.opendev.org/c/openstack/cinder/+/831186 has merged if you want to propose it for wallaby real quick | 13:18 |
| abishop | rosmaita: yep, I'm on it | 13:25 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: increase swap to 4G https://review.opendev.org/c/openstack/cinder/+/841591 | 13:36 |
| opendevreview | Alan Bishop proposed openstack/cinder stable/wallaby: Prevent temporary volume from being deleted accidentally https://review.opendev.org/c/openstack/cinder/+/831187 | 13:37 |
| abishop | rosmaita: ^^ | 13:37 |
| rosmaita | abishop: cool, ty | 13:37 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: mypy: api_utils https://review.opendev.org/c/openstack/cinder/+/840678 | 13:43 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: mypy: api_utils https://review.opendev.org/c/openstack/cinder/+/840678 | 13:46 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: mypy: cinder/volume/volume_types.py https://review.opendev.org/c/openstack/cinder/+/835881 | 13:48 |
| geguileo | abishop: will you have some time tomorrow to dig into some of the CI job failures that we can fix? | 15:03 |
| geguileo | abishop: sorry, that was meant for rosmaita | 15:04 |
| * geguileo facepalms | 15:04 | |
| geguileo | rosmaita: will you have some time tomorrow to dig into some of the CI job failures that we can fix? | 15:04 |
| opendevreview | Merged openstack/cinder master: JovianDSS: fix default value in documentation https://review.opendev.org/c/openstack/cinder/+/824995 | 15:04 |
| rosmaita | geguileo: do you mean by myself, or with you? | 15:04 |
| geguileo | rosmaita: with me, because I know what things need to be changed, but no idea where they are | 15:05 |
| geguileo | rosmaita: Today I saw a failure on a patch and after digging into it I saw 2 different kind of errors and how we should go about fixing them | 15:06 |
| rosmaita | geguileo: excellent, i will be happy to work with you tomorrow | 15:06 |
| geguileo | rosmaita: awesome ! | 15:06 |
| rosmaita | any time after 12:30 UTC | 15:07 |
| geguileo | rosmaita: ok, will ping you then | 15:08 |
| rosmaita | geguileo: sounds good | 15:08 |
| opendevreview | Merged openstack/cinder master: mypy: annotate remotefs https://review.opendev.org/c/openstack/cinder/+/733623 | 15:49 |
| raghavendrat | hi, it would be great if someone can have a look at https://review.opendev.org/c/openstack/cinder/+/824911 | 15:52 |
| raghavendrat | It has one +2. Thanks. | 15:52 |
| yuval | Hey so someone from datacore asked how to configure the comment-trigger-ci-build | 15:53 |
| yuval | the private chat stopped so I printting it here maybe it will useful for someone else | 15:53 |
| yuval | - install jenkins plugin: Gerrit Trigger (our version is 2.36.0) | 15:54 |
| yuval | - Manage Jenkins -> Uncategorized -> Gerrit Trigger -> Advanced -> in the Event List move comment-added to Filtered In section. | 15:54 |
| yuval | - job configurations: | 15:54 |
| yuval | 1. under Build Triggers section find Gerrit Trigger. | 15:54 |
| yuval | 2. Push Add and choose Comment Added Contains Regular Expression | 15:54 |
| yuval | 3. wirte the Regex you need for triggering the job, i.e ^run-Lightbits\sCI$ | 15:54 |
| yuval | dc_arunkv | 15:54 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: NVMe-oF: Consolidate code paths https://review.opendev.org/c/openstack/os-brick/+/836060 | 16:02 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: NVMe-oF: Disconnect subsystems https://review.opendev.org/c/openstack/os-brick/+/836062 | 16:02 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: Support shared_targets tristate value https://review.opendev.org/c/openstack/os-brick/+/836063 | 16:02 |
| geguileo | hemna whoami-rajat You both +2 this patch, do you think it needs more reviews or is it +W worthy? https://review.opendev.org/c/openstack/cinder/+/790492 | 16:20 |
| whoami-rajat | geguileo, done | 16:24 |
| geguileo | whoami-rajat: thanks! :-) | 16:25 |
| whoami-rajat | np | 16:25 |
| whoami-rajat | geguileo, not sure if you saw the mail from Előd Illés, but we need to do a cinderlib release before 24th June | 16:25 |
| geguileo | whoami-rajat: I missed it :-( | 16:26 |
| whoami-rajat | geguileo, oh, you can follow the subject [cinder][kolla][OSA][release] Yoga cycle-trailing release deadline | 16:27 |
| geguileo | whoami-rajat: thanks! | 16:27 |
| whoami-rajat | geguileo, we still have time but there are a bunch of patches that are open. Assuming we don't need to merge all, we can prioritize them someday, what do you think? | 16:44 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: TOX: Document install_command usage https://review.opendev.org/c/openstack/cinder/+/840525 | 16:45 |
| geguileo | whoami-rajat: I'll have to look into the open patches | 16:45 |
| opendevreview | Eric Harney proposed openstack/cinder master: Use modern type annotation format for collections https://review.opendev.org/c/openstack/cinder/+/839987 | 16:45 |
| geguileo | rosmaita: ^ I think that update on the patch makes it easier to understand why we cannot move the constraints from install_command into deps | 16:45 |
| opendevreview | Rico Lin proposed openstack/cinder master: Add image_conversion_disable config https://review.opendev.org/c/openstack/cinder/+/839793 | 16:45 |
| geguileo | rosmaita: not Eric's patch, but mine lol | 16:45 |
| whoami-rajat | geguileo, yeah, i meant we can have a meeting to merge all cinderlib changes | 16:46 |
| geguileo | oh, ok | 16:46 |
| geguileo | I'll see if I can add encryption support to cinderlib, since I recently used it to test the FIPS issue | 16:46 |
| geguileo | and doing it manually was a bit cumbersome | 16:46 |
| whoami-rajat | ack | 16:47 |
| ricolin | rosmaita: can you check if https://review.opendev.org/c/openstack/cinder/+/839793 targets your comments? thanks | 16:48 |
| rosmaita | geguileo: thanks, i completely forgot to circle back on the tox commment patch | 17:06 |
| rosmaita | ricolin: will look | 17:06 |
| rosmaita | geguileo: excellent update, thanks | 17:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Doc: Improve name_id documentation https://review.opendev.org/c/openstack/cinder/+/795751 | 17:54 |
| opendevreview | Merged openstack/cinder master: RBD: Don't flatten temporary resources https://review.opendev.org/c/openstack/cinder/+/790492 | 17:59 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Docs: Document clone_image driver method https://review.opendev.org/c/openstack/cinder/+/815660 | 18:03 |
| rosmaita | ricolin: left some comments for you ... thanks for the quick update! | 18:35 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Join do_setup and check_for_setup_error https://review.opendev.org/c/openstack/cinder/+/815659 | 19:22 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: increase swap to 4G https://review.opendev.org/c/openstack/cinder/+/841591 | 19:40 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: increase swap to 2G https://review.opendev.org/c/openstack/cinder/+/841632 | 19:45 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Reject force deletion of volumes used by Cinder services https://review.opendev.org/c/openstack/cinder/+/830901 | 19:48 |
| *** dviroel is now known as dviroel|afk | 21:09 | |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: use default swap size https://review.opendev.org/c/openstack/cinder/+/841644 | 21:40 |
| opendevreview | Merged openstack/cinder master: Doc: Improve name_id documentation https://review.opendev.org/c/openstack/cinder/+/795751 | 21:58 |
| ricolin | rosmaita: checking, thanks:) | 23:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!