Skip to content

Cherry-pick #20406 to 7.9: Makes metrics config option required in app_insights#20424

Merged
narph merged 1 commit intoelastic:7.9from
narph:backport_20406_7.9
Aug 4, 2020
Merged

Cherry-pick #20406 to 7.9: Makes metrics config option required in app_insights#20424
narph merged 1 commit intoelastic:7.9from
narph:backport_20406_7.9

Conversation

@narph
Copy link
Copy Markdown
Contributor

@narph narph commented Aug 4, 2020

Cherry-pick of PR #20406 to 7.9 branch. Original message:

What does this PR do?

Makes metrics config option required.

Why is it important?

Validation

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

@narph
Makes metrics config option required in app_insights (elastic#20406)
850602e 
* mofidy doc

* config

* changelog

* generate

* regen

* mage fmt

(cherry picked from commit 496e3c9)
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Aug 4, 2020
@narph narph self-assigned this Aug 4, 2020
@narph narph added [zube]: In Review backport Team:Integrations Label for the Integrations team labels Aug 4, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Aug 4, 2020

Pinging @elastic/integrations (Team:Integrations)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Aug 4, 2020
@zube zube bot added [zube]: Inbox needs_team Indicates that the issue/PR needs a Team:* label and removed [zube]: In Review labels Aug 4, 2020
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Aug 4, 2020
@botelastic
Copy link
Copy Markdown

botelastic bot commented Aug 4, 2020

This pull request doesn't have a Team:<team> label.

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Aug 4, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #20424 opened]

  • Start Time: 2020-08-04T12:05:43.834+0000

  • Duration: 61 min 55 sec

Test stats 🧪

Test Results
Failed 0
Passed 3482
Skipped 797
Total 4279

Steps errors

Expand to view the steps failures

  • Name: Install Go 1.14.4

    • Description: .ci/scripts/install-go.sh

    • Duration: 1 min 32 sec

    • Start Time: 2020-08-04T12:30:03.245+0000

    • log

@narph narph merged commit 1a74c7c into elastic:7.9 Aug 4, 2020
@narph narph deleted the backport_20406_7.9 branch August 4, 2020 13:54
@zube zube bot removed the [zube]: Done label Nov 3, 2020
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
@narph
Makes metrics config option required in app_insights (elastic#20406) (
65e3514 
elastic#20424)

* mofidy doc

* config

* changelog

* generate

* regen

* mage fmt

(cherry picked from commit a14cac5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@kaiyan-sheng kaiyan-sheng kaiyan-sheng approved these changes

Reviewers whose approvals may not affect merge requirements

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 @kaiyan-sheng