Conversation
|
Pinging @elastic/integrations (Team:Integrations) |
| categories: | ||
| - web | ||
| release: experimental | ||
| release: ga |
There was a problem hiding this comment.
I think this should be updated in the data stream manifests too.
There was a problem hiding this comment.
FTR "GA" for data streams means removing the "release" property at all (according to spec).
| release: experimental | ||
| release: ga | ||
| conditions: | ||
| kibana.version: "^7.14.0" |
There was a problem hiding this comment.
nit: is it bound with a particular Kibana release or not?
There was a problem hiding this comment.
It shouldn't. Why the version should be different if everything is the same? Maybe I'm missing something 🤔
There was a problem hiding this comment.
We had an offline chat with Jaime, I agree we can take the risk here and leave the constraint as is (please ignore this comment).
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪 |
mtojek
left a comment
There was a problem hiding this comment.
There are some unadressed comments.
|
@sayden Please take a look at the CI report. If the package is "GA", you don't need the property "release". |
|
Ok so I removed the |
What does this PR do?
Release Apache integration as GA
Checklist
changelog.ymlfile.manifest.ymlfile to point to the latest Elastic stack release (e.g.^7.13.0).