Project

Profile

Help

Issue #430

The pulp-profile-update yum client plugin does not clearly report errors to users

Added by rbarlow over 6 years ago. Updated over 2 years ago.

Status:
CLOSED - WONTFIX
Priority:
Low
Assignee:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
1. Low
Version:
Master
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

I noticed that running yum install <package> does not trigger a consumer profile update. This causes applicability data to be incorrect in the Pulp server. Restarting goferd is a workaround to getting the profile reported.

+ This bug was cloned from Bugzilla Bug #1097906 +

History

#1 Updated by rbarlow over 6 years ago

I am using yum-3.4.3-146.fc21.noarch, if that matters.

+ This comment was cloned from Bugzilla #1097906 comment 1 +

#2 Updated by rbarlow over 6 years ago

It turns out that our plugin is printing this error through the yum conduit:

Peer certificate commonName does not match host, expected localhost.localdomain, got guava.rdu.redhat.com

It would be nice if this was more clearly an error in reporting the profile.

+ This comment was cloned from Bugzilla #1097906 comment 2 +

#3 Updated by rbarlow over 6 years ago

Turns out that this was user error. I hadn't edited the server line in the /etc/pulp/.../consumer.conf file, but I had in the /root/.pulp/consumer.conf file. The yum plugin only reads the one from /etc/.../. However, I do think we have an issue here in reporting problems to users. Note the printout in the middle here about the peer certificate commonName not matching:

rbarlow@guava ~/d/p/pulp_rpm> (pulp_rpm) sudo yum install grinder_test_package-4.0
Loaded plugins: langpacks, pulp-profile-update
Resolving Dependencies
--> Running transaction check
---> Package grinder_test_package.noarch 0:3.0-1.fc14 will be updated
---> Package grinder_test_package.noarch 0:4.0-1.fc14 will be an update
--> Finished Dependency Resolution

Dependencies Resolved

====================================================================================================================
Package Arch Version Repository Size
====================================================================================================================
Updating:
grinder_test_package noarch 4.0-1.fc14 test-errata-install 1.6 k

Transaction Summary
====================================================================================================================
Upgrade 1 Package

Total download size: 1.6 k
Is this ok [y/d/N]: y
Downloading packages:
grinder_test_package-4.0-1.fc14.noarch.rpm | 1.6 kB 00:00:00
Running transaction check
Running transaction test
Transaction test succeeded
Running transaction
Updating : grinder_test_package-4.0-1.fc14.noarch 1/2
Cleanup : grinder_test_package-3.0-1.fc14.noarch 2/2
Peer certificate commonName does not match host, expected localhost.localdomain, got guava.rdu.redhat.com
Peer certificate commonName does not match host, expected localhost.localdomain, got guava.rdu.redhat.com
Verifying : grinder_test_package-4.0-1.fc14.noarch 1/2
Verifying : grinder_test_package-3.0-1.fc14.noarch 2/2

Updated:
grinder_test_package.noarch 0:4.0-1.fc14

Complete!

I think it would be best if we could make that error message more obviously tell the user that the plugin failed to report the profile to Pulp. I didn't even notice it was an error at all when I first ran it. It would have been better if it said something like "Error in reporting profile to Pulp: <error_message_here>".

I'm lowering the severity since it turns out that nothing is really broken.

+ This comment was cloned from Bugzilla #1097906 comment 3 +

#4 Updated by bmbouter over 6 years ago

  • Severity changed from Low to 1. Low

#5 Updated by bmbouter over 2 years ago

  • Status changed from NEW to CLOSED - WONTFIX

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

#6 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF