Skip to content

Conversation

@marciocloudflare
Copy link
Contributor

@marciocloudflare marciocloudflare commented Nov 21, 2024

Summary

Closes PCX-6618

Documentation checklist

Copy link
Contributor

@hyperlint-ai hyperlint-ai bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 9 total issue(s) found.

Co-authored-by: hyperlint-ai[bot] <154288675+hyperlint-ai[bot]@users.noreply.github.com>
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 21, 2024

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8db6e32
Status: ✅  Deploy successful!
Preview URL: https://930e2a3a.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://marcio-pcx6618-mwan-juniper.cloudflare-docs-7ou.pages.dev

View logs

```

```txt
admin@srx220> show configuration security policies from-zone trust to-zone cloudflare
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be (...) from-zone cloudflare to-zone trust instead?


```txt
set security zones security-zone cloudflare interfaces st0.0 host-inbound-traffic system-services all
set security zones security-zone cloudflare interfaces st0.0 host-inbound-traffic
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if this command should be referencing st0.1 instead (it appears in the output below).

**Or using the [qualified-next-hop](https://www.juniper.net/documentation/us/en/software/junos/static-routing/topics/ref/statement/qualified-next-hop-edit-routing-options.html) option:**

```txt
admin@srx220> show configuration routing-options | display set
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe this line should be removed. If we keep it, the next three lines would be output, not commands entered by the user. Can you check, please?

@marciocloudflare marciocloudflare merged commit a4e69c1 into production Nov 26, 2024
12 checks passed
@marciocloudflare marciocloudflare deleted the marcio-pcx6618-mwan-juniper branch November 26, 2024 09:57
harshil1712 pushed a commit that referenced this pull request Dec 3, 2024
…18320)

* added content

* cleaned markdown

* refined page

* added juniper to dev compat page

* Apply suggestions from code review

Co-authored-by: hyperlint-ai[bot] <154288675+hyperlint-ai[bot]@users.noreply.github.com>

* Apply suggestions from code review

Co-authored-by: Pedro Sousa <[email protected]>

* separated keys in kbd

---------

Co-authored-by: hyperlint-ai[bot] <154288675+hyperlint-ai[bot]@users.noreply.github.com>
Co-authored-by: Pedro Sousa <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants