Reviewed 9 of 9 files at r1.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @lukedirtwalker)

go/beacon_srv/internal/ifstate/ifstate.go, line 160 at r1 (raw file):

// and propagation are reset to the zero value. The return value indicates,
// whether the state is expired or revoked when the call returns.
func (intf *Interface) Expire() bool {

The doc string needs to be updated.
Probably, the method should now be called Revoked or similar.

go/beacon_srv/internal/ifstate/ifstate.go, line 179 at r1 (raw file):

// interface has been activated in the meantime and should not be revoked.
// This is indicated through an error.
func (intf *Interface) Revoke(rev *path_mgmt.SignedRevInfo) error {

Update doc string.
Should probably be called SetRevocation or similar

Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @oncilla)

go/beacon_srv/internal/ifstate/ifstate.go, line 160 at r1 (raw file):

Done.

go/beacon_srv/internal/ifstate/ifstate.go, line 179 at r1 (raw file):

Done.

Reviewed 6 of 6 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved

doc/BeaconService.md, line 139 at r2 (raw file):

The BS periodically evicts beacons that contain revoked interfaces.

### Interfaces State

💯 for remembering to update

Reviewable status: complete! all files reviewed, all discussions resolved

Reviewed 1 of 1 files at r3.
Reviewable status: complete! all files reviewed, all discussions resolved