Skip to content

Print experiment in output so incorrect usage is easier to recognize - #23043

Merged
inventhouse merged 1 commit into
masterfrom
bjh/bucketer-print-exp
Feb 20, 2020
Merged

Print experiment in output so incorrect usage is easier to recognize#23043
inventhouse merged 1 commit into
masterfrom
bjh/bucketer-print-exp

Conversation

@inventhouse

@inventhouse inventhouse commented Feb 7, 2020

Copy link
Copy Markdown
Contributor

Print experiment in output so incorrect usage like passing just a username instead of experiment name is easier to recognize

$ scripts/stable_bucketer bjh
For experiment bjh, bholt is in bucket: 1
Generated names:
bholt-bjh-0-rx3c5
bholt-bjh-1-pkkrx

@inventhouse
inventhouse requested a review from feanil February 7, 2020 15:33
@inventhouse

Copy link
Copy Markdown
Contributor Author

Not sure why these quality checks are failing, looks like some sort of Jenkins issue

@inventhouse
inventhouse requested a review from farhan February 7, 2020 15:38
…rname instead of experiment name is easier to recognize
@inventhouse
inventhouse force-pushed the bjh/bucketer-print-exp branch from 7a3587a to 4b19037 Compare February 19, 2020 20:17
@edx-status-bot

Copy link
Copy Markdown

Your PR has finished running tests. There were no failures.

@inventhouse
inventhouse merged commit e1e0258 into master Feb 20, 2020
@inventhouse
inventhouse deleted the bjh/bucketer-print-exp branch February 20, 2020 15:49
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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.

4 participants