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

Updated yum -y install yum-utils to yum -y upgrade #13555

Closed
wants to merge 1 commit into from

Conversation

brianrkleis
Copy link

Docker requires more updated libraries than yum-utils. I've tested yum-utils in a new Virtual Machine without updating any libs, and couldn't get docker working. For new environments this is a better command and for any environment.

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Docker requires more updated libraries than yum-utils. I've tested yum-utils in a new Virtual Machine without updating any libs, and couldn't get docker working. For new environments this is a better command and for any environment.
@netlify
Copy link

netlify bot commented Sep 17, 2021

✔️ Deploy Preview for docsdocker ready!

🔨 Explore the source changes: 2260180

🔍 Inspect the deploy log: https://app.netlify.com/sites/docsdocker/deploys/6144dc39b907130008c62a56

😎 Browse the preview: https://deploy-preview-13555--docsdocker.netlify.app

@usha-mandya
Copy link
Member

@thaJeztah Could you PTAL?

$ sudo yum install -y yum-utils
$ sudo yum -y upgrade
Copy link
Member

Choose a reason for hiding this comment

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

This looks to be addressing different goals;

 sudo yum install -y yum-utils

Is added here to make sure yum-config-manager is installed (which is used below).

yum upgrade will update / upgrade all packages and dependencies installed on the machine. While this could be a good recommendation, this would be somewhat out of scope for the installation instructions.

What's the exact problem you ran into? Did the installation fail, or did it warn that a newer version of a dependency was needed?

@thaJeztah
Copy link
Member

closing this, as I don't think this change is needed per my comment above

@thaJeztah thaJeztah closed this Nov 11, 2021
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