fix: Parse flexible shape name correctly for OCI API

- Extract base shape name (VM.Standard.E4.Flex) from full name (VM.Standard.E4.Flex-1-16)
- Parse CPU and memory config from shape suffix
- Use base shape name in LaunchInstance API call with separate ShapeConfig

Author: haimari

docs/flux-helmrelease-final.yaml

@@ -32,7 +32,13 @@ spec:
 
     settings:
       clusterName: "ashburn-ops-oke-stg"
-      logLevel: "info"
+      logLevel: "debug"
+    
+    # Add environment variable to enable OCI SDK debug logging
+    controller:
+      env:
+        - name: OCI_GO_SDK_DEBUG
+          value: "info"
 
     serviceAccount:
       create: false

pkg/providers/oci/client.go

@@ -21,6 +21,7 @@ import (
 	"encoding/base64"
 	"fmt"
 	"io"
+	"strconv"
 	"strings"
 	"time"
 
@@ -161,26 +162,44 @@ func (c *Client) LaunchInstance(ctx context.Context, nodeClaim *v1.NodeClaim, no
 			},
 		}
 
-		// Add shape config for flexible shapes
+		// Handle flexible shapes - extract base shape and config
+		actualShape := shape
 		if isFlexibleShape(shape) {
 			// Parse shape name to extract OCPUs and memory
 			// Format: VM.Standard.E4.Flex-1-16 (1 OCPU, 16GB memory)
 			var ocpus, memory int32 = 1, 16
-			if _, err := fmt.Sscanf(shape, "VM.Standard.E4.Flex-%d-%d", &ocpus, &memory); err == nil {
-				request.LaunchInstanceDetails.ShapeConfig = &core.LaunchInstanceShapeConfigDetails{
-					Ocpus:       common.Float32(float32(ocpus)),
-					MemoryInGBs: common.Float32(float32(memory)),
+			
+			// Extract base shape name and config
+			parts := strings.Split(shape, "-")
+			if len(parts) >= 3 {
+				// Get base shape (e.g., VM.Standard.E4.Flex)
+				actualShape = strings.Join(parts[:len(parts)-2], "-")
+				
+				// Try to parse CPU and memory from the last two parts
+				if cpu, err := strconv.Atoi(parts[len(parts)-2]); err == nil {
+					ocpus = int32(cpu)
+				}
+				if mem, err := strconv.Atoi(parts[len(parts)-1]); err == nil {
+					memory = int32(mem)
 				}
-				logger.Info("added shape config for flexible shape", 
-					"shape", shape, "ocpus", ocpus, "memory", memory)
 			}
+			
+			request.LaunchInstanceDetails.ShapeConfig = &core.LaunchInstanceShapeConfigDetails{
+				Ocpus:       common.Float32(float32(ocpus)),
+				MemoryInGBs: common.Float32(float32(memory)),
+			}
+			logger.Info("added shape config for flexible shape", 
+				"originalShape", shape, "actualShape", actualShape, "ocpus", ocpus, "memory", memory)
 		}
+		
+		// Update the shape in the request
+		request.LaunchInstanceDetails.Shape = &actualShape
 
 		// Log the launch request details including shape config
 		logFields := []interface{}{
 			"displayName", *request.LaunchInstanceDetails.DisplayName,
 			"subnet", *request.LaunchInstanceDetails.CreateVnicDetails.SubnetId,
-			"shape", shape,
+			"shape", actualShape,
 		}
 		if request.LaunchInstanceDetails.ShapeConfig != nil {
 			logFields = append(logFields,

scripts/test-launch-instance.sh

@@ -0,0 +1,28 @@
+#!/bin/bash
+# Test launching an instance with the same parameters as Karpenter
+
+COMPARTMENT_ID="ocid1.compartment.oc1..aaaaaaaalr5oi5mfqpjedsdsyn3vxn2fh2bltqezqrmk4bi7gaq6i245qnkq"
+IMAGE_ID="ocid1.image.oc1.iad.aaaaaaaaknzkygdhz6n3vmaovv3ouh2sxt7cudrdomefqfxnampnfrtpp6rq"
+SUBNET_ID="ocid1.subnet.oc1.iad.aaaaaaaaznwweno45m7klssbzt2kyl6qa4ec34335patiolemkti6d4ioita"
+SHAPE="VM.Standard.E4.Flex"
+
+# Get availability domain
+AD=$(oci iam availability-domain list --compartment-id "$COMPARTMENT_ID" --query "data[0].name" --raw-output)
+
+echo "Testing instance launch with:"
+echo "  Compartment: $COMPARTMENT_ID"
+echo "  Image: $IMAGE_ID"
+echo "  Subnet: $SUBNET_ID"
+echo "  Shape: $SHAPE"
+echo "  AD: $AD"
+
+# Test launching instance
+oci compute instance launch \
+  --compartment-id "$COMPARTMENT_ID" \
+  --availability-domain "$AD" \
+  --shape "$SHAPE" \
+  --shape-config '{"ocpus": 1, "memoryInGBs": 16}' \
+  --image-id "$IMAGE_ID" \
+  --subnet-id "$SUBNET_ID" \
+  --display-name "karpenter-test-manual" \
+  --assign-public-ip false