Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Margo Management Interface - Initial Construction/Proposal #19

Merged
merged 5 commits into from
Sep 17, 2024

Conversation

ajcraig
Copy link
Contributor

@ajcraig ajcraig commented Aug 5, 2024

Description

Initial Contribution towards defining the Margo Management Interface.

Issues Addressed

List and link all the issues addressed by this PR.

Change Type

Please select the relevant options:

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

Checklist

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

@ajcraig ajcraig requested a review from a team as a code owner August 5, 2024 20:49
@ajcraig ajcraig self-assigned this Aug 5, 2024
Copy link
Contributor

@arne-broering arne-broering left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Many Thanks for this very good contribution, Armand.
I only had a bit of difficulty reading, as the reading order is not quite clear yet.

Following todays call's discussion, I think the abstraction of the API definition could be possible: (1) defining all messages (YAMLs) in a separate spec section, and (2) defining how the messages are exchanged in specific "protocol bindings" sections (e.g., for HTTP REST or WASM or CoAP or ...).

@ajcraig ajcraig merged commit 2d4c630 into pre-draft Sep 17, 2024
2 checks passed
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.

5 participants