{"action":"created","issue":{"url":"https://api.github.com/repos/xcat2/xcat-core/issues/668","labels_url":"https://api.github.com/repos/xcat2/xcat-core/issues/668/labels{/name}","comments_url":"https://api.github.com/repos/xcat2/xcat-core/issues/668/comments","events_url":"https://api.github.com/repos/xcat2/xcat-core/issues/668/events","html_url":"https://github.com/xcat2/xcat-core/pull/668","id":128776020,"number":668,"title":"Fix pmr 88439,227,000 xcat bug 573","user":{"login":"hu-weihua","id":7370626,"avatar_url":"https://avatars.githubusercontent.com/u/7370626?v=3","gravatar_id":"","url":"https://api.github.com/users/hu-weihua","html_url":"https://github.com/hu-weihua","followers_url":"https://api.github.com/users/hu-weihua/followers","following_url":"https://api.github.com/users/hu-weihua/following{/other_user}","gists_url":"https://api.github.com/users/hu-weihua/gists{/gist_id}","starred_url":"https://api.github.com/users/hu-weihua/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hu-weihua/subscriptions","organizations_url":"https://api.github.com/users/hu-weihua/orgs","repos_url":"https://api.github.com/users/hu-weihua/repos","events_url":"https://api.github.com/users/hu-weihua/events{/privacy}","received_events_url":"https://api.github.com/users/hu-weihua/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"daniceexi","id":10385985,"avatar_url":"https://avatars.githubusercontent.com/u/10385985?v=3","gravatar_id":"","url":"https://api.github.com/users/daniceexi","html_url":"https://github.com/daniceexi","followers_url":"https://api.github.com/users/daniceexi/followers","following_url":"https://api.github.com/users/daniceexi/following{/other_user}","gists_url":"https://api.github.com/users/daniceexi/gists{/gist_id}","starred_url":"https://api.github.com/users/daniceexi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daniceexi/subscriptions","organizations_url":"https://api.github.com/users/daniceexi/orgs","repos_url":"https://api.github.com/users/daniceexi/repos","events_url":"https://api.github.com/users/daniceexi/events{/privacy}","received_events_url":"https://api.github.com/users/daniceexi/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/xcat2/xcat-core/milestones/7","html_url":"https://github.com/xcat2/xcat-core/milestones/2.9.3","labels_url":"https://api.github.com/repos/xcat2/xcat-core/milestones/7/labels","id":1408570,"number":7,"title":"2.9.3","description":"","creator":{"login":"whowutwut","id":10049070,"avatar_url":"https://avatars.githubusercontent.com/u/10049070?v=3","gravatar_id":"","url":"https://api.github.com/users/whowutwut","html_url":"https://github.com/whowutwut","followers_url":"https://api.github.com/users/whowutwut/followers","following_url":"https://api.github.com/users/whowutwut/following{/other_user}","gists_url":"https://api.github.com/users/whowutwut/gists{/gist_id}","starred_url":"https://api.github.com/users/whowutwut/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/whowutwut/subscriptions","organizations_url":"https://api.github.com/users/whowutwut/orgs","repos_url":"https://api.github.com/users/whowutwut/repos","events_url":"https://api.github.com/users/whowutwut/events{/privacy}","received_events_url":"https://api.github.com/users/whowutwut/received_events","type":"User","site_admin":false},"open_issues":8,"closed_issues":5,"state":"open","created_at":"2015-11-12T15:15:31Z","updated_at":"2016-01-27T18:39:01Z","due_on":null,"closed_at":null},"comments":3,"created_at":"2016-01-26T09:52:24Z","updated_at":"2016-01-28T02:36:19Z","closed_at":"2016-01-27T13:46:53Z","pull_request":{"url":"https://api.github.com/repos/xcat2/xcat-core/pulls/668","html_url":"https://github.com/xcat2/xcat-core/pull/668","diff_url":"https://github.com/xcat2/xcat-core/pull/668.diff","patch_url":"https://github.com/xcat2/xcat-core/pull/668.patch"},"body":"After debug. I found in ``xcatd`` we use number to indicate signal, such like ``12``, ``2``,  but unfortunately the same number has different meaning between AIX and LINUX.  So some our signal handler almost lose efficacy. that result in some unexpected problem, such like core dump. socket pipe broken.  \r\n\r\nI change some important signal to macro definition in aix 2.9. (not for all signal just be caution, we have customer is using it. we can't do test totally). \r\n\r\nI also enhance the robustness of ``do_discovery_process`` to avoid ``storable file failed`` in some unexpected case.  I adjust indent of ``do_discovery_process`` at the same time."},"comment":{"url":"https://api.github.com/repos/xcat2/xcat-core/issues/comments/175925540","html_url":"https://github.com/xcat2/xcat-core/pull/668#issuecomment-175925540","issue_url":"https://api.github.com/repos/xcat2/xcat-core/issues/668","id":175925540,"user":{"login":"chenglch","id":7165199,"avatar_url":"https://avatars.githubusercontent.com/u/7165199?v=3","gravatar_id":"","url":"https://api.github.com/users/chenglch","html_url":"https://github.com/chenglch","followers_url":"https://api.github.com/users/chenglch/followers","following_url":"https://api.github.com/users/chenglch/following{/other_user}","gists_url":"https://api.github.com/users/chenglch/gists{/gist_id}","starred_url":"https://api.github.com/users/chenglch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chenglch/subscriptions","organizations_url":"https://api.github.com/users/chenglch/orgs","repos_url":"https://api.github.com/users/chenglch/repos","events_url":"https://api.github.com/users/chenglch/events{/privacy}","received_events_url":"https://api.github.com/users/chenglch/received_events","type":"User","site_admin":false},"created_at":"2016-01-28T02:36:19Z","updated_at":"2016-01-28T02:36:19Z","body":"@daniceexi I have already discussed with @hu-weihua , in master branch, to avoid of merge conflict, I will replace all the signal number with signal name when I fixing the signal handler defect about xcat DB process."}}