Skip to content

Commit 218e2d2

Browse files
FoSixshinmog
authored andcommitted
fix(upgrade_assurance): error out panorama connections
1 parent 40b0d0d commit 218e2d2

File tree

4 files changed

+7
-4
lines changed

4 files changed

+7
-4
lines changed

plugins/modules/panos_active_in_ha.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
- pandevice can be obtained from PyPI U(https://pypi.python.org/pypi/pandevice)
3636
- pan-os-upgrade-assurance can be obtained from PyPI U(https://pypi.org/project/panos-upgrade-assurance)
3737
notes:
38-
- Only Firewalls are supported.
38+
- Panorama is not supported.
3939
- Check mode is not supported.
4040
extends_documentation_fragment:
4141
- paloaltonetworks.panos.fragments.transitional_provider
@@ -113,6 +113,7 @@ def main():
113113
force_fail=dict(type="bool", default=False),
114114
skip_config_sync=dict(type="bool", default=False),
115115
),
116+
panorama_error="This is a firewall only module",
116117
)
117118

118119
module = AnsibleModule(

plugins/modules/panos_readiness_checks.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
- pandevice can be obtained from PyPI U(https://pypi.python.org/pypi/pandevice)
3838
- pan-os-upgrade-assurance can be obtained from PyPI U(https://pypi.org/project/panos-upgrade-assurance)
3939
notes:
40-
- Only Firewalls are supported.
40+
- Panorama is not supported.
4141
- Check mode is not supported.
4242
extends_documentation_fragment:
4343
- paloaltonetworks.panos.fragments.transitional_provider
@@ -165,6 +165,7 @@ def main():
165165
force_fail=dict(type="bool", default=False),
166166
skip_force_locale=dict(type="bool", default=False),
167167
),
168+
panorama_error="This is a firewall only module",
168169
)
169170

170171
module = AnsibleModule(

plugins/modules/panos_snapshot_report.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@
4040
- pandevice can be obtained from PyPI U(https://pypi.python.org/pypi/pandevice)
4141
- pan-os-upgrade-assurance can be obtained from PyPI U(https://pypi.python.org/pypi/pan-os-upgrade-assurance)
4242
notes:
43-
- Only Firewalls are supported.
43+
- This is an offline module, no device connection is made.
4444
- Check mode is not supported.
4545
options:
4646
left_snapshot:

plugins/modules/panos_state_snapshot.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
- pandevice can be obtained from PyPI U(https://pypi.python.org/pypi/pandevice)
3838
- pan-os-upgrade-assurance can be obtained from PyPI U(https://pypi.python.org/pypi/pan-os-upgrade-assurance)
3939
notes:
40-
- Only Firewalls are supported.
40+
- Panorama is not supported.
4141
- Check mode is not supported.
4242
extends_documentation_fragment:
4343
- paloaltonetworks.panos.fragments.transitional_provider
@@ -135,6 +135,7 @@ def main():
135135
argument_spec=dict(
136136
state_areas=dict(type="list", default=["all"], elements="str")
137137
),
138+
panorama_error="This is a firewall only module",
138139
)
139140

140141
module = AnsibleModule(

0 commit comments

Comments
 (0)