Merge lp:~suligap/canonical-identity-provider/charm-drop-logfile-migration-tasks into lp:~ubuntuone-pqm-team/canonical-identity-provider/charm

[Przemysław Suliga]

Drop no longer needed temporary tasks for logfiles migration

Not needed anymore as per comments in
https://code.launchpad.net/~roadmr/canonical-identity-provider/charm-fix-talisker-logfile-migration/+merge/384659

[Simon Davy (bloodearnest)]

review: Approve

[Otto Co-Pilot (otto-copilot)]

Project setup failed
https://jenkins.ols.canonical.com/online-services/job/canonical-identity-provider_charm/10/

=== modified file 'playbook.yaml'
--- playbook.yaml	2020-05-25 15:57:19 +0000
+++ playbook.yaml	2020-05-28 12:03:30 +0000
@@ -12,7 +12,6 @@
     python: "{{ venv }}/bin/python"
     wheel_dir: "{{ current_dir }}/branches/wheels"
     logs_dir: "{{ basedir }}/logs"
-    sso_log: "{{ logs_dir }}/sso.log"
     oops_dir: "{{ basedir }}/logs/www-oops"
     conf_dir: "{{ basedir }}/etc"
     run_dir: "{{ basedir }}/run"
@@ -144,31 +143,6 @@
         group: "{{ user }}"
         state: "{% if readonly %}touch{% else %}absent{% endif %}"
 
-    # temporary task to handle migration to talisker logging
-    - name: test for linked logfile
-      tags:
-          - config-changed
-      stat: path={{ sso_log }}
-      register: log_file
-      when: talisker.stat.exists
-
-    # temporary task to handle migration to talisker logging
-    - name: mv old log file
-      tags:
-          - config-changed
-      command: mv {{ sso_log }} {{ sso_log }}.bak
-      when: talisker.stat.exists and not log_file.stat.islnk
-
-    - name: link log file
-      tags:
-          - config-changed
-      file:
-        path: "{{ logs_dir }}/sso.log"
-        src: /var/log/upstart/gunicorn.log
-        state: link
-      # temporary when until migrated to talisker
-      when: talisker.stat.exists
-
     # the next 2 tasks should really only run when the payload has *changed*
     - name: Install required sso packages
       tags: