-
Notifications
You must be signed in to change notification settings - Fork 470
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
Migrate obs-infraobs-integrations to package-spec v3 #5 #8234
Migrate obs-infraobs-integrations to package-spec v3 #5 #8234
Conversation
🌐 Coverage report
|
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.
LGTM - only question is about addition of beta
release tags to data streams. thanks!
@tommyers-elastic Do you know someone who can help with the CI issue we're facing with influxdb? Ref: #8171 (comment) |
We're good with mongodb tsdb testing ^^ |
Upgraded the stack version for influxdb to ^8.9.0 to fix the constant CI failures:
|
@tommyers-elastic Should I merge this PR? |
i'm happy to upgrade the stack version to make the CI more reliable. @lalit-satapathy how about you? |
@shmsr, Is the new conclusion that the CI is passing in 8.9.* and failing in 8.8.* for dimension_fields override? Our earlier understanding as per this, lead to conclude that, the issue is resolved in 8.8.* onwards. I am fine, in that is case to upgrade to 8.9. Need to summarise and to see if any changes needed in other packages. Example kafka? CC: @agithomas |
For some unknown reason it is only happening for After failing to conclude why the CI failed for |
So long as it consistently passes in 8.9 and fails in 8.8, we can conclude that some external change done in 8.9 stack. |
Package influxdb - 0.7.0 containing this change is available at https://epr.elastic.co/search?package=influxdb |
Package mongodb - 1.12.0 containing this change is available at https://epr.elastic.co/search?package=mongodb |
Cherry-picked changes for 2 packages to this branch so that #8171 gets unblocked:
Proposed commit message
Migrate the following package to use package-spec v3:
Here influxdb ci is failing for some unknown reason i.e., it failing with an error where it shouldn't. And mongodb needs to be tested to check if no data loss is there (tsdb).
Checklist
changelog.yml
file.