Skip to content

Commit bc47f38

Browse files
authored
Remove deprecated parameters (#202)
1 parent c777595 commit bc47f38

File tree

5 files changed

+8
-77
lines changed

5 files changed

+8
-77
lines changed

CHANGELOG.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
11
# Changelog
22

3+
## 0.5.0 (Unreleased)
4+
5+
BREAKING CHANGES:
6+
7+
Remove parameters deprecated in release `0.4.0`. To recap, these are `nginx_config_main_upload_*`, `nginx_config_upload_html_*`, and `nginx_config_stream_upload_*`. Use `nginx_config_upload` instead.
8+
39
## 0.4.2 (October 28, 2021)
410

511
BUG FIXES:
@@ -25,7 +31,7 @@ Efforts have been made to thoroughly test all these changes and make sure they w
2531

2632
DEPRECATION WARNINGS:
2733

28-
The `nginx_config_main_upload_*`, `nginx_config_upload_html_*`, and `nginx_config_stream_upload_*` parameters have been deprecated in favor of a newly introduced parameter, `nginx_config_upload_*` (previously `nginx_config_snippet_upload_*`). The new parameter provides greater flexibility in configuring your upload settings in addition to simplifying the upload Ansible tasks. The deprecated parameters will be removed in the next major release (0.5.0), due December 2021.
34+
The `nginx_config_main_upload_*`, `nginx_config_upload_html_*`, and `nginx_config_stream_upload_*` parameters have been deprecated in favor of a newly introduced parameter, `nginx_config_upload` (previously `nginx_config_snippet_upload_*`). The new parameter provides greater flexibility in configuring your upload settings in addition to simplifying the upload Ansible tasks. The deprecated parameters will be removed in the next major release (0.5.0), due December 2021.
2935

3036
BREAKING CHANGES:
3137

defaults/main/upload.yml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -2,18 +2,6 @@
22
# Enable uploading NGINX related files to your system.
33
# Default for uploading files is false.
44
# Default location of files is the files folder within the NGINX Config Ansible role.
5-
# (DEPRECATED) Upload the main NGINX configuration file.
6-
nginx_config_main_upload_enable: false
7-
nginx_config_main_upload_src: config/nginx.conf
8-
nginx_config_main_upload_dest: /etc/nginx/
9-
# (DEPRECATED) Upload HTTP NGINX configuration files.
10-
nginx_config_http_upload_enable: false
11-
nginx_config_http_upload_src: config/http/*.conf
12-
nginx_config_http_upload_dest: /etc/nginx/conf.d/
13-
# (DEPRECATED) Upload Stream NGINX configuration files.
14-
nginx_config_stream_upload_enable: false
15-
nginx_config_stream_upload_src: config/stream/*.conf
16-
nginx_config_stream_upload_dest: /etc/nginx/conf.d/
175
# Upload NGINX config files/snippets.
186
nginx_config_upload_enable: false
197
nginx_config_upload:

molecule/stable_push/converge.yml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,3 @@
2626
nginx_config_upload_ssl_key:
2727
- src: ../common/files/ssl/molecule.key
2828
dest: /etc/ssl/private
29-
30-
# DEPRECATED
31-
nginx_config_main_upload_enable: true
32-
nginx_config_main_upload_src: ../common/files/nginx.conf
33-
nginx_config_http_upload_enable: true
34-
nginx_config_http_upload_src: ../common/files/http/*.conf
35-
nginx_config_http_upload_dest: /etc/nginx/conf.d/deprecated_http

tasks/config/upload-config.yml

Lines changed: 0 additions & 53 deletions
Original file line numberDiff line numberDiff line change
@@ -1,57 +1,4 @@
11
---
2-
- name: (DEPRECATED) Upload NGINX main config
3-
block:
4-
- name: (DEPRECATED) Ensure NGINX main directory exists
5-
file:
6-
path: "{{ nginx_config_main_upload_dest | default('/etc/nginx/') }}"
7-
state: directory
8-
mode: 0755
9-
10-
- name: (DEPRECATED) Upload NGINX main config file
11-
copy:
12-
src: "{{ nginx_config_main_upload_src | default('conf/nginx.conf') }}"
13-
dest: "{{ nginx_config_main_upload_dest | default('/etc/nginx') }}"
14-
backup: true
15-
mode: 0644
16-
notify: (Handler - NGINX Config) Run NGINX
17-
when: nginx_config_main_upload_enable | bool
18-
19-
- name: (DEPRECATED) Upload NGINX HTTP config
20-
block:
21-
- name: (DEPRECATED) Ensure NGINX HTTP directory exists
22-
file:
23-
path: "{{ nginx_config_http_upload_dest | default('/etc/nginx/conf.d/') }}"
24-
state: directory
25-
mode: 0755
26-
27-
- name: (DEPRECATED) Upload NGINX HTTP config files
28-
copy:
29-
src: "{{ item }}"
30-
dest: "{{ nginx_config_http_upload_dest | default('/etc/nginx/conf.d/') }}"
31-
backup: true
32-
mode: 0644
33-
with_fileglob: "{{ nginx_config_http_upload_src }}"
34-
notify: (Handler - NGINX Config) Run NGINX
35-
when: nginx_config_http_upload_enable | bool
36-
37-
- name: (DEPRECATED) Upload NGINX stream config
38-
block:
39-
- name: (DEPRECATED) Ensure NGINX stream directory exists
40-
file:
41-
path: "{{ nginx_config_stream_upload_dest | default('/etc/nginx/conf.d/') }}"
42-
state: directory
43-
mode: 0755
44-
45-
- name: (DEPRECATED) Upload NGINX stream config files
46-
copy:
47-
src: "{{ item }}"
48-
dest: "{{ nginx_config_stream_upload_dest | default('/etc/nginx/conf.d/') }}"
49-
backup: true
50-
mode: 0644
51-
with_fileglob: "{{ nginx_config_stream_upload_src }}"
52-
notify: (Handler - NGINX Config) Run NGINX
53-
when: nginx_config_stream_upload_enable | bool
54-
552
- name: Upload NGINX config snippets
563
block:
574
- name: Ensure NGINX config snippet directories exist

tasks/main.yml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,7 @@
1515

1616
- name: Upload NGINX config
1717
include_tasks: "{{ role_path }}/tasks/config/upload-config.yml"
18-
when: nginx_config_main_upload_enable | bool
19-
or nginx_config_http_upload_enable | bool
20-
or nginx_config_stream_upload_enable | bool
21-
or nginx_config_upload_enable | bool
18+
when: nginx_config_upload_enable | bool
2219
or nginx_config_upload_html_enable | bool
2320
or nginx_config_upload_ssl_enable | bool
2421
tags: nginx_config_upload

0 commit comments

Comments
 (0)