Skip to content

set all the tags to 6.9 for test#199

Merged
DaedalusG merged 3 commits into
mainfrom
wg/rel/fix-readme-expected-test
Nov 13, 2025
Merged

set all the tags to 6.9 for test#199
DaedalusG merged 3 commits into
mainfrom
wg/rel/fix-readme-expected-test

Conversation

@DaedalusG
Copy link
Copy Markdown
Contributor

This script has been failing during releases resulting in the following error:


$ wget https://github.com/comby-tools/comby/releases/download/1.8.1/comby-1.8.1-x86_64-linux
--
  | chmod +x ./comby-1.8.1-x86_64-linux
  | mv comby-1.8.1-x86_64-linux /usr/local/bin/comby
  |  
  | sg release run test --workdir=. --config-from-commit
  |  
  | --2025-11-12 22:40:12--  https://github.com/comby-tools/comby/releases/download/1.8.1/comby-1.8.1-x86_64-linux
  | Resolving github.com (github.com)... 140.82.113.4
  | Connecting to github.com (github.com)\|140.82.113.4\|:443... connected.
  | HTTP request sent, awaiting response... 302 Found
  | Location: https://release-assets.githubusercontent.com/github-production-release-asset/180236648/9ad78da2-9bdc-4271-8865-d243d7fd0349?sp=r&sv=2018-11-09&sr=b&spr=https&se=2025-11-12T23%3A28%3A09Z&rscd=attachment%3B+filename%3Dcomby-1.8.1-x86_64-linux&rsct=application%2Foctet-stream&skoid=96c2d410-5711-43a1-aedd-ab1947aa7ab0&sktid=398a6654-997b-47e9-b12b-9515b896b4de&skt=2025-11-12T22%3A27%3A13Z&ske=2025-11-12T23%3A28%3A09Z&sks=b&skv=2018-11-09&sig=Iw0J8WV1JTDx498vZ3sqVR2AWnfHFo6%2Bbuhpn9Cc1zA%3D&jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmVsZWFzZS1hc3NldHMuZ2l0aHVidXNlcmNvbnRlbnQuY29tIiwia2V5Ijoia2V5MSIsImV4cCI6MTc2Mjk4OTAxMiwibmJmIjoxNzYyOTg3MjEyLCJwYXRoIjoicmVsZWFzZWFzc2V0cHJvZHVjdGlvbi5ibG9iLmNvcmUud2luZG93cy5uZXQifQ.GNIRljNnlCYPu2Xx8TcTNtvZnNyLkPETGz7HtaENUaU&response-content-disposition=attachment%3B%20filename%3Dcomby-1.8.1-x86_64-linux&response-content-type=application%2Foctet-stream [following]
  | --2025-11-12 22:40:12--  https://release-assets.githubusercontent.com/github-production-release-asset/180236648/9ad78da2-9bdc-4271-8865-d243d7fd0349?sp=r&sv=2018-11-09&sr=b&spr=https&se=2025-11-12T23%3A28%3A09Z&rscd=attachment%3B+filename%3Dcomby-1.8.1-x86_64-linux&rsct=application%2Foctet-stream&skoid=96c2d410-5711-43a1-aedd-ab1947aa7ab0&sktid=398a6654-997b-47e9-b12b-9515b896b4de&skt=2025-11-12T22%3A27%3A13Z&ske=2025-11-12T23%3A28%3A09Z&sks=b&skv=2018-11-09&sig=Iw0J8WV1JTDx498vZ3sqVR2AWnfHFo6%2Bbuhpn9Cc1zA%3D&jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmVsZWFzZS1hc3NldHMuZ2l0aHVidXNlcmNvbnRlbnQuY29tIiwia2V5Ijoia2V5MSIsImV4cCI6MTc2Mjk4OTAxMiwibmJmIjoxNzYyOTg3MjEyLCJwYXRoIjoicmVsZWFzZWFzc2V0cHJvZHVjdGlvbi5ibG9iLmNvcmUud2luZG93cy5uZXQifQ.GNIRljNnlCYPu2Xx8TcTNtvZnNyLkPETGz7HtaENUaU&response-content-disposition=attachment%3B%20filename%3Dcomby-1.8.1-x86_64-linux&response-content-type=application%2Foctet-stream
  | Resolving release-assets.githubusercontent.com (release-assets.githubusercontent.com)... 185.199.108.133, 185.199.109.133, 185.199.110.133, ...
  | Connecting to release-assets.githubusercontent.com (release-assets.githubusercontent.com)\|185.199.108.133\|:443... connected.
  | HTTP request sent, awaiting response... 200 OK
  | Length: 38581720 (37M) [application/octet-stream]
  | Saving to: 'comby-1.8.1-x86_64-linux'
  |  
  | comby-1.8.1-x86_64-linux                100%[===============================================================================>]  36.79M   149MB/s    in 0.2s
  |  
  | 2025-11-12 22:40:12 (149 MB/s) - 'comby-1.8.1-x86_64-linux' saved [38581720/38581720]
  |  
  | 👉 [     setup] Finding release manifest in "."
  | [     setup] No explicit branch name was provided, assuming current branch is the target: 6.10.0
  | [     setup] Found manifest for "terraform-aws-executors" (github.com/sourcegraph/terraform-aws-executors)
  | [      meta] Owners: sourcegraph/release
  | [      meta] Repository: github.com/sourcegraph/terraform-aws-executors
  | 👉 [      vars] Variables
  | [      vars] inputs.server.tag="6.10.0"
  | [      vars] version="v6.10.0"
  | [      vars] tag="6.10.0"
  | [      vars] git.branch="6.10.0"
  | [      vars] is_development="false"
  | [      vars] do_init="false"
  | [      vars] do_init_only="false"
  | [      vars] pipline_flags=""
  | [      vars] inputs.server.version="6.10.0"
  | [      vars] config="{\"version\":\"v6.10.0\",\"inputs\":\"server=6.10.0\",\"type\":\"minor\"}"
  | 👉 [      reqs] Checking requirements...
  | [      reqs] ✅ comby
  | [      reqs] ✅ GitHub CLI
  | 👉 [      test] Running testing steps for v6.10.0
  | 👉 [      step] Running step "changes:README"
  | [changes:README] expected 16 new version tags of "6.10.0" in README.md, got 0
  | [changes:README] Step failed: exit status 1
  | 🚨 Error: The command exited with status 1
  | user command error: exit status 1


This makes the target more explicit

Test plan

rerun release test

@DaedalusG DaedalusG enabled auto-merge (squash) November 13, 2025 00:50
@DaedalusG DaedalusG merged commit 17c6688 into main Nov 13, 2025
4 checks passed
@DaedalusG DaedalusG deleted the wg/rel/fix-readme-expected-test branch November 13, 2025 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants