8000 Implemented in-place upgrade and improved robustness by hkavula · Pull Request #3 · hetrixtools/agent-windows · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Implemented in-place upgrade and improved robustness #3

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

hkavula
Copy link
@hkavula hkavula commented Mar 18, 2025

Implemented in-place upgrade:

  • If 8000 no arguments are provided, the script prefers to reuse an existing Server ID if possible over existing with an error. (Ready for previous v2 agent, but only tested with existing v1 agent!)
  • Omission of '0' now means that no processes/services need to be monitored.
  • Postponed v1 agent uninstall so that its Server ID remains available during $SID initalization.
  • Changes are fully compatible with Install Code generated by hetrixtools.com

Improved robustness:

  • Moved v1 agent uninstall after v2 agent installation has completed and before it is started.
  • Added sanity checks to ensure v1 agent remains operational until v2 agent is ready to take over...

Jeroen Verburgh and others added 3 commits March 19, 2025 00:05
…t prefers to reuse an existing Server ID if possible. Omission of '0' now means no processes/services need to be monitored. Postponed v1 agent uninstall so that its Server ID remains available during $SID initalization.
…v2 agent is started. Added sanity checks to ensure v1 agent remains available until v2 agent is ready to start.
@hetrixtools
Copy link
Owner

We'll review the proposed changes and test them out. Thank you for your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0