-
Notifications
You must be signed in to change notification settings - Fork 284
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
Updated manifests for 2.0 release #1853
Updated manifests for 2.0 release #1853
Conversation
31661f8
to
14671a0
Compare
Looks like k-nn doesn't support qualifiers. |
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.
Hold on this PR for a bit as there is some bugs in the current setup.
Ok. |
@peterzhuamazon any rough eta on this ? thanks |
@peterzhuamazon Any update? |
Just wait for the obs PR to be merged then we can merge this manifest change. |
5628b4f
to
17c6619
Compare
Signed-off-by: vamsi-amazon <[email protected]>
@vamsi-amazon please resolve conflicts to this PR. |
17c6619
to
dbe21fa
Compare
Signed-off-by: vamsi-amazon <[email protected]>
@peterzhuamazon Resolved conflicts. |
@vamsi-amazon, I think plugins can do a bump but I'm not positive if it compiled locally. We might have to revert the OSD changes. |
Signed-off-by: vamsi-amazon [email protected]
Description
Adding builds for sql, observability and reporting plugins.
Issues Resolved
[List any issues this PR will resolve]
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.