Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ability to forward DCR article traffic to new article-rendering app #26816

Closed
wants to merge 2 commits into from

Conversation

cemms1
Copy link
Contributor

@cemms1 cemms1 commented Jan 16, 2024

What is the value of this and can you measure success?

Add a switch which when on will send traffic to the new article-rendering app for /Article DCR endpoints so that we can begin splitting the DCR apps in order to scale better with varying traffic levels and types of requests.

See guardian/dotcom-rendering#8351 for more information

What does this change?

Re-implements #26789 but this time with a feature switch to easily switch on or off. Also changes fewer endpoints for first iteration

@cemms1 cemms1 changed the title Add ability to start sending traffic to new article rendering app Add ability to forward DCR article traffic to new article-rendering app Jan 16, 2024
@cemms1 cemms1 marked this pull request as ready for review January 16, 2024 17:59
@cemms1 cemms1 requested a review from a team as a code owner January 16, 2024 17:59
@cemms1 cemms1 self-assigned this Jan 16, 2024
@ioannakok
Copy link
Contributor

@cemms1 Maybe before merging this, increase the latency threshold in DCR (frontend-rendering)?

@cemms1 cemms1 force-pushed the cemms1/add-rendering-switch branch from 10ddec5 to eba8f5f Compare January 17, 2024 15:44
@cemms1
Copy link
Contributor Author

cemms1 commented Jan 18, 2024

Closing in favour of #26821

@cemms1 cemms1 closed this Jan 18, 2024
@cemms1 cemms1 deleted the cemms1/add-rendering-switch branch January 18, 2024 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants