Ironic: save DB calls for getting flavor

The commit bceb6342994fc9d38a51a19e5b07652a5fdfd503 would read
the flavor a number of times instead of passing a flavor as a
a parameter.

This save an unnecesary DB access.

Change-Id: I4dc8739129a694d6ddae1ed9f1998663a2204d89
console: introduce a new exception InvalidConnectionInfo

Introduces a new exception InvalidConnectionInfo used when
an effectively connection_info is invalid.
Updates the code to take advantage of this new one.

Change-Id: I21940a14017b8aab5c3ee04c597cb05742a621eb
Mask passwords in exceptions and error messages

When a ProcessExecutionError is thrown by processutils.ssh_execute(),
the exception may contain information such as password. Upstream
applications that just log the message (as several appear to do)
could inadvertently expose these passwords to a user with read access to
the log files. It is therefore considered prudent to invoke
strutils.mask_password() on the command, stdout and stderr in the
exception. A test case has been added (to oslo-incubator) in order to 
ensure that all three are properly masked.

An earlier commit (853d8f9897f8563851441108a9be26b10908c076) failed
to address ssh_execute(). This change set addresses ssh_execute.

OSSA is aware of this change request.

Change-Id: Ie0caf32469126dd9feb44867adf27acb6e383958
Closes-Bug: #1377981
Correctly catch InstanceExists in servers create API

When osapi_compute_unique_server_name_scope is set to project
or global an attempt to create a server with a name which
already exists results in an InstanceExists exception being raised.
This was not being handled in the API layer causing a 500
status error code. This patch fixes it so the exception is caught
correctly and a 409 Conflict is returned instead

Change-Id: Ifee3476622b227991a4c2a2780ae7ff23787ccb9
Closes-Bug:1376936
Correct VERSION of NetworkRequest

The VERSION of the nova.objects.NetworkRequest object was incorrectly
still at 1.0, even though a 1.1 version had been incremented previously
that added a pci_request_id field to the object.  According to Dan on
IRC, there's no danger here, but this is good to clean up regardless.

From Dan: "the only case where you'd have an issue is if you had a
conductor with the changeset before pci_request was added, and a compute
running the code after"

Change-Id: I3d8c1ac22fb50f25cee03bb230fccad2b18deaf5
Closes-bug: #1378461
Merge "Correctly catch InstanceExists in servers create API"
Merge "Mask passwords in exceptions and error messages"
Merge "Correct VERSION of NetworkRequest"
Merge "console: introduce a new exception InvalidConnectionInfo"
Merge "Ironic: save DB calls for getting flavor"