Skip to content

Commit aca679b

Browse files
vrubezhnyvrubezhny
committed
Component create from Git fails if a devfile exists in git repo #3386
Fixes: #3386 Signed-off-by: Victor Rubezhny <vrubezhny@redhat.com>
1 parent 7466bf1 commit aca679b

File tree

4 files changed

+103
-17
lines changed

4 files changed

+103
-17
lines changed

‎src/webview/common/devfile.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,10 @@ import { StarterProject } from '../../odo/componentTypeDescription';
88
export type Devfile = {
99
name: string;
1010
id: string;
11-
port: number;
11+
port?: number;
1212
registryName: string;
1313
description: string;
14-
logoUrl: string;
14+
logoUrl?: string;
1515
supportsDebug: boolean;
1616
supportsDeploy: boolean;
1717
tags: string[];

‎src/webview/common/devfileListItem.tsx‎

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import { Check, Close } from '@mui/icons-material';
66
import { Box, Chip, Stack, Tooltip, Typography } from '@mui/material';
77
import * as React from 'react';
88
import { Devfile } from '../common/devfile';
9+
import DevfileLogo from '../../../images/context/devfile.png';
910

1011
export type DevfileListItemProps = {
1112
devfile: Devfile;
@@ -56,7 +57,7 @@ function DevfileListContent(props: DevfileListItemProps) {
5657
borderRadius: '4px',
5758
}}
5859
>
59-
<img src={props.devfile.logoUrl} style={{ maxWidth: '6em', maxHeight: '6em' }} />
60+
<img src={props.devfile.logoUrl ? props.devfile.logoUrl : DevfileLogo} style={{ maxWidth: '6em', maxHeight: '6em' }} />
6061
</Box>
6162
<Stack
6263
direction="column"
@@ -76,7 +77,11 @@ function DevfileListContent(props: DevfileListItemProps) {
7677
{props.devfile.name}
7778
</Typography>
7879
<Typography variant="body2" fontStyle="italic">
79-
from {props.devfile.registryName}
80+
from {
81+
(props.devfile.registryName ?
82+
props.devfile.registryName
83+
: 'Custom Devfile')
84+
}
8085
</Typography>
8186
</Stack>
8287
<Typography
@@ -98,17 +103,17 @@ function DevfileListContent(props: DevfileListItemProps) {
98103
icon={props.devfile.supportsDeploy ? <Check /> : <Close />}
99104
color={props.devfile.supportsDeploy ? 'success' : 'error'}
100105
/>
101-
{props.devfile.tags.map((tag, i) => {
106+
{(props.devfile.tags && props.devfile.tags.map((tag, i) => {
102107
if (i >= 4) {
103108
return;
104109
}
105110
return <Chip size="small" label={tag} key={tag} />;
106-
})}
107-
{props.devfile.tags.length > 4 && (
111+
}))}
112+
{(props.devfile.tags && props.devfile.tags.length > 4 && (
108113
<Tooltip title={props.devfile.tags.slice(4).join(', ')}>
109114
<Chip size="small" label="• • •" />
110115
</Tooltip>
111-
)}
116+
))}
112117
</Stack>
113118
</Stack>
114119
</Stack>

‎src/webview/create-component/createComponentLoader.ts‎

Lines changed: 82 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ import {
2929
} from '../common-ext/createComponentHelpers';
3030
import { loadWebviewHtml, validateGitURL } from '../common-ext/utils';
3131
import { Devfile, DevfileRegistry, TemplateProjectIdentifier } from '../common/devfile';
32+
import { DevfileV1 } from '../../util/devfileV1Type';
3233

3334
interface CloneProcess {
3435
status: boolean;
@@ -281,11 +282,19 @@ export default class CreateComponentLoader {
281282
action: 'cloneFailed',
282283
});
283284
} else {
285+
const isGirDevfileExists = await isDevfileExists(tmpFolder);
284286
void CreateComponentLoader.panel.webview.postMessage({
285287
action: 'devfileExists',
286-
data: await isDevfileExists(tmpFolder),
288+
data: isGirDevfileExists,
287289
});
288-
void CreateComponentLoader.getRecommendedDevfile(tmpFolder);
290+
291+
if (isGirDevfileExists) {
292+
// Use the Devfile existing in Gir-repo
293+
void CreateComponentLoader.getExistingDevfile(tmpFolder);
294+
} else {
295+
// Use recommended Devfile
296+
void CreateComponentLoader.getRecommendedDevfile(tmpFolder);
297+
}
289298
}
290299
break;
291300
}
@@ -295,7 +304,7 @@ export default class CreateComponentLoader {
295304
case 'createComponent': {
296305
const componentName: string = message.data.componentName;
297306
const portNumber: number = message.data.portNumber;
298-
let componentFolder: string;
307+
let componentFolder: string = '';
299308
try {
300309
if (message.data.isFromTemplateProject) {
301310
// from template project
@@ -337,14 +346,19 @@ export default class CreateComponentLoader {
337346
await fse.copy(tmpFolder.fsPath, componentFolder);
338347
}
339348
const devfileType = getDevfileType(message.data.devfileDisplayName);
340-
if (!await isDevfileExists(Uri.file(componentFolder))) {
349+
const componentFolderUri = Uri.file(componentFolder);
350+
if (!await isDevfileExists(componentFolderUri)) {
341351
await Odo.Instance.createComponentFromLocation(
342352
devfileType,
343353
componentName,
344354
portNumber,
345355
Uri.file(componentFolder),
346356
);
357+
} else {
358+
// Update component devfile with component'a selected name
359+
await CreateComponentLoader.updateDevfileWithComponentName(componentFolderUri, componentName);
347360
}
361+
348362
await sendTelemetry('newComponentCreated', {
349363
strategy,
350364
// eslint-disable-next-line camelcase
@@ -424,6 +438,67 @@ export default class CreateComponentLoader {
424438
}
425439
}
426440

441+
static async updateDevfileWithComponentName(ucomponentFolderUri: vscode.Uri, componentName: string): Promise<void> {
442+
const devFilePath = path.join(ucomponentFolderUri.fsPath, 'devfile.yaml');
443+
const file = await fs.readFile(devFilePath, 'utf8');
444+
const devfile = JSYAML.load(file.toString()) as any;
445+
if (devfile?.metadata?.name !== componentName) {
446+
devfile.metadata.name = componentName;
447+
await fs.unlink(devFilePath);
448+
const yaml = JSYAML.dump(devfile, { sortKeys: true });
449+
await fs.writeFile(devFilePath, yaml.toString(), 'utf-8');
450+
}
451+
}
452+
453+
static async getExistingDevfile(uri: Uri): Promise<void> {
454+
const devFileYamlPath = path.join(tmpFolder.fsPath, 'devfile.yaml');
455+
let rawDevfile: any;
456+
let supportsDebug = false; // Initial value
457+
let supportsDeploy = false; // Initial value
458+
try {
459+
void CreateComponentLoader.panel.webview.postMessage({
460+
action: 'getRecommendedDevfileStart'
461+
});
462+
const componentDescription = await Odo.Instance.describeComponent(uri.fsPath);
463+
if (componentDescription) {
464+
rawDevfile = componentDescription.devfileData.devfile;
465+
supportsDebug = componentDescription.devfileData.supportedOdoFeatures.debug;
466+
supportsDeploy = componentDescription.devfileData.supportedOdoFeatures.deploy;
467+
}
468+
} catch (Error) {
469+
// Will try reading the raw devfile
470+
} finally {
471+
if (!rawDevfile) {
472+
//Try reading the raw devfile
473+
const file = await fs.readFile(devFileYamlPath, 'utf8');
474+
rawDevfile = JSYAML.load(file.toString());
475+
}
476+
477+
void CreateComponentLoader.panel.webview.postMessage({
478+
action: 'getRecommendedDevfile'
479+
});
480+
481+
const devfile: Devfile = {
482+
description: rawDevfile.metadata.description,
483+
registryName: devFileYamlPath,
484+
name: rawDevfile.metadata.displayName ? rawDevfile.metadata.displayName : rawDevfile.metadata.name,
485+
id: rawDevfile.metadata.name,
486+
starterProjects: rawDevfile.starterProjects,
487+
tags: [],
488+
yaml: JSYAML.dump(rawDevfile),
489+
supportsDebug,
490+
supportsDeploy,
491+
} as Devfile;
492+
493+
void CreateComponentLoader.panel.webview.postMessage({
494+
action: 'recommendedDevfile',
495+
data: {
496+
devfile,
497+
},
498+
});
499+
}
500+
}
501+
427502
static async getRecommendedDevfile(uri: Uri): Promise<void> {
428503
let analyzeRes: AnalyzeResponse[] = [];
429504
let compDescriptions: ComponentTypeDescription[] = [];
@@ -444,7 +519,7 @@ export default class CreateComponentLoader {
444519
try {
445520
const devFileV1Path = path.join(uri.fsPath, 'devfile.yaml');
446521
const file = await fs.readFile(devFileV1Path, 'utf8');
447-
const devfileV1 = JSYAML.load(file.toString());
522+
const devfileV1 = JSYAML.load(file.toString()) as DevfileV1;
448523
await fs.unlink(devFileV1Path);
449524
analyzeRes = await Odo.Instance.analyze(uri.fsPath);
450525
compDescriptions = getCompDescription(analyzeRes);
@@ -475,7 +550,7 @@ export default class CreateComponentLoader {
475550
});
476551
const devfileRegistry: DevfileRegistry[] = getDevfileRegistries();
477552
const allDevfiles: Devfile[] = devfileRegistry.flatMap((registry) => registry.devfiles);
478-
const devfile: Devfile =
553+
const devfile: Devfile | undefined =
479554
compDescriptions.length !== 0
480555
? allDevfiles.find(
481556
(devfile) => devfile.name === compDescriptions[0].displayName,
@@ -515,7 +590,7 @@ function getDevfileType(devfileDisplayName: string): string {
515590
const devfileDescription: ComponentTypeDescription = Array.from(compDescriptions).find(
516591
(description) => description.displayName === devfileDisplayName,
517592
);
518-
return devfileDescription.name;
593+
return devfileDescription ? devfileDescription.name : devfileDisplayName;
519594
}
520595

521596
function getEndPoints(compDescription: ComponentTypeDescription): Endpoint[] {

‎src/webview/create-component/pages/fromExistingGitRepo.tsx‎

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -454,9 +454,15 @@ export function FromExistingGitRepo({ setCurrentView }) {
454454
setCurrentPage('fromGitRepo');
455455
}}
456456
createComponent={createComponentFromGitRepo}
457-
devfile={recommendedDevfile.isDevfileExistsInRepo ? undefined : selectedDevfile ? selectedDevfile : recommendedDevfile.devfile}
457+
devfile={
458+
recommendedDevfile.isDevfileExistsInRepo ?
459+
recommendedDevfile.devfile // An existing Git-Repo devfile
460+
: selectedDevfile ?
461+
selectedDevfile // A selected Devfile
462+
: recommendedDevfile.devfile // A recommended Devfile
463+
}
458464
initialComponentName={buildSanitizedComponentName(gitURL.url)}
459-
/>
465+
/>
460466
);
461467
case 'selectDifferentDevfile':
462468
return (

0 commit comments

Comments
 (0)
⚡