{"action":"created","comment":{"url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls/comments/181391119","pull_request_review_id":112014079,"id":181391119,"diff_hunk":"@@ -0,0 +1,56 @@\n+\"\"\"\n+Copyright (c) 2018 Red Hat, Inc\n+All rights reserved.\n+\n+This software may be modified and distributed under the terms\n+of the BSD license. See the LICENSE file for details.\n+\"\"\"\n+from copy import deepcopy\n+import re\n+\n+\n+# Negative regex used to exclude characters that are not allowed\n+# in naming a kubernetes resource\n+INVALID_KUBERNETES_NAME_CHARS = re.compile(r'[^a-z0-9\\.-]+')\n+\n+\n+class FilterModule(object):\n+    def filters(self):\n+        return {'with_isolated_workers': do_with_isolated_workers}\n+\n+\n+def do_with_isolated_workers(reactor_configs):\n+    \"\"\"Generate reactor configs for each worker cluster\n+\n+    :param reactor_configs: list<dict>, each dict should contain a name and\n+    a data key. The value of name key is used to name the config map object\n+    and the value of data key is a reactor config\n+\n+    :return: a new list of reactor configs that contains new reactor configs\n+    for each worker cluster in addition to the original reactor configs\n+    \"\"\"\n+    all_configs = list(reactor_configs)\n+\n+    for config in reactor_configs:\n+        clusters = config.get('data', {}).get('clusters', {})\n+        for arch, workers_info in clusters.items():\n+            for worker_info in workers_info:\n+                worker_info = deepcopy(worker_info)\n+                worker_info['enabled'] = True","path":"filter_plugins/with_isolated_workers.py","position":39,"original_position":39,"commit_id":"0e3d4201000c89add8ec8ee347c5e070a3ba4060","original_commit_id":"0e3d4201000c89add8ec8ee347c5e070a3ba4060","user":{"login":"rcerven","id":23337371,"avatar_url":"https://avatars0.githubusercontent.com/u/23337371?v=4","gravatar_id":"","url":"https://api.github.com/users/rcerven","html_url":"https://github.com/rcerven","followers_url":"https://api.github.com/users/rcerven/followers","following_url":"https://api.github.com/users/rcerven/following{/other_user}","gists_url":"https://api.github.com/users/rcerven/gists{/gist_id}","starred_url":"https://api.github.com/users/rcerven/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rcerven/subscriptions","organizations_url":"https://api.github.com/users/rcerven/orgs","repos_url":"https://api.github.com/users/rcerven/repos","events_url":"https://api.github.com/users/rcerven/events{/privacy}","received_events_url":"https://api.github.com/users/rcerven/received_events","type":"User","site_admin":false},"body":"why do we overwrite 'enabled' value?","created_at":"2018-04-13T13:40:55Z","updated_at":"2018-04-13T13:40:55Z","html_url":"https://github.com/projectatomic/ansible-role-osbs-namespace/pull/32#discussion_r181391119","pull_request_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls/32","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls/comments/181391119"},"html":{"href":"https://github.com/projectatomic/ansible-role-osbs-namespace/pull/32#discussion_r181391119"},"pull_request":{"href":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls/32"}}},"pull_request":{"url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls/32","id":181476806,"html_url":"https://github.com/projectatomic/ansible-role-osbs-namespace/pull/32","diff_url":"https://github.com/projectatomic/ansible-role-osbs-namespace/pull/32.diff","patch_url":"https://github.com/projectatomic/ansible-role-osbs-namespace/pull/32.patch","issue_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/issues/32","number":32,"state":"open","locked":false,"title":"Add support for generating reactor-config-map","user":{"login":"lcarva","id":5272931,"avatar_url":"https://avatars1.githubusercontent.com/u/5272931?v=4","gravatar_id":"","url":"https://api.github.com/users/lcarva","html_url":"https://github.com/lcarva","followers_url":"https://api.github.com/users/lcarva/followers","following_url":"https://api.github.com/users/lcarva/following{/other_user}","gists_url":"https://api.github.com/users/lcarva/gists{/gist_id}","starred_url":"https://api.github.com/users/lcarva/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lcarva/subscriptions","organizations_url":"https://api.github.com/users/lcarva/orgs","repos_url":"https://api.github.com/users/lcarva/repos","events_url":"https://api.github.com/users/lcarva/events{/privacy}","received_events_url":"https://api.github.com/users/lcarva/received_events","type":"User","site_admin":false},"body":"Signed-off-by: Luiz Carvalho <lucarval@redhat.com>","created_at":"2018-04-13T12:42:34Z","updated_at":"2018-04-13T13:40:55Z","closed_at":null,"merged_at":null,"merge_commit_sha":"8d6f7105169e24126a6f16417d32c9595c01c27b","assignee":null,"assignees":[],"requested_reviewers":[{"login":"sosiouxme","id":29690,"avatar_url":"https://avatars1.githubusercontent.com/u/29690?v=4","gravatar_id":"","url":"https://api.github.com/users/sosiouxme","html_url":"https://github.com/sosiouxme","followers_url":"https://api.github.com/users/sosiouxme/followers","following_url":"https://api.github.com/users/sosiouxme/following{/other_user}","gists_url":"https://api.github.com/users/sosiouxme/gists{/gist_id}","starred_url":"https://api.github.com/users/sosiouxme/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sosiouxme/subscriptions","organizations_url":"https://api.github.com/users/sosiouxme/orgs","repos_url":"https://api.github.com/users/sosiouxme/repos","events_url":"https://api.github.com/users/sosiouxme/events{/privacy}","received_events_url":"https://api.github.com/users/sosiouxme/received_events","type":"User","site_admin":false},{"login":"twaugh","id":6985468,"avatar_url":"https://avatars1.githubusercontent.com/u/6985468?v=4","gravatar_id":"","url":"https://api.github.com/users/twaugh","html_url":"https://github.com/twaugh","followers_url":"https://api.github.com/users/twaugh/followers","following_url":"https://api.github.com/users/twaugh/following{/other_user}","gists_url":"https://api.github.com/users/twaugh/gists{/gist_id}","starred_url":"https://api.github.com/users/twaugh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/twaugh/subscriptions","organizations_url":"https://api.github.com/users/twaugh/orgs","repos_url":"https://api.github.com/users/twaugh/repos","events_url":"https://api.github.com/users/twaugh/events{/privacy}","received_events_url":"https://api.github.com/users/twaugh/received_events","type":"User","site_admin":false},{"login":"bfontecc007","id":29286673,"avatar_url":"https://avatars2.githubusercontent.com/u/29286673?v=4","gravatar_id":"","url":"https://api.github.com/users/bfontecc007","html_url":"https://github.com/bfontecc007","followers_url":"https://api.github.com/users/bfontecc007/followers","following_url":"https://api.github.com/users/bfontecc007/following{/other_user}","gists_url":"https://api.github.com/users/bfontecc007/gists{/gist_id}","starred_url":"https://api.github.com/users/bfontecc007/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bfontecc007/subscriptions","organizations_url":"https://api.github.com/users/bfontecc007/orgs","repos_url":"https://api.github.com/users/bfontecc007/repos","events_url":"https://api.github.com/users/bfontecc007/events{/privacy}","received_events_url":"https://api.github.com/users/bfontecc007/received_events","type":"User","site_admin":false}],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls/32/commits","review_comments_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls/32/comments","review_comment_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls/comments{/number}","comments_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/issues/32/comments","statuses_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/statuses/0e3d4201000c89add8ec8ee347c5e070a3ba4060","head":{"label":"projectatomic:reactor-config-map","ref":"reactor-config-map","sha":"0e3d4201000c89add8ec8ee347c5e070a3ba4060","user":{"login":"projectatomic","id":6852258,"avatar_url":"https://avatars1.githubusercontent.com/u/6852258?v=4","gravatar_id":"","url":"https://api.github.com/users/projectatomic","html_url":"https://github.com/projectatomic","followers_url":"https://api.github.com/users/projectatomic/followers","following_url":"https://api.github.com/users/projectatomic/following{/other_user}","gists_url":"https://api.github.com/users/projectatomic/gists{/gist_id}","starred_url":"https://api.github.com/users/projectatomic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/projectatomic/subscriptions","organizations_url":"https://api.github.com/users/projectatomic/orgs","repos_url":"https://api.github.com/users/projectatomic/repos","events_url":"https://api.github.com/users/projectatomic/events{/privacy}","received_events_url":"https://api.github.com/users/projectatomic/received_events","type":"Organization","site_admin":false},"repo":{"id":105527357,"name":"ansible-role-osbs-namespace","full_name":"projectatomic/ansible-role-osbs-namespace","owner":{"login":"projectatomic","id":6852258,"avatar_url":"https://avatars1.githubusercontent.com/u/6852258?v=4","gravatar_id":"","url":"https://api.github.com/users/projectatomic","html_url":"https://github.com/projectatomic","followers_url":"https://api.github.com/users/projectatomic/followers","following_url":"https://api.github.com/users/projectatomic/following{/other_user}","gists_url":"https://api.github.com/users/projectatomic/gists{/gist_id}","starred_url":"https://api.github.com/users/projectatomic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/projectatomic/subscriptions","organizations_url":"https://api.github.com/users/projectatomic/orgs","repos_url":"https://api.github.com/users/projectatomic/repos","events_url":"https://api.github.com/users/projectatomic/events{/privacy}","received_events_url":"https://api.github.com/users/projectatomic/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/projectatomic/ansible-role-osbs-namespace","description":null,"fork":false,"url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace","forks_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/forks","keys_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/keys{/key_id}","collaborators_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/teams","hooks_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/hooks","issue_events_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/issues/events{/number}","events_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/events","assignees_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/assignees{/user}","branches_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/branches{/branch}","tags_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/tags","blobs_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/git/refs{/sha}","trees_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/git/trees{/sha}","statuses_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/statuses/{sha}","languages_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/languages","stargazers_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/stargazers","contributors_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/contributors","subscribers_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/subscribers","subscription_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/subscription","commits_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/commits{/sha}","git_commits_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/git/commits{/sha}","comments_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/comments{/number}","issue_comment_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/issues/comments{/number}","contents_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/contents/{+path}","compare_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/compare/{base}...{head}","merges_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/merges","archive_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/downloads","issues_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/issues{/number}","pulls_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls{/number}","milestones_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/milestones{/number}","notifications_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/labels{/name}","releases_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/releases{/id}","deployments_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/deployments","created_at":"2017-10-02T11:34:03Z","updated_at":"2018-03-26T17:12:24Z","pushed_at":"2018-04-13T12:42:35Z","git_url":"git://github.com/projectatomic/ansible-role-osbs-namespace.git","ssh_url":"git@github.com:projectatomic/ansible-role-osbs-namespace.git","clone_url":"https://github.com/projectatomic/ansible-role-osbs-namespace.git","svn_url":"https://github.com/projectatomic/ansible-role-osbs-namespace","homepage":null,"size":57,"stargazers_count":2,"watchers_count":2,"language":"Shell","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":6,"mirror_url":null,"archived":false,"open_issues_count":1,"license":null,"forks":6,"open_issues":1,"watchers":2,"default_branch":"master"}},"base":{"label":"projectatomic:master","ref":"master","sha":"510207098b2960f9a44f5887df7c76425c0af3a7","user":{"login":"projectatomic","id":6852258,"avatar_url":"https://avatars1.githubusercontent.com/u/6852258?v=4","gravatar_id":"","url":"https://api.github.com/users/projectatomic","html_url":"https://github.com/projectatomic","followers_url":"https://api.github.com/users/projectatomic/followers","following_url":"https://api.github.com/users/projectatomic/following{/other_user}","gists_url":"https://api.github.com/users/projectatomic/gists{/gist_id}","starred_url":"https://api.github.com/users/projectatomic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/projectatomic/subscriptions","organizations_url":"https://api.github.com/users/projectatomic/orgs","repos_url":"https://api.github.com/users/projectatomic/repos","events_url":"https://api.github.com/users/projectatomic/events{/privacy}","received_events_url":"https://api.github.com/users/projectatomic/received_events","type":"Organization","site_admin":false},"repo":{"id":105527357,"name":"ansible-role-osbs-namespace","full_name":"projectatomic/ansible-role-osbs-namespace","owner":{"login":"projectatomic","id":6852258,"avatar_url":"https://avatars1.githubusercontent.com/u/6852258?v=4","gravatar_id":"","url":"https://api.github.com/users/projectatomic","html_url":"https://github.com/projectatomic","followers_url":"https://api.github.com/users/projectatomic/followers","following_url":"https://api.github.com/users/projectatomic/following{/other_user}","gists_url":"https://api.github.com/users/projectatomic/gists{/gist_id}","starred_url":"https://api.github.com/users/projectatomic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/projectatomic/subscriptions","organizations_url":"https://api.github.com/users/projectatomic/orgs","repos_url":"https://api.github.com/users/projectatomic/repos","events_url":"https://api.github.com/users/projectatomic/events{/privacy}","received_events_url":"https://api.github.com/users/projectatomic/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/projectatomic/ansible-role-osbs-namespace","description":null,"fork":false,"url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace","forks_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/forks","keys_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/keys{/key_id}","collaborators_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/teams","hooks_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/hooks","issue_events_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/issues/events{/number}","events_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/events","assignees_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/assignees{/user}","branches_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/branches{/branch}","tags_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/tags","blobs_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/git/refs{/sha}","trees_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/git/trees{/sha}","statuses_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/statuses/{sha}","languages_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/languages","stargazers_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/stargazers","contributors_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/contributors","subscribers_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/subscribers","subscription_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/subscription","commits_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/commits{/sha}","git_commits_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/git/commits{/sha}","comments_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/comments{/number}","issue_comment_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/issues/comments{/number}","contents_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/contents/{+path}","compare_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/compare/{base}...{head}","merges_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/merges","archive_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/downloads","issues_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/issues{/number}","pulls_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls{/number}","milestones_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/milestones{/number}","notifications_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/labels{/name}","releases_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/releases{/id}","deployments_url":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/deployments","created_at":"2017-10-02T11:34:03Z","updated_at":"2018-03-26T17:12:24Z","pushed_at":"2018-04-13T12:42:35Z","git_url":"git://github.com/projectatomic/ansible-role-osbs-namespace.git","ssh_url":"git@github.com:projectatomic/ansible-role-osbs-namespace.git","clone_url":"https://github.com/projectatomic/ansible-role-osbs-namespace.git","svn_url":"https://github.com/projectatomic/ansible-role-osbs-namespace","homepage":null,"size":57,"stargazers_count":2,"watchers_count":2,"language":"Shell","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":6,"mirror_url":null,"archived":false,"open_issues_count":1,"license":null,"forks":6,"open_issues":1,"watchers":2,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls/32"},"html":{"href":"https://github.com/projectatomic/ansible-role-osbs-namespace/pull/32"},"issue":{"href":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/issues/32"},"comments":{"href":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/issues/32/comments"},"review_comments":{"href":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls/32/comments"},"review_comment":{"href":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/pulls/32/commits"},"statuses":{"href":"https://api.github.com/repos/projectatomic/ansible-role-osbs-namespace/statuses/0e3d4201000c89add8ec8ee347c5e070a3ba4060"}},"author_association":"CONTRIBUTOR"}}
{"actor":{"display_login":"rcerven"}}