Adding log_delivery parameter to CDN create_service#577
Adding log_delivery parameter to CDN create_service#577vikomall wants to merge 1 commit intopycontribs:masterfrom
Conversation
vikomall
force-pushed
the
cdn_log_directory
branch
from
32b6c34 to
6027890
Compare
pyrax/cloudcdn.py
Outdated
There was a problem hiding this comment.
Since this has to be True/False, this should be log_delivery=False
There was a problem hiding this comment.
Thanks @briancurtin for reviewing. Currently allowed value is {"enabled": "true/false"}, but in future if it supports some other key/values in this dictionary, for example {"enabled": "true", "log_path": "cloud-files-container-path"}, then we have to revisit this change.
So I am thinking current change is OK. Let me know what you think!
There was a problem hiding this comment.
If that happened, we'd just add a log_path=None argument and construct the appropriate body to go along with it.
There was a problem hiding this comment.
If we keep the current implementation, we need not touch it again.
There was a problem hiding this comment.
If we keep the current implementation, we will forever offer a sub-par experience.
vikomall
force-pushed
the
cdn_log_directory
branch
from
6027890 to
4695ed5
Compare
|
@briancurtin updated patch as suggested. |
2 participants
Adding log_delivery parameter to CDN create_service.
Issue #576