Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
justin0309 opened this issue Dec 1, 2023 · 10 comments
Open

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

justin0309 opened this issue Dec 1, 2023 · 10 comments
Assignees

Comments

@justin0309
Copy link

justin0309 commented Dec 1, 2023

This commit modification will cause test fail. Without these two lines of code, the test will pass.
e8015b3

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
Copy link
Member

gkeishin commented Dec 7, 2023

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
Copy link
Author

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
Copy link
Contributor

prkatti1 commented Dec 8, 2023

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

@justin0309
Copy link
Author

justin0309 commented May 31, 2024

@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
638a598
image

@gkeishin
Copy link
Member

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

@prkatti1
Copy link
Contributor

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

@justin0309
Copy link
Author

justin0309 commented May 31, 2024

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

@prkatti1
Copy link
Contributor

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

@justin0309
Copy link
Author

justin0309 commented Jun 7, 2024

@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
Copy link
Contributor

prkatti1 commented Jun 7, 2024

Sure, you can push the change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants