-
Notifications
You must be signed in to change notification settings - Fork 407
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 Managed Server Logs Data Stream #3067
[oracle_weblogic] Add Integration Package with Managed Server Logs Data Stream #3067
Conversation
🌐 Coverage report
|
packages/oracle_weblogic/data_stream/managed_server/elasticsearch/ingest_pipeline/default.yml
Show resolved
Hide resolved
@@ -0,0 +1,75 @@ | |||
--- | |||
description: Pipeline for processing Weblogic 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.
Please correct as Pipeline for processing WebLogic managed server logs
required: true | ||
show_user: true | ||
default: | ||
- /tmp/service_logs/managed-logs/ms-1.log |
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 correct the default path location
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.
Change looks good to me.
61873bb
to
c54d599
Compare
…ackage_oracle_weblogic_managed_server
/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.
PR looks good to me, I checked the logs for the failure.
Error: error running package system tests: could not complete test run: could not setup service: removing service logs failed: removing resource failed (path: /var/lib/jenkins/workspace/est-manager_integrations_PR-3067/.elastic-package/tmp/service_logs/adminserver): unlinkat /var/lib/jenkins/workspace/est-manager_integrations_PR-3067/.elastic-package/tmp/service_logs/adminserver/domain1.log: permission denied
Looks like you'll need to fix something with the permissions there.
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.
Just a couple of nits in comments to adjust, LGTM!
...ges/oracle_weblogic/_dev/deploy/docker/oracle_weblogic/container-scripts/startAdminServer.sh
Outdated
Show resolved
Hide resolved
...s/oracle_weblogic/_dev/deploy/docker/oracle_weblogic/container-scripts/startManagedServer.sh
Outdated
Show resolved
Hide resolved
…ntainer-scripts/startAdminServer.sh Co-authored-by: Yug Rajani <96041884+yug-elastic@users.noreply.github.com>
…ntainer-scripts/startManagedServer.sh Co-authored-by: Yug Rajani <96041884+yug-elastic@users.noreply.github.com>
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