647522 – repo delete does not unsubscribe consumers from that repo

Bug 647522 - repo delete does not unsubscribe consumers from that repo

Summary: repo delete does not unsubscribe consumers from that repo

Keywords:
Status:	CLOSED CURRENTRELEASE
Alias:	None
Product:	Pulp
Classification:	Retired
Component:	z_other
Sub Component:
Version:	unspecified
Hardware:	Unspecified
OS:	Unspecified
Priority:	high
Severity:	high
Target Milestone:	---
Target Release:	---
Assignee:	Pradeep Kilambi
QA Contact:	Preethi Thomas
Docs Contact:
URL:
Whiteboard:
Depends On:
Blocks:	pulp-verified
TreeView+	depends on / blocked

Reported:	2010-10-28 16:42 UTC by Pradeep Kilambi
Modified:	2011-08-16 14:03 UTC (History)
CC List:	1 user (show)
Fixed In Version:
Doc Type:	Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:	2011-08-16 14:03:18 UTC
Embargoed:

Attachments	(Terms of Use)

Description Pradeep Kilambi 2010-10-28 16:42:27 UTC

Description of problem:

When a pulp repo bound to consumers is deleted, the consumers are not unbound from it.

Consider this scenario:
* subscribe a consumer to repo A
* delete repo A
* perform actions such as errata list or errata install etc gets a traceback

File \"/home/pkilambi/code/pulp/src/pulp/server/api/repo.py\", line 95, in _get_existing_repo\n    raise PulpException(\"No Repo with id: %s found\" % id)\nPulpException: u'No Repo with id: test-repo found'\n"

Comment 2 Jay Dobies 2010-10-29 16:58:35 UTC

Fixed in 0.77.

Comment 3 Jay Dobies 2010-11-03 19:35:40 UTC

Fixed in build 0.78.

Comment 4 Preethi Thomas 2010-11-08 14:47:06 UTC

fails_qa

This is the error i get on my remote client with the use case from above.


[root@10 ~]# pulp-client -u admin -p admin errata list
error: operation failed: "Traceback (most recent call last):\n  File \"/usr/lib/python2.6/site-packages/pulp/server/webservices/controllers/base.py\", line 53, in report_error\n    return method(self, *args, **kwargs)\n  File \"/usr/lib/python2.6/site-packages/pulp/server/webservices/role_check.py\", line 105, in check_roles\n    result = f(instance, *fargs, **kw)\n  File \"/usr/lib/python2.6/site-packages/pulp/server/webservices/controllers/consumers.py\", line 380, in listerrata\n    return self.ok(consumer_api.listerrata(id, data['types']))\n  File \"/usr/lib/python2.6/site-packages/pulp/server/api/consumer.py\", line 430, in listerrata\n    return self._applicable_errata(consumer, types).keys()\n  File \"/usr/lib/python2.6/site-packages/pulp/server/api/consumer.py\", line 452, in _applicable_errata\n    for eid in self.repoapi.errata(repoid, types) ]\n  File \"/usr/lib/python2.6/site-packages/pulp/server/api/repo.py\", line 598, in errata\n    repo = self._get_existing_repo(id)\n  File \"/usr/lib/python2.6/site-packages/pulp/server/api/repo.py\", line 94, in _get_existing_repo\n    raise PulpException(\"No Repo with id: %s found\" % id)\nPulpException: u'No Repo with id: f13 found'\n"

Comment 5 Preethi Thomas 2010-11-08 19:12:37 UTC

I think I made mistake on testing this one. Will retest one more time.

Comment 6 Preethi Thomas 2010-11-10 15:04:28 UTC

verified
[root@preethi ~]# pulp-client -u admin -p redhat consumer bind --repoid=f13-updates
Successfully subscribed consumer [preethi.usersys.redhat.com] to repo [f13-updates]


