Skip to content

Commit

Permalink
[ML] Disable AIOps UI/APIs. (#132589)
Browse files Browse the repository at this point in the history
This disables the UI and APIs for Explain log rate spikes in the ML plugin since it will not be part of 8.3. Once 8.3 has been branched off, we can reenable it in main. This also adds a check to the API integration tests to run the tests only when the hard coded feature flag is set to true.
  • Loading branch information
walterra authored May 20, 2022
1 parent 57d783a commit b3aee17
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
2 changes: 1 addition & 1 deletion x-pack/plugins/aiops/common/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,4 +19,4 @@ export const PLUGIN_NAME = 'AIOps';
* This is an internal hard coded feature flag so we can easily turn on/off the
* "Explain log rate spikes UI" during development until the first release.
*/
export const AIOPS_ENABLED = true;
export const AIOPS_ENABLED = false;
8 changes: 6 additions & 2 deletions x-pack/test/api_integration/apis/aiops/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,17 @@
* 2.0.
*/

import { AIOPS_ENABLED } from '@kbn/aiops-plugin/common';

import { FtrProviderContext } from '../../ftr_provider_context';

export default function ({ loadTestFile }: FtrProviderContext) {
describe('AIOps', function () {
this.tags(['ml']);

loadTestFile(require.resolve('./example_stream'));
loadTestFile(require.resolve('./explain_log_rate_spikes'));
if (AIOPS_ENABLED) {
loadTestFile(require.resolve('./example_stream'));
loadTestFile(require.resolve('./explain_log_rate_spikes'));
}
});
}

0 comments on commit b3aee17

Please sign in to comment.