Skip to content

updated Dockerfile to use ubuntu 20.04 as base - #25435

Merged
mraarif merged 1 commit into
masterfrom
BOM-2068
Nov 3, 2020
Merged

updated Dockerfile to use ubuntu 20.04 as base#25435
mraarif merged 1 commit into
masterfrom
BOM-2068

Conversation

@mraarif

@mraarif mraarif commented Oct 26, 2020

Copy link
Copy Markdown
Contributor

No description provided.

@mraarif

mraarif commented Oct 26, 2020

Copy link
Copy Markdown
Contributor Author

needs https://github.com/edx/edx-platform/pull/25362 to merge to run container properly with mysql57

@mraarif
mraarif marked this pull request as ready for review October 27, 2020 13:56
@mraarif
mraarif requested a review from jmbowman October 29, 2020 09:39
@mraarif
mraarif force-pushed the BOM-2068 branch 2 times, most recently from 19d6341 to b84c13d Compare October 29, 2020 09:46

@jmbowman jmbowman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good other than the one noted suggestion.

Comment thread Dockerfile Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The line above can go away also, although you may need to add software-properties-common back to the list of packages below. See https://github.com/edx/edx-platform/pull/25205/files for the original change.

@edx-status-bot

Copy link
Copy Markdown

Your PR has finished running tests. There were no failures.

@mraarif
mraarif merged commit dafac0c into master Nov 3, 2020
@mraarif
mraarif deleted the BOM-2068 branch November 3, 2020 12:28
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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.

4 participants