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

Broken annotations for standard Ingress resource #1054

Closed
miry opened this issue May 17, 2018 · 4 comments
Closed

Broken annotations for standard Ingress resource #1054

miry opened this issue May 17, 2018 · 4 comments
Labels

Comments

@miry
Copy link

miry commented May 17, 2018

I use 7.0.0-rc.1 version.

I tried to create a standard resource with features https://github.com/appscode/voyager/blob/master/docs/guides/ingress/pod-placement.md#using-node-selector

apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: letsencrypt
  annotations:
    kubernetes.io/ingress.class: "voyager"
    ingress.appscode.com/type: HostPort
    ingress.appscode.com/replicas: '1'
    ingress.appscode.com/node-selector: '{"role": "edge"}'
    ingress.appscode.com/tolerations: '[{"key": "IngressOnly", "operator": "Equal", "value": "true", "effect": "NoSchedule"}]'
    ingress.appscode.com/workload-kind: DaemonSet
spec:
  rules:
    - host: 'logs.example.com
      http:
        paths:
        - path: /
          backend:
            serviceName: nginx
            servicePort: 80

And got:

 $ kubectl get daemonset               
NAME                  DESIRED   CURRENT   READY     UP-TO-DATE   AVAILABLE   NODE SELECTOR   AGE
voyager-letsencrypt   1         1         1         1            1           <none>          1m

There is missing tolerations and node selector.

@tamalsaha
Copy link
Contributor

Hi, please check the syntax for node selector and toleration https://appscode.com/products/voyager/7.0.0-rc.1/guides/ingress/pod-placement/ . Try with the correct syntax, it should work. :) Please reopen if you see any issues.

@miry
Copy link
Author

miry commented May 18, 2018

@tamalsaha in this guide there are example that valid for resource: apiVersion: voyager.appscode.com/v1beta1. And they are not valid for apiVersion: extensions/v1beta1.

@tamalsaha
Copy link
Contributor

@miry , I have added pr #1059 to fix this. Can you try the image appscode/voyager:node-sel and let me know if that solves this?

In general, we prefer and recommend using the CRD, since it has lot more feature support in a natural way than shoving everything into a string->string map (annotations).

@tamalsaha
Copy link
Contributor

This has been released as part of https://github.com/appscode/voyager/releases/tag/7.0.0-rc.2 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants