Skip to content

Commit

Permalink
[PR #6401/486c47f9 backport][stable-5] rhsm_release: removing require…
Browse files Browse the repository at this point in the history
…d=true for 'release' option (#6408)

rhsm_release: removing required=true for 'release' option (#6401)

Fix rhsm_release by removing required=true for 'release' option.

(cherry picked from commit 486c47f)

Co-authored-by: Felix Fontein <felix@fontein.de>
  • Loading branch information
patchback[bot] and felixfontein authored Apr 23, 2023
1 parent 5dc60cb commit af4ac48
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
2 changes: 2 additions & 0 deletions changelogs/fragments/6401-rhsm_release-required.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
bugfixes:
- "rhsm_release - make ``release`` parameter not required so it is possible to pass ``null`` as a value. This only was possible in the past due to a bug in ansible-core that now has been fixed (https://github.com/ansible-collections/community.general/pull/6401)."
12 changes: 6 additions & 6 deletions plugins/modules/packaging/os/rhsm_release.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@
options:
release:
description:
- RHSM release version to use (use null to unset)
required: true
- RHSM release version to use.
- To unset either pass C(null) for this option, or omit this option.
type: str
author:
- Sean Myers (@seandst)
Expand All @@ -34,17 +34,17 @@
# Set release version to 7.1
- name: Set RHSM release version
community.general.rhsm_release:
release: "7.1"
release: "7.1"
# Set release version to 6Server
- name: Set RHSM release version
community.general.rhsm_release:
release: "6Server"
release: "6Server"
# Unset release version
- name: Unset RHSM release release
community.general.rhsm_release:
release: null
release: null
'''

RETURN = '''
Expand Down Expand Up @@ -97,7 +97,7 @@ def set_release(module, release):
def main():
module = AnsibleModule(
argument_spec=dict(
release=dict(type='str', required=True),
release=dict(type='str'),
),
supports_check_mode=True
)
Expand Down

0 comments on commit af4ac48

Please sign in to comment.