Skip to content
This repository has been archived by the owner on May 17, 2024. It is now read-only.

feat: configure default-network-opts mtu #229

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

tobiashuste
Copy link
Member

If an MTU lower than 1500 is present, docker-compose inside CI jobs can lead to issues as docker-compose always defaults to networks with an MTU of 1500. With this change a user should not be required to set the MTU on their own in docker-compose.
Requires at least Docker 24.

@tobiashuste tobiashuste requested a review from a team as a code owner February 16, 2024 06:46
@tobiashuste tobiashuste force-pushed the use-default-network-opts branch from 8bce7d7 to 77e4d13 Compare February 16, 2024 06:49
@tobiashuste tobiashuste added the enhancement New feature or request label Feb 16, 2024
@tobiashuste tobiashuste force-pushed the use-default-network-opts branch from 77e4d13 to 44fdaf6 Compare February 16, 2024 07:25
@tobiashuste tobiashuste requested a review from Normo February 16, 2024 07:25
@tobiashuste
Copy link
Member Author

@Normo Could you please check this. I hope this helps to resolve the docker-compose MTU related issues.

@tobiashuste tobiashuste added this to the v1.15.0 milestone Feb 16, 2024
If an MTU lower than 1500 is present, docker-compose inside CI jobs can
lead to issues as docker-compose always defaults to networks with an MTU
of 1500. With this change a user should not be required to set the MTU
on their own in docker-compose.
Requires at least Docker 24.
@tobiashuste tobiashuste force-pushed the use-default-network-opts branch from 44fdaf6 to ad2b268 Compare February 16, 2024 07:49
Copy link
Member

@Normo Normo left a comment

Choose a reason for hiding this comment

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

Thanks @tobiashuste. This looks reasonable to me 👌

@tobiashuste tobiashuste merged commit 0f9c8b6 into hifis-net:main Feb 16, 2024
8 checks passed
@tobiashuste tobiashuste deleted the use-default-network-opts branch February 16, 2024 12:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants