Skip to content

Commit

Permalink
Merge pull request #16994 from jmchilton/job_metrics_fix
Browse files Browse the repository at this point in the history
Fixes for job metrics Galaxy markdown component.
  • Loading branch information
jmchilton authored Nov 9, 2023
2 parents f91693d + 200d03a commit ff45b63
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
2 changes: 1 addition & 1 deletion client/src/components/JobMetrics/JobMetrics.vue
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ const estimatedServerInstance = computed(() => {
</div>

<AwsEstimate
v-if="jobRuntimeInSeconds && coresAllocated && ec2Instances"
v-if="jobRuntimeInSeconds && coresAllocated && ec2Instances && shouldShowAwsEstimate"
:ec2-instances="ec2Instances"
:should-show-aws-estimate="shouldShowAwsEstimate"
:job-runtime-in-seconds="jobRuntimeInSeconds"
Expand Down
7 changes: 6 additions & 1 deletion client/src/components/Markdown/Elements/JobMetrics.vue
Original file line number Diff line number Diff line change
@@ -1,6 +1,11 @@
<template>
<b-card nobody>
<JobMetrics class="job-metrics" :job-id="args.job_id" />
<JobMetrics
class="job-metrics"
:job-id="args.job_id"
:should-show-aws-estimate="false"
:should-show-carbon-emissions-estimates="false"
:include-title="false" />
</b-card>
</template>

Expand Down

0 comments on commit ff45b63

Please sign in to comment.