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.

Sign up for GitHub

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

Jump to bottom

fix: configurable xqueue tele backend #159

Merged

jansenk merged 1 commit into master from jkantor/xqueue-dd

Jan 17, 2025

Member

jansenk commented Jan 16, 2025 •

edited

Loading

see /~https://github.com/edx/edx-internal/pull/12199 and openedx/xqueue#966

Make sure that the following steps are done before merging:

Have a Site Reliability Engineer review the PR if you don't own all of the services impacted.
If you are adding any new default values that need to be overridden when this change goes live, update internal repos and add an entry to the top of the CHANGELOG.
Performed the appropriate testing.


          fix: configurable xqueue tele backend

0c4de51

jansenk requested review from KristinAoki, BrandonHBodine, Rodra, nsprenkle, hajorg and jristau1984

January 16, 2025 15:59

jansenk mentioned this pull request

feat: add telemetry placeholder openedx/xqueue#966

Merged

KristinAoki approved these changes

View reviewed changes

jansenk merged commit f42bdca into master

3 checks passed

jansenk deleted the jkantor/xqueue-dd branch

January 17, 2025 15:40

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

Reviewers

KristinAoki KristinAoki approved these changes

BrandonHBodine Awaiting requested review from BrandonHBodine

Rodra Awaiting requested review from Rodra

nsprenkle Awaiting requested review from nsprenkle

hajorg Awaiting requested review from hajorg

jristau1984 Awaiting requested review from jristau1984

Labels

None yet