fix: fix issue with update method args

[benieric]

Issue #, if available:https://github.com/aws/sagemaker-core/issues/7

Description of changes:

• Only exclude resource attributes that are identifiers from the method args of update method
• More details in the github issue
  • With previous way, we are giving user impression that they can do something like:

    resource.attribute = updated_value
    resource.update()

    This way of updating is not possible for almost all cases since many attributes are not updatable by user (for example, training_job_status). As such we should only exclude resource identifiers (ie, "resource_name", "resource_arn", "id", "name", etc) from the method args for update().

FROM:

def update(
    self,
) -> Optional["Cluster"]:
    logger.debug("Creating cluster resource.")
    client = SageMakerClient().client

    operation_input_args = {
        "ClusterName": self.cluster_name,
        "InstanceGroups": self.instance_groups,
    }
    logger.debug(f"Input request: {operation_input_args}")
    # serialize the input request
    operation_input_args = Cluster._serialize(operation_input_args)
    logger.debug(f"Serialized input request: {operation_input_args}")

    # create the resource
    response = client.update_cluster(**operation_input_args)
    logger.debug(f"Response: {response}")
    self.refresh()

    return self

TO:

def update(
    self,
    instance_groups: List[ClusterInstanceGroupSpecification], # only exclude resource attributes that are identifiers
) -> Optional["Cluster"]:
    logger.debug("Creating cluster resource.")
    client = SageMakerClient().client

    operation_input_args = {
        "ClusterName": self.cluster_name,
        "InstanceGroups": instance_groups,
    }
    logger.debug(f"Input request: {operation_input_args}")
    # serialize the input request
    operation_input_args = Cluster._serialize(operation_input_args)
    logger.debug(f"Serialized input request: {operation_input_args}")

    # create the resource
    response = client.update_cluster(**operation_input_args)
    logger.debug(f"Response: {response}")
    self.refresh()

    return self

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.