Bug 1318718

Summary: [online_private_546] ClusterResourceOverride is not configured in master-config.yaml
Product: OpenShift Online Reporter: Qixuan Wang <qixuan.wang>
Component: PodAssignee: Luke Meyer <lmeyer>
Status: CLOSED CURRENTRELEASE QA Contact: Qixuan Wang <qixuan.wang>
Severity: medium Docs Contact:
Priority: high    
Version: 3.xCC: abhgupta, agrimm, aos-bugs, dakini, jokerman, mmccomas, qixuan.wang, twiest, xtian
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-05-23 15:09:03 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: 1303130    

Description Qixuan Wang 2016-03-17 15:17:02 UTC
Description of problem:
We need to add ClusterResourceOverride plugin on openshift dev-preview-int  master to override container resource request based on limit.

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

How reproducible:
Always

Steps to Reproduce:
1. Check master-config.yaml


Actual results:
ClusterResourceOverride plugin isn't configured so block container override related testing.

Expected results:
Please add the following to the master-config.yaml and then restart master service:

kubernetesMasterConfig:
  admissionConfig:
    pluginConfig:
      ClusterResourceOverride:
        configuration:
          apiVersion: v1
          kind: ClusterResourceOverrideConfig
          limitCPUToMemoryPercent: 100
          cpuRequestToLimitPercent: 25
          memoryRequestToLimitPercent: 50


Additional info:

Comment 1 Luke Meyer 2016-03-18 14:52:28 UTC
This should be already done in INT according to https://github.com/openshift/online/issues/18#issuecomment-196016797 - are we talking about the same system? The numbers are a bit different there too.

Comment 2 Stefanie Forrester 2016-03-18 16:47:13 UTC
The numbers were changed in the github issue after the initial request. Updated numbers are here:

          limitCPUToMemoryPercent: 200
          cpuRequestToLimitPercent: 6
          memoryRequestToLimitPercent: 60

Comment 3 Luke Meyer 2016-03-18 17:37:19 UTC
(In reply to Qixuan Wang from comment #0)
> We need to add ClusterResourceOverride plugin on openshift dev-preview-int 

Can you confirm exactly which cluster you're expecting this to be configured in and not seeing it?

Comment 4 Qixuan Wang 2016-03-21 07:31:53 UTC
I meant openshift dev-preview-int cluster. ClusterResourceOverride plugin was added quickly but something wrong, see https://bugzilla.redhat.com/show_bug.cgi?id=1318983

Comment 5 Abhishek Gupta 2016-03-29 00:38:39 UTC
Can you please check if this bug has been resolved with the latest INT rebuild?

Comment 6 Qixuan Wang 2016-03-29 03:25:14 UTC
Now the bug has been fixed in the latest INT with the following ratio:

limitCPUToMemoryPercent: 200
cpuRequestToLimitPercent: 6
memoryRequestToLimitPercent: 60

Package versions:
oc v3.2.0.6
kubernetes v1.2.0-36-g4a3f9c5
Docker version 1.9.1, build 78ee77d/1.9.1
kernel 3.10.0-327.10.1.el7.x86_64