Misleading MACAlreadyExists exception

[Josh Gachnang]

This code raises a MACAlreadyExists exception, when it should be something like UUIDAlreadyExists exception. I think this is due to get_by_uuid accepting a UUID or a MAC. If you add a UUID kwarg that is different than the get_test_port's UUID, the error goes away.

def _create_test_port(self, **kwargs):
        p = db_utils.get_test_port(**kwargs)
        return self.dbapi.create_port(p)

fake_port1 = self._create_test_port(node_id=123,
                                    address="aa:bb:cc:dd:ee:fe")
fake_port2 = self._create_test_port(node_id=123,
                                    id=42,
                                    address="aa:bb:cc:dd:ee:fb")

[Dmitry Tantsur (divius)]

Hi Josh! Could you give status update on this bug? If you're still working on it, please set status to In Progress and link your patch here, if any.

[Dmitry Tantsur (divius)]

Hi! Per global bugs clean-up, we're unassigning bugs that are without patch and other visible progress for more than 7 days. Please feel free to reassign yourself, set status to In Progress and leave some status comment, if this is wrong. Thank you for understanding!

Also, the only place where create_test_port is present is in test_pxe and uuid argument is used there. Could you clarify, whether this bug is present now?

[Launchpad Janitor (janitor)]

[Expired for Ironic because there has been no activity for 60 days.]