Skip to content

Commit

Permalink
Merge pull request #27 from onaio/remove-handler-that-havent-been-reg…
Browse files Browse the repository at this point in the history
…istered

Remove calls to handlers that have not been registered
  • Loading branch information
FrankApiyo authored Apr 26, 2024
2 parents 1b1b408 + 801ec5b commit 20a8459
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 11 deletions.
2 changes: 1 addition & 1 deletion meta/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ dependencies:
django_pip_paths: "{{ kpi_pip_paths }}"
django_pip_packages: "{{ kpi_django_pip_packages }}"
django_venv_path: "{{ kpi_venv_path }}"
django_pid_socks_dir: "{{ kpi_pid_socket_dir }}"
django_pid_socks_dir: "{{ kpi_pid_socket_dir }}"
django_pid_file: "{{ kpi_pid_file }}"
django_wsgi_socket: "{{ kpi_socket_file }}"
django_service_group: "{{ kpi_service_group }}"
Expand Down
5 changes: 1 addition & 4 deletions tasks/install-nvm.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,6 @@
with_items: "{{ kpi_npm_requirements }}"
environment:
PATH: "{{ kpi_nvm_node_dir }}/bin:{{ ansible_env.PATH }}"
notify:
- restart_service

- name: Install npm dependencies in package.json # noqa 301
command: "{{ kpi_nvm_npm_path }} install"
Expand All @@ -28,5 +26,4 @@
environment:
PATH: "{{ kpi_nvm_node_dir }}/bin:{{ ansible_env.PATH }}"
HOME: "{{ kpi_system_user_home }}"
notify:
- restart_service

6 changes: 0 additions & 6 deletions tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,6 @@
environment:
PATH: "{{ kpi_nvm_node_dir }}/bin:{{ ansible_env.PATH }}:{{kpi_system_user_home}}/.virtualenvs/kpi/bin"
HOME: "{{ kpi_system_user_home }}"
notify:
- restart_service

- name: Build client code (run build)
become: true
Expand All @@ -47,8 +45,6 @@
environment:
PATH: "{{ kpi_nvm_node_dir }}/bin:{{ ansible_env.PATH }}:{{kpi_system_user_home}}/.virtualenvs/kpi/bin"
HOME: "{{ kpi_system_user_home }}"
notify:
- restart_service

- name: Change ownership of all the KPI files
file:
Expand All @@ -66,8 +62,6 @@
virtualenv: "{{ kpi_venv_path }}"
settings: "{{ kpi_settings_app_name }}"
with_items: "{{ kpi_django_setup_commands }}"
notify:
- restart_service

- name: Ensure all KPI files are owned by the KPI user
file:
Expand Down

0 comments on commit 20a8459

Please sign in to comment.