Merge ~adam-collard/maas:bulk-update-lp1839788 into maas:master

Proposed by Adam Collard
Status: Merged
Approved by: Adam Collard
Approved revision: e1cc7b3cd183248ba6b4b5d0d087e67a642a6177
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~adam-collard/maas:bulk-update-lp1839788
Merge into: maas:master
Diff against target: 126 lines (+87/-1)
2 files modified
src/maasserver/websockets/handlers/config.py (+35/-1)
src/maasserver/websockets/handlers/tests/test_config.py (+52/-0)
Reviewer Review Type Date Requested Status
Christian Grabowski Approve
MAAS Lander Approve
Review via email: mp+452470@code.staging.launchpad.net

Commit message

feat(websocket): add config.bulk_update

LP:1839788

Description of the change

config.bulk_update is a new websocket endpoint which takes one named parameter "items" which is a mapping of config key (string) to config value

on success, it will return the mapping (keys to new values)
on any errors, it will return an error mapping of failed keys to error strings.

To post a comment you must log in.
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b bulk-update-lp1839788 lp:~adam-collard/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: SUCCESS
COMMIT: e1cc7b3cd183248ba6b4b5d0d087e67a642a6177

review: Approve
Revision history for this message
Christian Grabowski (cgrabowski) wrote :

+1

review: Approve

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches