Not in this diff but the warning log about finding the wrong number of endpoints is bogus.
if len(ps_endpoints) != 1: log.warning("found {} product-streams endpoints in region {}," " expecting one - not updating" " endpoint".format(ps_endpoints, region, len(ps_endpoints)))
It has two placeholders and three values to format in there.
Not in this diff but the warning log about finding the wrong number of endpoints is bogus.
if len(ps_endpoints) != 1:
log.warning( "found {} product-streams endpoints in region {},"
" expecting one - not updating"
" endpoint" .format( ps_endpoints, region,
len( ps_endpoints) ))
It has two placeholders and three values to format in there.
It's crying out for tests.