Skip to content

feat(ci): add an automated build for network-interfaces that can be copied into a ros2-modulo-control image#39

Merged
LouisBrunner merged 3 commits intodevelopfrom
feature/ci
May 30, 2023
Merged

feat(ci): add an automated build for network-interfaces that can be copied into a ros2-modulo-control image#39
LouisBrunner merged 3 commits intodevelopfrom
feature/ci

Conversation

@LouisBrunner
Copy link
Copy Markdown
Contributor

Description

Closes #38

Example build: https://github.com/aica-technology/network-interfaces/actions/runs/5059684431/jobs/9081636211

  • Bundle all 3 libraries in one image (Python, network-interfaces, communication-interfaces)
  • Final target is a partial installation that can be copied as COPY --from=net-iface / / to install zmq, network-interfaces, communication-interfaces, etc which is convenient for build and production images
  • Uses a Dockerfile similar to dynamic-components to snapshot apt packages to keep only what needs to be installed
  • No changes to any existing scripts/workflows

Review guidelines

Estimated Time of Review: 7 minutes

Related issues

Blocks:

hardware-collection PR

@LouisBrunner LouisBrunner requested review from domire8 and eeberhard May 23, 2023 16:25
@LouisBrunner LouisBrunner self-assigned this May 23, 2023
Comment thread CMakeLists.txt
@@ -0,0 +1,19 @@
cmake_minimum_required(VERSION 3.15)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

If you want, you can for now just focus on the network_interfaces. communication_interfaces is meant to replace it, but I'm not there yet. So technically, they are not meant to coexist. But up to you

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 isn't much effort to support both and I think it makes sense if communication_interfaces is the future.

@LouisBrunner LouisBrunner merged commit c2600a1 into develop May 30, 2023
@LouisBrunner LouisBrunner deleted the feature/ci branch May 30, 2023 08:23
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