Commit 774a8733 authored by Dimitrios Gogos's avatar Dimitrios Gogos
Browse files

fix: update standalone field mapping in app registration flow and translation example

parent ca29d89b
Loading
Loading
Loading
Loading
+3 −3
Changes for oeg/app-registration-flow.md: 3 added lines, 3 removed lines.
Original line number Diff line number Diff line
@@ -203,7 +203,7 @@ AppRegistrationTranslation
  requiredResources
    infraKind                 str         validated to be "kubernetes" only (ADR-0009); does NOT
                                          flow into any SRM field (domain_kind below is a fixed literal)
    isStandalone              bool        → resource_requirements.compute.standalone (required in the vendored schema)
    isStandalone              bool        → resource_requirements.standalone (required in the vendored schema)
    applicationResources
      cpuPool                 obj?        optional (GPU-only apps may omit it)
        cpu_millicores        int         numCPU × 1000
@@ -273,7 +273,7 @@ SRM creates all rows in one transaction.
|---|---|---|
| `compute.cpu_millicores` | `cpuPool.numCPU` + `gpuPool.numCPU` | × 1000 each (vCPU → millicores); summed if both pools present, `gpuPool` is sole source if `cpuPool` omitted (GPU-only apps) |
| `compute.memory_mb` | `cpuPool.memory` + `gpuPool.memory` | direct (already MB); summed if both pools present, `gpuPool` is sole source if `cpuPool` omitted (GPU-only apps) |
| `compute.standalone` | `isStandalone` | direct |
| `standalone` | `isStandalone` | direct; top-level field on `resource_requirements`, not nested under `compute` |
| `compute.accelerator.type` | *(fixed)* | `"gpu"` when gpuPool present (UC2) |
| `compute.accelerator.units` | *(fixed)* | `1` — CAMARA's `gpuPool` has no GPU-count field (UC2) |
| `compute.accelerator.memory_mb` | `gpuPool.gpuMemory` | × 1000 (GB → MB, decimal per CAMARA spec); UC2 only |
@@ -376,7 +376,7 @@ Compact end-to-end mapping across UC1–UC3 (rows marked UC2/UC3 apply only to t
| `appRepo.userName` | direct | `service_deployment_units[0].metadata.repo.user_ref` | SRM (UC1) |
| `appRepo.credentials` | **→ secret:// ref** | `service_deployment_units[0].metadata.repo.credentials` | SRM — secret ref only |
| `appRepo.authType` | direct | `service_deployment_units[0].metadata.repo.auth_type` | SRM (UC1) |
| `requiredResources.isStandalone` | direct | `resource_requirements.compute.standalone` | SRM |
| `requiredResources.isStandalone` | direct | `resource_requirements.standalone` | SRM |
| `cpuPool.numCPU` + `gpuPool.numCPU` | **× 1000** each (vCPU → millicores); summed if both present, gpuPool sole source if cpuPool omitted | `resource_requirements.compute.cpu_millicores` | SRM |
| `cpuPool.memory` + `gpuPool.memory` | direct (already MB); summed if both present, gpuPool sole source if cpuPool omitted | `resource_requirements.compute.memory_mb` | SRM |
| `cpuPool.topology.minNumberOfNodes` | direct | `resource_requirements.topology.min_nodes` | SRM |
+7 −7
Changes for oeg/translation-example-uc1.md: 7 added lines, 7 removed lines.
Original line number Diff line number Diff line
@@ -136,7 +136,6 @@ SRM creates all rows in one transaction.
        "compute": {
          "cpu_millicores": 2000,
          "memory_mb": 2048,
          "standalone": false,
          "storage": []
        },
        "topology": {
@@ -152,7 +151,8 @@ SRM creates all rows in one transaction.
            "port": 8080,
            "visibility": "external"
          }
        ]
        ],
        "standalone": false
      }
    }
  ],
@@ -247,7 +247,6 @@ has no source field for either (no GPU-count field exists in the vendored schema
        "compute": {
          "cpu_millicores": 2000,
          "memory_mb": 2048,
          "standalone": false,
          "accelerator": { "type": "gpu", "units": 1, "memory_mb": 16000 },
          "storage": []
        },
@@ -260,7 +259,8 @@ has no source field for either (no GPU-count field exists in the vendored schema
        "interfaces": [
          { "component": "nginx_component", "interface_id": "web_port_80",
            "protocol": "TCP", "port": 80, "visibility": "external" }
        ]
        ],
        "standalone": false
      }
    }
  ],
@@ -299,14 +299,14 @@ The kubernetes `requiredResources.additionalStorage` is a **string** (`"10GB"`),
        "compute": {
          "cpu_millicores": 2000,
          "memory_mb": 2048,
          "standalone": false,
          "storage": [ { "name": "data", "size_mb": 10000, "mount_point": "/data" } ]
        },
        "topology": { "min_nodes": 2, "min_node_cpu_millicores": 1000, "min_node_memory_mb": 1024 },
        "interfaces": [
          { "component": "nginx_component", "interface_id": "web_port_80",
            "protocol": "TCP", "port": 80, "visibility": "external" }
        ]
        ],
        "standalone": false
      }
    }
  ],
@@ -355,7 +355,7 @@ Field-by-field mapping for the UC1 request (container, private repo).
| `appRepo.userName` | `registry_user` | direct | `service_deployment_unit.metadata.repo.user_ref` | `"...metadata.repo.user_ref"` | `registry_user` |
| `appRepo.credentials` | `secret_token_123` | **→ secret:// ref** (raw credential stored in vault, substituted with a reference before the SRM call) | `service_deployment_unit.metadata.repo.credentials` | `"...metadata.repo.credentials"` | `secret://vault/...` |
| `appRepo.authType` | `DOCKER` | direct | `service_deployment_unit.metadata.repo.auth_type` | `"...metadata.repo.auth_type"` | `DOCKER` |
| `isStandalone` | `false` | direct | `resource_requirements.compute.standalone` | `"...compute.standalone"` | `false` |
| `isStandalone` | `false` | direct | `resource_requirements.standalone` | `"...resource_requirements.standalone"` | `false` |
| *(fixed)* | — | — | `resource_requirements.compute.storage` | `"...compute.storage"` | `[]` (UC1 has no additionalStorage) |
| `cpuPool.numCPU` | `2` | **× 1000** | `resource_requirements.compute.cpu_millicores` | `"...compute.cpu_millicores"` | `2000` |
| `cpuPool.memory` | `2048` | direct | `resource_requirements.compute.memory_mb` | `"...compute.memory_mb"` | `2048` |