search

openbmc / openbmc-test-automation

Apache License 2.0
98 stars 92 forks source link

[BUG]Verify SSH Login Access With Admin User #2232

Open justin0309 opened 7 months ago

justin0309 commented 7 months ago

This commit modification will cause test fail. Without these two lines of code, the test will pass. https://github.com/openbmc/openbmc-test-automation/commit/e8015b3e14fdfd0303af57b4aa9fa99837d14421

Redfish.Login Redfish.Delete /redfish/v1/AccountService/Accounts/new_admin

It seems like two lines should be added, but I don’t know why it failed.

image

gkeishin commented 7 months ago

The test intends to delete the newly created new_admin password after the SSH validation is done. Can you check if the user has been created?

justin0309 commented 7 months ago

The test intends to delete the newly created new_admin password after the SSH validation is done. Can you check if the user has been created?

image

prkatti1 commented 7 months ago

@justin0309 Pls open a defect against the build you are using, its not a test issue.

justin0309 commented 1 month ago

@prkatti1 openbmc code base: 2.14.0 auto-test : v4.0-stable The error message is still the same Modify the test based on the latest commit

Removed faillock file check to fix problem https://github.com/openbmc/openbmc-test-automation/commit/638a598989125c8fd96d8599813c5aec8451b85b image

gkeishin commented 1 month ago

@prkatti1 @swe12345 @Anvesh-Kumar-Rayankula Please check and respond

prkatti1 commented 1 month ago

Its not a test code issue, Pls open a defect against the build on which you have noticed this issue.

justin0309 commented 1 month ago

@prkatti1 what should I do? I don’t know where it should be classified? bmcweb?

prkatti1 commented 1 month ago

yes, it is allowing to create user but not allowing to delete user

justin0309 commented 1 month ago

@prkatti1 I ran the Verify SSH Login Access With Admin User test item again. The user is being used and cannot be deleted. Does this make it possible to suspect that the robot test case process needs to be corrected or that there may be other problems?

Jun 06 08:36:47  phosphor-user-manager[521]: User 'new_admin' deleted successfully
Jun 06 08:36:48  phosphor-user-manager[521]: User 'new_admin' created successfully
Jun 06 08:36:56  phosphor-user-manager[7432]: userdel: user new_admin is currently used by process 8687
Jun 06 08:36:56  phosphor-user-manager[521]: Command /usr/sbin/userdel execution failed, return code 8
Jun 06 08:36:56  phosphor-user-manager[521]: The operation failed internally.
Jun 06 08:36:56  phosphor-user-manager[521]: Delete User 'new_admin' failed
Jun 06 08:36:56  phosphor-user-manager[521]: The operation failed internally.

It is occupied by this process, but I don’t know who is using it. 8697 new_admi 3292 S -sh https://github.com/openbmc/openbmc-test-au SSHLibrary.Close All Connections should be added here. This way the test will pass Verify User Account Unlock Have the same problem

phosphor-user-manager[521]: User 'test_user' deleted successfully
Jun 06 09:21:25  phosphor-user-manager[521]: User 'test_user' created successfully
Jun 06 09:22:01  phosphor-user-manager[11685]: userdel: user test_user is currently used by process 11674
Jun 06 09:22:01  phosphor-user-manager[521]: Command /usr/sbin/userdel execution failed, return code 8
Jun 06 09:22:01  phosphor-user-manager[521]: The operation failed internally.
Jun 06 09:22:01  phosphor-user-manager[521]: Delete User 'test_user' failed
Jun 06 09:22:01  phosphor-user-manager[521]: The operation failed internally.

I think SSHLibrary . Close All Connections should be adjusted before Redfish . Delete /redfish/v1/AccountService/Accounts/test_user image

prkatti1 commented 1 month ago

Sure, you can push the change.