Comment 2 for bug 1966471
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to config (master) | #2 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
1b1ed1a
(Get the code!)
Reviewed: https:/
Committed: https:/
Submitter: "Zuul (22348)"
Branch: master
commit a7b47f6197597a6
Author: Douglas Henrique Koerich <email address hidden>
Date: Fri Mar 25 14:14:41 2022 -0300
Prevent overwrite of sensible FEC configuration
Regardless these precedent changes:
https:/
https:/
https:/
that intended to avoid the overwrite of values in database for FEC
device configuration with the actual (sometimes empty or temporary)
values coming from PCI inventory report, it has been still noticed that
under some unexpected reboot such sensible, stored data could be lost.
This change implements for FEC driver and VF driver the same approach
taken for number of configured VFs versus actual VFs introduced by the
changes:
https:/
https:/
https:/
From now on, the configured FEC driver and VF driver will be available
as "extra information" (in the 'extra_info' field, together with
configured number of VFs) that can be compared with the actual settings
collected from PCI inventory for the host.
Test Plan:
PASS: Fresh install of r/stx7.0 with subsequent FEC configuration;
PASS: Backup & restore of r/stx7.0 after FEC configuration;
PASS: Patch of r/stx5.0 with previous FEC configuration;
PASS: Upgrade from r/stx5.0 with previous FEC configuration to patched
r/stx6.0.
PASS: Upgrade from patched r/stx5.0 with previous FEC configuration to
patched r/stx6.0.
Closes-Bug: #1966471
Signed-off-by: Douglas Henrique Koerich <email address hidden>
Change-Id: I5e2e6f55856d5e