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

feat: reorg proto structure #158

Merged
merged 8 commits into from
Aug 30, 2024
Merged

feat: reorg proto structure #158

merged 8 commits into from
Aug 30, 2024

Conversation

mfiedorowicz
Copy link
Member

@mfiedorowicz mfiedorowicz commented Aug 29, 2024

  • upgrade buf to v2 (details)
  • reorganize proto definitions for ingester and reconciler services under diode.v1 package
  • use ingester go code from netboxlabs/diode-sdk-go (same existing diodepb go package) removing same code being generated into go sdk and diode-server internally
  • reconciler.proto generated into diode-server/reconciler/v1/reconcilerpb (as-is)

keep diode related protobuf definitions (ingester and reconciler) under diode-proto directory and diode.v1 package

Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
Copy link

github-actions bot commented Aug 29, 2024

Go test coverage

STATUS ELAPSED PACKAGE COVER PASS FAIL SKIP
🟢 PASS 1.24s github.com/netboxlabs/diode/diode-server/ingester 4.9% 9 0 0
🟢 PASS 0.01s github.com/netboxlabs/diode/diode-server/netbox 0.0% 0 0 0
🟢 PASS 1.56s github.com/netboxlabs/diode/diode-server/netboxdiodeplugin 6.8% 40 0 0
🟢 PASS 1.40s github.com/netboxlabs/diode/diode-server/reconciler 8.2% 27 0 0
🟢 PASS 1.34s github.com/netboxlabs/diode/diode-server/reconciler/changeset 78.6% 78 0 0
🟢 PASS 1.02s github.com/netboxlabs/diode/diode-server/server 85.7% 14 0 0
🟢 PASS 1.01s github.com/netboxlabs/diode/diode-server/version 100.0% 1 0 0

Total coverage: 80.3%

diode-server/Makefile Outdated Show resolved Hide resolved
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz mfiedorowicz changed the title [POC] feat: reorg proto structure feat: reorg proto structure Aug 30, 2024
@mfiedorowicz mfiedorowicz marked this pull request as ready for review August 30, 2024 15:20
@mfiedorowicz mfiedorowicz requested a review from natm as a code owner August 30, 2024 15:20
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@github-actions github-actions bot added documentation Improvements or additions to documentation markdown labels Aug 30, 2024
@mfiedorowicz mfiedorowicz merged commit bae91c5 into develop Aug 30, 2024
7 checks passed
@mfiedorowicz mfiedorowicz deleted the reorg-proto-structure branch August 30, 2024 16:21
@orb-ci
Copy link

orb-ci commented Sep 5, 2024

🎉 This PR is included in version 0.5.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@orb-ci orb-ci added the released label Sep 5, 2024
@orb-ci
Copy link

orb-ci commented Sep 5, 2024

🎉 This PR is included in version 0.5.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants