Commit 37f599ef by Seth Vidal Committed by Michael DeHaan

make sure latest outputs an error if the thing you've listed as 'latest'

is not there at all or even updateable
parent 1d04ec89
...@@ -217,7 +217,9 @@ def ensure(my, state, pkgspec): ...@@ -217,7 +217,9 @@ def ensure(my, state, pkgspec):
updates = my.doPackageLists(pkgnarrow='updates', patterns=[pkgspec]).updates updates = my.doPackageLists(pkgnarrow='updates', patterns=[pkgspec]).updates
avail = my.doPackageLists(pkgnarrow='available', patterns=[pkgspec]).available avail = my.doPackageLists(pkgnarrow='available', patterns=[pkgspec]).available
if not updates and not avail: if not updates and not avail:
# there nothing in updates matching this. if not my.doPackageLists(pkgnarrow='installed', patterns=[pkgspec]).installed:
msg = "No Package matching '%s' found available, installed or updated" % pkgspec
return { 'changed':False, 'failed':True, 'msg': msg }
return { 'changed':False,} return { 'changed':False,}
# we have something in updates or available # we have something in updates or available
...@@ -226,6 +228,7 @@ def ensure(my, state, pkgspec): ...@@ -226,6 +228,7 @@ def ensure(my, state, pkgspec):
else: else:
cmd = "yum -c %s -d1 -y update '%s'" % (yumconf, pkgspec) cmd = "yum -c %s -d1 -y update '%s'" % (yumconf, pkgspec)
rc, out, err = run_yum(cmd) rc, out, err = run_yum(cmd)
# FIXME if it is - update it and check to see if it applied # FIXME if it is - update it and check to see if it applied
# check to see if there is no longer an update available for the pkgspec # check to see if there is no longer an update available for the pkgspec
if rc: if rc:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment