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

Make nplb build in default target #4057

Conversation

niranjanyardi
Copy link
Contributor

@niranjanyardi niranjanyardi commented Aug 28, 2024

b/356067446

We plan on releasing nplb evergreen binaries to partners. This change adds it to the default target which gets currently built on github.
Remove testonly variable for nplb and it's dependencies.
Fix missing deps in nplb

@niranjanyardi niranjanyardi requested review from a team and joeltine as code owners August 28, 2024 00:52
@niranjanyardi niranjanyardi changed the base branch from main to 25.lts.1+ August 28, 2024 00:52
@niranjanyardi niranjanyardi marked this pull request as draft August 28, 2024 00:55
@niranjanyardi niranjanyardi marked this pull request as ready for review August 28, 2024 01:00
@niranjanyardi niranjanyardi marked this pull request as draft August 28, 2024 01:00
@niranjanyardi niranjanyardi force-pushed the make_nplb_build_in_default_25lts branch from d4a643b to 80ce1ba Compare August 28, 2024 01:03
@niranjanyardi niranjanyardi requested review from jonastsai and oxve and removed request for a team, joeltine, johnxwork, jonastsai and oxve August 28, 2024 01:06
@niranjanyardi niranjanyardi force-pushed the make_nplb_build_in_default_25lts branch 2 times, most recently from f758a30 to 4a81f52 Compare August 30, 2024 18:40
@niranjanyardi niranjanyardi marked this pull request as ready for review August 30, 2024 18:43
@niranjanyardi niranjanyardi force-pushed the make_nplb_build_in_default_25lts branch from 4a81f52 to 32d51ce Compare August 30, 2024 18:45
@niranjanyardi niranjanyardi marked this pull request as draft September 3, 2024 23:01
Change-Id: I433a8b7d4bd8054600c74daa12180ada11a443bc
@niranjanyardi niranjanyardi force-pushed the make_nplb_build_in_default_25lts branch 3 times, most recently from 162b214 to dea5ca6 Compare September 4, 2024 04:54
@niranjanyardi niranjanyardi force-pushed the make_nplb_build_in_default_25lts branch 2 times, most recently from c09b264 to 4c82597 Compare September 4, 2024 06:00
@niranjanyardi niranjanyardi force-pushed the make_nplb_build_in_default_25lts branch 4 times, most recently from 86bacbe to 5c6ed35 Compare September 4, 2024 08:01
@niranjanyardi
Copy link
Contributor Author

not working on this atm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant