Skip to content
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

Replace checked-in ZIPs with dynamic dependencies #415

Closed
Tracked by #1750
dblock opened this issue Apr 18, 2022 · 6 comments
Closed
Tracked by #1750

Replace checked-in ZIPs with dynamic dependencies #415

dblock opened this issue Apr 18, 2022 · 6 comments
Assignees
Labels
bug Something isn't working untriaged v2.0.0

Comments

@dblock
Copy link
Member

dblock commented Apr 18, 2022

Coming from opensearch-project/opensearch-build#1750, replace checked in ZIP(s) with dynamic dependencies.

./notifications/notifications/notifications/src/test/resources/notifications-core/opensearch-notifications-core-2.0.0.0-SNAPSHOT.zip
@dblock
Copy link
Member Author

dblock commented Apr 19, 2022

@dbbaughe Can you please help find an owner for this, it's part of 2.0 campaigns.

@dbbaughe dbbaughe assigned qreshi and unassigned dbbaughe Apr 19, 2022
@dbbaughe
Copy link
Contributor

@dblock Don't have bandwidth this week - added Mohammad who is leading this, he should be able to help find someone.

@qreshi
Copy link
Contributor

qreshi commented Apr 19, 2022

Since notifications-core is a local subproject, I'm guessing we'll want to use the locally built zip and not an uploaded artifact since it might not reflect local changes?

@elfisher
Copy link

Is this required for 2.0?

@qreshi
Copy link
Contributor

qreshi commented Apr 25, 2022

It's a 2.0 campaign but it's build specific so if we've missed the window on it, it shouldn't be a big deal.

@sbcd90
Copy link
Collaborator

sbcd90 commented May 5, 2022

this issue is fixed now.closing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working untriaged v2.0.0
Projects
None yet
Development

No branches or pull requests

5 participants