Sample Swift tests don't have SLA defined

Bug #1554857 reported by Andrey Danin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Rally
Fix Released
Low
Roman Vasilets

Bug Description

We should use

   sla:
     failure_rate: 0

in all samples that we have

Changed in rally:
status: New → Confirmed
Revision history for this message
Roman Vasilets (rvasilets) wrote :

We don't use sla section in the samples. Please look at other samples

Changed in rally:
status: Confirmed → Won't Fix
Revision history for this message
Boris Pavlovic (boris-42) wrote :

Roman,

Actually we should use, I would better refactor this bug

Changed in rally:
status: Won't Fix → Triaged
importance: Undecided → Low
description: updated
Changed in rally:
assignee: nobody → Roman Vasilets (rvasilets)
assignee: Roman Vasilets (rvasilets) → nobody
Revision history for this message
Roman Vasilets (rvasilets) wrote :

Will fix it for all samples and add hacking

Changed in rally:
assignee: nobody → Roman Vasilets (rvasilets)
Changed in rally:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on rally (master)

Change abandoned by Roman Vasilets (<email address hidden>) on branch: master
Review: https://review.openstack.org/348395

Revision history for this message
zhangzhihui (zhangzhang) wrote :

is anyone working for the bug?

Revision history for this message
Andrey Danin (gcon-monolake) wrote : Re: [Bug 1554857] Re: Sample Swift tests don't have SLA defined

I believe, no one is working.

On Wed, Jul 12, 2017 at 12:33 AM, zhangzhihui <email address hidden>
wrote:

> is anyone working for the bug?
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1554857
>
> Title:
> Sample Swift tests don't have SLA defined
>
> Status in Rally:
> In Progress
>
> Bug description:
> We should use
>
> sla:
> failure_rate: 0
>
> in all samples that we have
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/rally/+bug/1554857/+subscriptions
>

--
Andrei Danin
Senior Deployment Engineer
Mirantis, Inc.
Phone: 650-660-3607
Email: <email address hidden>

Revision history for this message
zhangzhihui (zhangzhang) wrote :

because the status of Roman's patch is abandoned, so i propose a new patch. please look at https://review.openstack.org/#/c/484225/

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

Reviewed: https://review.openstack.org/484225
Committed: https://git.openstack.org/cgit/openstack/rally/commit/?id=a9b88a5917ed3f23fa7411a1fd1998704c903a7e
Submitter: Jenkins
Branch: master

commit a9b88a5917ed3f23fa7411a1fd1998704c903a7e
Author: zhangzhang <email address hidden>
Date: Wed Jul 12 15:37:34 2017 -0400

    Add sla section to samples

    Add missing sla section and proper check
    This work is originally porposed by Roman[1]

    [1] https://review.openstack.org/#/c/348395/3

    Change-Id: I4507aa82080b41cac4b87aa810865da9ffe015e6
    Closes-bug: #1554857

Changed in rally:
status: In Progress → Fix Released
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.