Tuesday, 2022-03-01

opendevreviewMerged openstack/cinder master: [SVf] Manage host attachment using portsets  https://review.opendev.org/c/openstack/cinder/+/81735100:13
opendevreviewMerged openstack/cinder master: db: Remove unnecessary timezone configuration  https://review.opendev.org/c/openstack/cinder/+/82612300:42
TusharTgitehi core reviewers need a workflow +1 o this https://review.opendev.org/c/openstack/cinder/+/77398504:00
opendevreviewAtsushi Kawai proposed openstack/cinder master: Hitachi: Add port scheduler  https://review.opendev.org/c/openstack/cinder/+/82869606:57
opendevreviewFrancesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/82648409:09
opendevreviewVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Deploy with cephadm  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/82648409:57
*** bhagyashris is now known as bhagyashris|ruck11:30
raghavendrathi whoami-rajat__: are you online ?12:38
raghavendrathi whoami-rajat:12:56
raghavendratregarding: https://review.opendev.org/c/openstack/cinder/+/78605413:01
raghavendratThis is one of the FFE for Yoga. It as one +2.13:01
raghavendratif you get free time, it would be great to receive feedback. Thanks.13:01
rosmaitaTusharTgite: https://review.opendev.org/c/openstack/cinder/+/773985 is in the gate now13:13
TusharTgiterosmaita: i have question should i rebase its child patch now or after merge13:13
rosmaitaTusharTgite: depends on whether it's going to require a merge commit13:23
rosmaitaif the parent commit of your patch is the current HEAD of master, then there won't be a merge commit13:23
rosmaitaotherwise, there will be, and it's possible that could make you have to rebase your dependent patches again if you do it now13:23
rosmaitacurrent HEAD of master is a43427ef57, parent of your patch is 3bebea1, so probably better to wait13:25
TusharTgiterosmaita: ok thanks i'll wait then13:35
TusharTgiterosmaita: i need workflow on this one too has two plus 2 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/82822913:36
rosmaitagot it13:37
opendevreviewMerged openstack/cinder master: Reset state robustification for volume os-reset_status  https://review.opendev.org/c/openstack/cinder/+/77398514:39
opendevreviewMerged openstack/cinder-tempest-plugin master: Updating python testing as per Yoga testing runtime  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/82822914:56
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status  https://review.opendev.org/c/openstack/cinder/+/80403515:07
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group os-reset_status  https://review.opendev.org/c/openstack/cinder/+/80473515:08
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group-snapshot os-reset_status  https://review.opendev.org/c/openstack/cinder/+/80475715:08
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for backup os-reset_status  https://review.opendev.org/c/openstack/cinder/+/77819315:08
hemnarosmaita got a second?15:25
rosmaitawhat's up?15:25
hemnaI'm confused by this failure : https://zuul.opendev.org/t/openstack/build/c46efe4445bf447487177b370117f90315:25
hemnaI didn't modify that line, and that novaclient server.create() call doesn't even have a context param15:26
hemnai ran pylint against master and it worked fine15:28
hemnathe only thing I changed in that file is a few numeric values, that already existed.15:28
*** geguileo is now known as Guest91415:34
rosmaitahemna: no ideas from me ... at some point we need to address that pylint job, because it fails so often i don't think anyone pays attention to it any more15:36
rosmaitabut yeah, i don't see any way your code change could have made that fail15:37
hemnaok it's not just me then.   /me is confused15:43
eharneythose pylint failures look accurate, though15:44
eharneyprobably since the job is non-voting, a patch introduced them and it wasn't noticed15:44
opendevreviewWalt proposed openstack/cinder master: WIP: Fix capacity calculations  https://review.opendev.org/c/openstack/cinder/+/83124715:45
hemnaeharney, if I run tox -epylint against master it's fine15:46
eharneyoh, and it showed up here because you touched that file, since the pylint job only runs on files that are modified in that commit15:46
eharneyhaving it only run against files changed in that commit is confusing, tbh15:47
opendevreviewEric Harney proposed openstack/cinder master: Tests: Fix missing novaclient context arg in test_host_filters  https://review.opendev.org/c/openstack/cinder/+/83139515:50
eharneythat fixes it ^15:50
hemnahah ok cool.  15:51
hemnaI ran pylint against that file directly and didn't get that failure fwiw.15:51
*** hemna8 is now known as hemna16:07
*** Guest914 is now known as geguileo16:08
geguileowhoami-rajat: I answered your question, could you have another look at https://review.opendev.org/c/openstack/cinder/+/790492 please?16:09
whoami-rajatgeguileo, looking16:37
geguileowhoami-rajat: thanks.  You can do it tomorrow ;-)16:42
*** bhagyashris_ is now known as bhagyashris|ruck16:43
opendevreviewMerged openstack/cinder master: db: Add missing foreign keys, indexes to models  https://review.opendev.org/c/openstack/cinder/+/81322119:07
opendevreviewGhanshyam proposed openstack/cinder master: Add grenade-skip-level irrelevant-files config  https://review.opendev.org/c/openstack/cinder/+/83143819:09
opendevreviewJuan Pablo Suazo proposed openstack/cinder master: Adds conditional that prevents going out of index when len(parent_backup_shalist) < len(shalist).  https://review.opendev.org/c/openstack/cinder/+/83143919:09
hemnathat passed zuul, can we slam that one in?20:33
opendevreviewMerged openstack/cinder master: Tests: Fix missing novaclient context arg in test_host_filters  https://review.opendev.org/c/openstack/cinder/+/83139522:58

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