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

Management command to bootstrap versions for a team #871

Merged
merged 3 commits into from
Nov 15, 2024
Merged

Conversation

snopoke
Copy link
Collaborator

@snopoke snopoke commented Nov 15, 2024

Description

./manage.py enable_versioning_for_team --help

usage: manage.py enable_versioning_for_team [-h] [--force] [--continue-on-error] team

positional arguments:
  team                  The team slug to enable versioning for.

options:
  --force               Force enable versioning.
  --continue-on-error   Continue creating versions even if an error occurs.

This will create versions for all experiments in the team (that don't already have versions) and add the team to the feature flag.

It will stop if there are any errors unless the continue-on-error flag is set.

Use --force to run this for a team that already has the feature flag enabled.

Bonus

When I ran this locally it gave some errors when syncing tools to OpenAI. This was caused by a change I made recently to use 'strict' mode which some of our tools don't support. For now I've disabled strict mode if the tool doesn't support it.

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 83.33333% with 4 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
apps/assistants/sync.py 85.71% 2 Missing ⚠️
apps/teams/models.py 77.77% 2 Missing ⚠️
Additional details and impacted files

📢 Thoughts on this report? Let us know!

@snopoke snopoke merged commit 5a4e1f4 into main Nov 15, 2024
8 checks passed
@snopoke snopoke deleted the sk/version-rollout branch November 15, 2024 18:02
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