Comment 17 for bug 1185290

Reviewed: https://review.openstack.org/620165
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=3338d39a86eb2a31c5bb079aa1ac56fd88b52733
Submitter: Zuul
Branch: master

commit 3338d39a86eb2a31c5bb079aa1ac56fd88b52733
Author: Mohammed Naser <email address hidden>
Date: Mon Nov 26 16:31:33 2018 -0500

    Drop cruft code for all_tenants behaviour

    The behaviour for the Nova API is to require `all_tenants` to be
    included in order to be able to list instances for a specific
    project.

    While this behaviour can be a bit confusing, all official CLI
    clients currently automatically imply the addition of `all_tenants`
    to the API request if an explicit tenant/project is selected when
    listing instances.

    The TODO has lived in the code for 5 years now without any work
    being done on it. This patch drops the commented code which can
    later be updated with the appropriate microversion bump to allow
    for behaviour that doesn't require including all_tenants in the
    HTTP request for the future.

    Related-Bug: #1185290

    Change-Id: I9324f798b3c6e05991802bb6610fc357864c0c66