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
[7.17 backport] Update elastic-agent-client used in 7.17 releases #39224
[7.17 backport] Update elastic-agent-client used in 7.17 releases #39224
Conversation
f889fec
to
6a5f9bc
Compare
19c7d68
to
3f9f7b0
Compare
3f9f7b0
to
ee0b705
Compare
@@ -148,6 +153,7 @@ func (s *Server) Start() error { | |||
if ok := certPool.AppendCertsFromPEM(s.ca.Crt()); !ok { | |||
return errors.New("failed to append root CA", errors.TypeSecurity) | |||
} | |||
//nolint:gosec // G402: TLS MinVersion too low. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can also set MinVersion
if we don't want to disable the linter here
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
To double check, Fleet server was confirmed to bootstrap with these changes? |
Yes, last time i tested it bootstrapped locally correctly. We can also try to make a custom issue and check on ESS as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
proto.StartUpInfo
is backwards compatible, looks good.
Proposed commit message
Update the elastic-agent-client to the latest tagged release.
Update protobuf libraries.
elastic-agent will have v2 related functions disabled.
Checklist
I have made corresponding changes to the documentationI have made corresponding change to the default configuration filesI have added an entry inCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.