apache / apisix-ingress-controller

APISIX Ingress Controller for Kubernetes
https://apisix.apache.org/
Apache License 2.0
1.01k stars 344 forks source link

bug: Some CRD missing status sub-resource #1784

Open AlinsRan opened 1 year ago

AlinsRan commented 1 year ago

Current Behavior

Due to the lack of definition for status sub-resource, and status cannot be recorded.

Missing CRDs for status sub-resource:

Similar to #1619 , we need to add the definition of the status field and related e2e testing in the v2 apiversion.

https://github.com/apache/apisix-ingress-controller/blob/8e437006dc227c1bccf3fec93d70cdac40dc31a3/samples/deploy/crd/v1/ApisixUpstream.yaml#L827-L844

Environment

Chever-John commented 1 year ago

Let me try it:)

AlinsRan commented 1 year ago

Assigned, Thanks.

afzal442 commented 1 year ago

Hi, LMK if there is any update. I'd like to take it up.

cc @tao12345666333

tao12345666333 commented 1 year ago

Assigned. Thanks

Chever-John commented 1 year ago

Emmmm, there's some work going on right now. I can give it another two days. (fixing e2e-test phase

afzal442 commented 1 year ago

Alright @Chever-John ! Of course you can.

Chever-John commented 1 year ago

@afzal442 Thanks:)

I will keep you all informed if there is an update notice, for example, if I cannot proceed, and you are welcome to continue with the work. Have a happy day!