Commit 4c1dbbb4 authored by Alex Pooley's avatar Alex Pooley

Merge branch '351073-cleanup-publish-project-deleted-event-flag' into 'master'

Clean up feature flag publish_project_deleted_event to enable by default

See merge request gitlab-org/gitlab!80875
parents 6212eb6f 4d49dff5
...@@ -37,7 +37,7 @@ module Projects ...@@ -37,7 +37,7 @@ module Projects
system_hook_service.execute_hooks_for(project, :destroy) system_hook_service.execute_hooks_for(project, :destroy)
log_info("Project \"#{project.full_path}\" was deleted") log_info("Project \"#{project.full_path}\" was deleted")
publish_project_deleted_event_for(project) if Feature.enabled?(:publish_project_deleted_event, default_enabled: :yaml) publish_project_deleted_event_for(project)
current_user.invalidate_personal_projects_count current_user.invalidate_personal_projects_count
......
---
name: publish_project_deleted_event
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/78862
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/351073
milestone: '14.8'
type: development
group: group::pipeline insights
default_enabled: false
...@@ -36,18 +36,6 @@ RSpec.describe Projects::DestroyService, :aggregate_failures do ...@@ -36,18 +36,6 @@ RSpec.describe Projects::DestroyService, :aggregate_failures do
destroy_project(project, user, {}) destroy_project(project, user, {})
end end
context 'when feature flag publish_project_deleted_event is disabled' do
before do
stub_feature_flags(publish_project_deleted_event: false)
end
it 'does not publish an event' do
expect(Gitlab::EventStore).not_to receive(:publish).with(event_type(Projects::ProjectDeletedEvent))
destroy_project(project, user, {})
end
end
end end
shared_examples 'deleting the project with pipeline and build' do shared_examples 'deleting the project with pipeline and build' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment