Bug 797963 - Need to resolve differences between new 7.1.0 standalone.xml and migrated standalone.xml
Summary: Need to resolve differences between new 7.1.0 standalone.xml and migrated sta...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OKD
Classification: Red Hat
Component: Containers
Version: 2.x
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: ---
Assignee: Bill DeCoste
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-27 16:21 UTC by Dan McPherson
Modified: 2015-05-14 22:51 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-03-02 21:56:25 UTC
Target Upstream Version:


Attachments (Terms of Use)
7.0.2_standalone.xml (10.16 KB, text/xml)
2012-02-27 16:22 UTC, Dan McPherson
no flags Details
7.1.0 standalone.xml (14.15 KB, text/xml)
2012-02-27 16:22 UTC, Dan McPherson
no flags Details
migrated standalone.xml (16.31 KB, text/xml)
2012-02-27 16:23 UTC, Dan McPherson
no flags Details

Description Dan McPherson 2012-02-27 16:21:41 UTC
Description of problem:
Opening this bug so we don't forget to make any necessary changes.

Known issues:
- Remoting
- Was the copyright removed on purpose?
- Are we going to turn off console?
- Are we going to use env. consistently and migrate all the existing apps so we can change our code?



Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Dan McPherson 2012-02-27 16:22:15 UTC
Created attachment 566082 [details]
7.0.2_standalone.xml

Comment 2 Dan McPherson 2012-02-27 16:22:41 UTC
Created attachment 566083 [details]
7.1.0 standalone.xml

Comment 3 Dan McPherson 2012-02-27 16:23:03 UTC
Created attachment 566084 [details]
migrated standalone.xml

Comment 4 Dan McPherson 2012-02-28 04:21:23 UTC
From Scott:

1. Yes, copyright was removed as it is now located in the jboss-as-7.1.0.Final/LICENSE.txt file. I guess this is a general move we are making with projects.
2. We can remove the management-interfaces/http-interface section to not expose the web version of the management console, even though this is not usable by default because there are no admin users defined.
3. Yes, we should use the env. prefix consistently. We should be migrating or simply have users to that.

Comment 5 Bill DeCoste 2012-02-28 21:30:26 UTC
Disabled remoting, management, and console logging. Also updated the xslt.

Comment 6 Dan McPherson 2012-02-28 21:44:17 UTC
What about the env vars?

Comment 7 Bill DeCoste 2012-02-28 22:04:44 UTC
Just updated ${OPENSHIFT...} to ${env.OPENSHIFT...}

Comment 8 Dan McPherson 2012-02-29 00:26:53 UTC
I'm still not seeing env. in the migrated ones.

Comment 9 Xiaoli Tian 2012-02-29 10:36:40 UTC
Yeah, I also not seeing env. after migration. the following is copied from the merged one after migration:


      <loopback-address value="${OPENSHIFT_INTERNAL_IP}"/>
    </interface>
    <interface name="public">
      <loopback-address value="${OPENSHIFT_INTERNAL_IP}"/>

Comment 10 Bill DeCoste 2012-02-29 13:07:20 UTC
Dan and I fixed this last night - should be in the next build.

Comment 11 Xiaoli Tian 2012-03-01 07:06:05 UTC
Have tested this on devenv-stage_139 , environment variables have been changed to env.OPENSHIFT... like:
      <user-name>${env.OPENSHIFT_DB_USERNAME}</user-name>
      <password>${env.OPENSHIFT_DB_PASSWORD}</password>


Note You need to log in before you can comment on or make changes to this bug.