Remove the grafana_dashboard_link feature flag

a4e320a4 · Reuben Pereira · Mike Greiling · b98633ad · a4e320a4 · a4e320a4
--- a/app/assets/javascripts/operation_settings/index.js
+++ b/app/assets/javascripts/operation_settings/index.js
@@ -3,14 +3,6 @@ import store from './store';
 import ExternalDashboardForm from './components/external_dashboard.vue';

 export default () => {
-  /**
-   * This check can be removed when we remove
-   * the :grafana_dashboard_link feature flag
-   */
-  if (!gon.features.grafanaDashboardLink) {
-    return null;
-  }
-
  const el = document.querySelector('.js-operation-settings');

  return new Vue({

--- a/app/controllers/projects/environments_controller.rb
+++ b/app/controllers/projects/environments_controller.rb
@@ -13,7 +13,6 @@ class Projects::EnvironmentsController < Projects::ApplicationController
  before_action only: [:metrics, :additional_metrics, :metrics_dashboard] do
    push_frontend_feature_flag(:environment_metrics_use_prometheus_endpoint)
    push_frontend_feature_flag(:environment_metrics_show_multiple_dashboards)
-    push_frontend_feature_flag(:grafana_dashboard_link)
    push_frontend_feature_flag(:prometheus_computed_alerts)
  end


--- a/app/controllers/projects/settings/operations_controller.rb
+++ b/app/controllers/projects/settings/operations_controller.rb
@@ -5,10 +5,6 @@ module Settings
    class OperationsController < Projects::ApplicationController
      before_action :authorize_update_environment!

-      before_action do
-        push_frontend_feature_flag(:grafana_dashboard_link)
-      end
-
      helper_method :error_tracking_setting

      def show

--- a/changelogs/unreleased/remove-grafana-dashboard-link-feature-flag.yml
+++ b/changelogs/unreleased/remove-grafana-dashboard-link-feature-flag.yml
+---
+title: Link to an external dashboard from metrics dashboard
+merge_request: 29369
+author:
+type: added
--- a/spec/javascripts/monitoring/dashboard_spec.js
+++ b/spec/javascripts/monitoring/dashboard_spec.js
@@ -49,9 +49,6 @@ describe('Dashboard', () => {
    window.gon = {
      ...window.gon,
      ee: false,
-      features: {
-        grafanaDashboardLink: true,
-      },
    };

    store = createStore();
@@ -382,52 +379,26 @@ describe('Dashboard', () => {
  describe('external dashboard link', () => {
    beforeEach(() => {
      mock.onGet(mockApiEndpoint).reply(200, metricsGroupsAPIResponse);
-    });
-
-    describe('with feature flag enabled', () => {
-      beforeEach(() => {
-        component = new DashboardComponent({
-          el: document.querySelector('.prometheus-graphs'),
-          propsData: {
-            ...propsData,
-            hasMetrics: true,
-            showPanels: false,
-            externalDashboardUrl: '/mockUrl',
-          },
-          store,
-        });
-      });

-      it('shows the link', done => {
-        setTimeout(() => {
-          expect(component.$el.querySelector('.js-external-dashboard-link').innerText).toContain(
-            'View full dashboard',
-          );
-          done();
-        });
+      component = new DashboardComponent({
+        el: document.querySelector('.prometheus-graphs'),
+        propsData: {
+          ...propsData,
+          hasMetrics: true,
+          showPanels: false,
+          showTimeWindowDropdown: false,
+          externalDashboardUrl: '/mockUrl',
+        },
+        store,
      });
    });

-    describe('without feature flage enabled', () => {
-      beforeEach(() => {
-        window.gon.features.grafanaDashboardLink = false;
-        component = new DashboardComponent({
-          el: document.querySelector('.prometheus-graphs'),
-          propsData: {
-            ...propsData,
-            hasMetrics: true,
-            showPanels: false,
-            externalDashboardUrl: '',
-          },
-          store,
-        });
-      });
-
-      it('does not show the link', done => {
-        setTimeout(() => {
-          expect(component.$el.querySelector('.js-external-dashboard-link')).toBe(null);
-          done();
-        });
+    it('shows the link', done => {
+      setTimeout(() => {
+        expect(component.$el.querySelector('.js-external-dashboard-link').innerText).toContain(
+          'View full dashboard',
+        );
+        done();
      });
    });
  });