-
Notifications
You must be signed in to change notification settings - Fork 392
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
[oracle_weblogic] Add Integration Package with Deployed Application Data Stream #3121
[oracle_weblogic] Add Integration Package with Deployed Application Data Stream #3121
Conversation
/test |
🌐 Coverage report
|
1d20332
to
07151e7
Compare
/test |
2 similar comments
/test |
/test |
/test |
1 similar comment
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requested some changes
field: oracle_weblogic.deployed_application.deployment.state | ||
value: Shutdown | ||
ignore_empty_value: true | ||
if: ctx.oracle_weblogic.deployed_application.deployment.state_value == 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using a painless script with a HashMap
seems to be a better option here.
ignore_empty_value: true | ||
if: ctx.oracle_weblogic.deployed_application.deployment.state_value == 2 | ||
ignore_failure: true | ||
- remove: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please move remove
to the end
packages/oracle_weblogic/data_stream/deployed_application/fields/fields.yml
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
…ackage_oracle_weblogic_deployed_application Conflicts: packages/oracle_weblogic/_dev/build/docs/README.md packages/oracle_weblogic/changelog.yml packages/oracle_weblogic/docs/README.md packages/oracle_weblogic/manifest.yml
@@ -96,7 +98,7 @@ policy_templates: | |||
- http://localhost:8004 | |||
description: "Hosts for Oracle WebLogic metrics (example: http://localhost:8004)." | |||
- type: logfile | |||
title: Collect Oracle WebLogic logs. | |||
title: Collect Oracle WebLogic logs | |||
description: Collecting Oracle WebLogic logs of Admin Server, Managed Server, Domain and Access logs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description to cover all information of all the metrics that are fetched ( not just logs)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, made the changes (quick reference)
💚 CLA has been signed |
c85e892
to
251813d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
What does this PR do?
Checklist
changelog.yml
file.How to test this PR locally
elastic-package test
Related issues
Screenshots