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

fix(legacy): stop setting gateway on gateway-less subnets (#1860) #2009

Merged
merged 4 commits into from
Dec 17, 2020

Conversation

vtapia
Copy link
Contributor

@vtapia vtapia commented Dec 15, 2020

Done

Unify the behavior for both standard and relayed networks, populating the placeholder with the suggested gateway without setting the IP when the subnet does not have one configured.

QA

  • Configure a subnet without gateway
  • Go to related VLAN and configure DHCP
  • Verify that the gateway field has remained unchanged

Fixes

Fixes: #1860 .

Launchpad issue

lp#1903586

Backports

2.8 and 2.7 are affected too

@github-actions github-actions bot added the Legacy 🧟‍♂️ this issue refers to the legacy angularjs app. label Dec 15, 2020
Copy link
Contributor

@Caleb-Ellis Caleb-Ellis left a comment

Choose a reason for hiding this comment

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

Looks good! Just need to update the prepares provideDHCPAction on actionOptionChanged and populates suggested gateway test in src/app/controllers/tests/test_vlan_details.js

Copy link
Contributor

@Caleb-Ellis Caleb-Ellis left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thanks @vtapia

@Caleb-Ellis Caleb-Ellis merged commit 5a74d72 into canonical:master Dec 17, 2020
Caleb-Ellis pushed a commit that referenced this pull request Dec 17, 2020
)

* fix(legacy): stop setting gateway on gateway-less subnets (#1860)

Co-authored-by: Caleb Ellis <[email protected]>
Caleb-Ellis pushed a commit that referenced this pull request Dec 17, 2020
)

* fix(legacy): stop setting gateway on gateway-less subnets (#1860)

Co-authored-by: Caleb Ellis <[email protected]>
Caleb-Ellis pushed a commit that referenced this pull request Dec 17, 2020
)

* fix(legacy): stop setting gateway on gateway-less subnets (#1860)

Co-authored-by: Caleb Ellis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Legacy 🧟‍♂️ this issue refers to the legacy angularjs app. Review: Code +1 Review: QA +1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

gateway reappears after reconfiguring dhcp
2 participants