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

Restructure (split gRPC and server code, better naming, Makefile) #31

Merged
merged 5 commits into from
Sep 18, 2022

Conversation

sskaur
Copy link
Contributor

@sskaur sskaur commented Aug 2, 2022

These commits reorganize existing code but do not attempt to add any functionality.

@idryomov

Copy link

@PepperJo PepperJo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Sandy for taking the time to split the PR.
I only have one minor item otherwise LGTM.

@sskaur
Copy link
Contributor Author

sskaur commented Aug 15, 2022

I changed PersistentConfig to GatewayState and updated other things (partially from discussion in #14). I believe it is ready for another look. @PepperJo @idryomov @trociny

Copy link

@PepperJo PepperJo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changes Sandy. Just some minor things otherwise LGTM.

Copy link

@PepperJo PepperJo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@trociny trociny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@idryomov idryomov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise LGTM!

This commit reorganizes and renames files. It also adds
a Makefile to simplify protobuf compilation and use of
other basic commands like starting the server.

Signed-off-by: Sandy Kaur <[email protected]>
This commit splits gRPC API away from server code
that handles SPDK startup and monitoring.

Signed-off-by: Sandy Kaur <[email protected]>
Signed-off-by: Sandy Kaur <[email protected]>
Signed-off-by: Sandy Kaur <[email protected]>
@idryomov idryomov changed the title Restructure Restructure (split gRPC and server code, better naming, Makefile) Sep 18, 2022
@idryomov idryomov merged commit 5c1d529 into ceph:devel Sep 18, 2022
@sskaur sskaur mentioned this pull request Sep 19, 2022
@epuertat epuertat added this to the Milestone 2 milestone Nov 18, 2022
@sskaur sskaur deleted the restructure branch March 28, 2023 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

6 participants