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

IIS Integration - pipeline update for the log format #3516

Merged
merged 15 commits into from
Jun 21, 2022

Conversation

muthu-mps
Copy link
Contributor

@muthu-mps muthu-mps commented Jun 16, 2022

Type of change

  • Enhancement

What does this PR do?

Fixes the issue reported in #2347 . Supports the given log format

2021-11-28 00:00:04 W3SVC1 10.44.0.136 GET /cdgo SERVER-STATUS=200 443 - 67.43.156.13 HTTP/1.1 Chrome/2.12.1 - example.com 200 0 0 34018 303 46

Checklist

  • I have added an entry to my package's changelog.yml file.

How to test this PR locally

The new log format is added to the test-iis-access.log
Execute the pipeline test to verify the given log format is working.

Related issues

@muthu-mps muthu-mps self-assigned this Jun 16, 2022
@muthu-mps muthu-mps changed the title IIS Integration - pipeline update for the log format #2347 IIS Integration - pipeline update for the log format Jun 16, 2022
@elasticmachine
Copy link

elasticmachine commented Jun 16, 2022

💚 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 preview

Expand to view the summary

Build stats

  • Start Time: 2022-06-20T12:15:44.574+0000

  • Duration: 14 min 11 sec

Test stats 🧪

Test Results
Failed 0
Passed 84
Skipped 0
Total 84

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@elasticmachine
Copy link

elasticmachine commented Jun 16, 2022

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (2/2) 💚
Files 100.0% (2/2) 💚 3.432
Classes 100.0% (2/2) 💚 3.432
Methods 85.714% (30/35) 👎 -3.239
Lines 91.781% (268/292) 👍 1.836
Conditionals 100.0% (0/0) 💚

@muthu-mps muthu-mps added the enhancement New feature or request label Jun 16, 2022
@muthu-mps muthu-mps marked this pull request as ready for review June 16, 2022 11:04
@muthu-mps muthu-mps requested a review from a team as a code owner June 16, 2022 11:04
@muthu-mps muthu-mps added the Team:Service-Integrations Label for the Service Integrations team label Jun 16, 2022
@ishleenk17
Copy link
Contributor

GTG!

@muthu-mps muthu-mps merged commit 2641d16 into elastic:main Jun 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Team:Service-Integrations Label for the Service Integrations team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IIS access Integration failed with unexpected pipeline error
3 participants