Skip to content

feat: add real-time memory data model changes - #4

Open
madalazar wants to merge 1 commit into
rt-sup-data-model-cachefrom
rt-sup-data-model-memory
Open

feat: add real-time memory data model changes#4
madalazar wants to merge 1 commit into
rt-sup-data-model-cachefrom
rt-sup-data-model-memory

Conversation

@madalazar

Copy link
Copy Markdown
Owner

Description

The purpose of this PR is to update the Memory object to the both Application Description/Deployment and Device Capabilities manifests to match the RT SUP.
As part of this PR we also had to update the data model to pass the requiredResources from a deploymentProfile from the application description to the application deployment manifest

Issues Addressed

Change Type

Please select the relevant options:

  • Fix (change that resolves an issue)
  • New enhancement (change that adds specification content)
  • Content edits (change that edits existing content)

Checklist

  • I have read the CONTRIBUTING document.
  • My changes adhere to the established patterns, and best practices.

Testing

Capabilities.json

  "memory": {
                "size": "69 Gi"
        },

Application mafest:

- type: helm
    id: rt-benchmark-helm
    description: >
   ...
    components:
      ...

    requiredResources:
      memory:
        size: 1024Mi
      storage: 2Gi

logs:

.saveDeviceCapabilities:235","diagnostics":{"correlationId":"ef232351-ac75-4e50-9d56-988d3703a07d","resourceId":"","traceContext":{"spanId":"","traceId":""}},"instance":"r09s12-c","level":"info","msg":"V (MargoDeviceVendor): saveDeviceCapabilities, device=client-cdb2e476aa6679f0-1787575600, cpu=cpu[0].kind[0]={cores=4, class=low-power, frequency={baseMHz=1800, maxMHz=3500}, type=shared, architecture=amd64}; cpu[0].kind[1]={cores=2, class=performance, frequency={baseMHz=2100, maxMHz=4900}, type=isolated, architecture=amd64}, cache=cache[0]={level=L3, allocationTypes=[shared|exclusive], size=115200KB}; cache[1]={level=L3, allocationTypes=[shared|exclusive], size=115200KB}, memory=memory={size=69 Gi, bandwidthAllocationTypes=none}

Signed-off-by: Lazar, Madalina <madalina.lazar@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant