agama-project / agama / 18222230338

66%

master: 77%

Build Type

push

github

Committed by web-flow

Commit Message

feat: add a new config-based API

This pull request introduces two big changes:

* The implementation of the new HTTP API for the `l10n` module. Bear in
mind that the specification is still a work in progress.
* The new architecture for Agama services. The `l10n` module has been
heavily reorganized to:
  * support the new API.
  * make it easier to maintain in the future.

## The new HTTP API

Here is a brief explanation of how the new API should look like. For
further details, check [the ongoing
description](https://gist.github.com/joseivanlopez/ed9f4f87b214ff60d52a5f6c3897dc9d).

In general, it is designed around 3 main concepts:

* system: represents the current status of the running system. It is not
only about the configuration, but it can offer additional information
(e.g., the list of supported languages, products, hardware information,
etc.).
* config: represents the user configuration for the target system.
* proposal: represents what is going to happen during installation.

Here is the API summary (check whether we have done some changes):

~~~
GET             /system
GET             /extended_config
GET             /extended_config/{scope}
GET PUT PATCH   /config
GET PUT PATCH   /config/{scope}
GET POST PATCH  /questions
GET             /proposal
GET             /state
GET             /issues
POST            /action
~~~

This new design allows us to drastically reduce the complexity and the
amount of code needed by the old API.

Of course, we still need to extend it to support more use cases
(software, DASD, progress, etc.) but this one is a step in the direction
we want to take.

## The new architecture

Coming from a set of separate single-thread D-Bus services, we had have
time to identify a few limitations that we would like to overcome in
this redesign. Given that we expect several process to collaborate
(holding configuration, listening for system changes, etc.) we want to
implement something similar to an *very simplified* [actor
mod... (continued)

Run Details

2680 of 4679 branches covered (57.28%)

Branch coverage included in aggregate %.

91 of 104 new or added lines in 24 files covered. (87.5%)

161 existing lines in 19 files now uncovered.

24985 of 38819 relevant lines covered (64.36%)

196.88 hits per line