Rename job trace to job logs in frontend code
As mentioned in https://docs.gitlab.com/ee/administration/job_logs.html, we renamed job trace to job log in GitLab 12.5, but we still often use the term trace
internally because our code still uses the legacy name. We also use the term live trace
for https://docs.gitlab.com/ee/administration/job_logs.html#enabling-incremental-logging.
Javascript
$ git grep -i "job.*trace" | grep \.js
app/assets/javascripts/ide/stores/modules/pipelines/actions.js: .get(`${state.detailJob.path}/trace`, { params: { format: 'json' } })
app/assets/javascripts/jobs/store/actions.js: return Promise.all([dispatch('fetchJob'), dispatch('fetchTrace')]);
app/assets/javascripts/jobs/store/actions.js: * Job's Trace
app/assets/javascripts/jobs/store/getters.js: * Checks if it the job has trace.
app/assets/javascripts/jobs/store/getters.js: state.job.has_trace || (!isEmpty(state.job.status) && state.job.status.group === 'running');
app/assets/javascripts/jobs/store/mutations.js: // When the job still does not have a trace
Specs
ee/spec/frontend/environments_dashboard/components/mock_environment.json: "title": "This job does not have a trace."
ee/spec/frontend/security_dashboard/mock_data/jobs.js: downloadPath: '/debug-cov-fuzz-project/-/jobs/1133/artifacts/download?file_type=trace',
ee/spec/frontend/security_dashboard/mock_data/jobs.js: downloadPath: '/debug-cov-fuzz-project/-/jobs/1131/artifacts/download?file_type=trace',
spec/features/projects/jobs_spec.rb: context 'when job is failed but has no trace', :js do
spec/features/projects/jobs_spec.rb: describe "GET /:project/jobs/:id/trace.json" do
spec/frontend/environments/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/components/job_app_spec.js: const setupAndMount = ({ jobData = {}, traceData = {} } = {}) => {
spec/frontend/jobs/components/job_app_spec.js: const findJobTraceScrollTop = () => wrapper.find('[data-testid="job-controller-scroll-top"]');
spec/frontend/jobs/components/job_app_spec.js: const findJobTraceScrollBottom = () =>
spec/frontend/jobs/components/job_app_spec.js: const findJobTraceController = () => wrapper.find('[data-testid="job-raw-link-controller"]');
spec/frontend/jobs/components/job_app_spec.js: const findJobTraceEraseLink = () => wrapper.find('[data-testid="job-log-erase-link"]');
spec/frontend/jobs/components/job_app_spec.js: it('renders empty state when job does not have trace and is not running', () =>
spec/frontend/jobs/components/job_app_spec.js: it('does not render empty state when job does not have trace but it is running', () =>
spec/frontend/jobs/components/job_app_spec.js: it('does not render empty state when job has trace but it is not running', () =>
spec/frontend/jobs/components/job_app_spec.js: setupAndMount({ jobData: { has_trace: true } }).then(() => {
spec/frontend/jobs/components/job_app_spec.js: expect(findJobTraceScrollTop().exists()).toBe(true);
spec/frontend/jobs/components/job_app_spec.js: expect(findJobTraceScrollBottom().exists()).toBe(true);
spec/frontend/jobs/components/job_app_spec.js: expect(findJobTraceController().exists()).toBe(true);
spec/frontend/jobs/components/job_app_spec.js: expect(findJobTraceEraseLink().exists()).toBe(true);
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: title: 'This job does not have a trace.',
spec/frontend/jobs/mock_data.js: downloadPath: '/root/ci-project/-/jobs/2336/artifacts/download?file_type=trace',
spec/frontend/jobs/mock_data.js: downloadPath: '/root/test-job-artifacts/-/jobs/1982/artifacts/download?file_type=trace',
spec/frontend/jobs/store/actions_spec.js: { pagePath: 'job/872324/trace.json' },
spec/frontend/jobs/store/actions_spec.js: [{ type: types.SET_TRACE_OPTIONS, payload: { pagePath: 'job/872324/trace.json' } }],
spec/frontend/jobs/store/getters_spec.js: localState.job.has_trace = true;
spec/frontend/jobs/store/getters_spec.js: localState.job.has_trace = false;
spec/frontend/jobs/store/getters_spec.js: localState.job.has_trace = false;
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/pipelines/linked_pipelines_mock.json: "title": "This job does not have a trace."
spec/frontend/vue_shared/security_reports/mock_data.js: '/gitlab-org/secrets-detection-test/-/jobs/1399/artifacts/download?file_type=trace',
spec/frontend/vue_shared/security_reports/mock_data.js: '/gitlab-org/secrets-detection-test/-/jobs/1400/artifacts/download?file_type=trace',
spec/frontend/vue_shared/security_reports/mock_data.js: '/gitlab-org/secrets-detection-test/-/jobs/1401/artifacts/download?file_type=trace',
spec/frontend/vue_shared/security_reports/mock_data.js: '/gitlab-org/secrets-detection-test/-/jobs/1402/artifacts/download?file_type=trace',
spec/frontend/vue_shared/security_reports/mock_data.js: '/gitlab-org/secrets-detection-test/-/jobs/1399/artifacts/download?file_type=trace',
spec/frontend/vue_shared/security_reports/mock_data.js: '/gitlab-org/secrets-detection-test/-/jobs/1400/artifacts/download?file_type=trace',
spec/frontend/vue_shared/security_reports/mock_data.js: '/gitlab-org/secrets-detection-test/-/jobs/1401/artifacts/download?file_type=trace',
spec/frontend/vue_shared/security_reports/mock_data.js: '/gitlab-org/secrets-detection-test/-/jobs/1402/artifacts/download?file_type=trace',
spec/frontend/vue_shared/security_reports/mock_data.js: * These correspond to any jobs with trace data in the securityReportMergeRequestDownloadPathsQueryResponse above.
spec/frontend/vue_shared/security_reports/mock_data.js: path: '/gitlab-org/secrets-detection-test/-/jobs/1399/artifacts/download?file_type=trace',
spec/frontend/vue_shared/security_reports/mock_data.js: path: '/gitlab-org/secrets-detection-test/-/jobs/1400/artifacts/download?file_type=trace',
spec/frontend/vue_shared/security_reports/mock_data.js: path: '/gitlab-org/secrets-detection-test/-/jobs/1401/artifacts/download?file_type=trace',
spec/frontend/vue_shared/security_reports/mock_data.js: path: '/gitlab-org/secrets-detection-test/-/jobs/1402/artifacts/download?file_type=trace',
CSS Class
Remove the legacy CSS class: build-trace as this is shared between the project jobs page and the web ide
Related to #235814 (closed).
Edited by Jackie Porter