diff --git a/services/web/app/src/Features/Project/ProjectController.js b/services/web/app/src/Features/Project/ProjectController.js index 67ae7aa842..9afdc33369 100644 --- a/services/web/app/src/Features/Project/ProjectController.js +++ b/services/web/app/src/Features/Project/ProjectController.js @@ -715,7 +715,7 @@ const ProjectController = { if (!userId) return cb(null, '') SplitTestHandler.getTestSegmentation( userId, - 'pdf_caching_beta_full', + 'pdf_caching_5', (err, segmentation) => { if (err) { // Do not fail loading the editor. @@ -813,8 +813,7 @@ const ProjectController = { return false } const canSeeFeaturePreview = - user.alphaProgram || - (user.betaProgram && pdfCachingFeatureFlag.includes(flag)) + user.alphaProgram || pdfCachingFeatureFlag.includes(flag) if (!canSeeFeaturePreview) { // The user is not in the target group. return false diff --git a/services/web/config/settings.defaults.js b/services/web/config/settings.defaults.js index 076f375433..6047f4e7d2 100644 --- a/services/web/config/settings.defaults.js +++ b/services/web/config/settings.defaults.js @@ -347,16 +347,16 @@ module.exports = { ], }, { - id: 'pdf_caching_beta_full', - active: process.env.SPLIT_TEST_PDF_CACHING_BETA_ACTIVE === 'true', + id: 'pdf_caching_5', + active: process.env.SPLIT_TEST_PDF_CACHING_5_ACTIVE === 'true', variants: [ { id: 'collect-metrics-only', - rolloutPercent: 50, + rolloutPercent: 5, }, { id: 'collect-metrics-and-enable-caching', - rolloutPercent: 50, + rolloutPercent: 5, }, ], }, diff --git a/services/web/frontend/js/ide/pdf/controllers/PdfJsMetrics.js b/services/web/frontend/js/ide/pdf/controllers/PdfJsMetrics.js index b6f7f40fb8..39f24d1727 100644 --- a/services/web/frontend/js/ide/pdf/controllers/PdfJsMetrics.js +++ b/services/web/frontend/js/ide/pdf/controllers/PdfJsMetrics.js @@ -79,7 +79,7 @@ function submitCompileMetrics(metrics) { compileTimeServerE2E: timings.compileE2E, } sl_console.log('/event/compile-metrics', JSON.stringify(metrics)) - sendMB('compile-metrics-v3', leanMetrics, SAMPLING_RATE) + sendMB('compile-metrics-v4', leanMetrics, SAMPLING_RATE) } function submitPDFBandwidth(metrics) { @@ -91,5 +91,5 @@ function submitPDFBandwidth(metrics) { }) }) sl_console.log('/event/pdf-bandwidth', JSON.stringify(metrics)) - sendMB('pdf-bandwidth-v3', metricsFlat, SAMPLING_RATE) + sendMB('pdf-bandwidth-v4', metricsFlat, SAMPLING_RATE) } diff --git a/services/web/test/unit/src/Project/ProjectControllerTests.js b/services/web/test/unit/src/Project/ProjectControllerTests.js index 296ba2e2a9..27ebaa1d17 100644 --- a/services/web/test/unit/src/Project/ProjectControllerTests.js +++ b/services/web/test/unit/src/Project/ProjectControllerTests.js @@ -1230,11 +1230,6 @@ describe('ProjectController', function () { this.user.alphaProgram = true }) } - function tagBeta() { - beforeEach(function () { - this.user.betaProgram = true - }) - } beforeEach(function () { this.settings.enablePdfCaching = true @@ -1252,13 +1247,9 @@ describe('ProjectController', function () { tagAlpha() expectToCollectMetricsAndCachePDF() }) - describe('beta user', function () { - tagBeta() - expectToNotBeEnrolledAtAll() - }) }) - describe('during beta roll-out', function () { + describe('during regular roll-out', function () { describe('disabled', function () { showNoVariant() @@ -1273,17 +1264,13 @@ describe('ProjectController', function () { tagAlpha() expectToCollectMetricsAndCachePDF() }) - describe('beta user', function () { - tagBeta() - expectToNotBeEnrolledAtAll() - }) }) describe('variant=collect-metrics', function () { showVariant('collect-metrics') describe('regular user', function () { - expectToNotBeEnrolledAtAll() + expectToCollectMetricsOnly() }) describe('anonymous user', function () { tagAnonymous() @@ -1293,17 +1280,13 @@ describe('ProjectController', function () { tagAlpha() expectToCollectMetricsAndCachePDF() }) - describe('beta user', function () { - tagBeta() - expectToCollectMetricsOnly() - }) }) describe('variant=collect-metrics-and-enable-caching', function () { showVariant('collect-metrics-and-enable-caching') describe('regular user', function () { - expectToNotBeEnrolledAtAll() + expectToCollectMetricsAndCachePDF() }) describe('anonymous user', function () { tagAnonymous() @@ -1313,17 +1296,13 @@ describe('ProjectController', function () { tagAlpha() expectToCollectMetricsAndCachePDF() }) - describe('beta user', function () { - tagBeta() - expectToCollectMetricsAndCachePDF() - }) }) describe('variant=enable-caching-only', function () { showVariant('enable-caching-only') describe('regular user', function () { - expectToNotBeEnrolledAtAll() + expectToCachePDFOnly() }) describe('anonymous user', function () { tagAnonymous() @@ -1333,10 +1312,6 @@ describe('ProjectController', function () { tagAlpha() expectToCollectMetricsAndCachePDF() }) - describe('beta user', function () { - tagBeta() - expectToCachePDFOnly() - }) }) }) })