Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Adding new states in WarmState Adapter. #905

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

divyagayathri-hcl
Copy link
Contributor

@divyagayathri-hcl divyagayathri-hcl commented Aug 27, 2024

Summary:

Add additional warm reboot states.
New states: FROZEN, QUIESCENT, CHECKPOINTED & FAILED.

Build Results:

divya@6faa6de402dd:/sonic/src/sonic-p4rt/sonic-swss-common$ bazel build $BAZEL_BUILD_OPTS ...
2024/08/27 19:05:30 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, 544 targets configured).
INFO: Found 4 targets...
INFO: Elapsed time: 121.915s, Critical Path: 6.25s
INFO: 196 processes: 98 internal, 98 linux-sandbox.
INFO: Build completed successfully, 196 total actions

Test Results:

divya@6faa6de402dd:/sonic/src/sonic-p4rt/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, 544 targets configured).
INFO: Found 2 targets and 2 test targets...
INFO: Elapsed time: 0.632s, Critical Path: 0.19s
INFO: 3 processes: 1 internal, 2 linux-sandbox.
INFO: Build completed successfully, 3 total actions
//tests:saiaclschema_ut PASSED in 0.1s
//tests:status_code_util_test PASSED in 0.1s

Executed 2 out of 2 tests: 2 tests pass.
INFO: Build completed successfully, 3 total actions

@kishanps
Copy link

@mint570 @akarshgupta25 Can you pls add the relevant repo owners to review and merge this PR ?

Copy link

@jaanah-hcl jaanah-hcl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants