fix(k8s): modify inconsistent apiGroups

Radeity commented 10 months ago

Purpose of the PR

This bug is introduced by https://github.com/apache/incubator-hugegraph-computer/pull/259
Inconsistent apiGroups in ClusterRole definition, which will cause the following permission error during launching hugegraphcomputerjob-controller:

io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: GET at: https://10.96.0.1/apis/operator.hugegraph.apache.org/v1/hugegraphcomputerjobs. Message: Forbidden!Configured service account doesn't have access. Service account may have been revoked. hugegraphcomputerjobs.operator.hugegraph.apache.org is forbidden: User "system:serviceaccount:hugegraph-computer-operator-system:hugegraph-computer-operator-controller-manager" cannot list resource "hugegraphcomputerjobs" in API group "operator.hugegraph.apache.org" at the cluster scope.

Main Changes

Modify KubeBuilder comment and generate role.yaml with correct apiGroups.

Verifying these changes

[x] Trivial rework / code cleanup without any test coverage. (No Need)
[ ] Already covered by existing tests, such as (please modify tests here).
[ ] Need tests and can be verified as follows.

Does this PR potentially affect the following parts?

[x] Nope
[ ] Dependencies (add/update license info)
[ ] Modify configurations
[ ] The public API
[ ] Other affects (typed here)

Documentation Status

[ ] Doc - TODO
[ ] Doc - Done
[x] Doc - No Need

Radeity commented 10 months ago

cc @coderzc

Radeity commented 10 months ago

Hi @coderzc , CI meets TransportException, plz help rerun CI :D

codecov[bot] commented 10 months ago

Codecov Report

Merging #270 (ede614f) into master (a21a191) will increase coverage by 0.00%. The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #270   +/-   ##
=========================================
  Coverage     85.82%   85.83%           
- Complexity     3233     3234    +1     
=========================================
  Files           344      344           
  Lines         12124    12124           
  Branches       1092     1092           
=========================================
+ Hits          10406    10407    +1     
- Misses         1193     1194    +1     
+ Partials        525      523    -2

see 3 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

apache / incubator-hugegraph-computer