{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/openshift-ansible/issues/4822","repository_url":"https://api.github.com/repos/openshift/openshift-ansible","labels_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/4822/labels{/name}","comments_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/4822/comments","events_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/4822/events","html_url":"https://github.com/openshift/openshift-ansible/issues/4822","id":244669212,"number":4822,"title":"CFME Role is changing the maxImagesBulkImportedPerRepository value","user":{"login":"simon3z","id":130393,"avatar_url":"https://avatars1.githubusercontent.com/u/130393?v=4","gravatar_id":"","url":"https://api.github.com/users/simon3z","html_url":"https://github.com/simon3z","followers_url":"https://api.github.com/users/simon3z/followers","following_url":"https://api.github.com/users/simon3z/following{/other_user}","gists_url":"https://api.github.com/users/simon3z/gists{/gist_id}","starred_url":"https://api.github.com/users/simon3z/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/simon3z/subscriptions","organizations_url":"https://api.github.com/users/simon3z/orgs","repos_url":"https://api.github.com/users/simon3z/repos","events_url":"https://api.github.com/users/simon3z/events{/privacy}","received_events_url":"https://api.github.com/users/simon3z/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2017-07-21T13:29:49Z","updated_at":"2017-09-18T13:22:03Z","closed_at":"2017-09-14T13:54:46Z","author_association":"CONTRIBUTOR","body":"#### Description\r\n\r\nWhen installing CFME the `maxImagesBulkImportedPerRepository` configuration parameter is changed to `100`.\r\n\r\nThere is a configuration parameter `openshift_cfme_maxImagesBulkImportedPerRepository` which is a little bit misleading because it makes you think that it's limited to CFME only (instead it is applied cluster-wide).\r\n\r\nI understand the images are not following the tagging best-practices (fixed in 4.6 cc @carbonin) but I feel that installing an infrastructure application shouldn't silently change such a critical cluster-wide settings.\r\n\r\nI have the feeling that suggesting to set `openshift_cfme_maxImagesBulkImportedPerRepository` = `openshift_master_image_policy_config` won't be a full solution (also because of course it will most likely not work with the dockerhub repository).\r\n\r\n@gtanzillo maybe we can try to go with the correct solution and change the tagging strategy on dockerhub (maintaining the old one as well for backward-compatibility).\r\n\r\ncc @sdodson @tbielawa @fbladilo @gtanzillo @bazulay\r\n\r\n@joelddiaz this is particularly important for you because you probably don't want to find your cluster with `maxImagesBulkImportedPerRepository` magically configured to `100`.\r\n\r\n##### Version\r\n\r\n```\r\nansible 2.2.2.0\r\n  config file = /home/simon/src/openshift-ansible/ansible.cfg\r\n  configured module search path = Default w/o overrides\r\n```\r\n```\r\nopenshift-ansible-3.6.123-1\r\n```\r\n\r\n##### Steps To Reproduce\r\n1. Deploy CFME using the relevant role\r\n\r\n\r\n##### Expected Results\r\nThe `maxImagesBulkImportedPerRepository` value shouldn't be changed.\r\n\r\n##### Observed Results\r\nThe `maxImagesBulkImportedPerRepository` value is changed to `100`.\r\n"},"comment":{"url":"https://api.github.com/repos/openshift/openshift-ansible/issues/comments/330218413","html_url":"https://github.com/openshift/openshift-ansible/issues/4822#issuecomment-330218413","issue_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/4822","id":330218413,"user":{"login":"sdodson","id":48161,"avatar_url":"https://avatars2.githubusercontent.com/u/48161?v=4","gravatar_id":"","url":"https://api.github.com/users/sdodson","html_url":"https://github.com/sdodson","followers_url":"https://api.github.com/users/sdodson/followers","following_url":"https://api.github.com/users/sdodson/following{/other_user}","gists_url":"https://api.github.com/users/sdodson/gists{/gist_id}","starred_url":"https://api.github.com/users/sdodson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sdodson/subscriptions","organizations_url":"https://api.github.com/users/sdodson/orgs","repos_url":"https://api.github.com/users/sdodson/repos","events_url":"https://api.github.com/users/sdodson/events{/privacy}","received_events_url":"https://api.github.com/users/sdodson/received_events","type":"User","site_admin":false},"created_at":"2017-09-18T13:22:03Z","updated_at":"2017-09-18T13:22:03Z","author_association":"MEMBER","body":"@tbielawa why'd you say this wouldn't be a problem in 4.6? If this is being closed by a future PR lets leave it open until that PR is merged."}}
{"actor":{"display_login":"sdodson"}}