cohesity / management-sdk-python

Cohesity Management SDK for Python.
Apache License 2.0
21 stars 20 forks source link

[BUG / DOCUMENTATION INCOMPLETE] 'key_name' doesn't exist in KmsConfiguration class #61

Closed a-a-a-mc closed 2 years ago

a-a-a-mc commented 2 years ago

🐛 Bug Report

Can't create a KMS, I have this error : Response status code: 400, Response message: KMS Key Name is empty. By looking the attribute of a KMS created using GUI with kms_configuration_controller.get_kms_config() , i see an attribute named 'key_name'. This attribute is not in KmsConfiguration class, and it look like it is mandatory.

To Reproduce

  1. Trying to create an AWS kms
  2. Run it
  3. Response status code: 400, Response message: KMS Key Name is empty.

Expected behavior

Can created aws KMS

Actual Behavior

Response status code: 400, Response message: KMS Key Name is empty.

a-a-a-mc commented 2 years ago

Key Name is also not present on the doc : https://developer.cohesity.com/apidocs-64.html#/python/models/structures/kms-configuration

naveena-maplelabs commented 2 years ago

@aankri The documentation you're referring to is the older one, you can check the latest one available here

a-a-a-mc commented 2 years ago

@naveena-maplelabs I don't see the python SDK on your new doc ? By the way, the link on the cohesity doc ( https://docs.cohesity.com/6_6/Web/UserGuide/Content/RestAPI/RestAPIIntro.htm?tocpath=Tools%7C_____4 ) to the dev portal refer to the one linked above (6.4)

naveena-maplelabs commented 2 years ago

@aankri We generate individual SDK versions for different cluster versions, you can the refer the SDK(1.6.0) for 6.6.x release.

a-a-a-mc commented 2 years ago

@naveena-maplelabs can you please give the link ? There is no link for python 6.6 on this page too https://developer.cohesity.com/versions.html

a-a-a-mc commented 2 years ago

In any case, "key_name" is not present in the KmsConfiguration class, so this is not only a documentation issue . Please look at KmsConfiguration class :

    def __init__(self,
                 ca_certificate=None,
                 client_certificate=None,
                 client_key=None,
                 kmip_protocol_version=None,
                 server_ip=None,
                 server_name=None,
                 server_port=None,
                 server_type=None):
naveena-maplelabs commented 2 years ago

For creating the kms_config, use kms_create_request_parameters module. Controller reference

a-a-a-mc commented 2 years ago

Thanks, but I still need the link for the SDK doc up to date, do you have it ?

naveena-maplelabs commented 2 years ago

I currently dont have one, will share with you by EOD, thanks.

naveena-maplelabs commented 2 years ago

@aankri You can refer to the documentation available here. Page contains both V1 and V2 documentation, You can ignore the V2 API documentation which is a beta release.

naveena-maplelabs commented 2 years ago

Closing the ticket, please feel free to reopen if you have any queries.

a-a-a-mc commented 2 years ago

Thanks, i'm going to look at this neext week