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

bug: plugin config in APISIX but do not in declare yaml #974

Closed
GhangZh opened this issue Apr 20, 2022 · 6 comments
Closed

bug: plugin config in APISIX but do not in declare yaml #974

GhangZh opened this issue Apr 20, 2022 · 6 comments

Comments

@GhangZh
Copy link
Contributor

GhangZh commented Apr 20, 2022

Issue description

After upgrading apisix 1.13and apisix-ingress-controller 1.4 through helm, ingress-controller will automatically synchronize the current ar file to apisix and add the field plugin_config_id

apiVersion: apisix.apache.org/v2beta3
kind: ApisixRoute
metadata:
  name: test
  namespace: test
spec:
  http:
  - backends:
    - serviceName: test
      servicePort: 80
    match:
      hosts:
      - apisix-test.com
      paths:
      - /cluster/idc*
    name: test
    plugins:
    - name: proxy-rewrite
      enable: true
      config:
        regex_uri: ["^/cluster/idc(.*)", "/$1"]

image

apisix-ingress-controller log
image

curl http://127.0.0.1:9180/apisix/admin/plugin_configs -H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1'

image

Environment

  • your apisix-ingress-controller version (output of apisix-ingress-controller version --long):
  • your Kubernetes cluster version (output of kubectl version):
  • if you run apisix-ingress-controller in Bare-metal environment, also show your OS version (uname -a):

Minimal test code / Steps to reproduce

kubectl apply -f test.yaml

kubectl exec -it apisix-544b6cbf74-npr5z -n ingress-apisix bash

curl http://127.0.0.1:9180/apisix/admin/plugin_configs -H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1'

Actual result

No plugin_config_id

Error log

	warn	ingress/compare.go:186pluginConfig: 2930e8f3 in APISIX but do not in declare yaml
2022-04-20T16:13:34+08:00      
	warn	ingress/compare.go:186pluginConfig: 2930e8f3 in APISIX but do not in declare yaml

Expected result

No response

@GhangZh GhangZh changed the title bug: bug: plugin config in APISIX but do not in declare yaml Apr 20, 2022
@tao12345666333
Copy link
Member

The problem has been fixed

#836

@GhangZh
Copy link
Contributor Author

GhangZh commented Apr 21, 2022

The problem has been fixed

#836

Is there a way to circumvent it now. Even if I call the admin-api interface to remove it, the ingresscontroller will still add this field to apisix again

@tao12345666333
Copy link
Member

@GhangZh you need to use the master branch to build the image

@GhangZh
Copy link
Contributor Author

GhangZh commented Apr 21, 2022

@GhangZh you need to use the master branch to build the image

ok,thanks

@tao12345666333
Copy link
Member

@GhangZh After discussions we decided to release v1.4.1 #980

@tao12345666333
Copy link
Member

After v1.4.1 is released, you can upgrade to fix this issue. I will close this issue, we will track it with #980

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

No branches or pull requests

2 participants