Skip to content

Conversation

@rherrick
Copy link
Contributor

@rherrick rherrick commented Jun 2, 2020

What does this PR do?

The contents of the IngressRoute tab have whitespace errors. When you copy that and paste it into a file you get the following error:

ingressroute.traefik.containo.us/myingressroute created
error: error parsing traefik-4.yaml: error converting YAML to JSON: yaml: line 2: mapping values are not allowed in this context

This change corrects the format for that file so that it properly creates the ingress routes:

deployment.apps/whoami created
service/whoami created
deployment.apps/whoamitcp created
service/whoamitcp created
deployment.apps/whoamiudp created
service/whoamiudp created

Motivation

I'm trying to get Traefik working in Kubernetes to integrate into a deployable solution. It's hard to figure these things out when the samples don't work :)

More

  • Added/updated tests
  • Added/updated documentation

Copy link
Member

@juliens juliens left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ldez ldez left a comment

Choose a reason for hiding this comment

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

Thanks 👍

The contents of the IngressRoute tab have whitespace errors. When you copy that and paste it into a file you get the following error:

ingressroute.traefik.containo.us/myingressroute created
error: error parsing traefik-4.yaml: error converting YAML to JSON: yaml: line 2: mapping values are not allowed in this context

This change corrects the format for that file so that it properly creates the ingress routes:

deployment.apps/whoami created
service/whoami created
deployment.apps/whoamitcp created
service/whoamitcp created
deployment.apps/whoamiudp created
service/whoamiudp created
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.

5 participants