url_available() doesn't support the "mirror" method

Bug #1669751 reported by Michael Polenchuk
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Michael Polenchuk
Newton
Fix Committed
Medium
Sergii Golovatiuk

Bug Description

Detailed bug description:
 apt-get supports a "mirror" method that will automatically select a closest mirror based on geographical location, but ... deployments will be failing since url_available() function in connectivity_tests task doesn't support this method.

Steps to reproduce:
 Set mirror://mirrors.ubuntu.com/mirrors.txt as repository source.

Logs:
 Puppet (err): ERROR: Unable to fetch url 'mirror://mirrors.ubuntu.com/mirrors.txt', error 'undefined method `request_uri' for #<URI::Generic mirror://mirrors.ubuntu.com/mirrors.txt>'. Please verify node connectivity to this URL, or remove it from the settings page if it is invalid. on node node-4.domain.tld
/etc/puppet/modules/osnailyfacter/lib/puppet/parser/functions/url_available.rb:95:in `rescue in fetch'
/etc/puppet/modules/osnailyfacter/lib/puppet/parser/functions/url_available.rb:83:in `fetch'
/etc/puppet/modules/osnailyfacter/lib/puppet/parser/functions/url_available.rb:106:in `block (4 levels) in <top (required)>'

Workaround:
 Skip connectivity_tests task

Description of the environment:
 Fuel 10+

Changed in fuel:
milestone: none → 11.0
assignee: nobody → Michael Polenchuk (mpolenchuk)
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/441101

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-web (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/449494

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/ocata)

Fix proposed to branch: stable/ocata
Review: https://review.openstack.org/461145

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/461146

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/441101
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=ae1c80f8fbd1d7156075a1dbf61e832b00852bb7
Submitter: Jenkins
Branch: master

commit ae1c80f8fbd1d7156075a1dbf61e832b00852bb7
Author: Michael Polenchuk <email address hidden>
Date: Fri Mar 3 16:36:06 2017 +0400

    Support a "mirror" method of apt-get

    apt-get supports a "mirror" method that will automatically select a
    closest mirror based on geographical location. So extend URI library
    to handle repository sources with mirror://

    Change-Id: I7b8018af50e431614d0d5c87ad3f1c8bad94e4d9
    Closes-Bug: #1669751

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/newton)

Reviewed: https://review.openstack.org/461146
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=c8b04173d932cb57891117b96f19d9257c91057d
Submitter: Jenkins
Branch: stable/newton

commit c8b04173d932cb57891117b96f19d9257c91057d
Author: Michael Polenchuk <email address hidden>
Date: Fri Mar 3 16:36:06 2017 +0400

    Support a "mirror" method of apt-get

    apt-get supports a "mirror" method that will automatically select a
    closest mirror based on geographical location. So extend URI library
    to handle repository sources with mirror://

    Change-Id: I7b8018af50e431614d0d5c87ad3f1c8bad94e4d9
    Closes-Bug: #1669751

tags: added: in-stable-ocata
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/ocata)

Reviewed: https://review.openstack.org/461145
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=2ba8d872e8d38bacdbda42c04276be6c56d263cc
Submitter: Jenkins
Branch: stable/ocata

commit 2ba8d872e8d38bacdbda42c04276be6c56d263cc
Author: Michael Polenchuk <email address hidden>
Date: Fri Mar 3 16:36:06 2017 +0400

    Support a "mirror" method of apt-get

    apt-get supports a "mirror" method that will automatically select a
    closest mirror based on geographical location. So extend URI library
    to handle repository sources with mirror://

    Change-Id: I7b8018af50e431614d0d5c87ad3f1c8bad94e4d9
    Closes-Bug: #1669751

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-web (master)

Change abandoned by Andreas Jaeger (<email address hidden>) on branch: master
Review: https://review.opendev.org/449494
Reason: This repo is retired now, no further work will get merged.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library ocata-eol

This issue was fixed in the openstack/fuel-library ocata-eol release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.