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

changelog/no-changelog log stream widget on iis overview dashboard should query source:iis #9769

Merged
merged 1 commit into from
Jul 27, 2021

Conversation

mtoffl01
Copy link
Contributor

What does this PR do?

Changes the query of the Log Stream widget on the iis overview dashboard from "iis" to source:iis.

Motivation

The log stream widget on this dashboard currently queries "iis" but it should query source:iis. Since there is an OOTB integration pipeline for IIS, any formally recognized IIS logs will have the source:iis facet and will not necessarily have "iis" free text in the message attribute of the log. source:iis is a better query.

Additional Notes

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have changelog/ and integration/ labels attached

The log stream widget on this dashboard currently queries "iis" but it should query source:iis. Since there is an OOTB integration pipeline for IIS, any formally recognized IIS logs will have the source:iis facet and will not necessarily have "iis" free text in the message attribute of the log. source:iis is a better query.
@ChristineTChen ChristineTChen merged commit a9470de into master Jul 27, 2021
@ChristineTChen ChristineTChen deleted the mtoffl01/iis_overview_dashboard branch July 27, 2021 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants