mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [mesos] asekretenko commented on pull request #385: Fixed read of uninitialized variables detected with UBSAN.
Date Mon, 17 May 2021 18:04:56 GMT

asekretenko commented on pull request #385:
URL: https://github.com/apache/mesos/pull/385#issuecomment-841784763


   @cf-natali I'm also observing this on some platforms in an internal CI; reverted until
further investigation.
   
   One more note: after looking at the code that makes use of those attributes more carefully,
I'm realizing that our tests only work by miracle: the authorization callbacks that can be
cleared both by `~Master()` and by `~Slave()` belong to global libprocess state. Given that
we definitely have tests that are destroying slaves and masters in the middle of the test,
I'm now suprrised how this stuff works at all... most likely, we just don't have tests that
both rely on authn callbacks and destroy a master/an agent in the middle. Not sure if this
story has any relation to the cgroup destruction failure, though.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



Mime
View raw message