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

[EN] update FaaS & Serverless #2217

Merged
merged 12 commits into from
Mar 26, 2024

Conversation

johannes-manner
Copy link
Contributor

Update FaaS and Serverless definitions based on a scientific investigation of these terms

Serverless and FaaS are often used interchangeably when people talk about event-driven stateless functions.
I am a 6th year PhD student at the University of Bamberg, worked a lot on FaaS in the last years as you can see by my published list of papers.
The ambigous use of terms motivated me to make a structured assessment of scientific literature, how they use the terms FaaS and Serverless. A preprint to my paper can be found here.
The reference currently included in the text (Manner 2023) is the landing page (forwarded by doi.org) at IEEE as soon as the paper is published - I presented it last week on IEEE CLOUD 2023.

Hope we have a nice discussion here about these two terms :)

Checklist before opening this PR (put x in the checkboxes)

  • This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

@netlify
Copy link

netlify bot commented Jul 11, 2023

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit d1c782b
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/66035a739883030007e6135b
😎 Deploy Preview https://deploy-preview-2217--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added the lang/en for English label Jul 11, 2023
Signed-off-by: Johannes Manner <johannes.manner@web.de>
@johannes-manner johannes-manner force-pushed the serverless-faas-definition branch from 459786c to 8438de5 Compare July 11, 2023 07:36
@jihoon-seo
Copy link
Collaborator

To reviewers: you can hide whitespace changes with this link. It will make reviewing easier.

@jihoon-seo
Copy link
Collaborator

BTW welcome and thank you for your contribution! Please consider joining #glossary channel in CNCF Slack too!

Co-authored-by: Jihoon Seo <46767780+jihoon-seo@users.noreply.github.com>
Signed-off-by: Johannes Manner <johannes.manner@web.de>
@nate-double-u nate-double-u changed the title Updates definitions for FaaS and Serverless (EN) [EN] update FaaS & Serverless Jul 12, 2023
Signed-off-by: Johannes Manner <johannes.manner@web.de>
@johannes-manner
Copy link
Contributor Author

Can anyone of you, @jihoon-seo, @nate-double-u, @iamNoah1, rerun the Spellcheck CI? :)

Signed-off-by: Johannes Manner <johannes.manner@web.de>
@CathPag
Copy link
Collaborator

CathPag commented Jul 24, 2023

Thanks, @johannes-manner. To see where this is in the review pipeline and follow progress, please have a look at the project board.

@iamNoah1 iamNoah1 added the update term Updates to existing terms label Jul 31, 2023
@iamNoah1 iamNoah1 assigned iamNoah1 and unassigned iamNoah1 Sep 6, 2023
Copy link
Contributor

@rolfedh rolfedh left a comment

Choose a reason for hiding this comment

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

Excellent work, Johannes! I've shared some recommendations aimed at refining the clarity and conciseness of your text. You're welcome to incorporate any of these suggestions as you prefer. For enhanced maintainability, it would be beneficial to insert line breaks between sentences, rather than splitting them, as observed in certain sections. Additionally, it's advisable to link primarily to the official CNCF documentation for authoritative sources. When citing external references, ensure a variety of sources are used to avoid reliance on a single external site.

content/en/function-as-a-service.md Outdated Show resolved Hide resolved
content/en/function-as-a-service.md Outdated Show resolved Hide resolved
content/en/serverless.md Outdated Show resolved Hide resolved
content/en/serverless.md Outdated Show resolved Hide resolved
content/en/serverless.md Outdated Show resolved Hide resolved
content/en/function-as-a-service.md Outdated Show resolved Hide resolved
@nate-double-u
Copy link
Member

Hi @johannes-manner, thanks for your patience. @rolfedh has provided some good feedback - I know it's been a while, but I'd be happy if you're still interested in working through the process of getting your updates in.

iamNoah1 and others added 4 commits March 18, 2024 16:53
Co-authored-by: Rolfe Dlugy-Hegwer <rolfedh@users.noreply.github.com>
Signed-off-by: Noah Ispas <Noahispas@gmail.com>
Co-authored-by: Rolfe Dlugy-Hegwer <rolfedh@users.noreply.github.com>
Signed-off-by: Noah Ispas <Noahispas@gmail.com>
Co-authored-by: Rolfe Dlugy-Hegwer <rolfedh@users.noreply.github.com>
Signed-off-by: Noah Ispas <Noahispas@gmail.com>
Co-authored-by: Rolfe Dlugy-Hegwer <rolfedh@users.noreply.github.com>
Signed-off-by: Noah Ispas <Noahispas@gmail.com>
iamNoah1 and others added 3 commits March 18, 2024 16:53
Co-authored-by: Rolfe Dlugy-Hegwer <rolfedh@users.noreply.github.com>
Signed-off-by: Noah Ispas <Noahispas@gmail.com>
Co-authored-by: Rolfe Dlugy-Hegwer <rolfedh@users.noreply.github.com>
Signed-off-by: Noah Ispas <Noahispas@gmail.com>
Signed-off-by: Noah Ispas <Noahispas@gmail.com>
@johannes-manner
Copy link
Contributor Author

Hi, I got notified via mail of your pushes :)
Sorry for not responding - was a bussy time the last weeks.
I would try to look at the changes end of next week and contribute to it.
@iamNoah1, @nate-double-u is this timeline suitable for you?

Copy link
Collaborator

@iamNoah1 iamNoah1 left a comment

Choose a reason for hiding this comment

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

I added the suggestions of @rolfedh and hope to have resolved the merge conflict correctly. Thanks for everyone involved especially @johannes-manner for the first and biggest batch of work.

Copy link
Member

@nate-double-u nate-double-u left a comment

Choose a reason for hiding this comment

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

Thanks for this @johannes-manner and @rolfedh. These are some good changes.

/approve

Based on @johannes-manner's comment above, I'd like to wait a week before actually merging this in case they have some additional comments based on the changes made, but this is very close to landing 🙂

Correcting spelling.

Signed-off-by: Nate W <natew@cncf.io>

Signed-off-by: Nate W <natew@cncf.io>
@nate-double-u nate-double-u merged commit c2235f4 into cncf:main Mar 26, 2024
6 checks passed
@guillaumebernard84 guillaumebernard84 mentioned this pull request Aug 25, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/en for English update term Updates to existing terms
Projects
Development

Successfully merging this pull request may close these issues.

6 participants