diff options
| author | Lucas Alvares Gomes <lucasagomes@gmail.com> | 2016-11-01 11:30:26 +0000 |
|---|---|---|
| committer | Lucas Alvares Gomes <lucasagomes@gmail.com> | 2016-11-02 15:01:49 +0000 |
| commit | e0fd53d1830a76e8eb99be899b8302769b4fa649 (patch) | |
| tree | 7387694282b9326e54b1ccd71b60af76a19d60ed /releasenotes | |
| parent | f750bcd52124951db1beac95a00572c0548a01a6 (diff) | |
| download | ironic-e0fd53d1830a76e8eb99be899b8302769b4fa649.tar.gz | |
API: lookup() ignore malformed MAC addresses
This patch is adding code to ignore malformed MAC addresses which are
called as part of the lookup() mechanism. Prior to this patch, if a MAC
address wasn't in the expected format (six octets) the deployment of the
node would fail because the ramdisk wouldn't be able to lookup which
node it was in the Ironic database. One way to trigger this problem was
to deploy a node with an Infiniband Card which the MAC address (or GID)
contains 20 octets, that would result in a deployment failure even when
that NIC wasn't used by Ironic at all (not enrolled as a port).
The ListOfMacAdresses type was also deleted as part of this patch
because it wasn't used anywhere else.
Change-Id: I614fe63236985438d2f354d17a15d17649e72912
Closes-Bug: #1633585
Diffstat (limited to 'releasenotes')
| -rw-r--r-- | releasenotes/notes/lookup-ignore-malformed-macs-09e7e909f3a134a3.yaml | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/releasenotes/notes/lookup-ignore-malformed-macs-09e7e909f3a134a3.yaml b/releasenotes/notes/lookup-ignore-malformed-macs-09e7e909f3a134a3.yaml new file mode 100644 index 000000000..56c1725e3 --- /dev/null +++ b/releasenotes/notes/lookup-ignore-malformed-macs-09e7e909f3a134a3.yaml @@ -0,0 +1,8 @@ +--- +fixes: + - Fixes a problem where the deployment of a node would fail to continue + if a malformed MAC address was passed to the lookup mechanism in the + Ironic API. For example, if a node contains an Infiniband card, the + lookup used to fail because the agent ramdisk passes a MAC address + (or GID) with 20 octets (instead of the expected 6 octets) as part + of the lookup request. Invalid addresses are now ignored. |