[root@preethi ~]#  pulp-admin -u admin -p redhat errata list --consumerid=preethi.usersys.redhat.com
[u'FEDORA-2010-13978', u'FEDORA-2010-13549', u'FEDORA-2010-14252', u'FEDORA-2010-14890', u'FEDORA-2010-14258', u'FEDORA-2010-13631', u'FEDORA-2010-13430', u'FEDORA-2010-13435', u'FEDORA-2010-15953', u'FEDORA-2010-16286', u'FEDORA-2010-16122', u'FEDORA-2010-16284', u'FEDORA-2010-16288', u'FEDORA-2010-16655', u'FEDORA-2010-16711', u'FEDORA-2010-14624', u'FEDORA-2010-16713', u'FEDORA-2010-15353', u'FEDORA-2010-14621', u'FEDORA-2010-15507', u'FEDORA-2010-13968', u'FEDORA-2010-14244', u'FEDORA-2010-14245', u'FEDORA-2010-12523', u'FEDORA-2010-13405', u'FEDORA-2010-13404', u'FEDORA-2010-15945', u'FEDORA-2010-16336', u'FEDORA-2010-16332', u'FEDORA-2010-16859', u'FEDORA-2010-16646', u'FEDORA-2010-16645', u'FEDORA-2010-16237', u'FEDORA-2010-16883', u'FEDORA-2010-13656', u'FEDORA-2010-13158', u'FEDORA-2010-15375', u'FEDORA-2010-13084', u'FEDORA-2010-16382', u'FEDORA-2010-15201', u'FEDORA-2010-11032', u'FEDORA-2010-13152', u'FEDORA-2010-13559', u'FEDORA-2010-14682', u'FEDORA-2010-11530', u'FEDORA-2010-16478', u'FEDORA-2010-13520', u'FEDORA-2010-16722', u'FEDORA-2010-15120', u'FEDORA-2010-15121', u'FEDORA-2010-14235', u'FEDORA-2010-14347', u'FEDORA-2010-13145', u'FEDORA-2010-14651', u'FEDORA-2010-14078', u'FEDORA-2010-14107', u'FEDORA-2010-15917', u'FEDORA-2010-15763', u'FEDORA-2010-15764', u'FEDORA-2010-15911', u'FEDORA-2010-15957', u'FEDORA-2010-13540', u'FEDORA-2010-13708', u'FEDORA-2010-13547', u'FEDORA-2010-13544', u'FEDORA-2010-13791', u'FEDORA-2010-14778', u'FEDORA-2010-14779', u'FEDORA-2010-14672', u'FEDORA-2010-14770', u'FEDORA-2010-16095', u'FEDORA-2010-14355', u'FEDORA-2010-16243', u'FEDORA-2010-14350', u'FEDORA-2010-14787', u'FEDORA-2010-15710', u'FEDORA-2010-16238', u'FEDORA-2010-16009', u'FEDORA-2010-15998', u'FEDORA-2010-15994', u'FEDORA-2010-15993', u'FEDORA-2010-13389', u'FEDORA-2010-16480', u'FEDORA-2010-13383', u'FEDORA-2010-14701', u'FEDORA-2010-16612', u'FEDORA-2010-16571', u'FEDORA-2010-15079', u'FEDORA-2010-16273', u'FEDORA-2010-15070', u'FEDORA-2010-14415', u'FEDORA-2010-15705', u'FEDORA-2010-15978', u'FEDORA-2010-14627', u'FEDORA-2010-15977', u'FEDORA-2010-13693', u'FEDORA-2010-14625', u'FEDORA-2010-16377', u'FEDORA-2010-14656', u'FEDORA-2010-14654', u'FEDORA-2010-14718', u'FEDORA-2010-15988', u'FEDORA-2010-15542', u'FEDORA-2010-15544', u'FEDORA-2010-15166', u'FEDORA-2010-14088', u'FEDORA-2010-15165', u'FEDORA-2010-13895', u'FEDORA-2010-16266', u'FEDORA-2010-17112', u'FEDORA-2010-13364', u'FEDORA-2010-17059', u'FEDORA-2010-14401', u'FEDORA-2010-15960', u'FEDORA-2010-14409', u'FEDORA-2010-15081', u'FEDORA-2010-16589', u'FEDORA-2010-16588', u'FEDORA-2010-16107', u'FEDORA-2010-15086', u'FEDORA-2010-15008', u'FEDORA-2010-16468', u'FEDORA-2010-14229', u'FEDORA-2010-14640', u'FEDORA-2010-15246', u'FEDORA-2010-13818', u'FEDORA-2010-15242', u'FEDORA-2010-16820', u'FEDORA-2010-16629', u'FEDORA-2010-15526', u'FEDORA-2010-13058', u'FEDORA-2010-13423', u'FEDORA-2010-13426', u'FEDORA-2010-13375', u'FEDORA-2010-16110', u'FEDORA-2010-16294', u'FEDORA-2010-16594', u'FEDORA-2010-16595', u'FEDORA-2010-16593', u'FEDORA-2010-15529', u'FEDORA-2010-15012', u'FEDORA-2010-14636', u'FEDORA-2010-15520']

[root@preethi ~]#  pulp-admin -u admin -p redhat repo delete --id=f13-updatesSuccessful deleted repository [ f13-updates ]

[root@preethi ~]#  pulp-admin -u admin -p redhat errata list --consumerid=preethi.usersys.redhat.com
No errata available to list

[root@preethi ~]#

Comment 7 Preethi Thomas 2011-08-16 14:03:18 UTC

Closing with Community Release 15

pulp-0.0.223-4.

Note You need to log in before you can comment on or make changes to this bug.