commit 522cd98aecaf678c29206cf56db1f1fb17fd27d1
author Congbin Guo <guocb@google.com> Thu Oct 06 11:47:28 2022 -0700
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Oct 11 23:15:37 2022 +0000
tree 57059a17a0b8f0936aeedadbfa334d25e2442cb5
parent 9f7517aef3542212c0ad9474ebf91129a836460f

chore(sysmon): update process names to monitor

We add more chrome fleet services to monitor, e.g. fleet-tlw,
common-tls, etc. Also, we deprecated a legacy service, i.e.
gs_archive_server from the list.

BUG=b:238798008
TEST=unit test

Change-Id: I5157193b8fce42887f3288ed64ff037d41a1f31c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3937607
Commit-Queue: Congbin Guo <guocb@chromium.org>
Auto-Submit: Congbin Guo <guocb@chromium.org>
Tested-by: Congbin Guo <guocb@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>

scripts/sysmon/proc_metrics_unittest.py

diff --git a/scripts/sysmon/proc_metrics_unittest.py b/scripts/sysmon/proc_metrics_unittest.py
index 339152a..ef3d38e 100644
--- a/scripts/sysmon/proc_metrics_unittest.py
+++ b/scripts/sysmon/proc_metrics_unittest.py
@@ -170,18 +170,6 @@
                 _mock_process(
                     name="curl", cmdline=["curl", "server:port/path"]
                 ),
-                _mock_process(
-                    name="python",
-                    cmdline=[
-                        "python",
-                        "-m",
-                        "gs_archive_server",
-                        "-p",
-                        "18000",
-                        "-c",
-                        "127.0.0.1:8082",
-                    ],
-                ),
                 _mock_process(name="java", cmdline=["java", "-Xmx4g", "..."]),
                 _mock_process(
                     name="python",
@@ -195,23 +183,56 @@
                         "gs://eve-release/R100-14488.0.0/file",
                     ],
                 ),
+                _mock_process(
+                    name="common-tls",
+                    cmdline=["/opt/infra-tools/common-tls", "-port", "..."]
+                ),
+                _mock_process(
+                    name="fleet-tlw",
+                    cmdline=["/opt/infra-tools/fleet-tlw", "-port", "..."]
+                ),
+                _mock_process(
+                    name="drone-agent",
+                    cmdline=["/opt/infra-tools/drone-agent"]
+                ),
+                _mock_process(
+                    name="dnsmasq",
+                    cmdline=["dnsmasq", "..."]
+                ),
+                _mock_process(
+                    name="labservice",
+                    cmdline=["/opt/infra-tools/labservice", "-addr", "..."]
+                ),
+                _mock_process(
+                    name="cloud_sql_proxy",
+                    cmdline=[
+                        "/opt/cloud_sql_proxy", "-dir=/var/run/tko_proxy",
+                        "-instances=google.com:chromeos-lab:us-central1:tko",
+                        "-credential_file=..."
+                    ],
+                ),
             ]
             proc_metrics.collect_proc_info()
 
         setter = self.store.set
         calls = []
         calls.extend(_expected_calls_for("autoserv"))
+        calls.extend(_expected_calls_for("common-tls"))
         calls.extend(_expected_calls_for("curl"))
+        calls.extend(_expected_calls_for("dnsmasq"))
+        calls.extend(_expected_calls_for("drone-agent"))
+        calls.extend(_expected_calls_for("fleet-tlw"))
         calls.extend(_expected_calls_for("getty"))
-        calls.extend(_expected_calls_for("gs_archive_server"))
         calls.extend(_expected_calls_for("gs_offloader"))
         calls.extend(_expected_calls_for("gsutil"))
         calls.extend(_expected_calls_for("java"))
+        calls.extend(_expected_calls_for("labservice"))
         calls.extend(_expected_calls_for("lxc-attach"))
         calls.extend(_expected_calls_for("lxc-start"))
         calls.extend(_expected_calls_for("sshd"))
         calls.extend(_expected_calls_for("swarming_bot"))
         calls.extend(_expected_calls_for("sysmon"))
+        calls.extend(_expected_calls_for("tko_proxy"))
         calls.extend(_expected_calls_for("other"))
         setter.assert_has_calls(calls)
         self.assertEqual(len(setter.mock_calls), len(calls))