ubuntu / ubuntu-report

Report hardware and other collected metrics like installer or upgrade information
GNU General Public License v3.0
68 stars 29 forks source link

Gather architecture #6

Closed rfinnie closed 6 years ago

rfinnie commented 6 years ago

On a ppc64el qemu installation, the following information is gathered:

{
  "Version": "18.04",
  "CPU": [
    {
      "Vendor": "",
      "Family": "",
      "Model": "IBM pSeries (emulated by qemu)",
      "Stepping": ""
    }
  ],
  "RAM": 1,
  "Partitions": [
    31.5
  ],
  "Autologin": false,
  "LivePatch": false,
  "Timezone": "America/Los_Angeles"
}

The only way to discern this is a ppc64el installation is from the CPU model. The report should probably capture both dpkg --print-architecture (ppc64el) and uname -m (ppc64le).

didrocks commented 6 years ago

Doesn't "Model": "IBM pSeries (emulated by qemu)", implies ppc64el?

We tried to gather the minimal amount of data to not duplicate then, and though the CPU element would enable us to rebuild those. If this isn't the case, happy to add it.

rfinnie commented 6 years ago

It implies that, but is not programatically explicit (i.e. when aggregating data, you'd need to know that's a CPU string which would be returned). And for things like x86, it's not possible to tell if the user did an i386 or an amd64 install. And arm/arm64 doesn't appear to send the CPU dict at all:

{
   "Partitions" : [
      118.8,
      2
   ],
   "LivePatch" : false,
   "Timezone" : "Etc/UTC",
   "RAM" : 0.9,
   "Version" : "16.04",
   "Autologin" : false
}
didrocks commented 6 years ago

The installation media (32 or 64 bits) is reflected in the install stenza (doesn't work for upgrade though).

However, your point on arm* seems to be worth an upload with that fixed.

Do you mind opening as well a launchpad bug against ubuntu-report telling that arm* doesn't report the CPU type, and so, the architecture is needed? This will enable me to proceed an upload with those data.

Thanks!