[EDP] 'Pending' is the only job status with lowercase

Bug #1329526 reported by Andrew Lazarev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Sahara
Fix Released
Medium
Susanne Balle

Bug Description

For job status we use Oozie job statuses as is without any mapping. Oozie statuses are in uppercase (see http://oozie.apache.org/docs/3.1.3-incubating/client/apidocs/org/apache/oozie/client/Job.Status.html).

Moreover we hardcode oozie statuses in Sahara code and use them as job statuses.

The only status that differs is initial 'Pending' status (see screenshot).

Since we start using uppercase for job statuses I suggest to rename it to 'PENDING'.

Revision history for this message
Andrew Lazarev (alazarev) wrote :
tags: added: low-hanging-fruit
Changed in sahara:
assignee: nobody → Susanne Balle (susanne-balle)
Changed in sahara:
status: New → Triaged
milestone: none → juno-2
importance: Undecided → Low
Changed in sahara:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to sahara (master)

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

Revision history for this message
Susanne Balle (susanne-balle) wrote :
Changed in sahara:
status: In Progress → Fix Committed
Changed in sahara:
status: Fix Committed → In Progress
Changed in sahara:
status: In Progress → Fix Committed
Changed in sahara:
status: Fix Committed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to sahara (master)

Reviewed: https://review.openstack.org/107047
Committed: https://git.openstack.org/cgit/openstack/sahara/commit/?id=a6aea22d74b0e8b5a827ae5f273a2f071734fcd3
Submitter: Jenkins
Branch: master

commit a6aea22d74b0e8b5a827ae5f273a2f071734fcd3
Author: Susanne Balle <email address hidden>
Date: Tue Jul 15 09:18:34 2014 -0400

    Renamed Pending to PENDING
    fixes bug 1329526

    Change-Id: I53750cd9bf23df06678d555e5c1fecfcc98edca3

Changed in sahara:
status: In Progress → Fix Committed
Changed in sahara:
importance: Low → Medium
Changed in sahara:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in sahara:
milestone: juno-2 → 2014.2
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.