BMC username,password change through horizon UI is not working, through cli it works fine

[Enzo Candotti]

Brief Description

BMC change through horizon UI is not working, through cli it works fine

Severity

major

Steps to Reproduce

1)goto horizon https://<fip>:8443->host inventory->click on any host "edit host"

2)click on Board management, change the existing username, password and save

3)Now login to cli and verify the change , the change is not reflected

[sysadmin@controller-1 ~(keystone_admin)]$ system host-show controller-0
+------------------------+-------------------------------------------------------------------------+
| Property | Value |
+------------------------+-------------------------------------------------------------------------+
| action | none |
| administrative | unlocked |
| apparmor | disabled |
| availability | available |
| bm_ip | 128.224.64.231 |
| bm_type | ipmi |
| bm_username | sysadmin |
| boot_device | /dev/disk/by-path/pci-0000:01:00.0-scsi-0:2:0:0 |
| capabilities | {'is_max_cpu_configurable': 'configurable', 'stor_function': 'monitor', |
| | 'Personality': 'Controller-Standby'} |
| clock_synchronization | ntp |
| config_applied | 00783e0e-4391-4f3b-a3bc-d10f2dbf2589 |
| config_status | None |
| config_target | 00783e0e-4391-4f3b-a3bc-d10f2dbf2589 |
| console | ttyS0,115200n8 |
| created_at | 2022-11-29T10:28:54.283062+00:00 |
| device_image_update | None |
| hostname | controller-0 |
| id | 1 |
| install_output | text |
| install_state | None |
| install_state_info | None |
| inv_state | inventoried |
| invprovision | provisioned |
| location | {} |
| max_cpu_mhz_allowed | 3200 |
| max_cpu_mhz_configured | None |
| mgmt_ip | 192.168.204.2 |
| mgmt_mac | 3c:fd:fe:19:ee:00 |
| operational | enabled |
| personality | controller |
| reboot_needed | False |
| reserved | False |
| rootfs_device | /dev/disk/by-path/pci-0000:01:00.0-scsi-0:2:0:0 |
| serialid | None |
| software_load | 22.12 |
| subfunction_avail | available |
| subfunction_oper | enabled |
| subfunctions | controller,worker |
| task | |
| tboot | |
| ttys_dcd | False |
| updated_at | 2022-11-30T19:50:13.862871+00:00 |
| uptime | 117787 |
| uuid | 61cc4d9a-3f1c-4220-a939-41abe5877966 |
| vim_progress_status | services-enabled |
+------------------------+-------------------------------------------------------------------------+
[sysadmin@controller-1 ~(keystone_admin)]$

4)try changing it from cli, the change is reflected in horizon and on the system as well

[sysadmin@controller-1 ~(keystone_admin)]$ system host-update controller-0 bm_username=root bm_password=Li69nux*
+------------------------+-------------------------------------------------------------------------+
| Property | Value |
+------------------------+-------------------------------------------------------------------------+
| action | none |
| administrative | unlocked |
| apparmor | disabled |
| availability | available |
| bm_ip | 128.224.64.231 |
| bm_type | ipmi |
| bm_username | root |
| boot_device | /dev/disk/by-path/pci-0000:01:00.0-scsi-0:2:0:0 |
| capabilities | {'is_max_cpu_configurable': 'configurable', 'stor_function': 'monitor'} |
| clock_synchronization | ntp |
| config_applied | 00783e0e-4391-4f3b-a3bc-d10f2dbf2589 |
| config_status | None |
| config_target | 00783e0e-4391-4f3b-a3bc-d10f2dbf2589 |
| console | ttyS0,115200n8 |
| created_at | 2022-11-29T10:28:54.283062+00:00 |
| device_image_update | None |
| hostname | controller-0 |
| id | 1 |
| install_output | text |
| install_state | None |
| install_state_info | None |
| inv_state | inventoried |
| invprovision | provisioned |
| location | {} |
| max_cpu_mhz_allowed | 3200 |
| max_cpu_mhz_configured | None |
| mgmt_ip | 192.168.204.2 |
| mgmt_mac | 3c:fd:fe:19:ee:00 |
| operational | enabled |
| personality | controller |
| reboot_needed | False |
| reserved | False |
| rootfs_device | /dev/disk/by-path/pci-0000:01:00.0-scsi-0:2:0:0 |
| serialid | None |
| software_load | 22.12 |
| subfunction_avail | available |
| subfunction_oper | enabled |
| subfunctions | controller,worker |
| task | |
| tboot | |
| ttys_dcd | False |
| updated_at | 2022-11-30T19:50:13.862871+00:00 |
| uptime | 117787 |
| uuid | 61cc4d9a-3f1c-4220-a939-41abe5877966 |
| vim_progress_status | services-enabled |
+------------------------+-------------------------------------------------------------------------+
[sysadmin@controller-1 ~(keystone_admin)]$

