search

microsoft / vasim

Enhanced autoscaling with VASIM: Vertical Autoscaling Simulator Toolkit
MIT License
8 stars 3 forks source link

Parallelize tests #60

Closed bpkroth closed 20 hours ago

bpkroth commented 2 days ago

Investigate using pytest-xdist to parallelize tests.

bpkroth commented 1 day ago

Right now this doesn't work because multiple tests currently collide withone another's working directory when writing/deleting data from the filesystem.

This should probably be addressed through something like a ContextManager and TempDirectory that automatically cleanups up after itself.

ksaur commented 1 day ago

a stupidly easy way would just to name all of the test dirs differently

bpkroth commented 1 day ago

Yeah, that'd work for the tests. But I didn't look into whether this was fundamentally preventing simultaneous runs across instances for other purposes too.

From: Karla Saur @.> Sent: Tuesday, September 17, 2024 13:37 To: microsoft/vasim @.> Cc: Author @.>; Comment @.>; Subscribed @.***> Subject: Re: [microsoft/vasim] Parallelize tests (Issue #60)

a stupidly easy way would just to name all of the test dirs differently

— Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vasim/issues/60#issuecomment-2356636109 or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABQ53FF2TGYYKKBZWTK2NXDZXBZGPBFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLLDTOVRGUZLDORPXI6LQMWWES43TOVSUG33NNVSW45FGORXXA2LDOOJIFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVE4DGNRUGQZDEOBQQKSHI6LQMWSWS43TOVS2K5TBNR2WLKRSGUZDSNBXHAZDSN5HORZGSZ3HMVZKMY3SMVQXIZI. You are receiving this email because you authored the thread.

Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.