-
Notifications
You must be signed in to change notification settings - Fork 13
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
6.3.0 release documentation #676
Conversation
docs/config_dnat.md
Outdated
[`dynamic-source-nat`](config_command_guide.md#configure-authority-router-node-device-interface-network-interface-dynamic-source-nat): Defines the prefixes that need to be dynamically source natted for packets ingressing this interface. | ||
|
||
- [`local-ip`](config_command_guide.md#configure-authority-router-node-device-interface-network-interface-dynamic-source-nat-local-ip): For packets ingressing this interface, the IP that is source NAT'ed to the `remote-ip`. | ||
- [`remote-ip`](config_command_guide.md#configure-authority-router-node-device-interface-network-interface-dynamic-source-nat-remote-ip): For packets ingressing this interface, the IP where the `local-ip` will be source NAT'ed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe worth mentioning remote-ip
need to be /32
prefix.
docs/config_dnat.md
Outdated
@@ -0,0 +1,76 @@ | |||
--- | |||
title: Dynamic Source NAT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe more a question for @avinashbhat80 and @MichaelBaj, shall we use Dynamic Ingress Source NAT
to be more accurate naming for this feature?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do believe Dynamic Ingress Source NAT is more descriptive.
docs/config_static_nat.md
Outdated
@@ -0,0 +1,109 @@ | |||
--- | |||
title: Static NAT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this came from the func-spec. But I would like to propose this feature to be updated Static Bidirectional NAT
because the content is referring to bidirectional NAT
often.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The static NAT and dynamic source NAT parts look good 👍.
docs/config_static_nat.md
Outdated
|
||
### Non-SVR Traffic | ||
|
||
In order for non-SVR traffic (for example, LAN-to-LAN traffic traversing a single SSR) to take advantage of static-NAT addressing, you must disable egress source-nat at the service level by setting `service > source-nat` to `disabled` as shown below. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are two more ways to disable egress source nat that we want to include here
- disable egress source-nat at the network-interface level by setting
network-interface > source-nat
tofalse
and not configuringnetwork-interface > egress-source-nat-pool
authority
router SSR-router
name SSR-router
node SSR-node
name SSR-node
device-interface egress-LAN
name egress-LAN
network-interface egress-LAN
name egress-LAN
source-nat false
egress-source-nat-pool <DON'T CONFIGURE>
exit
exit
exit
exit
exit
- disable egress source-nat at the service-route level by not configuring
service-route > next-hop > source-nat-pool
authority
router SSR
service-route LAN-to-LAN-route
name LAN-to-LAN-route
service-name LAN-to-LAN
next-hop node egress-LAN
node-name SSR-node
interface egress-LAN
source-nat-pool <DON'T CONFIGURE>
exit
exit
exit
exit
docs/config_dnat.md
Outdated
|
||
Dynamic Source NAT translates multiple source IP addresses into a smaller pool of translated addresses and dynamic ports, which conserves public IP address space and provides the flexibility to source NAT a specific IP range. This supports scaling up sessions for an internal service. For example, in a corporate office with a SIP phone service where all phones have different IPs on port 5060, these internal IP addresses are source NAT’ed to a single external IP address. | ||
|
||
Dynamic Source NAT may also provide solutions for IP address conflicts, but because it is not mapping NAT one-to-one, it is not required to facilitate the destination NAT mapping for network connections from the external client to the internal client. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe more clearer to the reader that they should not be confused or expect this to work like bidirectional-nat
.
Dynamic Source NAT may also provide solutions for IP address conflicts, but the IP mapping is not one-to-one and does NOT provide destination NAT in the reverse direction. To facilitate the destination NAT mapping for network connections from the external client to the internal client, use
bidirectional-nat
(add a link here).
… make it more usable.
…adding network interface info.
…added the \ delimiter.
…change for the Uiso doc.
Co-authored-by: Kaushik Agrawal <[email protected]>
Co-authored-by: Kaushik Agrawal <[email protected]>
Co-authored-by: Kaushik Agrawal <[email protected]>
…y/docs into 6.3.0-release-documentation
docs/initialize_u-iso_other.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The url of this page still says initialize_u-iso_other
instead of the `advanced workflows name
docs/initialize_u-iso_other.md
Outdated
"dns-servers": [ | ||
"8.8.8.8", | ||
"1.1.1.1" | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: the indentation of this isn't consistent
docs/initialize_u-iso_other.md
Outdated
|
||
1. Serial Number (via dmidecode) | ||
2. Hostname | ||
3. UUID (via dmidecode) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@agrawalkaushik would the proper name for this be DMI Decode
or just via DMI
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Additionally, these defaults apply to mist managed
docs/initialize_u-iso_other.md
Outdated
|
||
`/etc/128T-hardware-bootstrapper/onboarding-config.json` | ||
`/etc/128T-hardware-bootstrapper/pre-bootstrap` | ||
`/etc/128T-hardware-bootstrapper/post-bootstrap` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From the web these are not showing up on multiple lines. Not sure what needs to be done to fix that
docs/initialize_u-iso_other.md
Outdated
- path: /etc/128T-hardware-bootstrapper/onboarding-config.json | ||
content: | | ||
{ | ||
"registration_code": "RegistrationCode", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should actually be registration-code
docs/about_releases.mdx
Outdated
@@ -24,6 +24,7 @@ Version `6.1.0` introduces changes to the SSR software release model. Every six | |||
|
|||
| Version | Initial GA Version | First Release Shipping Date | Latest GA Version | End of Software Engineering support | End of Support | | |||
| --| -- | -- | -- | -- | -- | | |||
| Release 6.3 | [6.3.0](release_notes_128t_6.3.md) | September 19, 2024 | [6.3.0](release_notes_128t_6.3.md) | June 19, 2025 | March 19, 2026 | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Release date needs to be updated. Should be 9/30
docs/releases.table.js
Outdated
{ | ||
version: '6.3.0', | ||
url: 'release_notes_128t_6.3#release-630-1', | ||
releaseDate: 'September 19, 2024', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Release date should be updated to 9/30.
|
||
### Limitations | ||
|
||
Traffic engineering includes a performance impact to the packet-per-second processing rate of the worker cores. When used in conjunction with other traffic engineering settings, performance is further impacted; each level of traffic engineering requires buffering for their scheduled objects. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't like the way this is worded. Suggestion:
Enabling traffic engineering will introduce a performance impact to the packet-per-second processing rate as the QoS engine works to ensure fairness of packet distribution under congestion scenarios. When used in conjunction with other traffic engineering settings (e.g., adjacency traffic engineering configured alongside device interface traffic engineering), performance may be further impacted.
==================================================== ============== ============ ============== ============= ====== ================== | ||
Metric Node Peer-name Peer-host Device-name Vlan Value | ||
==================================================== ============== ============ ============== ============= ====== ================== | ||
dequeue-cycle-count combo-east-a combo-west 172.16.102.2 11-red 0 7077545161474020 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the value 7077545161474020
accurate? That seems like a bogus number.
docs/cc_fips_otp_router_install.md
Outdated
config authority router RTR_EAST_COMBO node combo-east-2 ssh-settings inter-node host-key-checking yes | ||
``` | ||
|
||
To configure a new authorized key for ssh inter-node communitcation, use the [`create system connectivity authorized-keys`](cli_reference.md#create-system-connectivity-authorized-keys) command. This command adds an entry to the ssh authorized keys file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo. Please run through a spell checker for this large PR.
communitcation = communication
docs/release_notes_128t_6.3.md
Outdated
------ | ||
- **I95-49218 Filter OSPF routes using RIB Policy routes:** Use the `configure authority router routing rib-policy` command from either the routing default-instance (`configure authority router routing`) or inside `configure authority router routing vrf` to provide addtional filtering for OSPF routes. For more information see [`configure authority router routing rib-policy`](config_command_guide.md#configure-authority-router-routing-rib-policy) and [`configure authority router routing vrf rib-policy`](config_command_guide.md#configure-authority-router-routing-vrf-rib-policy). | ||
------ | ||
- **I95-49712 Configuration validation error uniformative:** Resolved an issue that when configuring an SSR, invalid configuration parameters were returning errors that were not specific enough to allow the user to locate the invalid configuration. Now when invalid configuration elements are identified during validation, the messages include relevant information for the invalid element, such as an IP address, node name, router name, interface names, etc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggest breaking up the sentence. Add a period after validation. Start the next sentence with The messages include...
docs/release_notes_128t_6.3.md
Outdated
------ | ||
- **I95-52337 Uninformative error when STEP is selected for the Conductor:** The error message now clearly states that STEP is not supported on the Conductor. | ||
------ | ||
- **I95-54844 Default to Multi-threading:** New session processing rates are now increased by default when the system has sufficient CPU resources, by using multiple CPU threads. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Default to Multi-threading for session processing
docs/release_notes_128t_6.3.md
Outdated
------ | ||
- **I95-54844 Default to Multi-threading:** New session processing rates are now increased by default when the system has sufficient CPU resources, by using multiple CPU threads. | ||
------ | ||
- **I95-55725 Highway crashes when peer-path routers are removed:** Resolved a race condition that could cause a crash in the highway worker-core packet-processor if peer routers are removed from the configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I95-55725 Highway crashes when peer-path routers are removed: Resolved a race condition that could cause a crash in the forwarding plane (highway) if peer routers are removed from the configuration.
docs/release_notes_128t_6.3.md
Outdated
------ | ||
- **I95-57099 Race condition causing crash in highway process when peer path timers expire:** Resolved an issue with handling BFD timers in multi-threaded environments. | ||
------ | ||
- **I95-57110 Crash seen during add and delete peers while sending traffic:** A race condition has been fixed that could cause a crash in the packet-processing highway process if a peer-path is removed from configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could cause a crash in the forwarding plane (highway) process
------ | ||
|
||
### Caveats | ||
<!-- markdown-link-check-disable --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This link check disable should not be needed. This tells me that the reference to the KB article is incorrect.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The link works and if i recall we had to do this because when it tried to build it would fail. We couldn't figure out why so we disabled the link. I'll investigate after the release if I don't get a chance before.
…l do that later tonight/tomorrow morning.
- pre-bootstrap | ||
- post-bootstrap | ||
1. Create a USB device named **BOOTSTRAP**. | ||
2. On the USB, create the following files: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the root partition? /
? Somewhere else?
docs/intro_installation_legacy.md
Outdated
The examples listed in this guide generally prefer running commands as a non-root user, except as noted, and prepend commands that must be run as a superuser with sudo. **The SSH Root login is not permitted.** | ||
::: | ||
|
||
## Installing SSR Software Versions 6.2.x and Earlier |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this section redundant since you created this legacy guide?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The legacy guide and the 6.3.0 install guide overviews follow the same format. So the Installing section is laid out the same in both - this one just makes it clear that this is for versions 6.2.x and earlier. Are you suggesting that I change the heading to "Installation Process"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed the heading to Installation Process.
|
||
A Mist-redirect ZTP process for Conductor-managed deployments is supported on Juniper branded hardware devices - the SSR1x0/1x00. See [Onboard an SSR Device to a Conductor](onboard_ssr_to_conductor.md) for details about this process. | ||
|
||
### Image-based Installation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be moved to the beginning with a pointer to the new guide?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think so; the image based install is part of the legacy process for 6.0.x to 6.2.x. 6.3.0 uses the U-ISO which is different than the earlier image based install.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added this to the paragraph:
Image-based Installation
SSR Version 6.3.0 provides support for image-based installs for both Conductor- and Mist-managed deployments. Earlier installations (6.0.x - 6.2.x) using the image-based install only support Mist-managed deployments.
@@ -17,7 +17,7 @@ export const releases = [ | |||
}, | |||
{ | |||
version: '6.1.10', | |||
url: 'release_notes_128t_6.1#release-6110-8', | |||
url: 'release_notes_128t_6.1#release-6110-8-lts', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of the other releases updated in the other release notes page need to be updated here as well.
No description provided.