Skip to content
This repository has been archived by the owner on Nov 2, 2023. It is now read-only.

[BUG] yurtingress create fail #121

Open
huiwq1990 opened this issue Oct 1, 2022 · 1 comment
Open

[BUG] yurtingress create fail #121

huiwq1990 opened this issue Oct 1, 2022 · 1 comment
Labels
kind/bug kind/bug

Comments

@huiwq1990
Copy link
Member

What happened:

When create yurtingress, yurtappmanager report clustrole yurt-app-manager-role not found.

What you expected to happen:

How to reproduce it (as minimally and precisely as possible):

helm upgrade --install -n kube-system yurt-app-manager openyurt/yurt-app-manager

cat<<EOF | kubectl apply -f -
apiVersion: apps.openyurt.io/v1alpha1
kind: YurtIngress
metadata:
  name: yurtingress-demo
spec:
  ingress_controller_replicas_per_pool: 2
  ingress_controller_image: huiwq1990/ingress-nginx-controller:v0.49.0
  ingress_webhook_certgen_image: docker.io/jettech/kube-webhook-certgen:v1.5.1
  pools:
    - name: sh-works
EOF

Anything else we need to know?:

Yurtingress depend clusterrole yurt-app-manager-role, as the code https://github.com/openyurtio/yurt-app-manager/blob/master/pkg/yurtappmanager/util/kubernetes/apply_addons.go#L45.
But yurtappmanager's clusterrole depend serviceaccount, as the code https://github.com/openyurtio/yurt-app-manager/blob/master/charts/yurt-app-manager/templates/clusterrole.yaml#L4.

Environment:

  • OpenYurt version:
  • Kubernetes version (use kubectl version):
  • OS (e.g: cat /etc/os-release):
  • Kernel (e.g. uname -a):
  • Install tools:
  • Others:

others

/kind bug

@huiwq1990 huiwq1990 added the kind/bug kind/bug label Oct 1, 2022
@rambohe-ch
Copy link
Member

rambohe-ch commented Oct 2, 2022

@huiwq1990 which version of helm charts did you deploy?
maybe you can have a try to use tag: openyurt-1.0.2 of openyurtio/openyurt-helm.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/bug kind/bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants