Add free-form text to qube for notes, comments, ...

qubesadmin part of adding free-form text to each qube for comments,
notes, descriptions, remarks, reminders, etc.

fixes: QubesOS/qubes-issues#899

requires: QubesOS/qubes-core-admin-client#334

Author: alimirjamali

icons/notes.svg

@@ -153,6 +153,7 @@ class VMSettingsWindow(ui_settingsdlg.Ui_SettingsDialog, QtWidgets.QDialog):
         ('devices', 3),
         ('applications', 4),
         ('services', 5),
+        ('notes', 6),
         ))
 
     # pylint: disable=too-many-positional-arguments
@@ -273,6 +274,9 @@ def __init__(self, vm_name, init_page="basic", qapp=None, qubesapp=None,
             self.warn_template_missing_apps.setVisible(
                 self.app_list_manager.has_missing)
 
+        ####### notes tab
+        self.__init_notes_tab__()
+
     def setup_application(self):
         self.qapp.setApplicationName(self.tr("Qube Settings"))
         self.qapp.setWindowIcon(QtGui.QIcon.fromTheme("qubes-manager"))
@@ -354,6 +358,9 @@ def __save_changes__(self):
             ret_tmp = self.__apply_services_tab__()
             if ret_tmp:
                 ret += [self.tr("Sevices tab:")] + ret_tmp
+            ret_tmp = self.__apply_notes_tab__()
+            if ret_tmp:
+                ret += [self.tr("Notes tab:")] + ret_tmp
         except qubesadmin.exc.QubesException as qex:
             ret.append(self.tr('Error while saving changes: ') + str(qex))
         except Exception as ex:  # pylint: disable=broad-except
@@ -1587,6 +1594,23 @@ def __init_services_tab__(self):
         self.service_line_edit.addItem(self.tr('(custom...)'))
         self.service_line_edit.setEditText("")
 
+    def __init_notes_tab__(self):
+        try:
+            self.notes.setPlainText(self.vm.get_notes())
+        except qubesadmin.exc.QubesException:
+            self.notes.setPlainText(
+                "Could not retrieve notes for this qube "
+            )
+            self.notes.setEnabled(False)
+
+    def __apply_notes_tab__(self):
+        try:
+            if self.notes.isEnabled():
+                if self.notes.toPlainText() != self.vm.get_notes():
+                    self.vm.set_notes(self.notes.toPlainText())
+        except qubesadmin.exc.QubesException as ex:
+            return str(ex)
+
     def __add_service__(self):
         srv = str(self.service_line_edit.currentText()).strip()
 

qubesmanager/settings.py

@@ -133,6 +133,8 @@ def settings_fixture(request, qapp, test_qubes_app) -> Tuple[
             'boot-mode.active': 'mode1', 'boot-mode.appvm-default': 'mode2'}
     )
 
+    expected_call = ('sys-net', 'admin.vm.notes.Get', None, None)
+    qapp.expected_calls[expected_call] = b'0\x00'
     test_qubes_app.update_vm_calls()
 
     if isinstance(request.param, dict):

qubesmanager/tests/test_vm_settings.py

@@ -18,6 +18,7 @@
     <file alias="killvm">icons/kill.svg</file>
     <file alias="log">icons/log.svg</file>
     <file alias="networking">icons/netvm.svg</file>
+    <file alias="notes">icons/notes.svg</file>
     <file alias="obsolete.svg">icons/obsolete.svg</file>
     <file alias="outdated">icons/outdated.svg</file>
     <file alias="paused">icons/paused.svg</file>

resources.qrc

@@ -1866,6 +1866,30 @@ The qube must be running to disable seamless mode. This setting is not persisten
          </item>
         </layout>
        </widget>
+       <widget class="QWidget" name="notes_tab">
+        <attribute name="icon">
+         <iconset resource="../resources.qrc">
+          <normaloff>:/notes</normaloff>:/notes</iconset>
+        </attribute>
+        <attribute name="title">
+         <string>Notes</string>
+        </attribute>
+        <layout class="QVBoxLayout" name="verticalLayout_notes_tab">
+         <item>
+          <layout class="QVBoxLayout" name="verticalLayout_notes">
+           <item>
+            <widget class="QPlainTextEdit" name="notes">
+             <property name="placeholderText">
+               <string>Notes and comments (up to 256KB of clear text)
+For user reference only.
+       </string>
+             </property>
+            </widget>
+           </item>
+          </layout>
+         </item>
+        </layout>
+       </widget>
       </widget>
      </item>
      <item>