Skip to content

Conversation

@relud
Copy link
Contributor

@relud relud commented Nov 6, 2025

fixes EXP-6110

expiration_policy:
delete_after_days: 760

- app_name: subplat_cirrus
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question (non-blocking): Based on the dependencies including glean-server, is this perhaps intended to be specific to the SubPlat backend? Or would it also be used for the SubPlat frontend?

If it is specific to the SubPlat backend, then I think it'd be good to change the app_name, v1_name, and app_id to be more similar to the values for the existing subscription_platform_backend app (plus the appropriate Cirrus-specific suffix).

Copy link
Contributor Author

@relud relud Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tl;dr this isn't backend specific

More precisely, while we expect only the backend to use it, we expect any experiments in the frontend to receive feature configuration from the backend, not directly from cirrus. This matches with accounts_cirrus which is also only used in the backend.

integrating with cirrus on the frontend would be for applications that don't have a suitable backend, e.g. static sites that only have client-side code.

Co-authored-by: Sean Rose <[email protected]>
@relud
Copy link
Contributor Author

relud commented Nov 7, 2025

thanks for the reviews, i will need someone else to merge this as I lack such powers.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants