Linkerd2: Increase the comprehensiveness of check --pre

Created on 9 Aug 2019  路  6Comments  路  Source: linkerd/linkerd2

What problem are you trying to solve?

check is awesome and provides a way for users to reason about whether linkerd can run effectively on their clusters. Unfortunately, it doesn't test for somethings. For example, while users can create ClusterRoles, it is possible that they don't have the broad permissions required to actually create these with the correct permissions required.

How should the problem be solved?

For --pre, it would provide better results to do something similar to what kubectl apply --server-dry-run does.

arecli good first issue help wanted prioritP1

All 6 comments

As part of this, it would be nice to add a check to validate that the injector is operating correctly. This could simply try and create a pod, and see if it was correctly injected. The --server-dry-run should go through the injector and give some information back for that. See #3290 and #2973 for two examples of this failure.

can i take this as per slack discussion ?

Go for it!

Hi @Pac23

I was about to start working on this issue and I was wondering if you were still interested in it.

Thanks!

Hi @Pac23

I was about to start working on this issue and I was wondering if you were still interested in it.

Thanks!

i found some of the bits of the solution to this issue slightly difficult as a i learned go only couple of weeks back,please take this. :sweat_smile:

Thanks @Pac23!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

steve-fraser picture steve-fraser  路  4Comments

briansmith picture briansmith  路  4Comments

alpeb picture alpeb  路  3Comments

manimaul picture manimaul  路  3Comments

zaharidichev picture zaharidichev  路  4Comments