rally verify doesn't work if there is no web service on 80 port

Bug #1423358 reported by Boris Pavlovic
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Rally
Fix Released
Medium
Nha Pham

Bug Description

https://github.com/stackforge/rally/blob/master/rally/verification/tempest/config.py#L233-L235

This code:

        horizon_url = ("http://" +
                       parse.urlparse(self.endpoint["auth_url"]).hostname)
        horizon_availability = (requests.get(horizon_url).status_code == 200)

Is bad for 2 reasons. It can take forever to get timeout and if timeout occurs it breaks everything. So I would prefer to add next code:

        try:
          horizon_url = ("http://" +
                         parse.urlparse(self.endpoint["auth_url"],
                         timeout=15).hostname)
          horizon_availability = (requests.get(horizon_url).status_code == 200)
        except requests.exceptions.Timeout:
          horizon_availability = False

So rally verify command want crash in this case.

Changed in rally:
status: New → Triaged
importance: Undecided → Medium
description: updated
Nha Pham (phqnha)
Changed in rally:
assignee: nobody → Nha Pham (phqnha)
Revision history for this message
Nha Pham (phqnha) wrote :
Changed in rally:
status: Triaged → Fix Released
Revision history for this message
Marian Krcmarik (mkrcmari) wrote :

It still takes forever, It would be nice to implement the timeout.

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.