[swss-common] Define new VLAN P4 table names #1015
Open
Conversation
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
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
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.
Summary:
Dedine new VLAN P4 table names
Bazel build and Test logs:
jaanahg@d2bbd80da15e:/sonic/src/sonic-swss-common$ bazel build $BAZEL_BUILD_OPTS ...
2025/05/07 06:40:38 Downloading https://releases.bazel.build/4.0.0/release/bazel-4.0.0-linux-x86_64...
Downloading: 46 MB out of 46 MB (100%)
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
INFO: Analyzed 4 targets (30 packages loaded, 547 targets configured).
INFO: Found 4 targets...
INFO: From Compiling googletest/src/gtest-death-test.cc:
external/com_google_googletest/googletest/src/gtest-death-test.cc: In function 'bool testing::internal::StackGrowsDown()':
external/com_google_googletest/googletest/src/gtest-death-test.cc:1301:24: warning: 'dummy' may be used uninitialized [-Wmaybe-uninitialized]
1301 | StackLowerThanAddress(&dummy, &result);
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
external/com_google_googletest/googletest/src/gtest-death-test.cc:1290:13: note: by argument 1 of type 'const void*' to 'void testing::internal::StackLowerThanAddress(const void*, bool*)' declared here
1290 | static void StackLowerThanAddress(const void* ptr, bool* result) {
| ^~~~~~~~~~~~~~~~~~~~~
external/com_google_googletest/googletest/src/gtest-death-test.cc:1299:7: note: 'dummy' declared here
1299 | int dummy;
| ^~~~~
INFO: Elapsed time: 26.908s, Critical Path: 5.39s
INFO: 199 processes: 100 internal, 99 linux-sandbox.
INFO: Build completed successfully, 199 total actions
jaanahg@d2bbd80da15e:/sonic/src/sonic-swss-common$ bazel test $BAZEL_BUILD_OPTS --cache_test_results=no ...
INFO: Build option --cache_test_results has changed, discarding analysis cache.
INFO: Analyzed 4 targets (0 packages loaded, 547 targets configured).
INFO: Found 2 targets and 2 test targets...
INFO: Elapsed time: 0.320s, Critical Path: 0.08s
INFO: 3 processes: 1 internal, 2 linux-sandbox.
INFO: Build completed successfully, 3 total actions
//tests:saiaclschema_ut PASSED in 0.0s
//tests:status_code_util_test PASSED in 0.0s
Executed 2 out of 2 tests: 2 tests pass.
INFO: Build completed successfully, 3 total actions
jaanahg@d2bbd80da15e:/sonic/src/sonic-swss-common$