{"action":"created","issue":{"url":"https://api.github.com/repos/mesosphere/dcos-commons/issues/1626","repository_url":"https://api.github.com/repos/mesosphere/dcos-commons","labels_url":"https://api.github.com/repos/mesosphere/dcos-commons/issues/1626/labels{/name}","comments_url":"https://api.github.com/repos/mesosphere/dcos-commons/issues/1626/comments","events_url":"https://api.github.com/repos/mesosphere/dcos-commons/issues/1626/events","html_url":"https://github.com/mesosphere/dcos-commons/pull/1626","id":258000147,"number":1626,"title":"[KAF-368] Make Kafka integration tests compatible with Confluent","user":{"login":"elezar","id":931393,"avatar_url":"https://avatars3.githubusercontent.com/u/931393?v=4","gravatar_id":"","url":"https://api.github.com/users/elezar","html_url":"https://github.com/elezar","followers_url":"https://api.github.com/users/elezar/followers","following_url":"https://api.github.com/users/elezar/following{/other_user}","gists_url":"https://api.github.com/users/elezar/gists{/gist_id}","starred_url":"https://api.github.com/users/elezar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/elezar/subscriptions","organizations_url":"https://api.github.com/users/elezar/orgs","repos_url":"https://api.github.com/users/elezar/repos","events_url":"https://api.github.com/users/elezar/events{/privacy}","received_events_url":"https://api.github.com/users/elezar/received_events","type":"User","site_admin":false},"labels":[{"id":649058275,"url":"https://api.github.com/repos/mesosphere/dcos-commons/labels/ready%20for%20review","name":"ready for review","color":"1d76db","default":false}],"state":"open","locked":false,"assignee":{"login":"elezar","id":931393,"avatar_url":"https://avatars3.githubusercontent.com/u/931393?v=4","gravatar_id":"","url":"https://api.github.com/users/elezar","html_url":"https://github.com/elezar","followers_url":"https://api.github.com/users/elezar/followers","following_url":"https://api.github.com/users/elezar/following{/other_user}","gists_url":"https://api.github.com/users/elezar/gists{/gist_id}","starred_url":"https://api.github.com/users/elezar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/elezar/subscriptions","organizations_url":"https://api.github.com/users/elezar/orgs","repos_url":"https://api.github.com/users/elezar/repos","events_url":"https://api.github.com/users/elezar/events{/privacy}","received_events_url":"https://api.github.com/users/elezar/received_events","type":"User","site_admin":false},"assignees":[{"login":"elezar","id":931393,"avatar_url":"https://avatars3.githubusercontent.com/u/931393?v=4","gravatar_id":"","url":"https://api.github.com/users/elezar","html_url":"https://github.com/elezar","followers_url":"https://api.github.com/users/elezar/followers","following_url":"https://api.github.com/users/elezar/following{/other_user}","gists_url":"https://api.github.com/users/elezar/gists{/gist_id}","starred_url":"https://api.github.com/users/elezar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/elezar/subscriptions","organizations_url":"https://api.github.com/users/elezar/orgs","repos_url":"https://api.github.com/users/elezar/repos","events_url":"https://api.github.com/users/elezar/events{/privacy}","received_events_url":"https://api.github.com/users/elezar/received_events","type":"User","site_admin":false}],"milestone":null,"comments":2,"created_at":"2017-09-15T10:47:20Z","updated_at":"2017-09-21T07:36:37Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/mesosphere/dcos-commons/pulls/1626","html_url":"https://github.com/mesosphere/dcos-commons/pull/1626","diff_url":"https://github.com/mesosphere/dcos-commons/pull/1626.diff","patch_url":"https://github.com/mesosphere/dcos-commons/pull/1626.patch"},"body":"This PR makes minor changes to the Kafka integration tests to make it simpler to copy these to Confluent (see mesosphere/confluent#59)\r\n\r\nThe biggest change was around checking for created topics, where I added a function to filter out topics starting with `_` so as to gracefully handle the `_confluent-metrics` topic.\r\n\r\nThis also contains changes to `test.sh` and `test-runner.sh` to allow these to be used from the CI system."},"comment":{"url":"https://api.github.com/repos/mesosphere/dcos-commons/issues/comments/331076991","html_url":"https://github.com/mesosphere/dcos-commons/pull/1626#issuecomment-331076991","issue_url":"https://api.github.com/repos/mesosphere/dcos-commons/issues/1626","id":331076991,"user":{"login":"elezar","id":931393,"avatar_url":"https://avatars3.githubusercontent.com/u/931393?v=4","gravatar_id":"","url":"https://api.github.com/users/elezar","html_url":"https://github.com/elezar","followers_url":"https://api.github.com/users/elezar/followers","following_url":"https://api.github.com/users/elezar/following{/other_user}","gists_url":"https://api.github.com/users/elezar/gists{/gist_id}","starred_url":"https://api.github.com/users/elezar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/elezar/subscriptions","organizations_url":"https://api.github.com/users/elezar/orgs","repos_url":"https://api.github.com/users/elezar/repos","events_url":"https://api.github.com/users/elezar/events{/privacy}","received_events_url":"https://api.github.com/users/elezar/received_events","type":"User","site_admin":false},"created_at":"2017-09-21T07:36:37Z","updated_at":"2017-09-21T07:36:37Z","author_association":"CONTRIBUTOR","body":"Thanks @benclarkwood. I have switched back to `:latest` and also created a PR to do the same in mesosphere/confluent#63. If i could get a review there too, that would be great."}}
{"actor":{"display_login":"elezar"}}