[orchagent]: Use priorities in orchagent #475
Merged
Conversation
This file contains 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
qiluo-msft
reviewed
Apr 17, 2018
| @@ -16,8 +16,10 @@ extern sai_object_id_t gSwitchId; | |||
| extern PortsOrch* gPortsOrch; | |||
| extern CrmOrch * gCrmOrch; | |||
|
|
|||
| const int fdborch_pri = 20; | |||
There was a problem hiding this comment.
fdborch_pri [](start = 10, length = 11)
It's inconvenient to scatter all the similar priority settings to many files.
qiluo-msft
reviewed
Apr 17, 2018
| @@ -36,6 +37,8 @@ const char config_db_key_delimiter = '|'; | |||
| #define CONFIGDB_KEY_SEPARATOR "|" | |||
| #define DEFAULT_KEY_SEPARATOR ":" | |||
|
|
|||
| const int default_orch_pri = 0; | |||
There was a problem hiding this comment.
default_orch_pri [](start = 10, length = 16)
Move it to class local?
lguohan
approved these changes
Apr 17, 2018
|
Hi Qi, I'm going to refactor the whole orchdaemon class to make it readable including priorities. I didn't want to refactor in this PR, otherwise this PR would be stuck for another week. |
EdenGri
pushed a commit
to EdenGri/sonic-swss
that referenced
this pull request
Feb 28, 2022
…onic-net#475) * Change PFC watchdog CONFIG_DB table name from PFC_WD_TABLE to PFC_WD Signed-off-by: Wenda Ni <wenni@microsoft.com>
oleksandrivantsiv
pushed a commit
to oleksandrivantsiv/sonic-swss
that referenced
this pull request
Mar 1, 2023
* Add default bridge id for bridge port id of type PORT in virtual switch * Update recording Virtual switch now supports Queue TYPE, so tests must be updated * Add specific comparison logic for tunnel map
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.
What I did
I've added support of Selectable priorities for orchagent.
I've put following priorities to orchagent tables:
All other orchagent run with lowest priority which is 0.
Why I did it
To prioritize messages inside of the orchagent message queue.
How I verified it
Build swss and run it
Details if related