Repository: ansible/ansible
Pull request: Unify ios_logging documentation with accepted options
##### SUMMARY
<!--- Describe the change below, including rationale and design decisions -->

<!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue -->
Fixes #61547

The documentation of the `name` parameter mentions options not supported by the module. This PR updates the description to remove them.

##### ISSUE TYPE
<!--- Pick one below and delete the rest -->
- Docs Pull Request

##### COMPONENT NAME
<!--- Write the short name of the module, plugin, task or feature below -->
ios_logging

Problem statement:
_net_logging and ios_logging dest: file
<!--- Verify first that your improvement is not already reported on GitHub -->
<!--- Also test if the latest release and devel branch are affected too -->
<!--- Complete *all* sections as described, this form is processed automatically -->

##### SUMMARY
<!--- Explain the problem briefly below, add suggestions to wording or structure -->
In net_logging documentation examples, 'file' is used as 'dest:' but not mentioned in parameters choices
In ios_logging, the parameter 'name' says "If value of dest is file it indicates file-name" but again 'file' is not stated as a valid 'dest' choice.
I can update the doc for net_logging as it seems file is a valid choice because it is used in examples but for ios_logging, I'm not sure if it's a typo or if 'file' should be valid too.
<!--- HINT: Did you know the documentation has an "Edit on GitHub" link on every page ? -->

##### ISSUE TYPE
- Documentation Report

##### COMPONENT NAME
<!--- Write the short name of the rst file, module, plugin, task or feature below, use your best guess if unsure -->
net_logging, ios_logging

##### ADDITIONAL INFORMATION
<!--- Describe how this improves the documentation, e.g. before/after situation or screenshots -->

<!--- HINT: You can paste gist.github.com links for larger files -->

Hints from issue discussion:
Files identified in the description:
* [`lib/ansible/modules/network/ios/ios_logging.py`](https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/network/ios/ios_logging.py)
* [`lib/ansible/modules/network/system/_net_logging.py`](https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/network/system/_net_logging.py)

If these files are inaccurate, please update the `component name` section of the description or use the `!component` bot command.

[click here for bot help](https://github.com/ansible/ansibullbot/blob/master/ISSUE_HELP.md)
<!--- boilerplate: components_banner --->
@Yannis100: Greetings! Thanks for taking the time to open this issue. In order for the community to handle your issue effectively, we need a bit more information.

Here are the items we could not find in your description:
- ansible version

Please set the description of this issue with this template:
https://raw.githubusercontent.com/ansible/ansible/devel/.github/ISSUE_TEMPLATE.md

[click here for bot help](https://github.com/ansible/ansibullbot/blob/master/ISSUE_HELP.md)
<!--- boilerplate: issue_missing_data --->
cc @NilashishC @Qalthos @danielmellado @ganeshrn @justjais @trishnaguha
[click here for bot help](https://github.com/ansible/ansibullbot/blob/master/ISSUE_HELP.md)
<!--- boilerplate: notify --->
@Yannis100, just so you are aware we have a dedicated Working Group for network.
You can find other people interested in this in `#ansible-network` on Freenode IRC
For more information about communities, meetings and agendas see https://github.com/ansible/community

[click here for bot help](https://github.com/ansible/ansibullbot/blob/master/ISSUE_HELP.md)
<!--- boilerplate: community_workgroups --->
Seems like a documentation mismatch.

Commit message: Unify ios_logging documentation with accepted options

Patch to review:
diff --git a/lib/ansible/modules/network/ios/ios_logging.py b/lib/ansible/modules/network/ios/ios_logging.py
index 902c7bf2bdd0e1..efc1592dc76d2a 100644
--- a/lib/ansible/modules/network/ios/ios_logging.py
+++ b/lib/ansible/modules/network/ios/ios_logging.py
@@ -41,9 +41,8 @@
     choices: ['on', 'host', 'console', 'monitor', 'buffered', 'trap']
   name:
     description:
-      - If value of C(dest) is I(file) it indicates file-name,
-        for I(user) it indicates username and for I(host) indicates
-        the host name to be notified.
+      - If value of C(dest) is I(host) C(name) should be specified,
+        which indicates hostname or IP address.
   size:
     description:
       - Size of buffer. The acceptable value is in range from 4096 to


Code review comments:
diff_hunk: @@ -41,9 +41,8 @@
     choices: ['on', 'host', 'console', 'monitor', 'buffered', 'trap']
   name:
     description:
-      - If value of C(dest) is I(file) it indicates file-name,
-        for I(user) it indicates username and for I(host) indicates
-        the host name to be notified.
+      - If value of C(dest) is I(host) C(name) should be specified,
+        which indicates hostname or IP address.
path: lib/ansible/modules/network/ios/ios_logging.py
text: @user1:
```suggestion
```

diff_hunk: @@ -41,9 +41,8 @@
     choices: ['on', 'host', 'console', 'monitor', 'buffered', 'trap']
   name:
     description:
-      - If value of C(dest) is I(file) it indicates file-name,
-        for I(user) it indicates username and for I(host) indicates
-        the host name to be notified.
+      - If value of C(dest) is I(host) C(name) should be specified,
path: lib/ansible/modules/network/ios/ios_logging.py
text: @user1:
```suggestion
      - The hostname or IP address of the destination.
      - Required when I(dest=host).
```