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

Fix CircleCI configuration #339

Open
wants to merge 1 commit into
base: channel/beta
Choose a base branch
from

Conversation

BrianHawley
Copy link

CircleCI sent me an email indicating that the config settings for this repo were set to use a deprecated image (Ubuntu 14.04) because we didn't specify the image to use.

The image option is required, per their docs; set it as recommended.

@BrianHawley BrianHawley force-pushed the fixes_20221216_circle_ci_config branch from e846427 to 7b384f9 Compare December 16, 2022 19:27
CircleCI sent me an email indicating that the config settings for this
repo were set to use a deprecated image (Ubuntu 14.04) because we didn't
specify the image to use.

The image option is required, per their docs; set it as recommended.
@BrianHawley BrianHawley force-pushed the fixes_20221216_circle_ci_config branch from 7b384f9 to 5e6364a Compare December 16, 2022 19:30
@BrianHawley
Copy link
Author

@fede-moya or @filipesperandio could you review and merge?

This should also be able to be cherry-picked to other new channels, though they really should be based on channel/beta anyways so that shouldn't be a problem. The particular setting only affects the process of creating a new version of the engine image in docker, and only affects the host process that is building the image. It doesn't affect the built image itself, which is based on Dockerfile instead of this config.

@BrianHawley
Copy link
Author

@fede-moya this change is still needed. Could you review?

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.

1 participant