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

Noise report using cloud providers #326

Open
krol3 opened this issue Jun 10, 2023 · 1 comment
Open

Noise report using cloud providers #326

krol3 opened this issue Jun 10, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@krol3
Copy link

krol3 commented Jun 10, 2023

Hi! I'm using an EKS cluster version 1.24, with Krane, It was installed Krane v0.1.1 release, using the helm installation.

Using the krane report --incluster

In the report generated by krane is mapped as danger default RBAC resources from EKS, this cause noise in the report.

I would like to see your opinion about this topic, I suppose that will be the same for GKE, AKS or OpenShift. Thank you.

Result:

{
  "summary": {
    "danger": 9,
    "warning": 9,
    "info": 1,
    "success": 35
  },
  "results": [
    {
      "id": "risky-any-resource-list",
      "status": "danger",
      "group_title": "Risky Roles/ClusterRoles allowing list action on all resources",
      "info": "Roles/ClusterRoles allowing list action on all resources. This might be dangerous. Review listed
Roles!",
      "items": [
        "ClusterRole aws-node in * namespace(s)",
        "ClusterRole eks:addon-manager in * namespace(s)",
        "ClusterRole ks-sa-roles in * namespace(s)",
      ]
    },
@krol3 krol3 added the enhancement New feature or request label Jun 10, 2023
@marcinc
Copy link
Contributor

marcinc commented Jul 26, 2023

This is certainly something that could be improved. As it stands the tool doesn't distinguish between vendor specific roles and custom roles. In the meantime you could perhaps look at whitelists. Bear in mind that currently helm doesn't support option for passing custom configuration files at install time. There is a PR opened around that though (#253) so the ability to pass these things in should land soon.

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

No branches or pull requests

2 participants