git-mirrors promoted from experiment, enabled by git-mirrors-path #1272
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This draft pull request promotes the
git-mirrors
functionality from experiment to standard.It still requires
git-mirrors-path
to be configured, otherwise it remains disabled. This makes it pretty safe to promote from experiment status, with a few caveats below.elastic-ci-stack-for-aws consideration
Note that elastic-ci-stack-for-aws prior to buildkite/elastic-ci-stack-for-aws#698 does provide a
git-mirrors-path
even when the stack chooses not to enable the experiment. That means shipping this PR could result in surprising/disruptive behaviour if it's used in an elastic-stack prior to that PR merged.test coverage
This PR regrettably removes test coverage for
git-mirrors
, because that coverage was being driven externally viaTEST_EXPERIMENTS
:agent/bootstrap/integration/main_test.go
Line 38 in f119ac1
… as well as special handling in BootstrapTester:
agent/bootstrap/integration/bootstrap_tester.go
Lines 111 to 119 in f119ac1
… and a special CI test entrypoint:
agent/.buildkite/steps/tests.sh
Lines 9 to 10 in f119ac1
I think this PR should add some test coverage back in, but via a dedicated test which sets up a
git-mirror-path
at at least traverses the happy path of the code.