Repository navigation
Prepare 0.16.0: fix the flaky basic e2e trigger, the Go version in actions/setup, and stale docs - #242
Merged
Conversation
…re 0.16.0 actions/setup installed Go 1.24, while go.mod requires go 1.26.0 since #227. setup-go exports GOTOOLCHAIN=local, so the build either fails or, where sudo drops that variable, downloads the go1.26.0 toolchain instead of using the latest 1.26 patch. It now installs "1.26", like CI. README: - Build from source: needs Go 1.26+; `make build` only builds, a plain `make` also runs license/lint/test (license-eye, golangci-lint); the binaries are swctl-<version>-<os>-<arch> for the seven platforms, not swctl-latest-<os>-amd64. - The GitHub Actions example used `apache/skywalking-cli@main`; the default branch is `master`, there is no `main`. CONTRIBUTING: - Compiling: Go 1.26+, seven binaries, not three. - How to release: the tag has no `v` prefix and is pushed only after `make release` succeeds; three links pointed at How-to-release.md anchors that do not exist; mention pre-release.sh and release.sh. How-to-release.md: fix the unclosed BuildKit link.
infra-e2e #132 (pulled in by the cf589b4 -> 56ac8be bump in #240) makes the http trigger stop after `times` requests. Before it, `times` only decided when the trigger reported its first result, and the requests kept going until the verify phase finished. The basic case's `times: 10` now means 30 seconds of traffic, and the nightly CI failed on 3 of the 4 nights since #240, every time on `metrics linear service_relation_client_cpm`, whose whole 30-minute window came back empty while the other 15 cases passed. `times: -1` (endless, infra-e2e #134) restores the continuous traffic, the same as the live-debugging case.
wankai123
approved these changes
Nov 7, 2026
This file contains hidden or 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
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.
Release prep for 0.16.0: the nightly CI flake, the Go version in
actions/setup, and stale build/release docs.actions/setupinstalls Go 1.24go.modhas requiredgo 1.26.0since #227, butactions/setup/action.yamlstill installed Go 1.24.actions/setup-goexportsGOTOOLCHAIN=local(visible in this repo's CI logs). Somake installin the action either fails on the go.mod version check or, wheresudodrops that variable, downloads thego1.26.0toolchain instead of the latest 1.26 patch. CI never runs this action, which is why nothing caught it. It now installs"1.26", like CI. Added to CHANGES.md under 0.16.0.Docs
make buildonly builds, while a plainmakealso runs license/lint/test and needslicense-eyeandgolangci-lint. The binaries areswctl-<version>-<os>-<arch>for seven platforms (checked withmake -n build), notswctl-latest-<os>-amd64.apache/skywalking-cli@main, but there is nomainbranch, onlymaster.git tag v${VERSION}and push before building. The tags have novprefix and are pushed only aftermake releasesucceeds. Three links pointed at How-to-release.md anchors that don't exist (#add-your-gpg-public-key,#make-the-internal-announcements,#call-a-vote-in-dev). The steps now also mentionscripts/pre-release.shandscripts/release.sh.Go version
No bump to Go 1.27 for this release. Go 1.26 is still supported, the other Go projects (infra-e2e, skywalking-go, skywalking-mcp, BanyanDB, Satellite) are on 1.26, and
govulncheckwithgo1.26.10reports no reachable vulnerability. The release binaries are built withGOTOOLCHAIN=go1.26.10, the same patch CI'sgo-version: "1.26"resolves to.Nightly CI: basic e2e trigger (was #241)
Nightly CI on master has failed on 3 of the 4 nights since #240 (1, 2, 3), after 26 green nights before it. Every failure is the same case in
Command Tests:The case's whole 30-minute window comes back empty. The other 15 cases pass.
Cause
#240 bumped
apache/skywalking-infra-e2efromcf589b4to56ac8be. That range includes infra-e2e #132, which makes the http trigger stop aftertimesrequests. Before it,timesonly decided when the trigger reported its first result, and the requests kept going until the verify phase finished. The basic case'stimes: 10withinterval: 3stherefore now means only 30 seconds of traffic (trigger has completed 10 requests and will stop.in the logs). That is not always long enough for the OAP to attribute the consumer's client-side calls to theproviderservice.Fix
times: -1(endless trigger, infra-e2e #134, already in the pinned SHA) restores the continuous traffic. Thelive-debuggingcase already uses it. Test config only, no change to swctl.