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

v5.16.3: Fix PosixWatchdog #1677

Merged
merged 14 commits into from
Oct 22, 2023
Merged

v5.16.3: Fix PosixWatchdog #1677

merged 14 commits into from
Oct 22, 2023

Conversation

Cyberboss
Copy link
Member

@Cyberboss Cyberboss commented Oct 18, 2023

🆑
Fixed issues with DreamDaemon referencing the original true path of instances when using the advanced watchdog on Linux. Hard linked copies of deployment directory structure are now used instead of symlinks.
If a deployment completes on Linux right before a world with a valid DMAPI reboots it may not have enough time to be applied and delayed until the next reboot.
/:cl:

Fixes #1680

  • Rename ISymlinkFactory to IFileSystemLinkFactory
  • Rename SwappableDmbProvider to SymlinkDmbProvider.
  • Extract WindowsWatchdog code into abstract AdvancedWatchdog.

@Cyberboss Cyberboss added Fix Fixes incorrect functionality Priority: High Fix as soon as possible Area: Watchdog With regard to managing DreamDaemon servers Posix Only Issue only applies to non-Windows deployments Watchdog: Advanced Issue with the WindowsWatchdog or PosixWatchdog labels Oct 18, 2023
@Cyberboss Cyberboss added this to the v5.16.3 milestone Oct 18, 2023
- IDmbProvider is now `AsyncDisposable` as opposed to `IDisposable`.
- Add hard link support to `ISymlinkFactory`.
- `PosixWatchdog` now has to mirror deployment structure as hard links before swapping. Because of this deployments may not immediately be applied if the reboots immediately after they were completed.
Move Windows specific functionality out to `WindowsWatchdog`.
@Cyberboss Cyberboss marked this pull request as ready for review October 21, 2023 22:27
@Cyberboss Cyberboss force-pushed the FixLinuxFuckingBullshit branch from 36d2115 to 5bb655e Compare October 21, 2023 22:56
github-actions[bot]
github-actions bot previously approved these changes Oct 21, 2023
@Cyberboss Cyberboss added the Refactor Refactor functionality for future improvements label Oct 21, 2023
@Cyberboss Cyberboss enabled auto-merge October 21, 2023 23:02
@Cyberboss Cyberboss linked an issue Oct 21, 2023 that may be closed by this pull request
github-actions[bot]
github-actions bot previously approved these changes Oct 22, 2023
github-actions[bot]
github-actions bot previously approved these changes Oct 22, 2023
github-actions[bot]
github-actions bot previously approved these changes Oct 22, 2023
@Cyberboss Cyberboss added the Release This is a release pull request label Oct 22, 2023
github-actions[bot]
github-actions bot previously approved these changes Oct 22, 2023
@Cyberboss Cyberboss added Area: Compiler With regard to managing the deployment process Component Issue Issue regarding the service components labels Oct 22, 2023
@Cyberboss Cyberboss changed the title Fix PosixWatchdog v5.16.3: Fix PosixWatchdog and bad DMAPI Deployment Timeouts Oct 22, 2023
@Cyberboss Cyberboss disabled auto-merge October 22, 2023 03:21
@Cyberboss Cyberboss enabled auto-merge October 22, 2023 03:21
@Cyberboss
Copy link
Member Author

Merging with [TGSDeploy]

github-actions[bot]
github-actions bot previously approved these changes Oct 22, 2023
github-actions[bot]
github-actions bot previously approved these changes Oct 22, 2023
@Cyberboss Cyberboss changed the title v5.16.3: Fix PosixWatchdog and bad DMAPI Deployment Timeouts v5.16.3: Fix PosixWatchdog Oct 22, 2023
@codecov
Copy link

codecov bot commented Oct 22, 2023

Codecov Report

Merging #1677 (cfca47d) into master (83117cf) will increase coverage by 0.00%.
The diff coverage is 87.17%.

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #1677    +/-   ##
========================================
  Coverage   94.92%   94.92%            
========================================
  Files         606      609     +3     
  Lines      126542   126725   +183     
  Branches     2788     2803    +15     
========================================
+ Hits       120119   120299   +180     
- Misses       5942     5945     +3     
  Partials      481      481            

@Cyberboss Cyberboss merged commit 708a5cb into master Oct 22, 2023
87 checks passed
@Cyberboss Cyberboss deleted the FixLinuxFuckingBullshit branch October 22, 2023 04:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Compiler With regard to managing the deployment process Area: Watchdog With regard to managing DreamDaemon servers Component Issue Issue regarding the service components Fix Fixes incorrect functionality Posix Only Issue only applies to non-Windows deployments Priority: High Fix as soon as possible Refactor Refactor functionality for future improvements Release This is a release pull request Watchdog: Advanced Issue with the WindowsWatchdog or PosixWatchdog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PosixWatchdog doesn't work
1 participant