dent-atrifact: Use Marvell FW v3.1.2

dentproject / dentOS

dentOS SwitchDev based NOS

Other

200 stars 59 forks source link

dent-atrifact: Use Marvell FW v3.1.2 #158

Closed taraschornyiplv closed 2 years ago

taraschornyiplv commented 2 years ago

Update dent-artifact submodule to include FW v3.1.2 from Marvell

Signed-off-by: Taras Chornyi taras.chornyi@plvision.eu

Mickey201 commented 2 years ago

Is it true, that only one bug is fixed?

Yes, Paul. only one issue was required to be resolved. Why do you sound surprised? We have a quite good validation team 🥇

paulmenzel commented 2 years ago

Is it true, that only one bug is fixed?

Yes, Paul. only one issue was required to be resolved. Why do you sound surprised? We have a quite good validation team

I didn’t mean to sound surprised. I just asked, because these terse commit messages leave room for interpretation.

It’d be great if you added a link to the issue/bug for the traffic dropping issue. As it’s binary firmware, it’s especially hard to verify changes, so it’‘d be great to have at least a way to reproduce the issue.

jmpolom commented 2 years ago

I agree with @paulmenzel that the lack of detail in commits, lack of a public detailed changelog or links to specific issues resolved make it nearly impossible for end users to determine what, exactly, is being fixed or supported when. We are to take it on faith or from third parties which is suboptimal. This needs to be improved.

snobleaws commented 2 years ago

Are we able to merge this?

taraschornyiplv commented 2 years ago

yes we are