Comment 2 for bug 1661065

Reviewed: https://review.openstack.org/427860
Committed: https://git.openstack.org/cgit/openstack/murano/commit/?id=1e9a625f9c3e050fd3478df5b066304cba4cda69
Submitter: Jenkins
Branch: master

commit 1e9a625f9c3e050fd3478df5b066304cba4cda69
Author: Felipe Monteiro <email address hidden>
Date: Wed Feb 1 13:41:46 2017 -0500

    Fixes various Cfapi bugs.

    Currently, murano.cfapi.cfapi throws 2 bugs, when certain
    branches are reached.

    1) An AttributeError is thrown by _get_muranoclient, because
       CONF.cfapi is accessed; however, cfapi is not a defined
       group in murano.common.conf. The group should instead be
       CONF.engine, where packages_service is located.

    2) A NameError can be thrown by get_last_operation, in the
       unlikely situation when m_environment.status is not contained
       in any of the cases enumerated in the if/elif branches;
       when this happens, 'resp' is returned without being defined,
       resulting in this error.

    3) Not a bug per se, but a LOG.warning statement is not formatted
       correctly in get_last_operation. It does not specify the
       instance_id in the error message.

    Change-Id: If91a0f8f700a9c0cda0ea85a1d37938f5ef8f98c
    Closes-Bug: #1661065