Bug 1026659 - max_domains_per_user should be removed from broker rails config file
Summary: max_domains_per_user should be removed from broker rails config file
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Online
Classification: Red Hat
Component: Pod
Version: 2.x
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: ---
: ---
Assignee: Jordan Liggitt
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-11-05 07:55 UTC by Johnny Liu
Modified: 2015-05-15 00:22 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-01-24 03:28:52 UTC
Target Upstream Version:

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Johnny Liu 2013-11-05 07:55:15 UTC 
Description of problem:
Check source code, looks like "max_domains_per_user" is already replaced by "default_max_domains".

Review /var/www/openshift/broker/config/environments/{production,development,test}.rb, found "max_domains_per_user" still exists in config file. It should be removed.

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

How reproducible:
Always

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:
Comment 1 Jordan Liggitt 2013-11-05 15:33:01 UTC 
Will merge in https://github.com/openshift/li/pull/2095 and https://github.com/openshift/origin-server/pull/4089
Comment 2 Jianwei Hou 2013-11-06 02:11:21 UTC 
Verified on devenv_3992, max_domains_per_user is removed.
Note You need to log in before you can comment on or make changes to this bug.