Skip to content

feature: Secure your fleet, NGINX One #731

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

mjang
Copy link
Contributor

@mjang mjang commented Jun 23, 2025

Proposed changes

Create end-to-end "use-case" documentation for admins / security engineers who want to use NGINX One Console to "Secure your fleet".

Fleets of NGINX deployments frequently include dozens and many more instances. With this use case, an admin/security engineer can set up the NGINX One Console to send them alerts for appropriate issues. Today, those issues include CVEs and other detected security "misconfigurations" identified by NGINX One Console.

This use case goes somewhat beyond NGINX One Console. This PR removes roadblocks to success in the following ways:

  • It clarifies what users need to access the NGINX One Console, specifically with:

    • Supporting checks of appropriate licenses
    • Describing the detailed process of setting up a tenant
  • It then shows users, step by step, how to set up notifications when one/more of their instances have CVEs and other detected security issues.

Replaces #637

Checklist

Before merging a pull request, run through this checklist and mark each as complete.

  • I have read the contributing guidelines
  • I have signed the F5 Contributor License Agreement (CLA)
  • I have rebased my branch onto main
  • I have ensured my PR is targeting the main branch and pulling from my branch from my own fork
  • I have ensured that the commit messages adhere to Conventional Commits
  • I have ensured that documentation content adheres to the style guide
  • If the change involves potentially sensitive changes1, I have assessed the possible impact
  • If applicable, I have added tests that prove my fix is effective or that my feature works
  • I have ensured that existing tests pass after adding my changes
  • If applicable, I have updated README.md and CHANGELOG.md

Footnotes

  1. Potentially sensitive changes include anything involving code, personally identify information (PII), live URLs or significant amounts of new or revised documentation. Please refer to our style guide for guidance about placeholder content.

@mjang mjang self-assigned this Jun 23, 2025
@mjang mjang requested review from a team as code owners June 23, 2025 14:11
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nginx-one NGINX One Console labels Jun 23, 2025
Copy link

Deploy Preview will be available once build job completes!

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

Copy link
Contributor

@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.

LGTM. Small LOGAF non-blocking edit suggestions.

If the PR replaces #637, then that should probably be closed.

@mjang mjang mentioned this pull request Jun 23, 2025
10 tasks
@jputrino
Copy link
Contributor

I find the order in which the docs are presented confusing. Is "Manage your fleet" really before "Get started"? Shouldn't "Manage your instances" include items like Connect your instances and Draft configurations?

I could see 3 top-level headings on the landing page: Get started, Secure your fleet, and Manage your fleet. Or maybe 4, if we also add one for Admin tasks (like RBAC, metrics, etc.).

@mjang
Copy link
Contributor Author

mjang commented Jun 25, 2025

I find the order in which the docs are presented confusing. Is "Manage your fleet" really before "Get started"? Shouldn't "Manage your instances" include items like Connect your instances and Draft configurations?

I could see 3 top-level headings on the landing page: Get started, Secure your fleet, and Manage your fleet. Or maybe 4, if we also add one for Admin tasks (like RBAC, metrics, etc.).

Will be addressed in an internal issue (num-200 in internal-docs repo)

@mjang mjang force-pushed the secure-your-fleet-r3 branch from 2caef64 to 7152a0d Compare July 2, 2025 16:52
@mjang mjang requested a review from a team as a code owner July 2, 2025 16:52
@github-actions github-actions bot added product/nim NGINX Instance Manager product/ngf Issues related to NGINX Gateway Fabric product/agent NGINX Agent tooling Back end, repository, Hugo, and all things not related to content product/controller NGINX Controller (EOS product) product/mesh NGINX Service Mesh (EOS product) labels Jul 2, 2025
@mjang
Copy link
Contributor Author

mjang commented Jul 8, 2025

If that change doesn't get in, then we need to rewrite the adding alerts part of configure alert policy to something like this:

  • Enable 'Show Advanced Fields'
  • Choose Matching RegEx of Alertname
  • Type in [name of CVE alert/security recommendation alert]

