Change-Id: I72c84444225cb44afcf8d6393b2c1c95060c556e

Change-Id: I7c7752a87f260e35ca1e7caeb7f3e21f729d23a6
(cherry picked from commit 921e87cc)

Change-Id: I461ce0eca5ef729401c3b0214083ea7bc7409a57

Change-Id: Ieddf59a559340cab6e314e0d31da67fdf2661fa1
(cherry picked from commit c6050ac3)

This patch improve the error handling by adding try except.
It also make sure that the response will send as json.
This also fix ugly error when delete port failed because
it doesn't not exist.

Change-Id: I975c768926ecbafa8a608c0e1fff49d12c021e30

This patch allow the sdn mech driver to bind ports which
are required for HA and DVR setups

Change-Id: I4257aa04e782b0f878aa862d3a77ecdf7b124be8

Syncing the tox, requirement, and test-requirement configuration
to be up-to-date with neutron's configurations. Also solved the
new pep8 error that would arise from updating flake8, which seam
to be a code globalization convention.

Change-Id: I94db65c156b2b8c3723a83498c691fb51ad66eaf

This commit fixes SDNMechanismDriver._check_physnet_confs
to properly perform physical network config checks.

before the fix, we would fail mech driver initialization
when physical_networks config is provided with specific
networks, bind_normal_ports is set to true and
bind_normal_ports_physnets is a subset of the provided
physical_networks.

In addtion, unit tests were added to cover this.

Change-Id: I9baf3345f59832618ab229914fc634c18e9ced4f

Using UFM is not in the POR at the moment, so we revert
it for now.

This reverts commit 6b5eae77.

Change-Id: Idcfc4d07f2a73b556ebfdedd0d083e7bbc5e5682

The neutron dhcp agent plug_new function was changed to have
a link_up parameter, this patch add the support for the new
parameter.

Change-Id: I2c79c262ac2761569b598ee4506bdfae4461e2eb

For MLNX5 devices need to explicity initialize empty
dict when fetching them via netdev API fails.

Change-Id: I7607f145d24864e3973f8dbc7debd6adba42d598

There are no plans to support it. It makes no sense to perform
the same operation and expect different results so we remove
the relevant logic.

Change-Id: I82585ba02d938625661914e59a60c7dca547bdbd

Change-Id: I14101d952dc154beb75445f8802a52170243ac61

Mainly to avoid possible issues when vf index
is extracted from the system and is not converted
to int. while current code does not suffer from this
symptom, the same approach is already used in netdev API.
here we do the same.

Change-Id: I4e657173a2f44e834af211a8dc39df62da2aadf1

- Move set vf guid via sysfs logic to its own method
  for both mlnx4 and mlnx5
- Change fallback logging to debug and minor rewording

Change-Id: Iff7b46e490ad043b7d7ddb9de36615c344ebc9f3

This patch uses netlink to get administrative port and node GUID
for VFs.

This requires kernel 5.5.0 or newer and pyrote2 0.5.7 or newer.
A fallback mechanism is put in place so that existing functionality
is preserved on systems which do not meet the requirement.

Change-Id: I47bd969e5d0622ac2c1f1f435650918e322e22f0

Original place was wrong as it should be located under utils folder
like the module it tests.

Change-Id: I147c978c315a07a6e6414a55e6dea2d07aa73598

Change-Id: I2496c1e0476874db33ecbb24dc2fb3c703844cd6

For MLX4 devices it is required to get a VF administrative
GUID according to the GID index and not the VF index.

Change-Id: I9286c70bb8fbb7f9f2e6eed0f5b880f8c8f8159b

As fetching GUIDs is not really a PCI related operation
the methods related need to be in ib_utils module

A followup patch will introduce a fix to these methods for
ConnectX-3 as fetching a GUID is done according to the VF's GID
index and not its VF index.

Change-Id: I57789b37a2be39b9c19129c3d7e2cd26ea66d52e

This patch aligns all *_idx methods to return an integer
instead of string for consistency.

This fixes an issue discovered when attempting to set PKEY
for connectX-3 VF

Change-Id: Ie877dfdf03f20b20a66eafd69c90d4c43a92b6dd

Change-Id: Id17f733ef015e3503b695bb3151322ae35499e1c

we require to invoke send_string/recv_string methods when
sending unicode.

Change-Id: I26804ae3666f57b183eb0e987a4d23a14e59035e

Co-authored-by: Moshe Levi <moshele@mellanox.com>

Change-Id: I9ecfaa798c8066d07bbbf3e22ab82ddc9b553274

This patch configure the mlnx sdn port binding
for InfiniBand physnet by default. It also disable the sdn
sync.

Change-Id: I8752f54d7c84c1c0c4032fd4f1bb49eef7069e19

This fix the issue when sync it disable and network
is not flat we skip the binding of the normal port.

This patch update the logic to always bind normal ports
for IB physnet

Change-Id: I37d662f13d287619ffe22e371ace155ad2d51841

This commits adds support for Admin up/down calls and
modifies plug_nic RPC to not automatically set the port's
admin state to up.

Consumers of the API (mellanox neutron agent) are required
to use the API on port update events from neutron.
(which they already do today)

Change-Id: Ie9e9c81b58d7dc5ea19ea49b19b36a28a849331b

This commit explicitly converts vf index arg to int
before calling pyroute.

This will allow consumers of this API to enjoy a greater
amount of freedom when using the API (e.g providing it as a string)

Change-Id: Iadc294d83521f100ed4e99d83ebc075695cb7141

Change-Id: Ib2019d569bdbb308f9b9484ea578d4d51f754315

The new field `fabric_type` can be 'IB' or 'ETH' to indicate the network fabric type
in order to use the correct client to process and populate the port or network
on the fabric.

Change-Id: I1a5b5ab572b868b82d56bc6e4d3d9ccc6215a4e8

This change performs some refactoring:

- remove internal DB dpenedency (eswitch_db, device_db)
  removing libvirt dependency altoghether

- Introduce EmbSwitch object to manage port operations on an
  embedded switch

- Introduce EswitchManager object to manage muliple embeddes
  switches.

- spearate infiniband specific functionality and move it to
  utils module.

- eswitchd API was modified to use PCI address as port identifier
  to reduce mac address dependency.

Change-Id: I79f9f356d0630e3efb2fa2624c3d9c2bad3d31fa

This change reduces the agent's dependency on mac address
when actin on port CRUD operations.

a minimal dependency is still kept since we rely on MAC
address to retrieve information from neutron server
(get_device_details RPC).

The following commit makes extensive changes to eswitchd
amongst the changes, get_vnics() response dict is now keyed
by PCI address and is reflected in this commit.

Due to the extent of the changes the two commits should be
merged in one pass and is partitioned here for easier review.

Change-Id: I762a6e851c8abac92fd74faddaa5700347c8352c

This commit clarifies two config opts description
that were recently added.

Change-Id: Ic94926a11b044facbca6851c7b17087d72b023aa