Bug 1194451 – [PATCH] Add dnf support to cloud-init

Bug 1194451 - [PATCH] Add dnf support to cloud-init

Summary:	[PATCH] Add dnf support to cloud-init

Status:	CLOSED RAWHIDE

Aliases:	None

Product:	Fedora
Classification:	Fedora
Component:	cloud-init (Show other bugs)
Sub Component:
Version:	rawhide
Hardware:	All Linux

Priority	unspecified Severity low
Target Milestone:	---
Target Release:	---
Assigned To:	Garrett Holmstrom
QA Contact:	Fedora Extras Quality Assurance
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2015-02-19 15:12 EST by Mike Ruckman
Modified:	2015-03-12 13:49 EDT (History)
CC List:	7 users (show)

See Also:
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 13:49:24 EDT
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
dnf-support patch (3.12 KB, application/mbox) 2015-02-19 15:12 EST, Mike Ruckman	no flags	Details
Add an attachment (proposed patch, testcase, etc.)

Groups: None (edit)

Description Mike Ruckman 2015-02-19 15:12:07 EST

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 17:35:07 EST

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 17:40:26 EST

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.