Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[202205] [Submodule] Update for sonic-restapi #16993

Merged
merged 1 commit into from
Oct 25, 2023

Conversation

prsunny
Copy link
Contributor

@prsunny prsunny commented Oct 24, 2023

Why I did it

Submodule update for sonic-restapi

ccad4a2 - 2023-10-17 : [Tunnel] Support co-existence of IPv4 and IPv6 tunnels (#147) [Prince Sunny]
c8fa96b - 2023-10-12 : Remove command to install libhiredis deb file (#146) [Saikrishna Arcot]

Work item tracking
  • Microsoft ADO 25072916:

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@prsunny prsunny requested a review from lguohan as a code owner October 24, 2023 23:06
@prsunny prsunny requested a review from yxieca October 25, 2023 17:44
@yxieca yxieca merged commit e367b00 into sonic-net:202205 Oct 25, 2023
15 checks passed
@yxieca
Copy link
Contributor

yxieca commented Oct 25, 2023

@xumia @liushilongbuaa do you know why sonic-restapi submodule is not advanced by the automation?

@liushilongbuaa
Copy link
Contributor

liushilongbuaa commented Oct 26, 2023

@yxieca ,

From 970edb10645ec193b307e70dc3a72854311190b6 Mon Sep 17 00:00:00 2001
From: Prince <[email protected]>
Date: Tue, 24 Oct 2023 23:01:26 +0000
Subject: [PATCH] [Submodule] Update for sonic-restapi

---
 src/sonic-restapi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/sonic-restapi b/src/sonic-restapi
index a69ba06560e..ccad4a24b3e 160000
--- a/src/sonic-restapi
+++ b/src/sonic-restapi
@@ -1 +1 @@
-Subproject commit a69ba06560eb9d2a9b5ace43e22e9665fa986d33
+Subproject commit ccad4a24b3ea2dd1afd3391f7e99864bd1414976

a69 is on master branch. ccad is on 202205 branch.
202205 don't have updates recently.
It's better to use correct branch to avoid potential issues.

@yxieca
Copy link
Contributor

yxieca commented Oct 26, 2023

@prsunny can you fix the submodule head?

yxieca added a commit that referenced this pull request Oct 26, 2023
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.

3 participants