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

Bug 1053897

Summary: [Doc] Foreman controller params should end with _host instead of _ip
Product: Red Hat OpenStack Reporter: Summer Long <slong>
Component: doc-Installation_and_Configuration_GuideAssignee: Tahlia Richardson <trichard>
Status: CLOSED CURRENTRELEASE QA Contact: Andrew Dahms <adahms>
Severity: high Docs Contact:
Priority: high    
Version: 4.0CC: cwolfe, jguiditt, morazi, rhos-maint, trichard, yeylon, zdover
Target Milestone: asyncKeywords: Documentation, Triaged
Target Release: 5.0 (RHEL 7)   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1053729 Environment:
Last Closed: 2015-02-13 06:01:40 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: 1053729    
Bug Blocks:    

Comment 2 Summer Long 2014-01-20 05:07:29 UTC
Scott, you'll need a tech review for this one first. Probably Jason Guiditta...

Comment 4 Jason Guiditta 2014-01-21 22:46:45 UTC
Scott, actually, those environment variables have been removed now, so all you need are the provisioning and gateway variables for running the setup script. This is not described directly in the BZ this one depends on:
https://bugzilla.redhat.com/show_bug.cgi?id=1053729

Rather it is in the one you see in Bruce's comments there:
https://bugzilla.redhat.com/show_bug.cgi?id=1049688#c4

Let me know if you need any further detail or explanation on this.

Comment 6 Jason Guiditta 2014-06-09 14:20:35 UTC
Sorry for the super delayed response, just got a reminder from bugzilla on this.  So, one thing I would like to note here.  It is not necessary to edit foreman_server.sh directly (and in fact increases the chance something will get messed up by the user accidentally).  Instead, it is better to set the two referenced variables as environment variables.  This can either be done inline like this:
# FOREMAN_GATEWAY='10.0.0.1' FOREMAN_PROVISIONING='true' bash -x foreman_server.sh

or, the above variables could be set in an env file and exported before use.

Comment 7 Zac Dover 2014-09-25 12:37:46 UTC
State of Play
-------------
The changes requested, namely that the referenced variables FOREMAN_GATEWAY and FOREMAN PROVISIONING should be set, are included in the document here:

http://documentation-devel.engineering.redhat.com/site/documentation/en-US/Red_Hat_Enterprise_Linux_OpenStack_Platform/4/html/Installation_and_Configuration_Guide/Configuring_the_Installer.html

Step 2 of this procedure reads:
  
  Open the /usr/share/openstack-foreman-installer/bin/foreman_server.sh 
  file in a text editor. 

Significant rewriting would be necessary if we were to follow Jason's suggestion in Comment 6 that we forego using a text editor to open the /usr/share/openstack-foreman-installer/bin/foreman_server.sh file and instead set the referenced variables by using environment variables. Jason's case for using exported environment variables is persuasive, but this will have to wait for inclusion later.

This bug displays significant mission creep. The original bug requested only that the Foreman controller parameters be changed from outdated parameters ending in "_ip" to newer, not-outdated controller parameters ending in "_host". That mission was complete as of Comment 4 or 5.

Comment 8 Zac Dover 2014-09-25 14:12:31 UTC
Remark
------
Well, I spoke too soon. This bug has been entirely superseded by this bug:

https://bugzilla.redhat.com/show_bug.cgi?id=1107833

This is one of those two-birds-with-one-stone situations.

I will update both of these bugs when I have made the necessary alterations.

Comment 9 Zac Dover 2014-09-25 14:57:29 UTC
Fixed in topic 19224:713013.
This topic and revision were added to cs15807.

Comment 10 Zac Dover 2014-09-25 15:07:21 UTC
Documentation Link
------------------
https://access.redhat.com/site/documentation/en-US/Red_Hat_Enterprise_Linux_OpenStack_Platform/4/html/Installation_and_Configuration_Guide/Configuring_the_Installer.html

What Changed
------------
The "Configuring the Foreman Installer" has been changed to read:

  Procedure . Configuring the Foreman Installer

  Log in to the system on which Foreman will be installed.
			
  1. Export the following variable, replacing address with either the IP address 
     or fully qualified domain name of the system on which Foreman will be 
     installed:
       # export FOREMAN_GATEWAY=address
  2. Export the following variable:
       # export FOREMAN_PROVISIONING=address
  3. The Foreman installer is now configured and ready for use.

NVR
---
Red_Hat_Enterprise_Linux_OpenStack_Platform-Installation_and_Configuration_Guide-4-en-US-4-79

Comment 11 Tahlia Richardson 2014-10-03 05:24:52 UTC
Content has changed slightly (from another bug). Now reads: 

Procedure 3.2. Configuring the Foreman Installer

    Log in to the system on which Foreman will be installed.

   1. Export the following variable, replacing address with either the IP address or fully qualified domain name of the system on which Foreman will be installed:

    # export FOREMAN_GATEWAY=address

   2. Export the following variable, replacing eth_name with the name of the network interface that Foreman will use to define the provisioning network:

    # export PROVISIONING_INTERFACE=eth_name

The Foreman installer is now configured and ready for use.

Comment 12 Andrew Dahms 2014-11-07 04:16:05 UTC
Checked in Red_Hat_Enterprise_Linux_OpenStack_Platform-Installation_and_Configuration_Guide-4-en-US-4-88.

Moving to VERIFIED.