Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

metadata.error_count が METADATA_RESOLVER_CIRCUIT_BREAK_COUNT 以上になっていると、expired であっても再取得が実行されない #1309

Open
shibafu528 opened this issue Oct 21, 2024 · 0 comments

Comments

@shibafu528
Copy link
Member

shibafu528 commented Oct 21, 2024

MetadataResolverServiceでのメタデータ解決時、expired_at で設定された時刻を過ぎると再取得判定になる。

この時、error_count が環境変数 METADATA_RESOLVER_CIRCUIT_BREAK_COUNT の値以上になっていると、サーキットブレイク処理が優先されて二度と再取得されなくなる。

expired_at によって再取得判定された場合、error_count を 0 に戻して取得すべきかもしれない。

expired_at を優先するか、サーキットブレイクのほうを優先するかはちょっと悩みどころ。
expiredならエラーが解消している可能性があるからリトライして良いとするか、エラーが出続けているのだからそちらを明示的にリセットしない限りはリトライすべきでないとするのか。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant