Open Stack

On Tuesday, 9 March 2021 23:40:08 CET Lee Yarwood wrote:
> On Tue, 9 Mar 2021 at 22:24, Jeremy Stanley <fungi at yuggoth.org> wrote:
> > On 2021-03-09 22:19:48 +0000 (+0000), Jeremy Stanley wrote:
> > > On 2021-03-09 22:18:12 +0000 (+0000), Jeremy Stanley wrote:
> > > > On 2021-03-09 22:08:18 +0000 (+0000), Lee Yarwood wrote:
> > > > > I reported the following bug last week but I've yet to get any real
> > > > > feedback after asking a few times in irc.
> > > > > 
> > > > > Running parallel iSCSI/LVM c-vol backends is causing random failures
> > > > > in CI
> > > > > https://bugs.launchpad.net/cinder/+bug/1917750
> > > > > 
> > > > > AFAICT tgtadm is causing this behaviour. As I've stated in the bug
> > > > > with Fedora 32 and lioadm I don't see the WWN conflict between the
> > > > > two
> > > > > backends. Does anyone know if using lioadm is an option on Focal?
> > > > 
> > > > https://docs.openstack.org/cinder/latest/admin/blockstorage-lio-iscsi-> > > > support.html seems to indicate that you just need to set it in
> > > > configuration. The package that document mentions looks like a distro
> > > > package
> > > > recommendation (does not exist under that name on PyPI) and the
> > > > equivalent lib on PyPI is included in cinder's requirements.txt
> > > > file, but I don't see either mentioned in the devstack source tree
> > > > so maybe that needs to be installed for DevStack-based jobs to take
> > > > advantage of it.
> > > 
> > > Oh, and this would be the package to add from Ubuntu Focal:
> > > 
> > > https://packages.ubuntu.com/focal/python3-rtslib-fb
> > 
> > Nevermind, DevStack installs the projects with pip, so the one in
> > cinder's requirements.txt should already be present. In that case,
> > yeah, just set it in the config?
> 
> Yes correct, my question was more to see if there were any known
> issues with using lioadm on Focal. Anyway I've pushed the following
> WIP change for devstack to switch over to lioadm when using Focal:
> 
> WIP cinder: Default CINDER_ISCSI_HELPER to lioadm on Ubuntu
> https://review.opendev.org/c/openstack/devstack/+/779624

For the record, we use the default (tgt) on the main cinder gates, as well as 
a lioadm job (defined in cinder-tempest-plugin). If I read the code correctly, 
that change would break tgt for everyone on ubuntu.

Please raise this in the cinder meeting (Wednesday).

-- 
Luigi