https://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2018-03-07T15:08:11ZCeph Dashboard - Bug #23265: FAIL: test_get (tasks.mgr.dashboard_v2.test_cluster_configuration.ClusterConfigurationTest)https://tracker.ceph.com/issues/23265?journal_id=1086962018-03-07T15:08:11ZRicardo Diasrdias@suse.com
<ul></ul><p>To reproduce this bug:</p>
<pre>
$ git checkout 480f078c147792c6bb234ee0c0b5ab9dbead5b05
$ cd src/pybind/mgr/dashboard_v2/
$ ./run-backend-api-tests.sh
</pre> Dashboard - Bug #23265: FAIL: test_get (tasks.mgr.dashboard_v2.test_cluster_configuration.ClusterConfigurationTest)https://tracker.ceph.com/issues/23265?journal_id=1086992018-03-07T15:40:40ZSebastian Wagner
<ul></ul><p>Here is the PR: <a class="external" href="https://github.com/ceph/ceph/pull/20782">https://github.com/ceph/ceph/pull/20782</a></p> Dashboard - Bug #23265: FAIL: test_get (tasks.mgr.dashboard_v2.test_cluster_configuration.ClusterConfigurationTest)https://tracker.ceph.com/issues/23265?journal_id=1087062018-03-07T16:26:35ZSebastian Wagner
<ul></ul><p>Some details. Before my commit,</p>
<p>1. <code>md_config_t::config_options</code> expected <code>Option::dump</code> to start an <code>object</code> section. This was no longer the case.<br />2. Therefore <code>md_config_t::config_options</code> did return a long array of things.<br />3. Therefore <code>ActivePyModules::get_python("config_objects") returned garbage<br />4. Therefore @src/pybind/mgr/dashboard_v2/controllers/cluster_configuration.py:ClusterConfiguration#list</code> raised an `IndexError` when filtering for a level.<br />5. Therefore <code>dashboard_v2</code> returned an Internal Server Error.<br />6. Therefore The test in <code>/ceph/qa/tasks/mgr/dashboard_v2/test_cluster_configuration.py, line 43, in test_get</code> failed.</p> Dashboard - Bug #23265: FAIL: test_get (tasks.mgr.dashboard_v2.test_cluster_configuration.ClusterConfigurationTest)https://tracker.ceph.com/issues/23265?journal_id=1087082018-03-07T17:58:12ZSebastian Wagner
<ul></ul><p>Turned out, this issue consists of two parts: the first one is described above, and the second one was a rename of "developer" to "dev", which was fixed by removing unused code.</p> Dashboard - Bug #23265: FAIL: test_get (tasks.mgr.dashboard_v2.test_cluster_configuration.ClusterConfigurationTest)https://tracker.ceph.com/issues/23265?journal_id=1087372018-03-08T07:51:15ZRicardo Diasrdias@suse.com
<ul></ul><p>Sebastian Wagner wrote:</p>
<blockquote>
<p>... the second one was a rename of "developer" to "dev", which was fixed by removing unused code.</p>
</blockquote>
<p>I don't understand how the issue is fixed just by removing unused code, the "rename" should break the dashboard frontend, right?</p> Dashboard - Bug #23265: FAIL: test_get (tasks.mgr.dashboard_v2.test_cluster_configuration.ClusterConfigurationTest)https://tracker.ceph.com/issues/23265?journal_id=1087382018-03-08T09:25:38ZSebastian Wagner
<ul></ul><blockquote>
<p>I don't understand how the issue is fixed just by removing unused code,</p>
</blockquote>
<p>The test failure was fixed by removing unused code. The bug is still present in the frontend code.</p> Dashboard - Bug #23265: FAIL: test_get (tasks.mgr.dashboard_v2.test_cluster_configuration.ClusterConfigurationTest)https://tracker.ceph.com/issues/23265?journal_id=1088082018-03-09T02:23:12ZKefu Chaitchaikov@gmail.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Resolved</i></li></ul> Dashboard - Bug #23265: FAIL: test_get (tasks.mgr.dashboard_v2.test_cluster_configuration.ClusterConfigurationTest)https://tracker.ceph.com/issues/23265?journal_id=1913512021-04-15T17:25:11ZErnesto Puerta
<ul><li><strong>Project</strong> changed from <i>mgr</i> to <i>Dashboard</i></li><li><strong>Category</strong> changed from <i>132</i> to <i>General</i></li></ul>