show command

[sysadmin@controller-1 ~(keystone_admin)]$ system host-show controller-0
+------------------------+-------------------------------------------------------------------------+
| Property | Value |
+------------------------+-------------------------------------------------------------------------+
| action | none |
| administrative | unlocked |
| apparmor | disabled |
| availability | available |
| bm_ip | 128.224.64.231 |
| bm_type | ipmi |
| bm_username | root |
| boot_device | /dev/disk/by-path/pci-0000:01:00.0-scsi-0:2:0:0 |
| capabilities | {'is_max_cpu_configurable': 'configurable', 'stor_function': 'monitor', |
| | 'Personality': 'Controller-Standby'} |
| clock_synchronization | ntp |
| config_applied | 00783e0e-4391-4f3b-a3bc-d10f2dbf2589 |
| config_status | None |
| config_target | 00783e0e-4391-4f3b-a3bc-d10f2dbf2589 |
| console | ttyS0,115200n8 |
| created_at | 2022-11-29T10:28:54.283062+00:00 |
| device_image_update | None |
| hostname | controller-0 |
| id | 1 |
| install_output | text |
| install_state | None |
| install_state_info | None |
| inv_state | inventoried |
| invprovision | provisioned |
| location | {} |
| max_cpu_mhz_allowed | 3200 |
| max_cpu_mhz_configured | None |
| mgmt_ip | 192.168.204.2 |
| mgmt_mac | 3c:fd:fe:19:ee:00 |
| operational | enabled |
| personality | controller |
| reboot_needed | False |
| reserved | False |
| rootfs_device | /dev/disk/by-path/pci-0000:01:00.0-scsi-0:2:0:0 |
| serialid | None |
| software_load | 22.12 |
| subfunction_avail | available |
| subfunction_oper | enabled |
| subfunctions | controller,worker |
| task | |
| tboot | |
| ttys_dcd | False |
| updated_at | 2022-11-30T19:54:50.529082+00:00 |
| uptime | 117787 |
| uuid | 61cc4d9a-3f1c-4220-a939-41abe5877966 |
| vim_progress_status | services-enabled |
+------------------------+-------------------------------------------------------------------------+
[sysadmin@controller-1 ~(keystone_admin)]$

Expected Behavior

bm username/password should be changed from UI as well, currently on through ci it is working

Actual Behavior

bm username/password change not working from UI

Reproducibility

100%

System Configuration
--------------------
All systems

Last Pass
---------
Unknown

Timestamp/Logs
---------
N/A

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/gui/+/868611

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/starlingx/gui/+/868611
Committed: https://opendev.org/starlingx/gui/commit/2cf939310efc5eca280cfa5c80fbc79b98ed2dd2
Submitter: "Zuul (22348)"
Branch: master

commit 2cf939310efc5eca280cfa5c80fbc79b98ed2dd2
Author: Enzo Candotti <email address hidden>
Date: Mon Dec 26 16:38:51 2022 -0300

    Fix 'Edit Host' modal

    When trying to edit a host, the modal is not able to save the changes
    unless the max_cpu_mhz_configured parameter is modified since it's
    set as required.

    This change fixes that problem by setting the max_cpu_mhz_configured as
    not required and removing the forced assignment during clean()
    function.

    Test Plan:
    PASS: Change BMC type, IP, User and Password. Verify changes
    are listed in system show command.
    PASS: Modify host location, CPU frequency, Clock Synchronization.
    Verify changes are listed in system show command.

    Closes-Bug: 2000484

    Signed-off-by: Enzo Candotti <email address hidden>
    Change-Id: Icb6d05fe170d9202e7cc01ede7154c8d2bb98c72