image (20)

@mjang mjang mentioned this pull request Jul 9, 2025
10 tasks
@mjang mjang force-pushed the secure-your-fleet-r3 branch 3 times, most recently from 2f9fbfe to cb81103 Compare July 15, 2025 18:17
@mjang
Copy link
Contributor Author

mjang commented Jul 16, 2025

@travisamartin thank you for the detailed review. I've accepted all but 2 of your suggestions.

@mjang mjang force-pushed the secure-your-fleet-r3 branch 2 times, most recently from 40b4b3d to 783a943 Compare July 18, 2025 14:25
@mjang mjang force-pushed the secure-your-fleet-r3 branch from b40ab0a to 5e39326 Compare July 30, 2025 19:36
This commit adds a new landing page archetype, which has the
ability to display various cards to highlight specific items. The
archetype includes inline guidance like other archetypes, including
explanations of new frontmatter parameters and a new card shortcode.

---------

Co-authored-by: Mike Jang <[email protected]>
Co-authored-by: Alan Dooley <[email protected]>

Co-authored-by: Alan Dooley <[email protected]>

Co-authored-by: Travis Martin <[email protected]>
@mjang mjang force-pushed the secure-your-fleet-r3 branch from 5de175a to 92057f0 Compare August 3, 2025 16:42
@mjang
Copy link
Contributor Author

mjang commented Aug 3, 2025

I now have set up and tested a working process for alerts with a production tenant of F5 Distributed Cloud. Here's a screenshot of an email notification

Sanitized_alert_email_message

@mjang mjang force-pushed the secure-your-fleet-r3 branch from 566259e to 1fc2ede Compare August 3, 2025 19:30
1. Select **Add Item**

You've now set up F5 Distributed Cloud to send you alerts from the NGINX One Console to your email. When an alert is triggered, you'll receive a message from **[email protected]**.

Copy link
Contributor Author

@mjang mjang Aug 5, 2025

Choose a reason for hiding this comment

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

Based on separate discussions, @jasonclopper @travisamartin

Suggested change
## Known issues
When you set up an email alert that recognizes a problem, you'll see the alert in:
- The F5 Distributed Cloud Console, in **Audit Logs & Alerts**, under **Notifications > Alerts**.
- An email with a subject line like **<number> Alert Requires Action**.
As defined in our [Alert Reference](https://docs.cloud.f5.com/docs-v2/platform/reference/alerts-reference), after a certain period of time, you may also receive an "Alert Resolved" email.
For CVEs, the authoritative source is in **NGINX One**, under **Manage > Instances > <Instance hostname>.** See the list of CVEs on the dashboard details for that instance.

Copy link
Contributor

@travisamartin travisamartin Aug 5, 2025

Choose a reason for hiding this comment

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

This section doesn’t describe the actual issue. The problem is that users may receive “Alert Resolved” emails even when the issue still exists, which is misleading.

If we don’t call out that behavior directly, I’m not sure “Known issues” is the right heading. As it stands, the text just tells users to defer to NGINX One Console as the source of truth.

Copy link
Contributor

Choose a reason for hiding this comment

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

Note: If you want to document this as a known issue, I recommend creating a KB article and linking to it from here. The KB system is the right place for that kind of content. It also helps make sure Global Services is aware of the problem.

Copy link
Contributor

Choose a reason for hiding this comment

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

The filename /secure-your-fleet/secure.md could be more descriptive.

Maybe /secure-your-fleet/set-up-security-alerts.md? That'd match the doc title.

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/agent NGINX Agent product/controller NGINX Controller (EOS product) product/mesh NGINX Service Mesh (EOS product) product/ngf Issues related to NGINX Gateway Fabric product/nginx-one NGINX One Console product/nim NGINX Instance Manager tooling Back end, repository, Hugo, and all things not related to content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants