Skip to content

Remove unsupported flow log limitations for ApplyOnForward policies#2555

Open
dimitri-nicolo wants to merge 1 commit intotigera:mainfrom
dimitri-nicolo:dimitri-EV-6405
Open

Remove unsupported flow log limitations for ApplyOnForward policies#2555
dimitri-nicolo wants to merge 1 commit intotigera:mainfrom
dimitri-nicolo:dimitri-EV-6405

Conversation

@dimitri-nicolo
Copy link
Contributor

Product Version(s):
Calico Enterprise 3.22+, Calico Cloud 22-2+

Issue:
EV-6372

Link to docs preview:

SME review:

  • An SME has approved this change.

DOCS review:

  • A member of the docs team has approved this change.

Additional information:
Remove the outdated limitation stating that flow logs on ApplyOnForward policies are not supported. This limitation no longer applies to Calico Enterprise 3.22+ and Calico Cloud.

Affected files:

  • calico-cloud/observability/elastic/flow/hep.mdx
  • calico-cloud_versioned_docs/version-22-2/observability/elastic/flow/hep.mdx
  • calico-enterprise/observability/elastic/flow/hep.mdx
  • calico-enterprise_versioned_docs/version-3.22-2/observability/elastic/flow/hep.mdx
  • calico-enterprise_versioned_docs/version-3.23-1/observability/elastic/flow/hep.mdx

Merge checklist:

  • Deploy preview inspected wherever changes were made
  • Build completed successfully
  • Test have passed

Copilot AI review requested due to automatic review settings March 2, 2026 20:50
@dimitri-nicolo dimitri-nicolo requested a review from a team as a code owner March 2, 2026 20:50
@netlify
Copy link

netlify bot commented Mar 2, 2026

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

Name Link
🔨 Latest commit 9073167
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/69a5f82829ebfa00087d7a95
😎 Deploy Preview https://deploy-preview-2555--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

netlify bot commented Mar 2, 2026

Deploy Preview succeeded!

Built without sensitive environment variables

Name Link
🔨 Latest commit 9073167
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/69a5f8289378fc0008eb5596
😎 Deploy Preview https://deploy-preview-2555--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: 67 (🔴 down 6 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.

@dimitri-nicolo
Copy link
Contributor Author

dimitri-nicolo commented Mar 2, 2026

@ctauchen could I please get a review? It's ready to merge if you don't have additional comments

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes an outdated limitation from the HostEndpoint (HEP) flow logs documentation, aligning Calico Enterprise (3.22+) and Calico Cloud (22-2+) docs with current product behavior for ApplyOnForward policies.

Changes:

  • Removed the “Flow logs on ApplyOnForward policies are currently not supported” limitation text from the relevant HEP flow log pages.
  • Applied the same removal consistently across unversioned (“next”) and the specified versioned documentation sets (Enterprise 3.22-2, 3.23-1; Cloud 22-2).

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
calico-cloud/observability/elastic/flow/hep.mdx Removes the outdated ApplyOnForward flow log limitation from Cloud “next” docs.
calico-cloud_versioned_docs/version-22-2/observability/elastic/flow/hep.mdx Removes the outdated limitation from Cloud 22-2 versioned docs.
calico-enterprise/observability/elastic/flow/hep.mdx Removes the outdated limitation from Enterprise “next” docs.
calico-enterprise_versioned_docs/version-3.22-2/observability/elastic/flow/hep.mdx Removes the outdated limitation from Enterprise 3.22-2 versioned docs.
calico-enterprise_versioned_docs/version-3.23-1/observability/elastic/flow/hep.mdx Removes the outdated limitation from Enterprise 3.23-1 versioned docs.

@ctauchen
Copy link
Collaborator

ctauchen commented Mar 3, 2026

LGTM, but need to check: If this is the result of code changes (bug fix being backported), are all the backports published? If not, we'll need to hold the versioned changes and publish with the appropriate patch release. @dimitri-nicolo

@dimitri-nicolo
Copy link
Contributor Author

The feature first appeared in v3.22.0-1.1 and it present in all v3.23 branches. So, it looks like we're good with respect to the currently affected versions for this change.

The one we'll have to wait for before making the update is the v3.21 branch, in which the change only appeared after v3.21.3. I can add it there when we bump that version

@ctauchen you should be good to merge this, thanks for checking

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