-
Notifications
You must be signed in to change notification settings - Fork 42
[Fleet / Agent ] support install / uninstall command from Agent #326
Comments
here are the tests I propose we add, to a new file agent_subcommands.feature: The other subcommands that are already tested are: The newly tested subcommands added here are: The as yet tested subcommands are: This only has one test for the new 'uninstall' command, and it uninstalls an active running installation. #Tests @install @stop-installed-agent @restart-installed-host @unenroll-installed-host @reenroll-installed-host @uninstall-installed-host @restart-installed-host |
Further to note, there is an install -f and an uninstall -f that we will use here. Without the -f we'd need to interactively try to pass info to the command line for Agent to process. @mdelapenya do you think that would be possible? it would be great to have both covered, but just the -f is enough to start. Also, the install command is really quite similar to the enroll, it takes basically the same arguments, here are the 2 to compare: elastic-agent enroll https://8040186313ec4a228ecc22ebee9ed123.us-central1.gcp.foundit.no:443 YW9mUm9YUUJWMUZuVTV1TlJNeTQ6WjBTQ3R6VFZUb0toeTBEQ05CcGV123== elastic-agent install -f --kibana-url https://8040186313ec4a228ecc22ebee9e123.us-central1.gcp.foundit.no:9243 --enrollment-token YW9mUm9YUUJWMUZuVTV1TlJNeTQ6WjBTQ3R6VFZUb0toeTBEQ05CcGV123== |
As commented yesterday in Slack, we are going to create an installer for TAR, which will be the only one using the install/uninstall subcommands. We are able to apply this installation mechanism to both Centos and Debian, although it could be expanded to cover other Linux distros having an official Docker image. |
I'm going to put some thoughts down for this.
From this issue: elastic/beats#21019
briefly:
the feature adds in new support, and leaves the old in place.
So we'll still want to test the enroll and unenroll as we are. but we can add in new feature tests for 'install' and 'uninstall' from the Agent side.
The text was updated successfully, but these errors were encountered: