Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • NetworkManager NetworkManager
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 180
    • Issues 180
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 13
    • Merge requests 13
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • NetworkManager
  • NetworkManagerNetworkManager
  • Merge requests
  • !1065

Merged
Created Jan 18, 2022 by Thomas Haller@thallerOwner

[th/device-manual-l3cd-prio] Revert "nm-device: prefer manually configured addresses to automatic"

  • Overview 5
  • Commits 4
  • Pipelines 5
  • Changes 4

This commit does not seem correct. The enum was moved with the declared intent to make manual IP configuration preferred. But the code comment in L3ConfigDataType and all the code there work under the assumption that higher (numerical) values mean more important. So the patch actually made manual the least important method.

I don't necessarily doubt that this fixes seemingly a scenario, but then it does the wrong fix. If the priorities are not honored correctly (or inverted), then the fix must address that, and not reorder some priorities. In particular, when most of the code claims to work the other way around.

Revert this commit. If this breaks something again, then we can re-investigate.

There also appears to be a CI test for this [1]. Let's see how/whether it regresses.

[1] NetworkManager-ci@9cd1b24c

This reverts commit af1903fe.

Edited Jan 20, 2022 by Thomas Haller
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: th/device-manual-l3cd-prio