Skip to content

[Due for payment 2025-08-20] Ping app deployers in Slack when CPs require conflict resolution #68299

Closed

Description

@Julesssss

Problem

App deployers often cherry-pick a PR to resolve deploy blockers. Occasionally these cherry-picks have conflicts with the target branch. In these cases a Github issue is created for the deployer to manually resolve and fix (example).

These are sometimes missed, which causes confusion and takes time to clean up. Also, we are shipping deploy blockers to production without realising.

Solution

In addition to creating the Github issue and assigning the relevant engineers, lets post in Slack to provide a more immediate notification.

To get this out asap I'm going to just tag @app_deployers. This can be improved by directly tagging the deployer, but that be done later -- we don't currently have a way of getting Slack Id from github ID

Activity

  1. self-assigned this
    on Aug 11, 2025
  2. added
    Awaiting PaymentAuto-added when associated PR is deployed to production
    and removed on Aug 13, 2025
  3. changed the title [-]Ping app deployers in Slack when CPs require conflict resolution[/-] [+][Due for payment 2025-08-20] Ping app deployers in Slack when CPs require conflict resolution[/+] on Aug 13, 2025
  4. melvin-bot commented on Aug 13, 2025

    @melvin-bot

    Reviewing label has been removed, please complete the "BugZero Checklist".

  5. melvin-bot commented on Aug 13, 2025

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.93-7 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

    If no regressions arise, payment will be issued on 2025-08-20. 🎊

  6. melvin-bot commented on Aug 20, 2025

    @melvin-bot

    Skipping the payment summary for this issue since all the assignees are employees or vendors. If this is incorrect, please manually add the payment summary SO.

  7. melvin-bot commented on Aug 27, 2025

    @melvin-bot

    @Julesssss Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

  8. Julesssss commented on Aug 27, 2025

    @Julesssss
    ContributorAuthor

    This is merged, for now it's fine to tag all

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions