Merge ~jslarraz/review-tools:add-dm-multipaht-iscsi-initiator-nvme-control-interfaces into review-tools:master
- Git
- lp:~jslarraz/review-tools
- add-dm-multipaht-iscsi-initiator-nvme-control-interfaces
- Merge into 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) |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Paulo Flabiano Smorigo | Approve | ||
|
Review via email:
|
|||
Commit message
Description of the change
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
| 1 | diff --git a/reviewtools/data/snapd-base-declaration.yaml b/reviewtools/data/snapd-base-declaration.yaml |
| 2 | index 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: |
| 71 | diff --git a/reviewtools/schemas/definitions.json b/reviewtools/schemas/definitions.json |
| 72 | index 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", |
| 99 | diff --git a/reviewtools/schemas/plugs.json b/reviewtools/schemas/plugs.json |
| 100 | index 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", |
| 112 | diff --git a/reviewtools/schemas/slots.json b/reviewtools/schemas/slots.json |
| 113 | index 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", |
LGTM