add nil_to_zero YACE config

[berler]

PR Description

Adds nil_to_zero config flag so YACE can be configured instead of hard-coding true.

Which issue(s) this PR fixes

Fixes #5075

Notes to the Reviewer

PR Checklist

• CHANGELOG.md updated
• Documentation added
• Tests updated
• Config converters updated

[mattdurham]

Overall looks good but we should add the conversion from static -> flow mode in cloudwatch_exporter.go

[berler]

Overall looks good but we should add the conversion from static -> flow mode in cloudwatch_exporter.go

Thanks, I think I've fixed that now. I'm not as familiar with this code so I'm not sure if I could still be missing some places.

[berler]

@mattdurham I think the staticconvert code currently skips converting static jobs:

agent/converter/internal/staticconvert/internal/build/cloudwatch_exporter.go

Line 32 in 0c708da

Static: []cloudwatch.StaticJob{},

-- should I fix this too?

There might be a few other things missing in this converter.

[berler]

@mattdurham Is this good to merge now? I fixed the converter so it now correctly converts static jobs.

[mattdurham]

Sorry was out Friday, @clayton-cornell can check the docs and then if you can resolve the conflict I will merge.

[berler]

Rebased to fix merge conflicts.

[clayton-cornell]

One small change for docs

[clayton-cornell]

Suggested change

	| `statistics` | `list(string)` | List of statistics to scrape. Ex: `"Minimum"`, `"Maximum"`, etc. | | yes |
	| `statistics` | `list(string)` | List of statistics to scrape. For example, `"Minimum"`, `"Maximum"`, etc. | | yes |

[berler]

@clayton-cornell I'm fine committing this, but I didn't actually change this text. I had to reformat because the table columns needed to get bigger to support the new row that I added.

[clayton-cornell]

@berler That's fine. It's just some general cleanup we can do in the doc content - I try to catch these (the e.g. or Ex) when I spot them.

[clayton-cornell]

@mattdurham Over to you.

[mattdurham]

LGTM