Merge ~bladernr/plainbox-provider-checkbox:1904965-cooper-lake-cpuid into plainbox-provider-checkbox:master

[Jeff Lane ]

bin/cpuid.py: add Cooper Lake CPUID to table

bin/cpuid.py: add Cooper Lake CPUID to table

[Jonathan Cave (jocave)]

Looks like a comma is missing

review: Needs Fixing

[Jeff Lane  (bladernr)]

Whoops!

Thanks, I fixed it.

review: Needs Resubmitting

[Jeff Lane  (bladernr)]

Self-approve

review: Approve

[Devices Certification Bot (ce-certification-qa)]

The merge was fine but running tests failed.

[xenial] [00:56:17] starting container
[bionic] [00:56:18] starting container
[focal] [00:56:20] starting container
Device project added to xenial-testing
Device project added to bionic-testing
Device project added to focal-testing
[xenial] [00:56:29] provisioning container
[bionic] [00:56:36] provisioning container
[focal] [00:56:37] provisioning container
[xenial] [00:56:43] Starting tests...
[xenial] Found a test script: ./requirements/container-tests-provider-checkbox
[bionic] [00:56:58] Starting tests...
[bionic] Found a test script: ./requirements/container-tests-provider-checkbox
[focal] [00:57:27] Starting tests...
[focal] Found a test script: ./requirements/container-tests-provider-checkbox
[xenial] [00:58:36] container-tests-provider-checkbox: FAIL
[xenial] output: https://paste.ubuntu.com/p/jhHfkzzJnZ/
[xenial] [00:58:39] Fixing file permissions in source directory
[xenial] [00:58:40] Destroying container
[bionic] [00:58:52] container-tests-provider-checkbox: FAIL
[bionic] output: https://paste.ubuntu.com/p/gFbqTqRtZw/
[bionic] [00:58:55] Fixing file permissions in source directory
[bionic] [00:58:55] Destroying container
[focal] [00:59:10] container-tests-provider-checkbox: FAIL
[focal] output: https://paste.ubuntu.com/p/hc7HwRrPQt/
[focal] [00:59:13] Fixing file permissions in source directory
[focal] [00:59:13] Destroying container

review: Needs Fixing

[Jeff Lane  (bladernr)]

why is flake8 complaining?
bladernr@galactica:~/development/plainbox-provider-checkbox/bin$ flake8 cpuid.py ; echo $?
0
bladernr@galactica:~/development/plainbox-provider-checkbox/bin$

[Jeff Lane  (bladernr)]

self-approve lets try this again.

review: Approve

diff --git a/bin/cpuid.py b/bin/cpuid.py
index 36d5b46..3eebf1e 100755
--- a/bin/cpuid.py
+++ b/bin/cpuid.py
@@ -88,12 +88,14 @@ CPUIDS = {
         "AMD EPYC":         ['0x800f12'],
         "AMD Lisbon":       ['0x100f81'],
         "AMD Magny-Cours":  ['0x100f91'],
+        "AMD Milan":        ['0xa00f11'],
         "AMD ROME":         ['0x830f10'],
         "Broadwell":        ['0x4067', '0x306d4', '0x5066', '0x406f'],
         "Canon Lake":       ['0x6066'],
         "Cascade Lake":     ['0x50655', '0x50656', '0x50657'],
         "Coffee Lake":      [
             '0x806ea', '0x906ea', '0x906eb', '0x906ec', '0x906ed'],
+        "Cooper Lake":      ['0x5065a', '0x5065b'],
         "Haswell":          ['0x306c', '0x4065', '0x4066', '0x306f'],
         "Ice Lake":         ['0x706e'],
         "Ivy Bridge":       ['0x306a', '0x306e'],