996806 - add product_name to crash_adu_by_build_signature

Status: RESOLVED FIXED

(Socorro :: Database, task)

Description

[Robert Helmer [:rhelmer]]

Per 915246 comment 36, we need to expose the product_name in the crash_adu_by_build_signature table.

Comment 1

[[github robot]]

Commits pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/ff64b6adcc483787c88636089d54a50458f2557b
bug 996806 - expose product_name in crash_adu_by_build_signature table

https://github.com/mozilla/socorro/commit/f46b77d4000e548c02ebe7b438a221df2ef5f3c5
Merge pull request #1998 from rhelmer/bug996806-expose-product_name

bug 996806 - expose product_name in crash_adu_by_build_signature table

Comment 2

[Robert Helmer [:rhelmer]]

Migration failed on stage:

sqlalchemy.exc.IntegrityError: (IntegrityError) column "product_name" contains null values
 'ALTER TABLE crash_adu_by_build_signature ADD COLUMN product_name CITEXT NOT NULL' {}

The migration includes a 30-day backfill, so we should just truncate the matview before adding the new column. There's no useful default here.

Comment 3

[Robert Helmer [:rhelmer]]

OK tested on stage, the change in PR https://github.com/mozilla/socorro/pull/2001 works

Comment 4

[[github robot]]

Commits pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/54c443285a86b618d47803809f47ce51c88d03ce
fix bug 996806 - truncate matview, this column cannot be migrated, backfill will cover it instead

https://github.com/mozilla/socorro/commit/487718afd7273ec18bc0edcd313420a14df54b74
Merge pull request #2001 from rhelmer/bug996806-expose-product_name

fix bug 996806 - truncate matview, this column cannot be migrated, backf...

Comment 5

[[github robot]]

Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/46d6a81adfb183b403090083d690e11e22178e66
Merge pull request #2004 from rhelmer/bugbug996806-restrict-sigreports-too

bug 915246 - ensure that product_version_id matches in sigreports and