1816967 - firefox-android: cleanup duplicate extra notify blocks in ui-test-apk config

Status: RESOLVED FIXED

(Release Engineering :: General, enhancement)

Description

[Aaron Train [:aaronmt]]

Re: https://github.com/mozilla-mobile/firefox-android/pull/720#discussion_r1104803463

Migration duplicated a couple extra Slack notify blocks. This can be set once in defaults.

Bonus:

"text": "Fenix :firefox: ${task.metadata.name} :x:\n "

Does the task or status object expose the app name that can be used there in the Slack text as Fenix is hard-coded?

Comment 1

[Geoff Brown [:gbrown]]

Related info:
https://github.com/taskcluster/taskcluster/blob/main/services/notify/schemas/v1/slack-request.yml
https://docs.taskcluster.net/docs/reference/core/notify/usage
https://api.slack.com/reference/block-kit/blocks

Comment 2

[Geoff Brown [:gbrown]]

(In reply to Aaron Train [:aaronmt] from comment #0)

Bonus:

"text": "Fenix :firefox: ${task.metadata.name} :x:\n "

Does the task or status object expose the app name that can be used there in the Slack text as Fenix is hard-coded?

I can't think of an easy way of providing the app name here, but ${task.metadata.name} should include the app name already, like "ui-test-apk-fenix-arm-nightly" vs "ui-test-apk-focus-arm-nightly". In that light, I'm suggesting we use a generic term to replace "Fenix":

"text": "firefox-android :firefox: ${task.metadata.name} :x:\n "

Comment 3

[BMO Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1816967 - Simplify and generalize slack notify for ui-test-apk kind (#832)

Comment 4

[Aaron Train [:aaronmt]]

Sound's good

Comment 5

[BMO Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1816967 - Simplify and generalize slack notify for ui-test-apk kind (backport #832) (#838)

Comment 6

[BMO Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1816967 - Simplify and generalize slack notify for ui-test-apk kind (backport #832) (#839)