Skip to content

Commit

Permalink
Deprecate haproxy modules, moved to pfsensible.haproxy
Browse files Browse the repository at this point in the history
  • Loading branch information
opoplawski committed Jan 16, 2024
1 parent 694d992 commit 9f6d276
Show file tree
Hide file tree
Showing 4 changed files with 20 additions and 0 deletions.
2 changes: 2 additions & 0 deletions changelogs/fragments/deprecate_haproxy.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
deprecated_features:
- The pfsensible_haproxy* modules have moved to the `pfsensible.haproxy` collection and will be removed from `pfsensible.core` in version 0.8.0.
10 changes: 10 additions & 0 deletions meta/runtime.yml
Original file line number Diff line number Diff line change
@@ -1 +1,11 @@
plugin_routing:
modules:
pfsense_haproxy_backend:
deprecation:
removal_version: 0.8.0
warning_text: Use pfsensible.haproxy.pfsense_haproxy_backend instead.
pfsense_haproxy_backend_server:
deprecation:
removal_version: 0.8.0
warning_text: Use pfsensible.haproxy.pfsense_haproxy_backend_server instead.
requires_ansible: ">=2.12"
4 changes: 4 additions & 0 deletions plugins/modules/pfsense_haproxy_backend.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,10 @@
description:
- Manage pfSense HAProxy backends
notes:
deprecated:
removed_in: 0.8.0
why: Moved to `pfsensible.haproxy`
alternative: Use M(pfsensible.haproxy.pfsense_haproxy_backend) instead.
options:
name:
description: The backend name.
Expand Down
4 changes: 4 additions & 0 deletions plugins/modules/pfsense_haproxy_backend_server.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,10 @@
description:
- Manage pfSense haproxy servers
notes:
deprecated:
removed_in: 0.8.0
why: Moved to `pfsensible.haproxy`
alternative: Use M(pfsensible.haproxy.pfsense_haproxy_backend) instead.
options:
backend:
description: The backend name.
Expand Down

2 comments on commit 9f6d276

@genofire
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you miss a lot of files for haproxy:

  • plugins/module_utils/haproxy_backend.py
  • plugins/module_utils/haproxy_backend_server.py
  • tests/unit/plugins/modules/test_pfsense_haproxy_backend.py
  • tests/unit/plugins/modules/test_pfsense_haproxy_backend_server.py
  • tests/unit/plugins/modules/fixtures/pfsense_haproxy_backend_server_config.xml
  • tests/unit/plugins/modules/fixtures/pfsense_haproxy_backend_config.xml

@opoplawski
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They are deprecated for now. Will be removed in 0.8.0.

Please sign in to comment.