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

Change-Id: Id9c3db01fde37123f161543125433e14d661a001

Change-Id: Ic35976e9b45f2f2b949ddd2b330f6efc8d07a8c9

Changed the way the method check the updated port such that it will
check for extra opts then the mac, if any of the two conditions is
not met, the updated port get generated again

Change-Id: Ic87443966f5e39742b813ea2ca770261716071d2

Changed the get_vf_device_type way of getting the type of pf netdevices
by using sysfs to read the driver name instead of relying on the PCI device
ID.

Change-Id: I6f1f722dffcf990e3ae6b514fb560596cea23c4b

Added the support for the https protocol in neo sessions by adding
two new configuration options called cert_verify and cert_path, that
tells the sdn wheter to verify the cert or not, and specify the path
for the wanted cert file or directory if needed (self signed certificates)
In case of directory, the cert should either be renamed to its hash
value, or have a link to it that is named the hash value of the cert
for the verification to be successful.

https://requests.kennethreitz.org/en/master/user/advanced/

Change-Id: Ibea26ace7c140c8c2a06c5a1e01dbb963f8fd612

Change-Id: I759557b1ea06e474cca74e7988a339155e2efea9

This change transitions the following operations to be
executed by pyroute2:

1. set VF Admin state
2. set net device link state
3. set GUID

Today, eswitchd uses sysfs extensively to perform much
of the operations.
This change aims to reduce the use sysfs and performs
these operations in a more linux, pythonic way by
using netlink and utilizing pyroute2 library

Elevation of privleges are done by privsep which is
a safer model to be used than the existing rootwrap.

- Introduce netdev_ops abstraction - netdev_opts
- Implement abstraction using pyroute2
- Use privsep to elevate privleges to perform operations
- extend device_db to contain pf net device
- Use netdev_ops in eswitch_handler
  - When setting GUID, fallback to old sysfs functionality
    in case of failure

Change-Id: I807500e57b156fbaad85f42d70e9636c55beaa3d

This commit introduces the use of privsep in the porject.
In this commit we introduce a pattern that future commits will build
on when there is a requirement to perform operation at elevated
privleges.

- Add privsep admin context
- Add privsep configurations + some clenaup in configuration of eswitchd
- sort devstack deployment with proper configuration files
- add sys_ops module that performs system operations
- elevate sys_ops write_sys method via privsep
- use the new sys_ops module instead of current rootwrap + echo usage
- remove 'echo' command from rootwrap

Change-Id: I0feede9631bd48fdce73054e654d30a132c627c2