Skip to content

Conversation

@ANISH-SR
Copy link
Contributor

@ANISH-SR ANISH-SR commented Jan 3, 2026

Summary of Changes

  • Enforced iface.status == InterfaceStatus::Activated before unlinking.
  • Returns DoubleZeroError::InvalidStatus if the interface is not activated.
  • CHANGELOG.md updated.

Testing Verification

  • rust & ci checks pass.

Closes #2230

@ANISH-SR ANISH-SR marked this pull request as draft January 3, 2026 06:58
@ANISH-SR ANISH-SR changed the title feat: iface activated [WIP] feat: iface activated Jan 10, 2026
Copy link
Contributor

@juan-malbeclabs juan-malbeclabs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@juan-malbeclabs juan-malbeclabs marked this pull request as ready for review January 12, 2026 17:31
@juan-malbeclabs juan-malbeclabs enabled auto-merge (squash) January 12, 2026 17:36
@juan-malbeclabs
Copy link
Contributor

@ANISH-SR The e2e test is failing: could you take a look? It seems the change may leave the interface in an invalid state. Thanks.

@ANISH-SR
Copy link
Contributor Author

Hi @juan-malbeclabs , I looked into the e2e failure and it seems to be quite a hassle here. if this change is not immediately needed, I'll go ahead and draft this for now. Rest assured, I'll recommit with some unit tests for the change. Thanks a ton :)

@ANISH-SR ANISH-SR marked this pull request as draft January 14, 2026 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

In UnlinkDeviceInterface, verify that iface.status == Activated

2 participants