Skip to content

Commit 7b19597

Browse files
datho7561datho7561
committed
Do not call getAllComponents() in activate
Part of #3850 Signed-off-by: David Thompson <davthomp@redhat.com>
1 parent eb06f1c commit 7b19597

File tree

5 files changed

+38
-31
lines changed

5 files changed

+38
-31
lines changed

‎src/extension.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,6 @@ export async function activate(extensionContext: ExtensionContext): Promise<unkn
204204
updateStatusBarItem(crcStatusItem, 'Stop CRC');
205205
void extendClusterExplorer();
206206

207-
await ComponentTypesView.instance.getAllComponents();
208207
await registerKubernetesCloudProvider();
209208
void startTelemetry(extensionContext);
210209
await verifyBinariesInRemoteContainer();

‎src/registriesView.ts‎

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import {
1010
QuickInputButtons, QuickPickItem, ThemeIcon, TreeDataProvider,
1111
TreeItem, TreeItemCollapsibleState, TreeView, Uri, window
1212
} from 'vscode';
13-
import { quickBtn, inputValue } from './util/inputValue';
1413
import {
1514
ComponentTypeAdapter,
1615
ComponentTypeDescription,
@@ -19,6 +18,7 @@ import {
1918
} from './odo/componentType';
2019
import { StarterProject } from './odo/componentTypeDescription';
2120
import { Odo } from './odo/odoWrapper';
21+
import { inputValue, quickBtn } from './util/inputValue';
2222
import { Progress } from './util/progress';
2323
import { vsCommand, VsCommandError } from './vscommand';
2424
import fetch = require('make-fetch-happen');
@@ -48,6 +48,13 @@ export class ComponentTypesView implements TreeDataProvider<ComponentType> {
4848
private readonly compDescriptions: Set<ComponentTypeDescription> = new Set<ComponentTypeDescription>();
4949
public subject: Subject<string> = new Subject<string>();
5050

51+
private initialComponentTypeLoadPromise: Promise<void>;
52+
53+
constructor() {
54+
this.initialComponentTypeLoadPromise = this.reloadComponentTypeList();
55+
void Progress.execFunctionWithProgress('Loading component types', () => this.initialComponentTypeLoadPromise);
56+
}
57+
5158
createTreeView(id: string): TreeView<ComponentType> {
5259
if (!this.treeView) {
5360
this.treeView = window.createTreeView(id, {
@@ -102,15 +109,16 @@ export class ComponentTypesView implements TreeDataProvider<ComponentType> {
102109
return this.registries;
103110
}
104111

105-
public getCompDescriptions(): Set<ComponentTypeDescription> {
112+
public async getCompDescriptions(): Promise<Set<ComponentTypeDescription>> {
113+
await this.initialComponentTypeLoadPromise;
106114
return this.compDescriptions;
107115
}
108116

109117
public getListOfRegistries(): Registry[] {
110118
return this.registries;
111119
}
112120

113-
public async getAllComponents(): Promise<void> {
121+
private async reloadComponentTypeList(): Promise<void> {
114122
return new Promise<void>((resolve) => {
115123
let isError = false;
116124
this.compDescriptions.clear();
@@ -362,7 +370,7 @@ export class ComponentTypesView implements TreeDataProvider<ComponentType> {
362370
void Progress.execFunctionWithProgress('Devfile registry is updating',async () => {
363371
const newRegistry = await Odo.Instance.addRegistry(regName, regURL, token);
364372
ComponentTypesView.instance.addRegistry(newRegistry);
365-
await ComponentTypesView.instance.getAllComponents();
373+
await ComponentTypesView.instance.reloadComponentTypeList();
366374
ComponentTypesView.instance.refresh(false);
367375
})
368376
}
@@ -389,7 +397,7 @@ export class ComponentTypesView implements TreeDataProvider<ComponentType> {
389397
await Odo.Instance.removeRegistry(registry.name);
390398
ComponentTypesView.instance.removeRegistry(registry);
391399
if (!isEdit) {
392-
await ComponentTypesView.instance.getAllComponents();
400+
await ComponentTypesView.instance.reloadComponentTypeList();
393401
}
394402
}
395403
}

‎src/webview/common-ext/createComponentHelpers.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ export function validatePortNumber(portNumber: number): string {
185185
*
186186
* @returns a list of the devfile registries with their devfiles attached
187187
*/
188-
export function getDevfileRegistries(): DevfileRegistry[] {
188+
export async function getDevfileRegistries(): Promise<DevfileRegistry[]> {
189189
const registries = ComponentTypesView.instance.getListOfRegistries();
190190
if (!registries || registries.length === 0) {
191191
throw new Error('No Devfile registries available. Default registry is missing');
@@ -198,7 +198,7 @@ export function getDevfileRegistries(): DevfileRegistry[] {
198198
} as DevfileRegistry;
199199
});
200200

201-
const components = ComponentTypesView.instance.getCompDescriptions();
201+
const components = await ComponentTypesView.instance.getCompDescriptions();
202202
for (const component of components) {
203203
const devfileRegistry = devfileRegistries.find(
204204
(devfileRegistry) => format(devfileRegistry.url) === format(component.registry.url),
@@ -261,8 +261,8 @@ export function getDevfileCapabilities(): string[] {
261261
*
262262
* @returns a list of the devfile tags
263263
*/
264-
export function getDevfileTags(url?: string): string[] {
265-
const devfileRegistries = getDevfileRegistries();
264+
export async function getDevfileTags(url?: string): Promise<string[]> {
265+
const devfileRegistries = await getDevfileRegistries();
266266

267267
const devfileTags: string[] = [
268268
...new Set(

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

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ import { ComponentTypesView } from '../../registriesView';
1818
import sendTelemetry from '../../telemetry';
1919
import { ExtensionID } from '../../util/constants';
2020
import { DevfileConverter } from '../../util/devfileConverter';
21+
import { DevfileV1 } from '../../util/devfileV1Type';
2122
import { getInitialWorkspaceFolder, selectWorkspaceFolder } from '../../util/workspace';
2223
import {
2324
getDevfileCapabilities,
@@ -29,7 +30,6 @@ import {
2930
} from '../common-ext/createComponentHelpers';
3031
import { loadWebviewHtml, validateGitURL } from '../common-ext/utils';
3132
import { Devfile, DevfileRegistry, TemplateProjectIdentifier } from '../common/devfile';
32-
import { DevfileV1 } from '../../util/devfileV1Type';
3333

3434
interface CloneProcess {
3535
status: boolean;
@@ -86,7 +86,7 @@ export default class CreateComponentLoader {
8686
});
8787

8888
const tagsSubscription = ComponentTypesView.instance.subject.subscribe(() => {
89-
sendUpdatedTags();
89+
void sendUpdatedTags();
9090
});
9191

9292
panel.onDidDispose(() => {
@@ -158,7 +158,7 @@ export default class CreateComponentLoader {
158158
case 'getDevfileTags': {
159159
void CreateComponentLoader.panel.webview.postMessage({
160160
action: 'devfileTags',
161-
data: getDevfileTags(),
161+
data: await getDevfileTags(),
162162
});
163163
break;
164164
}
@@ -360,7 +360,7 @@ export default class CreateComponentLoader {
360360
await fs.mkdir(componentFolder, {recursive: true});
361361
await fse.copy(tmpFolder.fsPath, componentFolder);
362362
}
363-
const devfileType = getDevfileType(message.data.devfileDisplayName);
363+
const devfileType = await getDevfileType(message.data.devfileDisplayName);
364364
const componentFolderUri = Uri.file(componentFolder);
365365
if (!await isDevfileExists(componentFolderUri)) {
366366
await Odo.Instance.createComponentFromLocation(
@@ -521,7 +521,7 @@ export default class CreateComponentLoader {
521521
action: 'getRecommendedDevfileStart'
522522
});
523523
analyzeRes = await Odo.Instance.analyze(uri.fsPath);
524-
compDescriptions = getCompDescription(analyzeRes);
524+
compDescriptions = await getCompDescription(analyzeRes);
525525
} catch (error) {
526526
if (error.message.toLowerCase().indexOf('failed to parse the devfile') !== -1) {
527527
const actions: Array<string> = ['Yes', 'Cancel'];
@@ -536,7 +536,7 @@ export default class CreateComponentLoader {
536536
const devfileV1 = JSYAML.load(file.toString()) as DevfileV1;
537537
await fs.unlink(devFileV1Path);
538538
analyzeRes = await Odo.Instance.analyze(uri.fsPath);
539-
compDescriptions = getCompDescription(analyzeRes);
539+
compDescriptions = await getCompDescription(analyzeRes);
540540
const endPoints = getEndPoints(compDescriptions[0]);
541541
const devfileV2 = DevfileConverter.getInstance().devfileV1toDevfileV2(
542542
devfileV1,
@@ -562,7 +562,7 @@ export default class CreateComponentLoader {
562562
void CreateComponentLoader.panel.webview.postMessage({
563563
action: 'getRecommendedDevfile'
564564
});
565-
const devfileRegistry: DevfileRegistry[] = getDevfileRegistries();
565+
const devfileRegistry: DevfileRegistry[] = await getDevfileRegistries();
566566
const allDevfiles: Devfile[] = devfileRegistry.flatMap((registry) => registry.devfiles);
567567
const devfile: Devfile | undefined =
568568
compDescriptions.length !== 0
@@ -583,8 +583,8 @@ export default class CreateComponentLoader {
583583
}
584584
}
585585

586-
function getCompDescription(devfiles: AnalyzeResponse[]): ComponentTypeDescription[] {
587-
const compDescriptions = ComponentTypesView.instance.getCompDescriptions();
586+
async function getCompDescription(devfiles: AnalyzeResponse[]): Promise<ComponentTypeDescription[]> {
587+
const compDescriptions = await ComponentTypesView.instance.getCompDescriptions();
588588
if (devfiles.length === 0) {
589589
return Array.from(compDescriptions);
590590
}
@@ -598,9 +598,9 @@ function getCompDescription(devfiles: AnalyzeResponse[]): ComponentTypeDescripti
598598
);
599599
}
600600

601-
function getDevfileType(devfileDisplayName: string): string {
601+
async function getDevfileType(devfileDisplayName: string): Promise<string> {
602602
const compDescriptions: Set<ComponentTypeDescription> =
603-
ComponentTypesView.instance.getCompDescriptions();
603+
await ComponentTypesView.instance.getCompDescriptions();
604604
const devfileDescription: ComponentTypeDescription = Array.from(compDescriptions).find(
605605
(description) => description.displayName === devfileDisplayName,
606606
);
@@ -679,11 +679,11 @@ function sendUpdatedCapabilities() {
679679
}
680680
}
681681

682-
function sendUpdatedTags() {
682+
async function sendUpdatedTags() {
683683
if (CreateComponentLoader.panel) {
684684
void CreateComponentLoader.panel.webview.postMessage({
685685
action: 'devfileTags',
686-
data: getDevfileTags(),
686+
data: await getDevfileTags(),
687687
});
688688
}
689689
}

‎src/webview/devfile-registry/registryViewLoader.ts‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -36,13 +36,13 @@ async function devfileRegistryViewerMessageListener(event: any): Promise<any> {
3636
})
3737
break;
3838
case 'getDevfileRegistries':
39-
RegistryViewLoader.sendUpdatedRegistries();
39+
await RegistryViewLoader.sendUpdatedRegistries();
4040
break;
4141
case 'getDevfileCapabilities':
4242
RegistryViewLoader.sendUpdatedCapabilities();
4343
break;
4444
case 'getDevfileTags':
45-
RegistryViewLoader.sendUpdatedTags();
45+
await RegistryViewLoader.sendUpdatedTags();
4646
break;
4747
case 'createComponent': {
4848
const { projectFolder, componentName } = event.data;
@@ -198,9 +198,9 @@ export default class RegistryViewLoader {
198198
return Promise.resolve();
199199
}
200200

201-
static sendUpdatedRegistries() {
201+
static async sendUpdatedRegistries() {
202202
if (panel) {
203-
let registries = getDevfileRegistries();
203+
let registries = await getDevfileRegistries();
204204
if (RegistryViewLoader.url) {
205205
registries = registries.filter((devfileRegistry) => devfileRegistry.url === RegistryViewLoader.url);
206206
}
@@ -220,24 +220,24 @@ export default class RegistryViewLoader {
220220
}
221221
}
222222

223-
static sendUpdatedTags() {
223+
static async sendUpdatedTags() {
224224
if (panel) {
225225
void panel.webview.postMessage({
226226
action: 'devfileTags',
227-
data: getDevfileTags(RegistryViewLoader.url),
227+
data: await getDevfileTags(RegistryViewLoader.url),
228228
});
229229
}
230230
}
231231
}
232232

233233
ComponentTypesView.instance.subject.subscribe(() => {
234-
RegistryViewLoader.sendUpdatedRegistries();
234+
void RegistryViewLoader.sendUpdatedRegistries();
235235
});
236236

237237
ComponentTypesView.instance.subject.subscribe(() => {
238238
RegistryViewLoader.sendUpdatedCapabilities();
239239
});
240240

241241
ComponentTypesView.instance.subject.subscribe(() => {
242-
RegistryViewLoader.sendUpdatedTags();
242+
void RegistryViewLoader.sendUpdatedTags();
243243
});

0 commit comments

Comments
 (0)
⚡