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

[DO NOT MERGE] generate vald gRPC code for Rust #2289

Closed
wants to merge 1 commit into from

Conversation

kmrmt
Copy link
Contributor

@kmrmt kmrmt commented Jan 10, 2024

Description:

  • add Rust code generated from proto files
    • The code is generated with buf
  • separate client code and server code
    • enable to merge the code
  • unnecessary files are also generated
    • e.g. discoverer, manager, etc.

Related Issue:

Versions:

  • Go Version: 1.21.5

  • Docker Version: 20.10.8

  • Kubernetes Version: v1.28.4

  • NGT Version: 2.1.6

  • Rust Version: 1.75.0

Checklist:

Special notes for your reviewer:

@vdaas-ci
Copy link
Collaborator

[CHATOPS:HELP] ChatOps commands.

  • 🙆‍♀️ /approve - approve
  • 🍱 /format - format codes and add licenses
  • /gen-test - generate test codes
  • 🏷️ /label - add labels
  • 🔚 2️⃣ 🔚 /label actions/e2e-deploy - run E2E deploy & integration test

Signed-off-by: Kosuke Morimoto <ksk@vdaas.org>
@kmrmt kmrmt force-pushed the feature/protocol-buffers/generate-rust-client branch from f72d151 to 17b70e3 Compare January 10, 2024 06:07
Copy link

cloudflare-pages bot commented Jan 10, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 17b70e3
Status: ✅  Deploy successful!
Preview URL: https://8f5f0a39.vald.pages.dev
Branch Preview URL: https://feature-protocol-buffers-gen.vald.pages.dev

View logs

@kmrmt kmrmt changed the title [WIP] generate vald gRPC code for Rust [DO NOT MERGE] generate vald gRPC code for Rust Jan 10, 2024
@kmrmt kmrmt closed this Jan 12, 2024
@kpango kpango deleted the feature/protocol-buffers/generate-rust-client branch May 8, 2024 08:50
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.

2 participants