Skip to content

[AVS-lite] Integrate docker-sonic-alpinevs with build files - #40

Merged
sonikajindal merged 1 commit into
sonic-net:masterfrom
sreeiyer-1:docker_update
Aug 17, 2026
Merged

[AVS-lite] Integrate docker-sonic-alpinevs with build files#40
sonikajindal merged 1 commit into
sonic-net:masterfrom
sreeiyer-1:docker_update

Conversation

@sreeiyer-1

Copy link
Copy Markdown
Contributor

Add build support for docker-sonic-alpinevs

These changes are on top of #37 Some of the things added are :

  1. Support for 'make target/docker-sonic-alpinevs.gz'
  2. Move pkt-handler and syncd to trixie
  3. pkt-handler made a seperate supervisor task
  4. Add check for genetlink support
  5. Always build pkt-handler to avoid the risk of missing changes
  6. Instead of copying the alpinevs-init.sh from src/services/init, using a fresh one for alpinevs-lite. Continue sharing the alpinevs-config.sh with sonic-alpine
  7. New topology file
  8. Lucius endpoint configurable through an environment variable
  9. Moved platform.json to the correct location. Added platform.json and hwsku.json for sonic-alpine, which hadn't been done earlier.

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

check_process_any pkt-handler "pkt-handler"

if copp_uses_genetlink; then
info "checking optional generic netlink prerequisite for CoPP GENETLINK hostifs"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why "optional"?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is required if the if-condition is true.

copp_uses_genetlink() checks if genetlink is required based on what is in copp_cfg.json. The if condition is true if copp_cfg refers to genetlink, and at that point genetlink is not optional. Will change it now.

Comment thread src/libsai-grpc/entrypoint.cc
sonikajindal
sonikajindal previously approved these changes Aug 14, 2026
@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

Signed-off-by: Sree Iyer <sbiyer@google.com>
@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@sreeiyer-1
sreeiyer-1 marked this pull request as ready for review August 15, 2026 00:49
@sreeiyer-1 sreeiyer-1 changed the title Integrate docker-sonic-vs with build files Integrate docker-sonic-alpinevs with build files Aug 15, 2026
@sonikajindal
sonikajindal merged commit 181cde9 into sonic-net:master Aug 17, 2026
5 checks passed
@sreeiyer-1 sreeiyer-1 changed the title Integrate docker-sonic-alpinevs with build files [AVS-lite] Integrate docker-sonic-alpinevs with build files Sep 5, 2026
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.

3 participants