[GitHub] rabbah commented on issue #3353: Fix the TLS issue with the CLI download

2018-04-02 Thread GitBox
rabbah commented on issue #3353: Fix the TLS issue with the CLI download  
URL: 
https://github.com/apache/incubator-openwhisk/pull/3353#issuecomment-377965035
 
 
   It could still happen for outdated python versions and replacing with curl 
is an alternative. That said we hadn't any new complaints about this issue.


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


[GitHub] rabbah commented on issue #3353: Fix the TLS issue with the CLI download

2018-03-02 Thread GitBox
rabbah commented on issue #3353: Fix the TLS issue with the CLI download  
URL: 
https://github.com/apache/incubator-openwhisk/pull/3353#issuecomment-370088307
 
 
   @ddragosd my apologies for not merging this sooner.
   it looks like the new change to resolve conflicts is
   
   ```
git diff
   diff --git a/ansible/roles/cli/tasks/deploy.yml 
b/ansible/roles/cli/tasks/deploy.yml
   index 8591263..76df94a 100644
   --- a/ansible/roles/cli/tasks/deploy.yml
   +++ b/ansible/roles/cli/tasks/deploy.yml
   @@ -26,10 +26,13 @@
#  straightforward.
#
- name: "Download release archive to build directory ..."
   -  get_url:
   -url: "{{ openwhisk_cli.remote.location }}/{{ 
openwhisk_cli.archive_name}}-{{ openwhisk_cli_tag }}-all.tgz"
   -dest: "{{ openwhisk_build_dir }}/{{ openwhisk_cli.archive_name }}.tgz"
   -headers: "{{ openwhisk_cli.remote.headers | default('') }}"
   +  local_action: >
   +  shell curl -L "{{ openwhisk_cli.remote.location }}/{{ 
openwhisk_cli.archive_name}}-{{ openwhisk_cli_tag }}-all.tgz" \
   +-o "{{ openwhisk_build_dir }}/{{ openwhisk_cli.archive_name }}.tgz"
   +#  get_url:
   +#url: "{{ openwhisk_cli.remote.location }}/{{ 
openwhisk_cli.archive_name}}-{{ openwhisk_cli_tag }}-all.tgz"
   +#dest: "{{ openwhisk_build_dir }}/{{ openwhisk_cli.archive_name }}.tgz"
   +#headers: "{{ openwhisk_cli.remote.headers | default('') }}"
  when: openwhisk_cli.installation_mode == "remote"
   ```
   
   shall i do it or want to do it?


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


[GitHub] rabbah commented on issue #3353: Fix the TLS issue with the CLI download

2018-02-26 Thread GitBox
rabbah commented on issue #3353: Fix the TLS issue with the CLI download  
URL: 
https://github.com/apache/incubator-openwhisk/pull/3353#issuecomment-368733349
 
 
   Thanks @ddragosd this fixes the symptom that I was also observing.


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