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

Updated Dockerfile to support multi-architecture builds #1589

Closed
wants to merge 4 commits into from

Conversation

l50
Copy link

@l50 l50 commented Mar 7, 2024

I wanted to run sliver in my ARM-based k8s cluster and quickly determined that
some changes would need to be made to facilitate that. This PR contains
the changes that were necessary to build an ARM64-based sliver container image. It can easily support additional architectures via the docker buildx build command (see Dockerfile).

Added:

  • Multi-arch build instructions for production and unit testing stages.
  • Conditional installation of gcc-multilib based on architecture in test and production stages.

Changed:

  • Removed explicit platform specification in FROM directives to support multi-architecture builds.
  • Refactored installation commands to include architecture-specific conditions.
  • Updated production stage to create additional directories in the sliver user's home directory.

…c installations.

**Added:**

- Multi-arch build instructions for production and unit testing stages.
- Conditional installation of `gcc-multilib` based on architecture in `test`
  and `production` stages.

**Changed:**

- Removed explicit platform specification in `FROM` directives to support
  multi-architecture builds.
- Refactored installation commands to include architecture-specific conditions.
- Updated `production` stage to create additional directories in the `sliver`
  user's home directory.
@l50 l50 requested a review from a team as a code owner March 7, 2024 03:43
@moloch--
Copy link
Member

This should be fixed with #1652 and #1654

@moloch-- moloch-- closed this Apr 30, 2024
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