https://forge.ispras.ru/https://forge.ispras.ru/favicon.ico?16490126692017-11-01T08:24:57ZOpen-Source ProjectsKlever - Feature #8536: Untie coverage report from any components namehttps://forge.ispras.ru/issues/8536?journal_id=319932017-11-01T08:24:57ZEvgeny Novikovnovikov@ispras.ru
<ul></ul><p>I suggest to use the same request as for uploading other reports but to introduce a new report type. The provided data should look like it was before. I.e. reports should look like:<br /><pre><code class="json syntaxhl" data-language="json"><span class="p">{</span><span class="w">
</span><span class="nl">"type"</span><span class="p">:</span><span class="w"> </span><span class="s2">"(sub-)job coverage"</span><span class="p">,</span><span class="w">
</span><span class="nl">"id"</span><span class="p">:</span><span class="w"> </span><span class="s2">"/0/tests/linux/usb/gadget/all/6caad7a"</span><span class="p">,</span><span class="w">
</span><span class="nl">"coverage"</span><span class="p">:</span><span class="w"> </span><span class="p">{</span><span class="w">
</span><span class="nl">"generic:memory"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-fs5d38v6.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:alloc:irq"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-ljt7h_hr.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:alloc:spinlock"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-vi01zanp.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:alloc:usb lock"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-6vs1uld4.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:arch:io"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-sxemg0cv.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:block:genhd"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-wfwbydt7.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:block:queue"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-qri7x44k.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:block:request"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-7pcs9pf4.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:drivers:base:class"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-4z5no1v0.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:drivers:base:dma-mapping"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-nwk79nyc.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:drivers:clk1"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-n9ibxsul.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:drivers:clk2"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-9m0d7apw.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:fs:sysfs"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-wxj4miy9.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:kernel:locking:mutex"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-c963s9_q.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:kernel:locking:rwlock"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-sybump4g.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:kernel:locking:spinlock"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-7yd4r5sj.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:kernel:module"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-1z5i1axi.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:kernel:rcu:srcu"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-3n0t27no.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:kernel:rcu:update:lock"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-82vicj4l.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:kernel:rcu:update:lock bh"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-1jkcfcbh.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:kernel:rcu:update:lock sched"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-dmxfrak1.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:kernel:sched:completion"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-954sm6bs.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:lib:find_bit"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-hsag_igs.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:lib:idr"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-dyt4nd7w.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:net:register"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-z56kqaj1.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:net:rtnetlink"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-p7o3y4ts.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:net:sock"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-08p0h95a.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:usb:coherent"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-cn0m0a3_.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:usb:dev"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-qlpr3910.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:usb:gadget"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-ww3gp6zp.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:usb:register"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-a7i_e0d3.zip"</span><span class="p">,</span><span class="w">
</span><span class="nl">"linux:usb:urb"</span><span class="p">:</span><span class="w"> </span><span class="s2">"2541-blrjtaya.zip"</span><span class="w">
</span><span class="p">}</span><span class="w">
</span><span class="p">}</span><span class="w">
</span></code></pre><br />And all corresponding archives should be attached.</p> Klever - Feature #8536: Untie coverage report from any components namehttps://forge.ispras.ru/issues/8536?journal_id=319942017-11-01T08:40:01ZEvgeny Novikovnovikov@ispras.ru
<ul></ul><p>For uploading so called verification coverage attached to verification reports nothing should be changed.</p> Klever - Feature #8536: Untie coverage report from any components namehttps://forge.ispras.ru/issues/8536?journal_id=319992017-11-01T14:46:01ZVladimir Gratinskiygratinskiy@ispras.ru
<ul><li><strong>Due date</strong> set to <i>11/01/2017</i></li><li><strong>Status</strong> changed from <i>New</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Implemented in branch feature_8536. Global coverage can be uploaded now only within "job coverage" report. Verfication reports covrerage uploading wasn't changed.</p> Klever - Feature #8536: Untie coverage report from any components namehttps://forge.ispras.ru/issues/8536?journal_id=320002017-11-02T08:49:45ZIlja Zakharovilja.zakharov@ispras.ru
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Open</i></li></ul><p>There will be no Sub-job components now. Instead of them we will use components with names Job. But Core also can have other components as children. So it is better not to rely on component names there at all. I tested the functionality in the branch and after removing exception at UploadReport.py:419 all works as expected, even collapsing reports works and coverage left saved at Job components reports. However, I kindly ask you to check that there is no any other code that relies on the assumption that total coverage should always be saved at Core/Sub-jobs reports (because it is not true now).</p> Klever - Feature #8536: Untie coverage report from any components namehttps://forge.ispras.ru/issues/8536?journal_id=320012017-11-02T08:55:55ZEvgeny Novikovnovikov@ispras.ru
<ul></ul><p>My suggestion is too always keep components having coverage independently on their names. Sounds simple but may be hard to be implemented. Indeed you can use one more restriction, namely, coverage is reported either within verification reports or/and for first level Core children. In other cases you can corrupt jobs.</p> Klever - Feature #8536: Untie coverage report from any components namehttps://forge.ispras.ru/issues/8536?journal_id=320072017-11-02T11:08:07ZVladimir Gratinskiygratinskiy@ispras.ru
<ul><li><strong>Status</strong> changed from <i>Open</i> to <i>Resolved</i></li></ul><p>Fixed.</p> Klever - Feature #8536: Untie coverage report from any components namehttps://forge.ispras.ru/issues/8536?journal_id=320972017-11-14T10:36:01ZEvgeny Novikovnovikov@ispras.ru
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Closed</i></li></ul><p>I merged the branch to master in <a class="changeset" title="Merge branch '8149-new-progress'" href="https://forge.ispras.ru/projects/klever/repository/331/revisions/459f75e70c9d13331a178a2b3f4f594f3dd370c4">459f75e7</a>.</p>