Skip to content

feat(test): Add more cloud tests #1879

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.

Sign up for GitHub

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

Jump to bottom
Open
craciunoiuc wants to merge 31 commits into
base: staging
Choose a base branch
from
Open

feat(test): Add more cloud tests #1879

craciunoiuc wants to merge 31 commits into from

Conversation

@craciunoiuc
Copy link
Member

@craciunoiuc craciunoiuc commented Sep 6, 2024

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 •
edited
Loading

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 mentioned this pull request Jan 21, 2025
Merged
4 tasks
@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
craciunoiuc added 15 commits July 1, 2025 11:35
@craciunoiuc
feat(test): Add tests for 'cloud instance get'
d4dade4 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud instance list'
dc34b1d 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud instance logs'
771f576 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud instance remove'
24a5783 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud instance start'
525dc5f 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud instance stop'
0ef8115 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Rename test suite to 'cloud'
41725e7 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
fix(test): Remove instance after running test
7b3a879 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud quotas'
1f43d11 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud metro list'
7fa631f 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud volume attach'
0974db9 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud volume detach'
c59f929 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud volume create'
7505235 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud volume delete'
4d848ab 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud volume get'
565b46a 
Signed-off-by: Cezar Craciunoiu <[email protected]>
craciunoiuc added 16 commits July 1, 2025 11:35
@craciunoiuc
feat(test): Add tests for 'cloud volume list'
a9856ee 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
fix(test): Increase minimum volume size in tests
24df094 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud volume import'
413d9a7 
Also add test fixtures for working with cpio archives.

Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud service create'
44f09da 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud service list'
9567b81 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud service get'
4d28708 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud service remove'
015abe2 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud tunnel'
206adec 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud scale init'
c72bd39 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud scale add'
21e447a 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud scale remove'
de9c552 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud scale reset'
0de9b24 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud scale get'
43df57c 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
feat(test): Add tests for 'cloud instance restart'
48f798d 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
fix(test): Bring all tests to a working state on staging
7440f16 
This still needs rebasing to break the changes of this
commit up and merge in the corresponding commits.

Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc
gomod(deps): Bump 'sdk.kraft.cloud' to the latest version
5704c51 
Signed-off-by: Cezar Craciunoiu <[email protected]>
@craciunoiuc craciunoiuc force-pushed the craciunoiuc/add-more-instance-tests branch from 032156f to 5704c51 Compare July 3, 2025 14:49
@craciunoiuc
Copy link
Member Author

craciunoiuc commented Oct 16, 2025

tagging @mdsahiloss as somehow I can't add you as a reviewer 👀

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

Reviewers

@nderjung nderjung Awaiting requested review from nderjung nderjung is a code owner

@ajpetersons ajpetersons Awaiting requested review from ajpetersons

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@craciunoiuc