Thursday, 2022-02-10

opendevreviewTakashi Kajinami proposed openstack/cinder master: Use the json format output of qemu-img info  https://review.opendev.org/c/openstack/cinder/+/79966300:04
opendevreviewAtsushi Kawai proposed openstack/cinder master: Hitachi: Add Target Port Assignment for VSP Driver  https://review.opendev.org/c/openstack/cinder/+/82806002:54
*** _pewp__ is now known as _pewp_07:37
*** bhagyashris__ is now known as bhagyashris11:26
opendevreviewAtsushi Kawai proposed openstack/cinder master: Hitachi: Add port scheduler  https://review.opendev.org/c/openstack/cinder/+/82869611:27
opendevreviewAtsushi Kawai proposed openstack/cinder master: Hitachi: Add port scheduler  https://review.opendev.org/c/openstack/cinder/+/82869611:28
*** dviroel|out is now known as dviroel|ruck11:30
geguileorosmaita: jungleboyj when you have a minute, please check this backport (all 5 previous patches have merged and the final one is blocked on this one): https://review.opendev.org/c/openstack/cinder/+/82510511:48
*** mdbooth2 is now known as mdbooth11:49
yuvalgeguileo: I responded to your comment - can you check it out: https://review.opendev.org/c/openstack/cinder/+/82808511:55
geguileoyuval: replied12:01
*** dviroel is now known as dviroel|ruck13:13
rosmaitageguileo: ack13:30
opendevreviewLior Friedman proposed openstack/os-brick master: 1) After md creation - wait up to 1 minute until md device is created since we see that under load sometimes the device is not created immediately 2) Fixed end raid - we see that when creating mutliple boot volumes and deleting them - some mds are left on host  https://review.opendev.org/c/openstack/os-brick/+/82264213:56
opendevreviewyuval proposed openstack/cinder master: [Lightos] standardize connector usage  https://review.opendev.org/c/openstack/cinder/+/82808513:59
yuval<geguileo> following your request ^13:59
rosmaitaanyone know what Lior Friedman's IRC nick is?14:21
opendevreviewLior Friedman proposed openstack/os-brick master: Robust md creation/ending  https://review.opendev.org/c/openstack/os-brick/+/82264214:24
raghavendrathi, requesting review on below patches:14:28
raghavendrathttps://review.opendev.org/c/openstack/cinder/+/82491114:29
raghavendrathttps://review.opendev.org/c/openstack/cinder/+/78605414:29
raghavendratAppreciate your feedback14:29
opendevreviewBrian Rosmaita proposed openstack/python-cinderclient stable/victoria: DNM: Migrate job from CentOS8 to Bionic  https://review.opendev.org/c/openstack/python-cinderclient/+/82609214:38
jungleboyjgeguileo: Approved. https://review.opendev.org/c/openstack/cinder/+/82510514:47
geguileojungleboyj: rosmaita  thank you both!14:47
jungleboyjOf course.14:47
rosmaitahth14:48
*** hemna8 is now known as hemna14:53
*** _pewp__ is now known as _pewp_14:53
rosmaitahey everyone ... please review os-brick patches ... here is a handy dashboard: http://tiny.cc/brick-patches15:20
TusharTgiterosmaita: jungleboyj i've updated commit msg zuul test are fine take a look at it https://review.opendev.org/c/openstack/cinder/+/77398515:41
rosmaitaTusharTgite: ack15:41
TusharTgiterosmaita: i need workflow +1 on this have two +2 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/79956415:42
rosmaitaok15:42
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status  https://review.opendev.org/c/openstack/cinder/+/77398515:58
TusharTgitejungleboyj: corrected the comment https://review.opendev.org/c/openstack/cinder/+/77398515:59
tobias-urdinam i understand it correctly that with https://review.opendev.org/c/openstack/cinder/+/805887 it doesn't make sense to set rbd_flatten_volume_from_snapshot=true ?16:07
*** priteau_ is now known as priteau16:11
*** tkajinam is now known as Guest21016:30
opendevreviewMerged openstack/cinder master: [SVf] Fixed Detach for multi-attach volumes  https://review.opendev.org/c/openstack/cinder/+/80951816:43
*** priteau is now known as priteau_16:48
*** priteau_ is now known as priteau16:48
opendevreviewMerged openstack/cinder stable/victoria: Expose volume_attachments in Volume OVO  https://review.opendev.org/c/openstack/cinder/+/82510517:00
opendevreviewMerged openstack/cinder stable/victoria: Fix: Race between attachment and volume deletion  https://review.opendev.org/c/openstack/cinder/+/82510617:00
opendevreviewLuigi Toscano proposed openstack/cinder-tempest-plugin master: WIP Add a CentOS Stream 9 LVM job  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/82874317:00
*** dviroel|ruck is now known as dviroel|ruck|afk17:16
opendevreviewBrian Rosmaita proposed openstack/python-cinderclient stable/victoria: DNM: Migrate job from CentOS8 to Bionic  https://review.opendev.org/c/openstack/python-cinderclient/+/82609218:22
yuvalHey18:59
yuvalZuul failed on my patch18:59
yuvalat the following test: {0} tempest.scenario.test_volume_backup_restore.TestVolumeBackupRestore.test_volume_backup_restore [280.355681s] ... FAILED18:59
yuvalfrom the traceback its doesnt look related to my patch19:00
yuvalhttps://zuul.opendev.org/t/openstack/build/af619f53c74140ba874387dce5aa4e4719:00
yuvalpatch: https://review.opendev.org/c/openstack/cinder/+/82808519:00
yuvalwelp19:04
*** dviroel|ruck|afk is now known as dviroel|ruck19:19
smcginnisyuval: That could just be a transient failure. You can leave a comment on your patch of just the string "recheck" to get it to run the tests again.19:36
abishoprosmaita (and anyone else): I'm working on a patch that wants to start a new yoga-policy-changes release not (similar to the xena-policy-changes note)19:54
abishopwill it be OK if I start the note, with the understanding that others will update the note as additional work proceeds during the rest of the yoga cycle?19:54
rosmaitaabishop: maybe the thing to do is have the note on its own review by itself19:56
rosmaitaso it only gets committed once19:56
rosmaitareviewers will have to remember to make sure a patch that does policy stuff also updates that release note patch19:56
abishoprosmaita: do you mean hold off starting the note, until such time when the whole note can be crafted?20:01
abishopin my patch I can certainly state the release note is "deferred"20:02
rosmaitaabishop: no, i was thinking that we put it on its own review, and update it as we go along20:02
abishopah, that makes sense20:03
rosmaitainstead of having an update to it on every patch20:03
rosmaitawell, i wonder whether it makes sense or not20:03
abishopso when I'm ready, I'll push two patches: 1. code chagnes, 2. release note for 1.20:04
rosmaitayou are right that it would be good to keep it updated so that we don't have to come back at the end and do this huge note20:04
abishopso 2. would not be merged until all the updates are done (end of yoga cycle)20:04
rosmaitayes, that's what i'm thinking20:04
opendevreviewAde Lee proposed openstack/cinder master: WIP/DNM: Add fips check jobs  https://review.opendev.org/c/openstack/cinder/+/82568820:14
opendevreviewBrian Rosmaita proposed openstack/python-cinderclient stable/victoria: Migrate job from CentOS8 to Bionic  https://review.opendev.org/c/openstack/python-cinderclient/+/82609220:15
opendevreviewAde Lee proposed openstack/cinder master: WIP/DNM: Add fips check jobs  https://review.opendev.org/c/openstack/cinder/+/82568820:15
*** dviroel|ruck is now known as dviroel|out21:49
opendevreviewBrian Rosmaita proposed openstack/os-brick master: Update requirements minima for Yoga release  https://review.opendev.org/c/openstack/os-brick/+/82880222:17
opendevreviewAlan Bishop proposed openstack/cinder master: Prepare unit test request contexts for secure RBAC  https://review.opendev.org/c/openstack/cinder/+/82880522:30
opendevreviewAlan Bishop proposed openstack/cinder master: Update deprecated policies and rules for Yoga  https://review.opendev.org/c/openstack/cinder/+/82880622:30
opendevreviewAlan Bishop proposed openstack/cinder master: WIP: Add release note for policy changes in Yoga  https://review.opendev.org/c/openstack/cinder/+/82880722:30
*** osmanlicilegi is now known as Guest023:49

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!