DatadogのContinuous Profilerを有効にしたら問題が一気に解決した

## はじめに
Datadogの[Continuous Profiler](https://docs.datadoghq.com/ja/profiler/)を有効にしたら、数か月悩んでいたパフォーマンスの問題が10分で解決したので、そのパワフルさを紹介していきます。

＃「Datadogすげー」な記事ではあるのですが、パフォーマンスに影響を与えていた原因自体は.NETの基本的なプログラミングの問題なので、個人的には自爆案件で毎年恒例のやらかしアドベントカレンダーに投稿しようかとも思った内容です。

## Continuous Profilerを設定する前の状況
Duende IdentityServerを利用してOIDCのIdPを構築しているのですが、ピーク時はTokenエンドポイントの呼び出しに10秒近くかかりアラートがバシバシ飛んでくる状況で、コンテナのCPU利用率が100%に張り付き、ヘルスチェックに応答できず、次々とOOM killerに殺されるみたいな状況が発生していました（念のための補足ですがDuende IdentityServerには性能劣化の問題はなく、自前のコードが原因です）。
![image.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/36535/0605930c-0232-98a3-83a9-54f079c20550.png)
もともとDatadog自体は導入していたので、APMのトレースを確認しては見たものの目立ったSPANは表示されず、特定のエンドポイントの原因というよりは何らかの原因によりCPU使用率が高騰し全体的に処理が遅くなっているということ以外はわからない状況でした。
![image.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/36535/422c8090-576b-9fa8-bd4c-ce3b19155db2.png)
また、調査のため１からDuende IdentityServerで構築したサンプルでは同様の問題が発生しないので、自前で実装したコードのどこかに問題はあるということだけはわかっていたのですが、とりあえず動いているという状況もあったのでコンテナ台数を増やして運用していました。

## DatadogのContinuous Profilerとは？
Datadogの[Continuous Profiler](https://docs.datadoghq.com/ja/profiler/)はメソッドの呼び出しレベルでパフォーマンスを追跡して継続的に監視するツールです。CPU、メモリ、I/Oのボトルネックを視覚的に表示してくれるので外形的な監視だけではわからない問題を特定することができます。

下のグラフはDuende IdentityServerのサンプルに、意図的に性能を劣化させるようなコードを仕込んだリクエストをContinuous Profilerで表示した例です。小さくて見づらいと思いますが、全体で930msかかる処理は大きく3つのブロック、2つのメソッド（[HasDuplicate](https://github.com/DuendeSoftware/IdentityServer/blob/main/src/IdentityServer/Extensions/IEnumerableExtensions.cs#L33)、[FindClientByIdAsync](https://github.com/DuendeSoftware/IdentityServer/blob/main/src/IdentityServer/Stores/InMemory/InMemoryClientStore.cs#L41)）で遅くなっていることがわかります。
![image.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/36535/7f36ca03-92ff-134d-112e-59f1304d5f35.png)

## Continuous Profilerを有効にする
.NETの場合Datadogをすでに導入しているのであれば、環境変数を追加するだけでContinuous Profilerが有効になります。詳細は[.NET プロファイラーの有効化](https://docs.datadoghq.com/ja/profiler/enabling/dotnet/?tab=linux)を参照してください。

前述の通りContinuous Profilerを有効にすると外形的な監視からはわからないようなメソッドごとの処理速度などを観測できるようになりますが、若干パフォーマンスのペナルティーがあるので気を付けてください。筆者の環境ではContinuous Profilerを有効にすることで2%程度のCPU増加が観測されました（点線が前週のCPU利用率、全体的にCPU利用率が上昇していることがわかる）。
![image.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/36535/b1633d53-0fa9-2776-fec1-532f30c45d6e.png)

## 原因を探る

Continuous Profilerで問題のメソッドを見ると、HasDuplicatesやFindClientByIdAsyncが内部で持っているClientの列挙に問題がありそうだなというのがすぐにわかります。
![image.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/36535/7f36ca03-92ff-134d-112e-59f1304d5f35.png)
クライアント定義の数なんてせいぜい数十の配列を列挙するだけで？と思いながらプロファイルの表示対象をOnly My CodeからAll Codeにすると...
![image.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/36535/4cf27403-04d8-1cd4-2bd8-a90f36873109.png)
あっ、単なる配列じゃなくてIConfigurationの検索から毎回評価されているようです。これは遅そうだ😢（ここまでプロファイラーを見始めてから5分）
![image.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/36535/28fd573f-8a9d-1320-cc38-6c65ed5b16ee.png)

## 修正する

IdentityServerの初期化コードで設定を引き渡している部分です。これまでIEnumerableで引き渡している部分をToArrayで具体化してから引き渡してあげます。

```diff:Program.cs
services.AddIdentityServer()
-    .AddInMemoryApiScopes(configuration.GetApiResourcesSection())
-    .AddInMemoryApiResources(configuration.GetApiResourcesSection())
-    .AddInMemoryIdentityResources(GetIdentityResources())
-    .AddInMemoryClients(configuration.GetIdentityClientsSection())
+    .AddInMemoryApiScopes(configuration.GetApiResourcesSection().ToArray())
+    .AddInMemoryApiResources(configuration.GetApiResourcesSection().ToArray())
+    .AddInMemoryIdentityResources(GetIdentityResources().ToArray())
+    .AddInMemoryClients(configuration.GetIdentityClientsSection().ToArray())
    ...略...
```    

同一メソッド内であれば、IEnumerableを複数列挙するようなコードはVisual Studioが指摘してくれるのですが、今回のようにミドルウェア内部で列挙されるようなものは見つけ出すのがかなり困難です。Continuous Profilerを導入していなかったら原因解明までにあと何か月かかったことか、、、

## 確認する

上記の修正を適用したタイミングから、サイト全体のレイテンシ（aspnet_core_request）は3秒から50ミリ秒に、CPU利用率は30%から0.3%まで改善しました。
![image.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/36535/8c785b59-02e6-d726-9bbb-539480c5f527.png)

## おわりに

IEnumerableのまま渡していたのは過去の自分なので完全に自爆案件なのですが、改めて可観測性の重要さを知る機会でした。