Skip to content

decision(delete-user-data): kit/extension parity – kit continues after an Eventarc publish failure, the extension aborts #3254

Closed

Description

@CorieW

delete-user-data kit/extension parity. Priority: low. The kit is intentionally more robust here; this needs recording, not fixing.

Difference

Reproduced

  • Kit: deployed with a channel the function couldn't publish to. Each event logged Failed to publish Eventarc event … 403 PERMISSION_DENIED at WARN, and deletion finished.
  • Extension: with eventarc.publisher removed from the instance service account, clearData threw 403 … eventarc.channels.publish on its first Firestore event. Auto-discovery and SEARCH_FUNCTION never ran, so posts/<uid>, custom/<uid>, groups/g1/members/<uid> and the <uid> collection were all left behind.
  • Kit, same IAM removal: every publish logged a WARN, and the same data was deleted.

Code

Resolution

Keep the kit's behaviour and record it as an accepted difference under "Decisions" in #3249.

Activity

  1. CorieW commented on Nov 3, 2026

    @CorieW
    MemberAuthor

    Decision: keep the kit's behaviour. Publish failures are logged as warnings and deletion carries on, the same call as firestore-send-email (#3290). Recorded under "Decisions" in #3249.

  2. self-assigned this
    on Nov 3, 2026
  3. CorieW commented on Nov 3, 2026

    @CorieW
    MemberAuthor

    Having another look

  4. reopened this on Nov 3, 2026
  5. CorieW commented on Nov 10, 2026

    @CorieW
    MemberAuthor

    Matched the firestore-bigquery-export kit, which handles this the same way: it catches publish failures, logs a warning and carries on (events.ts#L68-L76), and lists it under "Differences" in its README.

    firestore-bigquery-export kit delete-user-data kit
    Publish failure warning, carries on warning, carries on (no change)
    Listed in README "Differences" yes yes, from #3321

    Decision unchanged: keep the kit's behaviour.

  6. added a commit that references this issue on Nov 10, 2026
    3d2dfe7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

decisionNeeds a decision before work can proceedkitsparity

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions