fix(connect): clear RHSM spinner before the success line - #559
fix(connect): clear RHSM spinner before the success line#559Sanika10114 wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Hi, thanks for the PR! The fix itself looks correct — stopping the spinner before printing the success line resolves the malformed output.
However, the commit's author and Signed-off-by are set to Super User <root@localhost.localdomain>, which is not a real identity. Please amend the commit with your actual name and email (git commit --amend --author="Your Name <your@email>" --signoff) before this can be merged.
fd8f421 to
b726664
Compare
|
Hello @pkoprda, Thanks for the review. |
|
/packit build |
pkoprda
left a comment
There was a problem hiding this comment.
I found that the rhc connect with incorrect credentials is still broken. Can you also fix that?
# rhc connect --disable-feature analytics --disable-feature remote-management
Notice: ignoring preferences set via 'rhc configure features'.
Connecting fedora-44 to Red Hat. Enabled features: content.
This might take some time.
Username: foo
Password:
[/] Connecting to Red Hat Subscription Management... [𐄂] Cannot connect to Red Hat Subscription Management
[𐄂] Skipping generation of Red Hat repository file
[●] Analytics ... Skipped
[●] Remote Management ... Skipped
Manage your connected systems: https://red.ht/connector
The following errors were encountered during connect:
STEP ERROR
rhsm cannot connect to Red Hat Subscription Management: registering with RHSM: Invalid username or password. To create a login, please visit https://www.redhat.com/wapps/ugc/register.html
Please see /var/log/rhc/rhc.log for full details.
Also your commit needs to be GPG-signed, because I wasn't able to merge this MR even when I approved it. See the commits in the main branch: https://github.com/RedHatInsights/rhc/commits/main/, the newest commits have the label Verified, your commit doesn't have that. It is a requirement for all the commits under Red Hat Insights organization.
See the documentation on how to sign commits: https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits.
The connect spinner was still on the line when the checkmark was printed. Resolves: RHEL-244670 Signed-off-by: Sanika Parihar <sparihar@redhat.com>
b726664 to
ad8c28f
Compare
|
Hello @pkoprda Stopped the RHSM spinner on the failure path as well (verified with bad credentials). Also GPG-signed the commit. Many Thanks |
|
/packit build |
The connect spinner was still on the line when the checkmark was printed.
Resolves: RHEL-244670