{"action":"created","issue":{"url":"https://api.github.com/repos/tendermint/tendermint/issues/2815","repository_url":"https://api.github.com/repos/tendermint/tendermint","labels_url":"https://api.github.com/repos/tendermint/tendermint/issues/2815/labels{/name}","comments_url":"https://api.github.com/repos/tendermint/tendermint/issues/2815/comments","events_url":"https://api.github.com/repos/tendermint/tendermint/issues/2815/events","html_url":"https://github.com/tendermint/tendermint/issues/2815","id":379833136,"node_id":"MDU6SXNzdWUzNzk4MzMxMzY=","number":2815,"title":"Tendermint starts with inconsistent genesis.json","user":{"login":"srmo","id":7474512,"node_id":"MDQ6VXNlcjc0NzQ1MTI=","avatar_url":"https://avatars3.githubusercontent.com/u/7474512?v=4","gravatar_id":"","url":"https://api.github.com/users/srmo","html_url":"https://github.com/srmo","followers_url":"https://api.github.com/users/srmo/followers","following_url":"https://api.github.com/users/srmo/following{/other_user}","gists_url":"https://api.github.com/users/srmo/gists{/gist_id}","starred_url":"https://api.github.com/users/srmo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srmo/subscriptions","organizations_url":"https://api.github.com/users/srmo/orgs","repos_url":"https://api.github.com/users/srmo/repos","events_url":"https://api.github.com/users/srmo/events{/privacy}","received_events_url":"https://api.github.com/users/srmo/received_events","type":"User","site_admin":false},"labels":[{"id":99653654,"node_id":"MDU6TGFiZWw5OTY1MzY1NA==","url":"https://api.github.com/repos/tendermint/tendermint/labels/bug","name":"bug","color":"fad2fa","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2018-11-12T15:41:18Z","updated_at":"2018-11-12T19:58:48Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"**Tendermint version** (use `tendermint version` or `git rev-parse --verify HEAD` if installed from source): 0.26.1\r\n\r\n\r\n**ABCI app** (name for built-in, URL for self-written if it's publicly available): private\r\n\r\n**Environment**:\r\n- **OS** (e.g. from /etc/os-release): linux / `Linux test2 3.16.0-4-amd64 #1 SMP Debian 3.16.39-1+deb8u2 (2017-03-07) x86_64 GNU/Linux`\r\n- **Install tools**: ansible\r\n- **Others**: 3 nodes, 2 validators. `create_empty_blocks=false`\r\n\r\n\r\n**What happened**:\r\nStarting a fresh chain, using a genesis file coming from 0.22.8. Migrated the \"consensus_param\" section.\r\nTendermint starts and builds first empty block. Then logs errors\r\n```\r\n2018-11-12_15:37:28.75853 E[12116-11-12|15:37:28.758] Peer sent us invalid msg                     module=consensus peer=\"Peer{MConn{192.168.55.12:38432} ec325039be4a8997c5e1c740870d7f0c29b16cca in}\" msg=\"[HEARTBEAT Heartbeat{-1:7C31F891F553 2/00 (293) /215B9D2937A8.../}]\" err=\"Negative ValidatorIndex\"\r\n```\r\n\r\nLooking at the tendermint code, it is assumed that this happens because the genesis.json has no \"address\" field for any validator.\r\n\r\n**What you expected to happen**:\r\nTendermint should either not start because the address field is missing or not throw errors.\r\n\r\n**Have you tried the latest version**: yes/no\r\nlatest release\r\n\r\n**How to reproduce it** (as minimally and precisely as possible):\r\ninit 0.26.1 and remove address field from genesis\r\n\r\n**Logs (paste a small part showing an error (< 10 lines) or link a pastebin, gist, etc. containing more of the log file)**:\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/tendermint/tendermint/issues/comments/438009592","html_url":"https://github.com/tendermint/tendermint/issues/2815#issuecomment-438009592","issue_url":"https://api.github.com/repos/tendermint/tendermint/issues/2815","id":438009592,"node_id":"MDEyOklzc3VlQ29tbWVudDQzODAwOTU5Mg==","user":{"login":"srmo","id":7474512,"node_id":"MDQ6VXNlcjc0NzQ1MTI=","avatar_url":"https://avatars3.githubusercontent.com/u/7474512?v=4","gravatar_id":"","url":"https://api.github.com/users/srmo","html_url":"https://github.com/srmo","followers_url":"https://api.github.com/users/srmo/followers","following_url":"https://api.github.com/users/srmo/following{/other_user}","gists_url":"https://api.github.com/users/srmo/gists{/gist_id}","starred_url":"https://api.github.com/users/srmo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srmo/subscriptions","organizations_url":"https://api.github.com/users/srmo/orgs","repos_url":"https://api.github.com/users/srmo/repos","events_url":"https://api.github.com/users/srmo/events{/privacy}","received_events_url":"https://api.github.com/users/srmo/received_events","type":"User","site_admin":false},"created_at":"2018-11-12T19:58:48Z","updated_at":"2018-11-12T19:58:48Z","author_association":"CONTRIBUTOR","body":"Yeah, the non-validator is definitely broadcasting proposalHeartbeats. But it also receives the heartbeats from the other nodes\r\nCurrently trying to understand the code.\r\n```\r\n2018-11-12_19:56:32.44268 D[12116-11-12|19:56:32.442] Received proposal heartbeat message          module=consensus height=2 round=0 sequence=1343 valIdx=1 valAddr=8704010FD424990C77A9CB9368CFE56FB38C8E01\r\n2018-11-12_19:56:32.50015 D[12116-11-12|19:56:32.499] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=15\r\n2018-11-12_19:56:34.50059 D[12116-11-12|19:56:34.500] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=16\r\n2018-11-12_19:56:36.50125 D[12116-11-12|19:56:36.501] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=17\r\n2018-11-12_19:56:38.50224 D[12116-11-12|19:56:38.502] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=18\r\n2018-11-12_19:56:40.50310 D[12116-11-12|19:56:40.503] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=19\r\n2018-11-12_19:56:42.50358 D[12116-11-12|19:56:42.503] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=20\r\n2018-11-12_19:56:44.50452 D[12116-11-12|19:56:44.504] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=21\r\n2018-11-12_19:56:46.50565 D[12116-11-12|19:56:46.505] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=22\r\n2018-11-12_19:56:48.50649 D[12116-11-12|19:56:48.506] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=23\r\n2018-11-12_19:56:50.50677 D[12116-11-12|19:56:50.506] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=24\r\n2018-11-12_19:56:52.50784 D[12116-11-12|19:56:52.507] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=25\r\n2018-11-12_19:56:54.50818 D[12116-11-12|19:56:54.508] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=26\r\n2018-11-12_19:56:56.50842 D[12116-11-12|19:56:56.508] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=27\r\n2018-11-12_19:56:58.50899 D[12116-11-12|19:56:58.508] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=28\r\n2018-11-12_19:57:00.50962 D[12116-11-12|19:57:00.509] Broadcasting proposal heartbeat message      module=consensus height=2 round=0 sequence=29\r\n2018-11-12_19:57:02.18605 D[12116-11-12|19:57:02.185] Receive                                      module=consensus src=\"Peer{MConn{192.168.55.10:26656} fdf44b89af4211ec672ed11c0c717c970bc8a299 out}\" chId=32 msg=\"[HEARTBEAT Heartbeat{0:6F75C4F06F61 2/00 (1368) /71D22882F110.../}]\"\r\n2018-11-12_19:57:02.18605 D[12116-11-12|19:57:02.185] Received proposal heartbeat message          module=consensus height=2 round=0 sequence=1368 valIdx=0 valAddr=6F75C4F06F618B123761337680AD7D7BB9DCA5\r\n```"}}
{"actor":{"display_login":"srmo"}}