Skip to content

Commit a32be5d

Browse files
authored
Metrics bug: server_lib should be config_lib (#113)
* Metrics bug: server_lib should be config_lib * format
1 parent 958678c commit a32be5d

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

run_server.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
import jetstream_pt
2121
from absl import app, flags
2222
from jetstream.core import server_lib
23-
from jetstream.core.config_lib import ServerConfig
23+
from jetstream.core.config_lib import ServerConfig, MetricsServerConfig
2424
from jetstream_pt.config import FLAGS, create_engine_from_config_flags
2525

2626
flags.DEFINE_integer("port", 9000, "port to listen on")
@@ -49,11 +49,9 @@ def main(argv: Sequence[str]):
4949
)
5050
print(f"server_config: {server_config}")
5151

52-
metrics_server_config: server_lib.MetricsServerConfig | None = None
52+
metrics_server_config: MetricsServerConfig | None = None
5353
if FLAGS.prometheus_port != 0:
54-
metrics_server_config = server_lib.MetricsServerConfig(
55-
port=FLAGS.prometheus_port
56-
)
54+
metrics_server_config = MetricsServerConfig(port=FLAGS.prometheus_port)
5755

5856
# We separate credential from run so that we can unit test it with local credentials.
5957
# We would like to add grpc credentials for OSS.

0 commit comments

Comments
 (0)