bgp: idle hold timer should not be running outside idle state

During testing I'm seeing this in the `mgd` logs

```
WARN slog-rs: [transit] event idle hold timer expires not allowed in connect
```

This timer should not be running unless we're in the idle state.