Skip to content

[Bug]: AMS helm chart cannot set log4j2 because of wrong indentation #3549

@sqd

Description

@sqd

What happened?

When setting amoroConf.log4j2 to non-null value in values.yaml, the generated helm chart has an indentation error that makes it invalid yaml. The error looks like the following in amoro-deployment.yaml.

		Error: yaml: line 93: block sequence entries are not allowed in this context

Affects Versions

master

What table formats are you seeing the problem on?

No response

What engines are you seeing the problem on?

No response

How to reproduce

Set any non null value to amoroConf.log4j2 in values.yaml, and generate the helm template with helm template --debug ../amoro

Relevant log output

Error: yaml: line 93: block sequence entries are not allowed in this context

Anything else

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

  • I agree to follow this project's Code of Conduct

Metadata

Metadata

Assignees

No one assigned

    Labels

    type:bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions