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(subscriberdb): Handle NW NGC config nil check correctly #13081
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
|
1adf1f5
to
f060859
Compare
f060859
to
7189f05
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.
Looking good, I think this will fix the issue, I just wonder if we can make it a bit more stable.
In the Python client on the gateway, we currently fetch the suci profiles first and skip all other sync calls if the suci call fails. See client.py. Would it be better to do the suci call last? At the moment, the response doesn't seem to be used in the client.
7189f05
to
492c05a
Compare
b59952e
to
8afa51f
Compare
Signed-off-by: Moinuddin Khan <moinuddin.khan@wavelabs.ai>
8afa51f
to
d331f8e
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
…3081) Signed-off-by: Moinuddin Khan <moinuddin.khan@wavelabs.ai>
Signed-off-by: Moinuddin Khan moinuddin.khan@wavelabs.ai
feat(subscriberdb): Handle NW NGC config nil check correctly
Summary
Done the following as part of the PR:
Test Plan
Tested in local VM using docker based Orc8r and AGW.
Additional Information
#13036