G
YouTube TikTok Games
Skip to content

Support new commands in v9, replace old command GraphQL method - #134

Merged
wu-sheng merged 1 commit into
apache:masterfrom
mrproliu:v9-adapt
Jan 14, 2022
Merged

wu-sheng merged 1 commit into
apache:masterfrom
mrproliu:v9-adapt

Conversation

@mrproliu

Copy link
Copy Markdown
Contributor
  • Change the GraphQL method to the v9 version according to the OAP server version.
  • Add new sub-command get to instance and endpoint.
  • Add E2E test cases to 9.0.0 version check.

@mrproliu mrproliu added the enhancement New feature or request label Jan 14, 2022
@mrproliu mrproliu added this to the 0.10.0 milestone Jan 14, 2022
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #134 (ae9345c) into master (b3e4023) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #134   +/-   ##
=======================================
  Coverage   27.47%   27.47%           
=======================================
  Files          20       20           
  Lines         404      404           
=======================================
  Hits          111      111           
  Misses        286      286           
  Partials        7        7           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b3e4023...ae9345c. Read the comment docs.

@wu-sheng
wu-sheng merged commit 3278a47 into apache:master Jan 14, 2022
wu-sheng added a commit that referenced this pull request Nov 7, 2026
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.
wu-sheng added a commit that referenced this pull request Nov 7, 2026
…tions/setup, and stale docs (#242)

* Fix the Go version in actions/setup and stale build/release docs before 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.

* Keep the basic e2e trigger running until verification ends

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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants