Skip to content

Commit

Permalink
Merge pull request #507 from yaacov/add-missing-types
Browse files Browse the repository at this point in the history
Add global host and vm types
  • Loading branch information
yaacov committed Jun 25, 2023
2 parents e132505 + 7d66b66 commit 6fe9d0a
Show file tree
Hide file tree
Showing 21 changed files with 64 additions and 97 deletions.
1 change: 0 additions & 1 deletion packages/mocks/src/definitions/basic/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,5 @@ export * from './nicProfiles.mock';
export * from './plans.mock';
export * from './providers.mock';
export * from './storages.mock';
export * from './tlsCertificates.mock';
export * from './tree.mock';
export * from './vms.mock';
4 changes: 2 additions & 2 deletions packages/mocks/src/definitions/basic/providers.mock.ts
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
/* eslint-disable @cspell/spellchecker */
import {
ForkliftControllerModelGroupVersionKind,
InventoryByType,
OpenshiftProvider,
OpenstackProvider,
OVirtProvider,
ProviderModelGroupVersionKind as gvk,
ProvidersInventory,
V1beta1Provider,
V1beta1ProviderStatus,
VSphereProvider,
Expand Down Expand Up @@ -451,7 +451,7 @@ export const openshiftProviderHost: OpenshiftProvider = {
},
};

