Skip to content

Commit

Permalink
Merge pull request #1236 from cdapio/view-run-records-in-ui
Browse files Browse the repository at this point in the history
Show tooltip for pipeline runs based on  `app.run.records.ttl.days` configuration
  • Loading branch information
radhikav1 authored Aug 1, 2024
2 parents 23124fb + cc46a3a commit 55b418b
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,8 @@ const CurrentRunIndex = ({
runsCount - (runs.length - currentRunIndex)
);

const runLimit = Number(window.CDAP_CONFIG.cdap.runRecordsTtl) || 0;

const pipelineLink = getHydratorUrl({
stateName: 'hydrator.detail',
stateParams: {
Expand Down Expand Up @@ -192,7 +194,16 @@ const CurrentRunIndex = ({

return (
<div className="run-number-container run-info-container">
<h4 className="run-number">
<h4
className="run-number"
title={
runLimit > 0
? T.translate(`${PREFIX}.tooltipRunLimit`, {
runLimit,
}).toString()
: ''
}
>
{T.translate(`${PREFIX}.currentRunIndex`, {
currentRunIndex: runIndexInTotalRunsCount + 1,
numRuns: runsCount,
Expand Down
1 change: 1 addition & 0 deletions app/cdap/text/text-en.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2505,6 +2505,7 @@ features:
noInfo: Profile information unavailable
runsCurrentlyRunning: Runs currently running - select one to view
status: Status
tooltipRunLimit: Pipeline runs for only the last {runLimit} days are displayed
warnings: Warnings
startTime: Start time
TopPanel:
Expand Down
1 change: 1 addition & 0 deletions server/express.js
Original file line number Diff line number Diff line change
Expand Up @@ -257,6 +257,7 @@ function makeApp(authAddress, cdapConfig, uiSettings) {
hstsMaxAge: cdapConfig['hsts.max.age'],
hstsIncludeSubDomains: cdapConfig['hsts.include.sub.domains'],
hstsPreload: cdapConfig['hsts.preload'],
runRecordsTtl: cdapConfig['app.run.records.ttl.days'],
},
hydrator: {
previewEnabled: cdapConfig['enable.preview'] === 'true',
Expand Down

0 comments on commit 55b418b

Please sign in to comment.