logging copybara export CW_09#71
Draft
rmaddikery wants to merge 12 commits intoeclipse-score:mainfrom
Draft
Conversation
GIT_ORIGIN_SPP_REV_ID: dbd744b4b707c9279c12b2717f4e80be456938fc
GIT_ORIGIN_SPP_REV_ID: 6fd59fac1193f07b682bf5dd008ac6e2c9d03203
GIT_ORIGIN_SPP_REV_ID: b04d180da1a9375664c128050fc70943b93aba3e
- platform/aas/mw/log/detail GIT_ORIGIN_SPP_REV_ID: 9779b2453267c0166c1e92acfaf27bf67f8beca2
If we want to make sure that the rules C++ and Python loaded are the version we defined and not the one in Bazel, we need to explicitly add the load statement. This also becomes mandatory in Bazel 9. These changes have been done automatically with buildifier fix after adding native-cc and native-py. GIT_ORIGIN_SPP_REV_ID: c820f1811d8c6d42332706badb1e1a8a6f9b2dbc
GIT_ORIGIN_SPP_REV_ID: 1b4f6756ba4c1fc71667d7190bec17a68c23383b
The problem is that buildifier put the rule_cc inclusion in beginning of a file because it assumed that the copyright header is a comment for the next statement (cc_library, test_suite,...) due to a missing empty line in-between. GIT_ORIGIN_SPP_REV_ID: f8420e7798965cd7380b13c2a892378805e6f7b2
GIT_ORIGIN_SPP_REV_ID: b56a619ccccc3fd3fe672d0402be408710c08a1d
- Either the real or mock UdpStreamOutput should be linked into socketserverUT otherwise this leads to layout mismatch and heap corruption as seen in InitLogChannels - Fix clang_tidy_extra_checks for stub code parts GIT_ORIGIN_SPP_REV_ID: 1a9198cf43559e29c1bfba7af1b22d7132e70d3d
Remove session-based parser/callback tests for SET_LOG_CHANNEL_THRESHOLD, SET_MESSAGING_FILTERING_STATE, and SET_DEFAULT_LOG_LEVEL. Add behavior-focused direct-call tests to verify threshold filtering, filtering toggle behavior, and default log level behavior. Tested: bazel test --config=spp_host_gcc //platform/aas/pas/logging/test/ut/ut_logging:dltserverUT bazel test --config=arm64-qnx-unit-tests //platform/aas/pas/logging/test/ut/ut_logging:dltserverUT_microvm_qnx_5 GIT_ORIGIN_SPP_REV_ID: 657d932afb5b3a78581969bd99d717795cb60877
GIT_ORIGIN_SPP_REV_ID: 6a14ce49ce93029ecf9882f5aaecafe29be846cf
License Check Results🚀 The license check job ran with the Bazel command: bazel run //:license-checkStatus: Click to expand output |
|
The created documentation from the pull request is available at: docu-html |
- Required for migrating changes due to extra_clang_tidy_fixes
e8ee867 to
3baf953
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Notes for Reviewer
Pre-Review Checklist for the PR Author
Checklist for the PR Reviewer
Post-review Checklist for the PR Author
References
Closes #