Skip to content

Conversation

@developer-guy
Copy link
Contributor

@developer-guy developer-guy commented Nov 2, 2020

Description

This time, we are implementing Mutating Admission Webhook as OpenFaaS Function

Types of changes

  • New blog post
  • Updating an existing blog post
  • Updating part of the page page
  • Adding a new web-page

Checklist:

  • I have given attribution for any images I have used and have permission to use them under Copyright law
  • My code follows the writing-style of the publication and I have checked this
  • I've read the CONTRIBUTION guide
  • I have signed-off my commits with git commit -s

@developer-guy developer-guy force-pushed the feature/mutating-admission-webhook-as-function branch from f65c7e1 to fa01ad9 Compare November 2, 2020 15:25
@developer-guy
Copy link
Contributor Author

Changes have been made in #200 is merged with this PR.

@developer-guy developer-guy force-pushed the feature/mutating-admission-webhook-as-function branch from fa01ad9 to 6485838 Compare November 22, 2020 19:16
@developer-guy developer-guy force-pushed the feature/mutating-admission-webhook-as-function branch from 6485838 to e773bb3 Compare November 22, 2020 19:40
Copy link
Member

@alexellis alexellis left a comment

Choose a reason for hiding this comment

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

The basic flow is good, but the example needs work. We want users to identify with the use-case and to potentially try it out, not print hello world to the disk of every container. So please take a detailed read of the feedback and think about how we can make this something everyone will want to try.

@alexellis
Copy link
Member

/set title: Blog post for mutating webhooks

@derek derek bot changed the title feature: mutating webhook post ready Blog post for mutating webhooks Nov 23, 2020
@alexellis alexellis removed the request for review from Waterdrips November 23, 2020 09:43
@developer-guy developer-guy force-pushed the feature/mutating-admission-webhook-as-function branch 3 times, most recently from 7399a1f to 849da43 Compare November 25, 2020 21:22
@developer-guy
Copy link
Contributor Author

The basic flow is good, but the example needs work. We want users to identify with the use-case and to potentially try it out, not print hello world to the disk of every container. So please take a detailed read of the feedback and think about how we can make this something everyone will want to try.

Everything seems fine to me, I made changes according to your reviews, thank you

@developer-guy developer-guy force-pushed the feature/mutating-admission-webhook-as-function branch from 849da43 to 882b388 Compare November 25, 2020 21:36
Signed-off-by: Batuhan Apaydın <batuhan.apaydin@trendyol.com>
@developer-guy developer-guy force-pushed the feature/mutating-admission-webhook-as-function branch from 882b388 to eb0bc40 Compare November 25, 2020 21:44
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.

3 participants