-
Notifications
You must be signed in to change notification settings - Fork 19
Fix CI pipeline #180
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
Fix CI pipeline #180
Conversation
|
After a lot of head banging. I've come to the conclusion this DNS issues has to be on the circleCI side... Now we have the problem of these pipelines failing silently but that's for another time. However, here is example of a pipeline working with he docker properly pulling and running our test. I purposely changed the image over to the golang image that we use in the docker builds. You can see that the image was pulled/ran. I then made this minor change to get the pipeline to pass. The very next pipeline fails with DNS name resolution problems. You can look through all the pipelines from the |
|
One solution could be separating out the ssh test and docker tests. SSH seems to be fine within the containers and all docker commands seem to be more stable outside of the container... However I was picking up random docker pull failures for the alpine image even outside the dockerized thes.. see this, here is the line it failed on. It seems though running as separate jobs is fine. I don't know. This seems a bit strange to me 🤔 |
|
@SimonBaeumer do you have any opinions on proceeding? My only solution would be to separate out the docker execution test from the ssh test. It seems like circle ci really hates the docker executor spinning up another container with in a container. To be brief, running the docker integration as it’s own job seems to work all the time. |
|
Hey @SimonBaeumer CircelCI seems to be behaving now. It seems the latest bump-docker pipelines are passing. You want to give this. another shot. |
2bcd11a to
3131500
Compare
dylanhitt
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed the script to get it running. Mainly just putting the COPY back in the dockerfile even though mounting is the better option.. Not sure why circleci doesn't like that.
3131500 to
2d3e628
Compare
2d3e628 to
87783c5
Compare
Fixes #
Checklist
Linux,WindowsandmacOS?