Skip to content

PG-2839 percona-obs project versions --online prints version: null - #138

Merged
mohitj1988 merged 1 commit into
mainfrom
PG-2839
Oct 13, 2026
Merged

mohitj1988 merged 1 commit into
mainfrom
PG-2839

Conversation

@mohitj1988

Copy link
Copy Markdown
Collaborator

project versions --online printed version: null with no reason for packages OBS does build (e.g. etcd, geos, llvm-21 in ppg:staging:18):

  • Only the first succeeded repository (in name order) was asked for its binary list; when that one listed no readable package file, the others were never tried. Now every succeeded repository is tried in turn.
  • An aggregate was looked up only in its source project (e.g. ppg:common:deps). When that yields nothing (no build there, not visible, or an API error), the aggregating project, which holds the copied binaries, is now tried too.
  • Every OBS error was swallowed. _fetch_build_results and _fetch_pkg_versrel now log a warning with the error (and the URL at debug level, shown by --verbose), and a package still without a version gets a warning naming the projects searched.

Output format is unchanged (version: null when nothing is found).

`project versions --online` printed `version: null` with no reason for
packages OBS does build (e.g. etcd, geos, llvm-21 in ppg:staging:18):

- Only the first succeeded repository (in name order) was asked for its
  binary list; when that one listed no readable package file, the others
  were never tried. Now every succeeded repository is tried in turn.
- An aggregate was looked up only in its source project (e.g.
  ppg:common:deps). When that yields nothing (no build there, not visible,
  or an API error), the aggregating project, which holds the copied
  binaries, is now tried too.
- Every OBS error was swallowed. _fetch_build_results and
  _fetch_pkg_versrel now log a warning with the error (and the URL at
  debug level, shown by --verbose), and a package still without a version
  gets a warning naming the projects searched.

Output format is unchanged (`version: null` when nothing is found).
@mohitj1988
mohitj1988 requested a review from rjd15372 October 13, 2026 07:31
@mohitj1988
mohitj1988 merged commit 6168acc into main Oct 13, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants