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

feat(interop-tests): build test binary inside container #3441

Merged
merged 3 commits into from
Feb 10, 2023

Conversation

MarcoPolo
Copy link
Contributor

@MarcoPolo MarcoPolo commented Feb 9, 2023

Description

Building inside the container allows Windows and MacOS users to also build this binary. Thanks to a new feature from docker, --mount=type=cache, rebuilding layers is fast without any additional hacks.

Notes

continuation of #3439 for master. @thomaseizinger will backport these tests to older versions once we merge this.

Copy link
Contributor

@thomaseizinger thomaseizinger left a comment

Choose a reason for hiding this comment

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

LGTM apart from the "either" related changes!

@thomaseizinger thomaseizinger changed the title Update interop-tests feat(interop-tests): build test binary inside container Feb 10, 2023
@thomaseizinger
Copy link
Contributor

thomaseizinger commented Feb 10, 2023

@Mergifyio backport rust-v050-testplan rust-v049-testplan rust-v048-testplan

@mergify
Copy link
Contributor

mergify bot commented Feb 10, 2023

backport rust-v050-testplan rust-v049-testplan rust-v048-testplan

❌ No backport have been created

@thomaseizinger
Copy link
Contributor

thomaseizinger commented Feb 10, 2023

@thomaseizinger will backport these tests to older versions once we merge this.

Minor correction here, mergify will do it (hopefully)

@mergify mergify bot merged commit eeca244 into master Feb 10, 2023
@mergify mergify bot deleted the marco/rust-testplan-master branch February 10, 2023 06:14
mergify bot pushed a commit that referenced this pull request Feb 10, 2023
Building inside the container allows Windows and MacOS users to also build this binary. Thanks to a new feature from docker, `--mount=type=cache`, rebuilding layers is fast without any additional hacks.

(cherry picked from commit eeca244)

# Conflicts:
#	.dockerignore
@thomaseizinger
Copy link
Contributor

@Mergifyio backport rust-v0490-testplan

@mergify
Copy link
Contributor

mergify bot commented Feb 10, 2023

backport rust-v0490-testplan

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Feb 10, 2023
Building inside the container allows Windows and MacOS users to also build this binary. Thanks to a new feature from docker, `--mount=type=cache`, rebuilding layers is fast without any additional hacks.

(cherry picked from commit eeca244)

# Conflicts:
#	.dockerignore
#	Cargo.toml
@thomaseizinger
Copy link
Contributor

@Mergifyio backport rust-v0480-testplan

@mergify
Copy link
Contributor

mergify bot commented Feb 10, 2023

backport rust-v0480-testplan

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Feb 10, 2023
Building inside the container allows Windows and MacOS users to also build this binary. Thanks to a new feature from docker, `--mount=type=cache`, rebuilding layers is fast without any additional hacks.

(cherry picked from commit eeca244)

# Conflicts:
#	.dockerignore
#	Cargo.toml
thomaseizinger pushed a commit that referenced this pull request Feb 10, 2023
…) (#3442)

Co-authored-by: Marco Munizaga <git@marcopolo.io>
thomaseizinger pushed a commit that referenced this pull request Feb 10, 2023
…) (#3444)

Co-authored-by: Marco Munizaga <git@marcopolo.io>
thomaseizinger pushed a commit that referenced this pull request Feb 10, 2023
…) (#3443)

Co-authored-by: Marco Munizaga <git@marcopolo.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants