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.

Sign up for GitHub

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

Jump to bottom

[Ingest Manager] Refuse invalid stream values in configuration #18772

Closed
michalpristas opened this issue May 27, 2020 · 1 comment · Fixed by #19587
Closed

[Ingest Manager] Refuse invalid stream values in configuration #18772

michalpristas opened this issue May 27, 2020 · 1 comment · Fixed by #19587
Assignees
@michalpristas
Labels
bug Ingest Management:beta1 Group issues for ingest management beta1

Comments

@michalpristas
Copy link
Contributor

At the moment we try to fallback to default values if stream.namespace or stream.dataset has invalid value (empty string)

We should error out and refuse configuration in this case
@michalpristas michalpristas added bug Team:Ingest Management Ingest Management:beta1 Group issues for ingest management beta1 labels May 27, 2020
@michalpristas michalpristas self-assigned this May 27, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ingest-management (Team:Ingest Management)

@ph ph assigned ph and unassigned ph Jun 1, 2020
@michalpristas michalpristas mentioned this issue Jun 10, 2020
Merged
6 tasks
@michalpristas michalpristas mentioned this issue Jul 2, 2020
Merged
6 tasks
@michalpristas michalpristas mentioned this issue Jul 14, 2020
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@michalpristas michalpristas

Labels
bug Ingest Management:beta1 Group issues for ingest management beta1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Ingest Manager] Refuse invalid stream values in configuration michalpristas/beats
3 participants
@ph @michalpristas @elasticmachine