-
Notifications
You must be signed in to change notification settings - Fork 83
Platform, Backend Probe and Backend Capability APIs not unified between the in proc SDKs #1279
Copy link
Copy link
Open
Labels
Area-DotNet-SDK.NET SDK APIs, native bindings, packaging, samples, or tests..NET SDK APIs, native bindings, packaging, samples, or tests.Area-Node-SDKNode.js and TypeScript SDK APIs, bindings, packaging, CLI integration, or tests.Node.js and TypeScript SDK APIs, bindings, packaging, CLI integration, or tests.Area-Rust-SDKPublic Rust SDK APIs and integration with the MXC execution engine.Public Rust SDK APIs and integration with the MXC execution engine.Issue-BugSomething is not working as intended.Something is not working as intended.Priority2High-impact issue affecting key functionality; prioritize for the next appropriate release.High-impact issue affecting key functionality; prioritize for the next appropriate release.Release v1.0Issues tracked for the version 1.0 releaseIssues tracked for the version 1.0 release
Description
Activity
Metadata
Metadata
Assignees
Labels
Area-DotNet-SDK.NET SDK APIs, native bindings, packaging, samples, or tests..NET SDK APIs, native bindings, packaging, samples, or tests.Area-Node-SDKNode.js and TypeScript SDK APIs, bindings, packaging, CLI integration, or tests.Node.js and TypeScript SDK APIs, bindings, packaging, CLI integration, or tests.Area-Rust-SDKPublic Rust SDK APIs and integration with the MXC execution engine.Public Rust SDK APIs and integration with the MXC execution engine.Issue-BugSomething is not working as intended.Something is not working as intended.Priority2High-impact issue affecting key functionality; prioritize for the next appropriate release.High-impact issue affecting key functionality; prioritize for the next appropriate release.Release v1.0Issues tracked for the version 1.0 releaseIssues tracked for the version 1.0 release
Relevant area(s)
Linux, Windows, macOS
Brief description of your issue
Both Node and dot net in SDKs should use the mxc ffi layer to probe backend capability and availability.
Also: The node sdk currently uses the executables like wxc-exec to query for platform support> I noticed in here: https://github.com/microsoft/mxc/blob/main/sdk/node/src/platform.ts that there are probes for seatbelt, and bubble wrap. These probes go through their executables.
We should make sure these probes go through the
mxc_ffilayer for both node and dotnet. The Rust SDK should also have theseSteps to reproduce
Attempt to use node sdk and see that you any probe support uses the executables
Expected behavior
probe support for all non rust sdks use mxc_ffi layer. This includes dotnet as well. For rust SDK this support should already be there.
Actual behavior
existing probe support uses executables for node.