Skip to content
Snippets Groups Projects

Adding Geo support of Project-level Secure Files

Merged Darby Frey requested to merge secure-files-replication into master
All threads resolved!

What does this MR do and why?

Following the process described in #365595 (closed), this MR adds Geo-replication support for Project-level Secure Files.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Project-level Secure Files can be added to a project by a maintainer by going to the Settings > CI/CD. Scroll to the bottom of the page and expand the Secure Files section. Click the Upload File button, and upload any file smaller than 5 MB.

Docs: https://docs.gitlab.com/ee/ci/secure_files/#add-a-secure-file-to-a-project

MR acceptance checklist

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

Edited by Darby Frey

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
  • removed review request for @dbalexandre

  • Darby Frey added 1 commit

    added 1 commit

    • 0a8aa8ca - Adding code and tests for replicables_for_current_secondary

    Compare with previous version

  • Darby Frey added 1 commit

    added 1 commit

    • 07d377a7 - Adding code and tests for replicables_for_current_secondary

    Compare with previous version

  • Darby Frey added 1 commit

    added 1 commit

    • ce236896 - Renaming SecureFileState back to CiSecureFileState

    Compare with previous version

  • Darby Frey added 1 commit

    added 1 commit

    Compare with previous version

  • Darby Frey requested review from @dbalexandre

    requested review from @dbalexandre

  • Darby Frey changed the description

    changed the description

  • changed milestone to %15.3

  • removed review request for @dbalexandre

  • Darby Frey added 1 commit

    added 1 commit

    Compare with previous version

  • Darby Frey added 2 commits

    added 2 commits

    Compare with previous version

  • Darby Frey requested review from @dbalexandre

    requested review from @dbalexandre

  • removed review request for @dbalexandre

  • Darby Frey added 1 commit

    added 1 commit

    Compare with previous version

  • Darby Frey requested review from @dbalexandre

    requested review from @dbalexandre

  • Darby Frey mentioned in commit 4d5f51b1

    mentioned in commit 4d5f51b1

  • Darby Frey added 1638 commits

    added 1638 commits

    Compare with previous version

  • resolved all threads

  • Douglas Barbosa Alexandre approved this merge request

    approved this merge request

  • Douglas Barbosa Alexandre enabled an automatic merge when the pipeline for 792bca34 succeeds

    enabled an automatic merge when the pipeline for 792bca34 succeeds

  • :wave: @dbalexandre, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.

    For more info, please refer to the following links:

  • resolved all threads

  • Darby Frey mentioned in commit 3bd7723b

    mentioned in commit 3bd7723b

  • Darby Frey aborted the automatic merge because source branch was updated

    aborted the automatic merge because source branch was updated

  • Darby Frey added 75 commits

    added 75 commits

    Compare with previous version

  • resolved all threads

  • Douglas Barbosa Alexandre enabled an automatic merge when the pipeline for 5d3ac9d4 succeeds

    enabled an automatic merge when the pipeline for 5d3ac9d4 succeeds

  • mentioned in commit 39b8a2a5

  • mentioned in issue #368805 (closed)

  • added workflowstaging label and removed workflowcanary label

  • mentioned in issue #362561 (closed)

  • mentioned in issue #362564 (closed)

  • Darby Frey mentioned in merge request !172163 (closed)

    mentioned in merge request !172163 (closed)

  • Please register or sign in to reply
    Loading