Skip to content

Conversation

mengskysama
Copy link

No description provided.

Copy link
Member

@siggy siggy left a comment

Choose a reason for hiding this comment

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

@mengskysama Thanks for doing this! We'd like to merge part of this PR. The CRD -> TPR change is happening as part of #203, would you backing that part of your PR out, and then we can merge the rest?

---
kind: ThirdPartyResource
apiVersion: extensions/v1beta1
kind: CustomResourceDefinition
Copy link
Member

Choose a reason for hiding this comment

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

we're updating from TPR -> CRD in a larger doc update (#203). would you mind backing this particular change out of your PR?

kubectl apply -f k8s/certificates.yml
kubectl apply -f k8s/namerd.yml
kubectl apply -f k8s/linkerd-namerd-cni.yml
kubectl apply -f k8s/linkerd-namerd-cni-tls.yml
Copy link
Member

Choose a reason for hiding this comment

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

nice catch!


#### Daemonsets

If cluster enable RBAC please setting up the permission:
Copy link
Member

Choose a reason for hiding this comment

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

This same information is capture in the RBAC section further down this file. What do you think about moving that section up to here, and combining it with what you have documented here?


```bash
kubectl apply -f k8s/certificates.yml
kubectl apply -f k8s/namerd-legacy.yml
Copy link
Member

Choose a reason for hiding this comment

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

we'll handle this as part of #203.

@siggy siggy added the review label Jan 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants