Re: [virt-install PATCH v2 4/7] urldetect: return osdict.py:OsMedia instead of Libosinfo.Media

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 3/22/19 11:23 AM, Fabiano Fidêncio wrote:
> Consequently, let's:
> - rename urldetect::get_osinfo_media() to get_os_media()
> - rename the installertreemedia::_LocationData::osinfo_media to os_media
> - rename unattended::prepare_install_script's media argument to os_media
>   - and also rename media to os_media on its internal
>     _get_installation_source() method
> - rename osdict::get_install_script's media argument to os_media
> 
> Signed-off-by: Fabiano Fidêncio <fidencio@xxxxxxxxxx>
> ---
>  virtinst/installertreemedia.py |  8 ++++----
>  virtinst/osdict.py             |  3 ++-
>  virtinst/unattended.py         | 10 ++++++----
>  virtinst/urldetect.py          |  8 ++++----
>  4 files changed, 16 insertions(+), 13 deletions(-)
> 
> diff --git a/virtinst/installertreemedia.py b/virtinst/installertreemedia.py
> index 7c69c3fb..7e97c10a 100644
> --- a/virtinst/installertreemedia.py
> +++ b/virtinst/installertreemedia.py
> @@ -30,10 +30,10 @@ def _is_url(url):
>  
>  
>  class _LocationData(object):
> -    def __init__(self, os_variant, kernel_pairs, osinfo_media):
> +    def __init__(self, os_variant, kernel_pairs, os_media):
>          self.os_variant = os_variant
>          self.kernel_pairs = kernel_pairs
> -        self.osinfo_media = osinfo_media
> +        self.os_media = os_media
>  
>          self.kernel_url_arg = None
>          if self.os_variant:
> @@ -129,7 +129,7 @@ class InstallerTreeMedia(object):
>              if store:
>                  kernel_paths = store.get_kernel_paths()
>                  os_variant = store.get_osdict_info()
> -                osinfo_media = store.get_osinfo_media()
> +                os_media = store.get_os_media()
>              if has_location_kernel:
>                  kernel_paths = [
>                          (self._location_kernel, self._location_initrd)]

There were two other instances in this function of osinfo_media. I fixed
those and the two issues you pointed out, and pushed this series

Thanks,
Cole

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list




[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux