Merge ~jslarraz/review-tools:add-dm-multipaht-iscsi-initiator-nvme-control-interfaces into review-tools:master

Proposed by Jorge Sancho Larraz
Status: Merged
Merged at revision: 6a05f0174c1fd9e5c19ce009544e286170ddcaed
Proposed branch: ~jslarraz/review-tools:add-dm-multipaht-iscsi-initiator-nvme-control-interfaces
Merge into: review-tools:master
Diff against target: 124 lines (+29/-2)
4 files modified
reviewtools/data/snapd-base-declaration.yaml (+24/-0)
reviewtools/schemas/definitions.json (+3/-0)
reviewtools/schemas/plugs.json (+1/-1)
reviewtools/schemas/slots.json (+1/-1)
Reviewer Review Type Date Requested Status
Paulo Flabiano Smorigo Approve
Review via email: mp+498738@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Paulo Flabiano Smorigo (pfsmorigo) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/reviewtools/data/snapd-base-declaration.yaml b/reviewtools/data/snapd-base-declaration.yaml
2index ab29f87..94df4b3 100644
3--- a/reviewtools/data/snapd-base-declaration.yaml
4+++ b/reviewtools/data/snapd-base-declaration.yaml
5@@ -46,6 +46,9 @@
6 dm-crypt:
7 allow-installation: false
8 deny-auto-connection: true
9+ dm-multipath:
10+ allow-installation: false
11+ deny-auto-connection: true
12 docker-support:
13 allow-installation: false
14 deny-auto-connection: true
15@@ -75,6 +78,9 @@
16 ion-memory-control:
17 allow-installation: false
18 deny-auto-connection: true
19+ iscsi-initiator:
20+ allow-installation: false
21+ deny-auto-connection: true
22 kernel-firmware-control:
23 allow-installation: false
24 deny-auto-connection: true
25@@ -108,6 +114,9 @@
26 nvidia-drivers-support:
27 allow-installation: false
28 deny-auto-connection: true
29+ nvme-control:
30+ allow-installation: false
31+ deny-auto-connection: true
32 opengl-driver-libs:
33 allow-installation:
34 plug-snap-type:
35@@ -482,6 +491,11 @@
36 slot-snap-type:
37 - core
38 deny-auto-connection: true
39+ dm-multipath:
40+ allow-installation:
41+ slot-snap-type:
42+ - core
43+ deny-auto-connection: true
44 docker:
45 allow-installation: false
46 deny-connection: true
47@@ -662,6 +676,11 @@
48 slot-snap-type:
49 - core
50 deny-auto-connection: true
51+ iscsi-initiator:
52+ allow-installation:
53+ slot-snap-type:
54+ - core
55+ deny-auto-connection: true
56 jack1:
57 allow-installation:
58 slot-snap-type:
59@@ -914,6 +933,11 @@
60 slot-snap-type:
61 - core
62 deny-auto-connection: true
63+ nvme-control:
64+ allow-installation:
65+ slot-snap-type:
66+ - core
67+ deny-auto-connection: true
68 ofono:
69 allow-installation:
70 slot-snap-type:
71diff --git a/reviewtools/schemas/definitions.json b/reviewtools/schemas/definitions.json
72index e226003..55c50a2 100644
73--- a/reviewtools/schemas/definitions.json
74+++ b/reviewtools/schemas/definitions.json
75@@ -48,6 +48,7 @@
76 "device-buttons",
77 "display-control",
78 "dm-crypt",
79+ "dm-multipath",
80 "docker",
81 "docker-support",
82 "dsp",
83@@ -82,6 +83,7 @@
84 "intel-qat",
85 "io-ports-control",
86 "ion-memory-control",
87+ "iscsi-initiator",
88 "jack1",
89 "joystick",
90 "juju-client-observe",
91@@ -130,6 +132,7 @@
92 "nfs-mount",
93 "nomad-support",
94 "nvidia-drivers-support",
95+ "nvme-control",
96 "ofono",
97 "online-accounts-service",
98 "opengl",
99diff --git a/reviewtools/schemas/plugs.json b/reviewtools/schemas/plugs.json
100index b9a86a3..0c790ba 100644
101--- a/reviewtools/schemas/plugs.json
102+++ b/reviewtools/schemas/plugs.json
103@@ -1562,7 +1562,7 @@
104 }
105 ]
106 },
107- "^(?!(accel|account-control|accounts-service|acrn-support|adb-support|allegro-vcu|alsa|appstream-metadata|audio-playback|audio-record|auditd-support|autopilot-introspection|avahi-control|avahi-observe|block-devices|bluetooth-control|bluez|bool-file|broadcom-asic-control|browser-support|calendar-service|camera|can-bus|checkbox-support|cifs-mount|classic-support|confdb|contacts-service|content|core-support|cpu-control|cuda-driver-libs|cups|cups-control|custom-device|daemon-notify|dbus|dcdbas-control|desktop|desktop-launch|desktop-legacy|device-buttons|display-control|dm-crypt|docker|docker-support|dsp|dvb|egl-driver-libs|firewall-control|firmware-updater-support|fpga|framebuffer|fuse-support|fwupd|gbm-driver-libs|gconf|gpg-keys|gpg-public-keys|gpio|gpio-chardev|gpio-control|gpio-memory-control|greengrass-support|gsettings|hardware-observe|hardware-random-control|hardware-random-observe|hidraw|home|hostname-control|hugepages-control|i2c|iio|intel-mei|intel-qat|io-ports-control|ion-memory-control|jack1|joystick|juju-client-observe|kerberos-tickets|kernel-crypto-api|kernel-firmware-control|kernel-module-control|kernel-module-load|kernel-module-observe|kubernetes-support|kvm|libvirt|locale-control|location-control|location-observe|log-observe|login-session-control|login-session-observe|lxd|lxd-support|maliit|media-control|media-hub|microceph|microceph-support|microovn|microstack-support|mir|modem-manager|mount-control|mount-observe|mpris|multipass-support|netlink-audit|netlink-connector|netlink-driver|network|network-bind|network-control|network-manager|network-manager-observe|network-observe|network-setup-control|network-setup-observe|network-status|nfs-mount|nomad-support|nvidia-drivers-support|ofono|online-accounts-service|opengl|opengl-driver-libs|opengles-driver-libs|openvswitch|openvswitch-support|optical-drive|packagekit-control|password-manager-service|pcscd|personal-files|physical-memory-control|physical-memory-observe|pipewire|pkcs11|polkit|polkit-agent|posix-mq|power-control|ppp|process-control|ptp|pulseaudio|pwm|qualcomm-ipc-router|raw-input|raw-usb|raw-volume|remoteproc|removable-media|ros-opt-data|ros-snapd-support|screen-inhibit-control|screencast-legacy|scsi-generic|sd-control|serial-port|shared-memory|shutdown|snap-fde-control|snap-interfaces-requests-control|snap-refresh-control|snap-refresh-observe|snap-themes-control|snapd-control|spi|ssh-keys|ssh-public-keys|steam-support|storage-framework-service|system-backup|system-files|system-observe|system-packages-doc|system-source-code|system-trace|tee|thumbnailer-service|time-control|timeserver-control|timezone-control|tpm|u2f-devices|ubuntu-download-manager|udisks2|uhid|uinput|uio|unity7|unity8|unity8-calendar|unity8-contacts|upower-observe|usb-gadget|userns|vcio|vulkan-driver-libs|wayland|x11|xilinx-dma)).+$": {
108+ "^(?!(accel|account-control|accounts-service|acrn-support|adb-support|allegro-vcu|alsa|appstream-metadata|audio-playback|audio-record|auditd-support|autopilot-introspection|avahi-control|avahi-observe|block-devices|bluetooth-control|bluez|bool-file|broadcom-asic-control|browser-support|calendar-service|camera|can-bus|checkbox-support|cifs-mount|classic-support|confdb|contacts-service|content|core-support|cpu-control|cuda-driver-libs|cups|cups-control|custom-device|daemon-notify|dbus|dcdbas-control|desktop|desktop-launch|desktop-legacy|device-buttons|display-control|dm-crypt|dm-multipath|docker|docker-support|dsp|dvb|egl-driver-libs|firewall-control|firmware-updater-support|fpga|framebuffer|fuse-support|fwupd|gbm-driver-libs|gconf|gpg-keys|gpg-public-keys|gpio|gpio-chardev|gpio-control|gpio-memory-control|greengrass-support|gsettings|hardware-observe|hardware-random-control|hardware-random-observe|hidraw|home|hostname-control|hugepages-control|i2c|iio|intel-mei|intel-qat|io-ports-control|ion-memory-control|iscsi-initiator|jack1|joystick|juju-client-observe|kerberos-tickets|kernel-crypto-api|kernel-firmware-control|kernel-module-control|kernel-module-load|kernel-module-observe|kubernetes-support|kvm|libvirt|locale-control|location-control|location-observe|log-observe|login-session-control|login-session-observe|lxd|lxd-support|maliit|media-control|media-hub|microceph|microceph-support|microovn|microstack-support|mir|modem-manager|mount-control|mount-observe|mpris|multipass-support|netlink-audit|netlink-connector|netlink-driver|network|network-bind|network-control|network-manager|network-manager-observe|network-observe|network-setup-control|network-setup-observe|network-status|nfs-mount|nomad-support|nvidia-drivers-support|nvme-control|ofono|online-accounts-service|opengl|opengl-driver-libs|opengles-driver-libs|openvswitch|openvswitch-support|optical-drive|packagekit-control|password-manager-service|pcscd|personal-files|physical-memory-control|physical-memory-observe|pipewire|pkcs11|polkit|polkit-agent|posix-mq|power-control|ppp|process-control|ptp|pulseaudio|pwm|qualcomm-ipc-router|raw-input|raw-usb|raw-volume|remoteproc|removable-media|ros-opt-data|ros-snapd-support|screen-inhibit-control|screencast-legacy|scsi-generic|sd-control|serial-port|shared-memory|shutdown|snap-fde-control|snap-interfaces-requests-control|snap-refresh-control|snap-refresh-observe|snap-themes-control|snapd-control|spi|ssh-keys|ssh-public-keys|steam-support|storage-framework-service|system-backup|system-files|system-observe|system-packages-doc|system-source-code|system-trace|tee|thumbnailer-service|time-control|timeserver-control|timezone-control|tpm|u2f-devices|ubuntu-download-manager|udisks2|uhid|uinput|uio|unity7|unity8|unity8-calendar|unity8-contacts|upower-observe|usb-gadget|userns|vcio|vulkan-driver-libs|wayland|x11|xilinx-dma)).+$": {
109 "oneOf": [
110 {
111 "type": "string",
112diff --git a/reviewtools/schemas/slots.json b/reviewtools/schemas/slots.json
113index 12d0d65..0971a64 100644
114--- a/reviewtools/schemas/slots.json
115+++ b/reviewtools/schemas/slots.json
116@@ -1904,7 +1904,7 @@
117 }
118 ]
119 },
120- "^(?!(accel|account-control|accounts-service|acrn-support|adb-support|allegro-vcu|alsa|appstream-metadata|audio-playback|audio-record|auditd-support|autopilot-introspection|avahi-control|avahi-observe|block-devices|bluetooth-control|bluez|bool-file|broadcom-asic-control|browser-support|calendar-service|camera|can-bus|checkbox-support|cifs-mount|classic-support|confdb|contacts-service|content|core-support|cpu-control|cuda-driver-libs|cups|cups-control|custom-device|daemon-notify|dbus|dcdbas-control|desktop|desktop-launch|desktop-legacy|device-buttons|display-control|dm-crypt|docker|docker-support|dsp|dvb|egl-driver-libs|firewall-control|firmware-updater-support|fpga|framebuffer|fuse-support|fwupd|gbm-driver-libs|gconf|gpg-keys|gpg-public-keys|gpio|gpio-chardev|gpio-control|gpio-memory-control|greengrass-support|gsettings|hardware-observe|hardware-random-control|hardware-random-observe|hidraw|home|hostname-control|hugepages-control|i2c|iio|intel-mei|intel-qat|io-ports-control|ion-memory-control|jack1|joystick|juju-client-observe|kerberos-tickets|kernel-crypto-api|kernel-firmware-control|kernel-module-control|kernel-module-load|kernel-module-observe|kubernetes-support|kvm|libvirt|locale-control|location-control|location-observe|log-observe|login-session-control|login-session-observe|lxd|lxd-support|maliit|media-control|media-hub|microceph|microceph-support|microovn|microstack-support|mir|modem-manager|mount-control|mount-observe|mpris|multipass-support|netlink-audit|netlink-connector|netlink-driver|network|network-bind|network-control|network-manager|network-manager-observe|network-observe|network-setup-control|network-setup-observe|network-status|nfs-mount|nomad-support|nvidia-drivers-support|ofono|online-accounts-service|opengl|opengl-driver-libs|opengles-driver-libs|openvswitch|openvswitch-support|optical-drive|packagekit-control|password-manager-service|pcscd|personal-files|physical-memory-control|physical-memory-observe|pipewire|pkcs11|polkit|polkit-agent|posix-mq|power-control|ppp|process-control|ptp|pulseaudio|pwm|qualcomm-ipc-router|raw-input|raw-usb|raw-volume|remoteproc|removable-media|ros-opt-data|ros-snapd-support|screen-inhibit-control|screencast-legacy|scsi-generic|sd-control|serial-port|shared-memory|shutdown|snap-fde-control|snap-interfaces-requests-control|snap-refresh-control|snap-refresh-observe|snap-themes-control|snapd-control|spi|ssh-keys|ssh-public-keys|steam-support|storage-framework-service|system-backup|system-files|system-observe|system-packages-doc|system-source-code|system-trace|tee|thumbnailer-service|time-control|timeserver-control|timezone-control|tpm|u2f-devices|ubuntu-download-manager|udisks2|uhid|uinput|uio|unity7|unity8|unity8-calendar|unity8-contacts|upower-observe|usb-gadget|userns|vcio|vulkan-driver-libs|wayland|x11|xilinx-dma)).+$": {
121+ "^(?!(accel|account-control|accounts-service|acrn-support|adb-support|allegro-vcu|alsa|appstream-metadata|audio-playback|audio-record|auditd-support|autopilot-introspection|avahi-control|avahi-observe|block-devices|bluetooth-control|bluez|bool-file|broadcom-asic-control|browser-support|calendar-service|camera|can-bus|checkbox-support|cifs-mount|classic-support|confdb|contacts-service|content|core-support|cpu-control|cuda-driver-libs|cups|cups-control|custom-device|daemon-notify|dbus|dcdbas-control|desktop|desktop-launch|desktop-legacy|device-buttons|display-control|dm-crypt|dm-multipath|docker|docker-support|dsp|dvb|egl-driver-libs|firewall-control|firmware-updater-support|fpga|framebuffer|fuse-support|fwupd|gbm-driver-libs|gconf|gpg-keys|gpg-public-keys|gpio|gpio-chardev|gpio-control|gpio-memory-control|greengrass-support|gsettings|hardware-observe|hardware-random-control|hardware-random-observe|hidraw|home|hostname-control|hugepages-control|i2c|iio|intel-mei|intel-qat|io-ports-control|ion-memory-control|iscsi-initiator|jack1|joystick|juju-client-observe|kerberos-tickets|kernel-crypto-api|kernel-firmware-control|kernel-module-control|kernel-module-load|kernel-module-observe|kubernetes-support|kvm|libvirt|locale-control|location-control|location-observe|log-observe|login-session-control|login-session-observe|lxd|lxd-support|maliit|media-control|media-hub|microceph|microceph-support|microovn|microstack-support|mir|modem-manager|mount-control|mount-observe|mpris|multipass-support|netlink-audit|netlink-connector|netlink-driver|network|network-bind|network-control|network-manager|network-manager-observe|network-observe|network-setup-control|network-setup-observe|network-status|nfs-mount|nomad-support|nvidia-drivers-support|nvme-control|ofono|online-accounts-service|opengl|opengl-driver-libs|opengles-driver-libs|openvswitch|openvswitch-support|optical-drive|packagekit-control|password-manager-service|pcscd|personal-files|physical-memory-control|physical-memory-observe|pipewire|pkcs11|polkit|polkit-agent|posix-mq|power-control|ppp|process-control|ptp|pulseaudio|pwm|qualcomm-ipc-router|raw-input|raw-usb|raw-volume|remoteproc|removable-media|ros-opt-data|ros-snapd-support|screen-inhibit-control|screencast-legacy|scsi-generic|sd-control|serial-port|shared-memory|shutdown|snap-fde-control|snap-interfaces-requests-control|snap-refresh-control|snap-refresh-observe|snap-themes-control|snapd-control|spi|ssh-keys|ssh-public-keys|steam-support|storage-framework-service|system-backup|system-files|system-observe|system-packages-doc|system-source-code|system-trace|tee|thumbnailer-service|time-control|timeserver-control|timezone-control|tpm|u2f-devices|ubuntu-download-manager|udisks2|uhid|uinput|uio|unity7|unity8|unity8-calendar|unity8-contacts|upower-observe|usb-gadget|userns|vcio|vulkan-driver-libs|wayland|x11|xilinx-dma)).+$": {
122 "oneOf": [
123 {
124 "type": "string",

Subscribers

People subscribed via source and target branches