standard-test-roles

#54 Please consider to use a different name than 'list' for fuctions defined in inventory/standard-inventory-*
Closed: SOLVED by astepano. Opened by plautrba.

Closed: SOLVED
Reopen Issue 
^_^ git grep 'def list'
inventory/standard-inventory-docker:def list(subjects, docker_extra_args):
inventory/standard-inventory-local:def list():
inventory/standard-inventory-qcow2:def list(subjects): 
inventory/standard-inventory-rpm:def list(subjects):

list is a class defined in standard library and using same name could be confusing, e.g. 

 69         if opts.host:
 70             data = host(opts.host)
 71         else:
 72             data = list(opts.subjects)

Good point.

When work completes on PR #39, this will be addressed for standard-inventory-qcow2.

I'll follow-up to make sure the other inventory scripts are eventually updated in a similar manner.

This is fixed.

def inv_list():

Metadata Update from @astepano:
- Issue close_status updated to: SOLVED
- Issue status updated to: Closed (was: Open)
Metadata
None
None
None
None
None
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.