-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[BUG] Plugin Build Fails Due to missing nebula
Dependencies.
#5121
Comments
I discussed this with @mch2 offline. Would you please provide more context on this? |
We see same issue in ml-commons Github CI today which passed last Friday. |
I'm tagging this for 2.4.0 since it's blocking the last PR of security plugin for 2.4.0. |
Break all the build repo manifest checks: |
Just seeing this now with SQL plugin. |
The error in the original issue says |
@dblock PRs need to be merged to main(3.0.0) and then backported to 2.4. |
@cliu123 Just make the PR on 2.x and backport it the other direction. |
Looks like version 4.4.4 of the publish plugin depends on nebula-core 3.0.0 which is not in maven central. These are all very old versions of these plugins (latest is v19) but I think we can update to 4.6.0 or 4.10.0 and resolve the issue. |
@mch2 The above seems to imply that it worked before. Was it removed from maven? Weird. But yes upgrading to a newer 4.x should be harmless. Are you on it? |
@dblock Yeah it must have been removed but I'm not sure how to check that. Will PR to a 4.x version shortly. |
It just failed in 2.4 here |
Issue was found as old as April this year: |
Related dependabot: |
What do you think about updating 4.6.0. works because it specifies
|
@MaxKsyunz That's a good path. Do you have any idea why 4.4.0 stopped working? |
The change hasn't made it into 2.4.0 snapshot builds, reopening this issue until builds in all active branches are fully unblocked |
#5132 includes the nebula fix for 2.4 |
The last build for 2.4.0 is more than 3 days ago [1] so #5127 was not included. @opensearch-project/engineering-effectiveness does another build need to be triggered manually? [1] https://build.ci.opensearch.org/job/distribution-build-opensearch/6425/ |
I see 2.4.0 auto build disabled in opensearch-project/opensearch-build#2852. Already raised a while ago that we shouldn't be disabling that, so this is another data point. Who knows how to kick it? cc: @bbarani @peterzhuamazon |
Hey @dblock I see it was disabled because the Release candidate for 2.4.0 is locked. |
@prudhvigodithi what bad thing would happen if we didn't? |
I see the RC is fixed to specific build numbers opensearch:2.4.0.6425, opensearch-dashboards:2.4.0.4275, we can still continue to run the auto builds, but only the downloads with |
Latest does not imply "release candidate", and I don't think it should. |
We have to disable the build at this point for the release but we can introduce -rc input manifests later. Thanks. |
Yup. I'll take a look.
No, I couldn't figure out why |
@peterzhuamazon if we have release candidate ready, docker has the specific tag and urls have the buildNumber in their path, it should be okay to keep building right? Basically, whatever we will be shipping to customer or final product is locked now so whatever else it overrides shouldn't be a concern. |
We have published the latest snapshot. |
Describe the bug
https://github.com/opensearch-project/security/actions/runs/3415191485/jobs/5684117326
The text was updated successfully, but these errors were encountered: