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

Add e2e tests #165

Merged
merged 3 commits into from
Jun 17, 2024
Merged

Add e2e tests #165

merged 3 commits into from
Jun 17, 2024

Conversation

kvaps
Copy link
Member

@kvaps kvaps commented Jun 12, 2024

Add script that creates three VMs and installs Cozystack into them

@kvaps kvaps changed the title e2e Add e2e tests Jun 12, 2024
@kvaps kvaps requested review from gecube, themoriarti and nbykov0 June 12, 2024 17:20
@kvaps kvaps force-pushed the e2e branch 3 times, most recently from 5c1aff7 to a3729ae Compare June 12, 2024 17:40
Signed-off-by: Andrei Kvapil <[email protected]>
Copy link
Collaborator

@themoriarti themoriarti left a comment

Choose a reason for hiding this comment

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

After add MASQUERADE, it would be good to stop the virtual machines at the end of the test or add the possibility of configuration parameter for that.

ip link set cozy-srv$i up
ip link set cozy-srv$i master cozy-br0
done

Copy link
Collaborator

Choose a reason for hiding this comment

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

To be able to go outside add:

echo 1 > /proc/sys/net/ipv4/ip_forward
iptables -t nat -A POSTROUTING -s 192.168.123.0/24 -j MASQUERADE

Copy link
Member Author

Choose a reason for hiding this comment

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

I have to add check to test for this

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, just added this to tests

@kvaps kvaps requested a review from themoriarti June 17, 2024 16:04
Copy link
Collaborator

@themoriarti themoriarti left a comment

Choose a reason for hiding this comment

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

Now looks good.

@themoriarti themoriarti merged commit 2d40c85 into main Jun 17, 2024
@themoriarti themoriarti deleted the e2e branch June 17, 2024 17:14
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