Skip to content

docs(policy-activity): note that Log action rules are not tracked#2748

Open
tianfeng92 wants to merge 4 commits into
tigera:mainfrom
tianfeng92:add-notes-for-log-action-in-policy-activity
Open

docs(policy-activity): note that Log action rules are not tracked#2748
tianfeng92 wants to merge 4 commits into
tigera:mainfrom
tianfeng92:add-notes-for-log-action-in-policy-activity

Conversation

@tianfeng92
Copy link
Copy Markdown
Contributor

Summary

  • Add a limitation entry to the policy activity docs clarifying that action: Log rules do not appear in policy activity logs
  • The Log action is a diagnostic tool (kernel log / eBPF trace pipe) outside the policy evaluation scope tracked by policy activity
  • Only terminal actions (Allow, Deny, Pass) appear in policy activity logs

Test plan

  • Verify the docs site builds without errors
  • Review the limitation text for accuracy

🤖 Generated with Claude Code

Add a limitation entry clarifying that action: Log rules do not appear
in policy activity logs since the Log action is outside the policy
evaluation scope.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@tianfeng92 tianfeng92 requested a review from a team as a code owner May 22, 2026 20:48
@netlify
Copy link
Copy Markdown

netlify Bot commented May 22, 2026

Deploy Preview for calico-docs-preview-next ready!

Name Link
🔨 Latest commit c198131
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/6a10c96758aa2a000847f2a6
😎 Deploy Preview https://deploy-preview-2748--calico-docs-preview-next.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 project configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 22, 2026

Deploy Preview succeeded!

Built without sensitive environment variables

Name Link
🔨 Latest commit c198131
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/6a10c9672b30a50008c9b6e5
😎 Deploy Preview https://deploy-preview-2748--tigera.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 97 (🟢 up 28 from production)
Accessibility: 98 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

Copy link
Copy Markdown
Contributor

@dimitri-nicolo dimitri-nicolo left a comment

Choose a reason for hiding this comment

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

lgtm

@dimitri-nicolo
Copy link
Copy Markdown
Contributor

dimitri-nicolo commented May 22, 2026

@tianfeng92 Do you need to add this to the non-versioned docs as well?

tianfeng92 and others added 3 commits May 22, 2026 14:22
Move the Log action note from the versioned 3.23-1 policy-activity.mdx
to the current review-unused-network-policies.mdx where it belongs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@tianfeng92
Copy link
Copy Markdown
Contributor Author

@tianfeng92 Do you need to add this to the non-versioned docs as well?

@dimitri-nicolo Good call. Just added

@dimitri-nicolo
Copy link
Copy Markdown
Contributor

dimitri-nicolo commented May 22, 2026

@tianfeng92 Do you need to add this to the non-versioned docs as well?

@dimitri-nicolo Good call. Just added

there should be the same doc file in the non-versioned docs that we need to update. I expect that once we've added the new docs the versioned and non-versioned changes should mirror each other, unless the docs have been refactored in the non-versioned docs.

@tianfeng92
Copy link
Copy Markdown
Contributor Author

tianfeng92 commented May 22, 2026

@dimitri-nicolo

@tianfeng92 Do you need to add this to the non-versioned docs as well?

@dimitri-nicolo Good call. Just added

there should be the same doc file in the non-versioned docs that we need to update. I expect that once we've added the new docs the versioned and non-versioned changes should mirror each other, unless the docs have been refactored in the non-versioned docs.

@dimitri-nicolo The original file for the incoming 3.23 EP2 was removed in a previous PR. I added calico-enterprise/observability/review-unused-network-policies.mdx as the completed user instruction.

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