KMS service definition: `SignResult` wrapper is not used

Describe the bug The result of the sign operation is wrapped in a field called SignResult, but this is not specified as a resultWrapper in kms/2014-11-01/service-2.json:

https://github.com/boto/botocore/blob/3b7a7b329eada78d1026095a09c9928d3c491e37/botocore/data/kms/2014-11-01/service-2.json#L745-L752

Use elsewhere Amazon’s AWS SDKs have a SignResult model, taking the Android one as an example: https://github.com/aws-amplify/aws-sdk-android/blob/d4116c1909f1ecb5a31edf8588882a6181b73deb/aws-android-sdk-kms/src/main/java/com/amazonaws/services/kms/AWSKMSClient.java#L5660-L5687

This should be an easy fix, let me know if I can just open a PR for it or if there’s more that needs to be done.

Thanks!

[zbaylin]

@stobrien89 Thanks!

I actually realized that this is more broad than just Sign. I think every KMS operation is “wrapped” like this:

ListKeys:

<ListKeysResponse xmlns="https://trent.amazonaws.com/doc/2014-11-01/">
  <ListKeysResult>
    <KeyCount>1</KeyCount>
    <Keys>
      <member>
        [redacted]
      </member>
    </Keys>
    <Truncated>false</Truncated>
  </ListKeysResult>
  <ResponseMetadata>
    <RequestId>[redacted]</RequestId>
  </ResponseMetadata>
</ListKeysResponse>

[github-actions[bot]]

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see. If you need more assistance, please either tag a team member or open a new issue that references this one. If you wish to keep having a conversation with other community members under this issue feel free to do so.