Conversation
💚 Build Succeeded
Expand to view the summary
Build stats
|
ruflin
left a comment
There was a problem hiding this comment.
LGTM.
As soon as this gets merged, a PR will be opened to put this into snapshot. There it can be tested with the current master build of the agent and Kibana.
@mtojek This is a good example of a package that should go into snapshot but not yet staging or production.
packages/synthetics/manifest.yml
Outdated
| title: Monitor | ||
| description: Perform synthetic monitor checks | ||
| # config_templates: | ||
| # - name: synthetics/http |
There was a problem hiding this comment.
Nice to see this prototyped on how it might work!
mtojek
left a comment
There was a problem hiding this comment.
I don't see a definition for owner. Could you please define a team owning this package? Also, please add the https://github.com/elastic/package-storage to team's repositories with Write permissions.
|
@mtojek does 2bb10eb look ok for the owner entry? Our team has access to https://github.com/elastic/package-storage as well. |
What does this PR do?
We are doing some prototype work for synthetics to get a baseline package configured for our solution.
Our eventual goal is probably not to merge this exact code, but to aid us in the development of our central management story.
How to test this PR locally
Follow the instructions under
testing/environmentsto spin up the environment. The synthetics package should be included in the Ingest Manager when running this branch.Screenshots