docs: Add HLD for Bazel migration#2396
Draft
blorente wants to merge 12 commits into
Draft
Conversation
Collaborator
|
/azp run |
|
No pipelines are associated with this pull request. |
Collaborator
|
/azp run |
|
No pipelines are associated with this pull request. |
This was referenced Jun 20, 2026
Closed
bocon13
reviewed
Jun 23, 2026
|
|
||
| // TODO BL: Should we include a table here? | ||
|
|
||
| #### 4.b Migration |
Collaborator
There was a problem hiding this comment.
include an example of changes required to *mk to opt in
Author
There was a problem hiding this comment.
@bocon13 I have included a more thorough example of the changes in section 7.a (formerly 7.h, but I moved it up because I thought it would be more important than anything else in section 7). Please let me know what you think!
Collaborator
|
/azp run |
|
No pipelines are associated with this pull request. |
Collaborator
|
/azp run |
|
No pipelines are associated with this pull request. |
Collaborator
|
/azp run |
|
No pipelines are associated with this pull request. |
blorente
changed the title
Collaborator
|
/azp run |
|
No pipelines are associated with this pull request. |
bocon13
reviewed
Jun 29, 2026
| We have already implemented large parts of this document as proofs of concept. | ||
|
|
||
| Please see these publications about our work: | ||
| - Talk from OCP EMEA Summit 2026: https://www.youtube.com/watch?v=uSKCNDWuXjc |
bocon13
reviewed
Jun 29, 2026
bocon13
reviewed
Jun 29, 2026
bocon13
reviewed
Jun 29, 2026
|
|
||
| The community can use this period to experiment with Bazel, adopt it into their own builds, and generally gather information on whether this is a net benefit. | ||
|
|
||
| After the November release, the community will face a decision point: Should we adopt Bazel fully? |
Collaborator
There was a problem hiding this comment.
Should Bazel become a first class build system?
What does that mean?
- bazel may be the only option for some components (already true today for components like p4rt)
- bazel will handle more of dependency graph for these components (sonic deps might also need to be converted to bazel e.g. sonic-utils, py-common)
- reasonable build performance requires basic bazel infra (i.e. local cache, remote cache)
Collaborator
There was a problem hiding this comment.
reframe phase 1 around trial of build and bazel infra
bocon13
reviewed
Jun 29, 2026
bocon13
reviewed
Jun 29, 2026
bocon13
reviewed
Jun 29, 2026
bocon13
reviewed
Jun 29, 2026
bocon13
reviewed
Jun 29, 2026
bocon13
reviewed
Jun 29, 2026
bocon13
reviewed
Jun 29, 2026
bocon13
reviewed
Jun 29, 2026
Collaborator
|
/azp run |
|
No pipelines are associated with this pull request. |
blorente
force-pushed
the
blorente/add-bazel-hdl
branch
from
66cba30 to
a59a8a8
Compare
Collaborator
|
/azp run |
|
No pipelines are associated with this pull request. |
Collaborator
|
/azp run |
|
No pipelines are associated with this pull request. |
Signed-off-by: Borja Lorente <blorente.me@gmail.com>
Signed-off-by: Borja Lorente <blorente.me@gmail.com>
Signed-off-by: Borja Lorente <blorente.me@gmail.com>
Signed-off-by: Borja Lorente <blorente.me@gmail.com>
Signed-off-by: Borja Lorente <blorente.me@gmail.com>
Signed-off-by: Borja Lorente <blorente.me@gmail.com>
Signed-off-by: Borja Lorente <blorente.me@gmail.com>
… sections Signed-off-by: Borja Lorente <blorente.me@gmail.com>
Signed-off-by: Borja Lorente <blorente.me@gmail.com>
Signed-off-by: Borja Lorente <blorente.me@gmail.com>
Signed-off-by: Borja Lorente <blorente.me@gmail.com>
blorente
force-pushed
the
blorente/add-bazel-hdl
branch
from
3c6c7ce to
fdd5055
Compare
Collaborator
|
/azp run |
|
No pipelines are associated with this pull request. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add HLD describing a potential migration to Bazel, including the changes necessary in the current build so that the migration is as un-disruptive as possible.