From bd1e51f76c55f95f8852d63235fe28eaa034d9b1 Mon Sep 17 00:00:00 2001 From: Mark O Brien Date: Fri, 2 Sep 2022 11:49:26 +0100 Subject: [PATCH] kerneltest: review of current live version Signed-off-by: Mark O Brien --- docs/kerneltest/current_v1.2.1.rst | 59 ++++++++++++++++++++++++++++++ docs/kerneltest/index.rst | 2 + 2 files changed, 61 insertions(+) create mode 100644 docs/kerneltest/current_v1.2.1.rst diff --git a/docs/kerneltest/current_v1.2.1.rst b/docs/kerneltest/current_v1.2.1.rst new file mode 100644 index 0000000..aae59d0 --- /dev/null +++ b/docs/kerneltest/current_v1.2.1.rst @@ -0,0 +1,59 @@ +.. _current_v1.2.1: + +Current Live Version of Kernel Test App v1.2.1 +============================================== + +The current version of the Kernel Test App running is v1.2.1 +The code is no longer maintained and stale since 2017. There was an attempted +rewrite but this was never put live + +Resources +--------- + +Release 1.2.1: https://github.com/jmflinuxtx/kerneltest-harness/tree/c2d807506e2a9ac3ca6bc5b60405b5700a72719c + +What is in v1.21 +------------- + +* Tests are in the `tests` folder and appear to cover most functionality +* Uses flask `fas_openid` library for authentication +* Configuration file loading and defaults +* Forms for updating/creating release and submitting logs are in place +* Web routes implemented + + * `/` - index + * `/release/` - info about release version specified + * `/kernel/` - info about kernel version specified + * `/logs/` + * `/stats` + * `/upload` + * `/upload/autotest` + * `/upload/anonymous` + * `/login` + * `/logout` + * `/admin/new` - Add new release + * `/admin//edit` - edit existing release + + +Notes +----- + +* There is no documentation +* This is the live version and it works as is +* Web interface outdated +* Old unmaintained codebase(all functionality in single file) +* Local vagrant solution uses outdated version of Fedora and doesn't currently + work + +The Good Points +--------------- + +1. The application works as expected and provides necessary functionality + + +The Bad points +-------------- + +1. Code is unmaintaned and 5 years old +2. Web interface is outdated +3. No documentation diff --git a/docs/kerneltest/index.rst b/docs/kerneltest/index.rst index d7d1d78..663baa9 100644 --- a/docs/kerneltest/index.rst +++ b/docs/kerneltest/index.rst @@ -23,6 +23,7 @@ Requirements ------------ * Be able to upload logs anonymously, authenticated, super user +* Superuser should use api token or similar(no password login required) * Be able to view logs on front page showing super user ones * When clicking on the release it should show all logs for a release * Stats: https://apps.fedoraproject.org/kerneltest/stats (currently stored in postgres) @@ -53,6 +54,7 @@ Here is the list of ideas/things we discussed/looked at: :maxdepth: 1 rewrite_of_kerneltest_app + current_v1.2.1 continue_from_v2 enhancement_of_repository