Bug 1194451

Summary: [PATCH] Add dnf support to cloud-init
Product: [Fedora] Fedora Reporter: Mike Ruckman <mruckman>
Component: cloud-initAssignee: Garrett Holmstrom <gholms>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: unspecified    
Version: rawhideCC: apevec, gholms, Jan.van.Eldik, mattdm, p, shardy, s
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: cloud-init-0.7.6-3.fc22 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-03-12 17:49:24 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
dnf-support patch none

Description Mike Ruckman 2015-02-19 20:12:07 UTC
Created attachment 993770 [details]
dnf-support patch

Description of problem:
cloud-init doesn't currently support dnf when installing packages. F22 will need dnf support going forward. Provided patch adds dnf support and falls back to yum if dnf isn't installed.

Version-Release number of selected component (if applicable):
cloud-init-0.7.6-2

Comment 1 Garrett Holmstrom 2015-02-23 22:35:07 UTC
This patch actually gets a lot smaller when written against cloud-init 0.7.6.  :-)  Here are a couple comments:

  1. Why not just test whether or not /usr/bin/dnf exists instead of running an external command?
  2. The log messages should probably be debug messages.

I added a patch to that effect in cloud-init-0.7.6-3.fc22, so feel free to begin testing with that any time you wish.  cloud-init-0.7.6-4.20140218bzr1060.fc22 will be pushed shortly with some major changes, so it may be worth testing against that instead.

Comment 2 Garrett Holmstrom 2015-02-23 22:40:26 UTC
Update:  cloud-init-0.7.6-4.20140218bzr1060.fc22 isn't going to make it into F22 due to bug 1194577.  If you're testing rawhide it will appear there, but otherwise feel free to go ahead and test cloud-init-0.7.6-3.fc22.  Sorry about the confusion.