mirror of
https://github.com/GreptimeTeam/greptimedb.git
synced 2026-08-25 15:38:34 +00:00
1693b2727c
* refactor: port query regression runner to Rust Signed-off-by: discord9 <discord9@163.com> * ci: remove optional OTLP report plotter Signed-off-by: discord9 <discord9@163.com> * refactor: split query regression runner into modules Signed-off-by: discord9 <discord9@163.com> * style: use crate-qualified imports in query regression runner Signed-off-by: discord9 <discord9@163.com> * refactor: simplify query regression runner internals Signed-off-by: discord9 <discord9@163.com> * feat: abstract inspect-footer storage access behind object store destination Add an optional --destination <TOML> to inspect-footer (and --base-destination/--candidate-destination to finalize-remote) so the storage inspection reads DB data files through the opendal-backed object_store abstraction instead of bare std::fs. Local paths keep working unchanged via the --root shortcut (File backend); remote backends (S3/GCS/...) are described by a DestinationConfig TOML reusing the object-store crate's ObjectStoreConfig serde shape. - inspect_footer: list via ObjectStore::list + ObjectMeta filtering (parquet keys, non-zero size, metadata/ segment), read footers async via ParquetObjectReader + ParquetMetaDataReader with known file size (no extra HEAD); output JSON schema unchanged - finalize-remote: --base-data-home/--candidate-data-home become optional, mutually exclusive with the new --*-destination args - cmd deps: add object_store_opendal + datafusion_object_store - tests: fs-backend list+footer integration tests (metadata filtering, destination TOML mode, root/destination exclusivity) Signed-off-by: discord9 <55937128+discord9@users.noreply.github.com> * style: drop needless borrow in inspect footer test Fix clippy::needless_borrows_for_generic_args in the inspect-footer test (fs::create_dir_all(table.join("metadata"))). Missed by the earlier focused clippy run because it only covered --bin targets. Signed-off-by: discord9 <55937128+discord9@users.noreply.github.com> --------- Signed-off-by: discord9 <discord9@163.com> Signed-off-by: discord9 <55937128+discord9@users.noreply.github.com>
94 lines
3.9 KiB
Python
94 lines
3.9 KiB
Python
#!/usr/bin/env python3
|
|
# Copyright 2023 Greptime Team
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
|
|
"""Regression coverage for the outer OTLP lifecycle."""
|
|
|
|
import importlib.util
|
|
import json
|
|
import sys
|
|
import tempfile
|
|
import unittest
|
|
from pathlib import Path
|
|
from types import SimpleNamespace
|
|
from unittest.mock import patch
|
|
|
|
|
|
RUNNER_PATH = Path(__file__).resolve().parents[2] / ".github/scripts/query-regression-run.py"
|
|
SPEC = importlib.util.spec_from_file_location("query_regression_outer_otlp_under_test", RUNNER_PATH)
|
|
assert SPEC is not None and SPEC.loader is not None
|
|
runner = importlib.util.module_from_spec(SPEC)
|
|
sys.modules[SPEC.name] = runner
|
|
SPEC.loader.exec_module(runner)
|
|
|
|
|
|
class OtlpTraceOuterLifecycleTest(unittest.TestCase):
|
|
def test_stops_base_before_candidate_and_wires_rust_commands(self) -> None:
|
|
events: list[str] = []
|
|
commands: list[list[str]] = []
|
|
|
|
def run(command, **_kwargs):
|
|
commands.append(command)
|
|
phase = command[1]
|
|
events.append(phase)
|
|
output = Path(command[command.index("--output") + 1])
|
|
output.write_text(json.dumps({"status": "ok", "targets": []}))
|
|
return SimpleNamespace(returncode=0)
|
|
|
|
def start(target, component, _procs):
|
|
events.append(f"start:{target.name}:{component}")
|
|
|
|
def stop(target, component, _procs):
|
|
events.append(f"stop:{target.name}:{component}")
|
|
|
|
args = SimpleNamespace(http_timeout="1", otelgen_bin=Path("otelgen"))
|
|
ports = list(range(10000, 10016))
|
|
with (
|
|
tempfile.TemporaryDirectory() as tmpdir,
|
|
patch.object(runner, "allocate_ports", return_value=ports),
|
|
patch.object(runner, "start_component", side_effect=start),
|
|
patch.object(runner, "stop_component", side_effect=stop),
|
|
patch.object(runner.subprocess, "run", side_effect=run),
|
|
):
|
|
root = Path(tmpdir)
|
|
self.assertEqual(
|
|
runner.run_otlp_case(args, Path("case.toml"), root, Path("base-bin"), Path("candidate-bin"), Path("fixture"), Path("runner")),
|
|
0,
|
|
)
|
|
expected = [
|
|
[
|
|
"runner", "run-otlp-target", "--case", "case.toml", "--fixture-generator", "fixture",
|
|
"--otelgen-bin", "otelgen", "--http-port", "10004", "--target-name", "base",
|
|
"--work-dir", str(root / "base"), "--output", str(root / "base" / "report.json"), "--http-timeout", "1",
|
|
],
|
|
[
|
|
"runner", "run-otlp-target", "--case", "case.toml", "--fixture-generator", "fixture",
|
|
"--otelgen-bin", "otelgen", "--http-port", "10012", "--target-name", "candidate",
|
|
"--work-dir", str(root / "candidate"), "--output", str(root / "candidate" / "report.json"), "--http-timeout", "1",
|
|
],
|
|
[
|
|
"runner", "finalize-otlp", "--case", "case.toml", "--fixture-generator", "fixture",
|
|
"--base-result", str(root / "base" / "report.json"),
|
|
"--candidate-result", str(root / "candidate" / "report.json"),
|
|
"--output", str(root / "query-regression-report.json"),
|
|
],
|
|
]
|
|
self.assertEqual(commands, expected)
|
|
|
|
self.assertLess(events.index("stop:base:metasrv"), events.index("start:candidate:metasrv"))
|
|
|
|
|
|
if __name__ == "__main__":
|
|
unittest.main()
|