fix(sdk): accelerator type setting in kfp (#11373)

An api breaking change was introduced in 2.10 that removed deprecate
fields in the compilation step for the accelerator fields. This has
resulted in the driver being unable to fetch the old fields. This change
re-introduces the deprecated fields to allow for a proper timespan for
allowing driver to adjust to the new values.

Signed-off-by: Humair Khan <HumairAK@users.noreply.github.com>

sdk/python/kfp/compiler/pipeline_spec_builder.py

@@ -667,6 +667,10 @@ def convert_to_placeholder(input_value: str) -> str:
                         task.container_spec.resources.accelerator_type),
                     resource_count=convert_to_placeholder(
                         task.container_spec.resources.accelerator_count),
+                    type=convert_to_placeholder(
+                        task.container_spec.resources.accelerator_type),
+                    count=convert_to_placeholder(
+                        int(task.container_spec.resources.accelerator_count)),
                 ))
 
     return container_spec

sdk/python/test_data/pipelines/pipeline_with_resource_spec.yaml

@@ -61,6 +61,8 @@ deploymentSpec:
         image: gcr.io/my-project/my-fancy-trainer
         resources:
           accelerator:
+            count: '1'
+            type: tpu-v3
             resourceCount: '1'
             resourceType: tpu-v3
           resourceCpuLimit: '4'