Comment 2 for bug 907521

This is how the list roles method is defined in the wadl
Ref: https://github.com/openstack/keystone/blob/master/keystone/content/admin/OS-KSADM-admin.wadl

<!--Roles-->
<method name="GET" id="listRoles">
<doc xml:lang="EN" title="List Roles">
<p xmlns="http://www.w3.org/1999/xhtml">List roles.</p>

</doc>
<request>
<param name="serviceId" style="query" required="false" type="xsd:string"/> => Optional service id
<param name="marker" style="query" required="false" type="xsd:string"/>
<param name="limit" style="query" required="false" type="xsd:int"/>
</request>
<response status="200 203">
<representation mediaType="application/xml" element="identity:roles">
<doc xml:lang="EN">
<xsdxt:code href="../common/samples/roles.xml"/>
</doc>
</representation>
<representation mediaType="application/json">
<doc xml:lang="EN">
<xsdxt:code href="../common/samples/roles.json"/>
</doc>
</representation>
</response>
&commonFaults;
&getFaults;
</method>

Our list roles method should support the same call with an optional serviceId as our wadl supports the optional call with serviceId.