-
Notifications
You must be signed in to change notification settings - Fork 9
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
Updates in the logs #95
Conversation
a46343e
to
2ea5f1a
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.
Overall, it looks good. Adding some comments
@thakurajayL, you need to sign-off your commit(s) to successfully pass the |
37ad06e
to
fb9c009
Compare
@thakurajayL, you need to sign-off you last commit (the DCO filed for it) |
Signed-off-by: Ajay Lotan Thakur <[email protected]>
Signed-off-by: Ajay Lotan Thakur <[email protected]>
Signed-off-by: Ajay Lotan Thakur <[email protected]>
Co-authored-by: gab-arrobo <[email protected]> Signed-off-by: Ajay Lotan Thakur <[email protected]>
Co-authored-by: gab-arrobo <[email protected]> Signed-off-by: Ajay Lotan Thakur <[email protected]>
Co-authored-by: gab-arrobo <[email protected]> Signed-off-by: Ajay Lotan Thakur <[email protected]>
Co-authored-by: gab-arrobo <[email protected]> Signed-off-by: Ajay Lotan Thakur <[email protected]>
Co-authored-by: gab-arrobo <[email protected]> Signed-off-by: Ajay Lotan Thakur <[email protected]>
Signed-off-by: Ajay Lotan Thakur <[email protected]>
* add MongoDB functions to handle transactions Signed-off-by: Patricia Reinoso <[email protected]> * rename to supports transactions Signed-off-by: Patricia Reinoso <[email protected]> --------- Signed-off-by: Patricia Reinoso <[email protected]> Signed-off-by: Ajay Lotan Thakur <[email protected]>
Co-authored-by: gab-arrobo <[email protected]> Signed-off-by: Ajay Lotan Thakur <[email protected]>
1a31c65
to
38222ff
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.
+1 (tested changes with OnRamp using the replace
directive in the amf
pod/image)
Logging updates and code review.