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
chore(health): Bazelify health service #12654
Conversation
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In pairing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding a test!
2250587
to
9f9503c
Compare
Signed-off-by: Krisztián Varga <krisztian.varga@tngtech.com>
Signed-off-by: Krisztián Varga <krisztian.varga@tngtech.com>
Signed-off-by: Krisztián Varga <krisztian.varga@tngtech.com>
9f9503c
to
76e983d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - test refactoring discussed offline
* test(health): Add a unit test for the health service Signed-off-by: Krisztián Varga <krisztian.varga@tngtech.com> * chore(health): Bazelify health service Signed-off-by: Krisztián Varga <krisztian.varga@tngtech.com> * chore(health): Bazelify health service tests Signed-off-by: Krisztián Varga <krisztian.varga@tngtech.com>
Summary
This PR adds a new unit test for the health service, and bazelifies both the service and the new test.
Test Plan
Execute the service with Bazel on Devcontainer, Bazel-base and the VM:
bazel run lte/gateway/python/magma/health:health
Test the service with Bazel on Devcontainer, Bazel-base and the VM:
bazel test lte/gateway/python/magma/health/tests/...
Additional Information