Rishabh Karajgi

> You should use `env`, not `extraEnv`. `extraEnv` is already used globally for thing based config, so having one a string and another a list is confusing: > > https://github.com/apache/airflow/blob/2af19f16a4d94e749bbf6c7c4704e02aac35fc11/chart/values.yaml#L300-L307...

> Let's keep this PR to just env vars. Labels should be done in a separate PR. Had to merge the two PRs to generate a helm chart for testing....

@mayankbatra123 Is this PR feature complete? @jedcunningham can we expedite this for an upcoming astronomer feature?

> If the HTML page of the plugin is going to be visible to end users, it might be a good idea to fix/clean/style it up a bit. But otherwise...