Skip to content

Feat: add bazel build (S-Core Integration) - #488

Draft
theswiftfox wants to merge 4 commits into
mainfrom
feat/bazel-build
Draft

Feat: add bazel build (S-Core Integration)#488
theswiftfox wants to merge 4 commits into
mainfrom
feat/bazel-build

Conversation

@theswiftfox

Copy link
Copy Markdown
Contributor

Summary

Clean slate follow up for #348

Closes #439

Checklist

  • I have tested my changes locally
  • I have added or updated documentation
  • I have linked related issues or discussions
  • I have added or updated tests

Related

Notes for Reviewers


Elena Gantner elena.gantner@mercedes-benz.com, Mercedes-Benz Tech Innovation GmbH
Provider Information

Signed-off-by: Elena Gantner <elena.gantner@mercedes-benz.com>
Signed-off-by: Elena Gantner <elena.gantner@mercedes-benz.com>
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

✅ No CRAP regressions

↑ 0 regressed · ★ 0 new · ↔ 0 moved · ↓ 1 improved · 1617 unchanged · — 96 removed
✓ = clean, ▲ = moderate, ✗ = crappy; Δ = change since baseline; CC = cyclomatic complexity; Cov % = line coverage percentage

↓ 1 improved
CRAP Δ CC Cov % Function Location
4.0 -0.4 4 91.7 UdsManager::start_variant_detection_for_ecus cda-comm-uds/src/variant.rs:43
🔥 Top hot spots above threshold
CRAP CC Cov % Function Location
600.0 24 0.0 EcuManager::map_table_struct_from_uds cda-core/src/diag_kernel/payload_decode.rs:670
306.0 17 0.0 dop_variant_to_schema cda-core/src/diag_kernel/schema.rs:410
210.0 14 0.0 DoipDiagGateway::send_functional cda-comm-doip/src/lib.rs:648
210.0 14 0.0 UdsManager::send_functional_group cda-comm-uds/src/functional_group.rs:193
156.0 12 0.0 update_mdd_uncompressed cda-database/src/mdd_data.rs:364
156.0 12 0.0 has_new_artifacts cda-storage/src/recovery.rs:265
156.0 12 0.0 diag_service::functional_data_request cda-sovd/src/sovd/functions/functional_groups/data.rs:243
110.0 10 0.0 DiagServiceError::from cda-comm-doip/src/connections.rs:73
110.0 10 0.0 comparams::executions::id::get cda-sovd/src/sovd/components/ecu/operations.rs:273
90.0 9 0.0 params_to_schema cda-core/src/diag_kernel/schema.rs:188
90.0 9 0.0 add_phys_constr_range cda-core/src/diag_kernel/schema.rs:306
90.0 9 0.0 EcuManager::map_table_key_from_uds cda-core/src/diag_kernel/payload_decode.rs:622
90.0 9 0.0 establish_tls_ecu_connection cda-comm-doip/src/ecu_connection.rs:406
90.0 9 0.0 create_tls_stream cda-comm-doip/src/ecu_connection.rs:501
90.0 9 0.0 remove_descriptions_recursive cda-sovd/src/sovd.rs:923
85.5 16 35.2 service::executions::ecu_reset_handler cda-sovd/src/sovd/components/ecu/operations.rs:1346
72.0 8 0.0 UdsManager::send_functional_to_gateway cda-comm-uds/src/functional_group.rs:39
72.0 8 0.0 UdsManager::set_functional_state cda-comm-uds/src/functional_group.rs:398
72.0 8 0.0 comparams::executions::id::put cda-sovd/src/sovd/components/ecu/operations.rs:401
72.0 8 0.0 generate_config_cmd cda-main/src/lib.rs:213
70.6 10 15.4 DiagDataValue::try_into cda-core/src/diag_kernel.rs:211
60.3 11 25.9 load_config_with_storage_override cda-main/src/config.rs:147
56.0 7 0.0 build_mode_response cda-sovd/src/sovd/functions/functional_groups/modes.rs:162
56.0 7 0.0 get_components cda-sovd/src/sovd.rs:497
56.0 7 0.0 AppError::from cda-main/src/error.rs:45

