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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: modify the audit message for org update #2706

Merged

Conversation

littlejiancc
Copy link
Contributor

What type of this PR

/kind bug

What this PR does / why we need it:

fix: modify the audit message for org update

Which issue(s) this PR fixes:

Specified Reviewers:

/assign @

ChangeLog

Language Changelog
馃嚭馃嚫 English
馃嚚馃嚦 涓枃

Need cherry-pick to release versions?

/cherry-pick release/1.4

Add comment like /cherry-pick release/1.0 when this PR is merged.

For details on the cherry pick process, see the cherry pick requests section under CONTRIBUTING.md.

@erda-bot
Copy link
Member

Automated cherry pick can ONLY be triggered when this PR is MERGED!

@littlejiancc littlejiancc requested review from Effet and sfwn and removed request for Effet October 29, 2021 12:24
@codecov
Copy link

codecov bot commented Oct 29, 2021

Codecov Report

Merging #2706 (649c41d) into master (e7907ba) will increase coverage by 0.06%.
The diff coverage is 63.29%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2706      +/-   ##
==========================================
+ Coverage   16.68%   16.74%   +0.06%     
==========================================
  Files        1261     1261              
  Lines      128176   128238      +62     
==========================================
+ Hits        21381    21476      +95     
+ Misses     104600   104555      -45     
- Partials     2195     2207      +12     
Impacted Files Coverage 螖
apistructs/org.go 0.00% <酶> (酶)
modules/core-services/services/org/org.go 15.25% <63.29%> (+7.36%) 猬嗭笍
...ules/scheduler/executor/plugins/k8s/check_quota.go 34.45% <0.00%> (-2.14%) 猬囷笍
...mponents/cmp-dashboard-nodes/common/table/table.go 12.97% <0.00%> (-1.28%) 猬囷笍
pkg/resourcecalculator/calculator.go 84.93% <0.00%> (-0.78%) 猬囷笍
modules/cmp/component-protocol/cputil/util.go 21.52% <0.00%> (-0.30%) 猬囷笍
modules/core-services/services/project/project.go 9.84% <0.00%> (-0.10%) 猬囷笍
.../cmp-dashboard-workload-detail/podsTable/render.go 23.47% <0.00%> (酶)
.../components/cmp-dashboard-nodes/podTable/render.go 14.28% <0.00%> (+0.28%) 猬嗭笍
...p-dashboard-workloads-list/workloadChart/render.go 8.23% <0.00%> (+0.54%) 猬嗭笍
... and 4 more

@littlejiancc littlejiancc force-pushed the feature/org-update-audit branch 2 times, most recently from 151cb0c to 15fb6be Compare November 1, 2021 03:59
@sfwn
Copy link
Member

sfwn commented Nov 1, 2021

/approve

@erda-bot erda-bot merged commit 0845947 into erda-project:master Nov 1, 2021
@littlejiancc
Copy link
Contributor Author

/cherry-pick release/1.4

erda-bot pushed a commit to erda-bot/erda that referenced this pull request Nov 1, 2021
erda-bot added a commit that referenced this pull request Nov 1, 2021
Co-authored-by: littlejian <17816869670@163.com>
@littlejiancc littlejiancc deleted the feature/org-update-audit branch March 29, 2022 02:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

3 participants