Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MUST PASS CI 10 CONSECUTIVE TIMES BEFORE MERGING] Workaround for world.Export potentially hanging forever #1789

Merged
merged 7 commits into from
Feb 26, 2024

Conversation

Cyberboss
Copy link
Member

@Cyberboss Cyberboss commented Feb 23, 2024

🆑 DreamMaker API
Worked around an extremely rare BYOND bug where calling TgsNew() hung forever.
Fixed waiting on custom events never returning if TGS restarted while the event was processing.
/🆑

🆑
Fixed a rare race condition when suspending Windows processes.
/🆑

Merge with [DMDeploy]
Closes #1681

Also if this works create a BYOND bug.

I'm going to close #1681 with this because I haven't seen that particular issue in for every and it's the most closely related.
@Cyberboss Cyberboss added Fix Fixes incorrect functionality Area: DMAPI Communication between TGS and DM labels Feb 23, 2024
@Cyberboss Cyberboss added this to the v6.3.0 milestone Feb 23, 2024
Copy link

codecov bot commented Feb 23, 2024

Codecov Report

Merging #1789 (575d811) into master (888ce7c) will decrease coverage by 0.01%.
The diff coverage is 81.81%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1789      +/-   ##
==========================================
- Coverage   95.50%   95.50%   -0.01%     
==========================================
  Files         679      679              
  Lines      145941   145949       +8     
  Branches     3170     3172       +2     
==========================================
+ Hits       139380   139383       +3     
- Misses       6073     6079       +6     
+ Partials      488      487       -1     

@Cyberboss
Copy link
Member Author

3 succeeded

@Cyberboss Cyberboss removed this from the v6.3.0 milestone Feb 23, 2024
@Cyberboss
Copy link
Member Author

4 succeeded

@Cyberboss Cyberboss added this to the v6.4.0 milestone Feb 23, 2024
github-actions[bot]
github-actions bot previously approved these changes Feb 23, 2024
github-actions[bot]
github-actions bot previously approved these changes Feb 24, 2024
@Cyberboss
Copy link
Member Author

above marks 7th run?

@Cyberboss Cyberboss changed the base branch from dev to master February 25, 2024 17:56
@Cyberboss Cyberboss dismissed github-actions[bot]’s stale review February 25, 2024 17:56

The base branch was changed.

@Cyberboss
Copy link
Member Author

Next is 8

@Cyberboss Cyberboss modified the milestones: v6.4.0, v6.3.1 Feb 25, 2024
github-actions[bot]
github-actions bot previously approved these changes Feb 25, 2024
github-actions[bot]
github-actions bot previously approved these changes Feb 25, 2024
github-actions[bot]
github-actions bot previously approved these changes Feb 26, 2024
@Cyberboss Cyberboss enabled auto-merge February 26, 2024 19:03
@Cyberboss Cyberboss merged commit 57512a4 into master Feb 26, 2024
101 checks passed
@Cyberboss Cyberboss deleted the TheLastSpuriousCIError branch February 26, 2024 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: DMAPI Communication between TGS and DM Fix Fixes incorrect functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BYOND sometimes lies about the result of world.Export()
1 participant