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

Create README.md for Discovery folder with intents #128

Merged
merged 3 commits into from
Dec 15, 2023

Conversation

Kevsy
Copy link
Collaborator

@Kevsy Kevsy commented Aug 24, 2023

Added mapping of each API to intents

Fixes issue #129

Added mapping of each API to intents
@Kevsy
Copy link
Collaborator Author

Kevsy commented Aug 24, 2023

Fixes #129

@sergiofranciscoortiz
Copy link
Collaborator

sergiofranciscoortiz commented Aug 29, 2023

Some of the intents included in the README for Discovery API may also be included or fit better in one of the other API groups:

5 "I can ask the operator to provision my application server to the optimal MEC for a specific terminal..." - As it talks about provision could fit on WorkloadOrchestration

24 “I can map an application’s requirements to the best MEC for hosting it, based on application demands for CPU,Memory,Storage,GPU,bandwith,Network forecast, mobility” _(aligns with 4,5,8,9) - Could also be included in Workload Orchestration

Comment on lines 31 to 33
5. "I can ask the operator to provision my application server to the optimal MEC for a specific terminal, taking into account connectivity, resources (e.g. vCPU, Memory, network interfaces, storage, GPU) shortest network path, cost, network load, MEC platform load, application privacy considerations etc."
* "I can ask the operator to provision my application server to all MECs that meet these criteria (note this is not focussing on a specific terminal)"
* "I can ask the operator to provision my application server to a minimal set of MECs that meet these criteria across a given footprint (note this is not focussing on a specific terminal)"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think these intents related to provisioning should be moved to the Workload Orchestration API, otherwise this Discovery API risks becoming a superset of the Workload Orchestration API. There are quite a few Workload Orchestration APIs around artifacts, files, apps, appinstances, etc that would need to be duplicated here to be able to support these intents. I think it would be better to keep the APIs as independent as possible.

Copy link
Collaborator Author

@Kevsy Kevsy Sep 5, 2023

Choose a reason for hiding this comment

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

@maheshc01 please can you advise on @sergiofranciscoortiz and @gainsley Jon's questions above? Not sure if MEC Exposure & Experience Management is exclusively Discovery or arguably overlaps into Workload Orchestration.

Copy link
Collaborator

Choose a reason for hiding this comment

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

yes. i am good with moving this intent to workload orchestration as the intent specifically highlights need for provisioning.

Copy link
Collaborator

Choose a reason for hiding this comment

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

As commented during calls I think it make sense to move intent 5 to workload orchestration, as for intent 24 it can stay in discovery although it might be considered duplicated with other intents already included.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@sergiofranciscoortiz all commnets above now reflected with a new commit to the README in this PR, now ready to merge.

@Kevsy
Copy link
Collaborator Author

Kevsy commented Dec 14, 2023

@sergiofranciscoortiz please can you merge if you are happy with the changes made following the discussion? Then I can delete the branch.

Copy link
Collaborator

@sergiofranciscoortiz sergiofranciscoortiz left a comment

Choose a reason for hiding this comment

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

Accepted changes to facilitate branch removal, but since the discussion in October of what intents are covered by which sub api there have been relevant changes.

Now the goal is to merge current SED API with MVP API that already covers some of the intents included in this README ( for example intent 1), so we may need to reorganize the folder structure for resultant APIs.

@sergiofranciscoortiz sergiofranciscoortiz merged commit b469d28 into main Dec 15, 2023
@Kevsy Kevsy deleted the Kevsy-patch-7 branch January 15, 2024 16:21
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.

4 participants