Bug 1318976

Summary:	imagePolicyConfig should be in master-config by default
Product:	OpenShift Online	Reporter:	XiaochuanWang <xiaocwan>
Component:	Image Registry	Assignee:	Abhishek Gupta <abhgupta>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Wei Sun <wsun>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	3.x	CC:	abhgupta, aos-bugs, maszulik
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:08:25 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:

Description XiaochuanWang 2016-03-18 10:02:18 UTC

Description of problem:
imagePolicyConfig is required for how fast, how often, and how many images can be imported at a time. There is a requirement that allow imported images to be refreshed automatically from their upstreams:
As a user of OpenShift, I should be able to indicate that certain image stream tags should be periodically checked against their remote docker registries to determine whether a new image is available, and if so, import them. This import should leverage any image credentials I have available.

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

How reproducible:
Always

Steps to Reproduce:
Check if imagePolicyConfig is in master-config.

Actual results:
imagePolicyConfig is not in master-config

Expected results:
There should be imagePolicyConfig from master-config by default like:
imagePolicyConfig:
  disableScheduledImport: false
  maxImagesBulkImportedPerRepository: 5
  maxScheduledImageImportsPerMinute: 60
  scheduledImageImportMinimumIntervalSeconds: 900

Comment 1 Maciej Szulik 2016-03-21 12:38:27 UTC

This is already happening in origin. Tested with latest master https://github.com/openshift/origin/commit/2ddfc71bf4f24d35afbf4085b1e26ae64bffa107 and it works ok.

Comment 2 Abhishek Gupta 2016-03-29 00:30:55 UTC

This will likely need a release ticket to specify the desired configuration - I will get to this tomorrow.

Comment 3 Maciej Szulik 2016-03-30 09:34:54 UTC

I'm re-assigning this to Abishek since it's Online configuration right now and not a origin bug per se.

Comment 5 XiaochuanWang 2016-04-06 06:49:04 UTC

This is verified on origin. For On-line, there are two config as below:
imagePolicyConfig:
  disableScheduledImport: true
  maxImagesBulkImportedPerRepository: 3

According to PR the other 2 ones are dropped:
  maxScheduledImageImportsPerMinute: 1
  scheduledImageImportMinimumIntervalSeconds: 20

Seems it's by designed for online, so accept it as verified.