Remove ConsoleImage references (#980)

Signed-off-by: Lenin Alevski <alevsk.8772@gmail.com>
This commit is contained in:
Lenin Alevski
2021-08-25 11:23:11 -07:00
committed by GitHub
parent fa465d0ef3
commit 07bef3280d
10 changed files with 4 additions and 47 deletions

View File

@@ -18,10 +18,9 @@ package operatorapi
// list of all console environment constants // list of all console environment constants
const ( const (
ConsoleSubnetLicense = "CONSOLE_SUBNET_LICENSE" ConsoleSubnetLicense = "CONSOLE_SUBNET_LICENSE"
ConsoleOperatorSAToken = "CONSOLE_OPERATOR_SA_TOKEN" ConsoleOperatorSAToken = "CONSOLE_OPERATOR_SA_TOKEN"
ConsoleOperatorConsoleImage = "CONSOLE_OPERATOR_CONSOLE_IMAGE" MinIOSubnetLicense = "MINIO_SUBNET_LICENSE"
MinIOSubnetLicense = "MINIO_SUBNET_LICENSE"
// Constants for prometheus annotations // Constants for prometheus annotations
prometheusPath = "prometheus.io/path" prometheusPath = "prometheus.io/path"

View File

@@ -25,7 +25,6 @@ export interface ITenant {
deletion_date: string; deletion_date: string;
currentState: string; currentState: string;
image: string; image: string;
console_image: string;
instance_count: string; instance_count: string;
name: string; name: string;
namespace?: string; namespace?: string;
@@ -59,7 +58,6 @@ export interface ITenantCreator {
access_keys: string[]; access_keys: string[];
secret_keys: string[]; secret_keys: string[];
image: string; image: string;
console_image: string;
expose_minio: boolean; expose_minio: boolean;
expose_console: boolean; expose_console: boolean;
pools: IPoolModel[]; pools: IPoolModel[];

View File

@@ -317,7 +317,6 @@ const ObjectDetails = ({
let metadata = get(fileData, "user_metadata", {}); let metadata = get(fileData, "user_metadata", {});
setMetadata(metadata); setMetadata(metadata);
console.log("metadata:", res);
setMetadataLoad(false); setMetadataLoad(false);
}) })
.catch((error: ErrorResponseHandler) => { .catch((error: ErrorResponseHandler) => {

View File

@@ -94,7 +94,6 @@ const AddTenant = ({
const tenantName = fields.nameTenant.tenantName; const tenantName = fields.nameTenant.tenantName;
const selectedStorageClass = fields.nameTenant.selectedStorageClass; const selectedStorageClass = fields.nameTenant.selectedStorageClass;
const imageName = fields.configure.imageName; const imageName = fields.configure.imageName;
const consoleImage = fields.configure.consoleImage;
const customDockerhub = fields.configure.customDockerhub; const customDockerhub = fields.configure.customDockerhub;
const imageRegistry = fields.configure.imageRegistry; const imageRegistry = fields.configure.imageRegistry;
const imageRegistryUsername = fields.configure.imageRegistryUsername; const imageRegistryUsername = fields.configure.imageRegistryUsername;
@@ -221,7 +220,6 @@ const AddTenant = ({
enable_prometheus: true, enable_prometheus: true,
service_name: "", service_name: "",
image: imageName, image: imageName,
console_image: consoleImage,
expose_minio: exposeMinIO, expose_minio: exposeMinIO,
expose_console: exposeConsole, expose_console: exposeConsole,
pools: [ pools: [

View File

@@ -40,7 +40,6 @@ interface IConfigureProps {
classes: any; classes: any;
customImage: boolean; customImage: boolean;
imageName: string; imageName: string;
consoleImage: string;
customDockerhub: boolean; customDockerhub: boolean;
imageRegistry: string; imageRegistry: string;
imageRegistryUsername: string; imageRegistryUsername: string;
@@ -76,7 +75,6 @@ const Configure = ({
storageClasses, storageClasses,
customImage, customImage,
imageName, imageName,
consoleImage,
customDockerhub, customDockerhub,
imageRegistry, imageRegistry,
imageRegistryUsername, imageRegistryUsername,
@@ -164,14 +162,6 @@ const Configure = ({
pattern: /^((.*?)\/(.*?):(.+))$/, pattern: /^((.*?)\/(.*?):(.+))$/,
customPatternMessage: "Format must be of form: 'minio/minio:VERSION'", customPatternMessage: "Format must be of form: 'minio/minio:VERSION'",
}, },
{
fieldKey: "consoleImage",
required: false,
value: consoleImage,
pattern: /^((.*?)\/(.*?):(.+))$/,
customPatternMessage:
"Format must be of form: 'minio/console:VERSION'",
},
{ {
fieldKey: "logSearchImage", fieldKey: "logSearchImage",
required: false, required: false,
@@ -234,7 +224,6 @@ const Configure = ({
}, [ }, [
customImage, customImage,
imageName, imageName,
consoleImage,
logSearchImage, logSearchImage,
kesImage, kesImage,
logSearchPostgresImage, logSearchPostgresImage,
@@ -606,7 +595,6 @@ const mapState = (state: AppState) => ({
storageClasses: state.tenants.createTenant.storageClasses, storageClasses: state.tenants.createTenant.storageClasses,
customImage: state.tenants.createTenant.fields.configure.customImage, customImage: state.tenants.createTenant.fields.configure.customImage,
imageName: state.tenants.createTenant.fields.configure.imageName, imageName: state.tenants.createTenant.fields.configure.imageName,
consoleImage: state.tenants.createTenant.fields.configure.consoleImage,
customDockerhub: state.tenants.createTenant.fields.configure.customDockerhub, customDockerhub: state.tenants.createTenant.fields.configure.customDockerhub,
imageRegistry: state.tenants.createTenant.fields.configure.imageRegistry, imageRegistry: state.tenants.createTenant.fields.configure.imageRegistry,
imageRegistryUsername: imageRegistryUsername:

View File

@@ -32,7 +32,6 @@ interface IPreviewProps {
tenantName: string; tenantName: string;
customImage: boolean; customImage: boolean;
imageName: string; imageName: string;
consoleImage: string;
namespace: string; namespace: string;
selectedStorageClass: string; selectedStorageClass: string;
volumeSize: string; volumeSize: string;
@@ -55,7 +54,6 @@ const Preview = ({
tenantName, tenantName,
customImage, customImage,
imageName, imageName,
consoleImage,
namespace, namespace,
selectedStorageClass, selectedStorageClass,
volumeSize, volumeSize,
@@ -88,12 +86,6 @@ const Preview = ({
</TableCell> </TableCell>
<TableCell>{imageName}</TableCell> <TableCell>{imageName}</TableCell>
</TableRow> </TableRow>
<TableRow>
<TableCell align="right" className={classes.tableTitle}>
Console Image
</TableCell>
<TableCell>{consoleImage}</TableCell>
</TableRow>
</Fragment> </Fragment>
)} )}
@@ -145,7 +137,6 @@ const mapState = (state: AppState) => ({
state.tenants.createTenant.fields.nameTenant.selectedStorageClass, state.tenants.createTenant.fields.nameTenant.selectedStorageClass,
customImage: state.tenants.createTenant.fields.configure.customImage, customImage: state.tenants.createTenant.fields.configure.customImage,
imageName: state.tenants.createTenant.fields.configure.imageName, imageName: state.tenants.createTenant.fields.configure.imageName,
consoleImage: state.tenants.createTenant.fields.configure.consoleImage,
namespace: state.tenants.createTenant.fields.nameTenant.namespace, namespace: state.tenants.createTenant.fields.nameTenant.namespace,
volumeSize: state.tenants.createTenant.fields.tenantSize.volumeSize, volumeSize: state.tenants.createTenant.fields.tenantSize.volumeSize,
sizeFactor: state.tenants.createTenant.fields.tenantSize.sizeFactor, sizeFactor: state.tenants.createTenant.fields.tenantSize.sizeFactor,

View File

@@ -81,7 +81,6 @@ export interface ITenant {
name: string; name: string;
namespace: string; namespace: string;
image: string; image: string;
console_image: string;
pool_count: number; pool_count: number;
currentState: string; currentState: string;
instance_count: 4; instance_count: 4;

View File

@@ -53,7 +53,6 @@ const UpdateTenantModal = ({
}: IUpdateTenantModal) => { }: IUpdateTenantModal) => {
const [isSending, setIsSending] = useState<boolean>(false); const [isSending, setIsSending] = useState<boolean>(false);
const [minioImage, setMinioImage] = useState<string>(""); const [minioImage, setMinioImage] = useState<string>("");
const [consoleImage, setConsoleImage] = useState<string>("");
const [imageRegistry, setImageRegistry] = useState<boolean>(false); const [imageRegistry, setImageRegistry] = useState<boolean>(false);
const [imageRegistryEndpoint, setImageRegistryEndpoint] = const [imageRegistryEndpoint, setImageRegistryEndpoint] =
useState<string>(""); useState<string>("");
@@ -62,39 +61,30 @@ const UpdateTenantModal = ({
const [imageRegistryPassword, setImageRegistryPassword] = const [imageRegistryPassword, setImageRegistryPassword] =
useState<string>(""); useState<string>("");
const [validMinioImage, setValidMinioImage] = useState<boolean>(true); const [validMinioImage, setValidMinioImage] = useState<boolean>(true);
const [validConsoleImage, setValidConsoleImage] = useState<boolean>(true);
const validateImage = useCallback( const validateImage = useCallback(
(fieldToCheck: string) => { (fieldToCheck: string) => {
const pattern = new RegExp("^$|^((.*?)/(.*?):(.+))$"); const pattern = new RegExp("^$|^((.*?)/(.*?):(.+))$");
switch (fieldToCheck) { switch (fieldToCheck) {
case "consoleImage":
setValidConsoleImage(pattern.test(consoleImage));
break;
case "minioImage": case "minioImage":
setValidMinioImage(pattern.test(minioImage)); setValidMinioImage(pattern.test(minioImage));
break; break;
} }
}, },
[consoleImage, minioImage] [minioImage]
); );
useEffect(() => { useEffect(() => {
validateImage("minioImage"); validateImage("minioImage");
}, [minioImage, validateImage]); }, [minioImage, validateImage]);
useEffect(() => {
validateImage("consoleImage");
}, [consoleImage, validateImage]);
const closeAction = () => { const closeAction = () => {
closeModalAndRefresh(false); closeModalAndRefresh(false);
}; };
const resetForm = () => { const resetForm = () => {
setMinioImage(""); setMinioImage("");
setConsoleImage("");
setImageRegistry(false); setImageRegistry(false);
setImageRegistryEndpoint(""); setImageRegistryEndpoint("");
setImageRegistryUsername(""); setImageRegistryUsername("");
@@ -106,7 +96,6 @@ const UpdateTenantModal = ({
let payload = { let payload = {
image: minioImage, image: minioImage,
console_image: consoleImage,
enable_prometheus: true, enable_prometheus: true,
}; };
@@ -235,7 +224,6 @@ const UpdateTenantModal = ({
color="primary" color="primary"
disabled={ disabled={
!validMinioImage || !validMinioImage ||
!validConsoleImage ||
(imageRegistry && (imageRegistry &&
(imageRegistryEndpoint.trim() === "" || (imageRegistryEndpoint.trim() === "" ||
imageRegistryUsername.trim() === "" || imageRegistryUsername.trim() === "" ||

View File

@@ -63,7 +63,6 @@ const initialState: ITenantState = {
configure: { configure: {
customImage: false, customImage: false,
imageName: "", imageName: "",
consoleImage: "",
customDockerhub: false, customDockerhub: false,
imageRegistry: "", imageRegistry: "",
imageRegistryUsername: "", imageRegistryUsername: "",
@@ -531,7 +530,6 @@ export function tenantsReducer(
configure: { configure: {
customImage: false, customImage: false,
imageName: "", imageName: "",
consoleImage: "",
customDockerhub: false, customDockerhub: false,
imageRegistry: "", imageRegistry: "",
imageRegistryUsername: "", imageRegistryUsername: "",

View File

@@ -124,7 +124,6 @@ export interface INameTenantFields {
export interface IConfigureFields { export interface IConfigureFields {
customImage: boolean; customImage: boolean;
imageName: string; imageName: string;
consoleImage: string;
customDockerhub: boolean; customDockerhub: boolean;
imageRegistry: string; imageRegistry: string;
imageRegistryUsername: string; imageRegistryUsername: string;