Enrico Scholz <[email protected]> escreveu no dia quinta, 17/11/2022 à(s) 11:47:
> Jose Quaresma <[email protected]> writes: > > >> Transition to ThreadPoolExecutor > (eb6a6820928472ef194b963b606454e731f9486f) > >> broke he > >> > >> | Checking sstate mirror object availability: ... > >> > >> progress bar because the removed 'thread_worker' was still referenced > >> in an asynchronous function. As the result of the future is never > >> read, the resulting backtrace is silently discarded. > > > > I have tested the patch at the time and the progress bar works as > expected, > > maybe something breaks after that. > > Probably, you saw only the full 100% bar produced by > 'bb.event.ProcessFinished' > > But the patch does > > | - def checkstatus(thread_worker, arg): > | + def checkstatus(arg): > | ... > | bb.event.fire(bb.event.ProcessProgress(msg, > len(tasklist) - thread_worker.tasks.qsize()), d) > > This will always raise an exception and 'bb.event.ProcessProgress' will > never be fired. > I got where the problem came from, this patch serie has been tested on the autobuilder for quite some time and had several interactions so in one of them the patch [1] has been dropped. [1] https://lists.openembedded.org/g/openembedded-core/message/164564 Anyway your solution is much simpler than mine. Best regards, Jose > > > Can the num_tasks_done be moved to there and call the first > > next(num_tasks_done) so we can remove the +1 on the ProcessProgress > > I will send a v2 > > > > Enrico > -- Best regards, José Quaresma
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#173674): https://lists.openembedded.org/g/openembedded-core/message/173674 Mute This Topic: https://lists.openembedded.org/mt/95064126/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
