apache / rocketmq-operator

Apache RocketMQ Operator
https://rocketmq.apache.org/
Apache License 2.0
308 stars 126 forks source link

[ISSUE #185] Fix pod ready status reflect to cr #186

Closed drivebyer closed 6 months ago

drivebyer commented 9 months ago

What is the purpose of the change

https://github.com/apache/rocketmq-operator/issues/185

Brief changelog

Fix pod ready status reflect to cr.

Verifying this change

Apply:

apiVersion: rocketmq.apache.org/v1alpha1
kind: Broker
metadata:
  name: broker
  namespace: mcamel-system
spec:
  allowRestart: true
  brokerImage: apacherocketmq/rocketmq-broker:4.5.0-alpine-operator-0.3.0
  env:
  - name: BROKER_MEM
    valueFrom:
      configMapKeyRef:
        key: BROKER_MEM
        name: broker-config
  hostPath: /data/rocketmq/broker
  imagePullPolicy: Always
  nameServers: ""
  replicaPerGroup: 1
  resources:
    limits:
      cpu: 500m
      memory: 20Mi          ########### low memory here #############
    requests:
      cpu: 250m
      memory: 10Mi          ########### low memory here ############# 
  scalePodName: broker-0-master-0
  size: 1
  storageMode: EmptyDir
  volumeClaimTemplates:
  - metadata:
      annotations:
        volume.beta.kubernetes.io/storage-class: rocketmq-storage
      name: broker-storage
    spec:
      accessModes:
      - ReadWriteOnce
      resources:
        requests:
          storage: 8Gi
  volumes:
  - configMap:
      items:
      - key: broker-common.conf
        path: broker-common.conf
      name: broker-config
    name: broker-config

Status always empty

Please go through this checklist to help us incorporate your contribution quickly and easily.

Notice: It would be helpful if you could finish the following checklist (the last one is not necessary) before request the community to review your PR.