Bug 1781166
| Summary: | hammer ping command is asking for user id and password after upgrade(Use any existing or non-existing credentials) | |||
|---|---|---|---|---|
| Product: | Red Hat Satellite | Reporter: | Devendra Singh <desingh> | |
| Component: | Hammer | Assignee: | Rahul Bajaj <rabajaj> | |
| Status: | CLOSED ERRATA | QA Contact: | Gaurav Talreja <gtalreja> | |
| Severity: | high | Docs Contact: | ||
| Priority: | unspecified | |||
| Version: | 6.7.0 | CC: | apatel, egolov, kgaikwad, kkinge, mmccune, mshira, ofedoren, rabajaj, rankumar | |
| Target Milestone: | 6.9.0 | Keywords: | Reopened, Triaged | |
| Target Release: | Unused | |||
| Hardware: | x86_64 | |||
| OS: | Linux | |||
| Whiteboard: | ||||
| Fixed In Version: | tfm-rubygem-hammer_cli_foreman-2.3.0 | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | ||
| Clone Of: | ||||
| : | 1908855 (view as bug list) | Environment: | ||
| Last Closed: | 2021-04-21 14:10:49 UTC | Type: | Bug | |
| Regression: | --- | Mount Type: | --- | |
| Documentation: | --- | CRM: | ||
| Verified Versions: | Category: | --- | ||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
| Cloudforms Team: | --- | Target Upstream Version: | ||
| Embargoed: | ||||
(In reply to Devendra Singh from comment #0) > Description of problem: hammer ping command is asking for user id and > password after upgrade(Use any existing or non-existing credentials). > > Version-Release number of selected component (if applicable): > 6.7 > > How reproducible: > always > > Steps to Reproduce: > 1. Perform Upgrade from 6.0 to 6.7 > 2. Upgrade completed successfully > 3. Execute all upgrade related test cases and all of them were executed. > 4. After that, I log in on the system and tried to run "hammer ping" > command, and it asks for foreman credentials. > > Actual results: > Hammer ping command asks for credentials and it works with non-existing > credentials. > > # hammer ping > [Foreman] Username: skd;skd'sfejfdflfmdcxcrlf;gd > [Foreman] Password for skd;skd'sfejfdflfmdcxcrlf;gd: > database: > Status: ok > Server Response: Duration: 0ms > candlepin: > Status: ok > Server Response: Duration: 31ms > candlepin_auth: > Status: ok > Server Response: Duration: 28ms > pulp: > Status: ok > Server Response: Duration: 124ms > pulp_auth: > Status: ok > Server Response: Duration: 58ms > foreman_tasks: > Status: ok > Server Response: Duration: 8ms > > > > Expected results: > > Hammer ping command should not ask for credentials and If it asks then use > valid credentials that exist in the system. > > Additional info: Correcting reproduce step 1 1- Perform Upgrade from 6.6 to 6.7 Hello, I have tested this issue. Looks like after upgrade there is no issue with the `hammer ping` command. Although, after running a few QE tests, there is change in the foreman.yml file and default username and password are missing from that file which leads to further asking of username and password. Furthermore, I believe there is something wrong with the QE tests and that I dont see any code changes required for this bug. Therefore, I am closing this issue with a `NOTABUG`. Feel free to reopen the issue in case my finding are not appropriate :) Thank you, Moving this bug to POST for triage into Satellite since the upstream issue https://projects.theforeman.org/issues/31140 has been resolved. Hi, I tested this with the following steps, On Build Version: Satellite 6.9.0 Snap1 1. Tried `hammer ping` with Default/Initial admin password. 2. Changed admin password from UI to see if `hammer ping` works without being dependent on password specified in `.hammer/cli.modules.d/foreman.yml` And is it sufficient to test it on the newly installed satellite 6.9.0 instead of the upgraded satellite? Could you please confirm the above steps are sufficient for verification or we are missing something? Thank you, Gaurav Hello, Gaurav Talreja, you are working in the correct direction. To answer your question yes, it is okay to test on Sat 6.9.0. I am thinking maybe be we dont clear sessions when changing the password. Let me know your test results, if there is a bug, feel free to assign it to me. Thanks, Verified. Tested on Satellite 6.9.0 Snap 9 Version: rubygem-foreman_maintain-0.7.4-1.el7sat.noarch Steps: 1. `hammer ping` with Default/Initial admin password. 2. Changed admin password from WebUI to see if `hammer ping` works. 3. Changed password in `.hammer/cli.modules.d/foreman.yml` to see if `hammer ping` works. Observation: In all these steps followed, `hammer ping` works without the correct admin password and without being dependent on the password specified in `.hammer/cli.modules.d/foreman.yml`. I apologize for providing an incorrect version, Correcting it - Tested on version: tfm-rubygem-hammer_cli_foreman-2.3.0-1.el7sat.noarch Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory (Satellite 6.9 Satellite Tools Release), and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2021:1314 |
Description of problem: hammer ping command is asking for user id and password after upgrade(Use any existing or non-existing credentials). Version-Release number of selected component (if applicable): 6.7 How reproducible: always Steps to Reproduce: 1. Perform Upgrade from 6.0 to 6.7 2. Upgrade completed successfully 3. Execute all upgrade related test cases and all of them were executed. 4. After that, I log in on the system and tried to run "hammer ping" command, and it asks for foreman credentials. Actual results: Hammer ping command asks for credentials and it works with non-existing credentials. # hammer ping [Foreman] Username: skd;skd'sfejfdflfmdcxcrlf;gd [Foreman] Password for skd;skd'sfejfdflfmdcxcrlf;gd: database: Status: ok Server Response: Duration: 0ms candlepin: Status: ok Server Response: Duration: 31ms candlepin_auth: Status: ok Server Response: Duration: 28ms pulp: Status: ok Server Response: Duration: 124ms pulp_auth: Status: ok Server Response: Duration: 58ms foreman_tasks: Status: ok Server Response: Duration: 8ms Expected results: Hammer ping command should not ask for credentials and If it asks then use valid credentials that exist in the system. Additional info: