rabbah commented on a change in pull request #3955: Add an variable to set the 
timezone for all of components
URL: 
https://github.com/apache/incubator-openwhisk/pull/3955#discussion_r208915212
 
 

 ##########
 File path: ansible/README.md
 ##########
 @@ -359,3 +359,11 @@ limits:
 - The `limits.concurrentInvocations` represents the maximum concurrent 
invocations allowed per namespace.
 - The `limits.firesPerMinute` represents the allowed namespace trigger firings 
per minute.
 - The `limits.sequenceMaxLength` represents the maximum length of a sequence 
action.
+
+#### Set the timezone for containers
+The default timezone is `UTC`, which may differ from your servers, and makes 
it a litter difficult to analyse error logs, the configuration locates in 
[./group_vars/all](./group_vars/all) and may be changed by modifying the 
group_vars or by passing extra var to ansible-playbook(`-e docker_timzone=xxx`) 
for your specific environment.
 
 Review comment:
   Suggested rewrite:
   
   The default timezone for all system containers is `UTC`. The timezone may 
differ from your servers which could make it difficult to inspect logs. The 
timezone is configured globally in [group_vars/all](./group_vars/all) or by 
passing an extra variable `-e docker_timzone=xxx` when you run an 
`ansible-playbook`.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to