Bug 1008673

Summary:	Superfluous nectar.log file is created, but not used.
Product:	[Retired] Pulp	Reporter:	Jason Connor <jconnor>
Component:	user-experience	Assignee:	Jason Connor <jconnor>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Preethi Thomas <pthomas>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	Master	CC:	mmccune, pthomas
Target Milestone:	---
Target Release:	2.3.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-12-09 14:30:43 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:

Description Jason Connor 2013-09-16 19:53:28 UTC

Description of problem:
There is an /var/log/pulp/nectar.log file, but no messages are ever logged into it.

The messages from nectar are always routed into the /var/log/pulp/pulp.log file.


Version-Release number of selected component (if applicable):


How reproducible:
Always

Steps to Reproduce:
1. Install pulp
2. Run any pulp operation that uses nectar (notably: sync)
3. Observer that the nectar.log file remains empty

Actual results:
Empty and misleading log file

Expected results:
No extraneous log file

Additional info:

Comment 1 Jason Connor 2013-09-17 22:30:30 UTC

Discovered messages from the nectar packages did not have their own logger defined (instead, only messages from the requests package was using the nectar log file)

Added config entry that directs messages from the nectar package to the nectar log  file.

Fix in branch jconnor-1008673-unused-nectar-log-file

Comment 2 Jeff Ortel 2013-09-18 23:58:52 UTC

build: 2.3.0-0.14.alpha

Comment 3 Preethi Thomas 2013-09-19 15:11:03 UTC

verified
[root@pulp-v2-server ~]# rpm -qa python-nectar
python-nectar-1.1.1-1.el6.noarch
[root@pulp-v2-server ~]# rpm -qa pulp-server
pulp-server-2.3.0-0.14.alpha.el6.noarch


[root@pulp-v2-server ~]# tail -f /var/log/pulp/nectar.log 
2013-09-19 12:17:54,867 urllib3.connectionpool:INFO: Starting new HTTPS connection (1): cdn.redhat.com
2013-09-19 12:17:55,255 urllib3.connectionpool:INFO: Starting new HTTPS connection (1): cdn.redhat.com
2013-09-19 12:17:55,257 urllib3.connectionpool:INFO: Starting new HTTPS connection (1): cdn.redhat.com
2013-09-19 12:17:55,258 urllib3.connectionpool:INFO: Starting new HTTPS connection (1): cdn.redhat.com
2013-09-19 12:17:55,260 urllib3.connectionpool:INFO: Starting new HTTPS connection (1): cdn.redhat.com
2013-09-19 12:17:55,262 urllib3.connectionpool:INFO: Starting new HTTPS connection (1): cdn.redhat.com

Comment 4 Preethi Thomas 2013-12-09 14:30:43 UTC

Pulp 2.3 released.