{"action":"created","issue":{"url":"https://api.github.com/repos/automationbroker/bundle-lib/issues/147","repository_url":"https://api.github.com/repos/automationbroker/bundle-lib","labels_url":"https://api.github.com/repos/automationbroker/bundle-lib/issues/147/labels{/name}","comments_url":"https://api.github.com/repos/automationbroker/bundle-lib/issues/147/comments","events_url":"https://api.github.com/repos/automationbroker/bundle-lib/issues/147/events","html_url":"https://github.com/automationbroker/bundle-lib/issues/147","id":343819524,"node_id":"MDU6SXNzdWUzNDM4MTk1MjQ=","number":147,"title":"Bundles filtered log is hard to read","user":{"login":"jmrodri","id":17137,"node_id":"MDQ6VXNlcjE3MTM3","avatar_url":"https://avatars2.githubusercontent.com/u/17137?v=4","gravatar_id":"","url":"https://api.github.com/users/jmrodri","html_url":"https://github.com/jmrodri","followers_url":"https://api.github.com/users/jmrodri/followers","following_url":"https://api.github.com/users/jmrodri/following{/other_user}","gists_url":"https://api.github.com/users/jmrodri/gists{/gist_id}","starred_url":"https://api.github.com/users/jmrodri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmrodri/subscriptions","organizations_url":"https://api.github.com/users/jmrodri/orgs","repos_url":"https://api.github.com/users/jmrodri/repos","events_url":"https://api.github.com/users/jmrodri/events{/privacy}","received_events_url":"https://api.github.com/users/jmrodri/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-07-23T22:32:48Z","updated_at":"2018-07-23T22:41:21Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"Feels like the log statement shouldn't be all on one line. it's hard to read:\r\n\r\n```\r\ntime=\"2018-07-23T21:27:57Z\" level=info msg=\"Bundles filtered by white/blacklist filter:-> ansibleplaybookbundle/apb-tools-> ansibleplaybookbundle/origin-ansible-service-broker-> ansibleplaybookbundle/hello-world-> ansibleplaybookbundle/mediawiki123-> ansibleplaybookb     undle/ansible-service-broker-> ansibleplaybookbundle/apb-base-> ansibleplaybookbundle/origin-> ansibleplaybookbundle/py-zip-demo-> ansibleplaybookbundle/asb-installer-> ansibleplaybookbundle/photo-album-demo-app-> ansibleplaybookbundle/helm-bundle-base-> ansibleplayb     ookbundle/deploy-broker-> ansibleplaybookbundle/photo-album-demo-api-> ansibleplaybookbundle/apb-assets-base-> ansibleplaybookbundle/manageiq-apb-runner-> ansibleplaybookbundle/vnc-client-> ansibleplaybookbundle/mediawiki-> ansibleplaybookbundle/origin-service-catalo     g-> ansibleplaybookbundle/origin-recycler-> ansibleplaybookbundle/vnc-desktop-> ansibleplaybookbundle/origin-deployer-> ansibleplaybookbundle/origin-docker-registry-> ansibleplaybookbundle/origin-haproxy-router-> ansibleplaybookbundle/origin-pod-> ansibleplaybookbund     le/origin-sti-builder-> ansibleplaybookbundle/kubevirt-ansible\"\r\n```\r\nThe `->` should be in front of the apbs, but it looks like it is after.\r\n\r\n```golang\r\n     if len(filteredNames) != 0 {\r\n         var buffer bytes.Buffer\r\n         buffer.WriteString(\"Bundles filtered by white/blacklist filter:\")\r\n         for _, name := range filteredNames {\r\n             buffer.WriteString(fmt.Sprintf(\"-> %s\", name))\r\n         }\r\n         log.Infof(buffer.String())\r\n     }\r\n```"},"comment":{"url":"https://api.github.com/repos/automationbroker/bundle-lib/issues/comments/407223898","html_url":"https://github.com/automationbroker/bundle-lib/issues/147#issuecomment-407223898","issue_url":"https://api.github.com/repos/automationbroker/bundle-lib/issues/147","id":407223898,"node_id":"MDEyOklzc3VlQ29tbWVudDQwNzIyMzg5OA==","user":{"login":"jmrodri","id":17137,"node_id":"MDQ6VXNlcjE3MTM3","avatar_url":"https://avatars2.githubusercontent.com/u/17137?v=4","gravatar_id":"","url":"https://api.github.com/users/jmrodri","html_url":"https://github.com/jmrodri","followers_url":"https://api.github.com/users/jmrodri/followers","following_url":"https://api.github.com/users/jmrodri/following{/other_user}","gists_url":"https://api.github.com/users/jmrodri/gists{/gist_id}","starred_url":"https://api.github.com/users/jmrodri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmrodri/subscriptions","organizations_url":"https://api.github.com/users/jmrodri/orgs","repos_url":"https://api.github.com/users/jmrodri/repos","events_url":"https://api.github.com/users/jmrodri/events{/privacy}","received_events_url":"https://api.github.com/users/jmrodri/received_events","type":"User","site_admin":false},"created_at":"2018-07-23T22:41:21Z","updated_at":"2018-07-23T22:41:21Z","author_association":"CONTRIBUTOR","body":"If we keep it on a single line, I suggest we replace `-> ` with maybe ` |` \r\n\r\n```\r\ntime=\"2018-07-23T21:27:57Z\" level=info msg=\"Bundles filtered by white/blacklist filter: |ansibleplaybookbundle/apb-tools |ansibleplaybookbundle/origin-ansible-service-broker |ansibleplaybookbundle/hello-world |ansibleplaybookbundle/mediawiki123 |ansibleplaybookbundle/ansible-service-broker |ansibleplaybookbundle/apb-base |ansibleplaybookbundle/origin |ansibleplaybookbundle/py-zip-demo |ansibleplaybookbundle/asb-installer |ansibleplaybookbundle/photo-album-demo-app |ansibleplaybookbundle/helm-bundle-base |ansibleplayb  ookbundle/deploy-broker |ansibleplaybookbundle/photo-album-demo-api |ansibleplaybookbundle/apb-assets-base |ansibleplaybookbundle/manageiq-apb-runner |ansibleplaybookbundle/vnc-client |ansibleplaybookbundle/mediawiki |ansibleplaybookbundle/origin-service-catalog |ansibleplaybookbundle/origin-recycler |ansibleplaybookbundle/vnc-desktop |ansibleplaybookbundle/origin-deployer |ansibleplaybookbundle/origin-docker-registry |ansibleplaybookbundle/origin-haproxy-router |ansibleplaybookbundle/origin-pod |ansibleplaybookbundle/origin-sti-builder |ansibleplaybookbundle/kubevirt-ansible\"\r\n```"}}
{"actor":{"display_login":"jmrodri"}}