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

Cherry-pick #24156 to 7.11: Empty configuration options generate <no value> string for azure-eventhub input #24182

Merged
merged 4 commits into from
Mar 10, 2021

Conversation

narph
Copy link
Contributor

@narph narph commented Feb 23, 2021

Cherry-pick of PR #24156 to 7.11 branch. Original message:

What does this PR do?

makes sure to provide empty string values when input configuration is validated

Why is it important?

<no value> string is sent instead of empty value and sometimes passes required validation

Checklist

…enthub input (elastic#24156)

* work on no value

* changelog

(cherry picked from commit 916d4e0)
@narph narph added [zube]: In Review backport Team:Integrations Label for the Integrations team labels Feb 23, 2021
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Feb 23, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations (Team:Integrations)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Feb 23, 2021
@narph narph self-assigned this Feb 23, 2021
@narph narph requested a review from adriansr February 23, 2021 12:07
@elasticmachine
Copy link
Collaborator

elasticmachine commented Feb 23, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #24182 updated

  • Start Time: 2021-03-10T16:34:19.180+0000

  • Duration: 66 min 0 sec

  • Commit: 58a7c8f

Test stats 🧪

Test Results
Failed 0
Passed 6749
Skipped 1049
Total 7798

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 6749
Skipped 1049
Total 7798

@narph narph merged commit c46cc27 into elastic:7.11 Mar 10, 2021
@narph narph deleted the backport_24156_7.11 branch March 10, 2021 18:12
@zube zube bot removed the [zube]: Done label Jun 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Team:Integrations Label for the Integrations team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants