Skip to content

Conversation

@alecmerdler
Copy link
Contributor

Description

Wraps JSON.parse() in error handling and passes null for the template if the annotation is invalid.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1671125

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 5, 2019
Copy link
Member

@spadgett spadgett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 5, 2019
@spadgett
Copy link
Member

spadgett commented Feb 5, 2019

/test all

1 similar comment
@alecmerdler
Copy link
Contributor Author

/test all

@openshift-merge-robot
Copy link
Contributor

/retest

1 similar comment
@openshift-merge-robot
Copy link
Contributor

/retest

@spadgett
Copy link
Member

spadgett commented Feb 6, 2019

/test frontend
/test backend
/test console-e2e

@spadgett
Copy link
Member

spadgett commented Feb 6, 2019

/retest

2 similar comments
@spadgett
Copy link
Member

spadgett commented Feb 7, 2019

/retest

@spadgett
Copy link
Member

spadgett commented Feb 7, 2019

/retest

@spadgett
Copy link
Member

spadgett commented Feb 7, 2019

I think we might need to bump the memory request for our test pod

/retest

@spadgett
Copy link
Member

spadgett commented Feb 7, 2019

/retest

1 similar comment
@spadgett
Copy link
Member

spadgett commented Feb 8, 2019

/retest

@openshift-merge-robot openshift-merge-robot merged commit cad3827 into openshift:master Feb 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants