From 046d6a80eac3a610463babce1f6ecf7c3c30076a Mon Sep 17 00:00:00 2001 From: Mike McLean Date: Jan 20 2020 08:38:10 +0000 Subject: partial edits --- diff --git a/docs/source/release_notes/release_notes_1.20.rst b/docs/source/release_notes/release_notes_1.20.rst index 4c22b4a..383a5e1 100644 --- a/docs/source/release_notes/release_notes_1.20.rst +++ b/docs/source/release_notes/release_notes_1.20.rst @@ -124,22 +124,22 @@ less painful. API Changes ----------- -**getUser: default krb_princs value is changed to True** + +**getUser: Default krb_princs value changed to True** | PR: https://pagure.io/koji/pull-request/1872 -This argument was added in PR #1648, whose default value is ``False``. It is -used to control if to show the ``krb_principals`` list in the result of -``getUser``. It is better to be shown by default, as it may confuse people that -Kerberos principal was deleted. +This argument was added in PR #1648, with a default value of ``False``. It is +used to control whether the ``krb_principals`` field is included in the result of +``getUser``. -**drop buildMap API call** +**Drop buildMap API call** | PR: https://pagure.io/koji/pull-request/1755 It was designed for GC, but it is not used anymore. -**hub: new addArchiveType RPC** +**hub: New addArchiveType RPC** | PR: https://pagure.io/koji/pull-request/1149 @@ -149,24 +149,25 @@ Adds a new hub method for inserting new archivetype records. | PR: https://pagure.io/koji/pull-request/1893 -Previously database exception was propagated. Now, it is raising proper koji +Previously a database exception was propagated. Now, it is raising a more informative exception. + Web UI Changes -------------- -**browsable api** +**Browsable api** | PR: https://pagure.io/koji/pull-request/1821 -``koji list-api`` output browsable via web. +The web ui now offers API introspection similar to the ``koji list-api`` command. -**cluster health info page** +**Cluster health info page** | PR: https://pagure.io/koji/pull-request/1551 New web page showing current usage of build cluster. -**fix closing table tag** +**Fix closing table tag** | PR: https://pagure.io/koji/pull-request/1839 @@ -176,21 +177,25 @@ Fixed corrupted table. | PR: https://pagure.io/koji/pull-request/284 -Workaround before we have a proper web plugin API +This is a temporary solution to the problem of connecting the builds and tasks +generated by the +`containerbuild plugin `. -**human-friendly file sizes in taskinfo page** +**Human-friendly file sizes in taskinfo page** | PR: https://pagure.io/koji/pull-request/1820 + Builder Changes --------------- -**kojid: use binary msg for python3 in notification tasks** + +**kojid: Use binary msg for python3 in notification tasks** | PR: https://pagure.io/koji/pull-request/1892 -Fix encoding problems in notification +Fix encoding problems in notification tasks -**split admin_emails option for kojid** +**Split admin_emails option for kojid** | PR: https://pagure.io/koji/pull-request/1246 @@ -209,13 +214,13 @@ building SRPMs for fetching data from authenticated SCMs. | PR: https://pagure.io/koji/pull-request/821 | PR: https://pagure.io/koji/pull-request/1850 -**use --update for dist-repos if possible** +**Use --update for dist-repos if possible** | PR: https://pagure.io/koji/pull-request/1037 Improves speed of new distrepos. -**fix time type for restartHosts** +**Fix time type for restartHosts** | PR: https://pagure.io/koji/pull-request/1826 @@ -239,8 +244,10 @@ Maven3 file type added to ignored. In recent Fedora's maven is alternatives symlink. Original check now failed even if maven was installed. + System Changes -------------- + **QueryProcessor: fix countOnly for group sql** | PR: https://pagure.io/koji/pull-request/1845 @@ -248,7 +255,7 @@ System Changes WebUI returned an error on Users tab after multiple kerberos realms per user were introduced. -**limit distRepo tasks per tag** +**Limit distRepo tasks per tag** | PR: https://pagure.io/koji/pull-request/1869 | PR: https://pagure.io/koji/pull-request/1912 @@ -256,38 +263,42 @@ were introduced. Introduces ``distrepo.cancel_others`` extra flag for tags. If enabled, new distRepo task will cancel previous non-finished ones leaving only new one. -**do not use with statement with requests.get** +**Do not use with statement with requests.get** | PR: https://pagure.io/koji/pull-request/1854 Older python-requests doesn't handle correctly ``with`` statement, so we've avoided it for now. -**clean all unused `import` and reorder imports** +**Clean all unused `import` and reorder imports** | PR: https://pagure.io/koji/pull-request/763 Making our code PEP-8 compliant. -**fix CGRefundBuild to release build properly** +**Fix CGRefundBuild to release build properly** | PR: https://pagure.io/koji/pull-request/1853 Fixes for refunding failed/cancelled build. -**gitignore: exclude .vscode folder** +**gitignore: Exclude .vscode folder** | PR: https://pagure.io/koji/pull-request/1862 trivial change in `.gitignore` -**improve test and clean targets in Makefiles** +**Improve test and clean targets in Makefiles** | PR: https://pagure.io/koji/pull-request/723 -**remove old db constraint** +**Remove old db constraint** | PR: https://pagure.io/koji/pull-request/1790 +In some cases, an old and now invalid db constraint may have persisted after +older schema updates, due to naming variation. +This change ensures the constraint is removed. + **use BulkInsertProcessor for hub mass inserts** | PR: https://pagure.io/koji/pull-request/1714 @@ -295,7 +306,7 @@ trivial change in `.gitignore` Speed up mass inserts. -**rm old test code** +**Remove old test code** | PR: https://pagure.io/koji/pull-request/1798 @@ -303,21 +314,23 @@ Some files in the tree had bits of code that you could run if you executed the files directly as scripts. Now that we have unit tests and the "fakehub" tool, we do not need this code. -**hub: build for policy check should be build_id in host.tagBuild** +**hub: Build for policy check should be build_id in host.tagBuild** | PR: https://pagure.io/koji/pull-request/1797 -**rpm: remove %defattr** +**rpm: Remove %defattr** | PR: https://pagure.io/koji/pull-request/1800 -RHEL 5 and later do not require %defattr. +Remove the %defattr line from the rpm spec file, as this has not been needed +since RHEL 5. -**allow comma delimiter for allowed_methods** +**Use comma delimiter for allowed_methods** | PR: https://pagure.io/koji/pull-request/1745 -Example config says, that comma is allowed, but it was not true. +The ``allowed_methods`` configuration option for the ``save_failed_tree`` plugin +now expects comma-separated values, as the example config indicates. **hub: Fix issue with listing users and old versions of Postgres** @@ -327,7 +340,7 @@ Example config says, that comma is allowed, but it was not true. | PR: https://pagure.io/koji/pull-request/1753 -**clean python compiled binaries for non *.py code** +**Clean python compiled binaries for non *.py code** | PR: https://pagure.io/koji/pull-request/1695