Skip to content

Don't try to emit docker autodiscover event with missing container#10928

Merged
jsoriano merged 1 commit intoelastic:masterfrom
jsoriano:docker-autodiscover-panic
Feb 25, 2019
Merged

Don't try to emit docker autodiscover event with missing container#10928
jsoriano merged 1 commit intoelastic:masterfrom
jsoriano:docker-autodiscover-panic

Conversation

@jsoriano
Copy link
Copy Markdown
Member

If container or meta are nil, this will panic.

@jsoriano jsoriano added bug review libbeat containers Related to containers use case Team:Integrations Label for the Integrations team labels Feb 25, 2019
@jsoriano jsoriano self-assigned this Feb 25, 2019
@jsoriano jsoriano requested a review from roncohen February 25, 2019 12:44
@jsoriano jsoriano requested a review from a team as a code owner February 25, 2019 12:44
@jsoriano jsoriano merged commit a89b13f into elastic:master Feb 25, 2019
@jsoriano jsoriano deleted the docker-autodiscover-panic branch February 25, 2019 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug containers Related to containers use case libbeat review Team:Integrations Label for the Integrations team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants