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

Add use-case for Stateful CNF (0003) #165

Merged
merged 8 commits into from
Jun 21, 2021
Merged

Conversation

sishbi
Copy link
Contributor

@sishbi sishbi commented May 7, 2021

This use-case makes the case for a Stateful CNF and suggests a best practice for how Stateful CNFs should work in a kube-native environment.
It introduces the 5G core Convergent Charging System (CCS) as an example and includes 2 diagrams showing how the CCS interacts with other stateless CNFs in the 5G core network.

FYI: @taylor @iawells @jeffsaelens @Smitholi67

@taylor
Copy link
Member

taylor commented May 13, 2021

I think it would be good for the glossary to have terms for different types of state. Eg. ephemeral, cache, user data.
discussion

That could be a new PR.

Updated to resolve comments in PR, to relax the requirements to store state in memory.
@sishbi
Copy link
Contributor Author

sishbi commented Jun 14, 2021

@taylor Please can you add this to the agenda for 14/06 meeting to prompt reviewers to re-review this use-case.

@sishbi sishbi requested review from rannyh and removed request for Smitholi67 June 14, 2021 15:36
@taylor
Copy link
Member

taylor commented Jun 17, 2021

@sishbi I was OOO on the 14th. I'll bring it up on the 21st.

In the mean time everyone can review before then as well :)

... @electrocucaracha, @fkautz, @iawells, @ildikov, @michaelspedersen, @denverwilliams

Resolve comment about types of state vs what state is
Copy link
Member

@taylor taylor left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Collaborator

@pgoyal01 pgoyal01 left a comment

Choose a reason for hiding this comment

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

subject to a minor correction

Copy link
Collaborator

@rannyh rannyh left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Pankaj Goyal <52107136+pgoyal01@users.noreply.github.com>
Copy link
Collaborator

@nickolaev nickolaev left a comment

Choose a reason for hiding this comment

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

LGTM

@taylor
Copy link
Member

taylor commented Jun 21, 2021

+1 from @wavell and @tliron on CNF WG call Jun 21

@taylor taylor merged commit b1a9c21 into lfn-cnti:main Jun 21, 2021
@sishbi sishbi deleted the stateful-use-case branch June 21, 2021 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants