{"action":"created","issue":{"url":"https://api.github.com/repos/ahelsing/tickettest2/issues/1314","labels_url":"https://api.github.com/repos/ahelsing/tickettest2/issues/1314/labels{/name}","comments_url":"https://api.github.com/repos/ahelsing/tickettest2/issues/1314/comments","events_url":"https://api.github.com/repos/ahelsing/tickettest2/issues/1314/events","html_url":"https://github.com/ahelsing/tickettest2/issues/1314","id":78042953,"number":1314,"title":"Ready status override by multiple status replies","user":{"login":"MarshallBrinn","id":3009742,"avatar_url":"https://avatars.githubusercontent.com/u/3009742?v=3","gravatar_id":"","url":"https://api.github.com/users/MarshallBrinn","html_url":"https://github.com/MarshallBrinn","followers_url":"https://api.github.com/users/MarshallBrinn/followers","following_url":"https://api.github.com/users/MarshallBrinn/following{/other_user}","gists_url":"https://api.github.com/users/MarshallBrinn/gists{/gist_id}","starred_url":"https://api.github.com/users/MarshallBrinn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MarshallBrinn/subscriptions","organizations_url":"https://api.github.com/users/MarshallBrinn/orgs","repos_url":"https://api.github.com/users/MarshallBrinn/repos","events_url":"https://api.github.com/users/MarshallBrinn/events{/privacy}","received_events_url":"https://api.github.com/users/MarshallBrinn/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"MarshallBrinn","id":3009742,"avatar_url":"https://avatars.githubusercontent.com/u/3009742?v=3","gravatar_id":"","url":"https://api.github.com/users/MarshallBrinn","html_url":"https://github.com/MarshallBrinn","followers_url":"https://api.github.com/users/MarshallBrinn/followers","following_url":"https://api.github.com/users/MarshallBrinn/following{/other_user}","gists_url":"https://api.github.com/users/MarshallBrinn/gists{/gist_id}","starred_url":"https://api.github.com/users/MarshallBrinn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MarshallBrinn/subscriptions","organizations_url":"https://api.github.com/users/MarshallBrinn/orgs","repos_url":"https://api.github.com/users/MarshallBrinn/repos","events_url":"https://api.github.com/users/MarshallBrinn/events{/privacy}","received_events_url":"https://api.github.com/users/MarshallBrinn/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/ahelsing/tickettest2/milestones/65","html_url":"https://github.com/ahelsing/tickettest2/milestones/2.28","labels_url":"https://api.github.com/repos/ahelsing/tickettest2/milestones/65/labels","id":1117860,"number":65,"title":"2.28","description":null,"creator":{"login":"ahelsing","id":12136824,"avatar_url":"https://avatars.githubusercontent.com/u/12136824?v=3","gravatar_id":"","url":"https://api.github.com/users/ahelsing","html_url":"https://github.com/ahelsing","followers_url":"https://api.github.com/users/ahelsing/followers","following_url":"https://api.github.com/users/ahelsing/following{/other_user}","gists_url":"https://api.github.com/users/ahelsing/gists{/gist_id}","starred_url":"https://api.github.com/users/ahelsing/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ahelsing/subscriptions","organizations_url":"https://api.github.com/users/ahelsing/orgs","repos_url":"https://api.github.com/users/ahelsing/repos","events_url":"https://api.github.com/users/ahelsing/events{/privacy}","received_events_url":"https://api.github.com/users/ahelsing/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":6,"state":"open","created_at":"2015-05-19T10:26:23Z","updated_at":"2015-05-19T11:12:24Z","due_on":null,"closed_at":null},"comments":2,"created_at":"2015-05-19T11:12:24Z","updated_at":"2015-05-19T11:12:45Z","closed_at":null,"body":"When we have multiple status replies (for a stitched topology, say) we get in the 'green' from one status reply and then another status reply says 'unknown' and erases the green\r\n\r\nWe need to only use status from the authoritative AM.\n\n _Imported from trac ticket [#1314](http://trac.gpolab.bbn.com/proto-ch/ticket/1314),  created by mbrinn on 02-18-2015 at 10:31, last modified: 02-22-2015 at 12:21_\n"},"comment":{"url":"https://api.github.com/repos/ahelsing/tickettest2/issues/comments/103444177","html_url":"https://github.com/ahelsing/tickettest2/issues/1314#issuecomment-103444177","issue_url":"https://api.github.com/repos/ahelsing/tickettest2/issues/1314","id":103444177,"user":{"login":"MarshallBrinn","id":3009742,"avatar_url":"https://avatars.githubusercontent.com/u/3009742?v=3","gravatar_id":"","url":"https://api.github.com/users/MarshallBrinn","html_url":"https://github.com/MarshallBrinn","followers_url":"https://api.github.com/users/MarshallBrinn/followers","following_url":"https://api.github.com/users/MarshallBrinn/following{/other_user}","gists_url":"https://api.github.com/users/MarshallBrinn/gists{/gist_id}","starred_url":"https://api.github.com/users/MarshallBrinn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MarshallBrinn/subscriptions","organizations_url":"https://api.github.com/users/MarshallBrinn/orgs","repos_url":"https://api.github.com/users/MarshallBrinn/repos","events_url":"https://api.github.com/users/MarshallBrinn/events{/privacy}","received_events_url":"https://api.github.com/users/MarshallBrinn/received_events","type":"User","site_admin":false},"created_at":"2015-05-19T11:12:45Z","updated_at":"2015-05-19T11:12:45Z","body":"I've diagnosed the problem and it is a tricky one. Basically, whenever we give Jacks a manifest, it creates a new topology, getting rid of the old stuff and recreating it. So any formatting I did (setting things green e.g.) whenever a manifest comes in after a status message.\r\n\r\nThis is what is happening in this case, and could happen. In general we run the manifest/status in parallel for each AM (that is, we ask for all manifests, then as each one comes in, we do a a 'modify-topolgy' and then request the statues for that AM. \r\n\r\nBut if one of the manifests comes in after the status, then we see this problem. The box for the node that we painted based on the status no longer exists.\r\n\r\nThere are a bunch of ways to solve this but I think the best one is to keep a list of all the current status replies and apply them ALL after we get ANY manifest. \r\n\r\n\r\n\n\n_Trac comment by mbrinn (github user: MarshallBrinn) on 02-21-2015 at 11:25_\n"}}