buildbot/phase1, branch v21

phase1,phase2: GitPoller: fix deprecated API warning for pollinterval

2024-10-12T08:03:30Z

Fixes following warning: gitpoller.py:103: buildbot.warnings.DeprecatedApiWarning: [3.11.3 and later] pollinterval has been deprecated: please use pollInterval Signed-off-by: Petr Štetiar

add APK signing logic

2024-08-10T11:19:10Z

With this commit it's possible to sign APK package indexes (packages.adb) via the `signall.sh` script, which is run on the buildmaster. As a consequence `apk` must be available on the buildmaster. This is the final step to replace OPKG with APK. Signed-off-by: Paul Spooren

phase1: make IsNoMasterBuild function more flexible

2024-08-07T08:04:04Z

Work with both master/main Signed-off-by: Paul Spooren Signed-off-by: Christian Marangi

phase1: kmodprepare: add apk compat

2024-08-07T08:03:22Z

Ansuel reported, that with `USE_APK=y` there are no kmods available in the build artifacts, so lets fix it by extending the rsync's include pattern with the `apk` suffix to make it future proof. Reported-by: Christian Marangi Signed-off-by: Petr Štetiar

phase1: AnyBranchScheduler: make treeStableTimer configurable

2024-08-07T08:03:06Z

The AnyBranchScheduler will currently wait for 15 minutes before starting a build. If new changes are made on the same branch during this interval, the timer will be restarted. For staging repository we want faster feedback loop, so we're going to use much lower value, thus lets make this configurable and handled via Ansible. Signed-off-by: Petr Štetiar

phase1: make list of builders configurable

2024-06-01T12:16:36Z

Currently we always populate builders (build targets) from the Git tree, but for staging environment this is usually overkill as we might need just a bunch of targets, so lets make this configurable. Signed-off-by: Petr Štetiar

phase1: factor out populateTargetsForBranch

2024-06-01T11:21:23Z

Going to make builders (build targets) configurable, so lets factor current populateTargets into separate function populateTargetsForBranch which takes a branch as argument. No functional changes. Signed-off-by: Petr Štetiar

phase1,phase2: s/master/main for phase{1,2}

2024-04-13T04:52:44Z

More and more projects are switching their repositories to use the 'main' branch instead of the 'master' branch. This also includes many Linux upstream trees as well. Some trees are even removing their 'master' branches already. I think this is becoming more and more mainstream and expected of projects, so we should do the same. References: https://openwrt.org/voting/2023-02-27 Signed-off-by: Paul Spooren Signed-off-by: Petr Štetiar [commit facelift]

requirements-dev: use black 23.12.1 and ruff 0.1.9

2024-04-13T04:17:49Z

Fix ruff's recommendation: phase2/master.cfg:328:16: E721 Do not compare types, use `isinstance()` "Unlike a direct type comparison, isinstance will also check if an object is an instance of a class or a subclass thereof." Signed-off-by: Petr Štetiar

Add linting with flake8 tool

2024-04-13T04:17:49Z

flake8 is a Python tool that glues together pycodestyle, pyflakes, mccabe, and third-party plugins to check the style and quality of some Python code. Currently we've issue in phase2 in checksum step: /bin/sh: 2: Syntax error: ")" unexpected And it seems, that flake8 is able to spot places which might lead to such issues during runtime: phase2/master.cfg:733:151: W605 invalid escape sequence '\(' phase2/master.cfg:733:155: W605 invalid escape sequence '\)' phase2/master.cfg:733:158: W605 invalid escape sequence '\(' phase2/master.cfg:733:162: W605 invalid escape sequence '\)' So lets enable flake8 checking on the CI so we can spot similar places in the future and address them before deployment. We dont want to make current ongoing work on phase2 code harder, thus we don't touch that part yet, so we whitelist most of the checks for now. References: f0faed2970dd ("phase2: compute checksums") Signed-off-by: Petr Štetiar