<feed xmlns='http://www.w3.org/2005/Atom'>
<title>buildbot/phase1, branch v11</title>
<subtitle>LEDE buildbot configuration</subtitle>
<id>https://git.openwrt.org/buildbot/atom?h=v11</id>
<link rel='self' href='https://git.openwrt.org/buildbot/atom?h=v11'/>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/buildbot/'/>
<updated>2023-11-16T08:57:01Z</updated>
<entry>
<title>phase1: treat all branches equally for building</title>
<updated>2023-11-16T08:57:01Z</updated>
<author>
<name>Thibaut VARÈNE</name>
</author>
<published>2023-11-14T13:23:50Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/buildbot/commit/?id=b372bb5c19282e0eaa0b25f6717ed70afc2e74c5'/>
<id>urn:sha1:b372bb5c19282e0eaa0b25f6717ed70afc2e74c5</id>
<content type='text'>
Following discussion here:
https://lists.openwrt.org/pipermail/openwrt-devel/2023-November/041769.html

Signed-off-by: Thibaut VARÈNE &lt;hacks@slashdirt.org&gt;
</content>
</entry>
<entry>
<title>phase1: raise priority of tag builds</title>
<updated>2023-11-16T08:57:01Z</updated>
<author>
<name>Thibaut VARÈNE</name>
</author>
<published>2023-11-13T14:23:49Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/buildbot/commit/?id=34b4378d9a6f3cad8e728e9e91ba163a716c451e'/>
<id>urn:sha1:34b4378d9a6f3cad8e728e9e91ba163a716c451e</id>
<content type='text'>
Currently the buildmaster would only order tag builds within their own
branch, meaning that if e.g. a higher priority branch has normal
buildrequests (i.e. buildrequests comming from the AnyBranchScheduler),
and a lower priority branch has "tag" buildrequests (i.e. from the
Triggerable scheduler), the former builderequests would be served first,
deferring the build of e.g. release tags.

We want forced builds (release tag) to have maximum priority, regardless
of branch priority. This commit attempts to solve this problem by
leveraging the newly (as of buildbot 3.9.0) introduced "priority"
scheduler parameter, by raising the Triggerable scheduler buildrequests
priority, and then considering this higher priority in Builders' order.

The net result is that Builders are now prioritized if they have
pending higher priority buildrequest, still preserving the branch order.
In other words, tag requests are front run while preserving branch order,
meaning that if two branches have tag buildrequests, the higher priority
branch is still served first.

This requires buildbot 3.9.0

Signed-off-by: Thibaut VARÈNE &lt;hacks@slashdirt.org&gt;
</content>
</entry>
<entry>
<title>phase1: workaround dlprune recursive directory removal</title>
<updated>2023-07-24T09:06:09Z</updated>
<author>
<name>Petr Štetiar</name>
</author>
<published>2023-07-22T08:53:45Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/buildbot/commit/?id=30e2e5d800a87aebd8a87a78812a4b527c56bf2f'/>
<id>urn:sha1:30e2e5d800a87aebd8a87a78812a4b527c56bf2f</id>
<content type='text'>
Workarounds following issue:

 find: cannot delete ‘dl/ath10k-ct-firmware-2020-11-08’: Directory not empty

References: #13
Signed-off-by: Petr Štetiar &lt;ynezz@true.cz&gt;
</content>
</entry>
<entry>
<title>phase1: prepareFactory: use variables instead of list index</title>
<updated>2023-07-22T09:05:33Z</updated>
<author>
<name>Petr Štetiar</name>
</author>
<published>2023-06-18T05:32:22Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/buildbot/commit/?id=651a5c2aa158ddb7ad2920851696a75df1f2a86f'/>
<id>urn:sha1:651a5c2aa158ddb7ad2920851696a75df1f2a86f</id>
<content type='text'>
Thus make the code more readable.

Signed-off-by: Petr Štetiar &lt;ynezz@true.cz&gt;
</content>
</entry>
<entry>
<title>phase1: populateTargets: log branch name as well</title>
<updated>2023-07-22T09:05:32Z</updated>
<author>
<name>Petr Štetiar</name>
</author>
<published>2023-06-18T05:28:01Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/buildbot/commit/?id=e332f694130d40913083e52f454f7851b85000ac'/>
<id>urn:sha1:e332f694130d40913083e52f454f7851b85000ac</id>
<content type='text'>
So the progress is more verbose.

