Skip to content

[Nexthop] Add ImageBuilder orchestration for FBOSS Image Builder#836

Closed
raghav-nexthop wants to merge 3 commits intofacebook:mainfrom
nexthop-ai:raghav.distro-cli-image-builder-part9
Closed

[Nexthop] Add ImageBuilder orchestration for FBOSS Image Builder#836
raghav-nexthop wants to merge 3 commits intofacebook:mainfrom
nexthop-ai:raghav.distro-cli-image-builder-part9

Conversation

@raghav-nexthop
Copy link
Contributor

Pre-submission checklist

  • I've ran the linters locally and fixed lint errors related to the files I modified in this PR. You can install the linters by running pip install -r requirements-dev.txt && pre-commit install
  • pre-commit run

Summary

Add ImageBuilder class for orchestrating component builds and image assembly:

  • ImageBuilder: Main orchestration class for building FBOSS images
  • Compression support for build artifacts (zstd)
  • Component build coordination

Test Plan

  • Test coverage for compression functionality
  • Updated build_test.py to use ImageBuilder

@meta-cla meta-cla bot added the CLA Signed label Jan 26, 2026
@raghav-nexthop raghav-nexthop force-pushed the raghav.distro-cli-image-builder-part9 branch 5 times, most recently from 9d1cab9 to 27203a2 Compare January 29, 2026 03:14
@facebook-github-bot
Copy link
Contributor

@raghav-nexthop has updated the pull request. You must reimport the pull request before landing.

@raghav-nexthop raghav-nexthop force-pushed the raghav.distro-cli-image-builder-part9 branch from 27203a2 to 6a2dd29 Compare February 3, 2026 02:03
@facebook-github-bot
Copy link
Contributor

@raghav-nexthop has updated the pull request. You must reimport the pull request before landing.

@raghav-nexthop raghav-nexthop force-pushed the raghav.distro-cli-image-builder-part9 branch from 6a2dd29 to 5210f7b Compare February 5, 2026 01:32
@facebook-github-bot
Copy link
Contributor

@raghav-nexthop has updated the pull request. You must reimport the pull request before landing.

@kevin645
Copy link

kevin645 commented Feb 5, 2026

can we rebase

@raghav-nexthop raghav-nexthop force-pushed the raghav.distro-cli-image-builder-part9 branch from 5210f7b to 0c44014 Compare February 5, 2026 19:15
@facebook-github-bot
Copy link
Contributor

@raghav-nexthop has updated the pull request. You must reimport the pull request before landing.

@raghav-nexthop
Copy link
Contributor Author

I've rebased the entire PR stack.

@raghav-nexthop raghav-nexthop force-pushed the raghav.distro-cli-image-builder-part9 branch from 0c44014 to a0b2434 Compare February 5, 2026 22:41
@facebook-github-bot
Copy link
Contributor

@raghav-nexthop has updated the pull request. You must reimport the pull request before landing.

@raghav-nexthop raghav-nexthop force-pushed the raghav.distro-cli-image-builder-part9 branch from a0b2434 to 37338e8 Compare February 6, 2026 22:25
@facebook-github-bot
Copy link
Contributor

@raghav-nexthop has updated the pull request. You must reimport the pull request before landing.

@raghav-nexthop raghav-nexthop force-pushed the raghav.distro-cli-image-builder-part9 branch from 37338e8 to 6f62b36 Compare February 7, 2026 00:33
@facebook-github-bot
Copy link
Contributor

@raghav-nexthop has updated the pull request. You must reimport the pull request before landing.

@meta-codesync
Copy link

meta-codesync bot commented Feb 7, 2026

@kevin645 has imported this pull request. If you are a Meta employee, you can view this in D91737557.

@raghav-nexthop raghav-nexthop force-pushed the raghav.distro-cli-image-builder-part9 branch 2 times, most recently from 8ccfe84 to 669a50e Compare February 9, 2026 19:06
@facebook-github-bot
Copy link
Contributor

@raghav-nexthop has updated the pull request. You must reimport the pull request before landing.

@raghav-nexthop raghav-nexthop force-pushed the raghav.distro-cli-image-builder-part9 branch from 669a50e to 74e993e Compare February 10, 2026 02:26
@facebook-github-bot
Copy link
Contributor

@raghav-nexthop has updated the pull request. You must reimport the pull request before landing.

@raghav-nexthop raghav-nexthop force-pushed the raghav.distro-cli-image-builder-part9 branch from 74e993e to 25a40ad Compare February 10, 2026 02:34
@facebook-github-bot
Copy link
Contributor

@raghav-nexthop has updated the pull request. You must reimport the pull request before landing.

@raghav-nexthop raghav-nexthop force-pushed the raghav.distro-cli-image-builder-part9 branch from 25a40ad to 6f32566 Compare February 10, 2026 02:39
@facebook-github-bot
Copy link
Contributor

@raghav-nexthop has updated the pull request. You must reimport the pull request before landing.

Add abstract build component framework for managing build operations.

- Implement AbstractComponent base class for build components
- Add component lifecycle management (prepare, build, extract)
- Integrate with artifact store, download, and execute modules
- Enable extensible component-based build architecture

Tests utilizing the above infrastructure will be added when component build supports are included.
Add build entrypoint orchestration for component-based builds.

- Implement build entrypoint for coordinating component build workflows
- Add support for build configuration and execution management
- Include comprehensive unit tests for entrypoint functionality
Add ImageBuilder class for orchestrating component builds and image assembly.

- ImageBuilder: Main orchestration class for building FBOSS images
- Compression support for build artifacts (zstd)
- Component build coordination
- Test coverage for compression functionality
- Updated build_test.py to use ImageBuilder
@raghav-nexthop raghav-nexthop force-pushed the raghav.distro-cli-image-builder-part9 branch from 6f32566 to 1fed9eb Compare February 10, 2026 02:46
@facebook-github-bot
Copy link
Contributor

@raghav-nexthop has updated the pull request. You must reimport the pull request before landing.

@meta-codesync
Copy link

meta-codesync bot commented Feb 12, 2026

@kevin645 merged this pull request in ddc67cc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments