Cinder setup fails with "The conditional check 'cinder_backend_lvm_inuse | bool' failed"

[Chris Martin]

Encountered while running setup-infrastructure.yml, OSA stable/Newton commit 94a3b966b2f5603f2117b473f89477953e9328da from 7 October.

"TASK [Set default bind mounts] *************************************************
fatal: [r14r730-01]: FAILED! => {"failed": true, "msg": "The conditional check 'cinder_backend_lvm_inuse | bool' failed."

A very similar issue was reported by lovea in #openstack-ansible here, and he kept better notes than I did:
http://eavesdrop.openstack.org/irclogs/%23openstack-ansible/%23openstack-ansible.2016-08-19.log.html#t2016-08-19T15:44:27

Note that lovea was re-deploying, I was setting up a fresh environment, and we both experienced this.

Relevant part of my openstack_user_config.yml:

```
storage_hosts:
  r14r730-01:
    ip: 172.29.236.117

    container_vars:
      cinder_storage_availability_zone: cinder_jetstream_test
      cinder_default_availability_zone: cinder_jetstream_test
      cinder_backends:
        lvm:
          volume_backend_name: LVM_iSCSI
          volume_driver: cinder.volume.drivers.lvm.LVMVolumeDriver
          volume_group: cinder-volumes
          iscsi_ip_address: "{{ storage_address }}"
        limit_container_types: cinder_volume
```

When I commented out the line `iscsi_ip_address: "{{ storage_address }}"`, the playbook completed successfully.

It appears that some of the docs show a default of `iscsi_ip_address: "{{ cinder_storage_address }}"` instead of just `{{ storage_address }}`. I did not try this but perhaps that's what is expected?

http://docs.openstack.org/developer/openstack-ansible/install-guide/configure-user-config-examples.html - "storage_address"
http://docs.openstack.org/developer/openstack-ansible/install-guide/app-config-prod.html - "cinder_storage_address"

[Jean-Philippe Evrard (jean-philippe-evrard)]

When you commented out the line iscsi_ip_address: "{{ storage_address }}}" , is your cinder.conf properly configured with the proper IP?

It should.

If that's the case, we should just change the doc and tell it's automatically discovered.

We maybe have to test if the precedence of playbooks facts/inventory vars is still fine in the latest version of ansible.

Did you check the resulting cinder.conf file by forcing
iscsi_ip_address: <an ip of your choice, different than the auto-discovered one> ?

[Chris Martin (6-chris-z)]

When I commented out that line, I see something like "iscsi_ip_address = 172.29.236.36" in /etc/cinder/cinder.conf in each cinder container. The IP address corresponds to the container's interface on the container management subnet. Haven't tried using cinder yet but will report back if I have trouble.

[Alexandra Settle (alexandra-settle)]

Requires confirmation that this is an issue. Happy to take on board after confirmation.

[Chris Martin (6-chris-z)]

I've switched to using Ceph as my cinder back-end, so unfortunately I won't be helpful troubleshooting this going forward. Feel free to close if nobody else is having trouble with this..

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/392969

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/392969
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=f5d857864625cd9040d97bd73af1aebfc14d396c
Submitter: Jenkins
Branch: master

commit f5d857864625cd9040d97bd73af1aebfc14d396c
Author: Kevin Carter <email address hidden>
Date: Wed Nov 2 16:07:28 2016 -0500

    Move storage_address discovery into a single task

    The storage address variable needs to be defined before running other
    common tasks. The `storage_address` variable, when defined in a cinder
    storage backend, causes an exception at runtime as the host_var will be
    undefined. This patch moves the `storage_address` discovery set of tasks
    into its one task and allows the option to be known before running
    anything else.

    Closes-Bug: #1632393
    Change-Id: I547c72943396af92df2a298470e3f89b4d1e3a2d
    Signed-off-by: Kevin Carter <email address hidden>

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/393814

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/393814
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=9de8c2377264a4582139397b0f0fe16221d192f3
Submitter: Jenkins
Branch: stable/newton

commit 9de8c2377264a4582139397b0f0fe16221d192f3
Author: Kevin Carter <email address hidden>
Date: Wed Nov 2 16:07:28 2016 -0500

    Move storage_address discovery into a single task

    The storage address variable needs to be defined before running other
    common tasks. The `storage_address` variable, when defined in a cinder
    storage backend, causes an exception at runtime as the host_var will be
    undefined. This patch moves the `storage_address` discovery set of tasks
    into its one task and allows the option to be known before running
    anything else.

    Closes-Bug: #1632393
    Change-Id: I547c72943396af92df2a298470e3f89b4d1e3a2d
    Signed-off-by: Kevin Carter <email address hidden>
    (cherry picked from commit f5d857864625cd9040d97bd73af1aebfc14d396c)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/openstack-ansible 15.0.0.0b1 development milestone.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/openstack-ansible 14.0.2 release.