Update main_app.py

main_app.py

@@ -91,34 +91,19 @@ def _(os):
 
 @app.cell
 def _(client_instantiation_form, os):
-    if client_instantiation_form.value:
-        client_setup = client_instantiation_form.value
-    else:
-        client_setup = None
+    client_setup = client_instantiation_form.value or None
 
     ### Extract Credential Variables:
-    if client_setup is not None:
-        wx_url = client_setup["wx_region"]
-        wx_api_key = client_setup["wx_api_key"].strip()
-        os.environ["WATSONX_APIKEY"] = wx_api_key
-
-        if client_setup["project_id"] is not None:
-            project_id = client_setup["project_id"].strip()
-        else:
-            project_id = None
-
-        if client_setup["space_id"] is not None:
-            space_id = client_setup["space_id"].strip()
-        else:
-            space_id = None
+    if client_setup:
+        wx_url = client_setup["wx_region"] if client_setup["wx_region"] else "EU"
+        wx_api_key = client_setup["wx_api_key"].strip() if client_setup["wx_api_key"] else None
+        os.environ["WATSONX_APIKEY"] = wx_api_key or ""
 
+        project_id = client_setup["project_id"].strip() if client_setup["project_id"] else None
+        space_id = client_setup["space_id"].strip() if client_setup["space_id"] else None
     else:
         os.environ["WATSONX_APIKEY"] = ""
-        project_id = None
-        space_id = None
-        wx_api_key = None
-        wx_url = None
-
+        project_id = space_id = wx_api_key = wx_url = None
     return client_setup, project_id, space_id, wx_api_key, wx_url
 
 
@@ -196,41 +181,52 @@ def _(
 ):
     ### Instantiate the watsonx.ai client
     if client_setup:
-        wx_credentials = Credentials(url=wx_url, api_key=wx_api_key)
-
-        if project_id:
-            project_client = APIClient(
-                credentials=wx_credentials, project_id=project_id
-            )
-        else:
-            project_client = None
-
-        if space_id:
-            deployment_client = APIClient(credentials=wx_credentials, space_id=space_id)
-            client = deployment_client
-        else:
-            deployment_client = None
-            client = None
-
-        if project_client is not None:
-            task_credentials_details = setup_task_credentials(project_client)
-        elif deployment_client is not None:
-            task_credentials_details = setup_task_credentials(deployment_client)
-        elif client is not None:
-            task_credentials_details = setup_task_credentials(client)
-        elif project_client is not None and deployment_client is not None:
-            task_credentials_details = setup_task_credentials(project_client)
-        else:
-            task_credentials_details = None
-
+       wx_credentials = Credentials(url=wx_url, api_key=wx_api_key)
+        
+       try:
+           project_client = (
+               APIClient(credentials=wx_credentials, project_id=project_id)
+               if project_id
+               else None
+           )
+       except Exception:
+           project_client = None
+
+       try:
+           deployment_client = (
+               APIClient(credentials=wx_credentials, space_id=space_id)
+               if space_id
+               else None
+           )
+       except Exception:
+           deployment_client = None
+
+       try:
+           task_credentials_details = (
+              setup_task_credentials(project_client)
+              if project_client
+              else setup_task_credentials(deployment_client)
+              if deployment_client
+              else None
+           )
+           instantiation_success = True
+           instantiation_error = None
+       except Exception as e:
+           instantiation_success = False
+           instantiation_error = str(e)
+           task_credentials_details = None
     else:
-        wx_credentials = None
-        project_client = None
-        deployment_client = None
-        task_credentials_details = None
-        client = None
+       wx_credentials = project_client = deployment_client = task_credentials_details = None
+       instantiation_success = None
+       instantiation_error = None
 
-    return client, deployment_client, project_client, task_credentials_details
+    return (
+        deployment_client,
+        instantiation_error,
+        instantiation_success,
+        project_client,
+        task_credentials_details
+    )
 
 
 @app.cell
@@ -641,6 +637,7 @@ def _(ast, template, function_editor, mo, os):
 @app.cell
 def _(template):
     dependencies = template["dependencies"]
