Commit ef80b9d
authored
[action] [PR:19288] Add missing current_password_hash option in multi_passwd_ssh plugin configuration (sonic-net#483)
<!--
Please make sure you've read and understood our contributing guidelines;
https://github.com/sonic-net/SONiC/blob/gh-pages/CONTRIBUTING.md
Please provide following information to help code review process a bit easier:
-->
### Description of PR
<!--
- Please include a summary of the change and which issue is fixed.
- Please also include relevant motivation and context. Where should reviewer start? background context?
- List any dependencies that are required for this change.
-->
When attempting to run an Ansible module in a newer environment (Ansible [core 2.18.6] with Python 3.12), the following error occurs:
```
yutongzhang@sonic_mgmt:/data/sonic-mgmt/ansible$ ansible -i veos_vtb vlab-01 -m config_facts -a 'source=running host=vlab-01'
vlab-01 | FAILED! => {
"msg": "Requested entry (plugin_type: connection plugin: multi_passwd_ssh setting: current_password_hash ) was not defined in configuration."
}
```
This issue is caused by the multi_passwd_ssh connection plugin referencing an option named current_password_hash, which was not defined in its configuration schema.
This PR resolves the issue by explicitly adding current_password_hash to the plugin’s configuration documentation.
Summary:
Fixes # (issue)
### Type of change
<!--
- Fill x for your type of change.
- e.g.
- [x] Bug fix
-->
- [ ] Bug fix
- [ ] Testbed and Framework(new/improvement)
- [ ] New Test case
- [ ] Skipped for non-supported platforms
- [ ] Test case improvement
### Back port request
- [ ] 202205
- [ ] 202305
- [ ] 202311
- [ ] 202405
- [ ] 202411
- [ ] 202505
### Approach
#### What is the motivation for this PR?
When attempting to run an Ansible module in a newer environment (Ansible [core 2.18.6] with Python 3.12), the following error occurs:
```
yutongzhang@sonic_mgmt:/data/sonic-mgmt/ansible$ ansible -i veos_vtb vlab-01 -m config_facts -a 'source=running host=vlab-01'
vlab-01 | FAILED! => {
"msg": "Requested entry (plugin_type: connection plugin: multi_passwd_ssh setting: current_password_hash ) was not defined in configuration."
}
```
This issue is caused by the multi_passwd_ssh connection plugin referencing an option named current_password_hash, which was not defined in its configuration schema.
This PR resolves the issue by explicitly adding current_password_hash to the plugin’s configuration documentation.
#### How did you do it?
This PR resolves the issue by explicitly adding current_password_hash to the plugin’s configuration documentation.
#### How did you verify/test it?
We need to make sure that this change won't affect current test firstly -- test by pipeline itself. And then, we need to make sure that this change works in the new version -- test locally.
#### Any platform specific information?
#### Supported testbed topology if it's a new test case?
### Documentation
<!--
(If it's a new feature, new test case)
Did you update documentation/Wiki relevant to your implementation?
Link to the wiki page?
-->1 parent 0285776 commit ef80b9d
1 file changed
Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
55 | 55 | | |
56 | 56 | | |
57 | 57 | | |
| 58 | + | |
| 59 | + | |
58 | 60 | | |
59 | 61 | | |
60 | 62 | | |
| |||
0 commit comments