-
Notifications
You must be signed in to change notification settings - Fork 319
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
add {set,get,delete}_object_lock_config APIs #985
add {set,get,delete}_object_lock_config APIs #985
Conversation
d90957d
to
df33308
Compare
df33308
to
15b30f7
Compare
do we need to add functional tests for these APIs? |
Many existing APIs do not have functional tests. Not sure whether tests are needed for these APIs. |
57b0faf
to
c84527c
Compare
tests/unit/objectlockconfig.py
Outdated
<Years>3</Years> | ||
</DefaultRetention> | ||
</Rule> | ||
</ObjectLockConfiguration>""", |
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.
please fix the indentation here
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.
What do you mean by fixing indentation? Could you point me where exactly the problem?
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.
if you view the file xml is not indented ok
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.
The multi-lined string is indented correctly to python. Please print the string and let me know if I am missing it.
ccf35b9
to
012bece
Compare
012bece
to
e3cf139
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
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
No description provided.