Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[release-2.7] Allow multiline templatization #420

Conversation

willkutler
Copy link
Contributor

@willkutler willkutler commented Feb 13, 2023

Adds a new field, object-templates-raw, that allows users to paste in an object template. This makes multiline templates, like those using the range keyword, possible. It can be set instead of object-templates, but both cannot be set.

ref: https://issues.redhat.com/browse/ACM-2739?filter=-1

This is a backport of the fixes in open-cluster-management-io/config-policy-controller#95 and open-cluster-management-io/config-policy-controller#105

Signed-off-by: Will Kutler wkutler@redhat.com

Adds a new field, object-templates-raw, that allows users to paste in an object template. This makes multiline templates, like those using the range keyword, possible. It can be set instead of object-templates, but both cannot be set.

ref: https://issues.redhat.com/browse/ACM-2739?filter=-1

Signed-off-by: Will Kutler <wkutler@redhat.com>
@openshift-ci
Copy link

openshift-ci bot commented Feb 13, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: willkutler

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sonarcloud
Copy link

sonarcloud bot commented Feb 13, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

88.9% 88.9% Coverage
0.0% 0.0% Duplication

@mprahl mprahl changed the title Allow multiline templatization [release-2.7] Allow multiline templatization Feb 14, 2023
@mprahl
Copy link
Contributor

mprahl commented Feb 14, 2023

I asked in Slack to cherry-pick the initial commit and the commit with the bug fix rather than combining them so that the Git history is consistent.

@willkutler
Copy link
Contributor Author

closing as this has been merged as 2 cherry picked PRs

@willkutler willkutler closed this Feb 15, 2023
@willkutler willkutler deleted the 2.7-multiline-templatization-backport branch February 15, 2023 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants