Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add missing verb to the role of node lifecycle controller #1936

Merged
merged 1 commit into from
Jan 31, 2024

Conversation

crazytaxii
Copy link
Contributor

What type of PR is this?

/kind bug

What this PR does / why we need it:

Node lifecycle controller in yurt-manager maybe miss rbac settings.

Which issue(s) this PR fixes:

Fixes #1934

Does this PR introduce a user-facing change?

NONE

Copy link

codecov bot commented Jan 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (da03883) 52.04% compared to head (54fd83a) 52.04%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1936   +/-   ##
=======================================
  Coverage   52.04%   52.04%           
=======================================
  Files         174      174           
  Lines       21213    21213           
=======================================
  Hits        11040    11040           
  Misses       9199     9199           
  Partials      974      974           
Flag Coverage Δ
unittests 52.04% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rambohe-ch
Copy link
Member

@crazytaxii charts/yurt-manager/crds/apiextensions.k8s.io_v1_customresourcedefinition_blockaffinities.apiregistration.k8s.io.yaml file need to be ignored.

Signed-off-by: HF <crazytaxii666@gmail.com>
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@rambohe-ch rambohe-ch added approved approved lgtm lgtm labels Jan 31, 2024
@rambohe-ch
Copy link
Member

/lgtm

@rambohe-ch rambohe-ch merged commit 3c3b3cb into openyurtio:master Jan 31, 2024
13 checks passed
@crazytaxii crazytaxii deleted the fix branch January 31, 2024 08:49
zyjhtangtang pushed a commit to zyjhtangtang/openyurt that referenced this pull request Apr 16, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
approved approved lgtm lgtm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] node lifecycle controller in yurt-manager can not update status of node
2 participants