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 test container ftp #399

Merged
merged 8 commits into from
Aug 8, 2024
Merged

add test container ftp #399

merged 8 commits into from
Aug 8, 2024

Conversation

Yuhu-kth
Copy link
Collaborator

gonna add comments later

@Yuhu-kth
Copy link
Collaborator Author

Yuhu-kth commented Aug 4, 2024

Screenshot 2024-08-04 at 19 28 58
I tested all the containers with tag0.6.0, it turns out 23 out of 56 are not running after started

@Yuhu-kth
Copy link
Collaborator Author

Yuhu-kth commented Aug 6, 2024

Screenshot 2024-08-06 at 21 34 08 I tried to assign "15.15.15.15" to container1, and use container2 to ping. Container2 was assigned to "15.15.15.2" automatically. The ping is unsuccessful, checked the network is connected to container1 correctly and container1 is running properly, checked container2 was connected to the same network too. Don't know why. Tried to assign "15.15.15.2" to container1, then it can be pinged, like this Screenshot 2024-08-06 at 22 21 38 I made the error to see the output, here you can see packets are received. but it's weird that container2 was still assigned to "15.15.15.2", I thought it will be assigned to another ip-address. I wanna ask your idea about it, because maybe its because I'm not familiar with network. I'll check tmr anyways.

Copy link
Owner

@Limmen Limmen left a comment

Choose a reason for hiding this comment

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

Looks good in general, but do we need the bash file? otherwise remove it and then I will merge

@@ -0,0 +1,13 @@
docker network create --subnet=15.15.15.0/24 my_custom_network
Copy link
Owner

Choose a reason for hiding this comment

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

Do we need this file?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

deleted

@Limmen Limmen merged commit 3518898 into master Aug 8, 2024
@Limmen Limmen deleted the test_container_ftp branch August 8, 2024 19:43
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