Skip to content

canyonos test failing after canyonos deploy + stop (CAN-347) - #103

Open
Saaketh0 wants to merge 2 commits into
mainfrom
CAN-347-Test-Fail-After-Deploy
Open

Saaketh0 wants to merge 2 commits into
mainfrom
CAN-347-Test-Fail-After-Deploy

Conversation

@Saaketh0

Copy link
Copy Markdown
Collaborator

When canyonos deploy is ran and stopped, any subsequent canyonos tests will cause unexpected errors to the program. This is due to the fact that canyonos stop didn't remove the containers, just stopped them, unlike quit, resulting in canyonos test trying to access the containers and erroring because those containers are stopped.

This was a deeper issue with overlap between canyonos test and deploy, and I decided to try to completely decouple canyonos test deploys with the actual deploy path. This involves launching canyonos test with completely different containers, being prefixed with "test" and being located on the 9000 port range (not the best fix, but good for now). This allows test to occupy different containers from deploy, with no risk of clobbering/getting clobbered by anything.

I also set canyonos test to delete all test containers immediately upon exit, its a bandage solution in the sense of being unable to debug containers if it fails as they get shut down, but something i did for simplicity.

@Saaketh0 Saaketh0 self-assigned this Sep 11, 2026
@Saaketh0 Saaketh0 added the bug Something isn't working label Sep 11, 2026
@Saaketh0
Saaketh0 marked this pull request as ready for review September 11, 2026 22:52
@coderabbitai

coderabbitai Bot commented Sep 11, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 3af232d3-084b-4958-9b49-0756a96a006b


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant