Skip to content

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

Open

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.

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

Metadata

Metadata

Assignees

No one assigned

    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