-
Notifications
You must be signed in to change notification settings - Fork 459
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
RHEL8 spec update to require moby-engine or docker-ce #6972
Conversation
What about |
Just tested this. Does not support or |
Cool, so just test these scenarios on EL8 to be sure:
|
Defaults to docker repo |
Hmm, it would be nice if it preferred |
* RHEL8 spec update to require moby-engine or docker-ce (#6972) Co-authored-by: Andrew Smith <andsmi@microsoft.com> Co-authored-by: Damon Barry <damonbarry@users.noreply.github.com>
Adding a requires clause to spec for rhel8 "Requires: (moby-engine or docker-ce)", as well as moving check for docker group to %post.
Azure IoT Edge PR checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines and Best Practices
Testing Guidelines
Draft PRs
Draft
mode if it is:Note: We use the kodiakhq bot to merge PRs once the necessary checks and approvals are in place. When it merges a PR, kodiakhq converts the PR title to the commit title, PR description to the commit description, and squashes all the commits in the PR to a single commit. The net effect is that entire PR becomes a single commit. Please follow the best practices mentioned here for the PR title and description