Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Petro Bratash [email protected]
- Why I did it
pci-check.service and pcied daemon set different variables in STATE_DB.
pci-check.service set
"PCIE_STATUS|PCIE_DEVICES"
variable with type string,pcied daemon set
"PCIE_STATUS|" -> "PCIE_DEVICES"
variable with type hash.- How I did it
Change type for variable with PCI device status from
string
tohash
("PCIE_STATUS|PCIE_DEVICES" --> "PCIE_DEVICES" "status"), because haven't got API for work with string variable from Python.- How to verify it
- Which release branch to backport (provide reason below if selected)
- Description for the changelog
Also, in this PR fixed the issues:
PCIE_CHK_CMD
the variable contains the result of the command work's, and this result cyclically compares withEXPECTED
without rescanning devices.Added dependency to
database.service
, because whenpcie-check.service
start beforedatabase.service
will be next error:Related PR: sonic-net/sonic-platform-daemons#93
Fix for: #5349
- A picture of a cute animal (not mandatory but encouraged)