[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #7713 [Stem]: Stem should invalidate GETCONF cache



#7713: Stem should invalidate GETCONF cache
-----------------------------+----------------------------------------------
 Reporter:  atagar           |          Owner:  atagar        
     Type:  defect           |         Status:  needs_revision
 Priority:  minor            |      Milestone:                
Component:  Stem             |        Version:                
 Keywords:  controller easy  |         Parent:                
   Points:                   |   Actualpoints:                
-----------------------------+----------------------------------------------

Comment(by neena):

 The assertion is failing because it is executed before the
 _confchanged_listener can finish updating the cached value. One way to
 make the test pass would be to have it sleep for a few 100 milliseconds
 before running

 {{{
       self.assertEqual(['mno,tuv,wxyz'], controller.get_conf("nodefamily",
 multiple = True))
 }}}

 The right way to do this would probably be to, umm, lock a lock when
 CONF_CHANGED occurs and wait for it to be released in the get_conf call.

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/7713#comment:10>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs