mbox series

[0/2] oeqa/utils/gitarchive: put back local tags listing as third tag listing method

Message ID 20230823103520.72792-1-alexis.lothore@bootlin.com
Headers show
Series oeqa/utils/gitarchive: put back local tags listing as third tag listing method | expand

Message

Alexis Lothoré Aug. 23, 2023, 10:35 a.m. UTC 
Hello,
this is another attempt to properly fix tag computation in Autobuilder.

First attempt ([1]) completely replaced "git tag" with "git ls-remote" to
make sure to know about all tags when computing new tag name. It raised
some issue because oe-git-archive works with local repositories with no
remote configured (it uses explicit remote url in git calls), so a second
series introduced this provided url usage as a fallback ([2])

There are still scenarios which do not match the two above: [3] shows that
oe-build-perf-report works with directories with no remote configured (the
same directories as oe-git-archive, in fact), AND does not use urls (I
guess it uses accumulated result from previous run, but I got no access to
the AB machines to make sure about it). I feel that it may not be relevant
to try to force-propagate fallback urls to oe-build-perf-report, moreover
there may be other unidentified cases scattered in whole AB configuration
which would need the same fix. Hence this series proposes to reintroduce
the previous method to retrieve tags ("git tag") as last resort method,
with a warning when reached, to both fix current issues and allow to
diagnostic future issues if it makes the initial tagging issue appear
again.

[1] https://lore.kernel.org/openembedded-core/20230811125532.9427-1-alexis.lothore@bootlin.com/
[2] https://lore.kernel.org/openembedded-core/20230818141712.189294-1-alexis.lothore@bootlin.com/
[3] https://autobuilder.yoctoproject.org/typhoon/#/builders/133/builds/2125/steps/12/logs/stdio

Alexis Lothoré (2):
  oeqa/utils/gitarchive: allow to pass a logger to get_tags
  oeqa/utils/gitarchive: fall back to local tags when listing existing
    tags

 .../oeqa/selftest/cases/gitarchivetests.py    | 25 +++++++++-----
 meta/lib/oeqa/utils/gitarchive.py             | 34 ++++++++++++-------
 2 files changed, 38 insertions(+), 21 deletions(-)