Bug 921268 - virt-only pools are not showing with listall
Summary: virt-only pools are not showing with listall
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Candlepin
Classification: Community
Component: candlepin
Version: 0.9
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: ---
Assignee: William Poteat
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks: rhsm-rhel70 rhsm-2013
TreeView+ depends on / blocked
 
Reported: 2013-03-13 19:54 UTC by J.C. Molet
Modified: 2015-05-14 14:52 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-04-15 18:30:59 UTC
Embargoed:


Attachments (Terms of Use)

Description J.C. Molet 2013-03-13 19:54:46 UTC
Description of problem:
'virt_only = true' pools are somehow being filtered out when a user runs the listall command 

Version-Release number of selected component (if applicable):
#git show
commit 31c9da7824fcf733bd33432af2d326e12a028317
Author: Devan Goodwin <dgoodwin>
Date:   Wed Mar 13 09:50:58 2013 -0300


How reproducible:
always?

Steps to Reproduce:
1. get latest candlepin from master (or see version above)
2. deploy candlepin using TESTDATA=1 or this kind of voodoo:
export TESTDATA=1 && export FORCECERT=1 && export GENDB=1 && export HOSTNAME=$(hostname) && export IMPORTDIR= && cd /root/candlepin && buildconf/scripts/deploy && echo DEPLOYED
3. as the admin:admin user or any user with sufficient rights, register against this candlepin and then run listall either via 'subscription-manager list --available --all' or a call to:
 "/owners/<OWNER>/pools?consumer=<CONSUMER-ID>&listall=true"


Actual results:
Every pool listed is of System Type: Physical 

Expected results:
There would be some of System Type: Virtual, as provided by the ./buildconf/scripts/test_data.json which is loaded.  There are several items in there where virt_only = true

Additional info:
Normal automation from the subscription-manager side wouldn't find this since we compare our list available with the REST call, so this might need a dedicated spec-test.

Comment 1 William Poteat 2013-04-12 18:58:28 UTC
The test data is a bad way to go here. In fact its a bad way to test as it can hide issues from both the developers and QE. There are reasons why a pool would not show on --all. 

Start date is in the future.
The owner is not the same for the pool and the consumer.

I will look at the spec tests to see what is covered.

Comment 2 William Poteat 2013-04-15 18:19:51 UTC
A physical consumer will not get virtual_only pools. They are filtered from consideration at the server. The '--all' flag does not change that filtering.


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