{"action":"closed","issue":{"url":"https://api.github.com/repos/apache/incubator-pulsar/issues/2379","repository_url":"https://api.github.com/repos/apache/incubator-pulsar","labels_url":"https://api.github.com/repos/apache/incubator-pulsar/issues/2379/labels{/name}","comments_url":"https://api.github.com/repos/apache/incubator-pulsar/issues/2379/comments","events_url":"https://api.github.com/repos/apache/incubator-pulsar/issues/2379/events","html_url":"https://github.com/apache/incubator-pulsar/issues/2379","id":350898914,"node_id":"MDU6SXNzdWUzNTA4OTg5MTQ=","number":2379,"title":"Initializing cluster metadata should handle /managed-ledger path better","user":{"login":"sijie","id":1217863,"node_id":"MDQ6VXNlcjEyMTc4NjM=","avatar_url":"https://avatars2.githubusercontent.com/u/1217863?v=4","gravatar_id":"","url":"https://api.github.com/users/sijie","html_url":"https://github.com/sijie","followers_url":"https://api.github.com/users/sijie/followers","following_url":"https://api.github.com/users/sijie/following{/other_user}","gists_url":"https://api.github.com/users/sijie/gists{/gist_id}","starred_url":"https://api.github.com/users/sijie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sijie/subscriptions","organizations_url":"https://api.github.com/users/sijie/orgs","repos_url":"https://api.github.com/users/sijie/repos","events_url":"https://api.github.com/users/sijie/events{/privacy}","received_events_url":"https://api.github.com/users/sijie/received_events","type":"User","site_admin":false},"labels":[{"id":970640331,"node_id":"MDU6TGFiZWw5NzA2NDAzMzE=","url":"https://api.github.com/repos/apache/incubator-pulsar/labels/component/cli","name":"component/cli","color":"1d76db","default":false},{"id":1025509751,"node_id":"MDU6TGFiZWwxMDI1NTA5NzUx","url":"https://api.github.com/repos/apache/incubator-pulsar/labels/triage/week-33","name":"triage/week-33","color":"fbca04","default":false},{"id":399568949,"node_id":"MDU6TGFiZWwzOTk1Njg5NDk=","url":"https://api.github.com/repos/apache/incubator-pulsar/labels/type/bug","name":"type/bug","color":"ee0701","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/apache/incubator-pulsar/milestones/16","html_url":"https://github.com/apache/incubator-pulsar/milestone/16","labels_url":"https://api.github.com/repos/apache/incubator-pulsar/milestones/16/labels","id":3433482,"node_id":"MDk6TWlsZXN0b25lMzQzMzQ4Mg==","number":16,"title":"2.2.0-incubating","description":"","creator":{"login":"sijie","id":1217863,"node_id":"MDQ6VXNlcjEyMTc4NjM=","avatar_url":"https://avatars2.githubusercontent.com/u/1217863?v=4","gravatar_id":"","url":"https://api.github.com/users/sijie","html_url":"https://github.com/sijie","followers_url":"https://api.github.com/users/sijie/followers","following_url":"https://api.github.com/users/sijie/following{/other_user}","gists_url":"https://api.github.com/users/sijie/gists{/gist_id}","starred_url":"https://api.github.com/users/sijie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sijie/subscriptions","organizations_url":"https://api.github.com/users/sijie/orgs","repos_url":"https://api.github.com/users/sijie/repos","events_url":"https://api.github.com/users/sijie/events{/privacy}","received_events_url":"https://api.github.com/users/sijie/received_events","type":"User","site_admin":false},"open_issues":19,"closed_issues":232,"state":"open","created_at":"2018-06-18T20:11:19Z","updated_at":"2018-09-12T08:19:35Z","due_on":"2018-08-23T07:00:00Z","closed_at":null},"comments":3,"created_at":"2018-08-15T17:38:26Z","updated_at":"2018-09-12T08:19:35Z","closed_at":"2018-09-12T08:19:35Z","author_association":"CONTRIBUTOR","body":"`/managed-ledger` can be created automatically by brokers without initializing metadata. so in helm deployment, all pods are usually started at the same time. hence the race condition can happen between broker starts and initializing cluster metadata.\r\n\r\n- broker starts triggering creating `/managed-ledger`\r\n- cluster-metadata-initializer attempts to initialize cluster metadata and fail since `/managed-ledger` is already there.\r\n\r\nit can be handled by adding more checks in `initContainer` before starting brokers. However I think we also need to improve the logic on initializing cluster metadata to better handle such corner cases."}}
{"actor":{"display_login":"sijie"}}