Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1660517

Summary: [DOCS] Update Kuryr docs including removing tech preview
Product: OpenShift Container Platform Reporter: Jon Uriarte <juriarte>
Component: DocumentationAssignee: Aine Riordan <ariordan>
Status: CLOSED CURRENTRELEASE QA Contact: Jon Uriarte <juriarte>
Severity: high Docs Contact: Vikram Goyal <vigoyal>
Priority: high    
Version: 3.11.0CC: ahardin, aos-bugs, ariordan, jokerman, juriarte, ltomasbo, mcurry, mdulko, mmccomas, oblaut, racedoro
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1685481 (view as bug list) Environment:
Last Closed: 2019-04-02 05:33:48 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1685481    

Comment 1 Luis Tomas Bolivar 2018-12-19 13:11:14 UTC
I took a look to the documentation and have the next comments:

- We need to also update https://docs.openshift.com/container-platform/3.11/install_config/configuring_openstack.html
- At [1] (Installation section) we need to review the options that we are providing, as most of them are handled by the dynamic inventory. To my knowledge, the next ones should/may be needed (only the related to kuryr ones):
openshift_use_kuryr: True
openshift_use_openshift_sdn: False
openshift_master_open_ports:
- service: dns tcp
  port: 53/tcp
- service: dns udp
  port: 53/udp
openshift_node_open_ports:
- service: dns tcp
  port: 53/tcp
- service: dns udp
  port: 53/udp
use_trunk_ports: True
os_sdn_network_plugin_name: cni
openshift_node_proxy_mode: userspace

kuryr_openstack_public_net_id: 8711d122-4d92-4ece-aebb-178aacbb3a90

# if namespace isolation, the next two:
# openshift_kuryr_subnet_driver: namespace
# openshift_kuryr_sg_driver: namespace

- In addition I think this is not needed at the verification section anymore:
bootstrap-autoapprover-0            1/1       Running   0          3d    10.11.0.7       master-0.openshift.example.com.

- At [2] I have the next comments (though I see that is documentation without kuryr):
  - At octavia subsection (https://docs.openshift.com/container-platform/3.11/install_config/configuring_openstack.html#enabling-octavia-openstack-load-balancing-as-a-service-lbaas): we should include information about the template modification to increase the loadbalancer connection timeout, otherwise the installation may fail due to the next bug:
https://bugzilla.redhat.com/show_bug.cgi?id=1618685
  - At the cinder subsection (https://docs.openshift.com/container-platform/3.11/install_config/configuring_openstack.html#openstack-cinder-volumes): I thought recommendation was to use swift for the registry.
  - I'm missing the recommendation discussed at the hackfest of using VLAN networks to avoid double encapsulation when not using kuryr.

[1] https://docs.openshift.com/container-platform/3.11/install_config/configuring_kuryrsdn.html
[2] https://docs.openshift.com/container-platform/3.11/install_config/configuring_openstack.html

Comment 2 MichaƂ Dulko 2018-12-19 15:59:40 UTC
Page [1] could also use some clarifications:

   1. We should mention that Kuryr requires trunk ports and Open vSwitch
      firewall driver to be enabled in Neutron. This is already listed at
      [2], but not on general page on Kuryr's configuration.
   2. Example configuration lists `kuryr_openstack_username=admin`.
      Running with admin user shouldn't be recommended. In nested mode
      Kuryr is easily able to run on a regular tenant.
   3. We should add some recommendations on quotas values for that regular
      tenant mentioned above. I know we've started the discussion, I'm not
      sure if it was conclusive.

That's all the issues I've noticed.

[1] https://docs.openshift.com/container-platform/3.11/install_config/configuring_kuryrsdn.html
[2] https://docs.openshift.com/container-platform/3.11/architecture/networking/network_plugins.html#architecture-additional-concepts-kuryr-openstack

Comment 4 Aine Riordan 2019-01-16 22:44:09 UTC
@Jon Uriarte 

Removed references to tech preview.

Pull request https://github.com/openshift/openshift-docs/pull/13343

I will update this pull request to address comments 1 and 2.

Comment 8 Aine Riordan 2019-01-21 11:32:51 UTC
PR  https://github.com/openshift/openshift-docs/pull/13343  removes tech preview status only.
Subsequent comments will be handled in a separate PR.

Comment 13 Vikram Goyal 2019-04-02 05:33:48 UTC
This is now live. TP has been removed from 3.11