Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-ce
Commits
e89ec19d
Commit
e89ec19d
authored
Feb 22, 2021
by
Doug Stull
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Remove experiment feature flag for notification dot
- experiment was successful.
parent
f2919e1d
Changes
6
Hide whitespace changes
Inline
Side-by-side
Showing
6 changed files
with
16 additions
and
24 deletions
+16
-24
ee/app/helpers/ee/ci/runners_helper.rb
ee/app/helpers/ee/ci/runners_helper.rb
+3
-3
ee/changelogs/unreleased/298795-ci_notification_dot-experiment-successful-cleanup.yml
...795-ci_notification_dot-experiment-successful-cleanup.yml
+5
-0
ee/config/feature_flags/experiment/ci_notification_dot_experiment_percentage.yml
.../experiment/ci_notification_dot_experiment_percentage.yml
+0
-8
ee/spec/helpers/ee/ci/runners_helper_spec.rb
ee/spec/helpers/ee/ci/runners_helper_spec.rb
+8
-8
lib/gitlab/experimentation.rb
lib/gitlab/experimentation.rb
+0
-4
spec/lib/gitlab/experimentation_spec.rb
spec/lib/gitlab/experimentation_spec.rb
+0
-1
No files found.
ee/app/helpers/ee/ci/runners_helper.rb
View file @
e89ec19d
...
@@ -7,20 +7,20 @@ module EE
...
@@ -7,20 +7,20 @@ module EE
BUY_PIPELINE_MINUTES_NOTIFICATION_DOT
=
'buy_pipeline_minutes_notification_dot'
BUY_PIPELINE_MINUTES_NOTIFICATION_DOT
=
'buy_pipeline_minutes_notification_dot'
def
show_buy_pipeline_minutes?
(
project
,
namespace
)
def
show_buy_pipeline_minutes?
(
project
,
namespace
)
return
false
unless
experiment_enabled?
(
:ci_notification_dot
)
return
false
unless
::
Gitlab
.
dev_env_or_com?
show_out_of_pipeline_minutes_notification?
(
project
,
namespace
)
show_out_of_pipeline_minutes_notification?
(
project
,
namespace
)
end
end
def
show_pipeline_minutes_notification_dot?
(
project
,
namespace
)
def
show_pipeline_minutes_notification_dot?
(
project
,
namespace
)
return
false
unless
experiment_enabled?
(
:ci_notification_dot
)
return
false
unless
::
Gitlab
.
dev_env_or_com?
return
false
if
notification_dot_acknowledged?
return
false
if
notification_dot_acknowledged?
show_out_of_pipeline_minutes_notification?
(
project
,
namespace
)
show_out_of_pipeline_minutes_notification?
(
project
,
namespace
)
end
end
def
show_buy_pipeline_with_subtext?
(
project
,
namespace
)
def
show_buy_pipeline_with_subtext?
(
project
,
namespace
)
return
false
unless
experiment_enabled?
(
:ci_notification_dot
)
return
false
unless
::
Gitlab
.
dev_env_or_com?
return
false
unless
notification_dot_acknowledged?
return
false
unless
notification_dot_acknowledged?
show_out_of_pipeline_minutes_notification?
(
project
,
namespace
)
show_out_of_pipeline_minutes_notification?
(
project
,
namespace
)
...
...
ee/changelogs/unreleased/298795-ci_notification_dot-experiment-successful-cleanup.yml
0 → 100644
View file @
e89ec19d
---
title
:
Cleanup ci_notification_dot experiment
merge_request
:
54721
author
:
type
:
added
ee/config/feature_flags/experiment/ci_notification_dot_experiment_percentage.yml
deleted
100644 → 0
View file @
f2919e1d
---
name
:
ci_notification_dot_experiment_percentage
introduced_by_url
:
https://gitlab.com/gitlab-org/gitlab/-/merge_requests/27626
rollout_issue_url
:
https://gitlab.com/gitlab-org/growth/team-tasks/-/issues/101
milestone
:
'
12.10'
type
:
experiment
group
:
group::expansion
default_enabled
:
false
ee/spec/helpers/ee/ci/runners_helper_spec.rb
View file @
e89ec19d
...
@@ -21,13 +21,13 @@ RSpec.describe EE::Ci::RunnersHelper do
...
@@ -21,13 +21,13 @@ RSpec.describe EE::Ci::RunnersHelper do
end
end
context
'with a project and namespace'
do
context
'with a project and namespace'
do
context
'when
experiment is disabled
'
do
context
'when
not on dot com
'
do
let
(
:
experiment_status
)
{
false
}
let
(
:
dev_env_or_com
)
{
false
}
it
{
is_expected
.
to
be_falsey
}
it
{
is_expected
.
to
be_falsey
}
end
end
context
'when
experiment is enabled
'
do
context
'when
on dot com
'
do
it
{
is_expected
.
to
be_truthy
}
it
{
is_expected
.
to
be_truthy
}
context
'without a persisted project passed'
do
context
'without a persisted project passed'
do
...
@@ -82,15 +82,15 @@ RSpec.describe EE::Ci::RunnersHelper do
...
@@ -82,15 +82,15 @@ RSpec.describe EE::Ci::RunnersHelper do
end
end
context
'with notifications'
do
context
'with notifications'
do
let
(
:
experiment_status
)
{
true
}
let
(
:
dev_env_or_com
)
{
true
}
describe
'.show_buy_pipeline_minutes?'
do
describe
'.show_buy_pipeline_minutes?'
do
subject
{
helper
.
show_buy_pipeline_minutes?
(
project
,
namespace
)
}
subject
{
helper
.
show_buy_pipeline_minutes?
(
project
,
namespace
)
}
context
'when
experiment is "ci_notification_dot"
'
do
context
'when
on dot com
'
do
it_behaves_like
'minutes notification'
do
it_behaves_like
'minutes notification'
do
before
do
before
do
allow
(
helper
).
to
receive
(
:experiment_enabled?
).
with
(
:ci_notification_dot
).
and_return
(
experiment_status
)
allow
(
::
Gitlab
).
to
receive
(
:dev_env_or_com?
).
and_return
(
dev_env_or_com
)
end
end
end
end
end
end
...
@@ -100,7 +100,7 @@ RSpec.describe EE::Ci::RunnersHelper do
...
@@ -100,7 +100,7 @@ RSpec.describe EE::Ci::RunnersHelper do
subject
{
helper
.
show_pipeline_minutes_notification_dot?
(
project
,
namespace
)
}
subject
{
helper
.
show_pipeline_minutes_notification_dot?
(
project
,
namespace
)
}
before
do
before
do
allow
(
helper
).
to
receive
(
:experiment_enabled?
).
with
(
:ci_notification_dot
).
and_return
(
experiment_status
)
allow
(
::
Gitlab
).
to
receive
(
:dev_env_or_com?
).
and_return
(
dev_env_or_com
)
end
end
it_behaves_like
'minutes notification'
it_behaves_like
'minutes notification'
...
@@ -127,7 +127,7 @@ RSpec.describe EE::Ci::RunnersHelper do
...
@@ -127,7 +127,7 @@ RSpec.describe EE::Ci::RunnersHelper do
subject
{
helper
.
show_buy_pipeline_with_subtext?
(
project
,
namespace
)
}
subject
{
helper
.
show_buy_pipeline_with_subtext?
(
project
,
namespace
)
}
before
do
before
do
allow
(
helper
).
to
receive
(
:experiment_enabled?
).
with
(
:ci_notification_dot
).
and_return
(
experiment_status
)
allow
(
::
Gitlab
).
to
receive
(
:dev_env_or_com?
).
and_return
(
dev_env_or_com
)
end
end
context
'when the notification dot has not been acknowledged'
do
context
'when the notification dot has not been acknowledged'
do
...
...
lib/gitlab/experimentation.rb
View file @
e89ec19d
...
@@ -34,10 +34,6 @@
...
@@ -34,10 +34,6 @@
module
Gitlab
module
Gitlab
module
Experimentation
module
Experimentation
EXPERIMENTS
=
{
EXPERIMENTS
=
{
ci_notification_dot:
{
tracking_category:
'Growth::Expansion::Experiment::CiNotificationDot'
,
use_backwards_compatible_subject_index:
true
},
upgrade_link_in_user_menu_a:
{
upgrade_link_in_user_menu_a:
{
tracking_category:
'Growth::Expansion::Experiment::UpgradeLinkInUserMenuA'
,
tracking_category:
'Growth::Expansion::Experiment::UpgradeLinkInUserMenuA'
,
use_backwards_compatible_subject_index:
true
use_backwards_compatible_subject_index:
true
...
...
spec/lib/gitlab/experimentation_spec.rb
View file @
e89ec19d
...
@@ -7,7 +7,6 @@ require 'spec_helper'
...
@@ -7,7 +7,6 @@ require 'spec_helper'
RSpec
.
describe
Gitlab
::
Experimentation
::
EXPERIMENTS
do
RSpec
.
describe
Gitlab
::
Experimentation
::
EXPERIMENTS
do
it
'temporarily ensures we know what experiments exist for backwards compatibility'
do
it
'temporarily ensures we know what experiments exist for backwards compatibility'
do
expected_experiment_keys
=
[
expected_experiment_keys
=
[
:ci_notification_dot
,
:upgrade_link_in_user_menu_a
,
:upgrade_link_in_user_menu_a
,
:invite_members_version_a
,
:invite_members_version_a
,
:invite_members_version_b
,
:invite_members_version_b
,
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment