Skip to content
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

Update .fleet-agents mapping #89270

Merged
merged 1 commit into from
Sep 7, 2022

Conversation

AndersonQ
Copy link
Member

Update the .fleet-agents mapping to add the new outputs field. Due to elastic/fleet-server#1672 ther is the need to extend the current agent model. This PR updates the index mapping to reflect the changes made on fleet-server codebase.

Related issue: elastic/fleet-server#1672
Related PR: elastic/fleet-server#1684

Update the .fleet-agents mapping to add the new outputs field. See elastic/fleet-server#1672 for details.
@AndersonQ AndersonQ requested a review from a team as a code owner August 11, 2022 09:09
@AndersonQ AndersonQ requested review from michalpristas and blakerouse and removed request for a team August 11, 2022 09:09
@elasticsearchmachine elasticsearchmachine added v8.5.0 needs:triage Requires assignment of a team area label external-contributor Pull request authored by a developer outside the Elasticsearch team labels Aug 11, 2022
@AndersonQ AndersonQ requested a review from aleksmaus August 11, 2022 09:22
@AndersonQ AndersonQ added :Core/Infra/Core Core issues without another label Team:Core/Infra Meta label for core/infra team labels Aug 11, 2022
@AndersonQ AndersonQ self-assigned this Aug 11, 2022
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@elasticsearchmachine elasticsearchmachine removed the needs:triage Requires assignment of a team area label label Aug 11, 2022
@AndersonQ
Copy link
Member Author

/test

@AndersonQ AndersonQ closed this Aug 11, 2022
@AndersonQ AndersonQ reopened this Aug 11, 2022
Copy link
Member

@aleksmaus aleksmaus left a comment

Choose a reason for hiding this comment

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

lgtm. did you have a chance to test it?

@AndersonQ
Copy link
Member Author

lgtm. did you have a chance to test it?

not yet, I'm still checking how to run ES from source

Copy link
Contributor

@williamrandolph williamrandolph left a comment

Choose a reason for hiding this comment

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

This looks good to me. Is there anything I can do to help with testing?

@AndersonQ
Copy link
Member Author

I tested, it works :)

@AndersonQ
Copy link
Member Author

I'll just wait to merge this together with elastic/fleet-server#1684

@AndersonQ AndersonQ merged commit 3337493 into elastic:main Sep 7, 2022
@AndersonQ AndersonQ deleted the 2148-fleet-server-api-key branch September 7, 2022 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Core Core issues without another label external-contributor Pull request authored by a developer outside the Elasticsearch team >non-issue Team:Core/Infra Meta label for core/infra team v8.5.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants