sonic-swss-common icon indicating copy to clipboard operation
sonic-swss-common copied to clipboard

[Banner] Added BANNER_MESSAGE table to ConfigDB

Open SviatoslavBoichuk opened this issue 2 years ago • 8 comments
trafficstars

Why I did it

Added Banner feature related Config DB table according to HLD: https://github.com/sonic-net/SONiC/pull/1361

How I did it

Added Banner table name to schema.h.

SviatoslavBoichuk avatar Oct 20 '23 14:10 SviatoslavBoichuk

@SviatoslavBoichuk please handle conflict and add to PR description code changes and motivation

liat-grozovik avatar Dec 03 '23 13:12 liat-grozovik

@SviatoslavBoichuk please handle conflict and add to PR description code changes and motivation

Fixed conflicts, updated PR description

SviatoslavBoichuk avatar Dec 15 '23 10:12 SviatoslavBoichuk

/azpw run sonic-net.sonic-host-services

SviatoslavBoichuk avatar Jan 02 '24 09:01 SviatoslavBoichuk

/AzurePipelines run sonic-net.sonic-host-services

mssonicbld avatar Jan 02 '24 09:01 mssonicbld

No pipelines are associated with this pull request.

azure-pipelines[bot] avatar Jan 02 '24 09:01 azure-pipelines[bot]

/AzurePipelines run sonic-net.sonic-host-services

SviatoslavBoichuk avatar Jan 02 '24 09:01 SviatoslavBoichuk

Commenter does not have sufficient privileges for PR 826 in repo sonic-net/sonic-swss-common

azure-pipelines[bot] avatar Jan 02 '24 09:01 azure-pipelines[bot]

CLA Signed

The committers listed above are authorized under a signed CLA.

  • :white_check_mark: login: fastiuk / name: Yevhen Fastiuk (f3812af929392a1aee7a1ff6378bee43dcfce716)

/azpw run Azure.sonic-swss-common

fastiuk avatar Apr 27 '24 21:04 fastiuk

@liat-grozovik the checkers finally passed, could you please merge it so I can rerun tests for other repositories?

fastiuk avatar Apr 28 '24 16:04 fastiuk

@qiluo-msft , Can you please merge this PR ?

dprital avatar May 06 '24 16:05 dprital