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.

Sign up for GitHub

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

Jump to bottom

APIs to support streaming structured events #649

Closed
wants to merge 117 commits into from
Closed

APIs to support streaming structured events #649

wants to merge 117 commits into from

Conversation

renukamanavalan
Copy link
Contributor

Refer: HLD #954

  1. APIs to publish & receive
  2. Service APIs to cache events offline
  3. C-binding for APIs to facilitate use from golang.

renukamanavalan and others added 30 commits May 13, 2022 14:30
@renukamanavalan
first cut
d883c01
@renukamanavalan
first cut lib code with unit test
c223d04
@renukamanavalan
save it just in case, as VM under risk
7fbdcf1
@renukamanavalan
partial update, as vm at risk
9763d3a
@renukamanavalan
intermediate saving
d9d24b6
@renukamanavalan
First run code complete
d68c204
@renukamanavalan
self review update
d130b84
@renukamanavalan
partial compile
63db87a
@renukamanavalan
In middle of compilation
cc2bfe5
@renukamanavalan
compiled OK
d28399e
@renukamanavalan
common ut passes
f2694ea
@renukamanavalan
events_service unit tests complete
4c8373d
@renukamanavalan
test code in progress
2197e85
@renukamanavalan
evens publish covere by unit test
67fbe19
@renukamanavalan
events 75% covered by UT
9d64634
@renukamanavalan
subscribe UT done
e7f1bca
@renukamanavalan
Merge remote-tracking branch 'origin/syslog_telemetry_shared' into re…
2f13706 
…manava_events
@renukamanavalan
Drop internal readme
94153f5
@renukamanavalan
Update upon self review - mostly on comments
c4db46e
@renukamanavalan
More on comments update
e81160b
@renukamanavalan
minor fix of copy/paste error
9f7b2a8
@renukamanavalan
Comments update
35a6030
@renukamanavalan
more comments
6eb78d7
@renukamanavalan
minor updates; merged with master via shared
3328970
@renukamanavalan
Per review comments
66632f6
@renukamanavalan
Merge pull request #1 from renukamanavalan/remanava_events
c849258 
swss-common LIB update for events publish/subscribe
@renukamanavalan
minor updates
9ba11ed
@renukamanavalan
minor: name change for a typedef
d1a6f83
@renukamanavalan
Added EXIT code
bd28ea8
@renukamanavalan
Minor name change; Enabled py build for events
8f72258
@renukamanavalan renukamanavalan mentioned this pull request Jul 29, 2022
Closed
6 tasks
@kcudnik
Copy link
Contributor

kcudnik commented Jul 29, 2022

please fix build issues

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 2, 2022 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

zbud-msft and others added 18 commits August 2, 2022 15:01
@zbud-msft
Modify azp (#8)
d81441b 
* Add libzmq5 as dependency for ubuntu-20.04 build

* Add libzmq3-dev dependency to build script

Co-authored-by: Ubuntu <zain@zb-dev-vm.022x1jpnpm4u1iy2d325acts3c.yx.internal.cloudapp.net>
@zbud-msft
Modify azp (#9)
6b79a20 
* Add libzmq5 as dependency for ubuntu-20.04 build

* Add libzmq3-dev dependency to build script

* Add uuid-dev and libboost-serialization-dev dependencies

Co-authored-by: Ubuntu <zain@zb-dev-vm.022x1jpnpm4u1iy2d325acts3c.yx.internal.cloudapp.net>
@zbud-msft
Modify azp (#10)
40f0665 
* Add libzmq5 as dependency for ubuntu-20.04 build

* Add libzmq3-dev dependency to build script

* Add uuid-dev and libboost-serialization-dev dependencies

* Add dependencies for bazel build

Co-authored-by: Ubuntu <zain@zb-dev-vm.022x1jpnpm4u1iy2d325acts3c.yx.internal.cloudapp.net>
@zbud-msft
Merge branch 'sonic-net:master' into syslog_telemetry_shared
30f49d9
@renukamanavalan
Merge remote-tracking branch 'origin/syslog_telemetry_shared' into st…
0f58397 
…atistics
@renukamanavalan
compile errors
fe863a7
@renukamanavalan
compile errors
3c8e1f0
@renukamanavalan
Compile error from Ubuntu
36e18f7
@renukamanavalan
drop unused macro
fc65a91
@renukamanavalan
compile fix
0ff6c10
@renukamanavalan
compile fix
4d24e6f
@renukamanavalan
remove pedantic
e3db7b8
@renukamanavalan
restored ABORT_IF_NOT
2dab4e8
@renukamanavalan
revert flag add
7546a87
@renukamanavalan
comments correction
6de2566
@renukamanavalan
Merge pull request #13 from renukamanavalan/statistics
fd18ca8 
Minor updates
@renukamanavalan
Log every published event
7fd1e4a
@renukamanavalan
Merge pull request #14 from renukamanavalan/statistics
7976914 
Log every published event
@zbud-msft zbud-msft closed this Aug 8, 2022
@zbud-msft zbud-msft mentioned this pull request Aug 26, 2022
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zbud-msft zbud-msft Awaiting requested review from zbud-msft

@lguohan lguohan Awaiting requested review from lguohan

@liuh-80 liuh-80 Awaiting requested review from liuh-80

@kcudnik kcudnik Awaiting requested review from kcudnik

@qiluo-msft qiluo-msft Awaiting requested review from qiluo-msft

Assignees

@renukamanavalan renukamanavalan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@renukamanavalan @kcudnik @qiluo-msft @zbud-msft