Fix race condition in config handling (v7)

[f-blass]

Description of the Change

The CLI encounters race conditions when handling config files if multiple commands are executed in parallel.

Currently, the CLI writes a new config for every command executed due to the code in command_parser.go. However, it also deletes all temporary config files when reading the config, leading to race conditions between multiple processes. This PR introduces two changes to mitigate this problem:

1. Ignore "File Not Found" errors when attempting to delete a file. This change addresses the issue where multiple processes list and delete temporary files simultaneously.
2. Only delete temp config files older than 5 minutes. This change resolves the problem where one process creates a temporary config that it later wants to rename, while a second process deletes this temp config in the meantime. Orphan files should be rare, as they are normally pruned by the signal handler, as seen in

   cli/util/configv3/write_config.go

   Lines 63 to 66 in 04df8ae

   	func catchSignal(sig chan os.Signal, tempConfigFileName string) {
   	<-sig
   	_ = os.Remove(tempConfigFileName)
   	os.Exit(2)

Why Is This PR Valuable?

Enables usage of the CF CLI in scripts which execute commands in parallel

Applicable Issues

fixes #2232

How Urgent Is The Change?

Medium

Other Relevant Parties

Anyone using the CLI in scripts with parallel execution. Multiple users have encountered this issue, as discussed in #2232.

Related PRs

• main
• v8

[gururajsh]

LGTM