Skip to content

Add update-manager command for proxy agent scripts - #17

Open
fivetran-paulharper wants to merge 1 commit into
mainfrom
paulh/upgrade-management-script
Open

fivetran-paulharper wants to merge 1 commit into
mainfrom
paulh/upgrade-management-script

Conversation

@fivetran-paulharper

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI balanced review requested due to automatic review settings November 6, 2026 22:34

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues remain for the change enabling manager updates independently of the agent container.

No open issues 🎉

What changed in this PR

Adds update-manager to the Bash and PowerShell management scripts so users can update the manager independently of the agent container, without restarting it or requiring valid agent configuration.

Changes:

  • Downloads and validates a replacement script before replacing the installed manager, with temporary-file cleanup.
  • Adds tests for successful updates, URL overrides, proxy use, and failed or invalid downloads.
  • Documents the new command and manual update steps for older installations.
File Description
tests/​pwsh/​Manager.Tests.ps1 Adds PowerShell update tests and download stubbing.
tests/​bash/​stubs/​curl Supports output files and simulated download failures.
tests/​bash/​manager.bats Tests Bash updates, configuration independence, and failure cleanup.
README.md Documents manager updates and migration from older installations.
proxy-agent-manager.sh Implements Bash self-update before agent configuration validation.
proxy-agent-manager.ps1 Implements PowerShell self-update with proxy support and syntax validation.

Review effort: Balanced

@fivetran-paulharper
fivetran-paulharper marked this pull request as ready for review November 6, 2026 22:41
@fivetran-paulharper
fivetran-paulharper requested a review from a team November 6, 2026 22:41
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