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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

get execute env from report env #2064

Merged

Conversation

chengjoey
Copy link
Contributor

What type of this PR

Add one of the following kinds:
/kind bug

What this PR does / why we need it:

Which issue(s) this PR fixes:

Specified Reviewers:

/assign @your-reviewer

ChangeLog

Language Changelog
🇺🇸 English get execute env from report env
🇨🇳 中文 执行计划增加真实环境的显示

Need cherry-pick to release versions?

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.

@chengjoey chengjoey added bug dop devops plaoform labels Sep 23, 2021
@chengjoey
Copy link
Contributor Author

specify env:
image

empty env:
image

@codecov
Copy link

codecov bot commented Sep 23, 2021

Codecov Report

Merging #2064 (fbbae33) into master (01e3901) will increase coverage by 0.32%.
The diff coverage is 46.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2064      +/-   ##
==========================================
+ Coverage   15.17%   15.49%   +0.32%     
==========================================
  Files        1160     1161       +1     
  Lines      113308   113480     +172     
==========================================
+ Hits        17198    17588     +390     
+ Misses      94365    93913     -452     
- Partials     1745     1979     +234     
Impacted Files Coverage Δ
...ine/aop/plugins/pipeline/testplan_before/plugin.go 22.85% <0.00%> (-0.68%) ⬇️
...-test-plan-detail/components/executeInfo/render.go 6.09% <50.00%> (+1.14%) ⬆️
...ents/cmp-dashboard-workloads-list/filter/render.go 8.10% <0.00%> (-2.80%) ⬇️
...col/components/cmp-dashboard-pods/filter/render.go 7.14% <0.00%> (-2.37%) ⬇️
modules/msp/menu/menu.service.go 46.66% <0.00%> (-2.23%) ⬇️
modules/cmp/steve/proxy/cache_store.go 51.45% <0.00%> (-0.10%) ⬇️
modules/cmp/steve/server.go 0.00% <0.00%> (ø)
modules/cmp/steve/aggregator.go 0.00% <0.00%> (ø)
modules/cmp/steve/response.go 0.00% <0.00%> (ø)
... and 11 more

if err != nil {
return ""
}
return fmt.Sprintf("%v", configData.Meta[autotest.CmsCfgKeyDisplayName])
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how about Meta is nil.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

judge autotest.CmsCfgKeyDisplayName is existed in meta
add test if meta is nil

@chengjoey chengjoey force-pushed the fix/autotest-execute-config-name branch from cbff251 to fbbae33 Compare September 23, 2021 11:55
@sfwn
Copy link
Member

sfwn commented Sep 24, 2021

/approve

@erda-bot erda-bot merged commit 347b50a into erda-project:master Sep 24, 2021
@sfwn
Copy link
Member

sfwn commented Sep 24, 2021

/cherry-pick release/1.3

erda-bot pushed a commit to erda-bot/erda that referenced this pull request Sep 24, 2021
erda-bot added a commit that referenced this pull request Sep 26, 2021
Co-authored-by: chengjoey <30427474+chengjoey@users.noreply.github.com>
@chengjoey chengjoey deleted the fix/autotest-execute-config-name branch April 18, 2022 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved bug dop devops plaoform
Development

Successfully merging this pull request may close these issues.

3 participants