Release v0.91.0 Release PR by sgoggins · Pull Request #3349 · chaoss/augur

sgoggins · 2025-10-28T17:20:59Z

Description

Please include a summary of the change.

This PR fixes everything noted in the related PRS.

Notes for Reviewers

This version has been tested for 3 weeks at scale, and aside from a now known issue related to the GitHub events API, is stable.

Signed commits

Yes, I signed my commits.

Fix for GitHub Issue #3194 where releases collection was failing with psycopg2.errors.UniqueViolation on releases_pkey constraint. Root cause: GitHub API returns release IDs with trailing spaces that don't match existing trimmed database records. Changes: - Add str().strip() to release_id processing in get_release_inf() - Enhance duplicate detection in insert_release() with proper trimming - Add early duplicate detection to prevent unnecessary database operations Signed-off-by: mohsinm-dev <mohsin.mdev@gmail.com>

Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>

The code which parses the repos files interprets the first column as URL and the second one as repo group ID, but all the examples were doing the opposite. Signed-off-by: Mosè Giordano <m.giordano@ucl.ac.uk>

…/keys must be set The non-interactive Docker compose workflow requires all the variables `AUGUR_GIT{LA,HU}B_{USERNAME,API_KEY}` to be set even if they are going to be unused, otherwise you get stuck waiting for [the prompt](https://github.com/chaoss/augur/blob/b0bb3b80402ee5fcd84bec7334e58a41f9f5ec8a/scripts/install/config.sh#L18-L29) ``` You entered a blank line, are you sure? ``` Signed-off-by: Mosè Giordano <m.giordano@ucl.ac.uk>

docker compose: Make it crystal clear that all the GitHub/GitLab user/keys must be set

Set default to 0. Signed-off-by: Sean P. Goggins <outdoors@acm.org>

add date filter to contributer resolution logic queries

Signed-off-by: Sean P. Goggins <s@goggins.com>

Year not recognized without month and date. Signed-off-by: Sean P. Goggins <outdoors@acm.org>

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Fix examples of repos CSV files

`git reset --hard <remote>` isn't a correct command, the argument of `git reset` must be a tree-ish, which a remote name alone isn't. In some cases this command fails with ``` fatal: ambiguous argument 'origin': unknown revision or path not in the working tree. Use '--' to separate paths from revisions, like this: 'git <command> [<revision>...] -- [<file>...]' ``` Signed-off-by: Mosè Giordano <765740+giordano@users.noreply.github.com>

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Fix `git reset` command to include remote default branch

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

…ead-only components of the config Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Generated-by: Gpt-5 via cursor Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

…r each thing This is a tuple so it can potentially scale up without impacting the method signature Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

…add_value Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

replace worker vmem config item with individual process counts

github-actions · 2025-11-10T23:52:01Z

augur/application/cli/backend.py

    logger.info(f"The API is available at '{api_response.json()['route']}'")

-    processes = start_celery_worker_processes(float(worker_vmem_cap), disable_collection)
+    processes = start_celery_worker_processes((core_worker_count, secondary_worker_count, facade_worker_count), disable_collection)


[pylint] _{reported by reviewdog 🐶}
W0621: Redefining name 'processes' from outer scope (line 479) (redefined-outer-name)

Refactor config manager to use a hierarchical pattern. Discussed extensively in maintainer meetings the past two weeks, and docker build issues are now resolved.

github-actions · 2025-11-11T15:33:20Z

tests/test_classes/test_config_stores.py

+    return Mock()
+
+
+def test_jsonconfig_readonly_flags(mock_logger):


[pylint] _{reported by reviewdog 🐶}
W0621: Redefining name 'mock_logger' from outer scope (line 9) (redefined-outer-name)

github-actions · 2025-11-11T15:33:20Z

tests/test_classes/test_config_stores.py

+    assert cfg.empty is False
+
+
+def test_jsonconfig_empty_true_false(mock_logger):


[pylint] _{reported by reviewdog 🐶}
W0621: Redefining name 'mock_logger' from outer scope (line 9) (redefined-outer-name)

github-actions · 2025-11-11T15:33:20Z

tests/test_classes/test_config_stores.py

+    assert JsonConfig({"A": {}}, mock_logger).empty is False
+
+
+def test_jsonconfig_retrieve_has_get(mock_logger):


[pylint] _{reported by reviewdog 🐶}
W0621: Redefining name 'mock_logger' from outer scope (line 9) (redefined-outer-name)

github-actions · 2025-11-11T15:33:20Z

tests/test_classes/test_config_stores.py

+        ("add_value", ("X", "y", 2), {"ignore_existing": False}),
+    ],
+)
+def test_jsonconfig_mutations_raise_not_writable(mock_logger, callable_name, args, kwargs):


[pylint] _{reported by reviewdog 🐶}
W0621: Redefining name 'mock_logger' from outer scope (line 9) (redefined-outer-name)

github-actions · 2025-11-11T15:33:20Z

tests/test_classes/test_config_stores.py

+
+
+
+def test_fetching_real_defaults(mock_logger, mock_session):


[pylint] _{reported by reviewdog 🐶}
W0621: Redefining name 'mock_logger' from outer scope (line 9) (redefined-outer-name)

github-actions · 2025-11-11T15:33:20Z

tests/test_classes/test_config_stores.py

+
+
+
+def test_fetching_real_defaults(mock_logger, mock_session):


[pylint] _{reported by reviewdog 🐶}
W0621: Redefining name 'mock_session' from outer scope (line 13) (redefined-outer-name)

MoralCode

seems to work on my machine

mohsinm-dev and others added 30 commits July 27, 2025 19:38

add date filter to contributer resolution logic queries

8539825

Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>

dont use start date

fe85b3f

Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>

Merge branch 'main' into partial-recollection-facade

0df6bdd

remove unused method

6a6c760

Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>

bump rabbit image to 4.1

b8e1c79

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Merge branch 'main' into partial-recollection-facade

267f95e

revert

9ede8af

Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>

add toggle to toggle full collection of contributor resolution

f41e8f8

Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>

amend method of getting collection status record

c2e87ff

Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>

add config option in proper place

8ae5002

Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>

Fix examples of repos CSV files

4bb6554

The code which parses the repos files interprets the first column as URL and the second one as repo group ID, but all the examples were doing the opposite. Signed-off-by: Mosè Giordano <m.giordano@ucl.ac.uk>

Merge pull request #3300 from giordano/mg/do-not-remove-env-variables

e780526

docker compose: Make it crystal clear that all the GitHub/GitLab user/keys must be set

Change facade_contributor_full_recollect to 0

a87c6a0

Set default to 0. Signed-off-by: Sean P. Goggins <outdoors@acm.org>

Merge pull request #3253 from chaoss/partial-recollection-facade

473673c

add date filter to contributer resolution logic queries

update schema

fdc0005

Signed-off-by: Sean P. Goggins <s@goggins.com>

fixing description

512ff81

Signed-off-by: Sean P. Goggins <s@goggins.com>

Update date-released format in CITATION.cff

7f58fbc

Year not recognized without month and date. Signed-off-by: Sean P. Goggins <outdoors@acm.org>

Log facade messages based on the type requested

e4a618f

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Merge pull request #3299 from giordano/mg/repos-schema

3fb2bae

Fix examples of repos CSV files

refactor send_messages to remove a almost entirely duplicate code path

9ebc4eb

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Merge pull request #3312 from giordano/patch-1

60138d0

Fix `git reset` command to include remote default branch

remove from template

b1c04dc

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

remove from main route

9a92e17

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

remove the report routes

f07aa92

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

remove reports definition

2e60ffb

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

additional reports related things

a28c7b4

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

basic implementation

148680b

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

MoralCode and others added 18 commits November 10, 2025 15:30

add a base-config property to return the config as assembled by all r…

23a25df

…ead-only components of the config Signed-off-by: Adrian Edwards <adredwar@redhat.com>

adjust config init process to use the new base_config

16ea9fa

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

unused function

3960551

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

pass in a mock logger as part of the JsonConfig tests

bb7ae02

Generated-by: Gpt-5 via cursor Signed-off-by: Adrian Edwards <adredwar@redhat.com>

don't use an in-place reverse - its going to cause all sorts of issues

220aa0f

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

use a function that exists

3884fa8

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

add a test case for fetching real values

30e8e82

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

refactor start_celery_worker_processes to accept raw worker counts fo…

389788b

…r each thing This is a tuple so it can potentially scale up without impacting the method signature Signed-off-by: Adrian Edwards <adredwar@redhat.com>

apply same to start_celery_collection_processes

1bd2cba

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

fetch new values from config

267b903

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

fix unhashable type

05812c2

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

refactor redaction into a helper

53a2ae3

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

add_or_update_settings is used externally, replace it with a call to …

2d4fa30

…add_value Signed-off-by: Adrian Edwards <adredwar@redhat.com>

replace external calls to add_or_update_settings

9c1f422

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

make use of redaction function

aa6ce9d

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

fix: extract intended dict value from the returned list

de98f25

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

more fixes

9d19d48

Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Merge pull request #3337 from chaoss/remove-worker-vmem

934560d

replace worker vmem config item with individual process counts

github-actions bot reviewed Nov 10, 2025

View reviewed changes

Merge pull request #3378 from chaoss/config/hierarchy

a9f087e

Refactor config manager to use a hierarchical pattern. Discussed extensively in maintainer meetings the past two weeks, and docker build issues are now resolved.

github-actions bot reviewed Nov 11, 2025

View reviewed changes

sgoggins requested a review from MoralCode November 11, 2025 15:33

sgoggins assigned IsaacMilarky Nov 11, 2025

sgoggins marked this pull request as ready for review November 11, 2025 15:34

sgoggins added this to the v0.91.0 Release milestone Nov 11, 2025

sgoggins added this to Augur TSC Nov 11, 2025

sgoggins requested a review from Ulincsys November 11, 2025 23:34

MoralCode approved these changes Nov 11, 2025

View reviewed changes

sgoggins merged commit a886e28 into release Nov 11, 2025
22 of 23 checks passed

github-project-automation bot moved this to Main in Augur TSC Nov 11, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Release v0.91.0 Release PR#3349

Release v0.91.0 Release PR#3349
sgoggins merged 141 commits intoreleasefrom
main

sgoggins commented Oct 28, 2025

Uh oh!

github-actions bot Nov 10, 2025

Uh oh!

github-actions bot Nov 11, 2025

Uh oh!

github-actions bot Nov 11, 2025

Uh oh!

github-actions bot Nov 11, 2025

Uh oh!

github-actions bot Nov 11, 2025

Uh oh!

github-actions bot Nov 11, 2025

Uh oh!

github-actions bot Nov 11, 2025

Uh oh!

MoralCode left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

11 participants

		return Mock()


		def test_jsonconfig_readonly_flags(mock_logger):

		assert cfg.empty is False


		def test_jsonconfig_empty_true_false(mock_logger):

		assert JsonConfig({"A": {}}, mock_logger).empty is False


		def test_jsonconfig_retrieve_has_get(mock_logger):

Conversation

sgoggins commented Oct 28, 2025

Uh oh!

github-actions bot Nov 10, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

MoralCode left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

11 participants