Couple minor things inline

On Thu, 2010-06-03 at 20:32 +0200, [email protected] wrote:
> From: Jan Provaznik <[email protected]>
> 
> config files with repository links:
> image_descriptor_package_repositories.yml - used for package selection
> image_descriptor_platform_repositories.yml - used for selection of platform
> (only Fedora makes sense), I changed url for fedora to mirror (orig was too 
> slow)
> src/config/image_descriptor_targets.yml - config files with target names
> ---
>  .../image_descriptor_package_repositories.yml      |    7 +++++++
>  .../image_descriptor_platform_repositories.yml     |    8 ++++++++
>  src/config/image_descriptor_targets.yml            |   17 +++++++++++++++++
>  3 files changed, 32 insertions(+), 0 deletions(-)
>  create mode 100644 src/config/image_descriptor_package_repositories.yml
>  create mode 100644 src/config/image_descriptor_platform_repositories.yml
>  create mode 100644 src/config/image_descriptor_targets.yml
> 
> diff --git a/src/config/image_descriptor_package_repositories.yml 
> b/src/config/image_descriptor_package_repositories.yml
> new file mode 100644
> index 0000000..7e9e685
> --- /dev/null
> +++ b/src/config/image_descriptor_package_repositories.yml
> @@ -0,0 +1,7 @@
> +fedora:
> +  name: Fedora
> +  baseurl: 
> http://mirror.karneval.cz/pub/linux/fedora/linux/releases/12/Everything/x86_64/os/
This probably needs to be fedora 11 so it matches the jboss repo ^

That mirror may be better for you, but dog slow for me - any way we
could use the mirrorlist, like in .repo files?  If not, maybe include a
comment with a link to:
https://mirrors.fedoraproject.org/publiclist/Fedora/11/x86_64/
so it is easy for others to get hold of a local mirror

> +jboss:
> +  name: JBoss
> +  #baseurl: http://repo.oddthesis.org/cirras/packages/fedora/11/RPMS/x86_64/
> +  baseurl: http://repo.oddthesis.org/cirras/packages/fedora/11/RPMS/noarch/
> diff --git a/src/config/image_descriptor_platform_repositories.yml 
> b/src/config/image_descriptor_platform_repositories.yml
> new file mode 100644
> index 0000000..a50fa08
> --- /dev/null
> +++ b/src/config/image_descriptor_platform_repositories.yml
> @@ -0,0 +1,8 @@
> +fedora:
> +  name: Fedora
> +  # FIXME: orig url is slooow, use mirror
> +  baseurl: 
> http://mirror.karneval.cz/pub/linux/fedora/linux/releases/12/Everything/x86_64/os/
> +rhel:
> +  name: RHEL
> +  # this should be ignored, it's defined just to be showned in selection tag
> +  baseurl: 
> http://mirror.karneval.cz/pub/linux/fedora/linux/releases/12/Everything/x86_64/os/
> diff --git a/src/config/image_descriptor_targets.yml 
> b/src/config/image_descriptor_targets.yml
> new file mode 100644
> index 0000000..91848c9
> --- /dev/null
> +++ b/src/config/image_descriptor_targets.yml
> @@ -0,0 +1,17 @@
> +amazon_ec2:
> +  name: Amazon EC2
> +
> +rackspace:
> +  name: RackSpace
> +
> +disk_image:
> +  name: Disk Image
This one is not a valid target ^
> +
> +rhev:
> +  name: RHEV
This should be RHEV-M ^
> +
> +gogrid:
> +  name: GoGrid
> +
> +vmware_esx:
> +  name: VMWare ESX

Other than those trivial fixes, ACK

-j

_______________________________________________
deltacloud-devel mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/deltacloud-devel

Reply via email to