Do not cache result if running an interpreter fails

src/client/common/variables/environmentVariablesProvider.ts

@@ -4,7 +4,7 @@
 import { inject, injectable, optional } from 'inversify';
 import path from 'path';
 import { ConfigurationChangeEvent, Disposable, Event, EventEmitter, FileSystemWatcher, Uri } from 'vscode';
-import { traceError } from '../../logging';
+import { traceError, traceVerbose } from '../../logging';
 import { sendFileCreationTelemetry } from '../../telemetry/envFileTelemetry';
 import { IWorkspaceService } from '../application/types';
 import { PythonSettings } from '../configSettings';
@@ -61,6 +61,7 @@ export class EnvironmentVariablesProvider implements IEnvironmentVariablesProvid
         }
         const vars = await this._getEnvironmentVariables(resource);
         this.setCachedEnvironmentVariables(resource, vars);
+        traceVerbose('Dump environment variables', JSON.stringify(vars, null, 4));
         return vars;
     }
 

src/client/pythonEnvironments/base/info/environmentInfoService.ts

@@ -102,6 +102,9 @@ class EnvironmentInfoService implements IEnvironmentInfoService {
         this._getEnvironmentInfo(env, priority)
             .then((r) => {
                 deferred.resolve(r);
+                if (r === undefined) {
+                    this.cache.delete(normCasePath(interpreterPath));
+                }
             })
             .catch((ex) => {
                 deferred.reject(ex);

src/client/pythonEnvironments/common/environmentManagers/conda.ts

@@ -435,7 +435,7 @@ export class Conda {
     // eslint-disable-next-line class-methods-use-this
     private async getInfoImpl(command: string): Promise<CondaInfo> {
         const result = await exec(command, ['info', '--json'], { timeout: CONDA_GENERAL_TIMEOUT });
-        traceVerbose(`conda info --json: ${result.stdout}`);
+        traceVerbose(`${command} info --json: ${result.stdout}`);
         return JSON.parse(result.stdout);
     }