Rename config to policy and dataset to datastream#280
Rename config to policy and dataset to datastream#280mtojek merged 18 commits intoelastic:masterfrom
Conversation
|
Hi @mtojek, thanks for helping me get my registry running with this branch. I found that the package info endpoint doesn't contain Are you seeing the same? |
|
@jen-huang I might be off here but is the registry you are running built with this PR's changes in it: elastic/package-registry#641? |
|
@jen-huang , confirmed (I see the same results). It looks that you need to checkout the PR @ycombinator posted above (elastic/package-registry#641), then: In testing/environments, you need to update the reference to the Docker image: and finally: |
|
Thanks both!! I am seeing the correct package info now. Will continue with my testing. |
|
Another issue found: I think elastic/package-registry#641 is missing Example from system package info: "data_streams": [
{
"type": "logs",
- "name": "system.auth",
+ "dataset": "system.auth",
"title": "System auth logs",
"release": "experimental",
"ingest_pipeline": "default",
...
},
...
] |
|
I was able to monkey patch my local |
jen-huang
left a comment
There was a problem hiding this comment.
Didn't review actual code but tested this with elastic/kibana#78699 and it works well 👍
jonathan-buttner
left a comment
There was a problem hiding this comment.
I looked over the changes and they look good 👍
What does this PR do?
Checklist
Author's Checklist
How to test this PR locally
package-registry should be exposed under http://localhost:8080 .
Related issues
Screenshots