- Notifications
You must be signed in to change notification settings - Fork 513
OSQuery Elastic Managed package #273
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
Conversation
💔 Build Failed
Expand to view the summary
Build stats
Steps errors Expand to view the steps failures
|
| Pinging @elastic/integrations (Team:Integrations) |
| Pinging @elastic/siem (Team:SIEM) |
| The CI is unhappy at the moment. Please run |
mtojek left a comment
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.
CI notified about issues.
| @mtojek |
| Hm.. it's weird. Could you please push-force changes to restart CI job? I suppose you merged potential changes from master. EDIT: I see you're missing some changes from master branch. Please rebase it first and we can check then. |
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.
I think these json blobs might need to be decoded 🤔 . I think the scripts in this repo handle encoding them before they are copied to the package-storage repo.
Example decoded: https://github.com/elastic/integrations/blob/master/packages/aws/kibana/dashboard/aws-0eb5a6a0-694f-11ea-b0ac-95d4ecb1fecd.json#L18
Example encoded in package storage: https://github.com/elastic/package-storage/blob/production/packages/aws/0.2.4/kibana/dashboard/aws-0eb5a6a0-694f-11ea-b0ac-95d4ecb1fecd.json#L9
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.
These are from POC and sure will change as well. Will update to have it decoded.
| Rebased, so had to force push. |
| Please adjust the PR accordingly to changes introduced in #280 . Main changes:
|
| @aleksmaus I'm afraid this PR is not compliant with latest changes in the Integrations. Would you like to re-adjust it or prefer to close it? |
| will check what's up and update as needed. |
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
|
| Removed all the kibana files for now, these were for POC. |




What does this PR do?
OSQuery package ported from POC PR elastic/package-storage#98
Checklist
The latter requires osquerybeat that is currently is not a part of the agent distribution by default.
Screenshots
This doesn't need to be merged right away, the development is still pending on product input and the shape of the package is expected to change.