Signed-off-by: Petr Štetiar &lt;ynezz@true.cz&gt;
</content>
</entry>
<entry>
<title>phase1: do not leak targets between branches</title>
<updated>2023-07-22T09:05:31Z</updated>
<author>
<name>Petr Štetiar</name>
</author>
<published>2023-06-18T05:26:17Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/buildbot/commit/?id=3bbbc02a7f8b677026fbc856a336225e2630c050'/>
<id>urn:sha1:3bbbc02a7f8b677026fbc856a336225e2630c050</id>
<content type='text'>
Robert noticed, that after rename of `ipq807x` target in main branch to
`qualcommax/ipq807x` subtarget, that buildbot is trying to build this
new `qualcommax/ipq807x` subtarget on `openwrt-23.05` branch as well.

Thibaut later explained, that this is by design, his initial idea was to
find exhaustive list of all targets and let the `checkarch` step do the
final triaging.

I find this approach confusing, because if the subtarget is not present
in that branch, we shouldn't have builder for it configured as well.
Furthermore wasting roughly 5 minutes of precious buildworker time to
checkout all feeds and then just find out, that we're not going to use
those seems suboptimal as well.

So lets fix it by using builders for targets as present in respective
branches.

Fixes: #14
Reported-by: Robert Marko &lt;robimarko@gmail.com&gt;
Signed-off-by: Petr Štetiar &lt;ynezz@true.cz&gt;
</content>
</entry>
<entry>
<title>phase1: reformat with black</title>
<updated>2023-06-18T05:52:37Z</updated>
<author>
<name>Petr Štetiar</name>
</author>
<published>2023-06-18T05:19:32Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/buildbot/commit/?id=e56f359a2c2be5d1b6cd08e390087f242768a1b6'/>
<id>urn:sha1:e56f359a2c2be5d1b6cd08e390087f242768a1b6</id>
<content type='text'>
Making everything consistent, more readable, CI guarded.

Signed-off-by: Petr Štetiar &lt;ynezz@true.cz&gt;
</content>
</entry>
<entry>
<title>phase1: dlprune: fix cannot delete ‘dl/’: Not a directory</title>
<updated>2023-06-03T07:45:45Z</updated>
<author>
<name>Petr Štetiar</name>
</author>
<published>2023-06-01T05:56:49Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/buildbot/commit/?id=280a07b9e6ba928ce5495dc19ed68d17e990396c'/>
<id>urn:sha1:280a07b9e6ba928ce5495dc19ed68d17e990396c</id>
<content type='text'>
dlprune currently fails with following error:

 find: cannot delete ‘dl/’: Not a directory

Initial idea was to delete only stuff under download directory and not
the download directory itself, so lets adjust the find command
accordingly.

Fixes: 68b20ed67b5e ("phase1: prune unused files from dl/")
Signed-off-by: Petr Štetiar &lt;ynezz@true.cz&gt;
</content>
</entry>
<entry>
<title>phase1: satisfy getver.sh requirements</title>
<updated>2023-05-23T19:34:21Z</updated>
<author>
<name>Thibaut VARÈNE</name>
</author>
<published>2023-05-23T18:12:09Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/buildbot/commit/?id=2b415c0b17e9b96ea2933206a3466b8a52e32b35'/>
<id>urn:sha1:2b415c0b17e9b96ea2933206a3466b8a52e32b35</id>
<content type='text'>
scripts/getver.sh requires upstream branch tracking to correctly compute
revision number. Buildbot Git() step does not set branch tracking as it
checks out individual commits and then reset the target branch to it, so
in order for getver.sh to work, this workaround forcefully sets the
upstream branch.

This fixes malformed version code in filenames as observed on
openwrt-23.05:

r23001+3-38c150612c
instead of:
r23004-7f0db09513

Signed-off-by: Thibaut VARÈNE &lt;hacks@slashdirt.org&gt;
</content>
</entry>
<entry>
<title>phase1: cleanup redundant steps</title>
<updated>2023-05-23T02:29:44Z</updated>
<author>
<name>Thibaut VARÈNE</name>
</author>
<published>2023-05-22T18:20:12Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/buildbot/commit/?id=90af3d7ad4a72aab90e2b3bca853a91cf199a753'/>
<id>urn:sha1:90af3d7ad4a72aab90e2b3bca853a91cf199a753</id>
<content type='text'>
These 3 rm steps are unnecessary since these folders are already cleaned
by the Git() step. Thus reduce clutter in already pretty busy Factory.

Signed-off-by: Thibaut VARÈNE &lt;hacks@slashdirt.org&gt;
</content>
</entry>
</feed>
