Skip to content

Conversation

@rnitzan
Copy link
Contributor

@rnitzan rnitzan commented Dec 4, 2025

Proposed changes

Add dos k8s and dos k8s with EBPF
Refactor dos documentation

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@github-actions github-actions bot added documentation Improvements or additions to documentation product/dos Issues related to F5 DoS for NGINX labels Dec 4, 2025
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/docs/1533/

@rnitzan rnitzan requested review from ADubhlaoich, EyalPery, aknot242, pkozakpl and vadimkrishtalf5 and removed request for pkozakpl December 7, 2025 14:00
@rnitzan rnitzan marked this pull request as ready for review December 8, 2025 07:32
@rnitzan rnitzan requested a review from a team as a code owner December 8, 2025 07:32
Copy link
Member

@ADubhlaoich ADubhlaoich left a comment

Choose a reason for hiding this comment

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

Generally LGTM but you have updated metadata unnecessarily.

The nd-content-type key and the nd-product value F5DOSN are newer conventions that should be kept: I don't know why they were removed in this PR.

toc: true
# Types have a 1:1 relationship with Hugo archetypes, so you shouldn't need to change this
nd-content-type: how-to
nd-product: DOS
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
nd-product: DOS
nd-product: F5DOSN

toc: true
# Types have a 1:1 relationship with Hugo archetypes, so you shouldn't need to change this
nd-content-type: how-to
nd-product: DOS
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
nd-product: DOS
nd-product: F5DOSN

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation product/dos Issues related to F5 DoS for NGINX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants