Skip to content
Snippets Groups Projects

Add GET /projects/:id/remote_mirrors/:mirror_id/public_key API endpoint

Merged Jio Castillo requested to merge add-get-remote-mirror-public-key-endpoint into master
All threads resolved!

What does this MR do and why?

Adds the "GET /projects/:id/remote_mirrors/:mirror_id/public_key" API endpoint that extends the existing UI ability to obtain a remote mirrors public key to the Remote mirrors API.

Changelog: changed

References

Please include cross links to any resources that are relevant to this MR. This will give reviewers and future readers helpful context to give an efficient review of the changes introduced.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

  1. Create mirror with SSH authentication
  2. List a project’s remote mirrors to obtain the created remote mirror's ID
  3. Make a request to the new endpoint. Example:
curl --header "PRIVATE-TOKEN: <your_access_token>" "https://gitlab.example.com/api/v4/projects/42/remote_mirrors/101486/public_key"

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
  • Brendan Lynch
  • Jio Castillo added 1 commit

    added 1 commit

    • 338a3692 - Apply 2 suggestion(s) to 1 file(s)

    Compare with previous version

  • Jio Castillo requested review from @brendan777

    requested review from @brendan777

  • Brendan Lynch approved this merge request

    approved this merge request

  • added pipelinetier-2 label and removed pipelinetier-1 label

  • Before you set this MR to auto-merge

    This merge request will progress on pipeline tiers until it reaches the last tier: pipelinetier-3. We will trigger a new pipeline for each transition to a higher tier.

    Before you set this MR to auto-merge, please check the following:

    • You are the last maintainer of this merge request
    • The latest pipeline for this merge request is pipelinetier-3 (You can find which tier it is in the pipeline name)
    • This pipeline is recent enough (created in the last 8 hours)

    If all the criteria above apply, please set auto-merge for this merge request.

    See pipeline tiers and merging a merge request for more details.

  • Hi @brendan777 :wave:,

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

    This message was generated automatically. Improve it or delete it.

  • E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for 83f85d0d

    expand test summary
    +--------------------------------------------------------------+
    |                        suites summary                        |
    +---------+--------+--------+---------+-------+-------+--------+
    |         | passed | failed | skipped | flaky | total | result |
    +---------+--------+--------+---------+-------+-------+--------+
    | Create  | 3      | 0      | 0       | 0     | 3     | ✅     |
    | Govern  | 5      | 0      | 0       | 0     | 5     | ✅     |
    | Plan    | 1      | 0      | 0       | 0     | 1     | ✅     |
    | Monitor | 2      | 0      | 0       | 0     | 2     | ✅     |
    +---------+--------+--------+---------+-------+-------+--------+
    | Total   | 11     | 0      | 0       | 0     | 11    | ✅     |
    +---------+--------+--------+---------+-------+-------+--------+

    e2e-test-on-cng: :white_check_mark: test report for 83f85d0d

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Plan        | 86     | 0      | 8       | 0     | 94    | ✅     |
    | Govern      | 83     | 0      | 11      | 0     | 94    | ✅     |
    | Create      | 140    | 0      | 22      | 0     | 162   | ✅     |
    | Fulfillment | 2      | 0      | 7       | 0     | 9     | ✅     |
    | Verify      | 52     | 0      | 20      | 0     | 72    | ✅     |
    | Configure   | 0      | 0      | 3       | 0     | 3     | ➖     |
    | Data Stores | 33     | 0      | 10      | 0     | 43    | ✅     |
    | Secure      | 2      | 0      | 5       | 0     | 7     | ✅     |
    | ModelOps    | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Package     | 29     | 0      | 15      | 0     | 44    | ✅     |
    | Monitor     | 8      | 0      | 12      | 0     | 20    | ✅     |
    | Manage      | 1      | 0      | 9       | 0     | 10    | ✅     |
    | Release     | 5      | 0      | 1       | 0     | 6     | ✅     |
    | Ai-powered  | 0      | 0      | 2       | 0     | 2     | ➖     |
    | Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Growth      | 0      | 0      | 2       | 0     | 2     | ➖     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 443    | 0      | 128     | 0     | 571   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    Edited by ****
  • Jeff Park requested review from @garyh

    requested review from @garyh

  • Jeff Park approved this merge request

    approved this merge request

  • Jio Castillo added 321 commits

    added 321 commits

    • 338a3692...b817c1f4 - 318 commits from branch master
    • ff3c5bca - Add GET /projects/:id/remote_mirrors/:mirror_id/public_key API endpoint
    • 8a0d2c64 - Add documentation section
    • d2bae3c2 - Apply 2 suggestion(s) to 1 file(s)

    Compare with previous version

  • Jio Castillo added 1 commit

    added 1 commit

    • 83f85d0d - Apply 2 suggestion(s) to 1 file(s)

    Compare with previous version

  • Jio Castillo reset approvals from @brendan777 by pushing to the branch

    reset approvals from @brendan777 by pushing to the branch

  • Brendan Lynch approved this merge request

    approved this merge request

  • Gary Holtz approved this merge request

    approved this merge request

  • Gary Holtz resolved all threads

    resolved all threads

  • Gary Holtz enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • merged

  • Gary Holtz mentioned in commit 6cd33097

    mentioned in commit 6cd33097

  • Jio Castillo mentioned in issue #514005

    mentioned in issue #514005

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading