Merge pull request #24 from ZORDxDD/main

ci: add GitHub Actions CI for fmt, clippy, build and tests

Author: kmesh-bot

.github/workflows/ci.yml

@@ -0,0 +1,81 @@
+name: CI — fmt, clippy, build, test
+
+on:
+  push:
+    branches: [ main, master ]
+  pull_request:
+    branches: [ main, master ]
+
+env:
+  CARGO_CACHE_PATH: ~/.cargo/registry
+  CARGO_GIT_PATH: ~/.cargo/git
+  TARGET_DIR: target
+
+jobs:
+  # Fast checks that run in parallel (no build artifacts needed)
+  checks:
+    name: fmt + clippy
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      
+      - name: Initialize git submodules
+        run: git submodule update --init --recursive
+      
+      - name: Cache cargo
+        uses: actions/cache@v4
+        with:
+          path: |
+            ${{ env.CARGO_CACHE_PATH }}
+            ${{ env.CARGO_GIT_PATH }}
+          key: ${{ runner.os }}-cargo-checks-${{ hashFiles('**/Cargo.lock', 'envoy-data-plane-api/build.rs') }}
+      
+      - name: Install protoc
+        run: sudo apt-get update && sudo apt-get install -y protobuf-compiler
+      
+      - name: Setup Rust
+        uses: actions-rs/toolchain@v1
+        with:
+          toolchain: stable
+          components: rustfmt, clippy
+          profile: minimal
+      
+      - name: Check formatting
+        run: cargo fmt --all -- --check
+      
+      - name: Run clippy (fail on warnings)
+        run: cargo clippy --all-targets --all-features -- -D warnings
+
+  # Build and test that share artifacts
+  build-and-test:
+    name: build + test
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      
+      - name: Initialize git submodules
+        run: git submodule update --init --recursive
+      
+      - name: Cache cargo
+        uses: actions/cache@v4
+        with:
+          path: |
+            ${{ env.CARGO_CACHE_PATH }}
+            ${{ env.CARGO_GIT_PATH }}
+            ${{ env.TARGET_DIR }}
+          key: ${{ runner.os }}-cargo-build-${{ hashFiles('**/Cargo.lock', 'envoy-data-plane-api/build.rs') }}
+      
+      - name: Install protoc
+        run: sudo apt-get update && sudo apt-get install -y protobuf-compiler
+      
+      - name: Setup Rust
+        uses: actions-rs/toolchain@v1
+        with:
+          toolchain: stable
+          profile: minimal
+      
+      - name: Build (release)
+        run: cargo build --workspace --release --locked
+      
+      - name: Run tests
+        run: cargo test --workspace --release --locked

Makefile

@@ -0,0 +1,47 @@
+.PHONY: fmt fmt-check lint build test ci ci-parallel init
+
+fmt:
+	cargo fmt --all
+
+fmt-check:
+	cargo fmt --all -- --check
+
+lint:
+	cargo clippy --all-targets --all-features -- -D warnings
+
+build:
+	cargo build --workspace --release --locked
+
+test:
+	cargo test --workspace --release --locked
+
+# Sequential CI (keep for local/dev reproducibility)
+ci: init
+	cargo fmt --all -- --check
+	cargo clippy --all-targets --all-features -- -D warnings
+	cargo build --workspace --release --locked
+	cargo test --workspace --release --locked
+
+# Parallel CI: run fmt-check, lint and build concurrently; if build succeeds, run tests.
+ci-parallel: init
+	@echo "Starting parallel CI: fmt-check, lint, build"
+	@bash -ec '\
+	set -o pipefail; \
+	cargo fmt --all -- --check & pid_fmt=$$!; \
+	cargo clippy --all-targets --all-features -- -D warnings & pid_clippy=$$!; \
+	cargo build --workspace --release --locked & pid_build=$$!; \
+	wait $$pid_fmt; rc_fmt=$$?; \
+	wait $$pid_clippy; rc_clippy=$$?; \
+	wait $$pid_build; rc_build=$$?; \
+	if [ $$rc_fmt -ne 0 ]; then echo ">>> fmt-check failed (exit $$rc_fmt)"; fi; \
+	if [ $$rc_clippy -ne 0 ]; then echo ">>> clippy failed (exit $$rc_clippy)"; fi; \
+	if [ $$rc_build -ne 0 ]; then echo ">>> build failed (exit $$rc_build)"; fi; \
+	# Fail fast if any of the three failed (so PR check fails with non-zero code) \
+	if [ $$rc_fmt -ne 0 ] || [ $$rc_clippy -ne 0 ] || [ $$rc_build -ne 0 ]; then exit 1; fi; \
+	# If we reach here, build succeeded, so run tests \
+	cargo test --workspace --release --locked; \
+	'
+
+init:
+	@echo "Initializing git submodules..."
+	@git submodule update --init --recursive

orion-configuration/src/config.rs

@@ -57,7 +57,11 @@ impl Config {
         let runtime = self.runtime.update_from_env_and_options(opt);
         let max_cpus = num_cpus::get();
         if runtime.num_cpus() > max_cpus {
-            tracing::warn!(max_cpus, ORION_GATEWAY_CORES = runtime.num_cpus(), "Requested more cores than available CPUs");
+            tracing::warn!(
+                max_cpus,
+                ORION_GATEWAY_CORES = runtime.num_cpus(),
+                "Requested more cores than available CPUs"
+            );
         }
         if runtime.num_runtimes() > runtime.num_cpus() {
             tracing::warn!(

orion-proxy/tests/configs.rs

@@ -32,7 +32,6 @@ fn check_config_file(file_path: &str) -> Result<(), orion_error::Error> {
     with_current_dir(&d, || get_listeners_and_clusters(bootstrap).map(|_| ()))
 }
 
-
 #[traced_test]
 #[test]
 fn bootstrap_demo_static() -> Result<(), orion_error::Error> {