+    dependencies_string = ', '.join(dependencies)
     added_yaml_template = f"""dependencies:
           - pip
           - pip:
@@ -713,23 +710,27 @@ def _(mo, yaml_template, dependency_pack_name):
                 placeholder="Python Package for...",
                 label="Package Extension Name:",
                 kind="text",
-                value=f"Custom Python Package: {dependency_pack_name}",
+                value=f"{dependency_pack_name}_custom_python_pack",
+                max_length=int(31)
             ),
             software_spec_name=mo.ui.text(
                 placeholder="Software Spec Name",
                 label="Custom Software Spec Name:",
                 kind="text",
-                value=f"Extended Python Function Software Spec - {dependency_pack_name}",
+                value=f"{dependency_pack_name}_python_sw_spec",
+                max_length=int(31)
             ),
             package_description=mo.ui.text_area(
                 placeholder="Write a description for your package.",
                 label="Package Description:",
                 value=" ",
+                max_length=int(256)
             ),
             software_spec_description=mo.ui.text_area(
                 placeholder="Write a description for your software spec.",
                 label="Software Spec Description:",
                 value=" ",
+                max_length=int(256)
             ),
             package_type=mo.ui.dropdown(
                 pkg_types, label="Select your package type:", value="Conda Yaml"
@@ -754,7 +755,8 @@ def _(mo):
         )
         .batch(
             package_list=mo.ui.text_area(
-                placeholder="Add packages as a comma separated list (with or without versions)."
+                placeholder="Add packages as a comma separated list (with or without versions).", 
+                value=str(dependencies_string)
             ),
             return_full_list=mo.ui.checkbox(
                 value=False,
@@ -823,7 +825,7 @@ def _(client, create_yaml_tempfile, package_meta, uuid):
         pack_suffix = str(uuid.uuid4())[:4]
         pack_name = package_meta.value["package_name"]
         pe_metadata = {
-            client.package_extensions.ConfigurationMetaNames.NAME: f"{pack_name}_{pack_suffix}",
+            client.package_extensions.ConfigurationMetaNames.NAME: f"{pack_name}_{pack_suffix}"[:36],
             client.package_extensions.ConfigurationMetaNames.TYPE: package_meta.value[
                 "package_type"
             ],
@@ -1079,7 +1081,7 @@ def _(
         ss_name = package_meta.value["software_spec_name"]
 
         ss_metadata = {
-            client.software_specifications.ConfigurationMetaNames.NAME: f"{ss_name}_{ss_suffix}",
+            client.software_specifications.ConfigurationMetaNames.NAME: f"{ss_name}_{ss_suffix}"[:36],
             client.software_specifications.ConfigurationMetaNames.DESCRIPTION: package_meta.value[
                 "software_spec_description"
             ],
@@ -1722,9 +1724,10 @@ def _(client, mo, pd, upload_button, uuid):
         uuid_suffix = str(uuid.uuid4())[:4]
 
         deployment_name = mo.ui.text(
-            value=f"{uploaded_function_name.value}_deployed_{uuid_suffix}",
+            value=f"{uploaded_function_name.value}_{uuid_suffix}_dep"[:36],
             label="Deployment Name:",
             placeholder="<Must be completely unique>",
+            max_length=int(31)
         )
     else:
         hw_df = pd.DataFrame(data=[["ID", "Activate client."]], columns=["ID", "VALUE"])
@@ -1770,16 +1773,16 @@ def _(
             deployment_type.value == "Online (Function Endpoint)"
         ):  # Changed from "Online (Function Endpoint)"
             deployment_props = {
-                client.deployments.ConfigurationMetaNames.NAME: deployment_name.value,
+                client.deployments.ConfigurationMetaNames.NAME: str(deployment_name.value)[:36],
                 client.deployments.ConfigurationMetaNames.ONLINE: {},
                 client.deployments.ConfigurationMetaNames.HARDWARE_SPEC: {
                     "id": selected_hw_config
                 },
-                client.deployments.ConfigurationMetaNames.SERVING_NAME: deployment_name.value,
+                client.deployments.ConfigurationMetaNames.SERVING_NAME: str(deployment_name.value)[:36],
             }
         else:  # "Runnable Job" instead of "Batch (Runnable Jobs)"
             deployment_props = {
-                client.deployments.ConfigurationMetaNames.NAME: deployment_name.value,
+                client.deployments.ConfigurationMetaNames.NAME: str(deployment_name.value)[:36],
                 client.deployments.ConfigurationMetaNames.BATCH: {},
                 client.deployments.ConfigurationMetaNames.HARDWARE_SPEC: {
                     "id": selected_hw_config