In Neutron we created a tool to generate a Gerrit dashboard
that shows the most important reviews (bug fixes for high/critical bugs
and patches that implement approved blueprints) [1].
This patch adds a section to store the links of the project dashboards
in reviewday, copies the mentioned script in this repo and use it to
generate an updated link to the Gerrit dashboard.
[1] https://github.com/openstack/neutron/blob/master/tools/milestone-review-dash.py
Depends-on: I42ea25bdf69a1d75db24785db0e86396206fd48d
Change-Id: Id2e8590193fbf21a52a767323a77df9be4048b5a
This patch updates reviewday so that it generates
both an index.html and a separate data_table.html
which can be used for importing the reviewday HTML
table into an externally styled website.
The existing reviewday OpenStack style remains the same.
This patch should prove useful for sites wishing
to re-use the "data table" format as-is inside of a
separately themed web site.
Change-Id: If9a69235484038e4b10ee8a2c66ed834f5f6a8d2
Some projects use share a common Launchpad project for
reporting bugs (they don't always have split out
projects for these things). This patch updates
reviewday the --project-file format to support
specifying an altername 'launchpad_project' default.
Change-Id: I66afed988afe4c0cfa37d269fd397bcbb9e5badf
This patch updates the tox.ini file to create
virtualenv's using the new requirements files.
Also, adds a helper task to run reviewday in
a tox virtualenv. Useful on distros that
don't have launchpadlib packages handy...
Change-Id: I4fd2f70859d02a2402ed3b2387e1a6ffbcf588fd
Especially for bug review days it makes sense to sort the current
review list by review type (high prio bugfix, medium prio bugfix, ...)
This patch set introduces a new column "type" and allows a sorting for
it.
Change-Id: I3d37cccb8d6f75f6eebc2cf8ab9136ac50d4cf89
With the changeover to WIP being part of Workflow, the old WIP
detection no longer worked. This change makes WIP detection work
again and removes WIP reviews from the reviewday output.
Change-Id: Ic059b1e20708d3d6b6391fcebd7ff57afe7d7b0e
Closes-bug: 1321018
I stopped smoking in Hong Kong... so I think it
should be safe to remove this for now. If I decide
to pick up the habit again I can always add this
back in.
Change-Id: Ibc0d533aa295ba9a26c61995d63610cc53a2b11e
The "ReviewDay" header was far off to the right, fixing that with
edit to div and modification of CSS so it looks more like the
other pages on status.openstack.org.
Also add link to the new JSON file just below the header.
Change-Id: I45454594a20af5d20d2171899be25904a47b53ac
Also, updates our IO so we write to a .tmp file and
rename it only after the file IO is completed. This
wards off a potential race where a user could load
a page that hadn't been fully flushed to disk.
Change-Id: I04bfb7def801f11e3fcae83272879f660a8d1825
Because we were using a python tempfile the reviewday.json
file was readable/writable only by the user that runs
bin/reviewday. Ideally this file would be world readable so
that Apache can host it properly.
In this commit we drop the use of python tempfile in
favor of just plain old open() which respects the
normal file permission settings.
This should fix a Forbidden access error I get when
trying to access http://status.openstack.org/reviews/reviewday.json
today.
Closes-bug: #1273833
Change-Id: I9e9a76e29f5c71f2496eef96a6dbf2496c42d973
All non-WIP reviews should have a score of 1, this will ensure
that they are prioritized above WIP reviews. All reviews will
see an increase in score by 1 (WIP has a score of 0 now, all other
reviews start with a score of 1, and are modified up as appropriate).
Change-Id: I086cb01c7b8a71a9a1ec3cee2bd826f3d397c52e
WORKINPROGRESS reviews should not be at the top of the reviewday list
for a given project. The score will always be 0 for WIP reviews and
the subject line (in the generated report) will be prefixed with
a '(WIP): ' to indicate it is work-in-progress.
Change-Id: I32f7e08341dcbf1a0aa965a5f377c7fdc8613ead
Generate JSON data so that tools can easily make use of the info
that reviewday generates.
A new reviewday.json file is generated that has data like
{
"generated": "2014-01-03 22:22:33",
"projects": {
"identity-api": {
"https://review.openstack.org/#change,62961": {
"owner": "Dolph Mathews",
"score": 48,
"cause": "No link",
"feedback": {
"lowest": 1,
"highest": 1
},
"subject": "cleanup query_string documentation"
}, ... (for each review uri)
}, ... (for each project)
}
}
Change-Id: Idabafdadb3237410fa866c78d429cf46317fb3fe
Adds Ceilometer and Heat to SmokeStack results processing.
Also, bumps the jobs count a bit to give us headroom for
more results.
Change-Id: Iabd7f03c26c9b8167fb9088cbaa83c2cae1d75f8
Updates our job data handling so that it works with the latest
SmokeStack job format updates. This includes:
-new build status's for Failed, TestFail, and BuildFail
-new job types (cloudcue)
-display multiple jobs for each type (where appropriate)
Change-Id: I79fa21cfd489a2f524d4c98bc94b7a14b81ee831
Based on the number of days since the last patch was posted
Also added a tooltip to explain scores
Change-Id: I1f13a97ddd8714297a331be91146f9742bbb9766
Updates SmokeStack URL to be the SSL version.
Also, changes httplib2 so that we allow insecure SSL. This
is because we aren't using signed certificates with SmokeStack at the
moment.
Change-Id: I798c923efc07849be19521d7fc1730ca1e771838
This updates our LP calls to obtain blueprint information so that
they run faster. Previously calling spec.name would cause an LP hit
for each spec. Given that we call lp.specification many times during
a reviewday run this was causing slow runtimes.
What we do now is just scrap the blueprint name off of the Spec URL
(thus avoiding many of the extra LP hits).
Change-Id: I4f2cbcc262973b02e3a13b9bba1edde53d5e6d9e
Will print a warning message - so cron should email the owner of the job
when an unknown score key is encountered.
The example is blueprint db2-database which has "Not" as the priority
Change-Id: I4d43017be6a54bc3322d5aa754034a3d973d484d
Reviewed-on: https://review.openstack.org/36846
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Approved: Dan Prince <dprince@redhat.com>
Reviewed-by: Dan Prince <dprince@redhat.com>
Tested-by: Jenkins
The combo.css file was very complicated and difficult to
edit, pulled out unused sections so it's easier to
maintain and compatible with our s header.
Change-Id: I23b223d3c5af7db49da9ede47f69ef9792106888
Reviewed-on: https://review.openstack.org/31141
Approved: Dan Prince <dprince@redhat.com>
Reviewed-by: Dan Prince <dprince@redhat.com>
Tested-by: Jenkins
Switch http://status.openstack.org/reviews/ to use the common
status.o.o header and footer powered by common.js.
Change-Id: I6e742771bf9fe5b4732e0fc7abb82b8d2c623917
Reviewed-on: https://review.openstack.org/30168
Approved: Dan Prince <dprince@redhat.com>
Reviewed-by: Dan Prince <dprince@redhat.com>
Tested-by: Jenkins
The default timeout in LPlib is None which means it will just
hang. Lets set our default to 5 seconds to fix any queries
that hang.
This fixes an issue which started a week ago (when a new
version of LP was released) where reviewday would just hang
when trying to query LP for some of the bug/BP information.
Change-Id: Ia7af4efba543b25d220ccb193abbabc20b9f489f
Reviewed-on: https://review.openstack.org/28315
Approved: Dan Prince <dprince@redhat.com>
Reviewed-by: Dan Prince <dprince@redhat.com>
Tested-by: Jenkins
Add back a missing space which we accidentally removed in 932bfca.
This should fix issues causing zero results to display.
Change-Id: I86ae8a7217862811438dc15282415f47bf09bfb0
Reviewed-on: https://review.openstack.org/25545
Approved: Dan Prince <dprince@redhat.com>
Reviewed-by: Dan Prince <dprince@redhat.com>
Tested-by: Jenkins
Change-Id: I43efa5241ba0a93cf6272c849c6c3cc77c0b18a1
Reviewed-on: https://review.openstack.org/22995
Approved: Dan Prince <dprince@redhat.com>
Reviewed-by: Dan Prince <dprince@redhat.com>
Tested-by: Jenkins
Add a -o, --out-dir option to specify an output directory.
Change-Id: I0c8ac1aba309f9a34b2562b18da644efd9f37f4e
Reviewed-on: https://review.openstack.org/20839
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Approved: Dan Prince <dprince@redhat.com>
Reviewed-by: Dan Prince <dprince@redhat.com>
Tested-by: Jenkins
In our configuration puppet will manage the output directory, so it
is expected behavior for it to exist, removing warning. Also
switching to distutils.dir_util copy_tree since that allows for
copying of required supporting files into an existing output
directory.
Change-Id: I38b2c6ec47fd61814554a4b5007a83553b05aeb2
Reviewed-on: https://review.openstack.org/20647
Approved: Dan Prince <dprince@redhat.com>
Reviewed-by: Dan Prince <dprince@redhat.com>
Tested-by: Jenkins
Add tox.ini file and fix up several pep8 failures.
Change-Id: I3c64d2e790a8dfe37aa9091b001c3a5b85bc8586
Reviewed-on: https://review.openstack.org/20661
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Dan Prince <dprince@redhat.com>
Approved: Dan Prince <dprince@redhat.com>
Tested-by: Jenkins
Zuul