Bug 1194451 - [PATCH] Add dnf support to cloud-init
Summary: [PATCH] Add dnf support to cloud-init
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: cloud-init
Version: rawhide
Hardware: All
OS: Linux
unspecified
low
Target Milestone: ---
Assignee: Garrett Holmstrom
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-02-19 20:12 UTC by Mike Ruckman
Modified: 2015-03-12 17:49 UTC (History)
7 users (show)

Fixed In Version: cloud-init-0.7.6-3.fc22
Clone Of:
Environment:
Last Closed: 2015-03-12 17:49:24 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
dnf-support patch (3.12 KB, application/mbox)
2015-02-19 20:12 UTC, Mike Ruckman
no flags Details

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.


Note You need to log in before you can comment on or make changes to this bug.