Skip to content

Conversation

@trawler
Copy link
Contributor

@trawler trawler commented Jul 6, 2018

No description provided.

@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jul 6, 2018
@trawler trawler requested review from enxebre, spangenberg and squat July 6, 2018 11:38
Copy link
Contributor

@squat squat left a comment

Choose a reason for hiding this comment

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

Looks great @trawler

@squat
Copy link
Contributor

squat commented Jul 6, 2018

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 6, 2018
@spangenberg
Copy link
Contributor

/lgtm

@trawler trawler merged commit 5296ce0 into openshift:master Jul 6, 2018
@trawler trawler deleted the ingress_tls branch July 6, 2018 12:24
wking added a commit to wking/openshift-installer that referenced this pull request Jul 10, 2018
Both of these lines were touched by ae41b0a (Merge pull request openshift#22
from trawler/kube_tls_certs, 2018-07-09), but there's no need to use
Sprintf to inject string literals into the format template.  26930f2
(post-review fixes, 2018-07-07, openshift#22) fixed one such instance, but left
these.  The * instance dates back to bf61dc9
(installer/pkg/config-generator/tls: generate ingress certs,
2018-07-06, openshift#16).
stbenjam pushed a commit to stbenjam/installer that referenced this pull request Apr 30, 2019
Integrate coredns with the mdns plugin
stbenjam pushed a commit to stbenjam/installer that referenced this pull request Feb 10, 2021
Allow ClusterOperator to be set to Degraded state
clnperez added a commit to clnperez/installer that referenced this pull request Jul 2, 2021
add infrastructure asset and revendor

Signed-off-by: Christy Norman <[email protected]>
clnperez added a commit to clnperez/installer that referenced this pull request Jul 16, 2021
add infrastructure asset and revendor

Signed-off-by: Christy Norman <[email protected]>
clnperez added a commit to clnperez/installer that referenced this pull request Jul 20, 2021
add infrastructure asset and revendor

Signed-off-by: Christy Norman <[email protected]>
clnperez added a commit to clnperez/installer that referenced this pull request Jul 22, 2021
add infrastructure asset and revendor

Signed-off-by: Christy Norman <[email protected]>
clnperez added a commit to clnperez/installer that referenced this pull request Sep 2, 2021
add infrastructure asset and revendor

Signed-off-by: Christy Norman <[email protected]>
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/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants