-
Notifications
You must be signed in to change notification settings - Fork 258
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
Make type required when creating an escalation policy step #4356
Conversation
d0d2898
to
3233748
Compare
@@ -303,8 +303,11 @@ def test_should_be_considered_for_rbac_permissioning( | |||
NUM_ORGS = 5 | |||
settings.CLOUD_RBAC_ROLLOUT_PERCENTAGE = rollout_percentage | |||
|
|||
# make sure there are no other orgs around | |||
Organization.objects.all().delete() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we do this in a different way? Scary if someone runs this test connected to the wrong environment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After checking with Joey, it seems we can delete this test (RBAC rollout is already in progress and this will be cleanup soon anyways).
FWIW, we shouldn't run tests affecting any real db in any case, right? (they would break lots of things independently of this particular bit :-))
3233748
to
6093a3f
Compare
Related to https://github.com/grafana/oncall-private/issues/2689