MessagingCore - reference to a self.client only defined in children

[Vincent Françoise]

In the "check_api_version()" method of the MessagingCore class, we can see that a reference to a self.client is made albeit being undefined in its __init__() contructor.

The objective would be to fix this issue by either making sure it is defined in MessagingCore.__init__()

[OpenStack Infra (hudson-openstack)]

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

[Antoine Cabot (antoinecabot)]

Darren, your review is still in progress, could you please give a status on this fix ? thx

[Darren Shaw (dronshaw)]

Antoine,

I've had my free time exponentially taken by other responsibilities since the first patchset! Everything should be complete, except for unit testing within /tests/common/messaging/test_message_core.py

Given I may not have the ability to focus on this watcher issue for some time, is this something I could collaborate with another individual on? I'm not sure on the usual approach to this dilemma from a community standpoint would be. Thanks!

[Antoine Cabot (antoinecabot)]

no problem Darren, we will keep your code and create a new review with unit tests, I'd like to have this fixed in mitaka-2 that's why I asked you for an update.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/253856
Committed: https://git.openstack.org/cgit/openstack/watcher/commit/?id=2f0c1c12cf6cfa1921ab1c5e7b858b4f33800ff0
Submitter: Jenkins
Branch: master

commit 2f0c1c12cf6cfa1921ab1c5e7b858b4f33800ff0
Author: Darren Shaw <email address hidden>
Date: Sat Dec 5 18:53:44 2015 -0600

    Define self.client in MessagingCore

    Currently self.client is referenced within MessagingCore,
    but no definition is made in its constructor. Additionally
    self.client is defined in children classes of MessagingCore.
    This patchset defines self.client in the constructor of
    MessagingCore and removes the redefinition in its children.

    -self.client lazily loaded

    Co-Authored-By: v-francoise <email address hidden>
    Change-Id: I14525a175bf1ebde3d2636024ad2f2219c79d6e1
    Closes-Bug: #1521636