commit ffa0fe44315e8ad7fd978342dd4e8cd6871c01a3
Author: Jacek Tomasiak <email address hidden>
Date: Fri Feb 12 23:30:31 2016 +0100
Fix checks for Project/Tenant command line arguments
OS_PROJECT_ID/OS_TENANT_ID and OS_PROJECT_NAME/OS_TENANT_NAME are
supposed to be used interchangeably. Current implementation doesn't
handle this properly. It requires "tenant" parameters to be present
even if "project" arguments are provided. The check is done twice
which is redundant, inconsistent and catches some paths which don't
require those arguments at all.
The redundant check has been removed and better of two available
checks has been moved to proper place in the code.
Reviewed: https:/ /review. openstack. org/279868 /git.openstack. org/cgit/ openstack/ python- muranoclient/ commit/ ?id=ffa0fe44315 e8ad7fd978342dd 4e8cd6871c01a3
Committed: https:/
Submitter: Jenkins
Branch: master
commit ffa0fe44315e8ad 7fd978342dd4e8c d6871c01a3
Author: Jacek Tomasiak <email address hidden>
Date: Fri Feb 12 23:30:31 2016 +0100
Fix checks for Project/Tenant command line arguments
OS_ PROJECT_ ID/OS_TENANT_ ID and OS_PROJECT_ NAME/OS_ TENANT_ NAME are
supposed to be used interchangeably. Current implementation doesn't
handle this properly. It requires "tenant" parameters to be present
even if "project" arguments are provided. The check is done twice
which is redundant, inconsistent and catches some paths which don't
require those arguments at all.
The redundant check has been removed and better of two available
checks has been moved to proper place in the code.
Change-Id: I7f435aedf40fa1 6af8417c21d778e fc0c70979d6
Closes-Bug: #1545182