Skip to content

Regression in CI coverage for (at least) some Ethernet drivers after net_app code removal #13481

@pfalcon

Description

@pfalcon

48abdc2, which was part of #12855, removed echo_server (non-socket version) sample app, which was the "default" sample for networking smoke-testing. As such, it included many config overlays test configs (in sample.yaml) to cover various option combinations and drivers. All of them were lost.

Example of lost test configs are those added in commits 7117b73 and e779f91 - and those only the configs added literally 10 days before they were blind-removed like that, there're probably more useful configs added over last 2 years.

Bottom line:

  1. There were shared concerns that net_app removal from 1.14 is a hasty move.
  2. Instead of blind-removing (rm -rf *-style) everything like that, test config should have been ported over the other sample(s). (With RFC regarding what becomes a new default "smoke test app").
  3. Not done at right time, it needs to be done now.

Metadata

Metadata

Assignees

Labels

area: NetworkingbugThe issue is a bug, or the PR is fixing a bugpriority: mediumMedium impact/importance bug

Type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions