Bug 1771969

Summary: [Docs] Document defining static routes on RHV hosts
Product: Red Hat Enterprise Virtualization Manager Reporter: Dominik Holler <dholler>
Component: DocumentationAssignee: Rolfe Dlugy-Hegwer <rdlugyhe>
Status: CLOSED CURRENTRELEASE QA Contact: Roni <reliezer>
Severity: unspecified Docs Contact:
Priority: high    
Version: 4.4.0CC: lsurette, mavital, mburman, rdlugyhe, reliezer, srevivo
Target Milestone: ovirt-4.4.1Keywords: Documentation
Target Release: ---Flags: rdlugyhe: needinfo-
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: No Doc Update
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-07-24 19:20:46 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Network RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1107803    
Bug Blocks: 1771977    

Description Dominik Holler 2019-11-13 10:15:30 UTC
Thew new way of defining static routes proposed in https://bugzilla.redhat.com/show_bug.cgi?id=1311590#c20 should be added to the documentation.
The official documentation will supersede https://access.redhat.com/solutions/46060 for RHV-4.4

Comment 7 Roni 2020-07-13 09:24:37 UTC
Hi Rolfe

In the Add section at https://access.redhat.com/documentation/en-us/red_hat_virtualization/4.4-beta/html-single/administration_guide/index#proc-Adding-a-static-route-on-a-host
9.1.6. Adding a static route on a host

At the example below "metric: 100" can be removed, it not must:
routes:
  config:
  - destination: 192.168.123.0/24
    metric: 100                        <- should be removed from the example because it not required, if not entered it will get a default value from the system.
    next-hop-address: 192.168.178.1
    next-hop-interface: eth1



In the delete section:
https://access.redhat.com/documentation/en-us/red_hat_virtualization/4.4-beta/html-single/administration_guide/index#proc-Removing-a-static-route-on-a-host

At the example below: "[eth1]" should be changed to: "[{“name”: eth1}]"
routes:
  config:
  - destination: 192.168.123.0/24
    metric: 100
    next-hop-address: 192.168.178.
    next-hop-interface: eth1
    state: absent
interfaces: [eth1]      <- should be changed to:  [{“name”: eth1}]


Thx
Roni

Comment 8 Rolfe Dlugy-Hegwer 2020-07-14 14:50:26 UTC
Also this from https://bugzilla.redhat.com/show_bug.cgi?id=1771977#c5

"It is recommended to document that the custom static-route will be kept whenever 
the related route interface/bond exists and has an IP address. And will be removed if not.
As a result, there is different behavior between VM-Network and Non-VM-Network. 
A VM-Network is based on a bridge, while Non-VM-Network on an interface.
So replacing interface/bond (by moving the Network to other interfaces/bond) 
will not affect the route that related to VM-Network but will delete a route that related to a Non-VM-Network"

Comment 9 Rolfe Dlugy-Hegwer 2020-07-22 17:46:15 UTC
Thanks Roni. I've made the changes you suggested. 

Please review the changes here: https://gitlab.cee.redhat.com/rhci-documentation/docs-Red_Hat_Enterprise_Virtualization/-/merge_requests/1720

If it looks good, just comment "LGTM" below.

Comment 10 Roni 2020-07-23 06:14:00 UTC
LGTM
Thanks for fixing it

Comment 11 Rolfe Dlugy-Hegwer 2020-07-24 19:23:48 UTC
Merged and published.