…and 31 more, see CI artifact for the full report.

— 96 removed
  • MbedtlsError::name (was 272.0) — comm-mbedtls/mbedtls-rs/src/error.rs
  • TlsStream::poll_read (was 240.0) — comm-mbedtls/mbedtls-rs/src/async_stream.rs
  • TlsStream::async_handshake (was 72.0) — comm-mbedtls/mbedtls-rs/src/async_stream.rs
  • TlsStream::connect (was 42.0) — comm-mbedtls/mbedtls-rs/src/async_stream.rs
  • TlsStream::check_write_flush (was 42.0) — comm-mbedtls/mbedtls-rs/src/async_stream.rs
  • TlsStream::poll_write (was 42.0) — comm-mbedtls/mbedtls-rs/src/async_stream.rs
  • TlsStream::poll_shutdown (was 42.0) — comm-mbedtls/mbedtls-rs/src/async_stream.rs
  • MaxFragLen::to_raw (was 42.0) — comm-mbedtls/mbedtls-rs/src/ssl/config.rs
  • bio_send (was 30.0) — comm-mbedtls/mbedtls-rs/src/ssl/context.rs
  • bio_recv (was 30.0) — comm-mbedtls/mbedtls-rs/src/ssl/context.rs
  • SslStream::connect (was 30.0) — comm-mbedtls/mbedtls-rs/src/ssl/context.rs
  • SslStream::complete_handshake (was 30.0) — comm-mbedtls/mbedtls-rs/src/ssl/context.rs
  • SslStream::read (was 30.0) — comm-mbedtls/mbedtls-rs/src/ssl/context.rs
  • Error::from (was 20.0) — comm-mbedtls/mbedtls-rs/src/error.rs
  • TlsStream::flush_outgoing (was 20.0) — comm-mbedtls/mbedtls-rs/src/async_stream.rs
  • TlsStream::shutdown (was 20.0) — comm-mbedtls/mbedtls-rs/src/async_stream.rs
  • PrivateKey::from_file (was 20.0) — comm-mbedtls/mbedtls-rs/src/x509.rs
  • SslVerifyMode::to_raw (was 20.0) — comm-mbedtls/mbedtls-rs/src/ssl/config.rs
  • MbedtlsError::fmt (was 12.0) — comm-mbedtls/mbedtls-rs/src/error.rs
  • TlsStream::accept (was 12.0) — comm-mbedtls/mbedtls-rs/src/async_stream.rs
  • TlsStream::new_inner (was 12.0) — comm-mbedtls/mbedtls-rs/src/async_stream.rs
  • X509Certificate::from_pem (was 12.0) — comm-mbedtls/mbedtls-rs/src/x509.rs
  • X509Certificate::from_file (was 12.0) — comm-mbedtls/mbedtls-rs/src/x509.rs
  • PrivateKey::from_pem (was 12.0) — comm-mbedtls/mbedtls-rs/src/x509.rs
  • SslStream::new_inner (was 12.0) — comm-mbedtls/mbedtls-rs/src/ssl/context.rs

…and 71 more, see CI artifact for the full report.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

📊 Coverage Report (unit + DoIP/CAN/mixed integration)

Total line coverage: 75.3%

Full build artifacts

Signed-off-by: Elena Gantner <elena.gantner@mercedes-benz.com>
@theswiftfox

Copy link
Copy Markdown
Contributor Author

@FScholPer if you want you can test this PR for integration in s-core. It is built up cleanly from mainbranch and uses the external mbedtls module.
note: it is draft as the mbedtls is not yet merged to main and therefore the refs are not final.

Signed-off-by: Elena Gantner <elena.gantner@mercedes-benz.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add bazel build (S-Core Integration)

1 participant