Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(test): Add more cloud tests #1879

Open
wants to merge 28 commits into
base: staging
Choose a base branch
from

Conversation

craciunoiuc
Copy link
Member

Prerequisite checklist

  • Read the contribution guidelines regarding submitting new changes to the project;
  • Tested your changes against relevant architectures and platforms;
  • Ran make fmt on your commit series before opening this PR;
  • Updated relevant documentation.

Description of changes

@craciunoiuc craciunoiuc requested a review from nderjung September 6, 2024 07:11
@craciunoiuc craciunoiuc force-pushed the craciunoiuc/add-more-instance-tests branch from 3c2277d to bc12299 Compare September 6, 2024 11:42
@craciunoiuc craciunoiuc changed the title feat: Add tests for rest of instance commands & fix issues feat: Add more tests & fix issues found by them Sep 13, 2024
@craciunoiuc craciunoiuc force-pushed the craciunoiuc/add-more-instance-tests branch 6 times, most recently from 3831109 to 66001bd Compare September 18, 2024 15:26
@craciunoiuc
Copy link
Member Author

craciunoiuc commented Sep 19, 2024

TODO: wait_timeout_ms to be renamed to timeout_s

Need to change commit: 8903b43

Edit: Once @marcrittinghaus updates nodes we can do this, it will be backwards compatible until then

@craciunoiuc craciunoiuc force-pushed the craciunoiuc/add-more-instance-tests branch 2 times, most recently from d72134e to 37dac91 Compare January 15, 2025 10:44
@craciunoiuc craciunoiuc force-pushed the craciunoiuc/add-more-instance-tests branch from 37dac91 to 032156f Compare January 21, 2025 13:25
@craciunoiuc craciunoiuc changed the title feat: Add more tests & fix issues found by them feat(test): Add more cloud tests Jan 21, 2025
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.

1 participant