Define global in subchart json schema

[joebowbeer]

What issue type does this pull request address? (keep at least one, remove the others)
/kind bugfix

What does this pull request do? Which issues does it resolve? (use resolves #<issue_number> if possible)
resolves #1651

Please provide a short message that should be published in the vcluster release notes
Fixed an issue where vcluster helm chart could not be used as a subchart.

What else do we need to know?

I added the Global interface to config/config.go, regenerated the schema, ran helm lint, and verified helm template on a chart with a vcluster subchart succeeded.

[netlify]

✅ Deploy Preview for vcluster-docs canceled.

Built without sensitive environment variables

Name	Link
🔨 Latest commit	3ffc859
🔍 Latest deploy log	https://app.netlify.com/sites/vcluster-docs/deploys/661029abf61118000847d376

[heiko-braun]

Hi @joebowbeer thanks for the contribution. The schema.json is generated from [1] using [2].

Can you adjust the PR to get changes into [1]?

[1] https://github.com/loft-sh/vcluster/blob/main/config/config.go
[2]

vcluster/Justfile

Line 75 in 3b4b5fb

generate-config-schema:

[joebowbeer]

@heiko-braun instead of adding it to config.go, I wonder if it would be better to emit it in the generator:

https://github.com/loft-sh/vcluster/blob/main/hack/schema/main.go

Thoughts?

[FabianKramm]

@joebowbeer if global shows up in the values.yaml it needs to be in the config as well, as we pass the whole values.yaml to the syncer and that will strictly parse it. So I guess we need to add the following to the config.go:

Global interface{} `json:"global,omitempty"`

[joebowbeer]

@FabianKramm updated, PTAL