https://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2017-12-23T15:03:50ZCeph mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1040662017-12-23T15:03:50ZKefu Chaitchaikov@gmail.com
<ul></ul><p>/a/kchai-2017-12-23_05:58:55-rados-wip-kefu-testing-2017-12-23-1241-distro-basic-mira/1996509</p> mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1040722017-12-24T18:08:50ZKefu Chaitchaikov@gmail.com
<ul></ul><p>it's caused by <a class="external" href="https://github.com/ceph/ceph/pull/19369">https://github.com/ceph/ceph/pull/19369</a>. also this change breaks the upgrade test in a different way: <a class="external" href="http://pulpito.ceph.com/kchai-2017-12-24_02:20:05-rados-wip-kefu-testing-2017-12-23-1241-distro-basic-mira/1996875/">http://pulpito.ceph.com/kchai-2017-12-24_02:20:05-rados-wip-kefu-testing-2017-12-23-1241-distro-basic-mira/1996875/</a>, and <a class="external" href="https://github.com/ceph/ceph/pull/19369#commitcomment-26456668">https://github.com/ceph/ceph/pull/19369#commitcomment-26456668</a></p> mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1040732017-12-24T18:12:23ZKefu Chaitchaikov@gmail.com
<ul><li><strong>Project</strong> changed from <i>mgr</i> to <i>CephFS</i></li><li><strong>Component(FS)</strong> <i>MDSMonitor</i> added</li></ul> mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1040752017-12-24T18:17:41ZKefu Chaitchaikov@gmail.com
<ul><li><strong>Project</strong> changed from <i>CephFS</i> to <i>mgr</i></li><li><strong>Assignee</strong> set to <i>Patrick Donnelly</i></li></ul><p>hey Patrick, could you take a look?</p> mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1042152018-01-03T22:12:10ZPatrick Donnellypdonnell@redhat.com
<ul></ul><p>I'm inclined to revert [1] as it's not worth the trouble it's caused. There isn't a clean way to make the upgrade testing work and changing the JSON format broke the mgr.</p>
<p>[1] <a class="external" href="https://github.com/ceph/ceph/pull/19369">https://github.com/ceph/ceph/pull/19369</a></p> mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1042162018-01-03T23:26:57ZJohn Sprayjcspray@gmail.com
<ul></ul><p>+1 -- I'd revert that. The original format with the dicts is kind of ugly, but I don't think there's a strong enough need to spend effort managing the transition to lists (at least, unless someone else cares enough to do the whole job including dealing with upgrades from the mgr side).</p> mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1042192018-01-04T02:57:05ZZheng Yanukernel@gmail.com
<ul></ul><p>+1 for revert</p> mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1042432018-01-04T17:46:50ZPatrick Donnellypdonnell@redhat.com
<ul><li><strong>Status</strong> changed from <i>12</i> to <i>Resolved</i></li></ul><p>Fixed by f6e1a797d45d7d646835bad8e258908407b2695b.</p> mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1045602018-01-09T02:58:30ZKefu Chaitchaikov@gmail.com
<ul></ul><p><a class="external" href="http://pulpito.ceph.com/kchai-2018-01-08_13:52:40-rados-wip-kefu-testing-2018-01-08-1914-distro-basic-mira/2042789/">http://pulpito.ceph.com/kchai-2018-01-08_13:52:40-rados-wip-kefu-testing-2018-01-08-1914-distro-basic-mira/2042789/</a></p>
<p>i still have this issue in a upgrade test. but the branch being tested does include the revert change. weird enough..</p> mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1047362018-01-11T22:29:47ZPatrick Donnellypdonnell@redhat.com
<ul><li><strong>Related to</strong> <i><a class="issue tracker-1 status-3 priority-4 priority-default closed" href="/issues/21393">Bug #21393</a>: MDSMonitor: inconsistent role/who usage in command help</i> added</li></ul> mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1047372018-01-11T22:30:21ZPatrick Donnellypdonnell@redhat.com
<ul></ul><p>Kefu Chai wrote:</p>
<blockquote>
<p><a class="external" href="http://pulpito.ceph.com/kchai-2018-01-08_13:52:40-rados-wip-kefu-testing-2018-01-08-1914-distro-basic-mira/2042789/">http://pulpito.ceph.com/kchai-2018-01-08_13:52:40-rados-wip-kefu-testing-2018-01-08-1914-distro-basic-mira/2042789/</a></p>
<p>i still have this issue in a upgrade test. but the branch being tested does include the revert change. weird enough..</p>
</blockquote>
<p>Caused by the fix to <a class="issue tracker-1 status-3 priority-4 priority-default closed" title="Bug: MDSMonitor: inconsistent role/who usage in command help (Resolved)" href="https://tracker.ceph.com/issues/21393">#21393</a></p> mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1047382018-01-11T22:49:54ZPatrick Donnellypdonnell@redhat.com
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Fix Under Review</i></li></ul><p>Patrick Donnelly wrote:</p>
<blockquote>
<p>Kefu Chai wrote:</p>
<blockquote>
<p><a class="external" href="http://pulpito.ceph.com/kchai-2018-01-08_13:52:40-rados-wip-kefu-testing-2018-01-08-1914-distro-basic-mira/2042789/">http://pulpito.ceph.com/kchai-2018-01-08_13:52:40-rados-wip-kefu-testing-2018-01-08-1914-distro-basic-mira/2042789/</a></p>
<p>i still have this issue in a upgrade test. but the branch being tested does include the revert change. weird enough..</p>
</blockquote>
<p>Caused by the fix to <a class="issue tracker-1 status-3 priority-4 priority-default closed" title="Bug: MDSMonitor: inconsistent role/who usage in command help (Resolved)" href="https://tracker.ceph.com/issues/21393">#21393</a></p>
</blockquote>
<p><a class="external" href="https://github.com/ceph/ceph/pull/19926">https://github.com/ceph/ceph/pull/19926</a></p> mgr - Bug #22527: mgr.x runtime_error in json_spirit on upgradehttps://tracker.ceph.com/issues/22527?journal_id=1047772018-01-13T15:55:36ZSage Weilsage@newdream.net
<ul><li><strong>Status</strong> changed from <i>Fix Under Review</i> to <i>Resolved</i></li></ul>