Skip to content
Snippets Groups Projects

Fix detect-tests CI job

Merged Tianwen Chen requested to merge sh-fix-detect-tests-16-8 into 16-8-stable-ee
All threads resolved!

What does this MR do and why?

The detect-tests job is failing in https://gitlab.com/gitlab-renovate-forks/gitlab/-/jobs/6326977771 because ruby:3.1-slim doesn't come with gcc and make, which is needed to build native extensions. activesupport depends on i18n, which depends on racc. racc has a native extension.

This is needed because yesterday the i18n gem added racc as a runtime dependency in 1.14.3: https://github.com/ruby-i18n/i18n/pull/680

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

  • This MR is backporting a bug fix, documentation update, or spec fix, previously merged in the default branch.
  • The MR that fixed the bug on the default branch has been deployed to GitLab.com (not applicable for documentation or spec changes).
  • This MR has a severity label assigned (if applicable).
  • Set the milestone of the merge request to match the target backport branch version.
  • This MR has been approved by a maintainer (only one approval is required).
  • Ensure the e2e:package-and-test-ee job has either succeeded or been approved by a Software Engineer in Test.

Note to the merge request author and maintainer

If you have questions about the patch release process, please:

Edited by Stan Hu

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Tianwen Chen changed milestone to %16.10

    changed milestone to %16.10

  • Tianwen Chen added typebug label and removed typemaintenance label

    added typebug label and removed typemaintenance label

  • 2 Warnings
    :warning: Backporting to older releases requires an exception request process
    :warning: The e2e:package-and-test-ee job needs to succeed or have approval from a Software Engineer in Test.
    Read the "QA e2e:package-and-test-ee" section for more details.

    Pipeline Changes

    This merge request contains changes to the pipeline configuration for the GitLab project.

    Please consider the effect of the changes in this merge request on the following:

    Please consider communicating these changes to the broader team following the communication guideline for pipeline changes

    QA e2e:package-and-test-ee

    @tianwenchen, the package-and-test job must complete before merging this merge request.*

    If there are failures on the package-and-test pipeline, ping your team's associated Software Engineer in Test (SET) to confirm the failures are unrelated to the merge request. If there's no SET assigned, ask for assistance on the #test-platform Slack channel.

    If needed, you can retry the :repeat: danger-review job that generated this comment.

    Generated by :no_entry_sign: Danger

  • E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for 39de1292

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Create      | 54     | 0      | 8       | 0     | 62    | ✅     |
    | Verify      | 31     | 0      | 0       | 0     | 31    | ✅     |
    | Govern      | 67     | 0      | 0       | 0     | 67    | ✅     |
    | Plan        | 54     | 0      | 0       | 0     | 54    | ✅     |
    | Data Stores | 23     | 0      | 0       | 0     | 23    | ✅     |
    | Manage      | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Package     | 15     | 0      | 1       | 0     | 16    | ✅     |
    | Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Monitor     | 7      | 0      | 0       | 0     | 7     | ✅     |
    | Release     | 5      | 0      | 0       | 0     | 5     | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 258    | 0      | 10      | 0     | 268   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+

    e2e-package-and-test: :x: test report for 39de1292

    expand test summary
    +---------------------------------------------------------------------+
    |                           suites summary                            |
    +----------------+--------+--------+---------+-------+-------+--------+
    |                | passed | failed | skipped | flaky | total | result |
    +----------------+--------+--------+---------+-------+-------+--------+
    | GitLab Metrics | 2      | 0      | 1       | 0     | 3     | ✅     |
    | Govern         | 273    | 0      | 22      | 0     | 295   | ✅     |
    | Systems        | 8      | 0      | 0       | 0     | 8     | ✅     |
    | Package        | 208    | 18     | 17      | 9     | 243   | ❌     |
    | Plan           | 246    | 0      | 13      | 0     | 259   | ✅     |
    | Data Stores    | 119    | 0      | 13      | 1     | 132   | ✅     |
    | Manage         | 40     | 0      | 10      | 0     | 50    | ✅     |
    | Create         | 550    | 0      | 73      | 16    | 623   | ✅     |
    | Fulfillment    | 8      | 0      | 72      | 3     | 80    | ✅     |
    | Configure      | 1      | 0      | 9       | 0     | 10    | ✅     |
    | Analytics      | 7      | 0      | 0       | 0     | 7     | ✅     |
    | Verify         | 138    | 0      | 27      | 2     | 165   | ✅     |
    | Monitor        | 36     | 0      | 13      | 0     | 49    | ✅     |
    | Release        | 15     | 0      | 3       | 0     | 18    | ✅     |
    | Secure         | 6      | 0      | 3       | 0     | 9     | ✅     |
    | Growth         | 0      | 0      | 6       | 0     | 6     | ➖     |
    | ModelOps       | 0      | 0      | 12      | 0     | 12    | ➖     |
    +----------------+--------+--------+---------+-------+-------+--------+
    | Total          | 1657   | 18     | 294     | 31    | 1969  | ❌     |
    +----------------+--------+--------+---------+-------+-------+--------+
  • Stan Hu marked the checklist item This MR is backporting a bug fix, documentation update, or spec fix, previously merged in the default branch. as completed

    marked the checklist item This MR is backporting a bug fix, documentation update, or spec fix, previously merged in the default branch. as completed

  • Stan Hu marked the checklist item The MR that fixed the bug on the default branch has been deployed to GitLab.com (not applicable for documentation or spec changes). as completed

    marked the checklist item The MR that fixed the bug on the default branch has been deployed to GitLab.com (not applicable for documentation or spec changes). as completed

  • Stan Hu marked the checklist item This MR has a [severity label] assigned (if applicable). as completed

    marked the checklist item This MR has a [severity label] assigned (if applicable). as completed

  • Stan Hu marked the checklist item Set the milestone of the merge request to match the target backport branch version. as completed

    marked the checklist item Set the milestone of the merge request to match the target backport branch version. as completed

  • Stan Hu marked the checklist item This MR has been approved by a maintainer (only one approval is required). as completed

    marked the checklist item This MR has been approved by a maintainer (only one approval is required). as completed

  • Stan Hu marked the checklist item Ensure the e2e:package-and-test-ee job has either succeeded or been approved by a Software Engineer in Test. as completed

    marked the checklist item Ensure the e2e:package-and-test-ee job has either succeeded or been approved by a Software Engineer in Test. as completed

  • Stan Hu requested review from @stanhu

    requested review from @stanhu

  • Stan Hu marked this merge request as ready

    marked this merge request as ready

  • Stan Hu approved this merge request

    approved this merge request

    • Resolved by Stan Hu

      :wave: @stanhu, thanks for approving this merge request.

      This is the first time the merge request has been approved. Please ensure the e2e:package-and-test-ee job has succeeded. If there is a failure, a Software Engineer in Test (SET) needs to confirm the failures are unrelated to the merge request. If there's no SET assigned to this team, ask for assistance on the #test-platform Slack channel.

  • Stan Hu resolved all threads

    resolved all threads

  • Stan Hu enabled an automatic merge when the pipeline for 9cec7b51 succeeds

    enabled an automatic merge when the pipeline for 9cec7b51 succeeds

  • merged

  • Stan Hu mentioned in commit 1aff4fdc

    mentioned in commit 1aff4fdc

Please register or sign in to reply
Loading