Skip to content
Snippets Groups Projects

[BE] Provide script to run on console for bulk assignment

Merged Mohamed Hamda requested to merge 431619-script-draft into master
All threads resolved!

Purpose of this MR

  • The current MR presents the Code Suggestions Bulk User Assignment rake task.
  • The script accepts a CSV file path as a parameter and handles user assignments.
  • This task is specifically designated for manual use and not automated or running within the GitLab codebase.

Local Setup and Validation Instructions

For details on setting up and validating the script locally, kindly document the file in the MR.

Related to #431619

Edited by Mohamed Hamda

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
  • Mohamed Hamda marked this merge request as draft

    marked this merge request as draft

  • Contributor
    7 Warnings
    :warning: 7b19bc10: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
    :warning: 0103a6ef: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
    :warning: ed755a24: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines.
    :warning: ed755a24: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
    :warning: cdf6bf56: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines.
    :warning: 4c36e394: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
    :warning: f70090f5: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
    1 Message
    :book: This merge request adds or changes documentation files. A review from the Technical Writing team before you merge is recommended. Reviews can happen after you merge.

    Documentation review

    The following files require a review from a technical writer:

    The review does not need to block merging this merge request. See the:

    Reviewer roulette

    Changes that require review have been detected!

    Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:

    Category Reviewer Maintainer
    backend @joseph profile link current availability (UTC+0, 1 hour behind author) @engwan profile link current availability (UTC+8, 7 hours ahead of author)

    Please check reviewer's status!

    • available Reviewer is available!
    • unavailable Reviewer is unavailable!

    Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

    To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

    Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.

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

    Generated by :no_entry_sign: Danger

  • Mohamed Hamda requested review from @bhrai

    requested review from @bhrai

  • Mohamed Hamda added 2 commits

    added 2 commits

    • 637d2411 - Move to use a rake task instead of a script
    • e40448e4 - Add docs for usage

    Compare with previous version

  • A deleted user added documentation label

    added documentation label

  • Contributor

    E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for d676f8ed

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Create      | 8      | 0      | 3       | 0     | 11    | ✅     |
    | Govern      | 3      | 0      | 0       | 0     | 3     | ✅     |
    | Monitor     | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Plan        | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Package     | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Data Stores | 2      | 0      | 0       | 0     | 2     | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 21     | 0      | 4       | 0     | 25    | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
  • Mohamed Hamda marked this merge request as ready

    marked this merge request as ready

  • Mohamed Hamda requested review from @bhrai

    requested review from @bhrai

  • Mohamed Hamda changed the description

    changed the description

  • Bishwa Hang Rai requested review from @fneill

    requested review from @fneill

  • Fiona Neill
  • Fiona Neill
  • Fiona Neill
  • Fiona Neill
  • Fiona Neill
  • Fiona Neill
  • Mohamed Hamda added 3 commits

    added 3 commits

    Compare with previous version

  • Mohamed Hamda requested review from @fneill

    requested review from @fneill

  • Mohamed Hamda requested review from @bhrai

    requested review from @bhrai

  • Mohamed Hamda added 1 commit

    added 1 commit

    Compare with previous version

  • Mohamed Hamda added 1 commit

    added 1 commit

    Compare with previous version

  • Mohamed Hamda added 1 commit

    added 1 commit

    Compare with previous version

  • Bishwa Hang Rai
  • Mohamed Hamda added 2 commits

    added 2 commits

    Compare with previous version

  • Mohamed Hamda requested review from @bhrai

    requested review from @bhrai

  • Bishwa Hang Rai approved this merge request

    approved this merge request

  • Bishwa Hang Rai requested review from @vij

    requested review from @vij

  • Contributor

    @mhamda Some end-to-end (E2E) tests should run based on the stage label.

    Please start the trigger-omnibus-and-follow-up-e2e job in the qa stage and wait for the tests in the follow-up-e2e:package-and-test-ee pipeline to pass before merging this MR. Do not use Auto-merge, unless these tests have already completed successfully, because a failure in these tests do not block the auto-merge. (E2E tests are computationally intensive and don't run automatically for every push/rebase, so we ask you to run this job manually at least once.)

    To run all E2E tests, apply the pipeline:run-all-e2e label and run a new pipeline.

    E2E test jobs are allowed to fail due to flakiness. See current failures at the latest pipeline triage issue.

    Once done, apply the :white_check_mark: emoji on this comment.

    Team members only: for any questions or help, reach out on the internal #test-platform Slack channel.

  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar removed review request for @vij

    removed review request for @vij

  • Mohamed Hamda added 1 commit

    added 1 commit

    • 0103a6ef - Address maintainer review points

    Compare with previous version

  • Contributor

    Quarantined Fulfillment QA E2E Tests

    The following Fulfillment QA E2E tests are in quarantine. If your changes would be covered by these tests, please be sure to perform manual testing.

    Description Location
    Fulfillment with existing compute minutes packs behaves like Purchase storage adds additional storage to group namespace ./qa/specs/features/ee/browser_ui/11_fulfillment/purchase/purchase_storage_spec.rb[1:3:1:1]
    Fulfillment with an active subscription behaves like Purchase storage adds additional storage to group namespace ./qa/specs/features/ee/browser_ui/11_fulfillment/purchase/purchase_storage_spec.rb[1:2:1:1]
    Fulfillment without active subscription behaves like Purchase storage adds additional storage to group namespace ./qa/specs/features/ee/browser_ui/11_fulfillment/purchase/purchase_storage_spec.rb[1:1:1:1]
  • Mohamed Hamda requested review from @vij

    requested review from @vij

  • Fiona Neill approved this merge request

    approved this merge request

  • Hi @fneill :wave:,

    GitLab Bot has added the Technical Writing label because a Technical Writer has approved or merged this MR.

    This message was generated automatically. You're welcome to improve it.

  • Fiona Neill removed review request for @fneill

    removed review request for @fneill

  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar
  • Vijay Hawoldar removed review request for @vij

    removed review request for @vij

  • Mohamed Hamda added 1 commit

    added 1 commit

    • 7b19bc10 - Remove SaaS check and fix reviews

    Compare with previous version

  • Contributor

    Quarantined Fulfillment QA E2E Tests

    The following Fulfillment QA E2E tests are in quarantine. If your changes would be covered by these tests, please be sure to perform manual testing.

    Description Location
    Fulfillment without active subscription behaves like Purchase storage adds additional storage to group namespace ./qa/specs/features/ee/browser_ui/11_fulfillment/purchase/purchase_storage_spec.rb[1:1:1:1]
    Fulfillment with existing compute minutes packs behaves like Purchase storage adds additional storage to group namespace ./qa/specs/features/ee/browser_ui/11_fulfillment/purchase/purchase_storage_spec.rb[1:3:1:1]
    Fulfillment with an active subscription behaves like Purchase storage adds additional storage to group namespace ./qa/specs/features/ee/browser_ui/11_fulfillment/purchase/purchase_storage_spec.rb[1:2:1:1]
  • Mohamed Hamda requested review from @vij

    requested review from @vij

  • Bishwa Hang Rai removed review request for @bhrai

    removed review request for @bhrai

  • Vijay Hawoldar
  • Vijay Hawoldar removed review request for @vij

    removed review request for @vij

  • Mohamed Hamda mentioned in commit d676f8ed

    mentioned in commit d676f8ed

  • Mohamed Hamda added 1 commit

    added 1 commit

    Compare with previous version

  • Mohamed Hamda requested review from @vij

    requested review from @vij

  • Contributor

    Quarantined Fulfillment QA E2E Tests

    The following Fulfillment QA E2E tests are in quarantine. If your changes would be covered by these tests, please be sure to perform manual testing.

    Description Location
    Fulfillment with an active subscription behaves like Purchase storage adds additional storage to group namespace ./qa/specs/features/ee/browser_ui/11_fulfillment/purchase/purchase_storage_spec.rb[1:2:1:1]
    Fulfillment without active subscription behaves like Purchase storage adds additional storage to group namespace ./qa/specs/features/ee/browser_ui/11_fulfillment/purchase/purchase_storage_spec.rb[1:1:1:1]
    Fulfillment with existing compute minutes packs behaves like Purchase storage adds additional storage to group namespace ./qa/specs/features/ee/browser_ui/11_fulfillment/purchase/purchase_storage_spec.rb[1:3:1:1]
  • Vijay Hawoldar approved this merge request

    approved this merge request

  • Vijay Hawoldar resolved all threads

    resolved all threads

  • Vijay Hawoldar enabled an automatic merge when all merge checks for d676f8ed pass

    enabled an automatic merge when all merge checks for d676f8ed pass

  • Vijay Hawoldar canceled the automatic merge

    canceled the automatic merge

  • Vijay Hawoldar enabled an automatic merge when all merge checks for d676f8ed pass

    enabled an automatic merge when all merge checks for d676f8ed pass

  • merged

  • Vijay Hawoldar mentioned in commit eca41ca2

    mentioned in commit eca41ca2

  • added workflowstaging label and removed workflowcanary label

  • Ulises Fierro mentioned in issue #501110

    mentioned in issue #501110

  • Please register or sign in to reply
    Loading