🐛 Fix ENV VAR kv parsing to handle JSON values

[bHacklv]

What

JOB_KUBE_ANNOTATIONS environment variable can be used to define one or more Job pod annotations. The valid value can be a JSON value (see example below). Current logic to process the key:value pairs is not allowing to pass the JSON value.

• JOB_KUBE_ANNOTATIONS can have JSON as values
• This PR fixes the parsing of such values

Before fix:

Example of a valid istio k8s annotation that is incorrectly parsed without a fix:

proxy.istio.io/config={ 
"holdApplicationUntilProxyStarts": true, 
"proxyMetadata": { 
  "EXIT_ON_ZERO_ACTIVE_CONNECTIONS": "true" 
  } 
}

How

1. Parse ENV VAR value with Regex Pattern that checks for JSON values
2. Fallback to Regex Pattern that extracts k:v as it was done before with Splitter stream.

Recommended reading order

1. airbyte-commons-with-dependencies/src/main/java/io/airbyte/commons/workers/config/EnvUtils.java
2. airbyte-commons-with-dependencies/src/main/java/io/airbyte/commons/workers/config/WorkerConfigsProvider.java
3. airbyte-commons-with-dependencies/src/test/java/io/airbyte/commons/workers/config/EnvUtilsTest.java

Can this PR be safely reverted and rolled back?

• YES 💚
• NO ❌

[CLAassistant]

All committers have signed the CLA.

[colesnodgrass]

This PR will need to be updated against the main branch as it has dependencies which no longer exist within the platform (i.e. lombok).

[bHacklv]

This PR will need to be updated against the main branch as it has dependencies which no longer exist within the platform (i.e. lombok).

Sure, I will do it on Monday, thanks!

[bHacklv]

This PR will need to be updated against the main branch as it has dependencies which no longer exist within the platform (i.e. lombok).

Hi @colesnodgrass, I've finally got time to remove the usage of lombok.
It's ready for re-review!

[colesnodgrass]

/create-oss-pr