Skip to content

Conversation

@SimonBaeumer
Copy link
Member

Fixes #

Checklist

  • Added unit / integration tests for windows, macOS and Linux?
  • Added a changelog entry in CHANGELOG.md?
  • Updated the documentation (README.md, docs)?
  • Does your change work on Linux, Windows and macOS?

@SimonBaeumer SimonBaeumer changed the title Upgrade codeclimate test reporter Fix CI pipeline Nov 20, 2021
@dylanhitt
Copy link
Member

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 bump-docker-version and find sporadic successes. I'm a bit lost. Maybe in a few weeks it will resolve 🤷

@dylanhitt
Copy link
Member

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 🤔

@dylanhitt
Copy link
Member

@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.

@dylanhitt
Copy link
Member

Hey @SimonBaeumer CircelCI seems to be behaving now. It seems the latest bump-docker pipelines are passing.

You want to give this. another shot.

@dylanhitt dylanhitt force-pushed the upgrade-test-reporter branch 12 times, most recently from 2bcd11a to 3131500 Compare January 7, 2022 05:19
Copy link
Member

@dylanhitt dylanhitt left a 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.

@dylanhitt dylanhitt force-pushed the upgrade-test-reporter branch from 3131500 to 2d3e628 Compare February 16, 2022 03:43
@dylanhitt dylanhitt force-pushed the upgrade-test-reporter branch from 2d3e628 to 87783c5 Compare February 25, 2022 06:00
@dylanhitt dylanhitt merged commit b067197 into master Feb 25, 2022
@dylanhitt dylanhitt deleted the upgrade-test-reporter branch February 25, 2022 06:08
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.

3 participants