-
Notifications
You must be signed in to change notification settings - Fork 31
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
Add management mode enum to AgentInfo
metadata
#110
Add management mode enum to AgentInfo
metadata
#110
Conversation
Are we sure we need these changes? Why did this break only now? What did you attempt to do when it broke and what was the symptom — I can try to reproduce this on my end (from |
@ycombinator Sorry, should have explained that a bit more: the
I'm guessing the original code did this since it was a bit of a hack to install binaries without using
I figured it was a small enough change that I could just throw it in here, but we can spin it out into a different PR if you want. |
Thanks for the explanation, @fearful-symmetry. I pulled your magefile changes into their own PR here: #111. |
💚 Build Succeeded
History
|
Needed for elastic/beats#39403
This adds a
AgentManagedMode
enum to theAgentInfo
struct. This is in service of having beats send the fleet mode upstream via the user agent. Right now, management mode is only known to elastic-agent, so that data has to travel from Agent to Beats.This also tinkers with the magefile, as the
go-licenser
install was completely broken for me. This should be correct?