[incubator-openwhisk] branch master updated: Fix outdated `limit section` in ansible doc (#3445)

[ra...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

rabbah pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk.git


The following commit(s) were added to refs/heads/master by this push:
     new f6b855d  Fix outdated `limit section` in ansible doc (#3445)
f6b855d is described below

commit f6b855dc05903d8220da09f0a7e47232c14035f8
Author: Tzu-Chiao Yeh <su...@gmail.com>
AuthorDate: Sat Mar 17 23:21:42 2018 +0800

    Fix outdated `limit section` in ansible doc (#3445)
    
    Update the outdated default value of limits. Also, remove the unused
    section of environment/mac/group_vars/all, the detailed description in each feild is now pointed to the default group_vars.
---
 ansible/README.md | 23 ++++++++---------------
 1 file changed, 8 insertions(+), 15 deletions(-)

diff --git a/ansible/README.md b/ansible/README.md
index a31a062..7152772 100644
--- a/ansible/README.md
+++ b/ansible/README.md
@@ -305,24 +305,17 @@ the path to your OpenWhisk `ansible` directory contains spaces. To fix this, ple
 without spaces as there is no current fix available to this problem.
 
 #### Changing limits
-The default system throttling limits are configured in this file [./group_vars/all](./group_vars/all).
+The default system throttling limits are configured in this file [./group_vars/all](./group_vars/all) and may be changed by modifying the group_vars for your specific environment.
 ```
 limits:
-  invocationsPerMinute: "{{ limit_invocations_per_minute | default(120) }}"
-  concurrentInvocations: "{{ limit_invocations_concurrent | default(100) }}"
+  invocationsPerMinute: "{{ limit_invocations_per_minute | default(60) }}"
+  concurrentInvocations: "{{ limit_invocations_concurrent | default(30) }}"
   concurrentInvocationsSystem:  "{{ limit_invocations_concurrent_system | default(5000) }}"
   firesPerMinute: "{{ limit_fires_per_minute | default(60) }}"
   sequenceMaxLength: "{{ limit_sequence_max_length | default(50) }}"
 ```
-These values may be changed by modifying the `group_vars` for your environment. For example,
-mac users will find the limits in this file [./environments/mac/group_vars/all](./environments/mac/group_vars/all):
-```
-limit_invocations_per_minute: 60
-limit_invocations_concurrent: 30
-limit_invocations_concurrent_system: 5000
-limit_fires_per_minute: 60
-```
-- The `limit_invocations_per_minute` represents the allowed namespace action invocations per minute.
-- The `limit_invocations_concurrent` represents the maximum concurrent invocations allowed per namespace.
-- The `limit_invocations_concurrent_system` represents the maximum concurrent invocations the system will allow across all namespaces.
-- The `limit_fires_per_minute` represents the allowed namespace trigger firings per minute.
+- The `limits.invocationsPerMinute` represents the allowed namespace action invocations per minute.
+- The `limits.concurrentInvocations` represents the maximum concurrent invocations allowed per namespace.
+- The `limits.concurrentInvocationsSystem` represents the maximum concurrent invocations the system will allow across all namespaces.
+- The `limits.firesPerMinute` represents the allowed namespace trigger firings per minute.
+- The `limits.sequenceMaxLength` represents the maximum length of a sequence action.

-- 
To stop receiving notification emails like this one, please contact
rabbah@apache.org.