Yingzhen-ietf / BGP-QUIC

3 stars 1 forks source link

Restart Bit #1

Closed aretana closed 3 years ago

aretana commented 3 years ago

Why do we need a Restart bit in the Capability? I'm assuming that all the other capabilities still apply, so Graceful Restart would handle a restart. ??

Yingzhen-ietf commented 3 years ago

Hi,

So you’re recovered from IETF week. :)

Yes, all other capabilities still apply. The R bit here is to indicate this is a reset session (in case only one direction in QUIC bidrectional streams got reset) , not sure we really need it (We can put some requirement about bidirectional implementation, for example, if one direction is reset, the other MUST be reset also), but we can reserve a flag field. We need to decide whether we include the AFI/SAFI in this capability, or use the one in MP extension capability.

Thanks, Yingzhen

On Aug 6, 2021, at 1:12 PM, Alvaro Retana @.***> wrote:

Why do we need a Restart bit in the Capability? I'm assuming that all the other capabilities still apply, so Graceful Restart would handle a restart. ??

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Yingzhen-ietf/BGP-QUIC/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/AURDC47LJPHWQRBSZZA62FTT3Q633ANCNFSM5BWREW5Q. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

aretana commented 3 years ago

Yes, if the session is reset in one direction is MUST be reset in the other. This is important so we don't have to change the general behavior of BGP.

I don't think we need the AFI/SAFI in the capability. I'm about to post something (as a new issue) describing a couple more flags.