Skip to content

BSD: static network, set the mtu#894

Merged
TheRealFalcon merged 2 commits into
canonical:masterfrom
goneri:BSD-static-network-set-the-mtu_13881
May 14, 2021
Merged

BSD: static network, set the mtu#894
TheRealFalcon merged 2 commits into
canonical:masterfrom
goneri:BSD-static-network-set-the-mtu_13881

Conversation

@goneri
Copy link
Copy Markdown
Contributor

@goneri goneri commented May 10, 2021

In the case of a static network, we now set the MTU according to the
meta-data.

Copy link
Copy Markdown
Contributor

@TheRealFalcon TheRealFalcon left a comment

Choose a reason for hiding this comment

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

Thanks @goneri . Overall, the changes make sense. However, we're still supporting python 3.5 which doesn't support f-strings yet, so the Travis build is failing. Additionally, you have a flake8 issue you missed. Can you fix up the f-string and flake8 issue?

@TheRealFalcon TheRealFalcon self-assigned this May 10, 2021
@goneri
Copy link
Copy Markdown
Contributor Author

goneri commented May 10, 2021

Hi @TheRealFalcon, thanks for the feedback. I will adjust that.

@goneri goneri force-pushed the BSD-static-network-set-the-mtu_13881 branch from 7ce75b2 to 801fdd6 Compare May 11, 2021 18:04
@goneri goneri requested a review from TheRealFalcon May 11, 2021 18:49
@goneri goneri force-pushed the BSD-static-network-set-the-mtu_13881 branch from 801fdd6 to c23e1e9 Compare May 11, 2021 19:19
In the case of a static network, we now set the MTU according to the
meta-data.
@goneri goneri force-pushed the BSD-static-network-set-the-mtu_13881 branch from c23e1e9 to 061e2f1 Compare May 13, 2021 19:16
Copy link
Copy Markdown
Contributor

@TheRealFalcon TheRealFalcon left a comment

Choose a reason for hiding this comment

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

Thanks!

@TheRealFalcon TheRealFalcon merged commit 6fe1983 into canonical:master May 14, 2021
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.

2 participants