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

Update CI docker images with CircleCI python images #5720

Merged
merged 1 commit into from Jan 19, 2021

Conversation

emkll
Copy link
Contributor

@emkll emkll commented Jan 19, 2021

Status

Ready for review

Description of Changes

Fixes #5719

The container image provided by GCPs cloud-builders now recommends in its README the use of other docker containers, and is still running on Xenial. Since they are essentially swappable in place, here we use the containers that are maintained by the CI provider [2]

[1] https://github.com/GoogleCloudPlatform/cloud-builders
[2] https://circleci.com/developer/images/image/cimg/python

Testing

Deployment

Test/CI only

The docker container provided by GCPs cloud-builders now recommends in its README the use of other docker containers, and is still running on Xenial. Since they are essentially swappable in place, here we use the containers that are maintained by the CI provider [2]

[1] https://github.com/GoogleCloudPlatform/cloud-builders
[2] https://circleci.com/developer/images/image/cimg/python
@emkll emkll added this to Ready for Review in SecureDrop Team Board Jan 19, 2021
@codecov-io
Copy link

codecov-io commented Jan 19, 2021

Codecov Report

Merging #5720 (6660a1b) into develop (7b6f780) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #5720   +/-   ##
========================================
  Coverage    85.58%   85.58%           
========================================
  Files           52       52           
  Lines         3773     3773           
  Branches       471      471           
========================================
  Hits          3229     3229           
  Misses         439      439           
  Partials       105      105           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7b6f780...6660a1b. Read the comment docs.

Copy link
Contributor

@zenmonkeykstop zenmonkeykstop left a comment

Choose a reason for hiding this comment

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

CI is passing on both branches, this looks OK to go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Some CI targets are failing when dependencies are installed
3 participants