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

chore: circleci: Use image with go 1.20.3 installed #371

Merged
merged 5 commits into from
May 2, 2023

Conversation

vitsalis
Copy link
Member

@vitsalis vitsalis commented May 2, 2023

On the rpc-client repository I encountered issues due to the Ubuntu image being used having Go 1.18 installed instead of the 1.20 version that we use. These were fixed when I used an image with the appropriate version installed, so figured to do the same here.

Given that we use Docker for some of the build steps this fails. Will investigate further.

@vitsalis vitsalis marked this pull request as draft May 2, 2023 08:08
machine:
image: ubuntu-2204:2022.10.1
resource_class: large
docker:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Unfortunately for integration tests we are using and starting docker container, and last time I check it was impossible to do it in docker execution enviroment. That why we were always using https://circleci.com/docs/executor-intro/#linux-vm on babylon node side

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 changed this to install the correct go version in the docker image and it looks like we encounter no issues 🥳

@vitsalis vitsalis marked this pull request as ready for review May 2, 2023 08:52
@vitsalis
Copy link
Member Author

vitsalis commented May 2, 2023

Ok managed to resolve this using an orb, please check again! Also found a linting issue due to a deprecated method we did not find before due to using the incorrect golang version :)

@vitsalis vitsalis merged commit 813134b into dev May 2, 2023
@vitsalis vitsalis deleted the vitsalis/circle-ci-executor branch May 2, 2023 09:38
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