Skip to content

[jenkins] disable CI metrics for temporary feature branches#81938

Merged
spalger merged 1 commit intoelastic:masterfrom
spalger:implement/skip-ci-metrics-for-temp-feature-branches
Oct 28, 2020
Merged

[jenkins] disable CI metrics for temporary feature branches#81938
spalger merged 1 commit intoelastic:masterfrom
spalger:implement/skip-ci-metrics-for-temp-feature-branches

Conversation

@spalger
Copy link
Copy Markdown
Contributor

@spalger spalger commented Oct 28, 2020

With PRs like #81766, the CI metrics are marking the CI report as failed because the target branch is not getting baselines captured for it. We want to make it possible for these PRs to have metrics, but not required, and to do that the plan is to determine if the target branch for the PR matches the "branch" field in the root package.json file. When these don't match then CI metrics will become optional and CI report comments will not report as failures when the baseline metrics can't be determined.

@spalger spalger force-pushed the implement/skip-ci-metrics-for-temp-feature-branches branch 2 times, most recently from e41f747 to 968d538 Compare October 28, 2020 19:27
@spalger spalger force-pushed the implement/skip-ci-metrics-for-temp-feature-branches branch from 968d538 to 5029a7a Compare October 28, 2020 19:27
@spalger
Copy link
Copy Markdown
Contributor Author

spalger commented Oct 28, 2020

Tested to make sure that targetsTrackedBranch gets set to false correctly in #81946

https://kibana-ci.elastic.co/job/elastic+kibana+pipeline-pull-request/84408/execution/node/45/log/

19:32:01  repoInfo: [branch:test-ci-metrics-check, targetBranch:feature/runtime-field-editor, targetsTrackedBranch:false, commit:ec89d36fc5fc2fd44aa08293ccc1286c033b5043, mergeBase:af0f496b60c0e20a667c827179d0753fd80ec1b5]

@spalger spalger marked this pull request as ready for review October 28, 2020 19:34
@spalger spalger requested a review from a team as a code owner October 28, 2020 19:34
@spalger spalger added release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v7.10.0 v7.11.0 v8.0.0 labels Oct 28, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

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

@spalger spalger requested a review from brianseeders October 28, 2020 19:34
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@spalger spalger merged commit 271a799 into elastic:master Oct 28, 2020
@spalger spalger deleted the implement/skip-ci-metrics-for-temp-feature-branches branch October 28, 2020 21:08
spalger added a commit to spalger/kibana that referenced this pull request Oct 28, 2020
…81938)

Co-authored-by: spalger <spalger@users.noreply.github.com>
spalger added a commit to spalger/kibana that referenced this pull request Oct 28, 2020
…81938)

Co-authored-by: spalger <spalger@users.noreply.github.com>
spalger added a commit that referenced this pull request Oct 28, 2020
…81958)

Co-authored-by: spalger <spalger@users.noreply.github.com>

Co-authored-by: spalger <spalger@users.noreply.github.com>
spalger added a commit that referenced this pull request Oct 28, 2020
…81959)

Co-authored-by: spalger <spalger@users.noreply.github.com>

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v7.10.0 v7.11.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants