From 91a550f583dbc6b6e6ad8780d9d9b6a7debb0b88 Mon Sep 17 00:00:00 2001 From: Kamil Trzcinski Date: Wed, 21 Jun 2017 13:23:16 +0200 Subject: Disable environment list polling as it breaks deployboard --- app/controllers/projects/environments_controller.rb | 2 -- changelogs/unreleased/disable-environment-list-refresh.yml | 4 ++++ spec/controllers/projects/environments_controller_spec.rb | 6 ++++-- 3 files changed, 8 insertions(+), 4 deletions(-) create mode 100644 changelogs/unreleased/disable-environment-list-refresh.yml diff --git a/app/controllers/projects/environments_controller.rb b/app/controllers/projects/environments_controller.rb index 4630f451445..efe83776834 100644 --- a/app/controllers/projects/environments_controller.rb +++ b/app/controllers/projects/environments_controller.rb @@ -15,8 +15,6 @@ class Projects::EnvironmentsController < Projects::ApplicationController respond_to do |format| format.html format.json do - Gitlab::PollingInterval.set_header(response, interval: 3_000) - render json: { environments: EnvironmentSerializer .new(project: @project, current_user: @current_user) diff --git a/changelogs/unreleased/disable-environment-list-refresh.yml b/changelogs/unreleased/disable-environment-list-refresh.yml new file mode 100644 index 00000000000..62fd71496a0 --- /dev/null +++ b/changelogs/unreleased/disable-environment-list-refresh.yml @@ -0,0 +1,4 @@ +--- +title: Disable environment list refresh due to bug https://gitlab.com/gitlab-org/gitlab-ee/issues/2677 +merge_request: 12347 +author: diff --git a/spec/controllers/projects/environments_controller_spec.rb b/spec/controllers/projects/environments_controller_spec.rb index f6840578145..fcc79902a3c 100644 --- a/spec/controllers/projects/environments_controller_spec.rb +++ b/spec/controllers/projects/environments_controller_spec.rb @@ -58,9 +58,11 @@ describe Projects::EnvironmentsController do expect(json_response['stopped_count']).to eq 1 end - it 'sets the polling interval header' do + it 'does not set the polling interval header' do + # TODO, this is a temporary fix, see follow up issue: + # https://gitlab.com/gitlab-org/gitlab-ee/issues/2677 expect(response).to have_http_status(:ok) - expect(response.headers['Poll-Interval']).to eq("3000") + expect(response.headers['Poll-Interval']).to be_nil end end -- cgit v1.2.1