[GitHub] ningyougang opened a new pull request #2594: Fix bug on ansible variable db_prefix

[gi...@git.apache.org]

ningyougang opened a new pull request #2594: Fix bug on ansible variable db_prefix
URL: https://github.com/apache/incubator-openwhisk/pull/2594
 
 
   if `ansible node` and `target node` is not the same node, it will have problem.
   ```
   - include: db/recreateDb.yml
     vars:
       dbName: "{{ db.whisk.auth }}"
       forceRecreation: False
   ```
   the `db.whisk.auth` reference varaible : `db_prefix`, the `db_prefix` value is `ansible node`'s  `{{ ansible_user_id|lower }}_{{ ansible_hostname|lower }}_`
   
   ```
   - name: (re)start controller
     docker_container:
       name: controller{{ groups['controllers'].index(inventory_hostname) }}
       env:
          "DB_WHISK_AUTHS": "{{ db.whisk.auth }}"
          ...
          ...
    ```
   But above ansible task's  `db.whisk.auth`'s `db_prefix` value is `controller node`'s  `{{ ansible_user_id|lower }}_{{ ansible_hostname|lower }}_`
   
   So if `ansible node` is not same as `controller node`, the controller's ENV `DB_WHISK_AUTHS` will reference a `not exist auth db address`. 
   So it is better to write the `db_prefix` value to hardcode to keep the default running env well.
   If you don't like the `db_prefix` value, can change it what you like.
 
----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services