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.

Sign up for GitHub

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: [M3-6510] - Modifying Configuration Profile in Cloud breaks with VLANs #9053

Merged
merged 12 commits into from
Apr 26, 2023
Merged

fix: [M3-6510] - Modifying Configuration Profile in Cloud breaks with VLANs #9053

Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
7b07a51
refactor: [M3-6307] - MUIv5 Migration - SRC > Components > CopyTooltip
cpathipa Apr 20, 2023
7bf423f
Update packages/manager/src/components/CopyTooltip/CopyTooltip.storie…
cpathipa Apr 21, 2023
968ba1d
Merge remote-tracking branch 'origin/develop' into M3-6307
cpathipa Apr 25, 2023
7d35557
Update CHANGELOG.md
cpathipa Apr 25, 2023
d4b2e1b
Merge branch 'linode:develop' into M3-6307
cpathipa Apr 25, 2023
dced1f6
fix: [M3-6510] - modifying Configuration Profile in Cloud breaks with…
cpathipa Apr 25, 2023
b403986
Update CHANGELOG.md
cpathipa Apr 25, 2023
2c9c612
Update CHANGELOG.md
cpathipa Apr 25, 2023
ae03928
Update CHANGELOG.md
cpathipa Apr 26, 2023
b7cf073
Add missing type id for Interface
cpathipa Apr 26, 2023
202317c
Merge branch 'M3-6510' of github.com:cpathipa/manager into M3-6510
cpathipa Apr 26, 2023
c7bb561
PR feedback
cpathipa Apr 26, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/) and this p
- Event entities should only be linked for true labels
- Radio button hover effect #9031
- Prevent form submission unless action was taken (IP transfer & IP sharing modals) #5976
Inability to edit and save Linode Configurations #9053
cpathipa marked this conversation as resolved.
Show resolved Hide resolved

### Tech Stories:
- MUIv5 Migration - Components > CircleProgress #9028
Expand Down
7 changes: 6 additions & 1 deletion packages/manager/src/features/linodes/LinodesDetail/LinodeSettings/LinodeConfigDialog.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -425,6 +425,11 @@ const LinodeConfigDialog: React.FC<CombinedProps> = (props) => {
(thisOption) => thisOption.value === config?.root_device
)
);
const configInterfaces = config?.interfaces?.map(
cpathipa marked this conversation as resolved.
Show resolved Hide resolved
({ ipam_address, label, purpose }) => {
return { ipam_address, label, purpose };
}
);
resetForm({
values: {
useCustomRoot: isUsingCustomRoot(config.root_device),
Expand All @@ -438,7 +443,7 @@ const LinodeConfigDialog: React.FC<CombinedProps> = (props) => {
virt_mode: config.virt_mode,
helpers: config.helpers,
root_device: config.root_device,
interfaces: interfacesToState(config.interfaces),
interfaces: interfacesToState(configInterfaces),
cpathipa marked this conversation as resolved.
Show resolved Hide resolved
setMemoryLimit:
config.memory_limit !== 0 ? 'set_limit' : 'no_limit',
},
Expand Down