Bug 704194

Summary:

repo events should contain a 'path' record

Product:

[Retired] Pulp

Reporter:

Daniel Mach <dmach>

Component:

z_other

Assignee:

Jeff Ortel <jortel>

Status:

CLOSED CURRENTRELEASE

QA Contact:

Preethi Thomas <pthomas>

Severity:

high

Docs Contact:

Priority:

unspecified

Version:

unspecified

CC:

skarmark

Target Milestone:

---

Keywords:

Triaged

Target Release:

Sprint 24

Hardware:

Unspecified

OS:

Unspecified

Whiteboard:

Fixed In Version:

Doc Type:

Bug Fix

Doc Text:

Story Points:

---

Clone Of:

Environment:

Last Closed:

2012-02-24 20:12:14 UTC

Type:

---

Regression:

---

Mount Type:

---

Documentation:

---

CRM:

Verified Versions:

Category:

---

oVirt Team:

---

RHEL 7.3 requirements from Atomic Host:

Cloudforms Team:

---

Target Upstream Version:

Bug Depends On:

Bug Blocks:

563609

Attachments:

Description	Flags
Event listener	none

Description Daniel Mach 2011-05-12 12:02:15 UTC

For example repo.created event has no information about repo path.
That means I can't sync the repo elsewhere.

updated.content works as expected

Comment 1 Jeff Ortel 2011-06-10 14:45:57 UTC

You want the path part of the download URL, right?

Comment 2 Jeff Ortel 2011-06-10 14:47:10 UTC

Rather, the sync URL not the download URL

Comment 3 Jeff Ortel 2011-06-10 15:05:26 UTC

Path added.

commit: 41ffda11e6739f8034789c4a36c09e27a3b799e0

Comment 4 Daniel Mach 2011-06-10 16:32:41 UTC

(In reply to comment #2)
> Rather, the sync URL not the download URL

No, I mean the path on local file system.
I want it to rsync a repo elsewhere.

Comment 5 Jeff Ortel 2011-06-13 18:46:56 UTC

The event 'path' field now contains the absolute (file) path to the repo storage location in pulp.

commit: 84611ae9020cff3e6cf4872c32ea2b366f7ff55f

Comment 6 Jeff Ortel 2011-06-13 19:48:12 UTC

build: 0.190

Comment 7 Jeff Ortel 2011-09-02 18:50:41 UTC

Created attachment 521274 [details]
Event listener

Quick & dirty event listener.

Comment 8 Jeff Ortel 2011-09-02 18:56:27 UTC

Repo created events broken.  Fixed: 0c41f94dc8e99961b06836402cee7e72ea5f34da

How to verify:

1) edit /etc/pulp/pulp.conf and enable sending events.

[events]
send_enabled: true
recv_enabled: false

2) restart httpd
3) start the attached event listener.
4) create a repo
5) note in the event listener output the 'path' element in the event.
6) done

Comment 9 Jeff Ortel 2011-09-02 18:57:09 UTC

(In reply to comment #8)
> Repo created events broken.  Fixed: 0c41f94dc8e99961b06836402cee7e72ea5f34da
> 
> How to verify:
> 
> 1) edit /etc/pulp/pulp.conf and enable sending events.
> 
> [events]
> send_enabled: true
> recv_enabled: false
> 
> 2) restart httpd
> 3) start the attached event listener.

do: 'python listener.py' in a terminal window.

> 4) create a repo
> 5) note in the event listener output the 'path' element in the event.
> 6) done

Comment 10 Jeff Ortel 2011-09-02 19:18:11 UTC

build: 0.230

Comment 11 Preethi Thomas 2011-09-07 17:51:47 UTC

verified
[root@preethi ~]# rpm -q pulp
pulp-0.0.230-1.fc15.noarch

[root@preethi ~]# python listener.py 
subject: repo.created
event:
{'path': '/var/lib/pulp//repos/test2', 'name': 'test2', 'id': 'test2'}

Comment 12 Preethi Thomas 2012-02-24 20:12:14 UTC

Pulp v1.0 is released
Closed Current Release.