Merge pull request #196 from docker/miscellaneous-fixes-2

Miscellaneous UI fixes

Author: dgageot

src/extension/ui/src/FileUtils.ts

@@ -11,7 +11,6 @@ export const tryRunImageSync = async (client: v1.DockerDesktopClient, args: stri
     try {
         const result = await client.docker.cli.exec('run', args)
         if (result.stderr) {
-            console.error(result.stderr)
             showError(result.stderr)
         }
         return result.stdout || ''

src/extension/ui/src/Registry.ts

@@ -19,7 +19,6 @@ export const getRegistry = async (client: v1.DockerDesktopClient) => {
     const writeRegistryIfNotExists = async () => {
         const registry = await readFileInPromptsVolume(client, 'registry.yaml')
         if (!registry) {
-            console.log('writeRegistryIfNotExists: no registry')
             await writeFileToPromptsVolume(client, JSON.stringify({ files: [{ path: 'registry.yaml', content: 'registry: {}' }] }))
         }
     }
@@ -44,7 +43,6 @@ export const getStoredConfig = async (client: v1.DockerDesktopClient) => {
     const writeConfigIfNotExists = async () => {
         const config = await readFileInPromptsVolume(client, 'config.yaml')
         if (!config) {
-            console.log('writeConfigIfNotExists: no config')
             await writeFileToPromptsVolume(client, JSON.stringify({ files: [{ path: 'config.yaml', content: '{}' }] }))
         }
     }
@@ -65,14 +63,11 @@ export const getStoredConfig = async (client: v1.DockerDesktopClient) => {
 // Replace conflicting config values with registry values
 export const syncConfigWithRegistry = async (client: v1.DockerDesktopClient, registry: { [key: string]: { ref: string, config: any } }, config: { [key: string]: { [key: string]: ParsedParameters } }) => {
     if (Object.keys(registry).length === 0) {
-        console.log('No registry to sync with config.')
         return;
     }
     if (Object.keys(config).length === 0) {
-        console.log('No config to sync with registry.')
         return;
     }
-    console.log('SYNC STARTED. REGISTRY -> CONFIG', registry, config)
     const oldConfigString = JSON.stringify(config)
     for (const [registryItemName, registryItem] of Object.entries(registry)) {
         const configInRegistry = registryItem.config
@@ -84,26 +79,19 @@ export const syncConfigWithRegistry = async (client: v1.DockerDesktopClient, reg
     }
     const newConfigString = JSON.stringify(config)
     if (oldConfigString !== newConfigString) {
-        console.log('Updating config with new registry.', 'oldConfigString', oldConfigString, 'newConfigString', newConfigString)
         await writeFileToPromptsVolume(client, JSON.stringify({ files: [{ path: 'config.yaml', content: stringify(config) }] }))
     }
-    else {
-        console.log('No registry changes to sync with config.', 'oldConfigString', oldConfigString, 'newConfigString', newConfigString)
-    }
     return config
 }
 
 //  Replace conflicting registry values with config values
 export const syncRegistryWithConfig = async (client: v1.DockerDesktopClient, registry: { [key: string]: { ref: string, config: any } }, config: { [key: string]: { [key: string]: ParsedParameters } }) => {
     if (Object.keys(config).length === 0) {
-        console.log('No config to sync with registry.')
         return;
     }
     if (Object.keys(registry).length === 0) {
-        console.log('No registry to sync with config.')
         return;
     }
-    console.log('SYNC STARTED. CONFIG -> REGISTRY', config, registry)
     const oldRegString = JSON.stringify(registry)
     for (const [itemName, itemConfig] of Object.entries(config)) {
         const registryItem = registry[itemName]
@@ -117,11 +105,7 @@ export const syncRegistryWithConfig = async (client: v1.DockerDesktopClient, reg
     }
     const newRegString = JSON.stringify(registry)
     if (oldRegString !== newRegString) {
-        console.log('Updating registry with new config.', 'oldRegString', oldRegString, 'newRegString', newRegString)
         await writeFileToPromptsVolume(client, JSON.stringify({ files: [{ path: 'registry.yaml', content: stringify({ registry }) }] }))
     }
-    else {
-        console.log('No config changes to sync with registry.', 'oldRegString', oldRegString, 'newRegString', newRegString)
-    }
     return registry
 }