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 kubernetes examples configuration to use inputs #7080

Merged

Conversation

ph
Copy link
Contributor

@ph ph commented May 11, 2018

When we made the rename of the prospectors to input we overlooked the update of
the kubernetes examples files and readme.

Ref #6642

When we made the rename of the prospectors to input we overlooked the update of
the kubernetes examples files and readme.

Ref elastic#6642
@ph ph added Filebeat Filebeat needs_backport PR is waiting to be backported to other branches. labels May 11, 2018
@ph ph requested a review from exekias May 11, 2018 14:14
@ph
Copy link
Contributor Author

ph commented May 11, 2018

@exekias I am not too familiar with theses files and how to test them, but they were using prospectors instead of inputs, can you review theses changes?

Copy link
Contributor

@exekias exekias left a comment

Choose a reason for hiding this comment

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

Looking good 👍 WFG

@ph
Copy link
Contributor Author

ph commented May 11, 2018

@exekias Are theses files tested as part of the CI? I was suspicious because we were referencing a prospectors.d directory and I don't think we are explicitly creating it in any of our packages.

@exekias
Copy link
Contributor

exekias commented May 11, 2018

We test the manifest can be loaded to a mock kubernetes, so it has a valid format, but we don't really do an integration test, which would require running an actual kubernetes cluster (probably?).

I'm open to ideas to improve this, it would be nice to have some e2e testing here 😇

@ph
Copy link
Contributor Author

ph commented May 11, 2018

@exekias we will get there eventually :)

@exekias exekias merged commit cbc3f0e into elastic:master May 14, 2018
ph added a commit to ph/beats that referenced this pull request May 14, 2018
When we made the rename of the prospectors to input we overlooked the update of
the kubernetes examples files and readme.

Ref elastic#6642

(cherry picked from commit cbc3f0e)
@ph ph added v6.3.0 and removed needs_backport PR is waiting to be backported to other branches. labels May 14, 2018
exekias pushed a commit that referenced this pull request May 15, 2018
When we made the rename of the prospectors to input we overlooked the update of
the kubernetes examples files and readme.

Ref #6642

(cherry picked from commit cbc3f0e)
stevea78 pushed a commit to stevea78/beats that referenced this pull request May 20, 2018
When we made the rename of the prospectors to input we overlooked the update of
the kubernetes examples files and readme.

Ref elastic#6642
stevea78 pushed a commit to stevea78/beats that referenced this pull request May 20, 2018
When we made the rename of the prospectors to input we overlooked the update of
the kubernetes examples files and readme.

Ref elastic#6642
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…elastic#7095)

When we made the rename of the prospectors to input we overlooked the update of
the kubernetes examples files and readme.

Ref elastic#6642

(cherry picked from commit 343bbe4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants