Adrian Edwards

Results 148 issues of Adrian Edwards

task: augur.tasks.git.facade_tasks.analyze_commits_in_parallel exception: ValueError('task_id must not be empty. Got None instead.') repo: https://github.com/frrouting/frr Stack Trace ``` [...] [SQL: INSERT INTO augur_data.commits (repo_id, cmt_commit_hash, cmt_author_name, cmt_author_raw_email, cmt_author_email, cmt_author_date, cmt_committer_name, cmt_committer_raw_email, cmt_committer_email,...

bug
deployed version

Repo: https://github.com/confluentinc/azure-rest-api-specs Task: augur.tasks.github.releases.tasks.collect_releases Exception: `DataError('(psycopg2.errors.StringDataRightTruncation) value too long for type character(256)\n') ` Stack Trace ``` Traceback (most recent call last): File "/augur/.venv/lib/python3.11/site-packages/sqlalchemy/engine/base.py", line 1969, in _exec_single_context self.dialect.do_execute( File "/augur/.venv/lib/python3.11/site-packages/sqlalchemy/engine/default.py",...

bug
deployed version

I introduced a ResourceGoneException for when augur tries to query issues when a repo has them disabled It seems like this may not being handled.

bug
deployed version

**Description** per #3333, the toml package that we depend on is out of date. This PR proposes a way to replace it with the `tomllib` package, which is built in...

discussion
dependencies

We still have several periodic tasks configured in celery with fixed time intervals. These should be pulling their intervals from the config just like the materialized view task https://github.com/chaoss/augur/blob/6b48ab60443dddc6a03ce17a61b6b395ad54de35/augur/tasks/init/celery_app.py#L231-L260

good first issue
config

I saw this once fairly recently and I suspect it is a current issue preventing new instances/DBs being created on the main branch It seems to happen at the point...

bug
more info needed
tech debt

https://github.com/chaoss/augur/blob/34f5644e2738c91d2f6b2a4ce81035fa201d9094/augur/tasks/init/__init__.py#L7 every time this is called, a DB connection is opened that never gets closed. This should not be happening. This has been looked at by several maintainers several times...

database
testing

**Description** This PR resolves and reconciles the differences between the current, in-database data schema and the schema defined by the SQLAlchemy models. This is needed because, for the past few...

database
discussion
tech debt
high priority

**Description** This removes the `repo_id` from the list of natural keys passed to `insert_data` when processing pull requests. This prevents augur from trying to insert duplicate PRs from github on...

deployed version

**Description** @cdolfi has reported an issue (#3129) where repos that have moved and are redirecting when visited in a browser are not having their URLs updated to reflect the move....

bug
deployed version
discussion
disclosed-ai