Merge lp:~blake-rouse/maas/fix-1509417 into lp:~maas-committers/maas/trunk

[Blake Rouse]

Enable modification of storage tags on physical and virtual block devices in the available section.

This does not fully fix the bug. Has Rich still needs to fix the styling, so landing this branch will not mark the bug "Fix Committed".

[Lee Trager (ltrager)]

LGTM!

My only thought is that we should allow viewing and editing tags in either the filesystem or used disks and partitions section. Its a bit of a pain to have to make a disk available just to be able to view and edit its tags.

review: Approve

[Blake Rouse (blake-rouse)]

Thanks for the review.

Yeah I agree. I will be enabling that, just in a different branch.

[Mike Pontillo (mpontillo)]

Looks pretty good to me; I may have spotted an issue or two that you might want to look at. Take a look below and let me know what you think.

review: Approve

[Blake Rouse (blake-rouse)]

Thanks for the review. I will add the request comment.

[MAAS Lander (maas-lander)]

The attempt to merge lp:~blake-rouse/maas/fix-1509417 into lp:maas failed. Below is the output from the failed tests.

Get:1 http://security.ubuntu.com trusty-security InRelease [64.4 kB]
Ign http://nova.clouds.archive.ubuntu.com trusty InRelease
Get:2 http://nova.clouds.archive.ubuntu.com trusty-updates InRelease [64.4 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty Release.gpg
Hit http://nova.clouds.archive.ubuntu.com trusty Release
Get:3 http://security.ubuntu.com trusty-security/main Sources [98.0 kB]
Get:4 http://security.ubuntu.com trusty-security/universe Sources [31.0 kB]
Get:5 http://nova.clouds.archive.ubuntu.com trusty-updates/main Sources [241 kB]
Get:6 http://security.ubuntu.com trusty-security/main amd64 Packages [357 kB]
Get:7 http://nova.clouds.archive.ubuntu.com trusty-updates/universe Sources [143 kB]
Get:8 http://security.ubuntu.com trusty-security/universe amd64 Packages [117 kB]
Get:9 http://nova.clouds.archive.ubuntu.com trusty-updates/main amd64 Packages [637 kB]
Get:10 http://security.ubuntu.com trusty-security/main Translation-en [194 kB]
Get:11 http://security.ubuntu.com trusty-security/universe Translation-en [68.4 kB]
Get:12 http://nova.clouds.archive.ubuntu.com trusty-updates/universe amd64 Packages [326 kB]
Get:13 http://nova.clouds.archive.ubuntu.com trusty-updates/main Translation-en [309 kB]
Get:14 http://nova.clouds.archive.ubuntu.com trusty-updates/universe Translation-en [172 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty/main Sources
Hit http://nova.clouds.archive.ubuntu.com trusty/universe Sources
Hit http://nova.clouds.archive.ubuntu.com trusty/main amd64 Packages
Hit http://nova.clouds.archive.ubuntu.com trusty/universe amd64 Packages
Hit http://nova.clouds.archive.ubuntu.com trusty/main Translation-en
Hit http://nova.clouds.archive.ubuntu.com trusty/universe Translation-en
Ign http://nova.clouds.archive.ubuntu.com trusty/main Translation-en_US
Ign http://nova.clouds.archive.ubuntu.com trusty/universe Translation-en_US
Fetched 2,823 kB in 4s (652 kB/s)
Reading package lists...
sudo DEBIAN_FRONTEND=noninteractive apt-get -y \
     --no-install-recommends install apache2 authbind bind9 bind9utils build-essential bzr-builddeb chromium-browser chromium-chromedriver curl daemontools debhelper dh-apport dh-systemd distro-info dnsutils firefox freeipmi-tools git gjs ipython isc-dhcp-common libjs-angularjs libjs-jquery libjs-jquery-hotkeys libjs-yui3-full libjs-yui3-min libpq-dev make nodejs-legacy npm pep8 phantomjs postgresql pyflakes python-apt python-bson python-bzrlib python-convoy python-coverage python-crochet python-cssselect python-curtin python-dev python-distro-info python-django python-django-piston python-django-south python-djorm-ext-pgarray python-docutils python-extras python-fixtures python-flake8 python-formencode python-hivex python-httplib2 python-jinja2 python-jsonschema python-lxml python-mock python-netaddr python-netifaces python-nose python-oauth python-openssl python-paramiko python-pexpect python-pip python-pocket-lint python-psycopg2 python-pyinotify python-pyparsing python-seamicroclient python-simplejson python-simplestreams python-sphinx python-subunit python-tempita py...

[Mike Pontillo (mpontillo)]

Digging through that log is annoying. But I found this:

maasserver.tests.test_forms_interface.BondInterfaceFormTest.test__creates_bond_interface_with_parent_mac_address ... ERROR
maasserver.tests.test_populate_tags.TestPopulateTags.test__calls_do_populate_tags_with_clusters ... FAIL

======================================================================
ERROR: maasserver.tests.test_forms_interface.BondInterfaceFormTest.test__creates_bond_interface_with_parent_mac_address
----------------------------------------------------------------------
_StringException: Traceback (most recent call last):
  File "/tmp/tarmac/branch.IXGl4c/src/maastesting/runtest.py", line 68, in _run_user
    result = function(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/testtools/testcase.py", line 616, in _run_test_method
    return self._get_test_method()()
  File "/tmp/tarmac/branch.IXGl4c/src/maasserver/tests/test_forms_interface.py", line 516, in test__creates_bond_interface_with_parent_mac_address
    vlan = factory.make_VLAN(vid=10)
  File "/tmp/tarmac/branch.IXGl4c/src/maasserver/testing/factory.py", line 739, in make_VLAN
    vlan.save()
  File "/tmp/tarmac/branch.IXGl4c/src/maasserver/models/cleansave.py", line 37, in save
    self.full_clean()
  File "/usr/lib/python2.7/dist-packages/django/db/models/base.py", line 950, in full_clean
    raise ValidationError(errors)
ValidationError: {'__all__': [u'VLAN with this Vid and Fabric already exists.']}

======================================================================
FAIL: maasserver.tests.test_populate_tags.TestPopulateTags.test__calls_do_populate_tags_with_clusters
----------------------------------------------------------------------
_StringException: Traceback (most recent call last):
  File "/tmp/tarmac/branch.IXGl4c/src/maastesting/runtest.py", line 68, in _run_user
    result = function(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/testtools/testcase.py", line 616, in _run_test_method
    return self._get_test_method()()
  File "/tmp/tarmac/branch.IXGl4c/src/maasserver/tests/test_populate_tags.py", line 244, in test__calls_do_populate_tags_with_clusters
    populate_tags_module.tag_nsmap))
  File "/usr/lib/python2.7/dist-packages/testtools/testcase.py", line 406, in assertThat
    raise mismatch_error
MismatchError: Expected call: mock(((u'd3892b66-7db1-11e5-a501-fa163e312456', u'cluster-kKQCvD', u'nSaYxW5vfF7Ch4hpS6:QyP7Kw8baU347hG7r8:2yWUpBW23SLrmhzta49NND73Rtks5ndm'), (u'd38bb390-7db1-11e5-a501-fa163e312456', u'cluster-GMo8FL', u'N76yENgGPFVjbzXhSE:H3MNtwh824fB9ezxj6:N4XHy6nELjjNC26yZEvBMjdc3RmqeXJu'), (u'd38d73f6-7db1-11e5-a501-fa163e312456', u'cluster-bgk6Vv', u'dReXSBYBrD82JkcCpv:wApPKrTsbp3rsBLAfj:Eu77CdEkfYg8SJ4NZeGWzuk8EUdS8hQR')), u'tag-6TAjrG', u'//node', {u'lldp': u'lldp', u'lshw': u'lshw'})
Actual call: mock(((u'd38d73f6-7db1-11e5-a501-fa163e312456', u'cluster-bgk6Vv', u'dReXSBYBrD82JkcCpv:wApPKrTsbp3rsBLAfj:Eu77CdEkfYg8SJ4NZeGWzuk8EUdS8hQR'), (u'd3892b66-7db1-11e5-a501-fa163e312456', u'cluster-kKQCvD', u'nSaYxW5vfF7Ch4hpS6:QyP7Kw8baU347hG7r8:2yWUpBW23SLrmhzta49NND73Rtks5ndm'), (u'd38bb390-7db1-11e5-a501-fa163e312456', u'cluster-GMo8FL', u'N76yENgGPFVjbzXhSE:H3MN...

[MAAS Lander (maas-lander)]

The attempt to merge lp:~blake-rouse/maas/fix-1509417 into lp:maas failed. Below is the output from the failed tests.

Get:1 http://security.ubuntu.com trusty-security InRelease [64.4 kB]
Ign http://nova.clouds.archive.ubuntu.com trusty InRelease
Get:2 http://nova.clouds.archive.ubuntu.com trusty-updates InRelease [64.4 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty Release.gpg
Hit http://nova.clouds.archive.ubuntu.com trusty Release
Get:3 http://security.ubuntu.com trusty-security/main Sources [98.0 kB]
Get:4 http://security.ubuntu.com trusty-security/universe Sources [31.0 kB]
Get:5 http://nova.clouds.archive.ubuntu.com trusty-updates/main Sources [241 kB]
Get:6 http://security.ubuntu.com trusty-security/main amd64 Packages [357 kB]
Get:7 http://nova.clouds.archive.ubuntu.com trusty-updates/universe Sources [143 kB]
Get:8 http://nova.clouds.archive.ubuntu.com trusty-updates/main amd64 Packages [637 kB]
Get:9 http://security.ubuntu.com trusty-security/universe amd64 Packages [117 kB]
Hit http://security.ubuntu.com trusty-security/main Translation-en
Hit http://security.ubuntu.com trusty-security/universe Translation-en
Get:10 http://nova.clouds.archive.ubuntu.com trusty-updates/universe amd64 Packages [326 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty-updates/main Translation-en
Hit http://nova.clouds.archive.ubuntu.com trusty-updates/universe Translation-en
Hit http://nova.clouds.archive.ubuntu.com trusty/main Sources
Hit http://nova.clouds.archive.ubuntu.com trusty/universe Sources
Hit http://nova.clouds.archive.ubuntu.com trusty/main amd64 Packages
Hit http://nova.clouds.archive.ubuntu.com trusty/universe amd64 Packages
Hit http://nova.clouds.archive.ubuntu.com trusty/main Translation-en
Hit http://nova.clouds.archive.ubuntu.com trusty/universe Translation-en
Ign http://nova.clouds.archive.ubuntu.com trusty/main Translation-en_US
Ign http://nova.clouds.archive.ubuntu.com trusty/universe Translation-en_US
Fetched 2,080 kB in 3s (527 kB/s)
Reading package lists...
sudo DEBIAN_FRONTEND=noninteractive apt-get -y \
     --no-install-recommends install apache2 authbind bind9 bind9utils build-essential bzr-builddeb chromium-browser chromium-chromedriver curl daemontools debhelper dh-apport dh-systemd distro-info dnsutils firefox freeipmi-tools git gjs ipython isc-dhcp-common libjs-angularjs libjs-jquery libjs-jquery-hotkeys libjs-yui3-full libjs-yui3-min libpq-dev make nodejs-legacy npm pep8 phantomjs postgresql pyflakes python-apt python-bson python-bzrlib python-convoy python-coverage python-crochet python-cssselect python-curtin python-dev python-distro-info python-django python-django-piston python-django-south python-djorm-ext-pgarray python-docutils python-extras python-fixtures python-flake8 python-formencode python-hivex python-httplib2 python-jinja2 python-jsonschema python-lxml python-mock python-netaddr python-netifaces python-nose python-oauth python-openssl python-paramiko python-pexpect python-pip python-pocket-lint python-psycopg2 python-pyinotify python-pyparsing python-seamicroclient python-simplejson python-simplestreams python-sphinx python-subunit python-tempita python-testresources python-testscenarios python-te...

=== modified file 'src/maasserver/static/js/angular/controllers/node_details_storage.js'
--- src/maasserver/static/js/angular/controllers/node_details_storage.js	2015-10-23 16:34:24 +0000
+++ src/maasserver/static/js/angular/controllers/node_details_storage.js	2015-10-28 20:03:05 +0000
@@ -105,8 +105,6 @@
             }
         ];
 
-        $scope.editing = false;
-        $scope.editing_tags = false;
         $scope.column = 'model';
         $scope.has_disks = false;
         $scope.filesystems = [];
@@ -492,6 +490,10 @@
 
         // Return true if another disk exists with name.
         function isNameAlreadyInUse(name, exclude_disk) {
+            if(!angular.isArray($scope.node.disks)) {
+                return false;
+            }
+
             var i, j;
             for(i = 0; i < $scope.node.disks.length; i++) {
                 var disk = $scope.node.disks[i];
@@ -631,6 +633,7 @@
             return false;
         };
 
+        // Return true if the free space label should be shown.
         $scope.showFreeSpace = function(disk) {
             if(disk.type === "lvm-vg") {
                 return true;
@@ -1202,6 +1205,7 @@
             var selected = $scope.getSelectedAvailable();
             if(selected.length === 1) {
                 return (
+                    !selected[0].has_partitions &&
                     !$scope.hasUnmountedFilesystem(selected[0]) &&
                     selected[0].type !== "lvm-vg");
             }
@@ -1471,7 +1475,11 @@
                 $scope.availableNew.spares.length);
             var minSize = Number.MAX_VALUE;
             angular.forEach($scope.availableNew.devices, function(device) {
-                minSize = Math.min(minSize, device.original.available_size);
+                // Get the size of the device. For a block device it will be
+                // at available_size and for a partition it will be at size.
+                var deviceSize = (
+                    device.original.available_size || device.original.size);
+                minSize = Math.min(minSize, deviceSize);
             });
 
             // Calculate the new size.
@@ -1546,7 +1554,9 @@
             if(selected.length > 0) {
                 var i;
                 for(i = 0; i < selected.length; i++) {
-                    if($scope.hasUnmountedFilesystem(selected[i])) {
+                    if(selected[i].has_partitions) {
+                        return false;
+                    } else if($scope.hasUnmountedFilesystem(selected[i])) {
                         return false;
                     } else if(selected[i].type === "lvm-vg") {
                         return false;
@@ -1573,7 +1583,9 @@
         $scope.getNewVolumeGroupSize = function() {
             var total = 0;
             angular.forEach($scope.availableNew.devices, function(device) {
-                total += device.original.available_size;
+                // Add available_size or size if available_size is not set.
+                total += (
+                    device.original.available_size || device.original.size);
             });
             return ConverterService.bytesToUnits(total).string;
         };
@@ -1703,33 +1715,33 @@
             $scope.updateAvailableSelection(true);
         };
 
+        // Return true when tags can be edited.
+        $scope.canEditTags = function(disk) {
+            return disk.type !== "partition" && disk.type !== "lvm-vg";
+        };
+
         // Called to enter tag editing mode
-        $scope.editTags = function() {
-            if($scope.$parent.canEdit() && !$scope.editing) {
-                $scope.editing = true;
-                $scope.editing_tags = true;
-            }
-        };
-
-        // Called to cancel editing.
-        $scope.cancelTags = function() {
-            $scope.editing = false;
-            $scope.editing_tags = false;
-            updateDisks();
-        };
-
-        // Called to save the changes.
-        $scope.saveTags = function() {
-            $scope.editing = false;
-            $scope.editing_tags = false;
-
-            angular.forEach($scope.available, function(disk) {
-                var tags = [];
-                angular.forEach(disk.tags, function(tag) {
-                    tags.push(tag.text);
-                });
-                NodesManager.updateDiskTags(
-                    $scope.node, disk.block_id, tags);
+        $scope.availableEditTags = function(disk) {
+            disk.$options = {
+                editingTags: true,
+                tags: angular.copy(disk.tags)
+            };
+        };
+
+        // Called to cancel editing tags.
+        $scope.availableCancelTags = function(disk) {
+            disk.$options = {};
+        };
+
+        // Called to save the tag changes.
+        $scope.availableSaveTags = function(disk) {
+            var tags = [];
+            angular.forEach(disk.$options.tags, function(tag) {
+                tags.push(tag.text);
             });
+            NodesManager.updateDiskTags(
+                $scope.node, disk.block_id, tags);
+            disk.tags = disk.$options.tags;
+            disk.$options = {};
         };
     }]);

=== modified file 'src/maasserver/static/js/angular/controllers/tests/test_node_details_storage.js'
--- src/maasserver/static/js/angular/controllers/tests/test_node_details_storage.js	2015-10-23 18:39:01 +0000
+++ src/maasserver/static/js/angular/controllers/tests/test_node_details_storage.js	2015-10-28 20:03:05 +0000
@@ -252,7 +252,6 @@
 
     it("sets initial values", function() {
         var controller = makeController();
-        expect($scope.editing).toBe(false);
         expect($scope.column).toBe('model');
         expect($scope.has_disks).toBe(false);
         expect($scope.filesystems).toEqual([]);
@@ -3277,6 +3276,25 @@
             expect($scope.getNewRAIDSize()).toBe("2.0 MB");
         });
 
+        it("gets proper raid-0 size using size", function() {
+            var controller = makeController();
+            var disk0 = {
+                original: {
+                    size: 1000 * 1000
+                }
+            };
+            var disk1 = {
+                original: {
+                    size: 1000 * 1000
+                }
+            };
+            $scope.availableNew.spares = [];
+            $scope.availableNew.devices = [disk0, disk1];
+            $scope.availableNew.mode = $scope.getAvailableRAIDModes()[0];
+
+            expect($scope.getNewRAIDSize()).toBe("2.0 MB");
+        });
+
         it("gets proper raid-1 size", function() {
             var controller = makeController();
             var disk0 = {
@@ -3632,6 +3650,29 @@
 
             expect($scope.getNewVolumeGroupSize()).toBe("3.0 MB");
         });
+
+        it("return the total of all devices using size", function() {
+            var controller = makeController();
+            $scope.availableNew.devices = [
+                {
+                    original: {
+                        size: 1000 * 1000
+                    }
+                },
+                {
+                    original: {
+                        size: 1000 * 1000
+                    }
+                },
+                {
+                    original: {
+                        size: 1000 * 1000
+                    }
+                }
+            ];
+
+            expect($scope.getNewVolumeGroupSize()).toBe("3.0 MB");
+        });
     });
 
     describe("createVolumeGroupCanSave", function() {
@@ -3925,83 +3966,90 @@
         });
     });
 
-    describe("editTags", function() {
-
-        it("doesnt sets editing to true if cannot edit", function() {
-            var controller = makeController();
-            canEditSpy.and.returnValue(false);
-            $scope.editing = false;
-            $scope.editing_tags = false;
-            $scope.editTags();
-            expect($scope.editing).toBe(false);
-            expect($scope.editing_tags).toBe(false);
-        });
-
-        it("sets editing to true", function() {
-            var controller = makeController();
-            canEditSpy.and.returnValue(true);
-            $scope.editing = false;
-            $scope.editing_tags = false;
-            $scope.editTags();
-            expect($scope.editing).toBe(true);
-            expect($scope.editing_tags).toBe(true);
-        });
-    });
-
-    describe("cancelTags", function() {
-
-        it("sets editing to false", function() {
-            var controller = makeController();
-            $scope.editing = true;
-            $scope.editing_tags = true;
-            $scope.cancelTags();
-            expect($scope.editing).toBe(false);
-            expect($scope.editing_tags).toBe(false);
-        });
-
-        it("calls updateDisks", function() {
-            var controller = makeController();
-
-            // Updates disks so we can check that updateStorage
-            // is called.
-            node.disks = [
-                {
-                    id: 0,
-                    model: makeName("model"),
-                    tags: [],
-                    available_size: 0,
-                    filesystem: null,
-                    partitions: null
+    describe("canEditTags", function() {
+
+        it("returns false for partition", function() {
+            var controller = makeController();
+            expect($scope.canEditTags({
+                type: "partition"
+            })).toBe(false);
+        });
+
+        it("returns false for lvm-vg", function() {
+            var controller = makeController();
+            expect($scope.canEditTags({
+                type: "lvm-vg"
+            })).toBe(false);
+        });
+
+        it("returns true for physical", function() {
+            var controller = makeController();
+            expect($scope.canEditTags({
+                type: "physical"
+            })).toBe(true);
+        });
+
+        it("returns true for virtual", function() {
+            var controller = makeController();
+            expect($scope.canEditTags({
+                type: "virtual"
+            })).toBe(true);
+        });
+    });
+
+    describe("availableEditTags", function() {
+
+        it("sets $options", function() {
+            var controller = makeController();
+            var tags = [{}, {}];
+            var disk = {
+                tags: tags
+            };
+
+            $scope.availableEditTags(disk);
+            expect(disk.$options.editingTags).toBe(true);
+            expect(disk.$options.tags).toEqual(tags);
+            expect(disk.$options.tags).not.toBe(tags);
+        });
+    });
+
+    describe("availableCancelTags", function() {
+
+        it("clears $options", function() {
+            var controller = makeController();
+            var options = {};
+            var disk = { $options: options };
+
+            $scope.availableCancelTags(disk);
+            expect(disk.$options).toEqual({});
+            expect(disk.$options).not.toBe(options);
+        });
+    });
+
+    describe("availableSaveTags", function() {
+
+        it("calls NodesManager.updateDiskTags", function() {
+            var controller = makeController();
+            var tags = [
+                { text: "new" },
+                { text: "old" }
+            ];
+            var disk = {
+                block_id: makeInteger(0, 100),
+                tags: [],
+                $options: {
+                   editingTags:true,
+                   tags: tags
                 }
-            ];
-
-            $scope.nodeLoaded();
-            $rootScope.$digest();
-            var filesystems = $scope.filesystems;
-            var available = $scope.available;
-            var used = $scope.used;
-            $scope.editing = true;
-            $scope.editing_tags = true;
-            $scope.cancelTags();
-
-            // Verify cancel calls updateStorage but doesn't change any data
-            expect($scope.filesystems).toEqual(filesystems);
-            expect($scope.available).toEqual(available);
-            expect($scope.used).toEqual(used);
-        });
-    });
-
-    describe("saveTags", function() {
-
-        it("sets editing to false", function() {
-            var controller = makeController();
-
-            $scope.editing = true;
-            $scope.editing_tags = true;
-            $scope.saveTags();
-
-            expect($scope.editing).toBe(false);
-            expect($scope.editing_tags).toBe(false);
+            };
+            spyOn(NodesManager, "updateDiskTags");
+
+            $scope.availableSaveTags(disk);
+
+            expect(NodesManager.updateDiskTags).toHaveBeenCalledWith(
+                node, disk.block_id, ["new", "old"]);
+            expect(disk.$options).toEqual({});
+            expect(disk.tags).toEqual(tags);
         });
     });
 });

=== modified file 'src/maasserver/static/partials/node-details.html'
--- src/maasserver/static/partials/node-details.html	2015-10-28 18:53:35 +0000
+++ src/maasserver/static/partials/node-details.html	2015-10-28 20:03:05 +0000
@@ -828,11 +828,19 @@
                                     <div class="table__data table__column--15" colspan="2" data-ng-show="column === 'model'">{$ item.model $}</div>
                                     <div class="table__data table__column--15 ng-hide" colspan="2" data-ng-show="column === 'serial'">{$ item.serial $}</div>
                                     <div class="table__data table__tags table__column--15">
-                                        <span class="table__tag" data-ng-repeat="tag in item.tags" data-ng-hide="editing_tags">
+                                        <span class="table__tag" data-ng-repeat="tag in item.tags" data-ng-hide="item.$options.editingTags">
                                             <a href="#/node/?query=storage_tags:({$ tag.text $})">{$ tag.text $}</a>
                                         </span>
-                                        <tags-input ng-model="item.tags" data-ng-show="editing_tags" allow-tags-pattern="[\w-]+"></tags-input>
-                                        <a class="table__controls table__controls--secondary">edit</a>
+                                        <tags-input ng-model="item.$options.tags" data-ng-show="item.$options.editingTags" allow-tags-pattern="[\w-]+"></tags-input>
+                                        <a class="table__controls table__controls--secondary"
+                                            data-ng-hide="!canEditTags(item) || item.$options.editingTags"
+                                            data-ng-click="availableEditTags(item)">edit</a>
+                                        <a class="link-cta-ubuntu text-button"
+                                            data-ng-show="item.$options.editingTags"
+                                            data-ng-click="availableCancelTags(item)">Cancel</a>
+                                        <button class="cta-ubuntu"
+                                            data-ng-show="item.$options.editingTags"
+                                            data-ng-click="availableSaveTags(item)">Save</button>
                                     </div>
                                     <div class="table__data table__column--7">
                                         <div class="table__controls">