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

[CI] Remove unwanted console.log from pipeline.ts #204724

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

delanni
Copy link
Contributor

@delanni delanni commented Dec 18, 2024

Summary

Logging to stdout from this file would result uploading the logged string to buildkite as a pipeline definition, causing errors (https://buildkite.com/elastic/kibana-pull-request/builds/261721#0193d94b-f05c-41d6-9865-3d3c331a6cc4).

Also, adds an inline eslint rule to warn about console.log/stdout usage, as this has happened before by oversight.

Overlaps with: #204672

@delanni delanni added Team:Operations Team label for Operations Team release_note:skip Skip the PR/issue when compiling release notes backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development labels Dec 18, 2024
@delanni delanni requested a review from a team as a code owner December 18, 2024 11:27
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@delanni delanni force-pushed the fix-ci-pipeline-renovate branch from 1491e98 to 1cb56fd Compare December 18, 2024 11:28
@delanni delanni force-pushed the fix-ci-pipeline-renovate branch from 1cb56fd to 961499a Compare December 18, 2024 11:31
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #63 / serverless search UI Serverless Inference Management UI delete action "before each" hook for "deletes modal successfully with usage"

Metrics [docs]

✅ unchanged

History

*/

export function emitPipeline(pipelineStr: string) {
console.log(pipelineStr);
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we convert to Set to remove duplicates here as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants