fetch precompiled xk6 extension from grafana/xk6-sm
#966
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
roobre
force-pushed
the
xk6-sm-repo
branch
5 times, most recently
from
22c0782
to
b932ad2
Compare
roobre
force-pushed
the
xk6-sm-repo
branch
5 times, most recently
from
5b083ab
to
7154d3f
Compare
roobre
force-pushed
the
xk6-sm-repo
branch
7 times, most recently
from
af9c4e7
to
9ee2e6a
Compare
Otherwise, it is not currently possible for goreleaser to access files created by `sm-k6`, as goreleaser wipes `dist` before running.
roobre
changed the title
grafana/xk6-smgrafana/xk6-sm
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 PR removes the
smextension code from this repo and adjust Makefiles and Dockerfiles to pull k6 from https://github.com/grafana/xk6-sm.A bullet-point summary
$(DISTDIR)/$(OS)-$(ARCH)/sm-k6targets have been added tocurlk6 from grafana/xk6-sm releasessm-k6-nativeis a convenient alias for the host's OS/ARCH, whilesm-k6downloads sm-k6 for all known architecturestestorpackagetargets. They can be, but as the package target does not depend on build, I followed that pattern.make sm-k6is manually invoked in the pipeline.buildtarget needs to depend onsm-k6-native. If we decide to not keep dependencies, we can remove this when we ditch goreleaser (details on why this was needed are on a comment).Part of https://github.com/grafana/synthetic-monitoring/issues/160