Skip to content

test: add e2e instance test #1037

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

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

test: add e2e instance test #1037

wants to merge 11 commits into from

Conversation

Laure-di
Copy link
Contributor

No description provided.

@Laure-di Laure-di requested review from a team and remyleone as code owners June 20, 2025 09:08
@github-actions github-actions bot added webhosting Webhosting issues, bugs and feature requests container Container issues, bugs and feature requests vpcgw labels Jun 20, 2025
@github-actions github-actions bot removed webhosting Webhosting issues, bugs and feature requests container Container issues, bugs and feature requests vpcgw labels Jun 20, 2025
@Laure-di Laure-di force-pushed the e2e-tests branch 4 times, most recently from 510004c to a0115cb Compare June 20, 2025 10:53
Copy link

@Gnoale Gnoale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well done !

@Laure-di Laure-di force-pushed the e2e-tests branch 2 times, most recently from 67065c2 to 29ce792 Compare June 20, 2025 12:56
@Laure-di Laure-di force-pushed the e2e-tests branch 2 times, most recently from 7e0fc2f to eec183b Compare June 20, 2025 13:35
Gnoale
Gnoale previously approved these changes Jun 20, 2025
Gnoale
Gnoale previously approved these changes Jun 20, 2025
self.instanceAPI.detach_server_volume(
server_id=self._server.id, volume_id=volume.id
)
logger.info("✅ Volume {volume.id} has been detach")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should remove logging from the fixtures

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.

2 participants