Skip to content

Add new status code.#828

Merged
mint570 merged 2 commits intosonic-net:masterfrom
mint570:status_code_upstream
Mar 19, 2024
Merged

Add new status code.#828
mint570 merged 2 commits intosonic-net:masterfrom
mint570:status_code_upstream

Conversation

@mint570
Copy link
Contributor

@mint570 mint570 commented Nov 7, 2023

Add new status code.

SWSS_RC_NOT_EXECUTED: A request is not attempted to be performed. P4Orch will return this status for a request if there is any failure in the previous request in a batch.

SWSS_RC_FAILED_PRECONDITION: A request is not performed due to failure of meeting the preconditions.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Feb 29, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: prsunny / name: Prince Sunny (9e97270)
  • ✅ login: mint570 (7a94575)

@mint570 mint570 changed the title Add SWSS_RC_NOT_EXECUTED status code. Add new status code. Feb 29, 2024
Change-Id: I0e7fd00bba6e21770d4acf6965440463ccba5927
@qiluo-msft
Copy link
Contributor

@prsunny Could you help review?

@qiluo-msft qiluo-msft requested a review from prsunny March 11, 2024 23:54
@prsunny
Copy link
Contributor

prsunny commented Mar 18, 2024

lgtm, can you please provide details in description of whats the intention of the new code?

@mint570 mint570 merged commit e7917ac into sonic-net:master Mar 19, 2024
@mint570 mint570 deleted the status_code_upstream branch March 19, 2024 20:43
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.

4 participants