Skip to content

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

Merged
narph merged 4 commits intoelastic:7.11from
narph:backport_24156_7.11
Mar 10, 2021
Merged

Cherry-pick #24156 to 7.11: Empty configuration options generate <no value> string for azure-eventhub input#24182
narph merged 4 commits intoelastic:7.11from
narph:backport_24156_7.11

Conversation

@narph
Copy link
Copy Markdown
Contributor

@narph narph commented Feb 23, 2021 •
edited by zube bot
Loading

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

@narph
Empty configuration options generate <no value> string for azure-ev…
d5d4d29 
…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
Copy Markdown
Contributor

elasticmachine commented Feb 23, 2021

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
Copy Markdown
Contributor

elasticmachine commented Feb 23, 2021 •
edited by jenkins-beats-ci bot
Loading

💚 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

Reviewers

@jsoriano jsoriano jsoriano approved these changes

@adriansr adriansr Awaiting requested review from adriansr

Assignees

@narph narph

Labels

backport Team:Integrations Label for the Integrations team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@narph @elasticmachine @jsoriano