-
Notifications
You must be signed in to change notification settings - Fork 31
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Provide osd migration status alert for dr brownfield
Signed-off-by: Timothy Asir Jeyasingh <[email protected]>
- Loading branch information
1 parent
5646fc8
commit b596ace
Showing
14 changed files
with
327 additions
and
53 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
3 changes: 2 additions & 1 deletion
3
packages/mco/components/modals/app-failover-relocate/error-messages.tsx
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
3 changes: 2 additions & 1 deletion
3
packages/mco/components/modals/app-failover-relocate/subscriptions/error-messages.tsx
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
import { ODF_DOC_BASE_PATH, ODF_DOC_VERSION } from '@odf/shared/constants/doc'; | ||
|
||
export const ACM_DOC_VERSION = '2.9'; | ||
export const ACM_DOC_HOME = `https://access.redhat.com/documentation/en-us/red_hat_advanced_cluster_management_for_kubernetes/${ACM_DOC_VERSION}`; | ||
export const ACM_DOC_BASE_PATH = `${ACM_DOC_HOME}/html-single`; | ||
|
||
export const DOC_LINKS = { | ||
APPLY_POLICY: `${ODF_DOC_BASE_PATH}/configuring_openshift_data_foundation_disaster_recovery_for_openshift_workloads/index#apply-drpolicy-to-sample-application_manage-dr`, | ||
MDR_FAILOVER: `${ODF_DOC_BASE_PATH}/configuring_openshift_data_foundation_disaster_recovery_for_openshift_workloads/index#application-failover-between-managed-clusters_manage-dr`, | ||
MDR_RELOCATE: `${ODF_DOC_BASE_PATH}/configuring_openshift_data_foundation_disaster_recovery_for_openshift_workloads/index#relocating-application-between-managed-clusters_manage-dr`, | ||
DR_RELEASE_NOTES: `${ODF_DOC_BASE_PATH}/${ODF_DOC_VERSION}_release_notes/index#disaster_recovery`, | ||
ACM_OFFLINE_CLUSTER: `${ACM_DOC_BASE_PATH}/troubleshooting/index#troubleshooting-an-offline-cluster`, | ||
}; |
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,2 @@ | ||
export * from './disaster-recovery'; | ||
export * from './common'; | ||
export * from './doc-utils'; |
82 changes: 82 additions & 0 deletions
82
...cs/dashboards/persistent-internal/status-card/osd-migration/osd-migration-status.spec.tsx
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,82 @@ | ||
import React from 'react'; | ||
import { BLUESTORE, BLUESTORE_RDR } from '@odf/core/constants'; | ||
import { cleanup, render, screen, waitFor } from '@testing-library/react'; | ||
import '@testing-library/jest-dom/extend-expect'; | ||
import { MemoryRouter } from 'react-router-dom'; | ||
import { getOSDMigrationStatus } from '../../../../utils/osd-migration'; | ||
import { OSDMigration } from './osd-migration-status'; | ||
|
||
jest.mock('@odf/shared/status/icons', () => ({ | ||
RedExclamationCircleIcon: 'div', | ||
})); | ||
|
||
jest.mock('@openshift-console/dynamic-plugin-sdk-internal', () => ({ | ||
HealthBody: 'div', | ||
HealthItem: ({ title }) => <div>{title}</div>, | ||
ViewDocumentation: ({ text, doclink }) => <a href={doclink}>{text}</a>, | ||
HealthState: { | ||
OK: 'OK', | ||
}, | ||
})); | ||
|
||
jest.mock('../../../../utils/osd-migration'); | ||
afterEach(cleanup); | ||
|
||
describe('OSDMigrationStatus', () => { | ||
test('renders the component with COMPLETED status', async () => { | ||
const cephData = { | ||
status: { | ||
storage: { | ||
osd: { | ||
storeType: { | ||
[BLUESTORE_RDR]: 5, | ||
}, | ||
}, | ||
}, | ||
}, | ||
}; | ||
|
||
getOSDMigrationStatus.mockReturnValue('Completed'); | ||
|
||
render( | ||
<MemoryRouter> | ||
<OSDMigration cephData={cephData} /> | ||
</MemoryRouter> | ||
); | ||
|
||
await waitFor(() => { | ||
expect( | ||
screen.getByText('Cluster ready for Regional-DR setup.') | ||
).toBeInTheDocument(); | ||
}); | ||
}); | ||
|
||
test('renders the component with PENDING status', async () => { | ||
const cephData = { | ||
status: { | ||
storage: { | ||
osd: { | ||
storeType: { | ||
[BLUESTORE]: 10, | ||
[BLUESTORE_RDR]: 5, | ||
}, | ||
}, | ||
}, | ||
}, | ||
}; | ||
|
||
getOSDMigrationStatus.mockReturnValue('In Progress'); | ||
|
||
render( | ||
<MemoryRouter> | ||
<OSDMigration cephData={cephData} /> | ||
</MemoryRouter> | ||
); | ||
|
||
await waitFor(() => { | ||
expect( | ||
screen.getByText('Cluster OSDs are being migrated') | ||
).toBeInTheDocument(); | ||
}); | ||
}); | ||
}); |
125 changes: 125 additions & 0 deletions
125
...ges/ocs/dashboards/persistent-internal/status-card/osd-migration/osd-migration-status.tsx
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,125 @@ | ||
import * as React from 'react'; | ||
import { | ||
OSDMigrationStatus, | ||
BLUESTORE, | ||
BLUESTORE_RDR, | ||
} from '@odf/core/constants'; | ||
import { ODF_DR_DOC_HOME } from '@odf/shared/constants/doc'; | ||
import { RedExclamationCircleIcon } from '@odf/shared/status/icons'; | ||
import { CephClusterKind } from '@odf/shared/types'; | ||
import { useCustomTranslation } from '@odf/shared/useCustomTranslationHook'; | ||
import { ViewDocumentation } from '@odf/shared/utils/doc-utils'; | ||
import { HealthState } from '@openshift-console/dynamic-plugin-sdk'; | ||
import { | ||
HealthBody, | ||
HealthItem, | ||
} from '@openshift-console/dynamic-plugin-sdk-internal'; | ||
import { Divider, Flex, FlexItem } from '@patternfly/react-core'; | ||
import { InProgressIcon } from '@patternfly/react-icons'; | ||
import { | ||
getCephStoreType, | ||
getOSDMigrationStatus, | ||
} from '../../../../utils/osd-migration'; | ||
|
||
const calculateOSDMigration = ( | ||
cephData: CephClusterKind | ||
): [number, number, number] => { | ||
const migratedDevices = getCephStoreType(cephData)?.[BLUESTORE_RDR] || 0; | ||
const totalOsd = | ||
(getCephStoreType(cephData)?.[BLUESTORE] || 0) + migratedDevices; | ||
const percentageComplete = | ||
totalOsd !== 0 ? Math.round((migratedDevices / totalOsd) * 100) : 0; | ||
|
||
return [migratedDevices, totalOsd, percentageComplete]; | ||
}; | ||
|
||
export const OSDMigration: React.FC<OSDMigrationStatusProps> = ({ | ||
cephData, | ||
}) => { | ||
const { t } = useCustomTranslation(); | ||
const [migratedDevices, totalOsd, percentageComplete] = | ||
calculateOSDMigration(cephData); | ||
const migrationStatus: string = getOSDMigrationStatus(cephData); | ||
|
||
return ( | ||
<> | ||
{migrationStatus !== OSDMigrationStatus.PENDING && <Divider />} | ||
<HealthBody> | ||
<Flex alignItems={{ default: 'alignItemsCenter' }}> | ||
{migrationStatus === OSDMigrationStatus.COMPLETED && ( | ||
<> | ||
<FlexItem> | ||
<HealthItem | ||
title={t('Cluster ready for Regional-DR setup.')} | ||
state={HealthState.OK} | ||
/> | ||
</FlexItem> | ||
<FlexItem> | ||
<ViewDocumentation | ||
text={t('Setting up disaster recovery')} | ||
doclink={ODF_DR_DOC_HOME} | ||
/> | ||
</FlexItem> | ||
</> | ||
)} | ||
|
||
{migrationStatus === OSDMigrationStatus.IN_PROGRESS && ( | ||
<> | ||
<FlexItem className="pf-u-mt-xl"> | ||
<HealthItem | ||
icon={<InProgressIcon className="co-dashboard-icon" />} | ||
state={HealthState.OK} | ||
title={t('Cluster OSDs are being migrated')} | ||
/> | ||
</FlexItem> | ||
<FlexItem className="pf-u-mt-xl"> | ||
{t( | ||
'{{ percentageComplete }}% completed ({{ migratedDevices }}/{{ totalOsd }} remaining)', | ||
{ | ||
percentageComplete, | ||
migratedDevices, | ||
totalOsd, | ||
} | ||
)} | ||
</FlexItem> | ||
</> | ||
)} | ||
|
||
{migrationStatus === OSDMigrationStatus.FAILED && ( | ||
<> | ||
<FlexItem> | ||
<HealthItem | ||
state={HealthState.OK} | ||
icon={ | ||
<RedExclamationCircleIcon className="co-dashboard-icon" /> | ||
} | ||
title={t('Could not migrate cluster OSDs.')} | ||
/> | ||
</FlexItem> | ||
<FlexItem> | ||
<ViewDocumentation | ||
text={t('Check documentation')} | ||
doclink={ODF_DR_DOC_HOME} | ||
/> | ||
</FlexItem> | ||
<FlexItem align={{ default: 'alignRight' }}> | ||
{t( | ||
'{{ percentageComplete }}% completed ({{ migratedDevices }}/{{ totalOsd }} remaining)', | ||
{ | ||
percentageComplete, | ||
migratedDevices, | ||
totalOsd, | ||
} | ||
)} | ||
</FlexItem> | ||
</> | ||
)} | ||
</Flex> | ||
</HealthBody> | ||
</> | ||
); | ||
}; | ||
|
||
type OSDMigrationStatusProps = { | ||
cephData: CephClusterKind; | ||
}; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,39 @@ | ||
import { | ||
BLUESTORE, | ||
BLUESTORE_RDR, | ||
OSDMigrationStatus, | ||
} from '@odf/core/constants'; | ||
import { CephClusterKind } from '@odf/shared/types'; | ||
|
||
export function getCephStoreType(ceph: CephClusterKind) { | ||
return ceph?.status?.storage?.osd?.storeType; | ||
} | ||
|
||
export const getBluestoreCount = (ceph: CephClusterKind): number => { | ||
return getCephStoreType(ceph)?.[BLUESTORE] || 0; | ||
}; | ||
|
||
export const getBluestoreRdrCount = (ceph: CephClusterKind): number => { | ||
return getCephStoreType(ceph)?.[BLUESTORE_RDR] || 0; | ||
}; | ||
|
||
export const getOSDMigrationStatus = (ceph: CephClusterKind) => { | ||
if (!!ceph) { | ||
const bluestoreCount = getBluestoreCount(ceph); | ||
const bluestoreRdrCount = getBluestoreRdrCount(ceph); | ||
|
||
if (bluestoreCount > 0) { | ||
if (bluestoreRdrCount > 0) { | ||
return OSDMigrationStatus.IN_PROGRESS; | ||
} else { | ||
return OSDMigrationStatus.PENDING; | ||
} | ||
} else if (bluestoreRdrCount > 0) { | ||
return OSDMigrationStatus.COMPLETED; | ||
} | ||
} else { | ||
return OSDMigrationStatus.FAILED; | ||
} // TODO Add condition for migration failure | ||
|
||
return ''; | ||
}; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,12 @@ | ||
export const DISASTER_RECOVERY_TARGET_ANNOTATION = | ||
'ocs.openshift.io/clusterIsDisasterRecoveryTarget'; | ||
|
||
export enum OSDMigrationStatus { | ||
IN_PROGRESS = 'In Progress', | ||
PENDING = 'Pending', | ||
COMPLETED = 'Completed', | ||
FAILED = 'Failed', | ||
} | ||
|
||
export const BLUESTORE_RDR = 'bluestore-rdr'; | ||
export const BLUESTORE = 'bluestore'; |
Oops, something went wrong.