[Python-apps-team] Bug#844081: Reproducer

[Christian Geier]

Hi,
I could reproduce the issue and have fixed it, see the PR on github [1].
Depending on how urgent this is, you can either just take the os.sync()
commit and apply that to the version currently in Debian (which, while
brute force, certainly fixes this problem), or you might want to wait if
we find a more elegant solution.

Christian

[1] https://github.com/pimutils/khal/pull/543

Quoting Filip Pytloun (2016-12-11 13:16:56)
> Hello,
> 
> unfortunately I have no longer access to environment where this was
> happening. But if you were able to reproduce the issue and this fixed it
> for you, I'll apply it and make new release.
> 
> Anyway wouldn't it better to ensure data is written to disk directly
> during db updates and other operations? Eg. use O_SYNC for safety as
> these operations doesn't happen so often.
> 
> Filip
> 
> On 2016/12/11 01:14, Christian Geier wrote:
> > Hi Filip,
> > could you perhaps try to change all those sleep()s to `os.sync()`? For
> > me it seems to fix the issue.
> > 
> > See [0] for a patch.
> > 
> > If this doesn fix the issue, we obviously need to move the sync call out
> > of the tests and into the db update.
> > 
> > Best regards,
> > Christian
> > 
> > [0] https://github.com/pimutils/khal/commit/0b636f7633e86b9e136b06e9965cd3af0e3918f2
> >