export const MOCK_INVENTORY_PROVIDERS: InventoryByType = {
export const MOCK_INVENTORY_PROVIDERS: ProvidersInventory = {
vsphere: [vmwareProvider1, vmwareProvider2, vmwareProvider3],
ovirt: [rhvProvider1, rhvProvider1i, rhvProvider2, rhvProvider3],
openstack: [openstackProvider1, openstackProvider2],
Expand Down
35 changes: 0 additions & 35 deletions packages/mocks/src/definitions/basic/tlsCertificates.mock.ts

This file was deleted.

18 changes: 9 additions & 9 deletions packages/mocks/src/definitions/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ import { DateTime } from 'luxon';

import {
ForkliftControllerModelGroupVersionKind,
InventoryByType,
IoK8sApimachineryPkgApisMetaV1ObjectMeta,
OpenShiftNamespace,
OpenShiftNetworkAttachmentDefinition,
Expand All @@ -16,6 +15,7 @@ import {
OVirtStorageDomain,
OvirtTreeNode,
OVirtVM,
ProvidersInventory,
V1beta1Provider,
VSphereDataStore,
VSphereHostInventory,
Expand Down Expand Up @@ -44,7 +44,7 @@ export const disks = ({
ovirt,
openstack,
}: {
providers: InventoryByType;
providers: ProvidersInventory;
inventoryPath: string;
ovirt: { [uid: string]: OVirtDisk[] };
openstack: OpenstackVolumeType[];
Expand All @@ -62,7 +62,7 @@ export const vms = ({
vsphere,
ovirt,
}: {
providers: InventoryByType;
providers: ProvidersInventory;
inventoryPath: string;
vsphere: { [uid: string]: VSphereVM[] };
ovirt: { [uid: string]: OVirtVM[] };
Expand All @@ -77,7 +77,7 @@ export const hosts = ({
inventoryPath,
vsphere,
}: {
providers: InventoryByType;
providers: ProvidersInventory;
inventoryPath: string;
vsphere: { [uid: string]: VSphereHostInventory[] };
}) =>
Expand All @@ -91,7 +91,7 @@ export const namespaces = ({
inventoryPath,
openshift,
}: {
providers: InventoryByType;
providers: ProvidersInventory;
inventoryPath: string;
openshift: { [uid: string]: OpenShiftNamespace[] };
}) =>
Expand All @@ -107,7 +107,7 @@ export const networks = ({
ovirt,
openshift,
}: {
providers: InventoryByType;
providers: ProvidersInventory;
inventoryPath: string;
vsphere: { [uid: string]: VSphereNetwork[] };
ovirt: { [uid: string]: OVirtNetwork[] };
Expand All @@ -128,7 +128,7 @@ export const nicProfiles = ({
inventoryPath,
ovirt,
}: {
providers: InventoryByType;
providers: ProvidersInventory;
inventoryPath: string;
ovirt: { [uid: string]: OVirtNicProfile[] };
}) =>
Expand All @@ -145,7 +145,7 @@ export const storages = ({
openstack,
openshift,
}: {
providers: InventoryByType;
providers: ProvidersInventory;
inventoryPath: string;
vsphere: { [uid: string]: VSphereDataStore[] };
ovirt: { [uid: string]: OVirtStorageDomain[] };
Expand Down Expand Up @@ -177,7 +177,7 @@ export const trees = ({
ovirt,
openstack,
}: {
providers: InventoryByType;
providers: ProvidersInventory;
inventoryPath: string;
vsphere: {
hostTree: { [uid: string]: VSphereTreeNode };
Expand Down
8 changes: 8 additions & 0 deletions packages/types/src/types/ProviderHost.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
/** Unified file containing typed provider secrets */

import { OVirtHost, VSphereHost } from './provider';

/**
* General provider host inventory
*/
export type ProviderHost = OVirtHost | VSphereHost;
10 changes: 6 additions & 4 deletions packages/types/src/types/ProviderSecret.ts
Original file line number Diff line number Diff line change
@@ -1,9 +1,11 @@
/** Unified file containing typed provider secrets */

import { OpenShiftProviderSecret } from './secret/OpenShiftProviderSecret';
import { OpenstackProviderSecret } from './secret/OpenstackProviderSecret';
import { OVirtProviderSecret } from './secret/OVirtProviderSecret';
import { VSphereProviderSecret } from './secret/VSphereProviderSecret';
import {
OpenShiftProviderSecret,
OpenstackProviderSecret,
OVirtProviderSecret,
VSphereProviderSecret,
} from './secret';

export type ProviderType = 'openshift' | 'vsphere' | 'ovirt' | 'openstack';

Expand Down
8 changes: 8 additions & 0 deletions packages/types/src/types/ProviderVM.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
/** Unified file containing typed provider secrets */

import { OpenshiftVM, OpenstackVM, OVirtVM, VSphereVM } from './provider';

/**
* General provider virtual machine inventory
*/
export type ProviderVirtualMachine = OpenshiftVM | OVirtVM | VSphereVM | OpenstackVM;
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { OpenshiftProvider, OpenstackProvider, OVirtProvider, VSphereProvider }
/**
* Represents the inventory of providers, including their entities.
*/
export interface InventoryByType {
export interface ProvidersInventory {
openshift?: OpenshiftProvider[] | null;
openstack?: OpenstackProvider[] | null;
ovirt?: OVirtProvider[] | null;
Expand Down
32 changes: 0 additions & 32 deletions packages/types/src/types/TLSCertificate.ts

This file was deleted.

5 changes: 3 additions & 2 deletions packages/types/src/types/index.ts
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
// @index('./*', f => `export * from '${f.path}';`)
export * from './host';
export * from './InventoryByType';
export * from './k8s';
export * from './MustGatherResponse';
export * from './provider';
export * from './ProviderHost';
export * from './ProviderInventory';
export * from './ProviderSecret';
export * from './ProvidersInventory';
export * from './ProviderVM';
export * from './secret';
export * from './TLSCertificate';
// @endindex
7 changes: 7 additions & 0 deletions packages/types/src/types/k8s/K8sResourceCommon.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
import { IoK8sApimachineryPkgApisMetaV1ObjectMeta } from '../../models';

export interface K8sResourceCommon {
apiVersion?: string;
kind?: string;
metadata?: IoK8sApimachineryPkgApisMetaV1ObjectMeta;
}
7 changes: 7 additions & 0 deletions packages/types/src/types/k8s/K8sResourceCondition.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
export interface K8sResourceCondition {
type: string;
status: string;
lastTransitionTime?: string;
reason?: string;
message?: string;
}
Original file line number Diff line number Diff line change
@@ -1,10 +1,8 @@
import { IoK8sApimachineryPkgApisMetaV1ObjectMeta } from '../../models';

export interface Namespace {
export interface V1Namespace {
kind: 'Namespace';
apiVersion: string;

metadata?: IoK8sApimachineryPkgApisMetaV1ObjectMeta;
}

export type V1Namespace = Namespace;
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
import { IoK8sApimachineryPkgApisMetaV1ObjectMeta } from '../../models';

export interface NetworkAttachmentDefinition {
export interface V1NetworkAttachmentDefinition {
kind: 'NetworkAttachmentDefinition';
apiVersion: 'k8s.cni.cncf.io/v1';
metadata?: IoK8sApimachineryPkgApisMetaV1ObjectMeta;
}

export type V1NetworkAttachmentDefinition = NetworkAttachmentDefinition;
4 changes: 1 addition & 3 deletions packages/types/src/types/k8s/V1Secret.ts
Original file line number Diff line number Diff line change
@@ -1,12 +1,10 @@
import { IoK8sApimachineryPkgApisMetaV1ObjectMeta } from '../../models';

interface Secret {
export interface V1Secret {
apiVersion?: string;
kind?: string;
immutable?: boolean;
data?: Record<string, string>;
metadata?: IoK8sApimachineryPkgApisMetaV1ObjectMeta;
type?: string;
}

export type V1Secret = Secret;
2 changes: 1 addition & 1 deletion packages/types/src/types/k8s/index.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
// @index('./*.ts', f => `export * from '${f.path}';`)
export * from './IoK8sApiStorageV1StorageClass';
export * from './Namespace';
export * from './V1Namespace';
export * from './V1NetworkAttachmentDefinition';
export * from './V1Secret';
// @endindex
2 changes: 1 addition & 1 deletion packages/types/src/types/provider/openshift/Namespace.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { V1Namespace } from '../../k8s/Namespace';
import { V1Namespace } from '../../k8s/V1Namespace';

import { OpenshiftResource } from './Resource';

Expand Down
3 changes: 3 additions & 0 deletions packages/types/src/types/provider/openshift/VM.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
import { OpenshiftResource } from './Resource';

export type OpenshiftVM = OpenshiftResource;
1 change: 1 addition & 0 deletions packages/types/src/types/provider/openshift/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,5 @@ export * from './NetworkAttachmentDefinition';
export * from './Provider';
export * from './Resource';
export * from './StorageClass';
export * from './VM';
// @endindex
3 changes: 3 additions & 0 deletions packages/types/src/types/provider/vsphere/Host.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
import { VSphereResource } from './Resource';

export type VSphereHost = VSphereResource;
1 change: 1 addition & 0 deletions packages/types/src/types/provider/vsphere/index.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
// @index('./*.ts', f => `export * from '${f.path}';`)
export * from './DataStore';
export * from './Host';
export * from './model';
export * from './Network';
export * from './Provider';
Expand Down

0 comments on commit 6fe9d0a

Please sign in to comment.