Skip to content

elastic-agent manifests: align with elastic-agent repo; add comments#136394

Merged
tetianakravchenko merged 7 commits intoelastic:mainfrom
tetianakravchenko:elastic-agent-comments
Jul 25, 2022
Merged

elastic-agent manifests: align with elastic-agent repo; add comments#136394
tetianakravchenko merged 7 commits intoelastic:mainfrom
tetianakravchenko:elastic-agent-comments

Conversation

@tetianakravchenko
Copy link
Copy Markdown
Contributor

Signed-off-by: Tetiana Kravchenko tetiana.kravchenko@elastic.co

Summary

Align elastic-agent manifests with the elastic-agent repo: https://github.com/elastic/elastic-agent/blob/main/deploy/kubernetes/elastic-agent-managed-kubernetes.yaml for managed elastic-agent and standalone https://github.com/elastic/elastic-agent/blob/main/deploy/kubernetes/elastic-agent-standalone-kubernetes.yaml.

This PR also includes changes in PR - elastic/elastic-agent#708. should be merged before this PR.

Checklist

Delete any items that are not applicable to this PR.

Risk Matrix

Delete this section if it is not applicable to this PR.

Before closing this PR, invite QA, stakeholders, and other developers to identify risks that should be tested prior to the change/feature release.

When forming the risk matrix, consider some of the following examples and how they may potentially impact the change:

Risk Probability Severity Mitigation/Notes
Multiple Spaces—unexpected behavior in non-default Kibana Space. Low High Integration tests will verify that all features are still supported in non-default Kibana Space and when user switches between spaces.
Multiple nodes—Elasticsearch polling might have race conditions when multiple Kibana nodes are polling for the same tasks. High Low Tasks are idempotent, so executing them multiple times will not result in logical error, but will degrade performance. To test for this case we add plenty of unit tests around this logic and document manual testing procedure.
Code should gracefully handle cases when feature X or plugin Y are disabled. Medium High Unit tests will verify that any feature flag or plugin combination still results in our service operational.
See more potential risk examples

For maintainers

Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
- rolebindings
- roles
verbs: ["get", "list", "watch"]
- apiGroups: ["networking.k8s.io"]
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this wasn't added to the corresponding PR in elastic-agent repo - elastic/elastic-agent#381, was removed before merging elastic/elastic-agent@30b3291

Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
@tetianakravchenko tetianakravchenko marked this pull request as ready for review July 21, 2022 14:25
@tetianakravchenko tetianakravchenko requested a review from a team as a code owner July 21, 2022 14:25
@tetianakravchenko tetianakravchenko added release_note:enhancement backport:skip This PR does not require backporting labels Jul 21, 2022
@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Jul 21, 2022
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/fleet (Team:Fleet)

@MichaelKatsoulis
Copy link
Copy Markdown
Contributor

Before merging align with elastic/elastic-agent#708 (comment)

Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
@tetianakravchenko
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@MichaelKatsoulis
Copy link
Copy Markdown
Contributor

LGTM!

@tetianakravchenko tetianakravchenko merged commit ee3cfb6 into elastic:main Jul 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:enhancement Team:Fleet Team label for Observability Data Collection Fleet team v8.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants