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

Please document MTU settings #276

Open
smnbbrv opened this issue May 1, 2020 · 4 comments
Open

Please document MTU settings #276

smnbbrv opened this issue May 1, 2020 · 4 comments

Comments

@smnbbrv
Copy link

smnbbrv commented May 1, 2020

Hi there,

thank you for a great plugin!

Today I spent several hours with a quite specific issue which I still cannot fully understand. Probably it is about some specific config of the network we use for our cluster.

Actors:

  • Kubernetes
  • Drone CI & Drone Docker (this) plugin
  • Alpine image

Actually, what happens is exactly repeating this alpine issue: when I run apk update as one of the dockerfile's run command it freezes.

So, decreasing the MTU (I set 1000) solved the issue, however I need to go to the source code to find out there is a setting like that:

kind: pipeline
type: kubernetes
name: default

steps:
  - name: dockerize
    image: plugins/docker
    settings:
      ...
      mtu: 1000

Would be super if the setting gets documented. Thank you!

@tboerger
Copy link

I'm not sure about the documentation for that, there is no advanced configuration section within the plugin index, and the majority of users don't have to care about these options at all.

@BigeYoung
Copy link

Hello, dear developer.

In fact, many people are bothered by this problem:
https://discourse.drone.io/t/drone-on-kubernetes-with-docker-plugin-strange-network-problem/8420/5
https://discourse.drone.io/t/solved-issue-kubernetes-runner-cert-validation-issues-with-plugin-docker/7712/2
https://liejuntao001.medium.com/fix-docker-in-docker-network-issue-in-kubernetes-cc18c229d9e5

It seems that running docker in docker in Kubernetes is not so smooth.

So if we can document the MTU settings, it will be able to help many people save a lot of time!

@schaze
Copy link

schaze commented May 24, 2021

After over a year of intermittent issues with docker-in-docker builds on drone in kubernetes this finally helped. Please add a line of documentation for this. It will help A LOT of people!

@rule88
Copy link

rule88 commented Jun 8, 2021

This indeed should be part of documentation, especially as the resulting behavior is hard to debug from a developer point of view. In fact, the whole setup (kubernetes + drone) should result in an environment in which developers don't have infrastructure as a concern.

Documenting this setting with some reference to/over MTU sizes (and the effect if not set properly) would hopefully prevent others needing to make some deep dives in networking.

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

No branches or pull requests

5 participants