Remove references to '/ci' path while describing Runner's registration
What does this MR do?
Removes references to /ci
path while describing Runners registration process.
Are there points in the code the reviewer needs to double check?
Why was this MR needed?
In the past we were using URL with /ci
to register Runners, e.g. https://gitlab.com/ci
. Since few months until 9.0 release this was redundant (because Runner was automatically adding /ci
) but literally valid.
9.0 changed the situation, because Runner is no longer using /ci
prefix to access API. In Runner we've added a mechanism to remove existing /ci
, to make Runner work with existing configurations (this should be deprecated and removed in the future), but from our documentation and steps described on CI/CD settings page we should not suggest old URLs anymore. Especially, when Runner suggests new ones and this is confusing users: gitlab-org/gitlab-ci-multi-runner!547.
Screenshots (if relevant)
Does this MR meet the acceptance criteria?
-
Changelog entry added, if necessary -
Documentation created/updated -
API support added - Tests
-
Added for this feature/bug -
All builds are passing
-
-
Conform by the merge request performance guides -
Conform by the style guides -
Branch has no merge conflicts with master
(if it does - rebase it please) -
Squashed related commits together
What are the relevant issue numbers?
Merge request reports
Activity
@tmaczukin Can you rebase?
assigned to @tmaczukin
@ayufan Done
assigned to @ayufan
added 369 commits
-
ee40a507...ce6a48cb - 368 commits from branch
master
- 8ea244d1 - Remove references to '/ci' path while describing Runner's registration
-
ee40a507...ce6a48cb - 368 commits from branch
@tmaczukin Can you rebase?
assigned to @tmaczukin
changed milestone to %9.3
added Deliverable ~15078 labels
added 2305 commits
-
8ea244d1...e261b4b8 - 2304 commits from branch
master
- cd063101 - Remove references to '/ci' path while describing Runner's registration
-
8ea244d1...e261b4b8 - 2304 commits from branch
@ayufan Done
assigned to @ayufan
enabled an automatic merge when the pipeline for cd063101 succeeds
mentioned in commit 06638397
mentioned in merge request !11603 (merged)